@boyingliu01/xp-gate 0.33.0 → 0.33.2

package/hooks/gate-8.sh

@@ -59,10 +59,24 @@ fi
 # revisions and reports "0 commits scanned" on a pre-commit run -- so a staged
 # secret was never seen and Gate 8 always passed. `git --staged` scans
 # `git diff --cached`, which is exactly the pre-commit payload (#449).
+#
+# #616: `--all` is a different question, and the index is the wrong place to ask
+# it. Under `xp-gate check . --all` the user asked "is the whole tree clean", the
+# index is usually empty, `git --staged` legitimately found nothing, and that
+# nothing was counted as PASSED -- a secret sitting in a tracked file was
+# invisible to the one command meant to find it. The runner already exports
+# GATE_SCOPE (#589); this fragment just has to read it. `dir --no-git` scans the
+# filesystem so tracked-but-unmodified files are covered.
 GITLEAKS_REPORT="${TMPDIR:-/tmp}/gitleaks-report.json"
 rm -f "$GITLEAKS_REPORT"
-GITLEAKS_OUTPUT=$($GITLEAKS_CMD git --staged --redact --no-banner $GITLEAKS_CONFIG --report-format=json --report-path="$GITLEAKS_REPORT" 2>&1)
-GITLEAKS_EXIT=$?
+if [ "${GATE_SCOPE:-diff}" = "all" ]; then
+  echo "     Scope: --all (scanning the working tree, not the index)"
+  GITLEAKS_OUTPUT=$($GITLEAKS_CMD dir --no-git --redact --no-banner $GITLEAKS_CONFIG --report-format=json --report-path="$GITLEAKS_REPORT" 2>&1)
+  GITLEAKS_EXIT=$?
+else
+  GITLEAKS_OUTPUT=$($GITLEAKS_CMD git --staged --redact --no-banner $GITLEAKS_CONFIG --report-format=json --report-path="$GITLEAKS_REPORT" 2>&1)
+  GITLEAKS_EXIT=$?
+fi
 
 if [ "$GITLEAKS_EXIT" -eq 0 ]; then
   # Decide SKIP from the REPORT ARTIFACT, not from log wording. Parsing stdout
@@ -183,7 +197,22 @@ fi
 # if-condition (errexit-exempt); a non-zero hook exit lands in the else branch.
 if command -v detect-secrets-hook >/dev/null 2>&1 && [ -f ".secrets.baseline" ]; then
   DS_TMP="$(dirname "$(mktemp -u)")/ds-output.$$"
-  if git diff --cached --name-only --diff-filter=ACM -z \
+  # #616: same scope split as the gitleaks call above. Under `--all` the index is
+  # usually empty, so piping `git diff --cached -z` into xargs fed the hook zero
+  # files and it reported success without reading anything.
+  if [ "${GATE_SCOPE:-diff}" = "all" ]; then
+    if git ls-files -z | xargs -0 detect-secrets-hook --baseline .secrets.baseline > "$DS_TMP" 2>&1; then
+      echo "     ✅ detect-secrets PASSED - no new secrets beyond baseline (scope: --all)."
+    else
+      cat "$DS_TMP"
+      echo ""
+      echo "     ❌ BLOCKED - detect-secrets found secrets not in .secrets.baseline."
+      echo "   If these are false positives, regenerate the baseline (cross-platform):"
+      echo "     detect-secrets scan --update .secrets.baseline && git add .secrets.baseline"
+      rm -f "$DS_TMP"
+      exit 1
+    fi
+  elif git diff --cached --name-only --diff-filter=ACM -z \
       | xargs -0 detect-secrets-hook --baseline .secrets.baseline > "$DS_TMP" 2>&1; then
     echo "     ✅ detect-secrets PASSED - no new secrets beyond baseline."
   else

package/hooks/gate-9.sh

@@ -32,8 +32,20 @@ if [ -z "$SEMGREP_CMD" ]; then
   echo "     Gate 9: SAST Security (WARN, semgrep not installed)"
   GATE_9_STATUS="WARN"
 else
