@ngocsangairvds/vsaf 4.0.15 → 4.0.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngocsangairvds/vsaf",
-  "version": "4.0.15",
+  "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 {
@@ -224,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) ──
@@ -279,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 {
@@ -311,7 +311,7 @@ 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();
@@ -319,7 +319,21 @@ if (!verified) {
     verified = true;
   } catch (e) {
     log('❌', 'vds-cli not working — skills will be BLOCKED at runtime');
-    log('  ', `Error: ${e.message || e}`);
+    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}`);
+      }
+    }
   }
 }
 
@@ -329,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 │');