@boyingliu01/xp-gate 0.19.2 → 0.20.0

package/hooks/pre-commit

@@ -115,6 +115,30 @@ set -o pipefail
 # Helper Functions
 # ============================================================================
 
+# Run a TypeScript entrypoint under tsx without routing arguments through the
+# npx shim: Windows cmd.exe caps command lines at 8191 chars and long staged
+# file lists (Boy Scout, build integrity) overflow it, turning a healthy run
+# into a false BLOCK. Prefer the repo-local tsx CLI driven by node directly;
+# keep npx as fallback for consumer repos relying on a global install.
+# RUN_TSX_TIMEOUT (e.g. "120s") wraps the invocation in `timeout`.
+if ! declare -F run_tsx >/dev/null 2>&1; then
+  run_tsx() {
+    local _root="${PROJECT_ROOT:-$(pwd)}" _timeout=()
+    if [ -n "${RUN_TSX_TIMEOUT:-}" ]; then
+      _timeout=(timeout "$RUN_TSX_TIMEOUT")
+    fi
+    if [ -f "$_root/node_modules/tsx/dist/cli.mjs" ] && command -v node >/dev/null 2>&1; then
+      "${_timeout[@]}" node "$_root/node_modules/tsx/dist/cli.mjs" "$@"
+    else
+      "${_timeout[@]}" npx tsx "$@"
+    fi
+  }
+fi
+
+# Gate 5's vitest exit-code judgement (is_runner_infrastructure_error,
+# is_partial_threshold_only_exit, handle_test_failure) is sourced from
+# lib/test-failure.sh below, next to the other hook libraries.
+
 # Parse LCOV file and return coverage percentage
 # Usage: parse_lcov_coverage <lcov_file>
 # Returns: coverage percentage (e.g., "85.5")
@@ -190,6 +214,16 @@ PROJECT_ROOT_FOR_AUDIT="$(git rev-parse --show-toplevel 2>/dev/null || echo "$AU
 # Issue #370: source cross-platform now_ms() helper (node-preferred, regex-validated)
 source "${AUDIT_SCRIPT_DIR}/lib/now-ms.sh"
 
+# Issue #473 (+ #454): Gate 5's vitest exit-code judgement, sourced as a library
+# so the fail-closed contract is unit-testable without executing this hook.
+source "${AUDIT_SCRIPT_DIR}/lib/test-failure.sh"
+
+# Issue #477: Gate 2's jscpd config probe and outcome classification, same reason.
+source "${AUDIT_SCRIPT_DIR}/lib/jscpd-run.sh"
+
+# Issue #476: Gate 11's sprint-gate.sh resolution chain and decision translation.
+source "${AUDIT_SCRIPT_DIR}/lib/sprint-gate-report.sh"
+
 # Read max_duration_ms from .xp-gate-config.json (default 7200000 = 2 hours)
 _MAX_DURATION_MS=$(node -e "try{const c=JSON.parse(require('fs').readFileSync('${PROJECT_ROOT_FOR_AUDIT}/.xp-gate-config.json','utf8'));console.log((c.audit&&c.audit.max_duration_ms)||7200000);}catch(e){console.log(7200000);}" 2>/dev/null || echo "7200000")
 
@@ -254,7 +288,7 @@ echo ""
 GATE_0_START=$(gate_start_ms)
 GATE_0_STATUS="PASS"
 
-CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD)
+CURRENT_BRANCH=$(git symbolic-ref --short HEAD 2>/dev/null || git rev-parse --abbrev-ref HEAD 2>/dev/null || echo "HEAD")
 PROTECTED_BRANCHES="master develop trunk mainline"
 ROOT_DIR=$(git rev-parse --show-toplevel 2>/dev/null || echo ".")
 
@@ -402,7 +436,11 @@ Before committing:
   2. Add entry to CHANGELOG.md
   3. Run: node scripts/sync-version.cjs
 
-Or use --no-verify (discouraged) or include [skip-version-check] in commit message.
+Or use the bypass prefix — the commit message FIRST LINE must START with:
+  [skip-version-check] chore: <description>
+Allowed types: chore: docs: release:. The bypass only permits build
+tooling changes (adapters/, scripts/, hooks/); production source files
+still require a VERSION/CHANGELOG update.
 
 Staged files: $(echo "$STAGED_FILES" | tr '\n' ', ' | sed 's/,$//')
 "
@@ -429,6 +467,22 @@ if [ -z "$CHANGED_FILES" ]; then
   exit 0
 fi
 
+# Gate 5 test selection (#494): vitest.config.ts excludes the bundled mirror
+# subtrees and the skills/plugins source trees, so a path under one of them can
+# never be collected. When such a path is the only target, vitest exits 1 with
+# "No test files found" and Gate 5 reports a failing test run -- a commit that
+# touched nothing but a byte-identical mirror cannot land. Map each mirror path
+# to the canonical twin it copies, drop what has no twin, collapse duplicates.
+select_root_suite_targets() {
+  # Deliberately not `s|...|...|`: the alternation below contains `|`, which sed
+  # would read as the end of its pattern ("unknown option to `s'").
+  printf '%s\n' "$1" \
+    | sed -E 's#^src/npm-package/(principles|mutation|mock-policy|build-integrity)/#src/\1/#' \
+    | grep -vE '^(src/_wip/|plugins/|skills/|src/npm-package/(plugins|skills|principles|mutation|mock-policy|build-integrity)/)' \
+    | sed '/^$/d' \
+    | sort -u
+}
+
 any_changed_files_match() {
   local pattern
 
@@ -833,72 +887,16 @@ else
     echo "   ── Checking $CURRENT_LANG ──"
     case "$CURRENT_LANG" in
       typescript)
