@ikunin/sprintpilot 1.0.0 → 1.0.2

package/_Sprintpilot/scripts/lint-changed.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 
 const fs = require('node:fs');
 const path = require('node:path');
@@ -54,7 +53,9 @@ function localBin(name) {
   try {
     const stat = fs.statSync(p);
     if (stat.isFile()) return p;
-  } catch { /* ignore */ }
+  } catch {
+    /* ignore */
+  }
   return null;
 }
 
@@ -72,7 +73,8 @@ async function lintLanguage(lang, files) {
   if (lang === 'python') {
     if (await hasCli('ruff')) return runLinter('ruff', 'ruff', ['check'], files);
     if (await hasCli('flake8')) return runLinter('flake8', 'flake8', [], files);
-    if (await hasCli('pylint')) return runLinter('pylint', 'pylint', ['--output-format=text'], files);
+    if (await hasCli('pylint'))
+      return runLinter('pylint', 'pylint', ['--output-format=text'], files);
     return null;
   }
   if (lang === 'js-ts') {
@@ -83,15 +85,18 @@ async function lintLanguage(lang, files) {
     return null;
   }
   if (lang === 'rust') {
-    if (await hasCli('cargo')) return runLinter('cargo-clippy', 'cargo', ['clippy', '--message-format=short'], []);
+    if (await hasCli('cargo'))
+      return runLinter('cargo-clippy', 'cargo', ['clippy', '--message-format=short'], []);
     return null;
   }
   if (lang === 'go') {
-    if (await hasCli('golangci-lint')) return runLinter('golangci-lint', 'golangci-lint', ['run'], []);
+    if (await hasCli('golangci-lint'))
+      return runLinter('golangci-lint', 'golangci-lint', ['run'], []);
     return null;
   }
   if (lang === 'ruby') {
-    if (await hasCli('rubocop')) return runLinter('rubocop', 'rubocop', ['--format', 'simple'], files);
+    if (await hasCli('rubocop'))
+      return runLinter('rubocop', 'rubocop', ['--format', 'simple'], files);
     return null;
   }
   if (lang === 'java') {
@@ -108,25 +113,35 @@ async function lintLanguage(lang, files) {
     return null;
   }
   if (lang === 'c') {
-    if (await hasCli('cppcheck')) return runLinter('cppcheck', 'cppcheck', ['--enable=warning,style'], files);
+    if (await hasCli('cppcheck'))
+      return runLinter('cppcheck', 'cppcheck', ['--enable=warning,style'], files);
     if (await hasCli('clang-tidy')) return runLinter('clang-tidy', 'clang-tidy', [], files);
     return null;
   }
   if (lang === 'cpp') {
-    if (await hasCli('cppcheck')) return runLinter('cppcheck', 'cppcheck', ['--enable=warning,style', '--language=c++'], files);
+    if (await hasCli('cppcheck'))
+      return runLinter('cppcheck', 'cppcheck', ['--enable=warning,style', '--language=c++'], files);
     if (await hasCli('clang-tidy')) return runLinter('clang-tidy', 'clang-tidy', [], files);
     return null;
   }
   if (lang === 'csharp') {
-    if (await hasCli('dotnet')) return runLinter('dotnet-format', 'dotnet', ['format', '--verify-no-changes', '--diagnostics'], []);
+    if (await hasCli('dotnet'))
+      return runLinter(
+        'dotnet-format',
+        'dotnet',
+        ['format', '--verify-no-changes', '--diagnostics'],
+        [],
+      );
     return null;
   }
   if (lang === 'swift') {
-    if (await hasCli('swiftlint')) return runLinter('swiftlint', 'swiftlint', ['lint', '--quiet'], files);
+    if (await hasCli('swiftlint'))
+      return runLinter('swiftlint', 'swiftlint', ['lint', '--quiet'], files);
     return null;
   }
   if (lang === 'sql') {
-    if (await hasCli('sqlfluff')) return runLinter('sqlfluff', 'sqlfluff', ['lint', '--dialect', 'oracle'], files);
+    if (await hasCli('sqlfluff'))
+      return runLinter('sqlfluff', 'sqlfluff', ['lint', '--dialect', 'oracle'], files);
     return null;
   }
   if (lang === 'kotlin') {
@@ -135,7 +150,8 @@ async function lintLanguage(lang, files) {
     return null;
   }
   if (lang === 'php') {
-    if (await hasCli('phpstan')) return runLinter('phpstan', 'phpstan', ['analyse', '--no-progress'], files);
+    if (await hasCli('phpstan'))
+      return runLinter('phpstan', 'phpstan', ['analyse', '--no-progress'], files);
     if (await hasCli('phpcs')) return runLinter('phpcs', 'phpcs', [], files);
     return null;
   }
@@ -172,7 +188,10 @@ async function runOverride(name, files) {
 
 async function main() {
   const { opts } = parseArgs(process.argv.slice(2));
-  if (opts.help) { help(); process.exit(0); }
+  if (opts.help) {
+    help();
+    process.exit(0);
+  }
 
   const limit = parseInt(opts.limit || '100', 10);
   const outputFile = opts['output-file'] || '';
@@ -180,9 +199,9 @@ async function main() {
 
   const modified = await tryGitStdout(['diff', '--name-only', 'HEAD']);
   const untracked = await tryGitStdout(['ls-files', '--others', '--exclude-standard']);
-  const all = Array.from(
-    new Set([...splitLines(modified || ''), ...splitLines(untracked || '')])
-  ).filter(Boolean).sort();
+  const all = Array.from(new Set([...splitLines(modified || ''), ...splitLines(untracked || '')]))
+    .filter(Boolean)
+    .sort();
 
   if (all.length === 0) {
     log.out('No changed files to lint');

package/_Sprintpilot/scripts/lock.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 
 const fs = require('node:fs');
 const path = require('node:path');
@@ -88,12 +87,20 @@ function writeLockExclusive(lockFile, id) {
     fs.writeSync(fd, content, 0, 'utf8');
     wrote = true;
   } finally {
-    try { fs.closeSync(fd); } catch { /* ignore */ }
+    try {
+      fs.closeSync(fd);
+    } catch {
+      /* ignore */
+    }
     if (!wrote) {
       // writeSync failed (ENOSPC, EIO): leaving an empty lockfile behind
       // would look "corrupt" to the next acquirer and permanently wedge
       // the autopilot. Unlink so the next try can re-create cleanly.
-      try { fs.unlinkSync(lockFile); } catch { /* ignore */ }
+      try {
+        fs.unlinkSync(lockFile);
+      } catch {
+        /* ignore */
+      }
     }
   }
 }
@@ -143,7 +150,11 @@ function main() {
     // of them gets EEXIST.
     const info = readLockInfo(lockFile, staleSeconds);
     if (info.state === 'STALE') {
-      try { fs.unlinkSync(lockFile); } catch { /* ignore */ }
+      try {
+        fs.unlinkSync(lockFile);
+      } catch {
+        /* ignore */
+      }
       try {
         writeLockExclusive(lockFile, id);
         log.out(`ACQUIRED_STALE:${id}`);
@@ -176,7 +187,11 @@ function main() {
 
   if (action === 'release') {
     if (fs.existsSync(lockFile)) {
-      try { fs.unlinkSync(lockFile); } catch { /* ignore */ }
+      try {
+        fs.unlinkSync(lockFile);
+      } catch {
+        /* ignore */
+      }
       log.out('RELEASED');
     } else {
       log.out('NO_LOCK');
@@ -187,7 +202,8 @@ function main() {
   if (action === 'status') {
     const info = readLockInfo(lockFile, staleSeconds);
     if (info.state === 'FREE') log.out('Lock: free (no active session)');
-    else if (info.state === 'LOCKED') log.out(`Lock: ACTIVE — session ${info.id}, age ${info.ageMin}m`);
+    else if (info.state === 'LOCKED')
+      log.out(`Lock: ACTIVE — session ${info.id}, age ${info.ageMin}m`);
     else log.out(`Lock: STALE — session ${info.id}, age ${info.ageMin}m (will auto-remove)`);
   }
 }

package/_Sprintpilot/scripts/sanitize-branch.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 
 const crypto = require('node:crypto');
 
@@ -54,7 +53,10 @@ async function validateRefFormat(fullName) {
 
 async function main() {
   const { opts, positional } = parseArgs(process.argv.slice(2));
-  if (opts.help) { help(); process.exit(0); }
+  if (opts.help) {
+    help();
+    process.exit(0);
+  }
   const storyKey = positional[0];
   const prefix = opts.prefix ?? 'story/';
   const maxLength = parseInt(opts['max-length'] || '60', 10);

package/_Sprintpilot/scripts/stage-and-commit.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 
 const fs = require('node:fs');
 const path = require('node:path');
@@ -15,7 +14,9 @@ const {
 const log = require('../lib/runtime/log');
 
 function help() {
-  log.out("Usage: stage-and-commit.js --message 'msg' [--allowlist path] [--max-size-mb 1] [--file-list path] [--dry-run]");
+  log.out(
+    "Usage: stage-and-commit.js --message 'msg' [--allowlist path] [--max-size-mb 1] [--file-list path] [--dry-run]",
+  );
 }
 
 function splitOut(out) {
@@ -34,8 +35,9 @@ async function collectChanges() {
   // add-side list so we don't `git add` a path that no longer exists and
   // emit a spurious warning; the dedicated `git rm` loop handles them.
   const deletedSet = new Set(deleted);
-  const all = dedupeSorted([...splitOut(modified), ...splitOut(untracked)])
-    .filter((f) => !deletedSet.has(f));
+  const all = dedupeSorted([...splitOut(modified), ...splitOut(untracked)]).filter(
+    (f) => !deletedSet.has(f),
+  );
   return { all, deleted };
 }
 
@@ -52,7 +54,10 @@ function parseFileListMarkdown(filePath) {
 
 async function main() {
   const { opts } = parseArgs(process.argv.slice(2), { booleanFlags: ['dry-run'] });
-  if (opts.help) { help(); process.exit(0); }
+  if (opts.help) {
+    help();
+    process.exit(0);
+  }
 
   const message = opts.message ?? opts.m;
   const allowlist = opts.allowlist;
@@ -95,7 +100,9 @@ async function main() {
 
       if (!isAllowlisted(file, allowPatterns)) {
         if (lstat.size > MAX_SCAN_BYTES) {
-          warnings.push(`secret scan skipped for ${file} (size ${Math.floor(lstat.size / 1024)} KB > ${MAX_SCAN_BYTES / 1024} KB limit)`);
+          warnings.push(
+            `secret scan skipped for ${file} (size ${Math.floor(lstat.size / 1024)} KB > ${MAX_SCAN_BYTES / 1024} KB limit)`,
+          );
         } else if (!isBinary) {
           try {
             const raw = fs.readFileSync(file, 'utf8');
@@ -126,7 +133,8 @@ async function main() {
   if (fs.existsSync('.gitignore')) {
     // Exact line match — substring tests were fooled by the entry appearing
     // inside a comment (e.g. "# .autopilot.lock is auto-created").
-    const entries = fs.readFileSync('.gitignore', 'utf8')
+    const entries = fs
+      .readFileSync('.gitignore', 'utf8')
       .split(/\r?\n/)
       .map((l) => l.trim())
       .filter((l) => l && !l.startsWith('#'));

package/_Sprintpilot/scripts/sync-status.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 
 const fs = require('node:fs');
 const path = require('node:path');
@@ -40,12 +39,20 @@ function atomicWrite(targetPath, content) {
       try {
         fs.writeFileSync(targetPath, content, 'utf8');
       } finally {
-        try { fs.unlinkSync(tmp); } catch { /* best effort */ }
+        try {
+          fs.unlinkSync(tmp);
+        } catch {
+          /* best effort */
+        }
       }
       return;
     }
     // Any other error: clean up tmp so we don't leak cruft.
-    try { fs.unlinkSync(tmp); } catch { /* best effort */ }
+    try {
+      fs.unlinkSync(tmp);
+    } catch {
+      /* best effort */
+    }
     throw e;
   }
 }
@@ -78,7 +85,10 @@ function buildHeader(baseBranch, platform) {
 
 function main() {
   const { opts } = parseArgs(process.argv.slice(2));
-  if (opts.help) { help(); process.exit(0); }
+  if (opts.help) {
+    help();
+    process.exit(0);
+  }
 
   const story = opts.story;
   const statusFile = opts['git-status-file'];
@@ -102,12 +112,12 @@ function main() {
   // value", so we must NOT emit the field when the flag is absent. The
   // previous logic defaulted to 'false' and overwrote a prior 'true' every
   // call.
-  const hasWorktreeCleaned = Object.prototype.hasOwnProperty.call(opts, 'worktree-cleaned');
+  const hasWorktreeCleaned = Object.hasOwn(opts, 'worktree-cleaned');
   let worktreeCleaned;
   if (hasWorktreeCleaned) {
     const v = opts['worktree-cleaned'];
     // Accept 'true'/'false' strings (any case) and boolean true.
-    worktreeCleaned = (v === true || String(v).toLowerCase() === 'true') ? 'true' : 'false';
+    worktreeCleaned = v === true || String(v).toLowerCase() === 'true' ? 'true' : 'false';
   }
 
   const fields = [

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

@@ -177,7 +177,7 @@ Resolve:
     <action>STOP</action>
   </check>
 
-  <action>**Lock file** — run: `bash {{project_root}}/_Sprintpilot/scripts/lock.sh acquire`
+  <action>**Lock file** — run: `node {{project_root}}/_Sprintpilot/scripts/lock.js acquire`
   Output will be one of:
   - `ACQUIRED:<session-id>` → proceed
   - `ACQUIRED_STALE:<session-id>` → stale lock removed, proceed
@@ -189,18 +189,18 @@ Resolve:
   </check>
 
   <action>**Detect platform** — run:
-  `bash {{project_root}}/_Sprintpilot/scripts/detect-platform.sh --provider {{git.platform.provider}}`
+  `node {{project_root}}/_Sprintpilot/scripts/detect-platform.js --provider {{git.platform.provider}}`
   Output: `github`, `gitlab`, or `git_only`. Set `{{platform}}` to the output.
   Log: "Platform detected: {{platform}}"
   </action>
 
   <action>**Worktree health check** — run:
-  `bash {{project_root}}/_Sprintpilot/scripts/health-check.sh --base-branch {{base_branch}} --status-file {{status_file}}`
+  `node {{project_root}}/_Sprintpilot/scripts/health-check.js --base-branch {{base_branch}} --status-file {{status_file}}`
   Output classifies each worktree as CLEAN_DONE, COMMITTED, STALE, DIRTY, or ORPHAN.
   - CLEAN_DONE: `git worktree remove .worktrees/<name>` + `git worktree prune`
   - 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 `bash {{project_root}}/_Sprintpilot/scripts/create-pr.sh ...`
+    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}}`
     Then remove worktree.
   - STALE: `git worktree remove .worktrees/<name> --force` + prune
@@ -230,7 +230,7 @@ Resolve:
               **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"`.
         - 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 `bash {{project_root}}/_Sprintpilot/scripts/create-pr.sh --platform {{platform}} ...`
+          - 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)
     - If status IS "done" AND branch still exists AND `{{cleanup_on_merge}}` is true:
@@ -427,7 +427,7 @@ Resolve:
 <!-- GIT: Enter worktree before dev-story -->
 <check if="{{git_enabled}} AND {{next_skill}} is bmad-dev-story">
   <action>**Sanitize branch name** — run:
-  `bash {{project_root}}/_Sprintpilot/scripts/sanitize-branch.sh "{{current_story}}" --prefix "{{branch_prefix}}" --max-length 60`
+  `node {{project_root}}/_Sprintpilot/scripts/sanitize-branch.js "{{current_story}}" --prefix "{{branch_prefix}}" --max-length 60`
   Output: sanitized name (without prefix). Set `{{branch_name}}` = output.
   Full branch ref will be `{{branch_prefix}}{{branch_name}}`.
   </action>
@@ -530,7 +530,7 @@ pr_base: {{pr_base}}
   <!-- GIT: Lint, stage, and commit after dev-story -->
   <check if="{{git_enabled}} AND {{in_worktree}}">
     <action>**Lint changed files** — run:
-    `bash {{project_root}}/_Sprintpilot/scripts/lint-changed.sh --limit 100 --output-file lint-output.txt`
+    `node {{project_root}}/_Sprintpilot/scripts/lint-changed.js --limit 100 --output-file lint-output.txt`
     Log the output summary (non-blocking — lint never halts the autopilot).
     Set `{{lint_result}}` from the summary line.
     </action>
@@ -542,7 +542,7 @@ pr_base: {{pr_base}}
     - `{patch-title}` → from review finding title, fallback to "code review fix"
     Read the commit template from `git.commit_templates.story` in config (default: `feat({epic}): {story-title} ({story-key})`).
     Then run:
-    `bash {{project_root}}/_Sprintpilot/scripts/stage-and-commit.sh --message "feat({{epic}}): {{story-title}} ({{current_story}})" --allowlist {{project_root}}/_Sprintpilot/.secrets-allowlist`
+    `node {{project_root}}/_Sprintpilot/scripts/stage-and-commit.js --message "feat({{epic}}): {{story-title}} ({{current_story}})" --allowlist {{project_root}}/_Sprintpilot/.secrets-allowlist`
     Output: commit SHA. Set `{{story_commit}}` = output.
     Warnings (secrets, large files) printed to stderr — review but don't halt unless user says to.
     </action>
@@ -588,7 +588,7 @@ pr_base: {{pr_base}}
   <action>Sanitize branch name for `{{current_story}}` (same logic as step 3)</action>
   <action>Check if branch already registered in `{git_status_file}` for this story → skip if so</action>
   <action>Register branch in `{git_status_file}`:
-  `bash {{project_root}}/_Sprintpilot/scripts/sync-status.sh --story "{{current_story}}" --git-status-file "{{project_root}}/_bmad-output/implementation-artifacts/git-status.yaml" --branch "{{branch_prefix}}{{branch_name}}" --platform "{{platform}}" --base-branch "{{base_branch}}"`
+  `node {{project_root}}/_Sprintpilot/scripts/sync-status.js --story "{{current_story}}" --git-status-file "{{project_root}}/_bmad-output/implementation-artifacts/git-status.yaml" --branch "{{branch_prefix}}{{branch_name}}" --platform "{{platform}}" --base-branch "{{base_branch}}"`
   </action>
 </check>
 
@@ -741,7 +741,7 @@ Instruct: "Re-verify code review for story {{current_story}} — all patch findi
      - `{lint-result}` → `{{lint_result}}`
      - `{test-result}` → from last test run output
      - `{patch-count}` → number of patch commits
-  3. Run: `bash {{project_root}}/_Sprintpilot/scripts/create-pr.sh --platform {{platform}} --branch {{branch_prefix}}{{branch_name}} --base {{pr_base}} --title "{{story-title}} ({{current_story}})" --body "<filled template>"`
+  3. Run: `node {{project_root}}/_Sprintpilot/scripts/create-pr.js --platform {{platform}} --branch {{branch_prefix}}{{branch_name}} --base {{pr_base}} --title "{{story-title}} ({{current_story}})" --body "<filled template>"`
   4. Output: PR URL or "SKIPPED". Set `{{pr_url}}` = output.
   If creation fails → log warning, set `{{pr_url}}` = null, continue.
   </action>
@@ -800,7 +800,7 @@ Instruct: "Re-verify code review for story {{current_story}} — all patch findi
   </action>
 
   <action>**Write git status** to addon's own file (NEVER modify sprint-status.yaml) — runs AFTER checkout to base branch so the file persists in the working tree for the commit below:
-  `bash {{project_root}}/_Sprintpilot/scripts/sync-status.sh --story "{{current_story}}" --git-status-file "{{project_root}}/_bmad-output/implementation-artifacts/git-status.yaml" --branch "{{branch_prefix}}{{branch_name}}" --commit "{{story_commit}}" --patch-commits "{{patch_commits_csv}}" --push-status "{{push_status}}" --merge-status "{{merge_status}}" --pr-url "{{pr_url}}" --lint-result "{{lint_result}}" --worktree "{{project_root}}/.worktrees/{{current_story}}" --platform "{{platform}}" --base-branch "{{base_branch}}"`
+  `node {{project_root}}/_Sprintpilot/scripts/sync-status.js --story "{{current_story}}" --git-status-file "{{project_root}}/_bmad-output/implementation-artifacts/git-status.yaml" --branch "{{branch_prefix}}{{branch_name}}" --commit "{{story_commit}}" --patch-commits "{{patch_commits_csv}}" --push-status "{{push_status}}" --merge-status "{{merge_status}}" --pr-url "{{pr_url}}" --lint-result "{{lint_result}}" --worktree "{{project_root}}/.worktrees/{{current_story}}" --platform "{{platform}}" --base-branch "{{base_branch}}"`
   This writes to `git-status.yaml` (addon-owned). Sprint-status.yaml is BMAD-owned — updated by BMAD skills only.
   </action>
 
@@ -1043,7 +1043,7 @@ If the skill is not available or fails, generate a minimal README.md:
 
 <!-- GIT: Release lock -->
 <check if="{{git_enabled}}">
-  <action>Release lock: `bash {{project_root}}/_Sprintpilot/scripts/lock.sh release`</action>
+  <action>Release lock: `node {{project_root}}/_Sprintpilot/scripts/lock.js release`</action>
 </check>
 
 <action>Delete `{state_file}` — sprint complete</action>

package/bin/sprintpilot.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 
 const path = require('node:path');
 const { Command } = require('commander');
@@ -42,16 +41,24 @@ async function main() {
 
   program
     .name('sprintpilot')
-    .description('Sprintpilot — autopilot and multi-agent addon for BMad Method: autonomous story execution, parallel agents, git workflow')
+    .description(
+      'Sprintpilot — autopilot and multi-agent addon for BMad Method: autonomous story execution, parallel agents, git workflow',
+    )
     .version(await resolveVersion(), '-v, --version', 'Show version');
 
   program
     .command('install', { isDefault: true })
     .description('Install Sprintpilot into the current BMad Method project')
-    .option('--tools <list>', 'Comma-separated tools (claude-code,cursor,windsurf,cline,roo,trae,kiro,gemini-cli,github-copilot,all)')
+    .option(
+      '--tools <list>',
+      'Comma-separated tools (claude-code,cursor,windsurf,cline,roo,trae,kiro,gemini-cli,github-copilot,all)',
+    )
     .option('--dry-run', 'Preview without making changes')
     .option('--force', 'Skip backup of existing skills')
-    .option('--migrate-v1', 'Migrate from bmad-autopilot-addon v1 (auto-detected; this flag is for non-interactive CI)')
+    .option(
+      '--migrate-v1',
+      'Migrate from bmad-autopilot-addon v1 (auto-detected; this flag is for non-interactive CI)',
+    )
     .option('-y, --yes', 'Non-interactive mode')
     .action(async (options) => {
       try {

package/lib/commands/check-update.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('node:path');
 const fs = require('fs-extra');