feat(providers): add Kie provider for Claude, GPT, Grok, and Kimi models - #8391
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
There was a problem hiding this comment.
1 issue found across 24 files
Confidence score: 4/5
- In
apps/sim/providers/models.ts, accepting arbitrarykie/...IDs as custom models can let unsupported non-Claude IDs pass validation and then fail at execution; restrict the custom namespace to Claude IDs.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="apps/sim/providers/models.ts">
<violation number="1" location="apps/sim/providers/models.ts:3706">
P2: This pattern accepts arbitrary `kie/...` IDs as custom models, but non-Claude IDs must match the fixed Responses endpoint map and otherwise fail at execution. Restrict the custom namespace to Claude IDs; the 11 catalogued GPT/Grok/Kimi IDs still resolve through exact catalog matches.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
|
…treaming tool-call provider
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 25 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
…the request contract
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 26 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
…pes case-insensitively
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
kie/prefix: Claude (Opus 5.5 → Haiku 4.5), GPT 6 / 5.6 / 5.5, Grok 4.6 / 4.5 / 4.3, and Kimi K3low–xhigh){ code, msg };createKieFetchrewrites those into real HTTP errors so bad keys fail fast, rate limits retry, and failures never surface as empty completionsstreamto true, so the Responses and Anthropic cores now sendstream: falseexplicitly on non-streaming calls (a no-op for OpenAI/Anthropic)ResponsesProviderConfig.capabilityModellets a reseller's wire slug differ from the catalog id used for reasoning-effort lookupgetProviderHistoryProtocol), since Kie serves both Anthropic and Responses wire formats under one providerType of Change
Testing
conversation-smoke.test.tsresponses case); computed cost matches Kie's billed creditsproviders/kie/transport.test.ts(reverting the guard turns 6 of 10 red),history-adapters.test.ts, providers/memory/BYOK suites (2,204 tests)bun run lint,check:audits,docs-manifest:check, block registry check,type-checkChecklist
test-auditauthoring gate)