@spardutti/claude-skills 2.55.0 → 2.56.0

package/README.md

@@ -36,7 +36,7 @@ npx @spardutti/claude-skills
 ```
 
 ```text
-  Claude Skills Installer v2.55.0
+  Claude Skills Installer v2.56.0
 
   ── Frontend ──────────────────────────────
   ◉ react              ◯ tanstack-query
@@ -200,7 +200,7 @@ scripts/       validate-skills.mjs — checks skill length caps and reference in
                edit-check.sh — PostToolUse check on each edited file: line-limit.sh, lint-ratchet.mjs, .edit-rules.json
                diary-check.mjs — run at the end of /ship; says when the diaries hold enough for /add-rule
                line-limit.sh — the 200-line ratchet, shared by the edit check, the ship gate and preflight
-               gauntlet-selftest.sh — behavioural tests for the hooks (runs on pre-push)
+               gauntlet-selftest.sh — behavioural tests for the hooks (runs in preflight)
                gauntlet-survey.sh — what the hook would do in every repo under a directory
                preflight.sh — everything that must pass before a release
                check-freshness.mjs — compares each skill's `tracks:` versions to the registries
@@ -226,7 +226,7 @@ Skills live in `skills/<name>/SKILL.md`. Authoring conventions are in [CLAUDE.md
 - Before any release, run `bash scripts/preflight.sh` — validator, self-tests, a check that the
   hook scripts embedded in the CLI still match `scripts/`, detection against every repo under
   `~/projects`, and that the version is ahead of the registry. Do not publish while it is red.
-- Changing `scripts/gauntlet.sh`? Run `bash scripts/gauntlet-selftest.sh` (also on `pre-push`),
+- Changing `scripts/gauntlet.sh`? Run `bash scripts/gauntlet-selftest.sh` (also in preflight),
   and run `bash scripts/gauntlet-survey.sh ~/projects` before releasing. Detection is where
   this hook keeps breaking, because real repos are shaped in ways invented test repos aren't —
   the survey is a dry run that reports what each of your repos would get without executing

package/hooks/ship-gate-files.sh

@@ -35,31 +35,40 @@ test_then() {  # test_then <snapshot> <test>: its content when the snapshot was
   else git show "$BASE:$2" 2>/dev/null; fi
 }
 
-# Never `| grep -q` under pipefail: diff exits 1 on any difference, and an early
-# grep exit fails the writer, so a match reads as none.
-tests_lost_lines() {  # tests_lost_lines <snapshot> <family>
+# The repo-relative paths a test names in quotes (./, ../, @/ or ~/), extension dropped.
+test_names() {  # test_names <test path> <owner prefix> < test text
+  grep -oE "['\"](\.{1,2}/|@/|~/)[^'\"]+['\"]" | tr -d "'\"" | while IFS= read -r s; do
+    case "$s" in @/*|~/*) printf '%s%s\n' "$2" "${s#?/}" ;; *) printf '%s/%s\n' "$(dirname "$1")" "$s" ;; esac
+  done | awk -F/ '{ n = 0; for (i = 1; i <= NF; i++) { if ($i == "..") n--; else if ($i != "." && $i != "") p[++n] = $i }
+                    o = p[1]; for (i = 2; i <= n; i++) o = o "/" p[i]; print o }' | sed -E 's/\.(m|c)?[jt]sx?$//; s#/index$##'
+}
+
+# A test that lost a line can revive mutants only in the files it names, so only those lose their pass.
+# Never `| grep -q` under pipefail: diff exits 1 on any difference, and an early grep exit fails the writer.
+tests_lost_lines() {  # tests_lost_lines <snapshot> <family> <file> <owner prefix>
   [ -f "$1/manifest" ] || return 0
+  local f="${3%.*}"; f=${f%/index}
   while IFS= read -r t; do
     [ -n "$t" ] || continue
-    case "$(diff <(test_then "$1" "$t") <(cat "$t" 2>/dev/null))" in *$'\n<'*) return 0 ;; esac
+    case "$(diff <(test_then "$1" "$t") <(cat "$t" 2>/dev/null))" in *$'\n<'*) ;; *) continue ;; esac
+    [ -n "$({ test_then "$1" "$t"; cat "$t" 2>/dev/null || true; } | test_names "$t" "$4" | grep -xF "$f")" ] && return 0
   done <<< "$({ cat "$1/manifest"; changed_tests "$2"; } | sort -u)"
   return 1
 }
 
 # Sets OWNED to the files that still need mutating.
 files_split() {  # files_split <context> <owner> <label>
-  fam=$(family_of "$2/."); kept=""; rest=""
+  fam=$(family_of "$2/."); kept=""; rest=""; b=""; [ "$2" != "." ] && b="$2/"
   while IFS= read -r f; do
     [ -n "$f" ] || continue
     r=$(file_receipt "$1" "$f")
-    if [ -f "$r" ] && ! tests_lost_lines "$(cat "$r")" "$fam"; then kept="$kept ${f#"$2"/}"
+    if [ -f "$r" ] && ! tests_lost_lines "$(cat "$r")" "$fam" "$f" "${b}"; then kept="$kept ${f#"$2"/}"
     else rest="$rest$f"$'\n'; fi
   done <<< "$OWNED"
   OWNED=${rest%$'\n'}
   [ -n "$kept" ] || return 0
   echo "  $3 stryker — ok, not re-run: passed and unchanged since:$kept"
   echo "      A change in a file they call is not re-checked."
-  b=""; [ "$2" != "." ] && b="$2/"
   [ -z "$OWNED" ] && stryker_page_tests "$b" "$3"
   return 0
 }

package/hooks/ship-log-report.mjs

@@ -14,7 +14,7 @@ function records(dir) {
 export function tally(findings) {
   const by = {};
   for (const f of findings) {
-    const s = (by[f.by.split(":")[0]] ??= { found: 0, confirmed: 0, refuted: 0, unconfirmed: 0, fixed: 0 });
+    const s = (by[f.by.split(":")[0]] ??= { found: 0, confirmed: 0, refuted: 0, unconfirmed: 0, skipped: 0, fixed: 0 });
     s.found++; s[f.verdict]++; if (f.fixed) s.fixed++;
   }
   return by;

package/hooks/ship-log.mjs

@@ -6,7 +6,7 @@ import { git, parseReport, prSize } from "./ship-log-parse.mjs";
 import { printReport, tally } from "./ship-log-report.mjs";
 
 const HERE = dirname(resolve(process.argv[1]));
-const VERDICTS = ["confirmed", "refuted", "unconfirmed"];
+const VERDICTS = ["confirmed", "refuted", "unconfirmed", "skipped"];
 const now = () => new Date().toISOString().replace(/\.\d+Z$/, "Z");
 
 // gate-config.sh owns the config files and their order, so the gate and this agree on "on".

package/package.json

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