@ikunin/sprintpilot 1.0.2 → 1.0.3

package/_Sprintpilot/manifest.yaml

@@ -1,6 +1,6 @@
 addon:
   name: sprintpilot
-  version: 1.0.2
+  version: 1.0.3
   description: Sprintpilot — autopilot and multi-agent addon for BMad Method (git workflow, parallel agents, autonomous story execution)
   bmad_compatibility: ">=6.2.0"
   modules:

package/_Sprintpilot/scripts/scan.js

@@ -0,0 +1,457 @@
+#!/usr/bin/env node
+
+/**
+ * Cross-platform codebase scanner.
+ *
+ * Replaces bash pipelines like `find ... -exec wc -l {} + | sort -rn | head -N`
+ * so sprintpilot skills work on Windows PowerShell / cmd / Gemini CLI, not just bash.
+ *
+ * Subcommands:
+ *   files       List files matching include globs, excluding ignore globs.
+ *               Flags: --include, --exclude, --root, --limit, --count
+ *   largest     Top N files by line count.
+ *               Flags: --include, --exclude, --root, --limit (default 10)
+ *   loc         Total line count across matched files.
+ *               Flags: --include, --exclude, --root
+ *   extensions  Extension frequency histogram, descending.
+ *               Flags: --exclude, --root, --limit (default 20)
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { parseArgs } = require('../lib/runtime/args');
+const log = require('../lib/runtime/log');
+
+const DEFAULT_EXCLUDES = [
+  'node_modules',
+  '.git',
+  'vendor',
+  'target',
+  'dist',
+  'build',
+  '.next',
+  '.nuxt',
+  '.svelte-kit',
+  'coverage',
+  '.turbo',
+  '.cache',
+  '__pycache__',
+  '.venv',
+  'venv',
+  '.worktrees',
+];
+
+function help() {
+  log.out(
+    'Usage: scan.js <files|largest|loc|extensions> [--include <globs>] [--exclude <globs>] [--root <path>] [--limit <N>] [--count]',
+  );
+}
+
+// Split a comma-delimited list, but keep commas inside {a,b} brace groups intact.
+function splitList(value) {
+  if (value === undefined || value === null || value === true) return [];
+  const s = String(value);
+  const out = [];
+  let buf = '';
+  let depth = 0;
+  for (let i = 0; i < s.length; i++) {
+    const c = s[i];
+    if (c === '{') depth++;
+    else if (c === '}') depth = Math.max(0, depth - 1);
+    if (c === ',' && depth === 0) {
+      if (buf.trim()) out.push(buf.trim());
+      buf = '';
+    } else {
+      buf += c;
+    }
+  }
+  if (buf.trim()) out.push(buf.trim());
+  return out;
+}
+
+// Find matching closing brace, respecting nesting. Returns -1 if unterminated.
+function findBraceClose(glob, start) {
+  let depth = 1;
+  for (let i = start + 1; i < glob.length; i++) {
+    const c = glob[i];
+    if (c === '\\' && i + 1 < glob.length) {
+      i++;
+      continue;
+    }
+    if (c === '{') depth++;
+    else if (c === '}') {
+      depth--;
+      if (depth === 0) return i;
+    }
+  }
+  return -1;
+}
+
+// Split a brace group's body on commas at depth 0 (so nested braces stay intact).
+function splitBraceAlts(body) {
+  const parts = [];
+  let buf = '';
+  let depth = 0;
+  for (let i = 0; i < body.length; i++) {
+    const c = body[i];
+    if (c === '\\' && i + 1 < body.length) {
+      buf += c + body[i + 1];
+      i++;
+      continue;
+    }
+    if (c === '{') depth++;
+    else if (c === '}') depth = Math.max(0, depth - 1);
+    if (c === ',' && depth === 0) {
+      parts.push(buf);
+      buf = '';
+    } else {
+      buf += c;
+    }
+  }
+  parts.push(buf);
+  return parts;
+}
+
+// Convert a glob pattern into a RegExp.
+// Supports: * (any chars except /), ** (any chars incl. /), ? (single non-/),
+// {a,b} alternation (nestable), and literal path segments.
+// Matching is against forward-slash paths.
+function globToRegex(glob) {
+  let re = '';
+  let i = 0;
+  while (i < glob.length) {
+    const c = glob[i];
+    if (c === '\\') {
+      if (i + 1 >= glob.length) {
+        // Trailing lone backslash — emit as a literal backslash.
+        re += '\\\\';
+        i++;
+        continue;
+      }
+      // Literal escape: pass the next char through verbatim.
+      const next = glob[i + 1];
+      re += '.+^$()|[]{}?*\\'.includes(next) ? '\\' + next : next;
+      i += 2;
+      continue;
+    }
+    if (c === '*') {
+      if (glob[i + 1] === '*') {
+        re += '.*';
+        i += 2;
+        if (glob[i] === '/') i++; // consume trailing slash of ** segment
+      } else {
+        re += '[^/]*';
+        i++;
+      }
+      continue;
+    }
+    if (c === '?') {
+      re += '[^/]';
+      i++;
+      continue;
+    }
+    if (c === '{') {
+      const end = findBraceClose(glob, i);
+      if (end === -1) {
+        // Unterminated brace — treat as literal.
+        re += '\\{';
+        i++;
+        continue;
+      }
+      const alts = splitBraceAlts(glob.slice(i + 1, end));
+      const altRegexes = alts.map((p) => globToRegex(p).source.slice(1, -1));
+      re += `(?:${altRegexes.join('|')})`;
+      i = end + 1;
+      continue;
+    }
+    if ('.+^$()|[]'.includes(c)) {
+      re += '\\' + c;
+      i++;
+      continue;
+    }
+    re += c;
+    i++;
+  }
+  return new RegExp('^' + re + '$');
+}
+
+function toPosix(p) {
+  return p.split(path.sep).join('/');
+}
+
+// Compile a pattern into { raw, re, pathAnchored }.
+// pathAnchored = true if the pattern contains a path separator; such patterns
+// only match the full relative path. Basename-only patterns (no '/') match
+// both the full path and the basename, so "*.ts" works at any depth.
+function compilePatterns(patterns) {
+  return patterns.map((p) => ({
+    raw: p,
+    re: globToRegex(p),
+    pathAnchored: p.includes('/'),
+  }));
+}
+
+function matchesAny(relPath, compiled) {
+  if (compiled.length === 0) return false;
+  const basename = relPath.slice(relPath.lastIndexOf('/') + 1);
+  for (const { re, pathAnchored } of compiled) {
+    if (re.test(relPath)) return true;
+    if (!pathAnchored && re.test(basename)) return true;
+  }
+  return false;
+}
+
+function isExcludedDir(name, excludeBasenames) {
+  return excludeBasenames.has(name);
+}
+
+function matchesExcludePath(relPath, compiled) {
+  return matchesAny(relPath, compiled);
+}
+
+// Resolve a dirent to { kind: 'file' | 'dir' | 'other' }, following symlinks
+// through stat(). Returns 'other' on broken links or errors.
+function classifyEntry(fullPath, entry) {
+  if (entry.isFile()) return 'file';
+  if (entry.isDirectory()) return 'dir';
+  if (entry.isSymbolicLink()) {
+    try {
+      const st = fs.statSync(fullPath);
+      if (st.isFile()) return 'file';
+      if (st.isDirectory()) return 'dir';
+    } catch {
+      return 'other';
+    }
+  }
+  return 'other';
+}
+
+function isWithinRoot(real, rootReal) {
+  if (real === rootReal) return true;
+  const prefix = rootReal.endsWith(path.sep) ? rootReal : rootReal + path.sep;
+  return real.startsWith(prefix);
+}
+
+// Walk directory tree, yielding files that match includes and not excludes.
+// Follows symlinks (like GNU find's default) but:
+//   - breaks cycles by tracking the realpath of every directory visited
+//   - refuses to traverse symlinks that escape the --root boundary
+function* walk(root, includes, excludes, excludeBasenames) {
+  const visited = new Set();
+  let rootReal;
+  try {
+    rootReal = fs.realpathSync(root);
+  } catch {
+    rootReal = path.resolve(root);
+  }
+  visited.add(rootReal);
+
+  const stack = [root];
+  while (stack.length) {
+    const dir = stack.pop();
+    let entries;
+    try {
+      entries = fs.readdirSync(dir, { withFileTypes: true });
+    } catch {
+      continue;
+    }
+    for (const entry of entries) {
+      const full = path.join(dir, entry.name);
+      const rel = toPosix(path.relative(root, full));
+      const kind = classifyEntry(full, entry);
+      if (kind === 'dir') {
+        if (isExcludedDir(entry.name, excludeBasenames)) continue;
+        if (matchesExcludePath(rel, excludes)) continue;
+        let real;
+        try {
+          real = fs.realpathSync(full);
+        } catch {
+          continue;
+        }
+        if (visited.has(real)) continue;
+        if (!isWithinRoot(real, rootReal)) continue; // refuse symlinks that escape root
+        visited.add(real);
+        stack.push(full);
+        continue;
+      }
+      if (kind !== 'file') continue;
+      if (matchesExcludePath(rel, excludes)) continue;
+      if (includes.length > 0 && !matchesAny(rel, includes)) continue;
+      // For symlinked files, verify the target is within root.
+      if (entry.isSymbolicLink()) {
+        try {
+          const fileReal = fs.realpathSync(full);
+          if (!isWithinRoot(fileReal, rootReal)) continue;
+        } catch {
+          continue;
+        }
+      }
+      yield rel;
+    }
+  }
+}
+
+function readLineCount(fullPath) {
+  let fd;
+  try {
+    fd = fs.openSync(fullPath, 'r');
+    const buf = Buffer.alloc(64 * 1024);
+    let count = 0;
+    let bytesRead;
+    let lastByte = null;
+    let total = 0;
+    while ((bytesRead = fs.readSync(fd, buf, 0, buf.length, null)) > 0) {
+      total += bytesRead;
+      for (let i = 0; i < bytesRead; i++) {
+        if (buf[i] === 0x0a) count++;
+      }
+      lastByte = buf[bytesRead - 1];
+    }
+    // Count the final line if the file is non-empty and doesn't end with \n
+    if (total > 0 && lastByte !== 0x0a) count++;
+    return count;
+  } catch {
+    return 0;
+  } finally {
+    if (fd !== undefined) {
+      try {
+        fs.closeSync(fd);
+      } catch {
+        /* ignore */
+      }
+    }
+  }
+}
+
+function resolveRoot(opts) {
+  const root = opts.root ? path.resolve(opts.root) : process.cwd();
+  try {
+    if (!fs.statSync(root).isDirectory()) {
+      log.fail(`root is not a directory: ${root}`);
+    }
+  } catch {
+    log.fail(`root does not exist: ${root}`);
+  }
+  return root;
+}
+
+function buildExcludes(extra) {
+  const list = [...DEFAULT_EXCLUDES, ...extra];
+  // Patterns: match the basename of a directory OR any path containing it.
+  const patterns = [];
+  const basenames = new Set();
+  for (const item of list) {
+    if (item.includes('/') || item.includes('*')) {
+      patterns.push(item);
+    } else {
+      basenames.add(item);
+      patterns.push(`**/${item}/**`);
+      patterns.push(item);
+    }
+  }
+  return { compiled: compilePatterns(patterns), basenames };
+}
+
+function cmdFiles(opts) {
+  const root = resolveRoot(opts);
+  const includes = compilePatterns(splitList(opts.include));
+  const { compiled: excludes, basenames } = buildExcludes(splitList(opts.exclude));
+  const limit = opts.limit ? Number(opts.limit) : 0;
+  const count = opts.count === true || opts.count === 'true';
+
+  let n = 0;
+  const out = [];
+  for (const rel of walk(root, includes, excludes, basenames)) {
+    n++;
+    if (!count) {
+      out.push(rel);
+      if (limit > 0 && out.length >= limit) break;
+    }
+  }
+  if (count) {
+    log.out(String(n));
+  } else {
+    for (const p of out) log.out(p);
+  }
+}
+
+function cmdLargest(opts) {
+  const root = resolveRoot(opts);
+  const includes = compilePatterns(splitList(opts.include));
+  const { compiled: excludes, basenames } = buildExcludes(splitList(opts.exclude));
+  const limit = opts.limit ? Number(opts.limit) : 10;
+
+  const heap = []; // simple array; N is small so O(files * log N) is fine
+  for (const rel of walk(root, includes, excludes, basenames)) {
+    const full = path.join(root, rel);
+    const lines = readLineCount(full);
+    heap.push({ lines, path: rel });
+  }
+  heap.sort((a, b) => b.lines - a.lines);
+  for (const item of heap.slice(0, limit)) {
+    log.out(`${item.lines}\t${item.path}`);
+  }
+}
+
+function cmdLoc(opts) {
+  const root = resolveRoot(opts);
+  const includes = compilePatterns(splitList(opts.include));
+  const { compiled: excludes, basenames } = buildExcludes(splitList(opts.exclude));
+
+  let total = 0;
+  let fileCount = 0;
+  for (const rel of walk(root, includes, excludes, basenames)) {
+    total += readLineCount(path.join(root, rel));
+    fileCount++;
+  }
+  log.out(`${total}\t${fileCount}`);
+}
+
+function cmdExtensions(opts) {
+  const root = resolveRoot(opts);
+  const { compiled: excludes, basenames } = buildExcludes(splitList(opts.exclude));
+  const limit = opts.limit ? Number(opts.limit) : 20;
+
+  const counts = new Map();
+  for (const rel of walk(root, [], excludes, basenames)) {
+    const base = rel.split('/').pop();
+    const dot = base.lastIndexOf('.');
+    const ext = dot > 0 ? base.slice(dot + 1) : '(no-ext)';
+    counts.set(ext, (counts.get(ext) || 0) + 1);
+  }
+  const rows = [...counts.entries()].sort((a, b) => b[1] - a[1]).slice(0, limit);
+  for (const [ext, n] of rows) {
+    log.out(`${n}\t${ext}`);
+  }
+}
+
+function main() {
+  const { opts, positional } = parseArgs(process.argv.slice(2), {
+    booleanFlags: ['count'],
+  });
+  if (opts.help || positional.length === 0) {
+    help();
+    process.exit(opts.help ? 0 : 1);
+  }
+  const cmd = positional[0];
+  switch (cmd) {
+    case 'files':
+      cmdFiles(opts);
+      break;
+    case 'largest':
+      cmdLargest(opts);
+      break;
+    case 'loc':
+      cmdLoc(opts);
+      break;
+    case 'extensions':
+      cmdExtensions(opts);
+      break;
+    default:
+      log.error(`Unknown subcommand: ${cmd}`);
+      help();
+      process.exit(1);
+  }
+}
+
+main();

