Skip to content

test(fireworks): cover request-level extra headers#38518

Merged
Mason Daugherty (mdrxy) merged 1 commit into
masterfrom
mdrxy/fireworks/extra-headers
Jun 29, 2026
Merged

test(fireworks): cover request-level extra headers#38518
Mason Daugherty (mdrxy) merged 1 commit into
masterfrom
mdrxy/fireworks/extra-headers

Conversation

@mdrxy

@mdrxy Mason Daugherty (mdrxy) commented Jun 29, 2026

Copy link
Copy Markdown
Member

Fireworks chat users can pass request-specific headers to the SDK, but the integration did not have targeted coverage or examples for session-affinity and multi-turn headers. This adds explicit coverage for sync, async, and streaming calls, and documents the supported invocation patterns.

Changes

  • Documented ChatFireworks request-level extra_headers examples for session affinity and multi-turn sessions, plus the SDK-level prompt_cache_key alternative.
  • Added TestExtraHeaders coverage showing extra_headers reach top-level SDK kwargs for sync and streaming calls rather than being folded into extra_body.
  • Covered the async ainvoke path so request-specific headers are verified across the main call modes.

@github-actions github-actions Bot added fireworks `langchain-fireworks` package issues & PRs integration PR made that is related to a provider partner package integration internal size: S 50-199 LOC tests PRs made that add, modify, or remove tests labels Jun 29, 2026

@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

@mdrxy Mason Daugherty (mdrxy) merged commit 933adb0 into master Jun 29, 2026
56 checks passed
@mdrxy Mason Daugherty (mdrxy) deleted the mdrxy/fireworks/extra-headers branch June 29, 2026 04:13
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 integration PR made that is related to a provider partner package integration internal size: S 50-199 LOC tests PRs made that add, modify, or remove tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant