@spardutti/claude-skills 2.35.0 → 2.36.1

package/hooks/ship-gate-projects.sh

@@ -36,12 +36,16 @@ owner_of() {
 # Blaming an unreachable database for that sent one agent chasing Docker for 20 minutes.
 mutmut_unchecked() {  # mutmut_unchecked <label> <base> <count> <run log>
   echo "  $1 mutmut — UNPROVEN: $3 mutant(s) were never run."
-  stop='Failed to run clean test|failed to collect stats|Failed to collect list of tests|Stopping early'
+  # "Failed to run pytest with args" is one line carrying every selected test id — 20KB of
+  # them on one repo — so the line is cut. Unlisted, the gate blamed a database instead.
+  stop='Failed to run clean test|failed to collect stats|Failed to collect list of tests|Stopping early|Failed to run pytest with args|BadTestExecutionCommandsException'
   if grep -aqsE "$stop" "$4"; then
     echo "      mutmut stopped before testing a single mutant:"
-    grep -aE "^(FAILED|ERROR) |^[A-Za-z]*Error: |$stop" "$4" | tail -8 | sed 's/^/        /'
+    grep -aE "^(FAILED|ERROR) |^[A-Za-z]*Error: |$stop" "$4" | tail -8 | cut -c1-200 | sed 's/^/        /'
     echo "      It runs the whole suite, then these tests again in one process, so a"
     echo "      test that passes alone can fail here on state an earlier test left."
+    echo "      mutmut hides pytest's own error: set debug = true under [tool.mutmut]"
+    echo "      and run it again to read it."
     return
   fi
   echo "      They are recorded \"not checked\", so the suite proved nothing"
@@ -105,3 +109,17 @@ project_key() {  # project_key <owner> <tool>
       case "$(family_of "$f")" in "$fam"|any) printf '%s\n' "$f"; [ -f "$f" ] && cat "$f" ;; esac
     done; } | git hash-object --stdin
 }
+
+# A page test renders a whole screen, so it "kills" mutants in every util that screen calls.
+# Backoffice ran 23 minutes that way, and hid 104 mutants no logic test checked.
+stryker_page_tests() {  # stryker_page_tests <base> <label>
+  grep -qs configFile "$1"stryker.conf* && return 0
+  pages=$(git ls-files -co --exclude-standard -- "${1:-.}" | grep -E '\.test\.[jt]sx$' \
+          | grep -vE '/(hooks|queries)/|(^|/)\.stryker-tmp/' | head -3)
+  [ -n "$pages" ] || return 0
+  echo "  $2 stryker — UNPROVEN: page tests count as proof for logic, e.g."
+  printf '%s\n' "$pages" | sed 's/^/        /'
+  echo "      Give Stryker a Vitest config that runs logic tests only — see"
+  echo "      testing-best-practices/MUTATION-TESTING.md, \"Keep Page Tests Out Of The Run\"."
+  [ "$STATUS" = 0 ] && STATUS=2
+}

package/hooks/ship-gate-structure.sh

@@ -12,8 +12,9 @@ added_files() {
     git ls-files --others --exclude-standard 2>/dev/null; } | sort -u
 }
 
-has_subject() {  # has_subject <test path>: a file the test is named for, in the same folder
+has_subject() {  # has_subject <test path>: a file the test is named for, beside it or above its __tests__/
   d=$(dirname "$1")
+  [ "${d##*/}" = __tests__ ] && d=$(dirname "$d")
   stem=$(basename "$1" | sed -E 's/\.(test|spec)\.[^.]+$//')
   while :; do
     for e in ts tsx js jsx mjs cjs astro; do [ -f "$d/$stem.$e" ] && return 0; done

package/hooks/ship-gate.sh

@@ -25,8 +25,8 @@
 # Exit codes:
 #   0  clean — nothing over the line limit, nothing survived, nothing uncovered
 #   1  findings that must be dealt with before shipping
-#   2  ran, but could not prove the tests (a mutation tool is missing) — report,
-#      don't block; the output names exactly what to install and where
+#   2  ran, but could not prove the tests (a mutation tool is missing, or page tests
+#      count as proof) — report, don't block; the output names the fix and where
 #
 # Config: the same .claude/gauntlet.conf as the Stop hook.
 #   GAUNTLET_MAX_LINES=200   the per-file limit; 0 turns the check off
@@ -271,7 +271,7 @@ for owner in $OWNERS; do
     # 8 of them on one commit, all long since killed. --mutate already scopes
     # the run to the changed hunks, so incremental buys nothing here and costs
     # a cache that goes stale exactly the way mutmut's did.
-    CMD="${RUN}npx --no-install stryker run --mutate '${FLAGS#,}'"
+    CMD="${RUN}npx --no-install stryker run --mutate '${FLAGS#,}'"; stryker_page_tests "$base" "$label"
   elif [ -f "$base"package.json ]; then
     MISSING="$MISSING  $label needs Stryker:
       npm --prefix ${owner} i -D @stryker-mutator/core @stryker-mutator/vitest-runner

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spardutti/claude-skills",
-  "version": "2.35.0",
+  "version": "2.36.1",
   "description": "Guardrails for Claude Code — best-practice skills, planning commands, and delivery gates.",
   "type": "module",
   "repository": {