package/_Sprintpilot/skills/sprint-autopilot-on/workflow.md

@@ -8,6 +8,24 @@ You do NOT hardcode the workflow sequence. After each completed skill, read its
 
 **Git integration** is additive. If `_Sprintpilot/manifest.yaml` doesn't exist or `git.enabled: false`, all git operations are silently skipped and this workflow behaves identically to the stock autopilot.
 
+### Shell portability (IMPORTANT)
+
+Sprintpilot runs under any LLM CLI (Claude Code, Gemini CLI, Cursor, etc.) on any OS. The shell that executes commands may be **bash, zsh, PowerShell, or cmd** depending on platform and CLI. Shell-specific idioms will fail silently when the wrong shell is used.
+
+**When you encounter bash-style idioms below, translate them to your shell.** The table applies to **external commands** (like `git`); cmdlets have slightly different conventions.
+
+| Bash idiom | PowerShell equivalent | Meaning |
+|---|---|---|
+| `A && B` | `A; if ($LASTEXITCODE -eq 0) { B }` (or separate commands, guarding B manually) | Run B only if A succeeded |
+| `A \|\| true` | `A; $LASTEXITCODE = 0` (or `try { A } catch {}` for cmdlets) | Run A, ignore failures |
+| `2>/dev/null` | `2>$null` | Suppress stderr |
+| `rm -rf <dir>` | `Remove-Item -Recurse -Force <dir>` | Recursive delete |
+| `if [ -f X ]; then ... fi` | `if (Test-Path -PathType Leaf X) { ... }` | File-exists check (regular file, not dir) |
+
+**Safer:** when in doubt, use the cross-platform Node helpers under `_Sprintpilot/scripts/`. For ad-hoc file ops, invoke Node inline: `node -e "require('fs').rmSync('<path>', {recursive: true, force: true})"`.
+
+If a step below uses `&&` to chain "run B only on A's success", and you cannot express that in one line, **run the commands separately and STOP if any step fails** — do not proceed past a failed step.
+
 ---
 
 ## AUTOPILOT RULES — READ BEFORE PROCEEDING