-  # TypeScript: static analysis (tsc) + linting (eslint)
-  if ! command -v npx &> /dev/null; then
-    echo "ℹ️  npx not available — skipping TypeScript checks"
-    echo "⏭️  SKIPPED - Code quality (npx not available)"
-  else
-    # Verify tsc is real TypeScript compiler (npx may fetch deprecated tsc@2.0.4 placeholder)
-    TSC_OUTPUT=$(npx tsc --version 2>&1)
-    if echo "$TSC_OUTPUT" | grep -q "Version"; then
-      # Only run tsc if tsconfig.json exists (without it tsc prints help and exits non-zero)
-      if [ -f "tsconfig.json" ]; then
-        # Run TypeScript type checking
-        echo "Running TypeScript static analysis ($TSC_OUTPUT)..."
-        npx tsc --noEmit --skipLibCheck 2>&1 | head -30
-        TSC_EXIT=$?
-        if [ "$TSC_EXIT" -ne 0 ]; then
-          echo ""
-          echo "❌ BLOCKED - TYPE ERRORS detected"
-          echo "Fix the type errors above before committing."
-          exit 1
-        fi
-        echo "✅ PASSED - TypeScript static analysis."
-        
-        # Also check test files if they exist (Issue #293)
-        # Many projects exclude __tests__/ from tsconfig.json, so test file type
-        # errors silently accumulate.
-        if [ -d "src/__tests__" ] || [ -d "src/tests" ] || [ -d "tests" ] || [ -d "__tests__" ]; then
-          HAS_TESTS_IN_TSC=false
-          if npx tsc --noEmit --skipLibCheck --listFiles 2>/dev/null | grep -qE '__tests__/|\.test\.ts|\.spec\.ts'; then
-            HAS_TESTS_IN_TSC=true
-          fi
-          if [ "$HAS_TESTS_IN_TSC" = false ]; then
-            TSC_TEST_EXIT=0
-            if [ -f "tsconfig.tests.json" ]; then
-              echo "Checking test files with tsconfig.tests.json..."
-              npx tsc --noEmit --project tsconfig.tests.json 2>&1 | head -30
-              TSC_TEST_EXIT=$?
-            else
-              echo "Checking test files for type errors..."
-              TEMP_TSCONFIG=".tsconfig.withtests.json"
-              node -e "
-                const cfg = JSON.parse(require('fs').readFileSync('tsconfig.json','utf8'));
-                delete cfg.exclude;
-                cfg.include = cfg.include || ['src/**/*'];
-                cfg.include.push('src/**/__tests__/**','src/**/*.test.ts','src/**/*.spec.ts');
-                require('fs').writeFileSync('$TEMP_TSCONFIG', JSON.stringify(cfg, null, 2));
-              " 2>/dev/null
-              npx tsc --noEmit --project "$TEMP_TSCONFIG" --skipLibCheck 2>&1 | head -30
-              TSC_TEST_EXIT=$?
-              rm -f "$TEMP_TSCONFIG"
-            fi
-            if [ $TSC_TEST_EXIT -ne 0 ]; then
-              echo ""
-              echo "❌ BLOCKED - TYPE ERRORS in test files"
-              echo "Your project's tsconfig.json excludes test files from type checking."
-              echo "Add a tsconfig.tests.json to customise test file checking."
-              exit 1
-            fi
-          fi
-        fi
-      else
-        echo "ℹ️  SKIP - tsconfig.json not found (no TypeScript project config)"
-      fi
-    else
-      echo "ℹ️  SKIP - tsc not properly installed ($TSC_OUTPUT)"
-      echo "Install typescript (npm install typescript --save-dev) for type checking."
-    fi
+  # TypeScript: static analysis + linting (eslint). The type-check decision lives
+  # in githooks/lib/typecheck.sh so a project's own checker (vue-tsc, svelte-check,
+  # a `typecheck` script) is used instead of a hardcoded tsc that cannot resolve
+  # single-file components (#436).
+  source "${AUDIT_SCRIPT_DIR}/lib/typecheck.sh"
+  run_typescript_typecheck
+  TS_CHECK_EXIT=$?
+  if [ "$TS_CHECK_EXIT" -eq 1 ]; then
+    exit 1
+  fi
     
     # Run ESLint linting if config exists
     if [ -f ".eslintrc.json" ] || [ -f ".eslintrc.js" ] || [ -f ".eslintrc.cjs" ] || [ -f "eslint.config.js" ]; then
@@ -966,16 +964,30 @@ else
     # ── Biome lint/format check ──
     if [ -f "biome.json" ] || [ -f "biome.jsonc" ]; then
       echo "Running Biome lint/format check..."
-      if ! npx biome check --staged . 2>/dev/null; then
-        npx biome check . 2>&1 | head -50
-        echo ""
-        echo "❌ BLOCKED - Biome check failed"
-        echo "Fix the errors above before committing."
-        echo "Tip: Run 'npx biome check --write .' to auto-fix formatting and safe lint issues."
-        echo "     Run 'npx biome check --write --unsafe .' to also fix unsafe lint issues."
-        exit 1
+      BIOME_OUT=$(npx biome check --staged . 2>&1)
+      BIOME_STATUS=$?
+      if [ "$BIOME_STATUS" -ne 0 ]; then
+        # Biome exits 1 with `internalError/io - No files were processed` when the
+        # staged set yields nothing lintable (docs-only / config-only commit, or
+        # everything excluded by biome.json). That is "nothing to do", not a lint
+        # violation -- blocking on it makes documentation uncommittable (#458).
+        # This matches the hook's stated principle: tool available + check
+        # SUCCEEDS = PASS. Real violations still carry diagnostics, so they block.
+        if echo "$BIOME_OUT" | grep -qE 'internalError/io|No files were processed'; then
+          echo "$BIOME_OUT"
+          echo "✅ PASSED - Biome check (no lintable files staged)."
+        else
+          echo "$BIOME_OUT" | head -50
+          echo ""
+          echo "❌ BLOCKED - Biome check failed"
+          echo "Fix the errors above before committing."
+          echo "Tip: Run 'npx biome check --write .' to auto-fix formatting and safe lint issues."
+          echo "     Run 'npx biome check --write --unsafe .' to also fix unsafe lint issues."
+          exit 1
+        fi
+      else
+        echo "✅ PASSED - Biome check."
       fi
-      echo "✅ PASSED - Biome check."
     else
       if [ -f "package.json" ] && command -v node &> /dev/null; then
         HAS_BIOME=$(node -e "try{const p=JSON.parse(require('fs').readFileSync('package.json','utf8'));const d=p.devDependencies||{};const deps=p.dependencies||{};console.log((d['@biomejs/biome']||deps['@biomejs/biome'])?'yes':'no')}catch{console.log('no')}") && true
@@ -1002,7 +1014,6 @@ else
         exit 1
       fi
     fi
-  fi
     ;;
     
     python)
@@ -1013,8 +1024,22 @@ else
   else
     # Run Ruff (includes syntax check, linting)
     echo "Running Ruff linting (syntax + lint)..."
-    RUFF_OUTPUT=$(ruff check "$CHANGED_FILES" --output-format json 2>&1)
-    RUFF_EXIT=$?
+    # Filter to *.py/*.pyi into a bash array so each staged path is passed as
+    # ONE argument even when it contains whitespace; skip ruff when no Python
+    # files are staged so yaml/.gitignore are never parsed.
+    RUFF_ARGS=()
+    while IFS= read -r staged_file; do
+      case "$staged_file" in
+        *.py|*.pyi) RUFF_ARGS+=("$staged_file") ;;
+      esac
+    done <<< "$CHANGED_FILES"
+    if [ "${#RUFF_ARGS[@]}" -eq 0 ]; then
+      RUFF_OUTPUT="[]"
+      RUFF_EXIT=0
+    else
+      RUFF_OUTPUT=$(ruff check "${RUFF_ARGS[@]}" --output-format json 2>&1)
+      RUFF_EXIT=$?
+    fi
     if [ "$RUFF_EXIT" -ne 0 ]; then
       if [ -f ".xp-gate/lint-baseline.json" ] && command -v node &> /dev/null; then
         # Get baseline warnings for changed Python files
