@ngocsangairvds/vsaf 4.0.11 → 4.0.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngocsangairvds/vsaf",
-  "version": "4.0.11",
+  "version": "4.0.14",
   "description": "is it ready to use ?",
   "main": "packages/core/dist/index.js",
   "types": "packages/core/dist/index.d.ts",
@@ -20,13 +20,14 @@
   "files": [
     "packages/core/dist",
     "packages/cli/dist",
+    "scripts",
     "workflows",
     "commands",
     "skills"
   ],
   "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{};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})}\"",
+    "postinstall": "node scripts/postinstall.js",
     "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/packages/cli/dist/commands/init.d.ts

@@ -1,6 +1,2 @@
-export interface InitOptions {
-    /** Skill pack to auto-install. Default: 'sdlc'. Pass false to skip. */
-    pack?: string | false;
-}
-export declare function execInit(projectPath: string, options?: InitOptions): Promise<void>;
+export declare function execInit(projectPath: string): Promise<void>;
 //# sourceMappingURL=init.d.ts.map

package/packages/cli/dist/commands/init.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"init.d.ts","sourceRoot":"","sources":["../../src/commands/init.ts"],"names":[],"mappings":"AAmBA,MAAM,WAAW,WAAW;IAC1B,uEAAuE;IACvE,IAAI,CAAC,EAAE,MAAM,GAAG,KAAK,CAAC;CACvB;AAED,wBAAsB,QAAQ,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC,CAoDxF"}
+{"version":3,"file":"init.d.ts","sourceRoot":"","sources":["../../src/commands/init.ts"],"names":[],"mappings":"AAiBA,wBAAsB,QAAQ,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAwCjE"}

package/packages/cli/dist/commands/init.js

@@ -3,8 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.execInit = execInit;
 const fs_1 = require("fs");
 const path_1 = require("path");
-const skill_js_1 = require("./skill.js");
-const install_js_1 = require("./install.js");
 const DEFAULT_CONFIG = `provider: claude
 model: sonnet
 isolation:
@@ -18,30 +16,16 @@ isolation:
     - .vsaf/storage
   max_workspaces: 5
 `;
-async function execInit(projectPath, options) {
+async function execInit(projectPath) {
     const vsafDir = (0, path_1.join)(projectPath, '.vsaf');
     if ((0, fs_1.existsSync)(vsafDir)) {
         throw new Error(`.vsaf/ already exists at ${projectPath}. Remove it first to reinitialize.`);
     }
+    // Create .vsaf/ directory structure
     (0, fs_1.mkdirSync)((0, path_1.join)(vsafDir, 'workflows'), { recursive: true });
     (0, fs_1.mkdirSync)((0, path_1.join)(vsafDir, 'commands'), { recursive: true });
     (0, fs_1.mkdirSync)((0, path_1.join)(vsafDir, 'storage'), { recursive: true });
     (0, fs_1.writeFileSync)((0, path_1.join)(vsafDir, 'config.yaml'), DEFAULT_CONFIG, 'utf-8');
-    // Auto-install skill pack (default: sdlc)
-    const packName = options?.pack !== false ? (options?.pack ?? 'sdlc') : null;
-    if (packName) {
-        const packDir = (0, skill_js_1.resolveSource)(packName);
-        const packInfo = (0, skill_js_1.resolvePackInfo)(packName, projectPath);
-        const skillsDir = (0, path_1.join)(projectPath, '.claude', 'skills');
-        (0, install_js_1.deploySkills)(packDir, packName, packInfo.skillNames, skillsDir, {});
-        if (packInfo.bundledNames.length > 0) {
-            (0, install_js_1.deploySkills)(packDir, packName, packInfo.bundledNames, skillsDir, { noPrefix: true });
-        }
-        const prefixedNames = packInfo.skillNames
-            .filter(n => n !== 'setup')
-            .map(n => `${packName}-${n}`);
-        (0, skill_js_1.skillInit)(packName, projectPath, prefixedNames, { workflows: packInfo.workflows });
-    }
     // Create .mcp.json for MCP server registration
     const mcpPath = (0, path_1.join)(projectPath, '.mcp.json');
     const vsafServer = { type: 'stdio', command: 'vsaf', args: ['mcp'] };
@@ -73,5 +57,7 @@ async function execInit(projectPath, options) {
             (0, fs_1.writeFileSync)(mcpPath, JSON.stringify(existing, null, 2) + '\n', 'utf-8');
         }
     }
+    console.log(`\n[vsaf] Initialized .vsaf/ at ${projectPath}`);
+    console.log('[vsaf] Next: vsaf install sdlc  (deploy skills + dependencies)\n');
 }
 //# sourceMappingURL=init.js.map