-  # Get staged files filtered to Semgrep-supported languages
-  SEMGREP_FILES=$(git diff --cached --name-only --diff-filter=ACM 2>/dev/null | grep -E '\.(ts|tsx|js|jsx|py|go|java|c|cpp|cs|rb|php|scala|swift)$' || true)
+  # Get staged files filtered to Semgrep-supported languages.
+  #
+  # #616: under `--all` the index is usually empty, so the staged diff yielded no
+  # files and the gate reported SKIPPED -- "no supported language files changed"
+  # -- for a run whose whole purpose was to scan the tree. Same fail-open shape
+  # #589 fixed in Gate 3, one layer down: the CLI says the tree was checked and
+  # the fragment checks a diff that does not exist. The runner exports GATE_SCOPE;
+  # read it and enumerate tracked files instead.
+  if [ "${GATE_SCOPE:-diff}" = "all" ]; then
+    echo "     Scope: --all (scanning tracked files, not the index)"
+    SEMGREP_FILES=$(git ls-files 2>/dev/null | grep -E '\.(ts|tsx|js|jsx|py|go|java|c|cpp|cs|rb|php|scala|swift)$' || true)
+  else
+    SEMGREP_FILES=$(git diff --cached --name-only --diff-filter=ACM 2>/dev/null | grep -E '\.(ts|tsx|js|jsx|py|go|java|c|cpp|cs|rb|php|scala|swift)$' || true)
+  fi
 
   if [ -z "$SEMGREP_FILES" ]; then
     echo "     ⏭️  SKIPPED - SAST (no supported language files changed)"

package/hooks/pre-commit

@@ -3495,6 +3495,29 @@ else
     GATE_5_STATUS="PASS"
   fi
 fi
+
+# #665: a subset run is exempt from the GLOBAL coverage floor by design (#473),
+# but the exemption was never published, so the persisted report showed
+# `actual.coverage = 29` beside `thresholds.lines = 82` with `status = PASS` and
+# `reason = ""`. The verdict was correct; the RECORD was unreadable, and a record
+# is all `xp-gate audit` and the tech-debt trackers (#636, #638) ever see. Two
+# opposite situations -- "coverage really was 29% and the gate passed it" and
+# "a subset run, global floor not applicable" -- rendered identically.
+#
+# Derived from PARTIAL_TEST_RUN at ONE place instead of being set by hand at each
+# of the six branches that mark a partial run: a reason written at one call site
+# and forgotten at another is the drift that produced the empty reason here.
+if [ "${PARTIAL_TEST_RUN:-false}" = "true" ]; then
+  GATE_5_COVERAGE_APPLIES_TO="subset"
+  # Never overwrite a reason the branch itself recorded (docs_only_change,
+  # no_code_files_changed, ...) -- those explain a different verdict.
+  if [ -z "${GATE_5_DETAIL:-}" ] && [ "$GATE_5_STATUS" = "PASS" ]; then
+    GATE_5_DETAIL="partial_test_run_coverage_not_applicable"
+  fi
+else
+  GATE_5_COVERAGE_APPLIES_TO="project"
+fi
+
 record_gate_audit "gate-5" "tests-coverage" "$GATE_5_STATUS" "0" "$GATE_5_START" "${GATE_5_DETAIL:-}"
 
 # ============================================================================
@@ -3619,7 +3642,26 @@ else
         if [ -f "arch-go.yaml" ] || [ -f "arch-go.yml" ]; then
           if require_tool "arch-go" "Gate 6" "go install github.com/arch-go/arch-go@latest"; then
             echo "     Running arch-go for Go..."
+            # #630: #602 replaced the vacuous Java arm with a status-based verdict,
+            # but the Go arm kept the identical fail-open shape: it piped arch-go
+            # output into `head` and then printed "Go architecture validation
+            # completed" unconditionally. A pipeline's status is `head`'s -- 0 for
+            # ANY non-empty output -- so every arch-go result, contract violations
+            # included, printed "completed" and the commit landed. A gate that
+            # cannot fail is not a gate. Capture the checker's own status and
+            # block on it, as the Python arm above and run_java_build_integrity do.
             arch-go check 2>&1 | head -20