@@ -1070,9 +1095,14 @@ else
 
     # Ruff format check (formatting gate)
     if command -v ruff &> /dev/null; then
-      PY_FMT_FILES=$(echo "$CHANGED_FILES" | grep '\.py$' || true)
-      if [ -n "$PY_FMT_FILES" ]; then
-        RUFF_FMT_OUTPUT=$(ruff format --check $PY_FMT_FILES 2>&1)
+      FMT_ARGS=()
+      while IFS= read -r staged_file; do
+        case "$staged_file" in
+          *.py|*.pyi) FMT_ARGS+=("$staged_file") ;;
+        esac
+      done <<< "$CHANGED_FILES"
+      if [ "${#FMT_ARGS[@]}" -gt 0 ]; then
+        RUFF_FMT_OUTPUT=$(ruff format --check "${FMT_ARGS[@]}" 2>&1)
         RUFF_FMT_EXIT=$?
         if [ "$RUFF_FMT_EXIT" -ne 0 ]; then
           echo "$RUFF_FMT_OUTPUT" | head -20
@@ -1086,9 +1116,14 @@ else
     fi
 
     # Debug statement detection (breakpoint/pdb/ipdb)
-    PY_DEBUG_FILES=$(echo "$CHANGED_FILES" | grep '\.py$' || true)
-    if [ -n "$PY_DEBUG_FILES" ]; then
-      DEBUG_HITS=$(grep -n 'breakpoint()\|pdb\.set_trace()\|ipdb\.set_trace()\|import pdb\|import ipdb' $PY_DEBUG_FILES 2>/dev/null || true)
+    DEBUG_ARGS=()
+    while IFS= read -r staged_file; do
+      case "$staged_file" in
+        *.py|*.pyi) DEBUG_ARGS+=("$staged_file") ;;
+      esac
+    done <<< "$CHANGED_FILES"
+    if [ "${#DEBUG_ARGS[@]}" -gt 0 ]; then
+      DEBUG_HITS=$(grep -n 'breakpoint()\|pdb\.set_trace()\|ipdb\.set_trace()\|import pdb\|import ipdb' "${DEBUG_ARGS[@]}" 2>/dev/null || true)
       if [ -n "$DEBUG_HITS" ]; then
         echo ""
         echo "❌ BLOCKED - Debug statements detected in Python files:"
@@ -1101,9 +1136,19 @@ else
     
     # Optional: Type checking with mypy
     if command -v mypy &> /dev/null; then
-      echo "Running mypy type checking (optional)..."
-      mypy --ignore-missing-imports $(echo "$CHANGED_FILES" | grep "\.py$" || echo ".") 2>&1 | head -20 | tail -10
-      MYPY_EXIT=$?
+      MYPY_ARGS=()
+      while IFS= read -r staged_file; do
+        case "$staged_file" in
+          *.py|*.pyi) MYPY_ARGS+=("$staged_file") ;;
+        esac
+      done <<< "$CHANGED_FILES"
+      if [ "${#MYPY_ARGS[@]}" -eq 0 ]; then
+        MYPY_EXIT=0
+      else
+        echo "Running mypy type checking (optional)..."
+        mypy --ignore-missing-imports "${MYPY_ARGS[@]}" 2>&1 | head -20 | tail -10
+        MYPY_EXIT=$?
+      fi
       if [ "$MYPY_EXIT" -ne 0 ]; then
         echo ""
         echo "❌ BLOCKED - MYPI TYPE ERRORS detected"
@@ -1253,6 +1298,7 @@ record_gate_audit "gate-1" "code-quality" "$GATE_1_STATUS" "0" "$GATE_1_START"
  2>&1 echo ""
  2>&1 echo "→ Gate 2: Duplicate code detection..."
 GATE_2_START=$(gate_start_ms)
+GATE_2_STATUS="PASS"
 
 if [ "$PROJECT_LANG" = "documentation-only" ]; then
 echo "⏭️  SKIPPED - Duplicate code (documentation project)."
@@ -1267,13 +1313,34 @@ else
   fi
   
   echo "Running jscpd for duplicate code detection..."
-  jscpd --config jscpd.conf.json $(echo "$CHANGED_FILES" | tr '\n' ' ') 2>&1 | head -30
-  JSCPD_EXIT=$?
-  if [ "$JSCPD_EXIT" -ne 0 ]; then
+  # #477: only pass --config when the file actually exists, and let jscpd's report
+  # decide the message. Before this, a missing config produced "os error 2" and a
+  # non-zero exit that the gate translated into a fabricated "found duplicated
+  # code"; a real clone report that exited 0 was announced as a clean pass.
+  JSCPD_CONFIG=$(resolve_jscpd_config)
+  JSCPD_ARGS=()
+  if [ -n "$JSCPD_CONFIG" ]; then
+    JSCPD_ARGS=(--config "$JSCPD_CONFIG")
+  else
+    echo "ℹ️  No jscpd.conf.json found - using jscpd defaults"
+  fi
+  JSCPD_OUTPUT=$(mktemp)
+  JSCPD_EXIT=0
+  # shellcheck disable=SC2086 # pre-existing splitting of the file list; #477 changes the reporting, not the scan set
+  jscpd ${JSCPD_ARGS[@]+"${JSCPD_ARGS[@]}"} $(echo "$CHANGED_FILES" | tr '\n' ' ') > "$JSCPD_OUTPUT" 2>&1 || JSCPD_EXIT=$?
+  head -30 "$JSCPD_OUTPUT"
+  JSCPD_OUTCOME=$(classify_jscpd_run "$JSCPD_EXIT" "$(cat "$JSCPD_OUTPUT")")
+  rm -f "$JSCPD_OUTPUT"
+  if [ "$JSCPD_OUTCOME" = "duplicates" ]; then
     echo "ℹ️  jscpd found duplicated code (warning, not blocking by default)"
     echo "   Consider refactoring duplicate code blocks."
+  elif [ "$JSCPD_OUTCOME" = "error" ]; then
+    echo "⚠️  jscpd exited with code $JSCPD_EXIT without a report — the output above is its real error"
+    echo "⏭️  SKIPPED - Duplicate code (jscpd run error)"
+    GATE_2_STATUS="SKIP"
+  else
+    echo "✅ PASSED - jscpd duplicate code check completed."
   fi
