Skip to content

feat: alphabetical provider sorting and connected status for OpenCode/Kilo#12

Merged
aaditagrawal merged 1 commit intomainfrom
feat/opencode-kilo-alphabetical-providers
Mar 13, 2026
Merged

feat: alphabetical provider sorting and connected status for OpenCode/Kilo#12
aaditagrawal merged 1 commit intomainfrom
feat/opencode-kilo-alphabetical-providers

Conversation

@aaditagrawal
Copy link
Copy Markdown
Owner

@aaditagrawal aaditagrawal commented Mar 13, 2026 •
edited by coderabbitai bot
Loading

Summary

  • Sort providers alphabetically by name in model discovery for both OpenCode and Kilo server managers
  • Mark each provider's models with connected status from the provider.list API response so the UI can surface configured providers first in the picker
  • Pairs with existing client-side changes already on main that split the OpenCode/Kilo model picker into connected providers (shown first) and an expandable "All Providers" submenu

Test plan

  • Verify OpenCode model picker shows providers in alphabetical order
  • Verify Kilo model picker shows providers in alphabetical order
  • Verify connected providers appear before disconnected ones in the picker
  • Verify models within each provider group are sorted alphabetically

Summary by CodeRabbit

  • Bug Fixes
    • Enhanced error messaging for Kilo and OpenCode server managers to display detailed error information when encountering unhandled error types.

@aaditagrawal
feat: alphabetical provider sorting and connected status for OpenCode…
44d28cb 
…/Kilo

Sort providers alphabetically by name in model discovery for both OpenCode
and Kilo adapters. Mark each provider's models with connected status from
the provider.list API so the UI can surface configured providers first.
@github-actions github-actions bot added size:XS 0-9 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Mar 13, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 13, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e35cfd10-a280-4b99-8448-a117ee0e663c

📥 Commits

Reviewing files that changed from the base of the PR and between f7a9002 and 44d28cb.

📒 Files selected for processing (2)
  • apps/server/src/kiloServerManager.ts
  • apps/server/src/opencodeServerManager.ts
📝 Walkthrough

Walkthrough

Two server manager files are updated to enhance error message handling for unknown error types. Both kiloServerManager.ts and opencodeServerManager.ts now include default cases in their sessionErrorMessage functions that attempt to extract error details from error.data.message before falling back to constructing error strings using error.name.

Changes

Cohort / File(s) Summary
Error Message Handling Enhancement
apps/server/src/kiloServerManager.ts, apps/server/src/opencodeServerManager.ts		 Added default case to sessionErrorMessage function to handle unknown error types. Attempts to read error.data.message first, then falls back to constructed error string using error.name.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A catch-all clause hops into place,
For errors wearing an unknown face,
We peek at data.message with care,
Or craft a fallback, if nothing's there—
Better messages, hopping everywhere! 🌟

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The PR description covers the main changes and rationale, but the test plan section uses unchecked checkboxes suggesting incomplete testing rather than completed verification. Confirm whether the test plan items have been completed or are intended for reviewers; clarify the actual testing status before merge.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The PR title accurately describes the main changes: alphabetical provider sorting and connected status marking for OpenCode/Kilo server managers.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/opencode-kilo-alphabetical-providers
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

@aaditagrawal aaditagrawal merged commit 4de5477 into main Mar 13, 2026
8 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Mar 16, 2026
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size:XS 0-9 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@aaditagrawal