@ngocsangairvds/vsaf 4.0.4 → 4.0.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngocsangairvds/vsaf",
-  "version": "4.0.4",
+  "version": "4.0.6",
   "description": "README",
   "main": "packages/core/dist/index.js",
   "types": "packages/core/dist/index.d.ts",

package/skills/vds-skill/create-bitbucket-pr/SKILL.md

@@ -14,8 +14,14 @@ Before doing anything, run this check via Bash tool:
 ```bash
 source .claude/skills/_shared/vds-skill/credentials.sh 2>/dev/null
 MISSING=""
-{ command -v vds-cli >/dev/null 2>&1 || [[ -x .claude/bin/vds-cli ]]; } || MISSING="$MISSING vds-cli"
 [[ -x .claude/bin/vds-cli ]] && export PATH=".claude/bin:$PATH"
+if ! vds-cli --version >/dev/null 2>&1; then
+  MISSING="$MISSING vds-cli"
+  VDS_PATH=$(command -v vds-cli 2>/dev/null)
+  if [[ -n "$VDS_PATH" ]]; then
+    echo "NOTE: $VDS_PATH exists but fails to run (broken shim or missing venv)"
+  fi
+fi
 [[ -z "${VDS_BITBUCKET_TOKEN:-}" ]] && MISSING="$MISSING VDS_BITBUCKET_TOKEN"
 if [[ -n "$MISSING" ]]; then
   echo "BLOCKED — missing:$MISSING"

package/skills/vds-skill/create-jira-epic/SKILL.md

@@ -14,8 +14,14 @@ Before doing anything, run this check via Bash tool:
 ```bash
 source .claude/skills/_shared/vds-skill/credentials.sh 2>/dev/null
 MISSING=""
-{ command -v vds-cli >/dev/null 2>&1 || [[ -x .claude/bin/vds-cli ]]; } || MISSING="$MISSING vds-cli"
 [[ -x .claude/bin/vds-cli ]] && export PATH=".claude/bin:$PATH"
+if ! vds-cli --version >/dev/null 2>&1; then
+  MISSING="$MISSING vds-cli"
+  VDS_PATH=$(command -v vds-cli 2>/dev/null)
+  if [[ -n "$VDS_PATH" ]]; then
+    echo "NOTE: $VDS_PATH exists but fails to run (broken shim or missing venv)"
+  fi
+fi
 [[ -z "${VDS_JIRA_TOKEN:-}" ]] && MISSING="$MISSING VDS_JIRA_TOKEN"
 [[ -z "${VDS_JIRA_PROJECT_DEFAULT:-}" ]] && MISSING="$MISSING VDS_JIRA_PROJECT_DEFAULT"
 if [[ -n "$MISSING" ]]; then

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

@@ -193,10 +193,91 @@ if (!vdsCliFound) {
   log('  ', 'Verify:   command -v vds-cli && vds-cli --version');
 }
 
+// ── Step 5: Sync all vds-scripts packages (ensures subcommand binaries exist) ──
+
 if (vdsScriptsDir) {
   log('📂', `vds-scripts: ${vdsScriptsDir}`);
+  const uvRunner = join(vdsScriptsDir, 'scripts', 'worktree_uv.sh');
+  if (existsSync(uvRunner)) {
+    console.log('');
+    log('🔄', 'Syncing vds-scripts packages (uv sync --all-packages)...');
+    try {
+      execSync(`"${uvRunner}" sync --directory "${vdsScriptsDir}" --all-packages`, {
+        stdio: 'pipe',
+        encoding: 'utf-8',
+        timeout: 120000,
+        cwd: vdsScriptsDir,
+      });
+      log('✅', 'All vds-scripts packages synced');
+    } catch (e) {
+      // Fallback: try direct uv sync
+      try {
+        execSync('uv sync --all-packages', {
+          stdio: 'pipe',
+          encoding: 'utf-8',
+          timeout: 120000,
+          cwd: vdsScriptsDir,
+        });
+        log('✅', 'All vds-scripts packages synced (direct uv)');
+      } catch {
+        log('⚠️', 'Failed to sync vds-scripts packages — subcommands like confluence/jira may not work');
+        log('  ', `Fix: cd ${vdsScriptsDir} && uv sync --all-packages`);
+      }
+    }
+  }
 } else {
   log('⚠️', 'vds-scripts not found at .claude/vds-scripts/ or ~/.claude/vds-scripts/');
 }
 
