From cd5486a26abfaaf75d1369005ccf5bbf6bdf4232 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 24 Mar 2026 06:10:27 +0000
Subject: [PATCH] =?UTF-8?q?test:=20command=20=E2=80=94=20hints=20parser=20?=
 =?UTF-8?q?and=20altimate=20builtin=20registration?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Cover Command.hints() template placeholder extraction (7 tests) and verify
altimate-specific builtin commands (configure-claude, configure-codex,
discover-and-add-mcps) are properly registered with correct metadata.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

https://claude.ai/code/session_01LBpRnh2zYViskgkbqiTG8v
---
 .../test/command/command-resilience.test.ts   | 43 +++++++++++++++++++
 packages/opencode/test/command/hints.test.ts  | 34 +++++++++++++++
 2 files changed, 77 insertions(+)
 create mode 100644 packages/opencode/test/command/hints.test.ts

diff --git a/packages/opencode/test/command/command-resilience.test.ts b/packages/opencode/test/command/command-resilience.test.ts
index 6c01fbe1b..c54b64fee 100644
--- a/packages/opencode/test/command/command-resilience.test.ts
+++ b/packages/opencode/test/command/command-resilience.test.ts
@@ -90,6 +90,49 @@ describe("Command module", () => {
         expect(cmd).toBeUndefined()
       })
     })
+
+    test("altimate builtin commands are registered", async () => {
+      await withInstance(async () => {
+        const commands = await Command.list()
+        const names = commands.map((c) => c.name)
+        expect(names).toContain("configure-claude")
+        expect(names).toContain("configure-codex")
+        expect(names).toContain("discover-and-add-mcps")
+      })
+    })
+
+    test("discover-and-add-mcps has correct metadata", async () => {
+      await withInstance(async () => {
+        const cmd = await Command.get("discover-and-add-mcps")
+        expect(cmd).toBeDefined()
+        expect(cmd.source).toBe("command")
+        expect(cmd.description).toBe("discover MCP servers from external AI tool configs and add them")
+      })
+    })
+
+    test("discover-and-add-mcps template references mcp_discover tool", async () => {
+      await withInstance(async () => {
+        const cmd = await Command.get("discover-and-add-mcps")
+        const template = await cmd.template
+        expect(template).toContain("mcp_discover")
+      })
+    })
+
+    test("discover-and-add-mcps has $ARGUMENTS hint", async () => {
+      await withInstance(async () => {
+        const cmd = await Command.get("discover-and-add-mcps")
+        expect(cmd.hints).toContain("$ARGUMENTS")
+      })
+    })
+
+    test("configure-claude has correct metadata", async () => {
+      await withInstance(async () => {
+        const cmd = await Command.get("configure-claude")
+        expect(cmd).toBeDefined()
+        expect(cmd.source).toBe("command")
+        expect(cmd.description).toBe("configure /altimate command in Claude Code")
+      })
+    })
   })
 
   describe("user-defined commands from config", () => {
diff --git a/packages/opencode/test/command/hints.test.ts b/packages/opencode/test/command/hints.test.ts
new file mode 100644
index 000000000..b62d5a508
--- /dev/null
+++ b/packages/opencode/test/command/hints.test.ts
@@ -0,0 +1,34 @@
+import { describe, test, expect } from "bun:test"
+import { Command } from "../../src/command/index"
+
+describe("Command.hints: template placeholder extraction", () => {
+  test("extracts numbered placeholders in sorted order", () => {
+    expect(Command.hints("Do $2 then $1")).toEqual(["$1", "$2"])
+  })
+
+  test("deduplicates repeated placeholders", () => {
+    expect(Command.hints("Use $1 and $1 again")).toEqual(["$1"])
+  })
+
+  test("extracts $ARGUMENTS", () => {
+    expect(Command.hints("Run with $ARGUMENTS")).toEqual(["$ARGUMENTS"])
+  })
+
+  test("extracts both numbered and $ARGUMENTS", () => {
+    expect(Command.hints("Do $1 with $ARGUMENTS")).toEqual(["$1", "$ARGUMENTS"])
+  })
+
+  test("returns empty for templates with no placeholders", () => {
+    expect(Command.hints("Just a plain prompt")).toEqual([])
+  })
+
+  test("does not extract $ARGUMENTS as numbered", () => {
+    // $ARGUMENTS should only appear via the explicit check, not the numbered regex
+    expect(Command.hints("$ARGUMENTS only")).toEqual(["$ARGUMENTS"])
+  })
+
+  test("handles double-digit placeholders with lexicographic sort", () => {
+    // Note: sort is lexicographic, so $10 < $2. This documents current behavior.
+    expect(Command.hints("$10 then $2 then $1")).toEqual(["$1", "$10", "$2"])
+  })
+})