-  echo "✅ PASSED - jscpd duplicate code check completed."
     ;;
     
     python)
@@ -1331,7 +1398,8 @@ else
     esac
   done
 fi
-GATE_2_STATUS="PASS"
+# Set at the top of this gate; a branch may already have marked it SKIP (#477). The
+# audit must record what happened, not what the gate defaults to.
 record_gate_audit "gate-2" "duplicate-code" "$GATE_2_STATUS" "0" "$GATE_2_START"
 
 # ============================================================================
@@ -1392,7 +1460,7 @@ else
 
   SKIP_5A_BLOCK=false
   if [ "${SKIP_GATE_5A_BLOCK:-0}" = "1" ]; then
-    CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD)
+    CURRENT_BRANCH=$(git symbolic-ref --short HEAD 2>/dev/null || git rev-parse --abbrev-ref HEAD 2>/dev/null || echo "HEAD")
     if [ "$CURRENT_BRANCH" != "main" ] && [ "$CURRENT_BRANCH" != "master" ]; then
       REASON="${SKIP_GATE_5A_BLOCK_REASON:-not specified}"
       mkdir -p .xp-gate/reports
@@ -1617,7 +1685,15 @@ else
   # Gate 5b: Mock Density ADVISORY Scan (pure bash — no npx tsx overhead)
   # 30% = ADVISORY, 50% = suggests @mock-justified. Does NOT block commit.
   # ========================================================================
-  CHANGED_TEST_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E '\.(test|spec)\.(ts|tsx|js|jsx|py|go)$' || true)
+  CHANGED_TEST_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E '\.(test|spec)\.(ts|tsx|js|jsx|cjs|mjs|py|go)$' || true)
+
+  # Nested plugin sub-projects (plugins/<name>/) own their own vitest config and
+  # are excluded from the root suite (`vitest.config.ts` excludes `plugins/**`,
+  # because plugins/*/node_modules contains vendored tests). Feeding their test
+  # paths to the root vitest yields "No test files found" → false BLOCK, so drop
+  # them here and let the per-plugin suite cover them (CI runs `npm test` inside
+  # each plugin directory).
+  CHANGED_TEST_FILES=$(echo "$CHANGED_TEST_FILES" | grep -vE '^plugins/[^/]+/' || true)
 
   # Strip PROJECT_SUBDIR prefix from test file paths when running in subdirectory.
   # git diff returns repo-root relative paths, but after cd "$PROJECT_SUBDIR" (line 728)
@@ -1701,7 +1777,7 @@ else
       echo "⚠️  Gate 5c: req-ids.json is stale (specification.yaml updated since last check-alignment)"
       echo "   Run 'xp-gate check-alignment' to regenerate before next commit."
     else
-      CHANGED_TEST_FILES_ANNOTATED=$(echo "$CHANGED_FILES" | grep -E '\.(test|spec)\.(ts|tsx|js|jsx)$' || true)
+      CHANGED_TEST_FILES_ANNOTATED=$(echo "$CHANGED_FILES" | grep -E '\.(test|spec)\.(ts|tsx|js|jsx|cjs|mjs)$' || true)
       if [ -n "$CHANGED_TEST_FILES_ANNOTATED" ]; then
         echo "  → Checking @test REQ-XXX annotations..."
         ANNOTATION_ERRORS=0
@@ -1730,7 +1806,7 @@ else
         # Check reverse: any REQ in spec with no annotated test → visible FAIL (non-blocking)
         ALL_SPEC_REQS=$(jq -r '.[]' "$REQ_IDS_FILE" 2>/dev/null)
         ANNOTATED_ALL=""
-        for tf in $(echo "$CHANGED_FILES" | grep -E '\.(test|spec)\.(ts|tsx|js|jsx)$' || true); do
+        for tf in $(echo "$CHANGED_FILES" | grep -E '\.(test|spec)\.(ts|tsx|js|jsx|cjs|mjs)$' || true); do
           if [ -f "$tf" ]; then
             ANNOTATED_ALL="$ANNOTATED_ALL $(grep -oE '@test\s+REQ-[A-Z0-9-]+' "$tf" 2>/dev/null | sed 's/@test\s*//' || true)"
           fi
@@ -1784,16 +1860,41 @@ else
       # TypeScript: vitest with smart test selection
       if [ "$CURRENT_LANG" = "typescript" ] && { [ -f "package.json" ] || [ -f "$PROJECT_ROOT/package.json" ]; }; then
         if run_without_git_context npx vitest --version >/dev/null 2>&1; then
-          # Count changed test files to decide strategy
-          CHANGED_TEST_FILE_COUNT=$(echo "$CHANGED_TEST_FILES" | grep -c '.' || echo "0")
+          # Count changed test files to decide strategy. `grep -c '.'` prints its
+          # own zero AND exits 1 on an empty list, so `|| echo "0"` used to yield
+          # $'0\n0' -- the same two-value bug that false-blocked Gate 8 (#493).
+          # Bundled mirrors under src/npm-package/ are excluded from the root
+          # vitest project (vitest.config.ts, #418), so selecting them makes
+          # vitest exit 1 with "No test files found" and this gate reads that as
+          # failing tests: a false BLOCK for any commit that only touches mirrors
+          # (#494). Route both lists through the selector and announce it.
+          SELECTED_TEST_FILES=$(select_root_suite_targets "$CHANGED_TEST_FILES")
+          if [ "$SELECTED_TEST_FILES" != "$CHANGED_TEST_FILES" ]; then
+            echo "   ↪ Selection adjusted for bundled mirrors (#494):"
+            echo "     was: $(echo "$CHANGED_TEST_FILES" | tr '\n' ' ')"
+            echo "     now: $(echo "$SELECTED_TEST_FILES" | tr '\n' ' ')"
+          fi
+          CHANGED_TEST_FILE_COUNT=$(echo "$SELECTED_TEST_FILES" | grep -c '.' || true)
           
           # Also get changed source files (non-test) for related-test lookup
           CHANGED_SRC_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E '\.(ts|tsx|js|jsx)$' | grep -v '\.test\.' | grep -v '\.spec\.' | grep -v '__tests__' | grep -v '\.d\.ts$' | grep -v '__snapshots__' || true)
