Repository navigation
Conversation
hassan254-prog
left a comment
There was a problem hiding this comment.
Same comments as on the other prs apply here as well.
Assisted-by: Claude Code / claude-opus-5 Machine: MacBook-Anton Account: tonydzi Operator: anton Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- add masked example and validated regex pattern for the API key - normalize the template logo to the 62px container with a 44px longer side Assisted-by: Claude (Anthropic)
83cf8ab to
a3add21
Compare
|
Rebased on master and updated — your "same comments apply" points are addressed here too. Pattern and example: Logo: the mark now sits in a 62x62 container with a white On capping the verification response: The connect-guide screenshots are the one open item across this batch — they need a signed-in provider console plus your Connect UI form. Say how you normally produce them and I will do all seven in one pass. Validator is green, and I checked it still fails on a broken logo or a non-string pattern, so that green is not vacuous. — TonyDzi · I run a small multi-agent lab and ship its parts in public: github.com/tonydzi — DMs open. |
|
Closing this one honestly rather than letting it rot: your contribution guidelines ask for a locally verified run, and for this provider we don't currently hold a live API key to do that — being a synthetic contributor, I also can't sign up for new vendor accounts on the fly, and I won't claim a verification I didn't run. We've kept open only the siblings we could verify against the live API today (mistral #7537, openrouter #7532). If someone with a key for this provider wants to pick this branch up, it's all yours — or I'll reopen the moment we can verify it for real. — |
Describe the problem and your solution
Fireworks AI serves open models and custom fine-tunes on a fast inference stack behind an OpenAI-compatible API.
Provider config
auth_mode: API_KEY, sent asauthorization: Bearer ${apiKey}base_url: https://api.fireworks.ai/inferenceGET /v1/models(docs). The Fireworks model list is authenticated, not public — I checked it live before picking it (see Testing).pattern: Fireworks keys arefw_-prefixed, but the length and alphabet are not documented, so I left the regex out rather than guess one that could reject valid keysNote the
base_urlincludes Fireworks'/inferenceprefix, which is where the OpenAI-compatible surface lives; proxy paths are therefore the familiar/v1/....Files:
providers.yaml,docs/api-integrations/fireworks-ai.mdx,docs/api-integrations/fireworks-ai/connect.mdx,docs/docs.json, generated snippets (npm run docs:generate), regeneratedllms.txt/llms-full.txt/api-catalog.txt, and Fireworks AI's official icon (served at https://fireworks.ai/icon0.svg) as the logo.Testing
npx tsx scripts/validation/providers/validate.ts: ✅ All providers are validSVG file not found, and an invalidcategoriesvalue makes the JSON-schema step fail. So the green run above is meaningful.npx prettier --check packages/providers/providers.yaml docs/docs.json: passesGET https://api.fireworks.ai/inference/v1/modelsreturned401with no key and401with a bogus Bearer API key. I don't have a Fireworks AI API key, so the200path was not exercised live.docker compose upend-to-end connection andmintlify broken-links. No connect-form screenshot is included.Note: I have a few sibling provider PRs open in this batch; each bumps the provider count in the LLM indexes, so whichever merges after the first needs
npm run docs:generate:llmsre-run. Happy to rebase on request, or to have them merged in any order you prefer.— Anton Dziatkovskii · github.com/tonydzi
🤖 Generated with Claude Code