package/packages/cli/dist/commands/init.js.map

@@ -1 +1 @@
-{"version":3,"file":"init.js","sourceRoot":"","sources":["../../src/commands/init.ts"],"names":[],"mappings":";;AAwBA,4BAoDC;AA5ED,2BAAwE;AACxE,+BAAqC;AACrC,yCAAuE;AACvE,6CAA4C;AAE5C,MAAM,cAAc,GAAG;;;;;;;;;;;;CAYtB,CAAC;AAOK,KAAK,UAAU,QAAQ,CAAC,WAAmB,EAAE,OAAqB;IACvE,MAAM,OAAO,GAAG,IAAA,WAAI,EAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IAE3C,IAAI,IAAA,eAAU,EAAC,OAAO,CAAC,EAAE,CAAC;QACxB,MAAM,IAAI,KAAK,CAAC,4BAA4B,WAAW,oCAAoC,CAAC,CAAC;IAC/F,CAAC;IAED,IAAA,cAAS,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,WAAW,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC3D,IAAA,cAAS,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,UAAU,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC1D,IAAA,cAAS,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACzD,IAAA,kBAAa,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,aAAa,CAAC,EAAE,cAAc,EAAE,OAAO,CAAC,CAAC;IAErE,0CAA0C;IAC1C,MAAM,QAAQ,GAAG,OAAO,EAAE,IAAI,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,IAAI,IAAI,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;IAC5E,IAAI,QAAQ,EAAE,CAAC;QACb,MAAM,OAAO,GAAG,IAAA,wBAAa,EAAC,QAAQ,CAAC,CAAC;QACxC,MAAM,QAAQ,GAAG,IAAA,0BAAe,EAAC,QAAQ,EAAE,WAAW,CAAC,CAAC;QACxD,MAAM,SAAS,GAAG,IAAA,WAAI,EAAC,WAAW,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;QACzD,IAAA,yBAAY,EAAC,OAAO,EAAE,QAAQ,EAAE,QAAQ,CAAC,UAAU,EAAE,SAAS,EAAE,EAAE,CAAC,CAAC;QACpE,IAAI,QAAQ,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrC,IAAA,yBAAY,EAAC,OAAO,EAAE,QAAQ,EAAE,QAAQ,CAAC,YAAY,EAAE,SAAS,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;QACxF,CAAC;QACD,MAAM,aAAa,GAAG,QAAQ,CAAC,UAAU;aACtC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,KAAK,OAAO,CAAC;aAC1B,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,GAAG,QAAQ,IAAI,CAAC,EAAE,CAAC,CAAC;QAChC,IAAA,oBAAS,EAAC,QAAQ,EAAE,WAAW,EAAE,aAAa,EAAE,EAAE,SAAS,EAAE,QAAQ,CAAC,SAAS,EAAE,CAAC,CAAC;IACrF,CAAC;IAED,+CAA+C;IAC/C,MAAM,OAAO,GAAG,IAAA,WAAI,EAAC,WAAW,EAAE,WAAW,CAAC,CAAC;IAC/C,MAAM,UAAU,GAAG,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC;IACrE,MAAM,cAAc,GAAG,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,IAAI,EAAE,iBAAiB,EAAE,KAAK,CAAC,EAAE,CAAC;IAEjG,IAAI,CAAC,IAAA,eAAU,EAAC,OAAO,CAAC,EAAE,CAAC;QACzB,MAAM,SAAS,GAAG;YAChB,UAAU,EAAE;gBACV,IAAI,EAAE,UAAU;gBAChB,QAAQ,EAAE,cAAc;aACzB;SACF,CAAC;QACF,IAAA,kBAAa,EAAC,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,OAAO,CAAC,CAAC;IAC7E,CAAC;SAAM,CAAC;QACN,wDAAwD;QACxD,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,IAAA,iBAAY,EAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;QAC5D,IAAI,CAAC,QAAQ,CAAC,UAAU;YAAE,QAAQ,CAAC,UAAU,GAAG,EAAE,CAAC;QACnD,IAAI,OAAO,GAAG,KAAK,CAAC;QACpB,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,EAAE,CAAC;YAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,GAAG,UAAU,CAAC;YAAC,OAAO,GAAG,IAAI,CAAC;QAAC,CAAC;QACzF,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,EAAE,CAAC;YAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,GAAG,cAAc,CAAC;YAAC,OAAO,GAAG,IAAI,CAAC;QAAC,CAAC;QACrG,IAAI,OAAO,EAAE,CAAC;YACZ,IAAA,kBAAa,EAAC,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,OAAO,CAAC,CAAC;QAC5E,CAAC;IACH,CAAC;AACH,CAAC"}
+{"version":3,"file":"init.js","sourceRoot":"","sources":["../../src/commands/init.ts"],"names":[],"mappings":";;AAiBA,4BAwCC;AAzDD,2BAAwE;AACxE,+BAA4B;AAE5B,MAAM,cAAc,GAAG;;;;;;;;;;;;CAYtB,CAAC;AAEK,KAAK,UAAU,QAAQ,CAAC,WAAmB;IAChD,MAAM,OAAO,GAAG,IAAA,WAAI,EAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IAE3C,IAAI,IAAA,eAAU,EAAC,OAAO,CAAC,EAAE,CAAC;QACxB,MAAM,IAAI,KAAK,CAAC,4BAA4B,WAAW,oCAAoC,CAAC,CAAC;IAC/F,CAAC;IAED,oCAAoC;IACpC,IAAA,cAAS,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,WAAW,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC3D,IAAA,cAAS,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,UAAU,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC1D,IAAA,cAAS,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACzD,IAAA,kBAAa,EAAC,IAAA,WAAI,EAAC,OAAO,EAAE,aAAa,CAAC,EAAE,cAAc,EAAE,OAAO,CAAC,CAAC;IAErE,+CAA+C;IAC/C,MAAM,OAAO,GAAG,IAAA,WAAI,EAAC,WAAW,EAAE,WAAW,CAAC,CAAC;IAC/C,MAAM,UAAU,GAAG,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC;IACrE,MAAM,cAAc,GAAG,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,IAAI,EAAE,iBAAiB,EAAE,KAAK,CAAC,EAAE,CAAC;IAEjG,IAAI,CAAC,IAAA,eAAU,EAAC,OAAO,CAAC,EAAE,CAAC;QACzB,MAAM,SAAS,GAAG;YAChB,UAAU,EAAE;gBACV,IAAI,EAAE,UAAU;gBAChB,QAAQ,EAAE,cAAc;aACzB;SACF,CAAC;QACF,IAAA,kBAAa,EAAC,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,OAAO,CAAC,CAAC;IAC7E,CAAC;SAAM,CAAC;QACN,wDAAwD;QACxD,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,IAAA,iBAAY,EAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;QAC5D,IAAI,CAAC,QAAQ,CAAC,UAAU;YAAE,QAAQ,CAAC,UAAU,GAAG,EAAE,CAAC;QACnD,IAAI,OAAO,GAAG,KAAK,CAAC;QACpB,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,EAAE,CAAC;YAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,GAAG,UAAU,CAAC;YAAC,OAAO,GAAG,IAAI,CAAC;QAAC,CAAC;QACzF,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,EAAE,CAAC;YAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,GAAG,cAAc,CAAC;YAAC,OAAO,GAAG,IAAI,CAAC;QAAC,CAAC;QACrG,IAAI,OAAO,EAAE,CAAC;YACZ,IAAA,kBAAa,EAAC,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,OAAO,CAAC,CAAC;QAC5E,CAAC;IACH,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,kCAAkC,WAAW,EAAE,CAAC,CAAC;IAC7D,OAAO,CAAC,GAAG,CAAC,kEAAkE,CAAC,CAAC;AAClF,CAAC"}