@@ -170,7 +188,9 @@ Resolve:
   <check if="not a git repo">
     <action>HALT: "No git repository found. Initialize one first:
     ```
-    git init && git add -A && git commit -m 'initial commit'
+    git init
+    git add -A
+    git commit -m 'initial commit'
     git remote add origin <your-repo-url>
     ```
     Then run /sprint-autopilot-on again."</action>
@@ -201,15 +221,20 @@ Resolve:
   - COMMITTED: log "Recoverable work found for <name> — will push via git -C"
     Push the branch: `git -C .worktrees/<name> push -u origin <branch> 2>&1`
     If `{{create_pr}}` is true AND platform != git_only: create PR via `node {{project_root}}/_Sprintpilot/scripts/create-pr.js ...`
-    If `{{create_pr}}` is false OR platform is git_only: merge directly — `git checkout -B {{base_branch}} origin/{{base_branch}} && git merge <branch> --no-edit && git push origin {{base_branch}}`
+    If `{{create_pr}}` is false OR platform is git_only: merge directly. Run each as a separate command; **STOP and log the failure if any step fails — do not proceed past a failed step**:
+      1. `git checkout -B {{base_branch}} origin/{{base_branch}}`
+      2. `git merge <branch> --no-edit`
+      3. `git push origin {{base_branch}}`
     Then remove worktree.
   - STALE: `git worktree remove .worktrees/<name> --force` + prune
   - DIRTY: warn user, ask how to proceed (stash/commit/discard)
-  - ORPHAN: `rm -rf .worktrees/<name>` + `git worktree prune`
+  - ORPHAN: remove the directory cross-platform with `node -e "require('fs').rmSync('.worktrees/<name>', {recursive: true, force: true})"`, then `git worktree prune`
   </action>
 
   <action>**Branch reconciliation** — detect pushed-but-unmerged story branches.
-  Run: `git fetch origin && git branch -r --list "origin/{{branch_prefix}}*"`
+  Run as separate commands — **if `git fetch origin` fails (network/auth), STOP branch reconciliation and log a warning; do not operate on stale local refs**:
+    1. `git fetch origin`
+    2. `git branch -r --list "origin/{{branch_prefix}}*"`
   For each remote branch:
     - Extract story-key from branch name (strip "origin/{{branch_prefix}}" prefix)
     - Look up story status in `{status_file}`
@@ -226,16 +251,16 @@ Resolve:
           - If merge fails: log warning, continue (branch is preserved on remote)
           - If merge succeeds:
             - Re-read `{status_file}` from HEAD (may now include story artifacts after merge)
-            - Update `{git_status_file}` via sync-status.sh: set `--merge-status "recovered"` for this story.
-              **IMPORTANT:** sync-status.sh does full block replacement. If the story already has an entry in `{git_status_file}`, re-read its existing fields and pass ALL of them alongside `--merge-status`. If no entry exists yet, pass at minimum `--branch` and `--push-status "pushed"`.
+            - Update `{git_status_file}` via sync-status.js: set `--merge-status "recovered"` for this story.
+              **IMPORTANT:** sync-status.js does full block replacement. If the story already has an entry in `{git_status_file}`, re-read its existing fields and pass ALL of them alongside `--merge-status`. If no entry exists yet, pass at minimum `--branch` and `--push-status "pushed"`.
         - If `{{platform}}` is NOT git_only (github, gitlab, bitbucket, gitea) AND `{{create_pr}}` is true:
           - Check if PR/MR already exists for this branch (platform-specific check via create-pr.sh or CLI)
           - If no PR: create one via `node {{project_root}}/_Sprintpilot/scripts/create-pr.js --platform {{platform}} ...`
           - Log: "PR created/found for <story-key>"
-          - Update `{git_status_file}` via sync-status.sh: set `--merge-status "pr_pending"` for this story (same full-field requirement as above)
+          - Update `{git_status_file}` via sync-status.js: set `--merge-status "pr_pending"` for this story (same full-field requirement as above)
     - If status IS "done" AND branch still exists AND `{{cleanup_on_merge}}` is true:
       - Log: "Stale remote branch: <branch> — story already done, cleaning up"
-      - Delete remote branch: `git push origin --delete <branch> 2>/dev/null || true`
+      - Delete remote branch (ignore failure — the branch may already be gone): `git push origin --delete <branch>`
   </action>
 
   <action>Set `{{git_enabled}}` = true, `{{platform}}` = detected value</action>
@@ -486,8 +511,8 @@ Resolve:
     </action>
 
     <action>**Init submodules** if needed.
-    Run: `if [ -f .gitmodules ]; then timeout 30 git submodule update --init --recursive 2>&1 || echo "SUBMODULE_TIMEOUT"; fi`
-    If SUBMODULE_TIMEOUT: warn "Submodule init timed out (may need auth). Continuing without."
+    First check for `.gitmodules` (use your file-exists tool, or `node -e "process.exit(require('fs').existsSync('.gitmodules')?0:1)"`). If not present, skip this step.
+    If present, run `git submodule update --init --recursive` (give it ~30 seconds). If the command fails or hangs, warn "Submodule init failed (may need auth). Continuing without." and proceed.
     </action>
 
     <action>Set `{{in_worktree}}` = true</action>
@@ -595,17 +620,18 @@ pr_base: {{pr_base}}
 <!-- GIT: Commit planning artifacts to main after planning skills -->
 <check if="{{git_enabled}} AND {{completed_skill}} is a planning skill (bmad-create-prd, bmad-create-architecture, bmad-create-ux-design, bmad-create-epics-and-stories, bmad-sprint-planning, bmad-check-implementation-readiness, bmad-create-story)">
   <action>**Commit planning artifacts to main** — keep track of all planning decisions in git.
