@ikunin/sprintpilot 2.2.22 → 2.2.24

package/_Sprintpilot/bin/autopilot.js

@@ -1460,6 +1460,23 @@ function cmdStart(opts) {
       '[autopilot] WARN ma.parallel_stories=true but the state machine is not yet wired for parallel dispatch (planned for v2.3.0). Stories will run sequentially this session.\n',
     );
   }
+  if (profile.lint_enabled) {
+    // v2.2.24: lint_enabled wires verifyDevGreen → post-green-gates.js
+    // (lint-changed + lint-test-pitfalls + ci-parity scan). lint_blocking
+    // governs whether a failed gate halts the autopilot or passes
+    // through with a warning. The v2.2.23 "not wired" warning is gone —
+    // lint runs for real now.
+    ledger.append(
+      {
+        kind: 'state_transition',
+        detail: {
+          lint_enabled: true,
+          lint_blocking: !!profile.lint_blocking,
+        },
+      },
+      { projectRoot },
+    );
+  }
 
   // Worktree health check — once per session, after lock acquire so we
   // don't compete with another active session for the same .worktrees
@@ -1629,7 +1646,7 @@ function cmdRecord(opts) {
     profile.enabled === false && stateMachine.shouldSkipVerifyWhenGitDisabled(runtime.phase);
   let verifyResult;
   if (signal.status === 'success' && !isGitDisabledPhase) {
-    verifyResult = verifyMod.verify(runtime, signal.output, { projectRoot });
+    verifyResult = verifyMod.verify(runtime, signal.output, { projectRoot, profile });
     ledger.append(
       { kind: 'verify_result', phase: runtime.phase, ok: verifyResult.ok, issues: verifyResult.issues || [] },
       { projectRoot },
@@ -1638,7 +1655,7 @@ function cmdRecord(opts) {
     verifyResult = verifyMod.verifyWithOverride(
       runtime,
       signal.output || {},
-      { projectRoot },
+      { projectRoot, profile },
       signal.evidence || {},
     );
     ledger.append(

package/_Sprintpilot/lib/orchestrator/profile-rules.js

@@ -172,6 +172,16 @@ function flatToProfile(resolved, profileName) {
     // --stale-minutes. 0 disables the auto-takeover entirely (locks are
     // never considered stale; manual `autopilot off` required).
     lock_stale_timeout_minutes: coerceInt(get(resolved, 'git.lock.stale_timeout_minutes'), 30),
+    // git.lint.* — documented in modules/git/config.yaml as a future
+    // post-DEV_GREEN lint phase. Currently NOT wired into the state
+    // machine (no LINT_CHECK phase emitted). v2.2.23 plumbs the config
+    // to the typed Profile so users see the shape and cmdStart emits an
+    // experimental warning when lint_enabled=true (mirroring
+    // parallel_stories handling). Full state-machine integration is
+    // tracked for v2.3.0+.
+    lint_enabled: coerceBool(get(resolved, 'git.lint.enabled'), false),
+    lint_blocking: coerceBool(get(resolved, 'git.lint.blocking'), false),
+    lint_output_limit: coerceInt(get(resolved, 'git.lint.output_limit'), 100),
     // git.platform.provider + base_url — forwarded to create-pr.js when
     // the orchestrator opens or polls PRs. 'auto' delegates platform
     // detection to create-pr.js (currently defaults to github).

package/_Sprintpilot/lib/orchestrator/verify.js

@@ -164,6 +164,54 @@ function autoDetectTestFiles(ctx, baseBranch) {
   return out;
 }
 
+// Invoke scripts/post-green-gates.js when profile.lint_enabled is true.
+// Returns null when:
+//   - profile.lint_enabled is false / absent (feature opt-out)
+//   - the script is missing (partial install)
+//   - projectRoot is unset
+// Returns { failed: bool, summary?: string } on a real run.
+//
+// The script's contract: exit 0 = all gates pass, exit !=0 = at least
+// one gate failed. JSON report on stdout when invoked with --json (we
+// pass that flag). Failure summary captured for the issue message.
+function runPostGreenGates(ctx) {
+  if (!ctx || !ctx.profile || !ctx.profile.lint_enabled) return null;
+  if (!ctx.projectRoot) return null;
+  const scriptRel = nodePath.join('_Sprintpilot', 'scripts', 'post-green-gates.js');
+  const scriptAbs = nodePath.join(ctx.projectRoot, scriptRel);
+  let fs;
+  try {
+    fs = ctx.fs || nodeFs;
+    if (!fs.existsSync(scriptAbs)) return null;
+  } catch {
+    return null;
+  }
+  const cp = require('node:child_process');
+  try {
+    const r = cp.spawnSync(
+      'node',
+      [scriptAbs, '--json', '--project-root', ctx.projectRoot],
+      { encoding: 'utf8', stdio: ['ignore', 'pipe', 'pipe'], timeout: 120_000 },
+    );
+    if (r.status === 0) return { failed: false };
+    // Try to extract a brief summary from the JSON output. Fall back
+    // to the exit code if parsing fails.
+    let summary = `exit ${r.status}`;
+    try {
+      const parsed = JSON.parse(r.stdout || '{}');
+      if (parsed && parsed.failed_gate) summary = `failed_gate=${parsed.failed_gate}`;
+      if (parsed && parsed.first_issue) summary += `: ${parsed.first_issue}`;
+    } catch {
+      /* keep exit-code summary */
+    }
+    return { failed: true, summary };
+  } catch (_e) {
+    // Script crashed (e.g. ENOENT for node). Treat as non-failing —
+    // the lint phase should not gate the autopilot on its own bugs.
+    return null;
+  }
+}
+
 // Probe the underlying git state to confirm a STORY_DONE signal whose
 // `git_steps_completed` flag was omitted. Returns true iff:
 //   - commit_sha resolves locally (git cat-file -e <sha>)
@@ -227,6 +275,7 @@ function verify(state, signalOutput, context) {
     runner: (context && context.runner) || null,
     projectRoot: (context && context.projectRoot) || '.',
     augmented: (context && context.augmented) || null,
+    profile: (context && context.profile) || null,
   };
   const out = signalOutput || {};
   // Effective state: fall forward to signal.output for identity fields
@@ -381,6 +430,20 @@ function verifyDevGreen(state, out, ctx) {
       issues.push('tests_run must be a positive number (per AGENTS.md test-result format)');
     }
   }
+  // Post-GREEN gates: lint-changed + lint-test-pitfalls + ci-parity scan.
+  // Composed pipeline lives in scripts/post-green-gates.js. Only fires
+  // when profile.lint_enabled === true. Blocking vs non-blocking
+  // governed by profile.lint_blocking. Pre-2.2.24 the script existed
+  // and was documented as "called by the orchestrator after GREEN
+  // verify" but nothing actually invoked it.
+  const lintResult = runPostGreenGates(ctx);
+  if (lintResult) {
+    if (lintResult.failed && (ctx.profile && ctx.profile.lint_blocking)) {
+      issues.push(
+        `post-green-gates failed (lint_blocking=true): ${lintResult.summary || 'see ledger detail'}`,
+      );
+    }
+  }
   return { ok: issues.length === 0, issues };
 }
 

package/_Sprintpilot/manifest.yaml

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

package/package.json

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