@boyingliu01/xp-gate 0.28.0 → 0.29.0

package/hooks/pre-commit

@@ -285,6 +285,104 @@ gate_start_ms() {
   now_ms
 }
 
+# ---------------------------------------------------------------------------
+# #537: per-language coverage classification.
+#
+# A gate that SKIPs tells the reader nothing on its own — "documentation
+# project", "tool not installed" and "nothing changed" are wildly different
+# facts that used to collapse into one word. A PowerShell-primary repo could
+# therefore look fully covered while 3.3k lines were measured by nothing.
+#
+# classify_coverage_reason REASON GATE_STATUS
+#   -> covered | not_applicable | no_changes | tool_missing | unknown
+#
+# FAIL CLOSED: an unrecognised reason is `unknown`, never `covered`. `covered`
+# is reachable ONLY by a gate that actually passed (PASS / WARN /
+# PASS_WITH_WARNINGS) with no skip reason attached.
+# ---------------------------------------------------------------------------
+classify_coverage_reason() {
+  local reason="${1:-}"
+  local gate_status="${2:-}"
+
+  case "$reason" in
+    documentation_project)        echo "not_applicable" ;;
+    architecture_skip_exemption)  echo "not_applicable" ;;
+    no_source_files_changed)      echo "no_changes" ;;
+    # Gate 4 names the same concept "no_matching_files_changed"; Gate 3/6 use
+    # the shorter form. Both mean "the tool exists, nothing to look at".
+    no_matching_files_changed)    echo "no_changes" ;;
+    tool_missing|tool_not_installed|no_powershell_tool|no_archlint_detector|checker_not_found)
+                                  echo "tool_missing" ;;
+    # archlint / import-linter / arch-go / Java arch tests exist as concepts but
+    # this repo carries no config for the language, so nothing was measured.
+    architecture_not_configured)  echo "tool_missing" ;;
+    # Gate 4's checker ran but crashed. Not covered, not "tool missing" — the
+    # measurement was attempted and failed, so the honest answer is unknown.
+    checker_execution_error)      echo "unknown" ;;
+    "")
+      case "$gate_status" in
+        PASS|WARN|PASS_WITH_WARNINGS|PASSED) echo "covered" ;;
+        *) echo "unknown" ;;
+      esac
+      ;;
+    *) echo "unknown" ;;
+  esac
+}
+
+# #537: record where the language key set came from. PROJECT_LANGS is derived
+# from the CURRENTLY CHANGED files, not from a static inventory of the repo, so
+# a docs-only commit legitimately yields an empty set. Reporting the matrix
+# without this provenance would repeat the sin the matrix exists to fix.
+resolve_languages_source() {
+  local override_langs="${1:-}"
+  local config_langs="${2:-}"
+  local resolved_langs="${3:-}"
+  if [ -n "$override_langs" ]; then
+    echo "override"
+  elif [ -n "$config_langs" ]; then
+    echo "config"
+  elif [ -n "$resolved_langs" ]; then
+    echo "changed_files"
+  else
+    echo "none"
+  fi
+}
+
+# #537: resolve Gate 6's reason for ONE language.
+#
+# Reads the flattened GATE_6_LANG_REASONS ("lang=reason,lang=reason") because
+# the associate array that produced it is function-local to the gate and
+# invisible to generate_quality_report().
+#
+# The scalar GATE_6_REASON is a SUMMARY across languages and must NOT be used
+# as a per-language fallback when per-language detail exists: with langs
+# "typescript iac" where only iac lacks a detector, the scalar reads
+# no_archlint_detector and would relabel typescript — the exact language
+# archlint had just measured. A real probe run exposed this:
+#   langs=[typescript iac] reasons=[iac=no_archlint_detector]
+#
+# So: an explicit pair wins; otherwise there is no skip reason for this
+# language, and "no reason" + PASS status is precisely what `covered` means.
+# The scalar only speaks for every language when there is no detail at all
+# (config-missing / bash<4 paths).
+coverage_g6_reason_for() {
+  local lang="${1:-}"
+  local detail="${GATE_6_LANG_REASONS:-}"
+  local scalar="${GATE_6_REASON:-}"
+  local pairs=",${detail},"
+  local reason=""
+  case "$pairs" in
+    *",${lang}="*)
+      reason="${pairs#*",${lang}="}"
+      reason="${reason%%,*}"
+      ;;
+  esac
+  if [ -z "$reason" ] && [ -z "$detail" ] && [ -n "$scalar" ]; then
+    reason="$scalar"
+  fi
+  echo "$reason"
+}
+
 # Audit helper — records gate execution to .xp-gate/audit.jsonl
 # NEVER blocks commit on failure (try/catch around npx tsx).
 # Usage: record_gate_audit <gate_id> <gate_name> <passed> <issues_found> <start_ms>