+          # Nested plugin sub-projects are excluded from the root suite; their
+          # sources must not drive root-vitest related-test lookup either (it
+          # would resolve to plugin-local paths vitest cannot collect).
+          CHANGED_SRC_FILES=$(echo "$CHANGED_SRC_FILES" | grep -vE '^plugins/[^/]+/' || true)
+          # Mirror handling is identical for source files: their test-adjacent
+          # paths must never reach vitest as root-project targets (#494).
+          SELECTED_SRC_FILES=$(select_root_suite_targets "$CHANGED_SRC_FILES")
+          if [ "$SELECTED_SRC_FILES" != "$CHANGED_SRC_FILES" ]; then
+            echo "   ↪ Source selection adjusted for bundled mirrors (#494):"
+            echo "     was: $(echo "$CHANGED_SRC_FILES" | tr '\n' ' ')"
+            echo "     now: $(echo "$SELECTED_SRC_FILES" | tr '\n' ' ')"
+          fi
           # Strip PROJECT_SUBDIR prefix (same as CHANGED_TEST_FILES fix above)
-          if [ -n "$PROJECT_SUBDIR" ] && [ "$PROJECT_SUBDIR" != "." ] && [ -n "$CHANGED_SRC_FILES" ]; then
-            CHANGED_SRC_FILES=$(echo "$CHANGED_SRC_FILES" | sed "s|^${PROJECT_SUBDIR}/||")
+          if [ -n "$PROJECT_SUBDIR" ] && [ "$PROJECT_SUBDIR" != "." ] && [ -n "$SELECTED_SRC_FILES" ]; then
+            SELECTED_SRC_FILES=$(echo "$SELECTED_SRC_FILES" | sed "s|^${PROJECT_SUBDIR}/||")
           fi
-          CHANGED_SRC_COUNT=$(echo "$CHANGED_SRC_FILES" | grep -c '.' || echo "0")
+          CHANGED_SRC_COUNT=$(echo "$SELECTED_SRC_FILES" | grep -c '.' || true)
           
           # Decide strategy
           if [ "$CHANGED_TEST_FILE_COUNT" -gt 20 ] 2>/dev/null; then
@@ -1803,24 +1904,24 @@ else
             TESTS_EXIT_CODE=$?
             echo "$TESTS_OUTPUT" | head -60
             if [ "$TESTS_EXIT_CODE" -ne 0 ]; then
-              echo ""
-              echo "❌ BLOCKED - Tests FAILED"
-              exit 1
+              handle_test_failure "$TESTS_OUTPUT" "full test suite" || exit 1
             else
               echo "✅ PASSED - Full test suite passed (with coverage)."
             fi
           elif [ "$CHANGED_TEST_FILE_COUNT" -gt 0 ] 2>/dev/null; then
             # Run only changed test files with coverage
-            SMART_TARGETS=$(echo "$CHANGED_TEST_FILES" | tr '\n' ' ')
+            PARTIAL_TEST_RUN=true  # Mark as partial run for coverage check
+            SMART_TARGETS=$(echo "$SELECTED_TEST_FILES" | tr '\n' ' ')
             echo "Running ${CHANGED_TEST_FILE_COUNT} changed test file(s) with coverage..."
-            echo "   Target(s): $(echo "$CHANGED_TEST_FILES" | tr '\n' ' ')"
+            echo "   Target(s): $(echo "$SELECTED_TEST_FILES" | tr '\n' ' ')"
             TESTS_OUTPUT=$(run_without_git_context npx vitest run --coverage $SMART_TARGETS 2>&1)
             TESTS_EXIT_CODE=$?
             echo "$TESTS_OUTPUT" | head -60
             if [ "$TESTS_EXIT_CODE" -ne 0 ]; then
-              echo ""
-              echo "❌ BLOCKED - Tests FAILED in changed test files"
-              exit 1
+              # Distinguish a real test failure from a runner infrastructure
+              # error -- see #454. handle_test_failure routes both cases and
+              # keeps the message consistent across every Gate 5 branch.
+              handle_test_failure "$TESTS_OUTPUT" "changed test files" || exit 1
             else
               echo "✅ PASSED - Changed test files passed (with coverage)."
             fi
@@ -1828,7 +1929,7 @@ else
             # Source files changed but no test files → find related tests or run full
             echo "Source file(s) changed, finding related tests..."
             RELATED_TESTS=""
-            for src_file in $CHANGED_SRC_FILES; do
+            for src_file in $SELECTED_SRC_FILES; do
               base="${src_file%.*}"
               filename="${base##*/}"
               dir="$(dirname "$src_file")"
@@ -1862,9 +1963,7 @@ else
               TESTS_EXIT_CODE=$?
               echo "$TESTS_OUTPUT" | head -60
               if [ "$TESTS_EXIT_CODE" -ne 0 ]; then
-                echo ""
-                echo "❌ BLOCKED - Tests FAILED in related test files"
-                exit 1
+                handle_test_failure "$TESTS_OUTPUT" "related test files" || exit 1
               else
                 echo "✅ PASSED - Related test files passed (with coverage)."
               fi
@@ -1875,9 +1974,7 @@ else
               TESTS_EXIT_CODE=$?
               echo "$TESTS_OUTPUT" | head -60
               if [ "$TESTS_EXIT_CODE" -ne 0 ]; then
-                echo ""
-                echo "❌ BLOCKED - Tests FAILED"
-                exit 1
+                handle_test_failure "$TESTS_OUTPUT" "full test suite" || exit 1
               else
                 echo "✅ PASSED - Full test suite passed (with coverage)."
               fi
@@ -1894,13 +1991,22 @@ else
           TESTS_EXIT_CODE=$?
           echo "$TESTS_OUTPUT" | head -50
           if [ "$TESTS_EXIT_CODE" -ne 0 ]; then
-            echo ""
-            echo "❌ BLOCKED - Tests FAILED"
-            exit 1
+            # This branch is reached from the typescript adapter, so run_tests ran
+            # vitest (adapters/typescript.sh) and its output is vitest-shaped: the
+            # same three non-failure exits (#454 runner EPERM, #473 partial
+            # threshold, #498 no test files) apply here as in the four branches
+            # above. #454 left it unguarded because its own comment covers the
+            # NON-typescript fallback below, whose runner is pytest/go/jest; this
+            # one is not that branch.
+            handle_test_failure "$TESTS_OUTPUT" "typescript adapter fallback" || exit 1
           fi
         fi
       else
-        # Non-TypeScript: standard adapter test flow
+        # Adapter test flow for everything the TypeScript branch above did not
+        # claim. The label this branch carried (#454) was "Non-TypeScript", which
+        # is not what reaches it: a TypeScript project without a package.json also
+        # lands here, and its adapter runs vitest — which is how a test-less TS
+        # repo got false-blocked on "No test files found" (#498).
         # Skip for Python — tests already run with coverage in the coverage section below.
         # Running them here first without coverage doubles pre-commit time for no benefit.
         if [ "$CURRENT_LANG" = "python" ]; then
