refactor: use qsv --update-mcp-skill instead of separate qsv-skill-gen binary

jqnatividad · jqnatividad · commit 13380ba10374 · 2026-01-07T11:01:21.000-05:00
diff --git a/.claude/skills/src/mcp-server.ts b/.claude/skills/src/mcp-server.ts
@@ -127,7 +127,7 @@ class QsvMcpServer {
           console.error('');
         } else {
           console.error('ℹ️  To update skills manually, run:');
-          console.error('   cargo run --bin qsv-skill-gen --features all_features');
+          console.error('   qsv --update-mcp-skills');
           console.error('   Then restart the MCP server');
           console.error('');
         }
diff --git a/.claude/skills/src/update-checker.ts b/.claude/skills/src/update-checker.ts
@@ -228,7 +228,7 @@ export class UpdateChecker {
           `⚠️  qsv binary (${currentQsvVersion}) is newer than skills (${skillsVersion})`
         );
         recommendations.push(
-          `   Run: cargo run --bin qsv-skill-gen --features all_features`
+          `   Run: qsv --update-mcp-skills`
         );
         recommendations.push(
           `   Then restart the MCP server`
@@ -291,14 +291,12 @@ export class UpdateChecker {
     console.error('[UpdateChecker] Auto-regenerating skills...');
 
     return new Promise((resolve) => {
-      // Find the qsv repository root (should be ../.. from .claude/skills)
-      const repoRoot = join(this.skillsDir, '../..');
-
+      // Use qsv binary directly with --update-mcp-skills flag
+      // This is much simpler and doesn't require Rust toolchain
       const child = spawn(
-        'cargo',
-        ['run', '--bin', 'qsv-skill-gen', '--features', 'all_features'],
+        this.qsvBinaryPath,
+        ['--update-mcp-skills'],
         {
-          cwd: repoRoot,
           stdio: 'inherit'
         }
       );
@@ -314,7 +312,7 @@ export class UpdateChecker {
       });
 
       child.on('error', (error) => {
-        console.error('[UpdateChecker] ❌ Failed to spawn cargo:', error);
+        console.error('[UpdateChecker] ❌ Failed to spawn qsv:', error);
         resolve(false);
       });
     });

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ class QsvMcpServer {`
`127`	`127`	`console.error('');`
`128`	`128`	`} else {`
`129`	`129`	`console.error('ℹ️ To update skills manually, run:');`
`130`		`- console.error(' cargo run --bin qsv-skill-gen --features all_features');`
	`130`	`+ console.error(' qsv --update-mcp-skills');`
`131`	`131`	`console.error(' Then restart the MCP server');`
`132`	`132`	`console.error('');`
`133`	`133`	`}`