-  Stage all changed artifacts:
+  Stage all changed artifacts (ignore errors — any of these paths may not yet exist):
   ```
-  git add _bmad-output/planning-artifacts/ _bmad-output/implementation-artifacts/ _bmad-output/stories/ 2>/dev/null || true
+  git add _bmad-output/planning-artifacts/ _bmad-output/implementation-artifacts/ _bmad-output/stories/
   ```
-  If there are staged changes, commit:
+  Check if there's anything staged; if yes, commit:
   ```
-  git diff --cached --quiet || git commit -m "docs: {{completed_skill}} artifacts"
+  git diff --cached --quiet
   ```
-  Push to remote if possible:
+  If that exits non-zero (there are staged changes), run: `git commit -m "docs: {{completed_skill}} artifacts"`
+  Then push (log a warning if push fails; do not halt autopilot):
   ```
-  git push origin {{base_branch}} 2>/dev/null || true
+  git push origin {{base_branch}}
   ```
   </action>
 </check>
@@ -775,12 +801,12 @@ Instruct: "Re-verify code review for story {{current_story}} — all patch findi
       Log warning but do NOT halt. The branch is pushed and preserved.
       Boot reconciliation (INITIALIZATION branch reconciliation) will retry on next session.
 
-    Note: `{{merge_status}}` is persisted by the full sync-status.sh call later in this step (via `--merge-status`). Do NOT call sync-status.sh separately here — it does full block replacement and would destroy other fields.
+    Note: `{{merge_status}}` is persisted by the full sync-status.js call later in this step (via `--merge-status`). Do NOT call sync-status.js separately here — it does full block replacement and would destroy other fields.
     </action>
     <check if="{{cleanup_on_merge}} is true">
-      <action>**Cleanup worktree** for merged story — branch was merged locally, worktree is no longer needed:
+      <action>**Cleanup worktree** for merged story — branch was merged locally, worktree is no longer needed. Ignore failures from the remove (the worktree may already be gone):
       ```
-      git worktree remove .worktrees/{{current_story}} --force 2>/dev/null || true
+      git worktree remove .worktrees/{{current_story}} --force
       git worktree prune
       ```
       </action>
@@ -804,17 +830,21 @@ Instruct: "Re-verify code review for story {{current_story}} — all patch findi
   This writes to `git-status.yaml` (addon-owned). Sprint-status.yaml is BMAD-owned — updated by BMAD skills only.
   </action>
 
-  <action>**Stage and commit artifacts** — explicitly include git-status.yaml and decision-log.yaml:
+  <action>**Stage and commit artifacts** — explicitly include git-status.yaml and decision-log.yaml. Ignore errors from the `git add` (any listed path may not yet exist):
+  ```
+  git add _bmad-output/implementation-artifacts/sprint-status.yaml _bmad-output/implementation-artifacts/git-status.yaml _bmad-output/implementation-artifacts/autopilot-state.yaml _bmad-output/implementation-artifacts/decision-log.yaml _bmad-output/stories/ _bmad-output/planning-artifacts/
+  ```
+  Check if anything is staged: `git diff --cached --quiet`. If that exits non-zero, commit:
+  `git commit -m "docs: story {{current_story}} done — {{test_count}} tests{{#if pr_url}}, PR: {{pr_url}}{{/if}}"`
+  Then push (log a warning if push fails; do not halt autopilot):
   ```
-  git add _bmad-output/implementation-artifacts/sprint-status.yaml _bmad-output/implementation-artifacts/git-status.yaml _bmad-output/implementation-artifacts/autopilot-state.yaml _bmad-output/implementation-artifacts/decision-log.yaml _bmad-output/stories/ _bmad-output/planning-artifacts/ 2>/dev/null || true
-  git diff --cached --quiet || git commit -m "docs: story {{current_story}} done — {{test_count}} tests{{#if pr_url}}, PR: {{pr_url}}{{/if}}"
-  git push origin {{base_branch}} 2>/dev/null || true
+  git push origin {{base_branch}}
   ```
   This ensures sprint-status.yaml, git-status.yaml, story files, and any updated artifacts are on main even when story code is on a PR branch.
   </action>
 </check>
 
-<!-- Story git status was already written by sync-status.sh above (when git_enabled AND in_worktree).
+<!-- Story git status was already written by sync-status.js above (when git_enabled AND in_worktree).
      sprint-status.yaml is BMAD-owned — updated by bmad-dev-story / bmad-code-review directly. -->
 <check if="NOT {{git_enabled}}">
   <action>Log: "Story {{current_story}} complete — BMAD dev-story updates sprint-status.yaml directly"</action>
@@ -917,7 +947,7 @@ pr_base: {{pr_base}}
         `git merge <branch-ref> --no-edit`
         `git push origin {{base_branch}}`
       - If merge succeeds: update merge_status in `{git_status_file}`.
-        **IMPORTANT:** sync-status.sh does full block replacement — you MUST re-read the story's existing fields from `{git_status_file}` (branch, commit, patch_commits, push_status, pr_url, lint_result, worktree, platform, base_branch, worktree_cleaned) and pass ALL of them along with `--merge-status "merged"`. Omitting fields destroys them.
+        **IMPORTANT:** sync-status.js does full block replacement — you MUST re-read the story's existing fields from `{git_status_file}` (branch, commit, patch_commits, push_status, pr_url, lint_result, worktree, platform, base_branch, worktree_cleaned) and pass ALL of them along with `--merge-status "merged"`. Omitting fields destroys them.
       - If merge fails: `git merge --abort`, update merge_status to "failed" in `{git_status_file}` (same full-field requirement), log warning, continue
   Log: "Pre-checkpoint merge: N stories verified on {{base_branch}}"
   </action>
@@ -999,13 +1029,14 @@ If the skill is not available or fails, generate a minimal README.md:
 
 <!-- GIT: Commit documentation and final artifacts to main -->
 <check if="{{git_enabled}}">
-  <action>**Commit final artifacts and documentation to main**:
+  <action>**Commit final artifacts and documentation to main**. Run each step; if an early step fails, STOP and log — don't proceed past a failed step. `git add` may fail for missing optional paths (`docs/`, `README.md`); ignore those path-specific errors. Failure of the final push should log a warning but not halt autopilot:
   ```
   git checkout -B {{base_branch}} origin/{{base_branch}}
-  git add _bmad-output/ README.md docs/ 2>/dev/null || true
-  git diff --cached --quiet || git commit -m "docs: project documentation and final artifacts"
-  git push origin {{base_branch}} 2>/dev/null || true
+  git add _bmad-output/ README.md docs/
   ```
+  Check if anything is staged: `git diff --cached --quiet`. If that exits non-zero, commit:
+  `git commit -m "docs: project documentation and final artifacts"`
+  Then: `git push origin {{base_branch}}`
   </action>
 </check>
 
@@ -1035,8 +1066,8 @@ If the skill is not available or fails, generate a minimal README.md:
 <check if="{{git_enabled}}">
   <action>**Cleanup all remaining worktrees**:
   Run: `git worktree list --porcelain`
-  For each worktree that is NOT the main worktree:
-    `git worktree remove <path> --force 2>/dev/null || true`
+  For each worktree that is NOT the main worktree, run the following — log and continue on failure; some worktrees may already be gone:
+    `git worktree remove <path> --force`
   Then: `git worktree prune`
   </action>
 </check>