+            # The pipeline's own status is head's, so the checker's verdict has
+            # to come from PIPESTATUS[0] (#630). This is the same reading #602
+            # installed for Java via run_java_build_integrity.
+            _go_arch_rc="${PIPESTATUS[0]}"
+            if [ "$_go_arch_rc" -ne 0 ]; then
+              echo ""
+              echo "❌ BLOCKED - Go architecture contract violations (arch-go, exit $_go_arch_rc)"
+              echo "Fix the layering violation(s) above before committing."
+              GATE_6_REASON_BY_LANG["$CURRENT_LANG"]="architecture_tests_failed"
+              exit 1
+            fi
             echo "     ✅ Go architecture validation completed."
           else
             echo "     ❌ BLOCKED - arch-go not installed"
@@ -4583,7 +4625,7 @@ generate_quality_report() {
       "reason": "${GATE_5_DETAIL:-}",
       "thresholds": { "lines": ${COVERAGE_THRESHOLD}, "functions": null, "branches": null, "statements": null },
       "threshold_source": "${COVERAGE_THRESHOLD_SOURCE}",
-      "actual": { "coverage": "${COV_PCT}" },
+      "actual": { "coverage": "${COV_PCT}", "applies_to": "${GATE_5_COVERAGE_APPLIES_TO:-project}" },
       "failedTests": $FAILED_TESTS_JSON
     },
     "gate6_arch_boyscout": {
@@ -4681,7 +4723,14 @@ ENDJSON
   printf "  %-45s %s\n" "Gate 2: Duplicate Code" "${GATE_2_STATUS:-SKIP}"
   printf "  %-45s %s (warnings: ${CC_WARNINGS:-0})\n" "Gate 3: Complexity" "${GATE_3_STATUS:-SKIP}"
   printf "  %-45s %s (warnings: ${WARNING_COUNT:-0})\n" "Gate 4: Principles" "${GATE_4_STATUS:-SKIP}"
-  printf "  %-45s %s (coverage: ${COV_PCT}%%)\n" "Gate 5: Tests + Coverage" "${GATE_5_STATUS:-SKIP}"
+  # #665: a subset run's percentage describes a SUBSET, not the project, and the
+  # global floor was not applied. Printing the bare number next to a PASS reads as
+  # "29% passed an 82% floor". Say which population the number came from.
+  if [ "${GATE_5_COVERAGE_APPLIES_TO:-project}" = "subset" ]; then
+    printf "  %-45s %s (coverage: ${COV_PCT}%%, subset run -- global floor not checked)\n" "Gate 5: Tests + Coverage" "${GATE_5_STATUS:-SKIP}"
+  else
+    printf "  %-45s %s (coverage: ${COV_PCT}%%)\n" "Gate 5: Tests + Coverage" "${GATE_5_STATUS:-SKIP}"
+  fi
   printf "  %-45s %s\n" "Gate 6: Architecture + Boy Scout" "${GATE_6_STATUS:-SKIP}"
   printf "  %-45s %s\n" "Gate 7: IaC Security" "${GATE_7_STATUS:-SKIP}"
   printf "  %-45s %s\n" "Gate 8: Secret Scanning" "${GATE_8_STATUS:-SKIP}"

package/lib/__tests__/check-normalize-exit.test.js

@@ -0,0 +1,24 @@
+/**
+ * @test check-normalize-exit
+ * @intent Prove the CLI never hands a non-numeric verdict to process.exit (the #657 crash) and that ONLY the GATE_NOT_RUN sentinel is coerced to 0
+ * @covers AC-DSH-019
+ */
+const { normalizeGateExit } = require('../check.js');
+
+describe('normalizeGateExit (#657)', () => {
+  it('coerces the GATE_NOT_RUN string sentinel to 0 (honest skip, not a crash)', () => {
+    expect(normalizeGateExit('NOT_RUN')).toBe(0);
+  });
+
+  it('passes real numeric verdicts through unchanged', () => {
+    expect(normalizeGateExit(0)).toBe(0);
+    expect(normalizeGateExit(1)).toBe(1);
+    expect(normalizeGateExit(2)).toBe(2);
+  });
+
+  it('never returns a non-number for any input', () => {
+    for (const v of ['NOT_RUN', 0, 1, undefined, null]) {
+      expect(typeof normalizeGateExit(v)).toBe('number');
+    }
+  });
+});

package/lib/__tests__/shared-utils.test.js

@@ -109,4 +109,25 @@ describe('shared-utils', () => {
     });
   });
 
+  describe('copyHooks ships hook-module-resolution.sh (#655/#631)', () => {
+    it('@covers gate-script deployment: copyHooks installs hook-module-resolution.sh so pre-commit does not FATAL', () => {
+      // pre-commit loads hook-module-resolution.sh before any tier decision and
+      // fails closed. It was absent from copyHooks, so every fresh `init` /
+      // `setup-global` produced hooks that FATAL'd on every commit.
+      const pkgRoot = path.join(tmpRoot, 'node_modules', '@boyingliu01', 'xp-gate');
+      const hooksSrc = path.join(pkgRoot, 'hooks');
+      fs.mkdirSync(hooksSrc, { recursive: true });
+      for (const h of ['pre-commit', 'pre-push', 'post-commit', 'post-merge', 'hook-module-resolution.sh']) {
+        fs.writeFileSync(path.join(hooksSrc, h), `# ${h}`);
+      }
+      const destDir = path.join(tmpRoot, 'dest-hooks');
+      fs.mkdirSync(destDir, { recursive: true });
+
+      const { copyHooks } = require('../shared-utils');
+      copyHooks(pkgRoot, destDir);
+
+      expect(fs.existsSync(path.join(destDir, 'hook-module-resolution.sh'))).toBe(true);
+    });
+  });
+
 });

package/lib/__tests__/uninstall.test.js

@@ -309,6 +309,24 @@ describe('uninstall', () => {
     expect(cfg.uninstalled).toBeDefined();
   });
 