@@ -2902,6 +3000,15 @@ else
 
   # Check if architecture config exists (use $PROJECT_ROOT — hook may have cd'd to a subdir via ARCH-03)
   if [ -f "$PROJECT_ROOT/architecture.yaml" ] || [ -f "$PROJECT_ROOT/.architecturerc" ] || [ -f "$PROJECT_ROOT/.archlint.yaml" ]; then
+    # #537: per-language reason bookkeeping. Gate 3/4 are single-language gates
+    # and get away with one scalar reason; Gate 6 loops, so a scalar would lose
+    # information ("TS passed, PS has no detector" cannot be one string).
+    # Bash 4 associative array, with a scalar aggregate for the legacy field.
+    GATE_6_REASON=""
+    GATE_6_LANG_REASONS=""
+    if [ "${BASH_VERSINFO[0]:-0}" -ge 4 ]; then
+      declare -A GATE_6_REASON_BY_LANG=()
+    fi
     # Different architecture tools per language
     for CURRENT_LANG in $PROJECT_LANGS; do
     case "$CURRENT_LANG" in
@@ -2947,6 +3054,8 @@ else
          else
            echo "     ✅ TypeScript architecture validation completed (zero-tolerance mode). no baseline found."
          fi
+         # #537: archlint actually ran here, so there is no skip reason. The
+         # matrix reads an absent reason + PASS status as `covered`.
          ;;
       "python")
         # detect any import-linter carrier (incl. pyproject.toml [tool.importlinter])
@@ -2972,6 +3081,12 @@ else
         else
           echo "     ℹ️  No import-linter config found (.import-linter.yml/.importlinter/[tool.importlinter]) - skipping"
           echo "     ⏭️  SKIPPED - Python architecture validation (no config)"
+          # #537: no config means Python architecture was NOT measured. Without
+          # this the matrix would call it `covered` simply because the branch
+          # recorded nothing — the exact false-confidence the matrix prevents.
+          if [ "${BASH_VERSINFO[0]:-0}" -ge 4 ]; then
+            GATE_6_REASON_BY_LANG["$CURRENT_LANG"]="architecture_not_configured"
+          fi
         fi
         ;;
       "go")
@@ -2988,6 +3103,10 @@ else
         else
           echo "     ℹ️  No arch-go.yaml found - skipping Go architecture"
           echo "     ⏭️  SKIPPED - Go architecture validation (no config)"
+          # #537: same reasoning as the Python branch above.
+          if [ "${BASH_VERSINFO[0]:-0}" -ge 4 ]; then
+            GATE_6_REASON_BY_LANG["$CURRENT_LANG"]="architecture_not_configured"
+          fi
         fi
         ;;
       "java")
@@ -3004,19 +3123,63 @@ else
         else
           echo "     ℹ️  No architecture test files found - skipping"
           echo "     ⏭️  SKIPPED - Architecture validation (no tests)"
+          # #537: no test files means Java architecture was NOT measured.
+          if [ "${BASH_VERSINFO[0]:-0}" -ge 4 ]; then
+            GATE_6_REASON_BY_LANG["$CURRENT_LANG"]="architecture_not_configured"
+          fi
         fi
         ;;
       *)
         echo "     ℹ️  Architecture validation not configured for $CURRENT_LANG"
         echo "     ⏭️  SKIPPED - Architecture validation (not configured for $CURRENT_LANG)"
+        # #537: archlint ships no detector for this language. Say so in a way
+        # the quality report can read, instead of leaving the dimension silent.
+        if [ "${BASH_VERSINFO[0]:-0}" -ge 4 ]; then
+          GATE_6_REASON_BY_LANG["$CURRENT_LANG"]="no_archlint_detector"
+        fi
         ;;
     esac
     done  # end for CURRENT_LANG (architecture)
+
+    # #537: fold the per-language reasons into the legacy scalar. Homogeneous
+    # reasons survive as-is; a genuine mix becomes the `mixed` sentinel, which
+    # classify_coverage_reason() maps to `unknown` (fail closed) rather than
+    # pretending one language's verdict covers all of them.
+    #
+    # `declare -A` inside a function is FUNCTION-LOCAL in bash, and the report
+    # is written by generate_quality_report() — a different function. So the
+    # assoc array is flattened into this global string, which the report parses.
+    # (Found by running the hook: the matrix reported tool_missing for
+    # typescript because the lookup saw an empty array.)
+    GATE_6_LANG_REASONS=""
+    _g6_seen=""
+    _g6_mixed=0
+    for _g6_lang in $PROJECT_LANGS; do
+      _g6_r=""
+      if [ "${BASH_VERSINFO[0]:-0}" -ge 4 ]; then
+        _g6_r="${GATE_6_REASON_BY_LANG[$_g6_lang]:-}"
+      fi
+      if [ -n "$_g6_r" ]; then
+        GATE_6_LANG_REASONS="${GATE_6_LANG_REASONS:+$GATE_6_LANG_REASONS,}${_g6_lang}=${_g6_r}"
+        if [ -z "$_g6_seen" ]; then
+          _g6_seen="$_g6_r"
+        elif [ "$_g6_seen" != "$_g6_r" ]; then
+          _g6_mixed=1
+        fi
+      fi
+    done
+    if [ "$_g6_mixed" = "1" ]; then
+      GATE_6_REASON="mixed"
+    else
+      GATE_6_REASON="$_g6_seen"
+    fi
   else
     # Architecture config is REQUIRED — BLOCK if missing
     if [ -f "$PROJECT_ROOT/.architecture-skip" ]; then
       echo "     ⚠️  Architecture config missing but .architecture-skip present — allowed to skip"
       echo "     ⏭️  SKIPPED - Architecture validation (.architecture-skip exemption)"
+      # #537: an explicit exemption is "not applicable", not "tool missing".
+      GATE_6_REASON="architecture_skip_exemption"
     else
       echo ""
       echo "❌ ARCHITECTURE CONFIG MISSING - COMMIT BLOCKED"
