Skip to content

Added Fireworks as a provider#265

Open
ndrewGele wants to merge 1 commit intocharmbracelet:mainfrom
ndrewGele:add-fireworks-provider
Open

Added Fireworks as a provider#265
ndrewGele wants to merge 1 commit intocharmbracelet:mainfrom
ndrewGele:add-fireworks-provider

Conversation

@ndrewGele
Copy link
Copy Markdown

  • [ x ] I have read CONTRIBUTING.md.
  • [ x ] I have created a discussion that was approved by a maintainer (for new features).

Worth noting that my agent identified some discrepancies between which serverless models Fireworks advertises on their website and what their API returns, but I decided to use the API as a source of truth.
Ran it locally and asked a few of the cheaper models to write fizzbuzz for me as a test.

@ndrewGele ndrewGele requested a review from andreynering as a code owner April 26, 2026 15:30
@ndrewGele ndrewGele changed the title Added Fireworks as a provider, with some caveats Added Fireworks as a provider Apr 26, 2026
@meowgorithm
Copy link
Copy Markdown
Member

Thanks for the quick PR @ndrewGele! AFAIK, Fireworks has support for OpenAI and Anthropic-compatible endpoints, so we should be able to add this in without any Go code.

I think it also supports newer formats like the Responses API, but up to @andreynering which of these is the right shape.

@ndrewGele
Copy link
Copy Markdown
Author

I'll take a look at the linter issues later today.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ndrewGele ndrewGele force-pushed the add-fireworks-provider branch from a49b774 to 26bf09a Compare April 26, 2026 21:24
@ndrewGele
Copy link
Copy Markdown
Author

Thanks for the quick PR @ndrewGele! AFAIK, Fireworks has support for OpenAI and Anthropic-compatible endpoints, so we should be able to add this in without any Go code.

I think it also supports newer formats like the Responses API, but up to @andreynering which of these is the right shape.

Hey, missed this reply earlier, but thanks for the quick review on your end as well!
It sounds like there's a preference to keep things simply defined in the JSON over adding scrapers? If so, the new commits should address that.
Also noticed that a binary got committed initially, so I killed that with the force push.

@andreynering
Copy link
Copy Markdown
Member

Hi @ndrewGele,

Some notes:

  • I'd say we should use the OpenAI-compat endpoint as a default, unless we have reasons to believe the Anthropic-compatible one works better.
  • If this provider has a public /models endpoint, having a generator is prefferable. That means we'll be updating the list daily.

@andreynering
Copy link
Copy Markdown
Member

They have a /models endpoint, but it requires authentication. Would still be useful, tho.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants