Skip to content

refactor: remove colon from Deprecated auto models prefix#1062

Merged
markijbema merged 1 commit intomainfrom
remove-deprecated-colon
Mar 12, 2026
Merged

refactor: remove colon from Deprecated auto models prefix#1062
markijbema merged 1 commit intomainfrom
remove-deprecated-colon

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot Bot commented Mar 12, 2026

Summary

  • Remove the colon from the "Deprecated: " prefix on auto model names, changing it to "Deprecated " for cleaner display.
  • Updated the source logic in src/lib/kilo-auto-model.ts and the corresponding approval test snapshot in src/tests/openrouter-models-sorting.approved.json.

Verification

  • Grepped the entire codebase for Deprecated: — confirmed only an unrelated code comment in packages/db/src/schema.ts remains (not an auto model name).
  • Typecheck (pnpm typecheck) could not run due to missing tsgo binary in the environment; change is a string literal only, no type impact.

Visual Changes

N/A

Reviewer Notes

Straightforward string change — no logic or type changes. The only remaining Deprecated: in the codebase is a code comment in packages/db/src/schema.ts:1077 about a deprecated balance field, which is unrelated.

Change 'Deprecated: ' to 'Deprecated ' in auto model names for
cleaner display. Updates both the source logic in kilo-auto-model.ts
and the corresponding approval test snapshot.
@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot Bot commented Mar 12, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • src/lib/kilo-auto-model.ts
  • src/tests/openrouter-models-sorting.approved.json

Reviewed by gpt-5.4-20260305 · 227,971 tokens

@chrarnoldus chrarnoldus requested a review from markijbema March 12, 2026 17:08
@markijbema markijbema merged commit 26a4c2b into main Mar 12, 2026
18 checks passed
@markijbema markijbema deleted the remove-deprecated-colon branch March 12, 2026 18:41
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