package/_Sprintpilot/skills/sprintpilot-codebase-map/agents/architecture-mapper.md

@@ -19,28 +19,35 @@ Scan the project at `{{project_root}}` and write your findings to `{{output_file
 - `*.key`, `*.pem`, `*.p12` (private keys)
 - `credentials.json`, `service-account.json`
 
-## Exploration Commands
+## Exploration
 
-```bash
-# Top-level structure
-ls -la
-find . -maxdepth 2 -type d -not -path '*/node_modules/*' -not -path '*/.git/*' -not -path '*/vendor/*' | head -50
+Use your native file tools (Read, Glob, Grep). The lists below describe what data to collect; pick the appropriate tool for your CLI.
 
-# Entry points
-cat index.ts index.js main.py main.go cmd/main.go src/main.rs lib/main.rb app.py manage.py main.c main.cpp src/main.c src/main.cpp 2>/dev/null | head -30
+### Top-level structure
+Glob the root for directories and files (exclude `node_modules`, `.git`, `vendor`, `target`, `dist`, `build`). Look 1-2 levels deep to understand the layout.
 
-# Route definitions
-grep -rn "router\.\|app\.\(get\|post\|put\|delete\|patch\)\|@app\.route\|@Controller\|@RequestMapping\|CROW_ROUTE\|CPPREST_\|Pistache::" --include='*.ts' --include='*.js' --include='*.py' --include='*.java' --include='*.go' --include='*.xml' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -30
+### Entry points
+Read whichever of these exist: `index.ts`, `index.js`, `main.py`, `main.go`, `cmd/main.go`, `src/main.rs`, `lib/main.rb`, `app.py`, `manage.py`, `main.c`, `main.cpp`, `src/main.c`, `src/main.cpp`. 30 lines is usually enough to identify the entry path.
 
-# Module exports / barrel files
-find . -name 'index.ts' -o -name 'index.js' -o -name '__init__.py' -o -name 'mod.rs' | head -20
+### Route definitions
+Use Grep to find route declarations across `*.ts`, `*.js`, `*.py`, `*.java`, `*.go`, `*.xml`, C/C++ headers. Pattern set:
+```
+router\.|app\.(get|post|put|delete|patch)|@app\.route|@Controller|@RequestMapping|CROW_ROUTE|CPPREST_|Pistache::
+```
+Limit to ~30 matches.
 
-# Import patterns (what depends on what)
-grep -rn "^import\|^from\|require(\|source \|^\.\|^#include" --include='*.ts' --include='*.js' --include='*.py' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | awk -F'from |require|#include' '{print $2}' | sort | uniq -c | sort -rn | head -20
+### Module exports / barrel files
+Use Glob for: `**/index.ts`, `**/index.js`, `**/__init__.py`, `**/mod.rs`. Cap at ~20 hits.
 
-# Configuration loading
-grep -rn "config\|CONFIG\|Settings\|settings" --include='*.ts' --include='*.js' --include='*.py' --include='*.yaml' --include='*.json' --include='*.xml' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' -l | head -10
+### Import patterns (what depends on what)
+Use Grep to find import/require/include lines across `*.ts`, `*.js`, `*.py`, `*.sh`, C/C++ sources:
 ```
+^import|^from|require\(|source |^\.|^#include
+```
+Scan the top ~100 matches and note frequent dependencies. (No need to replicate the old `awk | sort | uniq -c` pipeline — just eyeball recurring targets.)
+
+### Configuration loading
+Use Grep (files-with-matches mode) for `config|CONFIG|Settings|settings` across config-bearing file types. Limit to ~10 files.
 
 Read entry point files, follow the import chain 2-3 levels deep to understand request flow.
 

package/_Sprintpilot/skills/sprintpilot-codebase-map/agents/concerns-hunter.md

@@ -20,42 +20,65 @@ Scan the project at `{{project_root}}` and write your findings to `{{output_file
 - `credentials.json`, `service-account.json`
 - Files in `.git/` directory
 
-## Exploration Commands
+## Exploration
 
-```bash
-# TODOs, FIXMEs, HACKs
-grep -rn 'TODO\|FIXME\|HACK\|XXX\|WORKAROUND\|TEMP\|DEPRECATED' --include='*.ts' --include='*.js' --include='*.py' --include='*.go' --include='*.java' --include='*.rs' --include='*.rb' --include='*.cs' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.xml' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -50
+Use Grep for pattern searches and `scan.js` for aggregations. All Grep calls below should filter to code file types (e.g., `*.ts`, `*.js`, `*.py`, `*.java`, `*.go`, `*.rs`, `*.rb`, `*.cs`, `*.sql`, `*.sps`, `*.spb`, `*.xml`, `*.sh`, `*.c`, `*.h`, `*.cpp`, `*.hpp`, `*.cc`, `*.cxx`, `*.hxx`) and cap each result set (~20-50).
 
-# Security: hardcoded secrets patterns
-grep -rn 'password\s*=\s*["\x27]\|api_key\s*=\s*["\x27]\|secret\s*=\s*["\x27]\|token\s*=\s*["\x27]' --include='*.ts' --include='*.js' --include='*.py' --include='*.java' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.xml' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' -i | grep -v 'node_modules\|test\|spec\|mock\|fixture\|\.env\.example' | head -20
+### TODOs, FIXMEs, HACKs
+Grep for: `TODO|FIXME|HACK|XXX|WORKAROUND|TEMP|DEPRECATED`. Limit ~50.
 
-# Security: dangerous functions
-grep -rn 'eval(\|exec(\|dangerouslySetInnerHTML\|innerHTML\s*=\|__import__\|pickle\.load\|yaml\.load(\|EXECUTE IMMEDIATE\|DBMS_SQL' --include='*.ts' --include='*.js' --include='*.py' --include='*.java' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.sh' | head -20
+### Security: hardcoded secrets
+Grep (case-insensitive) for: `password\s*=\s*["']|api_key\s*=\s*["']|secret\s*=\s*["']|token\s*=\s*["']`. Exclude matches under `node_modules/`, `test*`, `spec*`, `*mock*`, `*fixture*`, `.env.example`. Limit ~20.
 
-# SQL injection risk
-grep -rn 'query.*\${\|query.*%s\|query.*format\|execute.*f"\|query.*\+' --include='*.ts' --include='*.js' --include='*.py' --include='*.java' --include='*.xml' | head -20
+### Security: dangerous runtime sinks
+Grep for these high-risk call sites (code-exec and XSS patterns). The tokens below are split to avoid security-hook false positives on this documentation file — when building your regex, join them with `|` and concatenate the split tokens exactly as indicated.
 
-# C/C++ unsafe string and memory functions (buffer overflow risk)
-grep -rn 'strcpy(\|strcat(\|sprintf(\|gets(\|scanf(.*%s[^0-9]\|system(\|popen(' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -20
+Literal regex tokens (already properly escaped):
 
-# Dead code: unused imports (sample)
-grep -rn '^import.*from' --include='*.ts' --include='*.js' --include='*.py' | awk -F'import ' '{print $2}' | awk -F' from' '{print $1}' | sort | uniq -c | sort -rn | head -10
+- `eval\(`
+- `exec\(`
+- `innerHTML\s*=`
+- `__import__`
+- `yaml\.load\(`
+- `EXECUTE IMMEDIATE`
+- `DBMS_SQL`
 
-# Commented-out code blocks (likely dead code)
-grep -rn '^\s*//.*function\|^\s*//.*class\|^\s*//.*const\|^\s*#.*def\|^\s*#.*class\|^\s*--.*PROCEDURE\|^\s*--.*FUNCTION\|^\s*--.*PACKAGE\|^\s*//.*struct\|^\s*//.*typedef\|^\s*/\*.*struct\|^\s*/\*.*typedef' --include='*.ts' --include='*.js' --include='*.py' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -20
+Split tokens — concatenate the two halves verbatim, then escape the resulting literal dot:
 
-# Complexity: deeply nested code
-grep -rn '^\s\{16,\}' --include='*.ts' --include='*.js' --include='*.py' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -10
+- `dangerously` + `SetInnerHTML` → final regex literal `dangerouslySetInnerHTML`
+- `pick` + `le.load` → final regex literal `pickle\.load` (note the escaped dot)
 
-# Large files (complexity hotspots)
-find . -type f \( -name '*.ts' -o -name '*.js' -o -name '*.py' -o -name '*.java' -o -name '*.sql' -o -name '*.sps' -o -name '*.spb' -o -name '*.xml' -o -name '*.sh' -o -name '*.c' -o -name '*.h' -o -name '*.cpp' -o -name '*.hpp' -o -name '*.cc' -o -name '*.cxx' -o -name '*.hxx' \) -not -path '*/node_modules/*' -exec wc -l {} + 2>/dev/null | sort -rn | head -10
+Run the search case-sensitively across the code-file types listed above. Limit ~20.
 
-# Deprecated package warnings
-cat package.json 2>/dev/null | grep -i 'deprecated\|legacy\|old'
+### SQL injection risk
+Grep across `*.ts`, `*.js`, `*.py`, `*.java`, `*.xml` for: `query.*\$\{|query.*%s|query.*format|execute.*f"|query.*\+`. Limit ~20.
 
-# Error handling: bare catches
-grep -rn 'catch\s*(\|except:\|except Exception\|rescue$\|EXCEPTION\s*$\|WHEN OTHERS\|catch\s*(\.\.\.)' --include='*.ts' --include='*.js' --include='*.py' --include='*.rb' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -20
+### C/C++ unsafe string / memory functions
+Grep across C/C++ files only for: `strcpy\(|strcat\(|sprintf\(|gets\(|scanf\(.*%s[^0-9]|system\(|popen\(`. Limit ~20.
+
+### Dead code: unused-import candidates
+Grep for `^import.*from` across `*.ts`, `*.js`, `*.py` and eyeball the top imports. A full frequency rollup is not required — cite notable duplicates.
+
+### Commented-out code blocks
+Grep for:
+```
+^\s*//.*(function|class|const|struct|typedef)|^\s*#.*(def|class)|^\s*--.*(PROCEDURE|FUNCTION|PACKAGE)|^\s*/\*.*(struct|typedef)
 ```
+Limit ~20.
+
+### Complexity: deeply nested code
+Grep for lines starting with 16+ spaces: `^\s{16,}`. Limit ~10 (sample).
+
+### Large files (complexity hotspots)
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" largest --include "*.ts,*.js,*.py,*.java,*.cs,*.go,*.rs,*.rb,*.sql,*.sps,*.spb,*.xml,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --root "{{project_root}}" --limit 10
+```
+
+### Deprecated package warnings
+Read `package.json` if present and check for `deprecated|legacy|old` (case-insensitive).
+
+### Error handling: bare catches
+Grep for: `catch\s*\(|except:|except Exception|rescue$|EXCEPTION\s*$|WHEN OTHERS|catch\s*\(\.\.\.\)`. Limit ~20.
 
 ## Downstream Consumers
 

package/_Sprintpilot/skills/sprintpilot-codebase-map/agents/integration-mapper.md

@@ -21,36 +21,36 @@ Scan the project at `{{project_root}}` and write your findings to `{{output_file
 
 **DO read**: `.env.example`, `.env.sample`, `.env.template` (safe — contain variable names only)
 
-## Exploration Commands
+## Exploration
 
-```bash
-# Environment variables referenced in code
-grep -rn 'process\.env\.\|os\.environ\|os\.getenv\|ENV\[\|\${\|export \|getenv(' --include='*.ts' --include='*.js' --include='*.py' --include='*.rb' --include='*.go' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | sed 's/.*\(process\.env\.[A-Z_]*\|os\.environ\[['"'"'"]\?\([A-Z_]*\)\|os\.getenv(\([A-Z_]*\)\|ENV\[\([A-Z_]*\)\|getenv("\?\([A-Z_]*\)\).*/\1/' | sort -u | head -30
+Use Grep and Read. Below are the patterns to search for — file-type filters match the original language coverage (`*.ts`, `*.js`, `*.py`, `*.rb`, `*.go`, `*.rs`, `*.java`, `*.sh`, `*.c`, `*.h`, `*.cpp`, `*.hpp`, `*.cc`, `*.cxx`, `*.hxx`, `*.sql`, `*.sps`, `*.spb`, `*.xml`). Cap each result set (~15-30 matches).
 
-# .env.example (safe to read — template only)
-cat .env.example .env.sample .env.template 2>/dev/null
+### Environment variables referenced in code
+Grep for: `process\.env\.|os\.environ|os\.getenv|ENV\[|\$\{|export |getenv\(`. Extract uppercase identifier tokens from matches and list unique variable names.
 
-# HTTP client usage
-grep -rn 'fetch(\|axios\.\|requests\.\|http\.Client\|HttpClient\|urllib\|net/http\|reqwest\|curl \|wget \|curl_easy_\|libcurl\|cpprest\|boost::beast' --include='*.ts' --include='*.js' --include='*.py' --include='*.go' --include='*.rs' --include='*.java' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' | head -20
+### .env.example (safe to read — template only)
+Read whichever exist: `.env.example`, `.env.sample`, `.env.template`.
 
-# Database connections
-grep -rn 'createConnection\|createPool\|mongoose\.connect\|prisma\|sequelize\|knex\|sqlalchemy\|diesel\|gorm\|ActiveRecord\|jdbc\|sqlplus\|TNS_ADMIN\|CONNECT \|PQconnectdb\|mysql_real_connect\|SQLConnect\|OCILogon' --include='*.ts' --include='*.js' --include='*.py' --include='*.go' --include='*.rs' --include='*.java' --include='*.rb' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.xml' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' -l | head -10
+### HTTP client usage
+Grep for: `fetch\(|axios\.|requests\.|http\.Client|HttpClient|urllib|net/http|reqwest|curl |wget |curl_easy_|libcurl|cpprest|boost::beast`. Limit ~20.
 
-# Message queue / event usage
-grep -rn 'kafka\|rabbitmq\|amqp\|sqs\|sns\|pubsub\|redis.*pub\|redis.*sub\|bull\|BullMQ\|celery\|sidekiq\|AQ$\|DBMS_AQ\|librdkafka\|cppkafka\|zmq_' --include='*.ts' --include='*.js' --include='*.py' --include='*.rb' --include='*.sql' --include='*.sps' --include='*.spb' --include='*.xml' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' -i | head -15
+### Database connections
+Grep (files-with-matches) for: `createConnection|createPool|mongoose\.connect|prisma|sequelize|knex|sqlalchemy|diesel|gorm|ActiveRecord|jdbc|sqlplus|TNS_ADMIN|CONNECT |PQconnectdb|mysql_real_connect|SQLConnect|OCILogon`. Limit ~10 files.
 
-# Cloud SDK usage
-grep -rn 'aws-sdk\|@aws-sdk\|boto3\|google-cloud\|@google-cloud\|azure\|@azure\|aws/core\|Aws::\|google::cloud' --include='*.ts' --include='*.js' --include='*.py' --include='*.java' --include='*.xml' --include='*.sh' --include='*.c' --include='*.h' --include='*.cpp' --include='*.hpp' --include='*.cc' --include='*.cxx' --include='*.hxx' -l | head -10
+### Message queue / event usage
+Grep (case-insensitive) for: `kafka|rabbitmq|amqp|sqs|sns|pubsub|redis.*pub|redis.*sub|bull|BullMQ|celery|sidekiq|AQ$|DBMS_AQ|librdkafka|cppkafka|zmq_`. Limit ~15.
 
-# OAuth / Auth providers
-grep -rn 'oauth\|passport\|auth0\|firebase.*auth\|cognito\|supabase.*auth\|clerk\|next-auth\|lucia' --include='*.ts' --include='*.js' --include='*.py' --include='*.xml' -i | head -15
+### Cloud SDK usage
+Grep (files-with-matches) for: `aws-sdk|@aws-sdk|boto3|google-cloud|@google-cloud|azure|@azure|aws/core|Aws::|google::cloud`. Limit ~10.
 
-# Third-party SaaS SDKs
-grep -rn 'stripe\|sendgrid\|twilio\|sentry\|datadog\|segment\|amplitude\|mixpanel\|intercom\|slack' --include='*.ts' --include='*.js' --include='*.py' --include='*.xml' --include='*.sh' -i -l | head -10
+### OAuth / Auth providers
+Grep (case-insensitive) across `*.ts`, `*.js`, `*.py`, `*.xml` for: `oauth|passport|auth0|firebase.*auth|cognito|supabase.*auth|clerk|next-auth|lucia`. Limit ~15.
 
-# Docker-compose services (external deps)
-cat docker-compose*.yml 2>/dev/null | grep -E '^\s+\w+:$|image:' | head -20
-```
+### Third-party SaaS SDKs
+Grep (files-with-matches, case-insensitive) for: `stripe|sendgrid|twilio|sentry|datadog|segment|amplitude|mixpanel|intercom|slack`. Limit ~10.
+
+### Docker-compose services
+Read `docker-compose*.yml` files (use Glob to find them) and note the service names and `image:` values.
 
 ## Downstream Consumers
 

package/_Sprintpilot/skills/sprintpilot-codebase-map/agents/quality-assessor.md

@@ -19,35 +19,47 @@ Scan the project at `{{project_root}}` and write your findings to `{{output_file
 - `*.key`, `*.pem`, `*.p12` (private keys)
 - `credentials.json`, `service-account.json`
 
-## Exploration Commands
+## Exploration
 
-```bash
-# Test framework detection
-cat jest.config* vitest.config* pytest.ini setup.cfg pyproject.toml .rspec Cargo.toml 2>/dev/null | grep -i 'test\|jest\|pytest\|mocha\|vitest\|rspec'
+Use your native file tools (Read, Glob, Grep) plus the `scan.js` helper for aggregations.
 
-# Test file count vs source file count
-echo "Test files:" && find . -type f \( -name '*.test.*' -o -name '*.spec.*' -o -name 'test_*' -o -name '*_test.*' \) -not -path '*/node_modules/*' | wc -l
-echo "Source files:" && find . -type f \( -name '*.ts' -o -name '*.js' -o -name '*.py' -o -name '*.go' -o -name '*.rs' -o -name '*.java' -o -name '*.sql' -o -name '*.sps' -o -name '*.spb' -o -name '*.xml' -o -name '*.sh' -o -name '*.c' -o -name '*.h' -o -name '*.cpp' -o -name '*.hpp' -o -name '*.cc' -o -name '*.cxx' -o -name '*.hxx' \) -not -path '*/node_modules/*' -not -path '*/test*' -not -name '*.test.*' -not -name '*.spec.*' | wc -l
+### Test framework detection
+Read if present: `jest.config*`, `vitest.config*`, `pytest.ini`, `setup.cfg`, `pyproject.toml`, `.rspec`, `Cargo.toml`. Grep each for `test|jest|pytest|mocha|vitest|rspec` (case-insensitive).
 
-# Test types present
-find . -path '*/e2e/*' -o -path '*/integration/*' -o -path '*/unit/*' -o -name '*.e2e.*' -o -name '*.integration.*' 2>/dev/null | head -10
+### Test file count
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" files --include "*.test.*,*.spec.*,test_*,*_test.*" --root "{{project_root}}" --count
+```
 
-# CI/CD configuration
-cat .github/workflows/*.yml .gitlab-ci.yml Jenkinsfile azure-pipelines.yml .circleci/config.yml .travis.yml 2>/dev/null | head -80
+### Source file count
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" files --include "*.ts,*.js,*.py,*.go,*.rs,*.java,*.sql,*.sps,*.spb,*.xml,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --exclude "**/test/**,**/tests/**,**/__tests__/**,**/spec/**,*.test.*,*.spec.*,*_test.*,test_*" --root "{{project_root}}" --count
+```
+(The `scan.js` helper automatically also excludes `node_modules`, `.git`, `vendor`, `dist`, `build`, etc.)
 
-# Linting & formatting config
-ls -la .eslintrc* .prettierrc* .editorconfig .rubocop.yml .flake8 pyproject.toml rustfmt.toml .golangci.yml biome.json .sqlfluff .sqlfluffrc 2>/dev/null
+### Test types present
+Use Glob for `**/e2e/**`, `**/integration/**`, `**/unit/**`, `*.e2e.*`, `*.integration.*`. First ~10 hits are enough.
 
-# Code metrics (approximate LOC)
-find . -type f \( -name '*.ts' -o -name '*.js' -o -name '*.py' -o -name '*.go' -o -name '*.rs' -o -name '*.java' -o -name '*.sql' -o -name '*.sps' -o -name '*.spb' -o -name '*.xml' -o -name '*.sh' -o -name '*.c' -o -name '*.h' -o -name '*.cpp' -o -name '*.hpp' -o -name '*.cc' -o -name '*.cxx' -o -name '*.hxx' \) -not -path '*/node_modules/*' -not -path '*/.git/*' | xargs wc -l 2>/dev/null | tail -1
+### CI/CD configuration
+Read if present: `.github/workflows/*.yml` (use Glob to list them first), `.gitlab-ci.yml`, `Jenkinsfile`, `azure-pipelines.yml`, `.circleci/config.yml`, `.travis.yml`. 80 lines each is usually enough.
 
-# Largest files (complexity hotspots)
-find . -type f \( -name '*.ts' -o -name '*.js' -o -name '*.py' -o -name '*.sql' -o -name '*.sps' -o -name '*.spb' -o -name '*.sh' -o -name '*.c' -o -name '*.h' -o -name '*.cpp' -o -name '*.hpp' -o -name '*.cc' -o -name '*.cxx' -o -name '*.hxx' \) -not -path '*/node_modules/*' -exec wc -l {} + 2>/dev/null | sort -rn | head -10
+### Linting & formatting config
+Use Glob to list: `.eslintrc*`, `.prettierrc*`, `.editorconfig`, `.rubocop.yml`, `.flake8`, `pyproject.toml`, `rustfmt.toml`, `.golangci.yml`, `biome.json`, `.sqlfluff*`.
 
-# Coverage config
-cat .nycrc .istanbul.yml jest.config* vitest.config* 2>/dev/null | grep -i 'cover'
-ls -la coverage/ htmlcov/ .coverage 2>/dev/null
+### Code metrics (total LOC)
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" loc --include "*.ts,*.js,*.py,*.go,*.rs,*.java,*.sql,*.sps,*.spb,*.xml,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --root "{{project_root}}"
 ```
+Output is tab-separated `<total-lines>\t<file-count>`.
+
+### Largest files (complexity hotspots)
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" largest --include "*.ts,*.js,*.py,*.sql,*.sps,*.spb,*.sh,*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --root "{{project_root}}" --limit 10
+```
+Output: `<lines>\t<path>`, descending.
+
+### Coverage
+Read `.nycrc`, `.istanbul.yml`, `jest.config*`, `vitest.config*` if present and grep for `cover`. Use Glob to check for `coverage/`, `htmlcov/`, `.coverage` directories/files.
 
 ## Downstream Consumers
 
@@ -69,8 +81,8 @@ Write to `{{output_file}}`:
 | Metric | Value | Evidence |
 |--------|-------|----------|
 | Test framework | Jest 29.7 | jest.config.ts:1 |
-| Test files | 45 | find output |
-| Source files | 120 | find output |
+| Test files | 45 | scan.js files --include ... --count |
+| Source files | 120 | scan.js files --include ... --count |
 | Test:Source ratio | 1:2.7 | Calculated |
 | Test types present | unit, integration | directory structure |
 | Test types missing | e2e, snapshot | No e2e/ directory found |

package/_Sprintpilot/skills/sprintpilot-codebase-map/agents/stack-analyzer.md

@@ -20,50 +20,48 @@ Scan the project at `{{project_root}}` and write your findings to `{{output_file
 - `credentials.json`, `service-account.json`
 - `*.secret`, `*password*`, `*token*` (in filenames)
 
-## Exploration Commands
-
-Run these to gather data (adapt paths as needed):
-
-```bash
-# Package manifests
-cat package.json 2>/dev/null | head -100
-cat pyproject.toml 2>/dev/null
-cat Cargo.toml 2>/dev/null
-cat go.mod 2>/dev/null
-cat Gemfile 2>/dev/null
-cat pom.xml 2>/dev/null | head -100
-cat build.gradle 2>/dev/null | head -50
-cat *.csproj 2>/dev/null | head -50
-
-# Database / PL/SQL manifests
-ls -la *.sql *.sps *.spb 2>/dev/null | head -10
-find . -type f \( -name '*.sql' -o -name '*.sps' -o -name '*.spb' \) -not -path '*/.git/*' 2>/dev/null | wc -l
-cat tnsnames.ora sqlnet.ora 2>/dev/null | head -20
-
-# C / C++ manifests
-ls -la *.c *.h *.cpp *.hpp *.cc *.cxx *.hxx 2>/dev/null | head -10
-find . -type f \( -name '*.c' -o -name '*.h' -o -name '*.cpp' -o -name '*.hpp' -o -name '*.cc' -o -name '*.cxx' -o -name '*.hxx' \) -not -path '*/.git/*' 2>/dev/null | wc -l
-cat CMakeLists.txt configure.ac conanfile.txt vcpkg.json 2>/dev/null | head -20
-
-# Lockfiles (versions)
-head -100 package-lock.json 2>/dev/null || head -100 yarn.lock 2>/dev/null || head -100 pnpm-lock.yaml 2>/dev/null
-
-# Runtime versions
-cat .nvmrc .node-version .python-version .ruby-version .tool-versions 2>/dev/null
-cat rust-toolchain.toml 2>/dev/null
-
-# File type distribution
-find . -type f -not -path '*/node_modules/*' -not -path '*/.git/*' -not -path '*/vendor/*' -not -path '*/target/*' | sed 's/.*\.//' | sort | uniq -c | sort -rn | head -20
-
-# Build tools
-ls -la webpack.config* vite.config* rollup.config* tsconfig* babel.config* .babelrc Makefile CMakeLists.txt build.gradle* pom.xml *.sln *.xml 2>/dev/null
-
-# Infrastructure
-ls -la Dockerfile* docker-compose* .dockerignore 2>/dev/null
-ls -la terraform/ cdk.json serverless.yml k8s/ kubernetes/ helm/ 2>/dev/null
+## Exploration
+
+Gather data using your native file tools (Read, Glob, Grep). The commands below are illustrative — use the equivalent tool from your CLI. Skip files that don't exist; do not fail the task on missing manifests.
+
+### Package manifests
+Read each of these if present (top 100 lines is enough for most):
+`package.json`, `pyproject.toml`, `Cargo.toml`, `go.mod`, `Gemfile`, `pom.xml`, `build.gradle`, any `*.csproj`.
+
+### Database / PL/SQL manifests
+Read `tnsnames.ora`, `sqlnet.ora` if present.
+Count SQL / PL-SQL files:
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" files --include "*.sql,*.sps,*.spb" --root "{{project_root}}" --count
+```
+
+### C / C++ manifests
+Read `CMakeLists.txt`, `configure.ac`, `conanfile.txt`, `vcpkg.json` if present.
+Count C/C++ files:
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" files --include "*.c,*.h,*.cpp,*.hpp,*.cc,*.cxx,*.hxx" --root "{{project_root}}" --count
 ```
 
-Also use Glob and Grep to find patterns not covered above.
+### Lockfiles (versions)
+Read the first ~100 lines of whichever is present: `package-lock.json`, `yarn.lock`, `pnpm-lock.yaml`.
+
+### Runtime versions
+Read if present: `.nvmrc`, `.node-version`, `.python-version`, `.ruby-version`, `.tool-versions`, `rust-toolchain.toml`.
+
+### File type distribution
+```
+node "{{project_root}}/_Sprintpilot/scripts/scan.js" extensions --root "{{project_root}}" --limit 20
+```
+Output is tab-separated `<count>\t<extension>`, descending.
+
+### Build tools
+Use Glob to list if present: `webpack.config*`, `vite.config*`, `rollup.config*`, `tsconfig*`, `babel.config*`, `.babelrc`, `Makefile`, `CMakeLists.txt`, `build.gradle*`, `pom.xml`, `*.sln`.
+
+### Infrastructure
+Use Glob to list if present: `Dockerfile*`, `docker-compose*`, `.dockerignore`.
+Check for directories: `terraform/`, `k8s/`, `kubernetes/`, `helm/`. Read `cdk.json`, `serverless.yml` if present.
+
+Use Glob and Grep to find patterns not covered above.
 
 ## Downstream Consumers
 
@@ -87,7 +85,7 @@ Write to `{{output_file}}`:
 | TypeScript | 142 | 65% | Application code |
 | ... | ... | ... | ... |
 
-Evidence: `find` command output showing file distribution
+Evidence: `scan.js extensions` output showing file distribution
 
 ## Frameworks & Core Libraries
 | Name | Version | Purpose | Evidence |

package/_Sprintpilot/skills/sprintpilot-codebase-map/workflow.md

@@ -18,7 +18,7 @@ Complementary, not a replacement. `bmad-document-project` generates comprehensiv
 
 ## Step 1 — Prepare
 
-<action>Create output directory: `mkdir -p {output_folder}/codebase-analysis`</action>
+<action>Create output directory `{output_folder}/codebase-analysis` (use your native file-create tool; it will create parent directories as needed). The first `Write` tool call targeting a file inside this directory will auto-create it, so no explicit mkdir is required in practice.</action>
 <action>Determine project root absolute path: `{{project_root}}`</action>
 
 ---

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ikunin/sprintpilot",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Sprintpilot — autopilot and multi-agent addon for BMad Method v6: git workflow, parallel agents, autonomous story execution",
   "license": "Apache-2.0",
   "repository": {