test: command — hints parsing and altimate builtin completeness

Guard against regressions in Command.hints() template placeholder extraction
(used by every command for TUI argument hints) and ensure all altimate-specific
builtin commands (configure-claude, configure-codex, discover-and-add-mcps)
are registered — the latter would have caught the regression fixed in 528af75.

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

https://claude.ai/code/session_018jF7cHcSoARmZRt6Wb7m9o

Author: claude

packages/opencode/test/command/altimate-commands.test.ts

@@ -0,0 +1,64 @@
+import { describe, test, expect } from "bun:test"
+import { Command } from "../../src/command/index"
+import { Instance } from "../../src/project/instance"
+import { tmpdir } from "../fixture/fixture"
+
+async function withInstance(fn: () => Promise<void>) {
+  await using tmp = await tmpdir({ git: true })
+  await Instance.provide({ directory: tmp.path, fn })
+}
+
+describe("Altimate builtin commands", () => {
+  test("all altimate-specific commands are registered", async () => {
+    await withInstance(async () => {
+      const commands = await Command.list()
+      const names = commands.map((c) => c.name)
+      // These are the altimate_change commands that must ship with the package.
+      // Regression guard: commit 528af75 fixed discover-and-add-mcps not shipping.
+      expect(names).toContain("configure-claude")
+      expect(names).toContain("configure-codex")
+      expect(names).toContain("discover-and-add-mcps")
+      expect(names).toContain("feedback")
+    })
+  })
+
+  test("Command.Default includes all altimate constants", () => {
+    expect(Command.Default.CONFIGURE_CLAUDE).toBe("configure-claude")
+    expect(Command.Default.CONFIGURE_CODEX).toBe("configure-codex")
+    expect(Command.Default.DISCOVER_MCPS).toBe("discover-and-add-mcps")
+    expect(Command.Default.FEEDBACK).toBe("feedback")
+  })
+
+  test("discover-and-add-mcps has correct metadata and template", async () => {
+    await withInstance(async () => {
+      const cmd = await Command.get("discover-and-add-mcps")
+      expect(cmd).toBeDefined()
+      expect(cmd.name).toBe("discover-and-add-mcps")
+      expect(cmd.source).toBe("command")
+      expect(cmd.description).toBe("discover MCP servers from external AI tool configs and add them")
+      const template = await cmd.template
+      expect(template).toContain("mcp_discover")
+      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.name).toBe("configure-claude")
+      expect(cmd.source).toBe("command")
+      expect(cmd.description).toBe("configure /altimate command in Claude Code")
+    })
+  })
+
+  test("configure-codex has correct metadata", async () => {
+    await withInstance(async () => {
+      const cmd = await Command.get("configure-codex")
+      expect(cmd).toBeDefined()
+      expect(cmd.name).toBe("configure-codex")
+      expect(cmd.source).toBe("command")
+      expect(cmd.description).toBe("configure altimate skill in Codex CLI")
+    })
+  })
+})

packages/opencode/test/command/hints.test.ts

@@ -0,0 +1,32 @@
+import { describe, test, expect } from "bun:test"
+import { Command } from "../../src/command/index"
+
+describe("Command.hints — template placeholder extraction", () => {
+  test("extracts numbered placeholders in order", () => {
+    expect(Command.hints("Do $1 then $2")).toEqual(["$1", "$2"])
+  })
+
+  test("deduplicates repeated numbered placeholders", () => {
+    expect(Command.hints("Use $1 and $1 again")).toEqual(["$1"])
+  })
+
+  test("extracts $ARGUMENTS", () => {
+    expect(Command.hints("Run with $ARGUMENTS")).toEqual(["$ARGUMENTS"])
+  })
+
+  test("numbered placeholders before $ARGUMENTS", () => {
+    expect(Command.hints("Do $2 then $1 with $ARGUMENTS")).toEqual(["$1", "$2", "$ARGUMENTS"])
+  })
+
+  test("returns empty for no placeholders", () => {
+    expect(Command.hints("Plain text with no variables")).toEqual([])
+  })
+
+  test("handles template with only $ARGUMENTS", () => {
+    expect(Command.hints("If $ARGUMENTS contains --scope global")).toEqual(["$ARGUMENTS"])
+  })
+
+  test("only recognises $N and $ARGUMENTS — not $OTHER or $FOO", () => {
+    expect(Command.hints("Use $OTHER and $FOO")).toEqual([])
+  })
+})