Skip to content

Update model recommendations and defaults - #6050

Merged
chrarnoldus merged 5 commits into
mainfrom
utopian-fright
Sep 10, 2026
Merged

Update model recommendations and defaults#6050
chrarnoldus merged 5 commits into
mainfrom
utopian-fright

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add ~z-ai/glm-flash-latest and map it to zai/glm-5.3-flash for Vercel routing
  • update Claude Fable latest from 5 to 5.1
  • recommend DeepSeek V4.1 Flash and GLM 5.3 Flash ahead of Kimi and MiniMax
  • remove Claude Sonnet 5 and GLM 5.3 from the recommended model list
  • change PRIMARY_DEFAULT_MODEL to z-ai/glm-5.3-flash
  • use PRIMARY_DEFAULT_MODEL for every Auto Balanced and Auto Efficient fallback path

Testing

  • pnpm --filter web test --runInBand src/lib/ai-gateway/auto-model/resolution.test.ts src/tests/openrouter-models-config.test.ts (31 tests passed)
  • pnpm --filter @kilocode/trpc run build
  • formatted changed files with oxfmt
  • git diff --check

@chrarnoldus chrarnoldus self-assigned this Sep 10, 2026
Comment thread apps/web/src/lib/ai-gateway/models.ts
@kilo-code-bot

kilo-code-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Unifying Auto Balanced and Auto Efficient static fallbacks onto PRIMARY_DEFAULT_MODEL is consistent, fully referenced, and covered by tests.

Files Reviewed (3 files)
  • apps/web/src/lib/ai-gateway/auto-model/index.ts
  • apps/web/src/lib/ai-gateway/auto-model/resolution.test.ts
  • apps/web/src/lib/ai-gateway/auto-model/resolution.ts
Previous Review Summaries (3 snapshots, latest commit 22b60f6)

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

Previous review (commit 22b60f6)

Status: No Issues Found | Recommendation: Merge

Executive Summary

Incremental review of the GLM Flash efficient fallback and PRIMARY_DEFAULT_MODEL switch found no remaining defects.

Files Reviewed (5 files)
  • apps/web/src/lib/ai-gateway/auto-model/index.ts
  • apps/web/src/lib/ai-gateway/auto-model/resolution.test.ts
  • apps/web/src/lib/ai-gateway/auto-model/resolution.ts
  • apps/web/src/lib/ai-gateway/models.ts
  • apps/web/src/tests/openrouter-models-config.test.ts

Previous review (commit a2365e7)

Status: No Issues Found | Recommendation: Merge

Executive Summary

Incremental review of the restored CLAUDE_SONNET_CURRENT_MODEL_ID import confirmed PRIMARY_DEFAULT_MODEL is defined again; no remaining defects.

Files Reviewed (1 files)
  • apps/web/src/lib/ai-gateway/models.ts

Previous review (commit 3fcb378)

Status: 1 Issue Found | Recommendation: Address before merge

Executive Summary

Removing CLAUDE_SONNET_CURRENT_MODEL_ID from the models.ts import leaves PRIMARY_DEFAULT_MODEL undefined and will fail typecheck.

Overview

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

CRITICAL

File Line Issue
apps/web/src/lib/ai-gateway/models.ts 11 CLAUDE_SONNET_CURRENT_MODEL_ID still used by PRIMARY_DEFAULT_MODEL but dropped from the import
Files Reviewed (8 files)
  • apps/web/src/lib/ai-gateway/models.ts - 1 issue
  • apps/web/src/lib/ai-gateway/latest-model-aliases.ts
  • apps/web/src/lib/ai-gateway/providers/anthropic.constants.ts
  • apps/web/src/lib/ai-gateway/providers/deepseek.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.test.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.ts
  • apps/web/src/lib/ai-gateway/providers/zai.ts
  • apps/web/src/tests/openrouter-models-config.test.ts

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 105.4K · Output: 6K · Cached: 155.6K

Review guidance: REVIEW.md from base branch main

@chrarnoldus chrarnoldus changed the title Update recommended flash models Update flash model recommendations and defaults Sep 10, 2026
@chrarnoldus chrarnoldus changed the title Update flash model recommendations and defaults Update model recommendations and defaults Sep 10, 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