Skip to content

Add a bunch of BYOK providers#3704

Open
chrarnoldus wants to merge 3 commits into
mainfrom
christiaan/byok
Open

Add a bunch of BYOK providers#3704
chrarnoldus wants to merge 3 commits into
mainfrom
christiaan/byok

Conversation

@chrarnoldus
Copy link
Copy Markdown
Contributor

No description provided.

@chrarnoldus chrarnoldus self-assigned this Jun 3, 2026
export const DirectUserByokInferenceProviderIdSchema = z.enum([
'byteplus-coding',
'chutes-byok',
'crofai',
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.

SUGGESTION: 'crofai' is placed before 'codestral' here, breaking alphabetical ordering. codestral sorts before crofai.

Suggested change
'crofai',
'codestral',
'crofai',

import xiaomiTokenPlanAms from './xiaomi-token-plan-ams';
import xiaomiTokenPlanSgp from './xiaomi-token-plan-sgp';
import zaiCoding from './zai-coding';
import crofai from './crofai';
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.

SUGGESTION: The four new imports (crofai, martian, orcarouter, synthetic) are appended at the bottom, breaking the alphabetical ordering of the existing imports. They should be interleaved with the existing ones.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Jun 3, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

All previously flagged import-ordering issues have been resolved in the latest commit; no new issues found.

Files Reviewed (21 files)
  • apps/web/src/lib/ai-gateway/providers/get-provider.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/byteplus-coding.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/chutes-byok.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/crofai.ts (new)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/direct-byok-definitions.ts — ✅ fixed (imports now alphabetical)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/direct-byok-meta.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/index.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/kimi-coding.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/martian.ts (new)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/neurowatt.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/ollama-cloud.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/orcarouter.ts (new)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/sync-direct-byok.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/synthetic.ts (new)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/types.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/xiaomi-token-plan-ams.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/xiaomi-token-plan-sgp.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/zai-coding.ts
  • apps/web/src/lib/ai-gateway/providers/model-settings.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/inference-provider-id.ts — ✅ fixed ('crofai' now after 'codestral')
  • apps/web/src/lib/ai-gateway/experiments/build-direct-provider.ts

Reviewed by claude-4.6-sonnet-20260217 · 144,491 tokens

Review guidance: REVIEW.md from base branch main

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.

1 participant