Skip to content

feat: support Azure user BYOK via Vercel - #5989

Merged
chrarnoldus merged 4 commits into
mainfrom
satisfying-brother
Sep 9, 2026
Merged

feat: support Azure user BYOK via Vercel#5989
chrarnoldus merged 4 commits into
mainfrom
satisfying-brother

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • expose Azure OpenAI as a user BYOK provider for Vercel-backed models
  • validate and forward Azure apiKey, resourceName, and optional deployment model mappings
  • add masked structured-credential entry and Azure setup guidance to the BYOK UI
  • cover Azure credential validation and Vercel request transformation

Verification

  • pnpm exec oxfmt on changed files
  • git diff --check
  • local test suites intentionally not run; CI will handle tests

@chrarnoldus chrarnoldus self-assigned this Sep 9, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/components/organizations/byok/BYOKKeysManager.tsx
  • apps/web/src/routers/byok-router.ts
Previous Review Summaries (2 snapshots, latest commit f68acd3)

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

Previous review (commit f68acd3)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • apps/web/src/components/organizations/byok/BYOKKeysManager.tsx
  • apps/web/src/lib/ai-gateway/providers/vercel/index.ts
  • apps/web/src/routers/byok-router.ts

Previous review (commit dcbd9ac)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (8 files)
  • apps/web/src/components/organizations/byok/BYOKKeysManager.tsx
  • apps/web/src/lib/ai-gateway/providers/openrouter/inference-provider-id.test.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/inference-provider-id.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/types.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/index.ts
  • apps/web/src/routers/byok-router.test.ts
  • apps/web/src/routers/byok-router.ts

Reviewed by grok-4.6 · Input: 54.4K · Output: 6.2K · Cached: 155.1K

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.

2 participants