+  it('AC-02: uninstall in global mode unsets even when git returns a forward-slash path (#656)', async () => {
+    // Windows: git returns "C:/Users/.../hooks" while expectedPath uses
+    // backslashes. A raw === skipped the unset, leaving a dangling hooksPath.
+    setupGlobalInstall();
+    const expectedHooksPath = globalHooksDir();
+    mockExecGlobalHooksPath(expectedHooksPath.replace(/\\/g, '/'));
+    const { uninstall } = require('../uninstall');
+
+    const result = await uninstall([]);
+
+    expect(result).toBe(0);
+    expect(execSpy).toHaveBeenCalledWith(
+      expect.stringContaining('git config --global --unset core.hooksPath'),
+      expect.any(Object)
+    );
+    expect(warnSpy).not.toHaveBeenCalledWith(expect.stringContaining('does not match'));
+  });
+
   it('AC-02: uninstall in global mode skips unset when hooksPath does not match', async () => {
     setupGlobalInstall();
     mockExecGlobalHooksPath('/some/other/path');

package/lib/check.js

@@ -43,12 +43,31 @@ function parseCheckArgs(args) {
   };
 }
 
+/**
+ * Coerce a runGate() verdict into a valid process exit code (#657). The only
+ * non-numeric verdict is GATE_NOT_RUN (a string sentinel meaning "did not run");
+ * it must become 0, because a gate that honestly reported nothing to analyse is
+ * not a failure — and passing the string to process.exit() crashes Node.
+ * @param {number|string} verdict
+ * @returns {number}
+ */
+function normalizeGateExit(verdict) {
+  return typeof verdict === 'number' ? verdict : 0;
+}
+
 async function check(args) {
   const opts = parseCheckArgs(args);
 
   if (opts.list) return listGates();
   if (opts.changed) return runChangedBrief();
-  if (opts.gateNumber && !opts.all && !opts.gateIds) return runGate(opts.gateNumber, opts.pathArg);
+  // #657: runGate() can return the GATE_NOT_RUN sentinel (a string) when a gate
+  // has no module/fallback. Returning it straight to the caller let it reach
+  // process.exit(), which throws ERR_INVALID_ARG_TYPE and dumps a stack trace.
+  // Map it to exit 0 here (the gate honestly reported "did not run"); a real
+  // failure code passes through unchanged.
+  if (opts.gateNumber && !opts.all && !opts.gateIds) {
+    return normalizeGateExit(await runGate(opts.gateNumber, opts.pathArg));
+  }
   if (opts.all) return checkAllGates(opts.pathArg);
   if (opts.gateIds && opts.gateIds.length > 0) return runGatesByIds(opts.gateIds, opts.pathArg);
 
@@ -179,6 +198,7 @@ async function checkAllGates(target) {
 
 async function runGatesByIds(gateIds, target) {
   let failures = 0;
+  let skipped = 0;
   let ran = 0;
   const resolvedGates = [];
   const unknown = [];
@@ -201,8 +221,10 @@ async function runGatesByIds(gateIds, target) {
   }
 
   for (const resolvedId of resolvedGates) {
+    // #618/#657: a NOT_RUN sentinel is a skip, not a pass and not a failure.
     const code = await runGate(resolvedId, target);
-    if (code !== 0) failures += 1;
+    if (code === GATE_NOT_RUN) skipped += 1;
+    else if (code !== 0) failures += 1;
     ran += 1;
     console.log('');
   }
@@ -213,7 +235,8 @@ async function runGatesByIds(gateIds, target) {
     return 1;
   }
 
-  console.log(`[xp-gate check] ${ran - failures}/${ran} gate(s) passed`);
+  const tally = { passed: ran - failures - skipped, failed: failures, warned: 0, skipped, total: ran };
+  console.log(`[xp-gate check] ${formatGateTally(tally)}`);
   return failures === 0 ? 0 : 1;
 }
 
@@ -356,4 +379,4 @@ function listGates() {
   return 0;
 }
 
-module.exports = { check, checkAllGates, formatGateTally, runChangedBrief, fileExtension, printBriefSummary };
+module.exports = { check, checkAllGates, formatGateTally, runChangedBrief, fileExtension, printBriefSummary, normalizeGateExit };

package/lib/gate-runner.js

@@ -252,7 +252,12 @@ function runGateAdapter(scriptPath, cwd = process.cwd(), opts = {}) {
     execFileSync(
       bashCommand,
       ['--noprofile', '--norc', '-c', wrap, 'xp-gate', commonScript, nowMsScript, scriptPath, process.execPath, listFile],
-      { stdio: 'inherit', cwd }
+      // #660: the pre-commit hook exports GATE_DIR before sourcing a fragment, but
+      // the CLI path never did, so gate-3.sh looked for $GATE_DIR/lib/ccn-ratchet.sh
+      // (empty), fell back to <repo>/githooks/lib, and falsely BLOCKED on any
+      // project without vendored gates. The fragment lives beside its lib/, so
+      // point GATE_DIR at its own directory.
+      { stdio: 'inherit', cwd, env: { ...process.env, GATE_DIR: scriptDir } }
     );
   } finally {
     removeFileList(listFile);

package/lib/mattpocock-license-verdict.json

@@ -2,6 +2,6 @@
   "verdict": "PASS",
   "license_type": "MIT",
   "sha256": "0e7ac423bf2c6e223b7c5b156f8cf72da49d748e56a1641402c31f22ad07dbb5",
-  "checked_at": "2026-10-30T12:14:24.278Z",
+  "checked_at": "2026-10-30T13:23:44.380Z",
   "source": "network"
 }

package/lib/shared-utils.js

@@ -37,7 +37,12 @@ function copyHooks(srcDir, destDir) {
   // post-commit carries the audit-attribution flush and --no-verify detection.
   // It was absent here, so neither ever ran for installed users (the source
   // existed but no installer copied it).
-  ['pre-commit', 'pre-push', 'post-commit', 'post-merge'].forEach(hook => {
+  //
+  // hook-module-resolution.sh (#655/#631) is loaded by pre-commit BEFORE any tier
+  // decision and fails closed — omitting it made every commit FATAL on a fresh
+  // install. It is generated, but it must still ship beside the hooks.
+  const hookFiles = ['pre-commit', 'pre-push', 'post-commit', 'post-merge', 'hook-module-resolution.sh'];
+  hookFiles.forEach(hook => {
     const src = path.join(srcDir, 'hooks', hook);
     const dest = path.join(destDir, hook);
     if (fs.existsSync(src)) {

package/lib/uninstall.js

@@ -333,7 +333,11 @@ function unsetGitConfigIfMatch(item) {
     console.log(`  ${item.label} — not set`);
     return true;
   }
-  if (currentPath === item.expectedPath) {
+  // #656: git returns a forward-slash path ("C:/Users/.../hooks"), while
+  // item.expectedPath is built with path.join (backslashes on Windows). A raw
+  // === compares these as different and the unset is silently skipped, leaving
+  // a dangling core.hooksPath. Normalize both sides to an absolute, real path.
+  if (sameHooksPath(currentPath, item.expectedPath)) {
     unsetHooksPath();
     console.log(`  Unset ${item.label}`);
     return true;
@@ -342,6 +346,22 @@ function unsetGitConfigIfMatch(item) {
   return false;
 }
 
+/**
+ * True when two hook-directory paths refer to the same location (#656), tolerant
+ * of forward/back slashes, trailing separators, and case differences on Windows.
+ * @param {string} a
+ * @param {string} b
+ * @returns {boolean}
+ */
+function sameHooksPath(a, b) {
+  const norm = (p) => {
+    let r = path.resolve(p);
+    try { r = fs.realpathSync.native(r); } catch { /* path need not exist */ }
+    return process.platform === 'win32' ? r.toLowerCase() : r;
+  };
+  return norm(a) === norm(b);
+}
+
 /**
  * Execute a single plan item (file/dir/gitconfig).
  * @returns {boolean} true on success, false on skip/failure

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Mock layering policy enforcement — Gate M3 of pre-push hook. Ensures integration tests use real implementations for internal dependencies, mock external dependencies, and annotate pending mocks with removal plans. Combines project scope scanning, mock decision engine, and per-file validation into a single pipeline.

package/mutation/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MUTATION KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 **Gate M** (incremental mutation testing) + **Gate M2** helpers (test-layer detection used by `src/mock-policy/`). Pre-push quality gate. TypeScript-only; uses Stryker.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/xp-gate",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "bin/xp-gate.js"
@@ -10,6 +10,7 @@
     "hooks/",
     "adapters/",
     "lib/",
+    "gates/",
     "skills/",
     "plugins/",
     "dsh.bundle.patch.yml",

package/plugins/claude-code/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Claude Code. Includes 12 pre-commit gates (Gate 0-11), 8 pre-push gates, Sprint Flow (6 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/claude-code/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/claude-code/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/claude-code/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/dsh/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/dsh-plugin-xp-gate",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "description": "XP-Gate deterministic quality gates + AI workflow skills as a DeepSeek Harness (DSH) native plugin: gate-check / gate-principles / gate-arch tools + 12 SKILL.md skills",
   "type": "module",
   "main": "./lib/index.js",

package/plugins/dsh/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/dsh/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/dsh/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/opencode/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/opencode-plugin",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "type": "module",
   "main": "index.ts",
   "description": "XP-Gate quality gates + AI workflow skills + Sprint Flow TUI sidebar for OpenCode",

package/plugins/opencode/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/opencode/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/opencode/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/qoder/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Qoder. Includes 12 pre-commit gates (Gate 0-11), 8 pre-push gates, Sprint Flow (6 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/qoder/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/qoder/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/qoder/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/principles/AGENTS.md

@@ -1,9 +1,9 @@
 # PRINCIPLES CHECKER MODULE
 
 **Generated:** 2026-10-30
-**Commit:** 3c5e9aaf
+**Commit:** b45021b6
 **Branch:** main
-**Version:** 0.33.0.0
+**Version:** 0.33.2.0
 
 ## OVERVIEW
 Clean Code & SOLID principles checker — **Gate 4** of pre-commit. 14 rules × 9 language adapters, SARIF 2.1.0 output. Houses the **Boy Scout Rule** enforcement engine (Gate 6) and warning-baseline storage.