feat: enhance filterSlashCommands to prioritize prefix matches and maintain input order

Author: qorzj

packages/cli/src/tests/slash-commands.test.ts

@@ -41,10 +41,54 @@ test("filterSlashCommands matches partial prefixes", () => {
   assert.deepEqual(matched, ["skill-writer", "skills"]);
 });
 
+test("filterSlashCommands ranks /model before /commondao for /mo", () => {
+  const items = buildSlashCommands([
+    { name: "commondao", path: "/skills/commondao/SKILL.md", description: "CommonDAO guide" },
+  ]);
+  assert.deepEqual(
+    filterSlashCommands(items, "/mo").map((item) => item.name),
+    ["model", "commondao"]
+  );
+});
+
+test("filterSlashCommands preserves order within match groups without mutating input", () => {
+  const items = buildSlashCommands(
+    ["commondao", "motion", "demo", "model-helper"].map((name) => ({
+      name,
+      path: `/skills/${name}/SKILL.md`,
+      description: name,
+    }))
+  );
+  const original = [...items];
+  assert.deepEqual(
+    filterSlashCommands(items, "/mo").map((item) => item.name),
+    ["motion", "model-helper", "model", "commondao", "demo"]
+  );
+  assert.deepEqual(items, original);
+});
+
+test("filterSlashCommands matches and prioritizes prefixes case-insensitively", () => {
+  const items = buildSlashCommands(
+    ["ComMonDAO", "Motion"].map((name) => ({
+      name,
+      path: `/skills/${name}/SKILL.md`,
+      description: name,
+    }))
+  );
+  assert.deepEqual(
+    filterSlashCommands(items, "/MO").map((item) => item.name),
+    ["Motion", "model", "ComMonDAO"]
+  );
+});
+
 test("filterSlashCommands returns all entries on bare slash", () => {
   const items = buildSlashCommands(skills);
   const matched = filterSlashCommands(items, "/");
-  assert.equal(matched.length, items.length);
+  assert.deepEqual(matched, items);
+});
+
+test("filterSlashCommands returns nothing when no names match", () => {
+  assert.deepEqual(filterSlashCommands(buildSlashCommands(skills), "/missing"), []);
 });
 
 test("filterSlashCommands returns nothing for non-slash tokens", () => {

packages/cli/src/ui/core/slash-commands.ts

@@ -119,7 +119,17 @@ export function filterSlashCommands(items: SlashCommandItem[], token: string): S
   if (!query) {
     return items;
   }
-  return items.filter((item) => item.name.toLowerCase().includes(query));
+  const prefixMatches: SlashCommandItem[] = [];
+  const otherMatches: SlashCommandItem[] = [];
+  for (const item of items) {
+    const name = item.name.toLowerCase();
+    if (name.startsWith(query)) {
+      prefixMatches.push(item);
+    } else if (name.includes(query)) {
+      otherMatches.push(item);
+    }
+  }
+  return [...prefixMatches, ...otherMatches];
 }
 
 export function findExactSlashCommand(items: SlashCommandItem[], token: string): SlashCommandItem | null {