@@ -1910,9 +2016,18 @@ else
           TESTS_EXIT_CODE=$?
           echo "$TESTS_OUTPUT" | head -50
           if [ "$TESTS_EXIT_CODE" -ne 0 ]; then
-            echo ""
-            echo "❌ BLOCKED - Tests FAILED"
-            exit 1
+            if [ "$CURRENT_LANG" = "typescript" ]; then
+              # vitest output, so the same three non-failure exits are judged
+              # here as in the branches above (#454, #473, #498).
+              handle_test_failure "$TESTS_OUTPUT" "typescript adapter (no package.json)" || exit 1
+            else
+              # Genuinely non-vitest runners (pytest/go/jest/...): their output
+              # shape is not vitest's, so the guard could excuse a real failure.
+              # They keep blocking on any non-zero exit (#454).
+              echo ""
+              echo "❌ BLOCKED - Tests FAILED"
+              exit 1
+            fi
           else
             echo "✅ PASSED - Unit tests passed."
           fi
@@ -2031,8 +2146,9 @@ else
         " 2>/dev/null)
         if [ "$COVERAGE_PERCENT" != "parse_error" ] && [ -n "$COVERAGE_PERCENT" ]; then
           if [ "$COVERAGE_PERCENT" -lt 80 ]; then
-            # If only subset of tests ran (partial run or related tests), warn instead of block
-            if [ "${PARTIAL_TEST_RUN:-false}" = "true" ] || { [ -n "${CHANGED_TEST_FILE_COUNT:-}" ] && [ "${CHANGED_TEST_FILE_COUNT:-0}" -gt 0 ] 2>/dev/null; }; then
+            # Only a subset test run (smart selection) leaves partial coverage data;
+            # a full suite run must block on <80% even when many test files changed.
+            if [ "${PARTIAL_TEST_RUN:-false}" = "true" ]; then
               echo "⚠️  Partial coverage ${COVERAGE_PERCENT}% (subset test run, not full suite)."
               echo "    Run 'npx vitest run --coverage' for full coverage check."
             else
@@ -2072,6 +2188,64 @@ else
         echo "⚠️  coverage.py not available, cannot enforce Python coverage threshold."
       fi
       ;;
+    "java")
+      COVERAGE_ENFORCED=true
+      # JaCoCo writes target/site/jacoco/ (Maven) or build/reports/jacoco/test/
+      # (Gradle), i.e. neither path probed by the default branch below, so Java
+      # needs its own branch to be measurable at all.
+      jacoco_csv=""
+      for jacoco_candidate in \
+        "target/site/jacoco/jacoco.csv" \
+        "build/reports/jacoco/test/jacocoTestReport.csv"; do
+        [ -f "$jacoco_candidate" ] && { jacoco_csv="$jacoco_candidate"; break; }
+      done
+
+      if [ -n "$jacoco_csv" ]; then
+        # Aggregate INSTRUCTION_MISSED / INSTRUCTION_COVERED over every class row.
+        # Columns are located by header name because their position is not part of
+        # the CSV's stable contract.
+        COVERAGE_PERCENT=$(awk -F, '
+          NR == 1 {
+            for (i = 1; i <= NF; i++) {
+              if ($i == "INSTRUCTION_MISSED") missed_col = i;
+              if ($i == "INSTRUCTION_COVERED") covered_col = i;
+            }
+            next;
+          }
+          missed_col && covered_col {
+            missed += $missed_col;
+            covered += $covered_col;
+          }
+          END {
+            total = missed + covered;
+            if (total > 0) printf "%.2f", (covered / total) * 100;
+            else print "";
+          }
+        ' "$jacoco_csv" 2>/dev/null) || COVERAGE_PERCENT=""
+
+        if [ -n "$COVERAGE_PERCENT" ]; then
+          # Compare on the unrounded value so 79.5% cannot pass as 80%.
+          COVERAGE_ROUNDED=${COVERAGE_PERCENT%.*}
+          if awk -v pct="$COVERAGE_PERCENT" 'BEGIN { exit !(pct < 80) }'; then
+            if [ "${PARTIAL_TEST_RUN:-false}" = "true" ]; then
+              echo "⚠️  Partial coverage ${COVERAGE_ROUNDED}% (subset test run, not full suite)."
+              echo "    Run the full test suite for a complete Java coverage check."
+            else
+              echo "❌ BLOCKED - Java coverage ${COVERAGE_ROUNDED}% below 80% threshold"
+              exit 1
+            fi
+          else
+            echo "Java coverage: ${COVERAGE_ROUNDED}% ✅ (≥ 80%)"
+          fi
+        else
+          echo "⚠️  Could not parse Java coverage from ${jacoco_csv} (no instruction data)."
+        fi
+      else
+        echo "⚠️  No JaCoCo report found. Run 'mvn verify' or 'gradle jacocoTestReport'"
+        echo "    so that target/site/jacoco/jacoco.csv exists — otherwise Java coverage"
+        echo "    cannot be enforced (see also: JaCoCo detection in adapters/java.sh)."
+      fi
+      ;;
     "go")
       # Go enforcement already handled inline in Stage 1 (lines 933-942)
       ;;
@@ -2134,6 +2308,10 @@ else
         echo "⚠️  coverage.xml not found. Ensure Pester coverage was run."
       fi
       ;;
+    "iac")
+      # IaC has no coverage concept — must not borrow the TypeScript report
+      # that Stage 1 leaves behind (same rule as the PowerShell branch).
+      ;;
     *)
       # Default: attempt to parse coverage from common formats regardless of language
       COVERAGE_ENFORCED=true
@@ -2157,10 +2335,18 @@ else
       fi
       if [ -n "$COVERAGE_PERCENT" ] && [ "$COVERAGE_PERCENT" != "parse_error" ] && [ "$COVERAGE_PERCENT" -gt 0 ]; then
         if [ "$COVERAGE_PERCENT" -lt 80 ]; then
-          echo "❌ BLOCKED - ${CURRENT_LANG} coverage ${COVERAGE_PERCENT}% below 80% threshold"
-          exit 1
+          # Only a subset test run (smart selection) warns — the full suite may
+          # still reach ≥80%; a full suite run must block.
+          if [ "${PARTIAL_TEST_RUN:-false}" = "true" ]; then
+            echo "⚠️  Partial coverage ${COVERAGE_PERCENT}% (subset test run, not full suite)."
+            echo "    Run the full test suite for a complete ${CURRENT_LANG} coverage check."
+          else
+            echo "❌ BLOCKED - ${CURRENT_LANG} coverage ${COVERAGE_PERCENT}% below 80% threshold"
+            exit 1
+          fi
+        else
+          echo "${CURRENT_LANG} coverage: ${COVERAGE_PERCENT}% ✅ (≥ 80%)"
         fi
