Skip to content

Conversation

@ariane-emory
Copy link
Contributor

@ariane-emory ariane-emory commented Nov 24, 2025

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

Screenshot 2025-11-24 at 2 44 48 AM

After

Screenshot 2025-11-24 at 2 44 34 AM

ariane-emory and others added 2 commits November 24, 2025 02:24
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.
@ariane-emory ariane-emory marked this pull request as draft November 24, 2025 07:28
@ariane-emory ariane-emory marked this pull request as ready for review November 24, 2025 07:32
@rekram1-node
Copy link
Collaborator

would it be more aesthetic to sort by length? That;s what we do in the docs
Screenshot 2025-11-24 at 1 50 25 AM

@ariane-emory
Copy link
Contributor Author

ariane-emory commented Nov 24, 2025

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.

@rekram1-node
Copy link
Collaborator

Yeah that's valid

@rekram1-node rekram1-node merged commit 75c0c0a into sst:dev Nov 24, 2025
@ariane-emory
Copy link
Contributor Author

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 opencode.json file... that'd theoretically give the user more control, since they could manually sort them however they wanted by editing the JSON, but really I don't know if that control would really be worth the effort of editing the JSON.

shuv1337 pushed a commit to Latitudes-Dev/shuvcode that referenced this pull request Nov 24, 2025
shuv1337 added a commit to Latitudes-Dev/shuvcode that referenced this pull request Nov 24, 2025
* 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]>
@ariane-emory ariane-emory deleted the feature/alphabetic-mcp-display-order branch November 27, 2025 04:07
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.

3 participants