@spardutti/claude-skills 2.35.0 → 2.36.0

package/hooks/ship-gate-projects.sh

@@ -105,3 +105,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.0",
   "description": "Guardrails for Claude Code — best-practice skills, planning commands, and delivery gates.",
   "type": "module",
   "repository": {