-
Notifications
You must be signed in to change notification settings - Fork 3.4k
tweak: display MCPs in alphabetic order in the sidebar. #4680
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tweak: display MCPs in alphabetic order in the sidebar. #4680
Conversation
Display MCP servers in alphabetical order in the session sidebar for consistent and predictable ordering. Previously, MCPs appeared in the order their connections completed, which varied based on startup time and network latency.
…ane-emory/opencode into feature/alphabetic-mcp-display-order
|
Well... I'm undecided, but kind of leaning towards alphabetical myself so far. Either of these options (length based or alphabetical) would be more aesthetic than the current state, where the order is just some arbitrary consequence of what order some Promises resolve in... To me, alphabetical does feel kind of like it would be the least surprising.... what prompted me making this change was that I'd just added the 3 z.ai MCPs and was surprised and somewhat confused to find that not only were these three Z-prefixed MCPs not all at the bottom of the list, but that they weren't even all together (notice in the before screenshot, basic-memory and calculator-mcp occur in between two of the zai MCPs, preceded by two zai MCPs that just so happened to occur next to one another). If I started opencode and saw the alphabetical list of MCPs, that would feel so natural that I wouldn't even think about it twice.... us English speakers see so many alphabetically ordered lists in our lives that our brains have been trained to pass them over without a second thought. Length based might briefly surprise me and I'd think "Why are they sorted this way?" but it'd likely only take my brain a couple of seconds to puzzle out that length explained the sorting, so I'd get over the momentary surprise pretty quickly. Ultimately your call, either option would be less surprising than the current Promise-resolution-timing ordering. |
|
Yeah that's valid |
|
Another possible ordering that I considered (but haven't tried implementing yet) was to match the display order to the order of definition in the |
Co-authored-by: Github Action <[email protected]>
* ci: fix action * feature: optional selectedListItemText element in themes and luminance-based fallback to solve 4369 (sst#4572) Co-authored-by: knanao <[email protected]> Co-authored-by: knanao <[email protected]> * ignore: update download stats 2025-11-23 * ignore: update @solidjs/start dependency and fix console redirect handling Updates the @solidjs/start dependency to latest version and removes deprecated getResponseHeaders usage from auth callback. Also adds error handling for workspace ID lookup to prevent redirect failures. * Update Nix flake.lock and hashes * tweak: fix bool * sync * ci: stuff * Update Nix flake.lock and hashes * ci: stuff * Update Nix flake.lock and hashes * fix bash tool wsl * ci: ignore * release: v1.0.106 * Update Nix flake.lock and hashes * ci: ignore * token * ci: secret * chore: format code * ci: ignore * ci: ignore * ci: ignore * ci: remove log * tweak: slight improvements to title gen * wip: zen * wip: zen * chore: format code * fix: add explicit fallback model and prevent direct opencode provider calls (sst#4653) Co-authored-by: Aiden Cline <[email protected]> * nix: bundle js dist with bun and patch tree-sitter wasm paths (sst#4644) Co-authored-by: Aiden Cline <[email protected]> Co-authored-by: Github Action <[email protected]> * chore: format code * fix: fatal: undefined is not an object (evaluating 'color.buffer') * bump copilot plugin, give better error message for copilot (sst#4678) * release: v1.0.107 * tweak: modified files sidebar * fix: persist light vs dark mode * chore: format code * tweak: display MCPs in alphabetic order in the sidebar. (sst#4680) Co-authored-by: Github Action <[email protected]> * TUI: fix: add null check for user.time in duration calculation (sst#4679) * ignore: update download stats 2025-11-24 * docs: add Ollama Cloud provider setup to providers.mdx (sst#4693) * non-corpo loading spinner * chore: format code * release: v1.0.108 * textarea highlight cursor color * chore: format code * wip: zen * ci: nix hash * add autoupdate: notify * zen: add opus 4.5 * wip(share): more styling * wip(share): more styling * wip(share): more styling * wip(share): enterprise favicon * fix(console): favicon issues * chore: format code * fix(console): remove extra favicon * adjust bundled provider logic, fix tree shaking stuff (sst#4708) * wip: zen * zen: display reasoning tokens * fix(desktop): layout * Add --refresh flag to models command (sst#4707) * zen: byok for gemini * remove hardcoded openrouter provider * release: v1.0.109 * chore: format code * Update Nix flake.lock and hashes --------- Co-authored-by: Aiden Cline <[email protected]> Co-authored-by: Ariane Emory <[email protected]> Co-authored-by: knanao <[email protected]> Co-authored-by: knanao <[email protected]> Co-authored-by: GitHub Action <[email protected]> Co-authored-by: Dax Raad <[email protected]> Co-authored-by: opencode <[email protected]> Co-authored-by: Frank <[email protected]> Co-authored-by: Huang Qi <[email protected]> Co-authored-by: Albert O'Shea <[email protected]> Co-authored-by: Aiden Cline <[email protected]> Co-authored-by: Carli Samuele <[email protected]> Co-authored-by: Frankie Seabrook <[email protected]> Co-authored-by: Sebastian Herrlinger <[email protected]> Co-authored-by: Adam <[email protected]> Co-authored-by: Adam Hosker <[email protected]>

Display MCP servers in alphabetical order in the session sidebar for consistent and predictable ordering. Previously, MCPs appeared in the order their connections completed, which varied based on startup time and network latency.
Before
After