fix(vscode): match slash commands on the command name only

A description is prose, so matching it pulled in commands unrelated to what
was typed: "/sk" reached "/yolo" because its description contains those
letters, and the selection sat on it. Matching is now limited to the command
name, which is what the user is typing and the only part they can predict.

Author: elkaix

apps/vscode/test/slash-menu.test.ts

@@ -42,14 +42,15 @@ describe("scoreCommand", () => {
     expect(Number.isFinite(scoreCommand(RESEARCH_SKILL, "reswrit"))).toBe(true);
   });
 
-  it("still falls back to the description when nothing matches the name", () => {
-    expect(Number.isFinite(scoreCommand(CUSTOM_THEME, "color"))).toBe(true);
-  });
-
-  it("ranks every name match above a description match", () => {
-    expect(rank([CUSTOM_THEME, command("theme-picker", "Nothing to see.")], "theme")).toEqual([
-      "theme-picker",
-      "custom-theme",
+  it("never matches on the description", () => {
+    // "/sk" used to reach "/yolo" because its description contains those
+    // letters, so the highlight sat on an unrelated command.
+    const yolo = command("yolo", "Skip every approval prompt for this session.");
+    expect(scoreCommand(yolo, "sk")).toBe(NO_MATCH);
+    expect(rank([yolo, command("sub-skill"), command("invoke-skill")], "sk")).toEqual([
+      "sub-skill",
+      "invoke-skill",
     ]);
+    expect(scoreCommand(CUSTOM_THEME, "color")).toBe(NO_MATCH);
   });
 });

apps/vscode/webview-ui/src/components/inputarea/hooks/slash-command-match.ts

@@ -25,10 +25,12 @@ function matchesAnyWordPrefix(text: string, query: string): boolean {
 }
 
 /**
- * Lower is a better match, `NO_MATCH` means the command is filtered out. Every
- * way of matching the name outranks the description: the name is what the user
- * is typing, and matching a long description loosely lets nearly every command
- * through — which is what made the menu look unfiltered.
+ * Lower is a better match, `NO_MATCH` means the command is filtered out.
+ *
+ * Only the command name is matched. A description is prose — matching it pulls
+ * in commands that have nothing to do with what was typed (`/sk` reaching
+ * `/yolo` because its description happens to contain those letters), and the
+ * user cannot tell why they are listed.
  */
 export function scoreCommand(command: SlashCommandInfo, query: string): number {
   const name = command.name.toLowerCase();
@@ -45,14 +47,7 @@ export function scoreCommand(command: SlashCommandInfo, query: string): number {
     return 2;
   }
   // Forgiving tier: skipped letters and dropped separators still match.
-  if (isSubsequence(letters(name), letters(q))) {
-    return 3;
-  }
-  const description = command.description;
-  if (matchesAnyWordPrefix(description, q)) {
-    return 4;
-  }
-  return description.toLowerCase().includes(q) ? 5 : NO_MATCH;
+  return isSubsequence(letters(name), letters(q)) ? 3 : NO_MATCH;
 }
 
 /** Commands that match `query`, best match first. An empty query keeps the original order. */