-        echo "${CURRENT_LANG} coverage: ${COVERAGE_PERCENT}% ✅ (≥ 80%)"
       else
         echo "⚠️  Could not determine ${CURRENT_LANG} coverage percentage. No standard coverage report files found."
       fi
@@ -2301,10 +2487,20 @@ else
          fi
          ;;
       "python")
-        if [ -f ".import-linter.yml" ] || [ -f "import_linter_config.yml" ]; then
+        # detect any import-linter carrier (incl. pyproject.toml [tool.importlinter])
+        if [ -f "$PROJECT_ROOT/.import-linter.yml" ] || [ -f "$PROJECT_ROOT/import_linter_config.yml" ] || [ -f "$PROJECT_ROOT/.importlinter" ] || grep -q '^\[tool\.importlinter\]' "$PROJECT_ROOT/pyproject.toml" 2>/dev/null; then
           if require_tool "lint-imports" "Gate 6" "pip install import-linter"; then
             echo "     Running import-linter for Python..."
-            lint-imports 2>&1 | head -20
+            # run at PROJECT_ROOT (hook may have cd'd); block on exit code
+            LINT_IMPORTS_OUTPUT=$(cd "$PROJECT_ROOT" && lint-imports 2>&1)
+            LINT_IMPORTS_EXIT=$?
+            echo "$LINT_IMPORTS_OUTPUT" | head -30
+            if [ "$LINT_IMPORTS_EXIT" -ne 0 ]; then
+              echo ""
+              echo "❌ BLOCKED - Python architecture contract violations (import-linter)"
+              echo "Fix the broken contracts above before committing."
+              exit 1
+            fi
             echo "     ✅ Python architecture validation completed."
           else
             echo "     ❌ BLOCKED - import-linter not installed"
@@ -2312,7 +2508,7 @@ else
             exit 1
           fi
         else
-          echo "     ℹ️  No .import-linter.yml found - skipping Python architecture"
+          echo "     ℹ️  No import-linter config found (.import-linter.yml/.importlinter/[tool.importlinter]) - skipping"
           echo "     ⏭️  SKIPPED - Python architecture validation (no config)"
         fi
         ;;
@@ -2402,20 +2598,30 @@ else
   if [ -n "$BOY_SCOUT_SCRIPT" ]; then
     echo "     Checking Boy Scout Rule compliance..."
     
-    # Separate new files and modified files
-    NEW_FILES=$(git diff --cached --name-only --diff-filter=A | tr '\n' ' ')
-    MODIFIED_FILES=$(git diff --cached --name-only --diff-filter=M | tr '\n' ' ') 
+    # Separate new files and modified files.
+    #
+    # src/npm-package/** is a byte-identical MIRROR of canonical sources
+    # (plugins/**, src/principles/**, githooks/**) produced by
+    # sync-package-content.js at publish time. Checking it here double-counts
+    # every warning: the mirror path enters the baseline instead of the source
+    # path, so a violation gets recorded against a path nobody edits and the
+    # real source path stays unbudgeted. Exclude it and let the mirroring be
+    # governed by Mirror Parity in CI, which compares content rather than
+    # re-linting it. (See the same exclusion in adapters/powershell.sh.)
+    MIRROR_EXCLUDE='^src/npm-package/'
+    NEW_FILES=$(git diff --cached --name-only --diff-filter=A | grep -v "$MIRROR_EXCLUDE" | tr '\n' ' ')
+    MODIFIED_FILES=$(git diff --cached --name-only --diff-filter=M | grep -v "$MIRROR_EXCLUDE" | tr '\n' ' ')
     
     if [ -z "$NEW_FILES" ] && [ -z "$MODIFIED_FILES" ]; then
       echo "     ✅ No new or modified files to check."
     else
       # Run Boy Scout Rule enforcement
-      if command -v npx &> /dev/null; then
+      if command -v npx &> /dev/null || [ -f "${PROJECT_ROOT:-$(pwd)}/node_modules/tsx/dist/cli.mjs" ]; then
         # Skip if no files to check
         if [ -z "$(echo "$NEW_FILES" | tr -d ' ')" ] && [ -z "$(echo "$MODIFIED_FILES" | tr -d ' ')" ]; then
           echo "     ✅ No new or modified source files for Boy Scout check."
         else