+// ── Step 6: Verify vds-cli actually works (catch broken shims) ──
+
+console.log('');
+log('🔍', 'Verifying vds-cli...');
+
+let verified = false;
+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
+  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}` },
+    }).trim();
+    log('✅', `vds-cli ${ver} — working (via .claude/bin/vds-cli)`);
+    verified = true;
+  } catch {
+    // Check if a shim exists but is broken
+    try {
+      const shimPath = execSync('command -v vds-cli', { stdio: 'pipe', encoding: 'utf-8' }).trim();
+      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');
+    }
+  }
+}
+
+// ── Step 7: Print verification commands for user ──
+
+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 confluence --help   # Confluence subcommand │');
+console.log('  │  vds-cli jira --help         # Jira subcommand      │');
+console.log('  │  vds-cli bitbucket --help    # Bitbucket subcommand │');
+console.log('  └──────────────────────────────────────────────────────┘');
+
+if (!verified) {
+  console.log('');
+  log('⚠️', 'vds-cli is NOT working — skills will be BLOCKED at runtime.');
+  log('  ', 'Fix vds-cli first, then run the verify commands above.');
+}
+
 console.log('\n[vds-skill] Setup complete ✅\n');

package/skills/vds-skill/push-prd/SKILL.md

@@ -19,8 +19,15 @@ Before doing anything, run this check via Bash tool:
 ```bash
 source .claude/skills/_shared/vds-skill/credentials.sh 2>/dev/null
 MISSING=""
-{ command -v vds-cli >/dev/null 2>&1 || [[ -x .claude/bin/vds-cli ]]; } || MISSING="$MISSING vds-cli"
 [[ -x .claude/bin/vds-cli ]] && export PATH=".claude/bin:$PATH"
+if ! vds-cli --version >/dev/null 2>&1; then
+  MISSING="$MISSING vds-cli"
+  # Show why: broken shim vs truly missing
+  VDS_PATH=$(command -v vds-cli 2>/dev/null)
+  if [[ -n "$VDS_PATH" ]]; then
+    echo "NOTE: $VDS_PATH exists but fails to run (broken shim or missing venv)"
+  fi
+fi
 [[ -z "${VDS_CONFLUENCE_TOKEN:-}" ]] && MISSING="$MISSING VDS_CONFLUENCE_TOKEN"
 [[ -z "${VDS_CONFLUENCE_SPACE_DEFAULT:-}" ]] && MISSING="$MISSING VDS_CONFLUENCE_SPACE_DEFAULT"
 if [[ -n "$MISSING" ]]; then

package/skills/vds-skill/push-srs/SKILL.md

@@ -19,8 +19,14 @@ Before doing anything, run this check via Bash tool:
 ```bash
 source .claude/skills/_shared/vds-skill/credentials.sh 2>/dev/null
 MISSING=""
-{ command -v vds-cli >/dev/null 2>&1 || [[ -x .claude/bin/vds-cli ]]; } || MISSING="$MISSING vds-cli"
 [[ -x .claude/bin/vds-cli ]] && export PATH=".claude/bin:$PATH"
+if ! vds-cli --version >/dev/null 2>&1; then
+  MISSING="$MISSING vds-cli"
+  VDS_PATH=$(command -v vds-cli 2>/dev/null)
+  if [[ -n "$VDS_PATH" ]]; then
+    echo "NOTE: $VDS_PATH exists but fails to run (broken shim or missing venv)"
+  fi
+fi
 [[ -z "${VDS_CONFLUENCE_TOKEN:-}" ]] && MISSING="$MISSING VDS_CONFLUENCE_TOKEN"
 [[ -z "${VDS_CONFLUENCE_SPACE_DEFAULT:-}" ]] && MISSING="$MISSING VDS_CONFLUENCE_SPACE_DEFAULT"
 if [[ -n "$MISSING" ]]; then

package/skills/vds-skill/search-confluence/SKILL.md

@@ -14,8 +14,14 @@ Before doing anything, run this check via Bash tool:
 ```bash
 source .claude/skills/_shared/vds-skill/credentials.sh 2>/dev/null
 MISSING=""
-{ command -v vds-cli >/dev/null 2>&1 || [[ -x .claude/bin/vds-cli ]]; } || MISSING="$MISSING vds-cli"
 [[ -x .claude/bin/vds-cli ]] && export PATH=".claude/bin:$PATH"
+if ! vds-cli --version >/dev/null 2>&1; then
+  MISSING="$MISSING vds-cli"
+  VDS_PATH=$(command -v vds-cli 2>/dev/null)
+  if [[ -n "$VDS_PATH" ]]; then
+    echo "NOTE: $VDS_PATH exists but fails to run (broken shim or missing venv)"
+  fi
+fi
 [[ -z "${VDS_CONFLUENCE_TOKEN:-}" ]] && MISSING="$MISSING VDS_CONFLUENCE_TOKEN"
 [[ -z "${VDS_JIRA_TOKEN:-}" ]] && MISSING="$MISSING VDS_JIRA_TOKEN"
 [[ -z "${VDS_CONFLUENCE_SPACE_DEFAULT:-}" ]] && MISSING="$MISSING VDS_CONFLUENCE_SPACE_DEFAULT"