Skip to content

fix(fireworks): filter invalid tool calls from v1 content - #39805

Merged
ccurme (ccurme) merged 1 commit into
masterfrom
ccurme/fireworks/filter-invalid-tool-calls
Aug 20, 2026
Merged

fix(fireworks): filter invalid tool calls from v1 content#39805
ccurme (ccurme) merged 1 commit into
masterfrom
ccurme/fireworks/filter-invalid-tool-calls

Conversation

@ccurme

Copy link
Copy Markdown
Collaborator

Fireworks rejects LangChain's invalid_tool_call content block when malformed or partial tool-call history is replayed. Filter that unsupported block during v1-to-Chat-Completions conversion while retaining AIMessage.invalid_tool_calls for the existing diagnostic tool_calls serialization.

Covered by a focused request-serialization regression test that also verifies adjacent text is preserved.

Made by Open SWE

Co-authored-by: open-swe[bot] <open-swe@users.noreply.github.com>
@github-actions github-actions Bot added fireworks `langchain-fireworks` package issues & PRs fix For PRs that implement a fix integration PR made that is related to a provider partner package integration internal size: XS < 50 LOC labels Aug 20, 2026
@ccurme
ccurme (ccurme) marked this pull request as ready for review August 20, 2026 14:14

@open-swe open-swe Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in WebView Open SWE trace

@ccurme
ccurme (ccurme) merged commit e727daa into master Aug 20, 2026
54 checks passed
@ccurme
ccurme (ccurme) deleted the ccurme/fireworks/filter-invalid-tool-calls branch August 20, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fireworks `langchain-fireworks` package issues & PRs fix For PRs that implement a fix integration PR made that is related to a provider partner package integration internal size: XS < 50 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant