Repository navigation
fix(anthropic): remove deprecated temperature and top_p parameters (Anthropic API 2026-10-21) - #1488
Conversation
|
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes.
|
| Layer / File(s) | Summary |
|---|---|
Generation request parameter mapping archive/deprecated-providers/anthropic-ai/src/index.ts |
All four generation methods stop sending temperature and top_p. This removes the previous default temperature values and topP mapping. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix
Suggested reviewers: venomhare
Merge Risk: 
Minimal · previous review 3e061
-
The parameter cleanup has no identified merge-blocking issue. Complete the planned tests and staging verification before release.
Pre-merge checks: 
5
-
Full details: Description check
-
Explanation
The description gives a clear summary, explains the reason for the change, identifies affected call sites, documents validation results, and states that the test suite was not run. It does not reproduce every template heading or confirm related issues, docs, and changesets, but it contains the key information needed for review.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
3e0619b to
a4a07de
Compare
a4a07de to
c212759
Compare
Summary
This pull request updates anthropic call sites for recipe: Anthropic API 2026-10-21: temperature, top_p, top_k removal.
What changed
archive/deprecated-providers/anthropic-ai/src/index.tsWhy
Risk assessment
Risk level: LOW
Deterministic AST recipe applied cleanly with low blast radius.
Proof (How this was checked)
parse_okandresidual_zero).Not verified
This PR was not validated by running your test suite. Please run CI before merging.
What you must check
How to close or opt out
Summary by cubic
Removes the deprecated
temperatureandtop_pparameters from all four Anthropic API call sites so requests stay compatible with the Anthropic API versioning effective 2026-10-21.temperatureandtopPprovider options are now silently ignored instead of being sent to the API, so generation no longer uses the previous temperature defaults and may produce different results depending on the provider's defaults.Written for commit c212759. Summary will update on new commits.
Summary by CodeRabbit