mirror of
https://github.com/BerriAI/litellm.git
synced 2026-10-25 05:54:53 +00:00
2754da5391
* fix(friendliai): drop litellm-internal assistant fields before sending Claude Code replays thinking blocks on every turn after the first. The Anthropic adapter turns them into thinking_blocks plus reasoning_content on the chat assistant message, and FriendliAI rejects any message key it does not know, so multi-turn Claude Code sessions on friendliai/ models failed with a 500 after the first tool call FriendliAI accepts reasoning_content and GLM reads it back, so only thinking_blocks and provider_specific_fields are dropped. together_ai already had this exact helper as a private function, so it moves to common_utils next to strip_litellm_internal_message_fields and both providers share it Related to #32188 * test(friendliai): cover a replayed tool call turn The regression fixture is now the shape Claude Code replays after a tool call: an assistant turn with thinking_blocks, tool_calls and no content, followed by the matching tool result. The test asserts both survive the sync and async request paths unchanged * ci: retrigger checks * ci: retrigger benchmarks after runner timeout * chore(friendliai): mark inherited list signatures as mutable-ok The type-discipline gate on main now caps LIT001 at the merge-base count, so the _transform_messages overloads need the same inherited-contract marker together_ai uses for its identical override * ci(test-unit): add tests/unit/llms/friendliai to the llms-providers shard * test(friendliai): check the outbound FriendliAI body through the proxy for chat and messages replays Both replay shapes go through a real proxy to a scripted FriendliAI upstream, and the test reads the request body FriendliAI received. It fails at the merge base, where thinking_blocks and provider_specific_fields reach the upstream, and passes with the field stripping in place. --------- Co-authored-by: mateo-berri <277851410+mateo-berri@users.noreply.github.com> |
||
|---|---|---|
| .. | ||
| auto-close-duplicates.yml | ||
| auto_update_price_and_context_window.yml | ||
| check-schema-sync.yml | ||
| codeql.yml | ||
| codspeed.yml | ||
| compat-matrix-image.yml | ||
| conventional-commits.yml | ||
| cost-map-guard.yml | ||
| create-rc-branch.yml | ||
| create-release-branch.yml | ||
| deploy-dev-on-main.yml | ||
| duplicate_issue_check.yml | ||
| e2e_record_replay.yml | ||
| guard-fork-dependencies.yml | ||
| image-scan.yml | ||
| issue-keyword-labeler.yml | ||
| issue_classifier.yml | ||
| issue_fixed_comment.yml | ||
| issue_label_claude_code.yml | ||
| issue_label_sync.yml | ||
| lens-install-smoke.yml | ||
| linear-release.yml | ||
| mutation-test.yml | ||
| osv-scan.yml | ||
| publish-lint-base-counts.yml | ||
| scorecard.yml | ||
| stale.yml | ||
| sync-schema.yml | ||
| sync-together-ai-models.yml | ||
| test-e2e-redis-chaos.yml | ||
| test-linting.yml | ||
| test-mcp-dependency-resolution.yml | ||
| test-merge-smoke.yml | ||
| test-rust.yml | ||
| test-terraform-modules.yml | ||
| test-terraform-provider.yml | ||
| test-unit.yml | ||
| test-vscode-extension.yml | ||
| weekly_load_anomaly.yml | ||
| zizmor.yml | ||