[cliffhall]

Closes #2782

What changed

The Tools sidebar used to list tools in whatever order the server returned from tools/list. It now sorts them alphabetically by the label each row shows: the title when the tool has one, otherwise the name.

• compareToolsByLabel (clients/web/src/utils/toolUtils.ts): a pure comparator built on resolveDisplayLabel and an Intl.Collator with sensitivity: "base" and numeric: true. Sorting ignores case and compares numbers by value, so tool_2 comes before tool_10. Labels that differ only in case count as equal, and because Array.prototype.sort is stable, they keep the server's order.
• ToolControls: sorts after filtering. The listed tools and the "Excluded (SEP-2243)" section are both sorted.

Selection is unaffected

Each row key is still taken from the tool's position in the server's original list, before filtering and sorting. So toolRowKey, findToolByRowKey and the duplicate-name handling (#1957, #2001) behave exactly as before. A new test clicks a tool that moved from source index 3 to the top of the list and checks that it still reports 3:git_commit.

Tests

• toolUtils.test.ts: sorts by title when present, otherwise by name; ignores case; compares numbers by value; equal labels compare as 0.
• ToolControls.test.tsx: rows render in sorted order; the order holds while a search narrows the list; a moved row reports its source-position key; excluded tools are sorted too.

npm run local:gate passes.

Screenshots

Both screenshots use a test server that returns 10 tools in shuffled order (1280×900).

Before (server order):

After (sorted):

🤖 Generated with Claude Code

[Copilot]

🟡 Changes recommended

Excluded rows sort by title but display only names, so their visible ordering can remain nonalphabetical.

1 open issue

• Sort excluded tools by their displayed name · New

What changed in this PR

Adds alphabetical sorting to the web Tools sidebar for #2782, while preserving source-position row keys used to select duplicate-name tools.

Changes:

• Adds a label comparator with case-insensitive, numeric-aware ordering.
• Sorts regular and excluded tools after filtering.
• Adds tests for ordering, filtered results, and selection identity.

File	Description
clients/​web/​src/​utils/​toolUtils.ts	Adds the displayed-label comparator.
clients/​web/​src/​utils/​toolUtils.test.ts	Tests label precedence, case handling, and numeric ordering.
clients/​web/​src/​components/​groups/​ToolControls/​ToolControls.tsx	Sorts filtered rows while retaining source-position keys.
clients/​web/​src/​components/​groups/​ToolControls/​ToolControls.test.tsx	Tests rendered order and selection after sorting.

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

The focused changes match displayed labels, preserve selection identity, and include targeted regression coverage with no unresolved findings.

No open issues 🎉

1 resolved since last review

• Sort excluded tools by their displayed name

_{Review effort: Balanced}

[cliffhall]

Copilot review loop closed — round 2 came back clean.

• Round 1: one finding. Excluded rows were sorted by their resolved title but display only tool.name, so the visible order could be wrong. Fixed in 8e789d7: excluded rows now sort by name through a new compareToolLabels helper, and a regression test uses titles that would reverse the order.
• Round 2: no inline comments, nothing suppressed, approval recommended. Per the review procedure, a clean round ends the loop.