Skip to content

Add Fireworks fallback for CanopyWave - #571

Closed
jahooma wants to merge 6483 commits into
mainfrom
jahooma/fireworks-fallback
Closed

Add Fireworks fallback for CanopyWave#571
jahooma wants to merge 6483 commits into
mainfrom
jahooma/fireworks-fallback

Conversation

@jahooma

@jahooma jahooma commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Adds a server-side fallback from CanopyWave to Fireworks for supported models when CanopyWave returns availability failures such as 429, 5xx, or "No available workers".
If CanopyWave is not configured, supported models now route directly to Fireworks instead of failing on the missing key.
This keeps the existing provider billing/logging paths intact and covers both streaming and non-streaming completions.
Validated with bun test web/src/app/api/v1/chat/completions/__tests__/completions.test.ts and bun run --cwd web typecheck.

James Grugett and others added 30 commits March 24, 2026 12:32
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…481)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…vements (#486)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

ghost 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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@codebuff-team

ghost commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

This PR modifies files under web/src/app/api/v1/chat/completions/_post.ts and its test file, which fall under the web/ path excluded from this mirror's review scope per repo policy. Regardless of the change's technical merit (the CanopyWave→Fireworks fallback logic and the added test coverage for both streaming and non-streaming paths look reasonable and well-tested), this repo cannot accept changes to web/. If this is meant to be ported to the private tree, it should go through that internal process rather than this mirror.

@codebuff-team codebuff-team added bot:triaged Classified by the community triage bot pr:out-of-scope Touches paths the public mirror does not accept labels Aug 20, 2026
@victorxheng
victorxheng force-pushed the jahooma/fireworks-fallback branch from b0ec624 to 9064d9c Compare August 31, 2026 21:01
@github-actions

ghost commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

PR checks failed

A couple of things need fixing before this can be reviewed.
None of them are about the code itself.

This PR touches paths the public mirror does not accept.

  • web/src/app/api/v1/chat/completions/__tests__/completions.test.ts
  • web/src/app/api/v1/chat/completions/_post.ts

Backend, database, billing and deployment code is not part of
this repository. A change to those paths cannot be merged here
regardless of its quality. See CONTRIBUTING.md for the paths
that are in scope.


Edit the PR and this check re-runs automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:triaged Classified by the community triage bot pr:out-of-scope Touches paths the public mirror does not accept

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants