Skip to content

Enable routing to Vercel for Anthropic and OpenAI models#641

Open
chrarnoldus wants to merge 3 commits intomainfrom
christiaan/vercel
Open

Enable routing to Vercel for Anthropic and OpenAI models#641
chrarnoldus wants to merge 3 commits intomainfrom
christiaan/vercel

Conversation

@chrarnoldus
Copy link
Contributor

No description provided.

@chrarnoldus chrarnoldus marked this pull request as ready for review March 2, 2026 21:28
@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Mar 2, 2026

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
src/lib/providers/vercel/index.ts 74 OpenAI models are now also routed to Vercel (not just Anthropic as PR title suggests). The old exclusion noted gpt-5.3-codex-api-preview didn't exist on Vercel — confirm this is resolved.
src/lib/providers/vercel/index.ts (removed code) The context-1m-2025-08-07 beta flag was dropped from the Vercel Anthropic path — see Other Observations.
Other Observations (not in diff)

Issues found in removed/unchanged code that cannot receive inline comments:

File Line Issue
src/lib/providers/vercel/index.ts (removed ~line 153) The old applyVercelSettings added context-1m-2025-08-07 to the anthropic-beta header for Vercel-routed Anthropic requests (per Vercel docs). This beta flag is now completely removed from the Vercel path. If Vercel still requires this flag for 1M context window support, Anthropic models routed through Vercel may silently lose access to extended context.
Files Reviewed (2 files)
  • src/lib/providers/anthropic.ts - 0 new issues (existing comment covers the dual-header change)
  • src/lib/providers/vercel/index.ts - 2 issues

Fix these issues in Kilo Cloud

# Conflicts:
#	src/lib/providers/vercel/index.ts
@chrarnoldus chrarnoldus changed the title Enable routing to Vercel for Anthropic models Enable routing to Vercel for Anthropic and OpenAI models Mar 3, 2026
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