@@ -3563,6 +3726,37 @@ generate_quality_report() {
   local REPORT_FILE="${STATUS_DIR}/${BRANCH}.json"
   mkdir -p "$(dirname "$REPORT_FILE")"
 
+  # ── #537: per-language coverage matrix ───────────────────────────────────────
+  # Answers "was this dimension actually measured for this language?" instead of
+  # letting every SKIP read as a pass. Built by whitelist splicing (the same
+  # technique as DEGRADED_JSON above) rather than jq, because jq is an OPTIONAL
+  # dependency here (see the `command -v jq` guard earlier in this file) and a
+  # missing jq must not silently cost us the whole matrix.
+  #
+  # Safety: every spliced value is one of the five enum literals below — no
+  # quotes, backslashes or newlines can reach the JSON. Language names are
+  # whitelisted too, and a name that fails the whitelist is reported in
+  # coverageWarnings instead of being silently dropped.
+  local COVERAGE_MATRIX_JSON=""
+  local COVERAGE_WARNINGS_JSON=""
+  local _cm_lang _cm_g3 _cm_g4 _cm_g6 _cm_entry
+  for _cm_lang in $PROJECT_LANGS; do
+    if ! echo "$_cm_lang" | grep -qE '^[a-z][a-z0-9+]*$'; then
+      COVERAGE_WARNINGS_JSON="${COVERAGE_WARNINGS_JSON:+$COVERAGE_WARNINGS_JSON, }\"unsupported_language_name\""
+      continue
+    fi
+    _cm_g3=$(classify_coverage_reason "${GATE_3_REASON:-}" "${GATE_3_STATUS:-PASS}")
+    _cm_g4=$(classify_coverage_reason "${GATE_4_REASON:-}" "${GATE_4_STATUS:-PASS}")
+    # Gate 6 reason for this language — extracted so the rule is testable
+    # rather than inlined (see coverage_g6_reason_for below).
+    _cm_g6_reason=$(coverage_g6_reason_for "$_cm_lang")
+    _cm_g6=$(classify_coverage_reason "$_cm_g6_reason" "${GATE_6_STATUS:-PASS}")
+    _cm_entry="\"${_cm_lang}\": {\"gate3_complexity\": \"${_cm_g3}\", \"gate4_principles\": \"${_cm_g4}\", \"gate6_architecture\": \"${_cm_g6}\"}"
+    COVERAGE_MATRIX_JSON="${COVERAGE_MATRIX_JSON:+$COVERAGE_MATRIX_JSON, }${_cm_entry}"
+  done
+  local LANGUAGES_SOURCE
+  LANGUAGES_SOURCE=$(resolve_languages_source "${_RESOLVED_LANG_OVERRIDE:-}" "${_CONFIG_LANGS:-}" "${PROJECT_LANGS:-}")
+
   cat > "$REPORT_FILE" << ENDJSON
 {
   "reportVersion": "2.0",
@@ -3581,6 +3775,9 @@ generate_quality_report() {
     "score": $SCORE_JSON,
     "verdict": "$VERDICT"
   },
+  "languagesSource": "${LANGUAGES_SOURCE}",
+  "coverageMatrix": {${COVERAGE_MATRIX_JSON}},
+  "coverageWarnings": [${COVERAGE_WARNINGS_JSON}],
   "gates": {
     "gate1_static_analysis": {
       "name": "Code Quality (Static + Lint + Shell)",
@@ -3616,6 +3813,7 @@ generate_quality_report() {
     "gate6_arch_boyscout": {
       "name": "Architecture + Boy Scout Rule",
       "status": "${GATE_6_STATUS:-PASS}",
+      "reason": "${GATE_6_REASON:-}",
       "boyScoutPassed": $BS_PASSED,
       "boyScoutBlocked": $BS_BLOCKED
     },

package/hooks/pre-push

@@ -153,8 +153,15 @@ fi
 DOC_ONLY=true
 SOURCE_EXTENSIONS="\.py$|\.js$|\.cjs$|\.mjs$|\.ts$|\.tsx$|\.java$|\.go$|\.rs$|\.cpp$|\.c$|\.swift$|\.kt$|\.sh$|\.dart$|\.ps1$|\.m$|\.mm$|\.h$|\.hpp$"
 
+# Extensionless hook scripts and test files are CODE, not documentation.
+# Measured consequence of the old list: a commit touching githooks/pre-push and
+# its .bats suite was classified "Documentation-only push (no source code
+# files)" and the walkthrough was skipped — the single most safety-critical
+# script in the repo, exempt from review by a filename-shape accident.
+NON_EXT_SOURCE_RE='^(githooks/(pre-commit|pre-push|post-merge|install\.sh|verify\.sh)|[^/]+\.bats)$|/__tests__/'
+
 for file in $PUSHED_FILES; do
-  if echo "$file" | grep -qE "$SOURCE_EXTENSIONS"; then
+  if echo "$file" | grep -qE "$SOURCE_EXTENSIONS" || echo "$file" | grep -qE "$NON_EXT_SOURCE_RE"; then
     DOC_ONLY=false
     break
   fi
@@ -954,6 +961,7 @@ else
 # GATE MW: VALIDATE CODE WALKTHROUGH RESULT FILE
 # ============================================================================
 RESULT_FILE=".code-walkthrough-result.json"
+MW_SKIPPED=0
 
 # Skipped on main/master by design, exactly like Gate ML and Gate UI above.
 # Work is already walkthrough-reviewed on the feature branch before it lands, so
@@ -962,6 +970,7 @@ RESULT_FILE=".code-walkthrough-result.json"
 # the release commit itself changes nothing anyone reviewed. githooks/AGENTS.md
 # documented this skip; the code did not implement it (drift fixed here).
 if [ "$CURRENT_BRANCH" = "main" ] || [ "$CURRENT_BRANCH" = "master" ]; then
+  MW_SKIPPED=1
   echo ""
   echo "   ℹ️  Gate MW skipped (pushing to ${CURRENT_BRANCH} — pre-reviewed via branch walkthrough)"
   echo ""
@@ -1038,13 +1047,20 @@ fi
 # ============================================================================
 echo ""
 echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+# Say what actually happened. Printing "VERIFIED" on a skipped gate would be a
+# claim the hook cannot support — the same dishonesty (evidence asserting an
+# unperformed check) that Gate MW exists to prevent.
+if [ "$MW_SKIPPED" = "1" ]; then
+  echo "   ℹ️  GATE MW: CODE WALKTHROUGH SKIPPED (main-branch push)"
+else
   echo "   ✅ GATE MW: CODE WALKTHROUGH VERIFIED"
+fi
 echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
 echo ""
 echo "Branch: $CURRENT_BRANCH"
 echo "Files changed: $FILES_CHANGED"
 echo "Lines: +$LINES_ADDED -$LINES_DELETED"
-if [ -f "$RESULT_FILE" ]; then
+if [ "$MW_SKIPPED" != "1" ] && [ -f "$RESULT_FILE" ]; then
   echo ""
   echo "Code walkthrough result:"
   echo "  Commit: $EXPECTED_COMMIT"

package/lib/mattpocock-license-verdict.json

@@ -2,6 +2,6 @@
   "verdict": "PASS",
   "license_type": "MIT",
   "sha256": "0e7ac423bf2c6e223b7c5b156f8cf72da49d748e56a1641402c31f22ad07dbb5",
-  "checked_at": "2026-10-19T12:19:52.510Z",
+  "checked_at": "2026-10-19T16:16:19.910Z",
   "source": "network"
 }

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
 **Generated:** 2026-10-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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.28.0",
+  "version": "0.29.0",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "./bin/xp-gate.js"

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

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.28.0",
+  "version": "0.29.0",
   "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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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.28.0",
+  "version": "0.29.0",
   "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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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.28.0",
+  "version": "0.29.0",
   "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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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.28.0",
+  "version": "0.29.0",
   "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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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.

package/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.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/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-19
-**Commit:** 07d3036f
+**Commit:** 88bfe6a6
 **Branch:** main
-**Version:** 0.28.0.0
+**Version:** 0.29.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.