Skip to content

Indicate when the model list is empty - #11085

Merged
chrarnoldus merged 2 commits into
mainfrom
indicate-empty-model-list
Jun 11, 2026
Merged

Indicate when the model list is empty#11085
chrarnoldus merged 2 commits into
mainfrom
indicate-empty-model-list

Conversation

@chrarnoldus

Copy link
Copy Markdown
Collaborator

Summary

  • Track whether any models are available when model lookup fails
  • Include an explicit empty-model-list indication in CLI and session errors
  • Add focused formatter coverage and a patch changeset

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Jun 10, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (5 files)
  • .changeset/clear-empty-models.md
  • packages/opencode/src/cli/error.ts
  • packages/opencode/src/provider/provider.ts
  • packages/opencode/src/session/prompt.ts
  • packages/opencode/test/kilocode/cli/error.test.ts

lgtm — the incremental commit correctly scopes the empty-model check to the specific provider rather than checking all providers globally, and sets modelsEmpty: false for the "provider not found" branch where model availability is indeterminate.


Reviewed by claude-4.6-sonnet-20260217 · 172,499 tokens

Review guidance: REVIEW.md from base branch main

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus
chrarnoldus merged commit f31d370 into main Jun 11, 2026
21 checks passed
@chrarnoldus
chrarnoldus deleted the indicate-empty-model-list branch June 11, 2026 08:02
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…list

Indicate when the model list is empty
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