Skip to content

feat(ai-gateway): add Eden AI direct BYOK - #5270

Merged
chrarnoldus merged 4 commits into
mainfrom
feat/edenai-direct-byok
Aug 14, 2026
Merged

feat(ai-gateway): add Eden AI direct BYOK#5270
chrarnoldus merged 4 commits into
mainfrom
feat/edenai-direct-byok

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • add Eden AI as a direct BYOK provider using its OpenAI-compatible v3 API
  • synchronize models.dev metadata against Eden AI's live model catalog
  • force Eden AI model families through Chat Completions and translate reasoning settings
  • cover protocol selection, request transforms, and nullable live catalog metadata

Testing

  • local tests intentionally not run; CI handles validation

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Aug 14, 2026
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
Comment thread apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (31 files)

Incremental review of changes since e42eebb6: this pass adds per-chat-API base URL overrides (apiUrlOverrides / base_url_overrides) across the provider plumbing, enables Eden AI's Anthropic Messages (/v3/v1) and Responses endpoints alongside Chat Completions, and removes the blanket edenai AI-SDK provider override in favor of model-family detection. All Provider/DirectByokProvider constructions, the single upstreamRequest call site, and related tests were updated consistently; the previously reported reasoning: { enabled: false } issue remains resolved with regression coverage.

  • apps/web/src/app/api/openrouter/[...path]/route.test.ts
  • apps/web/src/app/api/openrouter/[...path]/route.ts
  • apps/web/src/lib/ai-gateway/experiments/build-direct-provider.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/alibaba-token-plan.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
  • apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.test.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/inceptron-byok.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/index.test.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/kimi-coding.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/martian.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/morph.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/neurowatt.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/nvidia-byok.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/ollama-cloud.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/opencode-go.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/orcarouter.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/synthetic.ts
  • 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/get-provider.ts
  • apps/web/src/lib/ai-gateway/providers/model-settings.ts
  • apps/web/src/lib/ai-gateway/providers/provider-definitions.ts
  • apps/web/src/lib/ai-gateway/providers/types.ts
  • apps/web/src/lib/ai-gateway/providers/upstream-request.generation.test.ts
  • apps/web/src/lib/ai-gateway/providers/upstream-request.ts
  • apps/web/src/tests/openrouterApi.timeout.test.ts
Previous Review Summaries (2 snapshots, latest commit e42eebb)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit e42eebb)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)

Incremental review of changes since 24ca612e; the previously reported reasoning: { enabled: false } issue is resolved by mapping it to reasoning_effort: "none" with regression test coverage.

  • apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.test.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.ts

Previous review (commit 24ca612)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.ts 14 Reasoning configs without effort (e.g. reasoning: { enabled: false }) are silently discarded by the request transform, so an explicit disable-reasoning signal is lost
Files Reviewed (8 files)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/direct-byok-definitions.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/direct-byok-meta.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.test.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/edenai.ts - 1 issue
  • apps/web/src/lib/ai-gateway/providers/direct-byok/sync-direct-byok.test.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/sync-direct-byok.ts
  • apps/web/src/lib/ai-gateway/providers/model-settings.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/inference-provider-id.ts

Reviewed by kimi-k3 · Input: 139.9K · Output: 8.5K · Cached: 1.1M

Review guidance: REVIEW.md from base branch main

chrarnoldus and others added 2 commits August 14, 2026 09:57
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus
chrarnoldus merged commit accd037 into main Aug 14, 2026
13 checks passed
@chrarnoldus
chrarnoldus deleted the feat/edenai-direct-byok branch August 14, 2026 14:08
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.

2 participants