Skip to content

fix(ai-gateway): use Poolside key for free Laguna - #4980

Open
chrarnoldus wants to merge 4 commits into
mainfrom
fix/poolside-free-byok
Open

fix(ai-gateway): use Poolside key for free Laguna#4980
chrarnoldus wants to merge 4 commits into
mainfrom
fix/poolside-free-byok

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • route poolside/laguna-s-2.1:free through Vercel using the Vercel-specific model ID restored from fix(ai-gateway): route Laguna through Vercel #4977
  • add the Poolside BYOK credential when no user BYOK is configured and POOLSIDE_FREE_API_KEY is set
  • preserve user BYOK precedence and skip the fallback when the environment variable is unset
  • document POOLSIDE_FREE_API_KEY

Testing

  • pnpm --filter web test -- --runInBand apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.test.ts
  • pnpm --filter web typecheck
  • pnpm --filter web lint
  • pnpm exec oxfmt --list-different apps/web/src/lib/ai-gateway/providers/vercel/index.ts apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.ts apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.test.ts ENVIRONMENT.md
  • git diff --check origin/main...HEAD

@chrarnoldus chrarnoldus self-assigned this Aug 3, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (5 files)
  • ENVIRONMENT.md
  • apps/web/src/lib/ai-gateway/providers/vercel/index.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.test.ts
Previous Review Summary (commit 17da342)

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

Previous review (commit 17da342)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • apps/web/src/lib/ai-gateway/providers/vercel/index.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts
  • ENVIRONMENT.md

Reviewed by kimi-k3 · Input: 39.3K · Output: 6.9K · Cached: 228.4K

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