Skip to content

Fix: respect max_completion_tokens in thinking feat - #18946

Merged
Sameerlite merged 1 commit into
mainfrom
litellm_add_max_completion_tokens_with_thinking_budget
Jan 12, 2026
Merged

Sameerlite merged 1 commit into
mainfrom
litellm_add_max_completion_tokens_with_thinking_budget

Conversation

@Sameerlite

Copy link
Copy Markdown
Contributor

Relevant issues

Fix #18939

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🐛 Bug Fix

Changes

image

@vercel

vercel Bot commented Jan 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
litellm Ready Ready Preview, Comment Jan 12, 2026 7:16am

@Sameerlite
Sameerlite marked this pull request as ready for review January 12, 2026 12:43
@Sameerlite
Sameerlite merged commit 98f1a0d into main Jan 12, 2026
60 of 63 checks passed
@ishaan-berri
ishaan-berri deleted the litellm_add_max_completion_tokens_with_thinking_budget branch March 26, 2026 22:29
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…ion_tokens_with_thinking_budget

Fix: respect max_completion_tokens in thinking feat

This branch was successfully deployed

1 active deployment
Preview — 508e4da4 Deployed Jan 12, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Anthropic Translation Layer Do Not Respect max_tokens or max_completion_tokens

1 participant