fix(registry): dedupe getAllCommands so aliases are not emitted twice

Aliases register the same Command object under more than one path
('speech generate' -> speechSynthesize, 'search web' -> searchQuery), so the
traversal in getAllCommands returned it once per path.

config export-schema maps that list straight to tool schemas, so it emitted 19
entries with only 17 distinct names — mmx_speech_synthesize and mmx_search_query
each appeared twice. Duplicate function names are rejected by both the Anthropic
and OpenAI tool APIs, so the exported schema could not be used as-is.

Dedupe by object identity: callers of getAllCommands want the set of distinct
commands, not the set of invocation paths. Alias routing is unaffected.

Author: shoemoney

src/registry.ts

@@ -58,9 +58,17 @@ class CommandRegistry {
   }
 
   getAllCommands(): Command[] {
+    // Aliases register the same Command object under more than one path
+    // ('speech generate' -> speechSynthesize, 'search web' -> searchQuery), so
+    // a plain traversal yields it once per alias. Dedupe by identity: callers
+    // want the set of distinct commands, not the set of invocation paths.
+    const seen = new Set<Command>();
     const commands: Command[] = [];
     const traverse = (node: CommandNode) => {
-      if (node.command) commands.push(node.command);
+      if (node.command && !seen.has(node.command)) {
+        seen.add(node.command);
+        commands.push(node.command);
+      }
       for (const child of node.children.values()) {
         traverse(child);
       }