Skip to content

Return retryable error for auto-routing selection failures - #5993

Merged
chrarnoldus merged 3 commits into
mainfrom
broadleaf-eagle
Sep 9, 2026
Merged

Return retryable error for auto-routing selection failures#5993
chrarnoldus merged 3 commits into
mainfrom
broadleaf-eagle

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • describe the failure as an auto-routing selection problem instead of blaming organization model restrictions
  • return HTTP 503 so clients can treat the failed routing decision as transient and retryable
  • align routing comments and focused test expectations with the new behavior

Testing

  • Local tests were not run, as requested; CI handles validation.
  • git diff --check passed.

@chrarnoldus chrarnoldus self-assigned this Sep 9, 2026
Comment thread apps/web/src/lib/ai-gateway/llm-proxy-helpers.ts
@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/lib/ai-gateway/llm-proxy-helpers.ts
  • apps/web/src/app/api/openrouter/[...path]/route.test.ts
Previous Review Summaries (2 snapshots, latest commit 9b47d94)

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

Previous review (commit 9b47d94)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • apps/web/src/app/api/openrouter/[...path]/route.test.ts

Previous review (commit c642eed)

Status: 1 Issues Found | Recommendation: Address before merge

Overview

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

WARNING

File Line Issue
apps/web/src/lib/ai-gateway/llm-proxy-helpers.ts 332 Tests still assert the old error copy
Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/llm-proxy-helpers.ts - 1 issue
  • apps/web/src/app/api/openrouter/[...path]/route.ts - 0 issues

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 77.1K · Output: 7.7K · Cached: 295.2K

Review guidance: REVIEW.md from base branch main

@chrarnoldus chrarnoldus changed the title Rephrase auto-routing selection error Return retryable error for auto-routing selection failures Sep 9, 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.

2 participants