@spardutti/claude-skills 2.54.2 → 2.55.1

package/README.md

@@ -36,7 +36,7 @@ npx @spardutti/claude-skills
 ```
 
 ```text
-  Claude Skills Installer v2.54.2
+  Claude Skills Installer v2.55.1
 
   ── 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

@@ -14,14 +14,13 @@ changed_tests() {  # changed_tests <family>
   done
 }
 
-# Everything a file's verdict rests on besides the file itself and the tests.
-# Tests are left out: adding to them can only kill more, so only a lost line counts.
+# Everything a file's verdict rests on besides the file itself and the tests: the base, the gate,
+# and the test setup. Other code is left out, so one fix no longer re-mutates every clean file.
 files_context() {  # files_context <owner> <owned files>
   fam=$(family_of "$1/.")
   { printf '%s\n' "$PROJECT_DIR" "$1" "$(git rev-parse "$BASE" 2>/dev/null)"
     cat "$HERE/ship-gate.sh" "$HERE/ship-gate-projects.sh" "$HERE/ship-gate-files.sh" "$HERE/gate-config.sh" "$HERE/line-limit.sh"
-    printf '%s\n' "$CHANGED" | grep -vE '\.(md|mdx|txt|rst|adoc)$' | grep -vE "$TEST_RE" \
-      | grep -vxF -f <(printf '%s\n' "$2") | while IFS= read -r f; do
+    printf '%s\n' "$CHANGED" | grep -E "$SETUP_RE" | while IFS= read -r f; do
       [ -n "$f" ] || continue
       case "$(family_of "$f")" in "$fam"|any) printf '%s\n' "$f"; [ -f "$f" ] && cat "$f" ;; esac
     done; } | git hash-object --stdin
@@ -36,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/lib/setup-claude-md.mjs

@@ -27,6 +27,7 @@ The hooks and skills in \`.claude/\` come from \`@spardutti/claude-skills\`. Fac
 - **Make it mandatory on some files:** put \`gate-paths: "src/**/*.tsx"\` under \`metadata:\`, quoted. A bare top-level \`paths:\` makes Claude Code drop the skill. Add \`tracks: react@19\` to fire only when the project depends on that package.
 - **Hook settings** live in \`.claude/gauntlet.conf\` (shell syntax): \`GAUNTLET_TEST\`, \`GAUNTLET_TYPECHECK\`, \`GAUNTLET_BASE\` (the trunk branch, if not develop/dev/main), \`GAUNTLET_MAX_LINES\`, \`GAUNTLET_IGNORE_FILES\`, \`GAUNTLET_OFF=1\`.
 - **Shipping:** open, merge or release a PR only through the \`/ship\` command; load it every time, even when you remember the steps. Never run \`ship-gate.sh\` with \`--force\` or \`--baseline\` unless the user asks.
+- **Tests:** The Stop hook runs the changed files' tests when you end your turn. Don't run them yourself to check; run one test file only to see a new test fail first.
 - **Never edit \`.claude/hooks/\`** — a sync overwrites it. Update with \`npx @spardutti/claude-skills --sync\`.`;
 
 const BLOCKS = [

package/lib/sync.mjs

@@ -4,6 +4,7 @@ import { readdir } from "node:fs/promises";
 import { join, basename } from "node:path";
 import { installSkills, installCommands, installRequiredAgents } from "./install.mjs";
 import { setupHook } from "./setup-hook.mjs";
+import { setupClaudeMd } from "./setup-claude-md.mjs";
 import { readManifest, writeManifest, computeOrphans, removeArtifacts, MANIFEST_FILE } from "./manifest.mjs";
 
 const SEARCH_DEPTH = 5;
@@ -28,7 +29,7 @@ export async function runSync(dir, catalog, version) {
   if (skills.length > 0) { console.log(); await installSkills(skills, dir); }
   if (commands.length > 0) { console.log(); await installCommands(commands, dir); }
   const { installed } = await installRequiredAgents(commands, catalog.agents, dir);
-  if (existsSync(join(dir, ".claude", "hooks", "skill-gate.sh"))) { console.log(); await setupHook(dir); }
+  if (existsSync(join(dir, ".claude", "hooks", "skill-gate.sh"))) { console.log(); await setupHook(dir); await setupClaudeMd(dir); }
 
   await writeManifest(dir, {
     catalogVersion: version,

package/package.json

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