-          BOY_SCOUT_OUTPUT=$(npx tsx $BOY_SCOUT_SCRIPT \
+          BOY_SCOUT_OUTPUT=$(run_tsx "$BOY_SCOUT_SCRIPT" \
             --new-files "$(echo "$NEW_FILES" | tr ' ' ',')" \
             --modified-files "$(echo "$MODIFIED_FILES" | tr ' ' ',')" \
             --baseline ".warnings-baseline.json" 2>&1)
@@ -2446,8 +2652,8 @@ else
         echo "     ✅ PASSED - Boy Scout Rule compliance."
         fi
       else
-        echo "     ℹ️  npx not available - skipping Boy Scout Rule"
-        echo "     ⏭️  SKIPPED - Boy Scout Rule (Node.js/npx not available)"
+        echo "     ℹ️  npx/tsx not available - skipping Boy Scout Rule"
+        echo "     ⏭️  SKIPPED - Boy Scout Rule (Node.js/tsx not available)"
       fi
     fi
   else
@@ -2503,7 +2709,7 @@ if [ -f "package.json" ] && [ -f "tsconfig.json" ]; then
   if [ -n "$GATE_9_SCRIPT" ]; then
     CHANGED_FILES_CSV=$(echo "$CHANGED_FILES" | tr ' ' ',')
     GATE_9_OUTPUT=$(mktemp)
-    if timeout 120s npx tsx "$GATE_9_SCRIPT" --changed-files "$CHANGED_FILES_CSV" --project-root "$PROJECT_ROOT" --timeout 115000 > "$GATE_9_OUTPUT" 2>&1; then
+    if RUN_TSX_TIMEOUT=120s run_tsx "$GATE_9_SCRIPT" --changed-files "$CHANGED_FILES_CSV" --project-root "$PROJECT_ROOT" --timeout 115000 > "$GATE_9_OUTPUT" 2>&1; then
       GATE_9_STATUS="PASS"
       echo "     ✅ Build integrity check passed."
     else
@@ -2531,8 +2737,9 @@ else
   echo "     ⏭️  SKIPPED - Build integrity (not a TypeScript project)"
 fi
 
-GATE_9_END=$(date +%s)
-GATE_9_DURATION=$((GATE_9_END - GATE_9_START))
+GATE_9_END=$(gate_start_ms)
+GATE_9_DURATION_MS=$((GATE_9_END - GATE_9_START))
+GATE_9_DURATION=$((GATE_9_DURATION_MS / 1000))
 echo "✅ Gate 9 completed in ${GATE_9_DURATION}s"
 record_gate_audit "gate-9" "build-integrity" "$GATE_9_STATUS" "0" "$GATE_9_START"
 echo ""
@@ -2565,29 +2772,66 @@ fi
 GATE_11_START=$(date +%s)
 GATE_11_STATUS="PASS"
 
-SPRINT_GATE_SCRIPT=""
-if [ -f "$GATE_DIR/sprint-gate.sh" ]; then
-  SPRINT_GATE_SCRIPT="$GATE_DIR/sprint-gate.sh"
-elif [ -f "$(git rev-parse --show-toplevel 2>/dev/null)/githooks/sprint-gate.sh" ]; then
-  SPRINT_GATE_SCRIPT="$(git rev-parse --show-toplevel)/githooks/sprint-gate.sh"
-fi
+# BEGIN SPRINT_GATE_SCRIPT RESOLUTION
+# Every tier is a *directory*; the resolver returns the first one that actually holds
+# sprint-gate.sh. The chain used to stop at the gate-script directories, but the flat
+# global layout ($GATE_DIR -> ~/.config/xp-gate/adapters) carries gate-3/4/7/8/9/10.sh
+# and never this script: update-hooks installs sprint-gate.sh under
+# ~/.config/xp-gate/hooks, which nothing probed, so a correctly installed machine was
+# told the file was missing (#476).
+SPRINT_GATE_CANDIDATES=(
+  "$GATE_DIR"
+  "$(git rev-parse --show-toplevel 2>/dev/null)/githooks"
+  "$SCRIPT_DIR"
+  "$HOME/.config/xp-gate/hooks"
+)
+SPRINT_GATE_SCRIPT=$(resolve_sprint_gate_script ${SPRINT_GATE_CANDIDATES[@]+"${SPRINT_GATE_CANDIDATES[@]}"})
+# END SPRINT_GATE_SCRIPT RESOLUTION
+
+echo "→ Gate 11: Sprint Flow Enforcement..."
 
 if [ -n "$SPRINT_GATE_SCRIPT" ]; then
-  if ! bash "$SPRINT_GATE_SCRIPT" --pre-commit; then
+  # The decision JSON was piped straight to the terminal with no heading, so it landed
+  # under Gate 10's banner and a legitimate "skip: not a sprint project" looked like a
+  # bare machine object. The exit status stays the authority for blocking; the JSON only
+  # decides which words the user reads.
+  SPRINT_GATE_EXIT=0
+  SPRINT_GATE_OUTPUT=$(bash "$SPRINT_GATE_SCRIPT" --pre-commit 2>&1) || SPRINT_GATE_EXIT=$?
+  SPRINT_GATE_DECISION=$(sprint_gate_decision "$SPRINT_GATE_OUTPUT")
+  SPRINT_GATE_DETAIL=$(sprint_gate_detail "$SPRINT_GATE_OUTPUT")
+
+  if [ "$SPRINT_GATE_EXIT" -ne 0 ]; then
     GATE_11_STATUS="BLOCK"
-    echo "❌ BLOCKED - Gate 11: Sprint Flow Enforcement"
+    echo "❌ BLOCKED - Gate 11: Sprint Flow (${SPRINT_GATE_DETAIL:-sprint gate denied})"
+    printf '%s\n' "$SPRINT_GATE_OUTPUT" | sed 's/^/     /'
     record_gate_audit "gate-11" "sprint-flow" "$GATE_11_STATUS" "1" "$GATE_11_START"
     # Sprint gate failure is a hard block — exit immediately
     exit 1
   fi
+
+  case "$SPRINT_GATE_DECISION" in
+    skip)
+      echo "⏭️  SKIPPED - Gate 11: Sprint Flow (${SPRINT_GATE_DETAIL:-no reason given})"
+      GATE_11_STATUS="SKIP"
+      ;;
+    allow)
+      echo "✅ PASSED - Gate 11: Sprint Flow (${SPRINT_GATE_DETAIL:-sprint state validated})"
+      ;;
+    *)
+      # A run that succeeded but said nothing this gate understands. Pass on the exit
+      # status, but keep the raw output visible rather than inventing a verdict.
+      echo "✅ PASSED - Gate 11: Sprint Flow (no recognisable decision in the gate output)"
+      printf '%s\n' "$SPRINT_GATE_OUTPUT" | sed 's/^/     /'
+      ;;
+  esac
 else
-  echo "⏭️  SKIPPED - Gate 11: Sprint Flow (sprint-gate.sh not found)"
+  echo "⏭️  SKIPPED - Gate 11: Sprint Flow (sprint-gate.sh not found; searched: $(IFS=', '; printf '%s' "${SPRINT_GATE_CANDIDATES[*]}"))"
   GATE_11_STATUS="SKIP"
 fi
 
 GATE_11_END=$(date +%s)
 GATE_11_DURATION=$((GATE_11_END - GATE_11_START))
-echo "✅ Gate 11 completed in ${GATE_11_DURATION}s"
+echo "   Gate 11 ($GATE_11_STATUS) completed in ${GATE_11_DURATION}s"
 record_gate_audit "gate-11" "sprint-flow" "$GATE_11_STATUS" "0" "$GATE_11_START"
 echo ""
 # ============================================================================
@@ -2647,8 +2891,10 @@ generate_quality_report() {
   if [ -n "$TESTS_OUTPUT" ] && echo "$TESTS_OUTPUT" | grep -qi "fail\|error\|FAILED"; then
     # Extract failure lines: "FAIL src/foo.test.ts" or "  ✗ test name"
     local FAIL_LINES
-    FAIL_LINES=$(echo "$TESTS_OUTPUT" | grep -E "^\s*(FAIL|✗|failed|AssertionError)" | head -10 | \
-      sed 's/"/\\"/g' | awk '{ printf "%s%s%s", sep, "\"", $0; sep="," }')
+    # errexit/pipefail-safe: grep returns 1 on zero matches (see #489) and this
+    # function is also sourced by consumer wrappers that may enable errexit
+    FAIL_LINES=$(echo "$TESTS_OUTPUT" | grep -E "^\s*(FAIL|✗|failed|AssertionError)" | sed -n '1,10p' | \
+      sed 's/"/\\"/g' | awk '{ printf "%s%s%s", sep, "\"", $0; sep="," }') || true
     if [ -n "$FAIL_LINES" ]; then
       FAILED_TESTS_JSON="[$FAIL_LINES]"
     fi