@ngocsangairvds/vsaf 4.0.10 → 4.0.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngocsangairvds/vsaf",
-  "version": "4.0.10",
+  "version": "4.0.12",
   "description": "is it ready to use ?",
   "main": "packages/core/dist/index.js",
   "types": "packages/core/dist/index.d.ts",
@@ -26,7 +26,7 @@
   ],
   "scripts": {
     "build": "npm run build --workspaces",
-    "postinstall": "node -e \"const fs=require('fs'),p=require('path'),r=p.resolve;const d=r(process.cwd(),'node_modules','@vsaf');fs.mkdirSync(d,{recursive:true});const t=r(d,'core');try{fs.rmSync(t,{recursive:true,force:true})}catch{};fs.symlinkSync(r(process.cwd(),'packages','core','dist'),t,'dir')\"",
+    "postinstall": "node -e \"const fs=require('fs'),p=require('path'),r=p.resolve;const d=r(process.cwd(),'node_modules','@vsaf');fs.mkdirSync(d,{recursive:true});const t=r(d,'core');try{fs.rmSync(t,{recursive:true,force:true})}catch{};const s=r(process.cwd(),'packages','core','dist');try{fs.symlinkSync(s,t,process.platform==='win32'?'junction':'dir')}catch{fs.cpSync(s,t,{recursive:true})}\"",
     "prepack": "cp package.json package.json.bak && node -e \"const f='package.json',j=JSON.parse(require('fs').readFileSync(f));delete j.workspaces;require('fs').writeFileSync(f,JSON.stringify(j,null,2)+'\\n')\"",
     "postpack": "mv package.json.bak package.json",
     "test": "vitest run",

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

@@ -10,8 +10,8 @@
  *   node install-deps.mjs [projectPath]
  */
 
-import { existsSync, readFileSync, writeFileSync, mkdirSync, cpSync, chmodSync } from 'fs';
-import { join, dirname } from 'path';
+import { existsSync, readFileSync, writeFileSync, mkdirSync, cpSync, chmodSync, unlinkSync } from 'fs';
+import { join, dirname, delimiter } from 'path';
 import { homedir } from 'os';
 
 const projectPath = process.argv[2] || process.cwd();
@@ -227,19 +227,37 @@ if (!vdsCliFound) {
 if (vdsScriptsDir) {
   log('📂', `vds-scripts: ${vdsScriptsDir}`);
   if (existsSync(join(vdsScriptsDir, 'pyproject.toml'))) {
-    console.log('');
-    log('🔄', 'Syncing vds-scripts packages (uv sync --all-packages)...');
+    // Check uv is available before attempting sync
+    let uvAvailable = false;
     try {
-      execSync('uv sync --all-packages', {
-        stdio: 'pipe',
-        encoding: 'utf-8',
-        timeout: 120000,
-        cwd: vdsScriptsDir,
-      });
-      log('✅', 'All vds-scripts packages synced');
+      execSync('uv --version', { stdio: 'pipe', encoding: 'utf-8', timeout: 10000 });
+      uvAvailable = true;
     } catch {
-      log('⚠️', 'Failed to sync vds-scripts packages — subcommands like confluence/jira may not work');
-      log('  ', `Fix: cd ${vdsScriptsDir} && uv sync --all-packages`);
+      log('⚠️', 'uv (Python package manager) not found — required for vds-scripts');
+      if (process.platform === 'win32') {
+        log('  ', 'Install: powershell -c "irm https://astral.sh/uv/install.ps1 | iex"');
+        log('  ', '  or:    winget install astral-sh.uv');
+      } else {
+        log('  ', 'Install: curl -LsSf https://astral.sh/uv/install.sh | sh');
+        log('  ', '  or:    brew install uv');
+      }
+    }
+
+    if (uvAvailable) {
+      console.log('');
+      log('🔄', 'Syncing vds-scripts packages (uv sync --all-packages)...');
+      try {
+        execSync('uv sync --all-packages', {
+          stdio: 'pipe',
+          encoding: 'utf-8',
+          timeout: 120000,
+          cwd: vdsScriptsDir,
+        });
+        log('✅', 'All vds-scripts packages synced');
+      } catch {
+        log('⚠️', 'Failed to sync vds-scripts packages — subcommands like confluence/jira may not work');
+        log('  ', `Fix: cd ${vdsScriptsDir} && uv sync --all-packages`);
+      }
     }
   }
 } else {
@@ -252,31 +270,43 @@ console.log('');
 log('🔍', 'Verifying vds-cli...');
 
 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();
   log('✅', `vds-cli ${ver} — working`);
   verified = true;
 } catch {
-  // Try with project-local wrapper in PATH
+  // vds-cli on PATH failed — check if a broken shim exists and auto-remove it
+  try {
+    const whichCmd = process.platform === 'win32' ? 'where' : 'which';
+    const shimPath = execSync(`${whichCmd} vds-cli`, { stdio: 'pipe', encoding: 'utf-8' }).trim().split('\n')[0];
+    // Shim exists but doesn't work — remove it automatically
+    log('🗑️', `Removing broken shim: ${shimPath}`);
+    try {
+      unlinkSync(shimPath);
+      log('✅', 'Broken shim removed');
+    } catch (e) {
+      log('⚠️', `Cannot auto-remove shim (${e.code || e.message})`);
+      log('  ', `Manual fix: del "${shimPath}"  then re-run: vsaf install vds-skill`);
+    }
+  } catch {
+    // No shim found — that's fine
+  }
+}
+
+// Second pass: try project-local wrapper
+if (!verified) {
   try {
     const wrapperPath = join(projectPath, '.claude', 'bin');
     const ver = execSync('vds-cli --version', {
       stdio: 'pipe', encoding: 'utf-8', timeout: 15000,
-      env: { ...process.env, PATH: `${wrapperPath}:${process.env.PATH}` },
+      env: { ...process.env, PATH: `${wrapperPath}${delimiter}${process.env.PATH}` },
     }).trim();
     log('✅', `vds-cli ${ver} — working (via .claude/bin/vds-cli)`);
     verified = true;
   } catch {
-    // Check if a shim exists but is broken
-    try {
-      const whichCmd = process.platform === 'win32' ? 'where' : 'which';
-      const shimPath = execSync(`${whichCmd} vds-cli`, { stdio: 'pipe', encoding: 'utf-8' }).trim().split('\n')[0];
-      log('❌', `vds-cli BROKEN — ${shimPath} exists but fails to run`);
-      log('  ', 'The shim likely points to a deleted venv or missing vds-scripts directory.');
-      log('  ', `Fix: rm ${shimPath}  then re-run: vsaf install vds-skill`);
-    } catch {
-      log('❌', 'vds-cli not found');
-    }
+    log('❌', 'vds-cli not working — skills will be BLOCKED at runtime');
   }
 }