Skip to content

Update normalizeKilocodeModel to exclude 'kilo/auto'#692

Merged
chrarnoldus merged 5 commits intomainfrom
chrarnoldus-patch-1
Mar 2, 2026
Merged

Update normalizeKilocodeModel to exclude 'kilo/auto'#692
chrarnoldus merged 5 commits intomainfrom
chrarnoldus-patch-1

Conversation

@chrarnoldus
Copy link
Contributor

@chrarnoldus chrarnoldus commented Feb 28, 2026

Fix for kilo/auto not working in cloud agents

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

kilo-code-bot bot commented Mar 1, 2026

Code Review Summary

Status: 3 Issues Found | Recommendation: Address before merge

Overview

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

WARNING

File Line Issue
cloud-agent-next/src/persistence/model-utils.ts 5 Broad startsWith('kilo/auto') check will match all auto models including kilo/auto-free, kilo/auto-small, and any future kilo/auto* model — confirm this is intentional
src/lib/providers/index.ts 112 Breaking change: prefix changed from kilo/ to kilo-internal/ for custom LLM lookup — existing DB records and clients must be aligned

SUGGESTION

File Line Issue
cloud-agent-next/src/persistence/model-utils.test.ts 27 Missing test cases for existing auto models kilo/auto-free and kilo/auto-small
Files Reviewed (3 files)
  • cloud-agent-next/src/persistence/model-utils.ts - 1 issue
  • cloud-agent-next/src/persistence/model-utils.test.ts - 1 issue
  • src/lib/providers/index.ts - 1 issue

Fix these issues in Kilo Cloud

@chrarnoldus chrarnoldus merged commit 863e227 into main Mar 2, 2026
12 checks passed
@chrarnoldus chrarnoldus deleted the chrarnoldus-patch-1 branch March 2, 2026 08:25
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