package/scripts/postinstall.js

@@ -0,0 +1,73 @@
+#!/usr/bin/env node
+
+/**
+ * Postinstall script — create @vsaf/core workspace link
+ *
+ * Monorepo workspace links are stripped during `npm pack` (prepack removes
+ * "workspaces" from package.json). This script recreates the link so that
+ * `require('@vsaf/core')` works after `npm install -g`.
+ *
+ * Strategy (in order):
+ *   1. symlink (macOS/Linux) or junction (Windows — no admin needed)
+ *   2. cpSync fallback (copy dist/ contents)
+ *   3. Fail with clear instructions
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+const cwd = process.cwd();
+const nsDir = path.resolve(cwd, 'node_modules', '@vsaf');
+const target = path.resolve(nsDir, 'core');
+const source = path.resolve(cwd, 'packages', 'core', 'dist');
+
+// Skip if source doesn't exist (e.g. fresh clone before build)
+if (!fs.existsSync(source)) {
+  process.exit(0);
+}
+
+// Ensure @vsaf/ namespace directory
+fs.mkdirSync(nsDir, { recursive: true });
+
+// Clean existing link/dir
+try { fs.rmSync(target, { recursive: true, force: true }); } catch {}
+
+// Attempt 1: symlink (dir on Unix, junction on Windows)
+try {
+  const type = process.platform === 'win32' ? 'junction' : 'dir';
+  fs.symlinkSync(source, target, type);
+  process.exit(0);
+} catch (e1) {
+  // Attempt 2: copy
+  try {
+    fs.cpSync(source, target, { recursive: true });
+    process.exit(0);
+  } catch (e2) {
+    // Both failed — show clear instructions
+    console.error('\n' + '='.repeat(60));
+    console.error('  VSAF postinstall failed: cannot link @vsaf/core');
+    console.error('='.repeat(60));
+    console.error(`\n  Symlink error: ${e1.message}`);
+    console.error(`  Copy error:    ${e2.message}`);
+
+    if (process.platform === 'win32') {
+      console.error('\n  Windows fix (choose one):');
+      console.error('  1. Enable Developer Mode:');
+      console.error('     Settings > Update & Security > For Developers > Developer Mode ON');
+      console.error('     Then re-run: npm install -g @ngocsangairvds/vsaf');
+      console.error('');
+      console.error('  2. Run as Administrator:');
+      console.error('     Right-click terminal > "Run as administrator"');
+      console.error('     Then re-run: npm install -g @ngocsangairvds/vsaf');
+      console.error('');
+      console.error('  3. Manual copy:');
+      console.error(`     xcopy /E /I "${source}" "${target}"`);
+    } else {
+      console.error('\n  Fix: check file permissions on node_modules/');
+      console.error(`  Manual: cp -r "${source}" "${target}"`);
+    }
+
+    console.error('\n' + '='.repeat(60) + '\n');
+    process.exit(1);
+  }
+}

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,49 @@ 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})`);
+      if (process.platform === 'win32') {
+        log('  ', 'Fix: open cmd as Administrator, then run:');
+        log('  ', `  del "${shimPath}"`);
+        log('  ', '  vsaf install vds-skill');
+      } else {
+        log('  ', `Fix: rm "${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');
   }
 }