@ngocsangairvds/vsaf 4.0.14 → 4.0.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngocsangairvds/vsaf",
-  "version": "4.0.14",
+  "version": "4.0.16",
   "description": "is it ready to use ?",
   "main": "packages/core/dist/index.js",
   "types": "packages/core/dist/index.d.ts",

package/skills/vds-skill/install-deps.mjs

@@ -179,7 +179,7 @@ let vdsCliFound = false;
 
 // Check if vds-cli is already on PATH
 try {
-  const version = execSync('vds-cli --version', { stdio: 'pipe', encoding: 'utf-8' }).trim();
+  const version = execSync('vds-cli version', { stdio: 'pipe', encoding: 'utf-8' }).trim();
   log('✅', `vds-cli ${version} (on PATH)`);
   vdsCliFound = true;
 } catch {
@@ -197,7 +197,7 @@ if (!vdsCliFound && vdsScriptsDir) {
 // Project-local vds-cli wrapper — created by vsaf install vds-skill
 const { execFileSync } = require('child_process');
 try {
-  execFileSync('uv', ['run', '--directory', ${JSON.stringify(vdsScriptsDir)}, '--package', 'vds-cli', 'vds-cli', ...process.argv.slice(2)], { stdio: 'inherit' });
+  execFileSync('uv', ['run', '--directory', ${JSON.stringify(vdsScriptsDir)}, '--package', 'vds-cli', 'vds-cli', ...process.argv.slice(2)], { stdio: 'inherit', shell: true });
 } catch (e) {
   process.exit(e.status || 1);
 }
@@ -210,7 +210,12 @@ try {
     }
     log('✅', `vds-cli wrapper created: ${wrapperPath}`);
     log('  ', `Points to: ${vdsScriptsDir}`);
-    log('💡', 'Add to PATH:  export PATH=".claude/bin:$PATH"');
+    if (process.platform === 'win32') {
+      log('💡', 'Add to PATH:  set "PATH=.claude\\bin;%PATH%"');
+      log('  ', 'PowerShell:   $env:PATH = ".claude\\bin;$env:PATH"');
+    } else {
+      log('💡', 'Add to PATH:  export PATH=".claude/bin:$PATH"');
+    }
     vdsCliFound = true;
   }
 }
@@ -219,7 +224,7 @@ if (!vdsCliFound) {
   log('⚠️', 'vds-cli not found — required for non-dry-run execution');
   log('  ', 'Option 1: Clone vds-scripts into .claude/vds-scripts/ then re-run install');
   log('  ', 'Option 2: Install vds-cli globally (pip install / Viettel internal)');
-  log('  ', 'Verify:   vds-cli --version');
+  log('  ', 'Verify:   vds-cli version');
 }
 
 // ── Step 5: Sync all vds-scripts packages (ensures subcommand binaries exist) ──
@@ -254,8 +259,9 @@ if (vdsScriptsDir) {
           cwd: vdsScriptsDir,
         });
         log('✅', 'All vds-scripts packages synced');
-      } catch {
+      } catch (e) {
         log('⚠️', 'Failed to sync vds-scripts packages — subcommands like confluence/jira may not work');
+        if (e.stderr) log('  ', `Error: ${e.stderr.trim().split('\n')[0]}`);
         log('  ', `Fix: cd ${vdsScriptsDir} && uv sync --all-packages`);
       }
     }
@@ -273,7 +279,7 @@ let verified = false;
 
 // First pass: try vds-cli on PATH
 try {
-  const ver = execSync('vds-cli --version', { stdio: 'pipe', encoding: 'utf-8', timeout: 15000 }).trim();
+  const ver = execSync('vds-cli version', { stdio: 'pipe', encoding: 'utf-8', timeout: 15000 }).trim();
   log('✅', `vds-cli ${ver} — working`);
   verified = true;
 } catch {
@@ -305,14 +311,29 @@ try {
 if (!verified) {
   try {
     const wrapperPath = join(projectPath, '.claude', 'bin');
-    const ver = execSync('vds-cli --version', {
+    const ver = execSync('vds-cli version', {
       stdio: 'pipe', encoding: 'utf-8', timeout: 15000,
       env: { ...process.env, PATH: `${wrapperPath}${delimiter}${process.env.PATH}` },
     }).trim();
     log('✅', `vds-cli ${ver} — working (via .claude/bin/vds-cli)`);
     verified = true;
-  } catch {
+  } catch (e) {
     log('❌', 'vds-cli not working — skills will be BLOCKED at runtime');
+    if (e.stderr) log('  ', `stderr: ${e.stderr.toString().trim().split('\n').slice(0, 3).join('\n         ')}`);
+    else log('  ', `Error: ${e.message || e}`);
+
+    // Diagnostic: test uv run directly to isolate the issue
+    if (vdsScriptsDir) {
+      try {
+        const uvVer = execSync(`uv run --directory ${JSON.stringify(vdsScriptsDir)} --package vds-cli vds-cli version`, {
+          stdio: 'pipe', encoding: 'utf-8', timeout: 15000,
+        }).trim();
+        log('  ', `Direct uv run works: ${uvVer}`);
+        log('  ', 'Issue is with the wrapper script, not uv/vds-cli');
+      } catch (uvErr) {
+        log('  ', `Direct uv run also fails: ${uvErr.stderr ? uvErr.stderr.toString().trim().split('\n')[0] : uvErr.message}`);
+      }
+    }
   }
 }
 
@@ -322,7 +343,7 @@ console.log('');
 console.log('  ┌──────────────────────────────────────────────────────┐');
 console.log('  │  Verify commands (run these to confirm setup):       │');
 console.log('  ├──────────────────────────────────────────────────────┤');
-console.log('  │  vds-cli --version          # CLI is executable     │');
+console.log('  │  vds-cli version            # CLI is executable     │');
 console.log('  │  vds-cli confluence --help   # Confluence subcommand │');
 console.log('  │  vds-cli jira --help         # Jira subcommand      │');
 console.log('  │  vds-cli bitbucket --help    # Bitbucket subcommand │');