@spardutti/claude-skills 2.51.2 → 2.51.3

package/README.md

@@ -36,7 +36,7 @@ npx @spardutti/claude-skills
 ```
 
 ```text
-  Claude Skills Installer v2.25.0
+  Claude Skills Installer v2.51.2
 
   ── Frontend ──────────────────────────────
   ◉ react              ◯ tanstack-query
@@ -76,7 +76,8 @@ install adds the part that does not negotiate:
 - A `Stop` hook that runs your typecheck and tests on the changed files every turn,
   and refuses to let the turn end red.
 - A delivery gate that **changes your code on purpose and checks whether any test
-  notices.** When none do, the PR does not open.
+  notices.** When none do, the PR does not open. The same gate runs one
+  `/code-review` round for logic bugs and fixes what it finds.
 
 That last one is the difference. Your agent says the tests pass. This finds out.
 

package/hooks/ship-gate-files.sh

@@ -110,3 +110,22 @@ stryker_findings() {  # stryker_findings <output> <globbed paths, comma-wrapped>
     END { if (pend != "" && kind == "s") print pend }
   '
 }
+
+# A survivor needs a test run against it. Survivors with 0 tests per mutant and
+# nothing killed mean the runner filtered every test out (stryker-js#6210, Vitest 5).
+stryker_zero_tests() {  # stryker_zero_tests <output> <label> <seconds>
+  grep -q '^\[Survived\]' <<< "$1" && grep -q 'Ran 0\.00 tests per mutant' <<< "$1" || return 1
+  [ "$(awk -F'|' '/^ *All files/ { gsub(/ /, "", $4); print $4 }' <<< "$1")" = 0 ] || return 1
+  echo "  $2 stryker (${3}s) — its survivors ran 0 tests, so they prove nothing and are dropped."
+  echo "      On Vitest 5 this is stryker-js#6210: pin vitest@4 until a fixed runner ships."
+}
+
+# Sets ALL from a Stryker run. Fails when a broken run leaves nothing provable.
+stryker_judge() {  # uses OUT GLOBBED FCTX RC owner label T from ship-gate.sh
+  ALL=$(stryker_findings "$OUT" "$GLOBBED")
+  if stryker_zero_tests "$OUT" "$label" "$T"; then
+    ALL=$(grep -v '^\[Survived\]' <<< "$ALL"); [ -n "$ALL" ]; return
+  fi
+  [ -n "$FCTX" ] && [ "$RC" -eq 0 ] && files_record "$FCTX" "$owner" "$ALL"
+  return 0
+}

package/hooks/ship-gate.sh

@@ -324,8 +324,7 @@ for owner in $OWNERS; do
   [ "$TOOL" = mutmut ] && [ $RC -eq 0 ] && { printf '%s\n' "$OUT" > "$PKEY.out"; cp "$MLOG" "$PKEY.log" 2>/dev/null; }
   [ -n "$SCOPE_RE" ] && [ $RC -eq 0 ] && OUT=$(printf '%s\n' "$OUT" | grep -E "^[[:space:]]*($SCOPE_RE)\.x")
   if [ "$TOOL" = stryker ]; then
-    ALL=$(stryker_findings "$OUT" "$GLOBBED")
-    [ -n "$FCTX" ] && [ $RC -eq 0 ] && files_record "$FCTX" "$owner" "$ALL"
+    stryker_judge || { [ "$STATUS" = 0 ] && STATUS=2; continue; }
   else
     ALL=$(printf '%s\n' "$OUT" | grep -E '[Ss]urvived' \
           | grep -vE '^[[:space:]]*[#│|+-]|# *surviv')

package/lib/scaffold-stryker.mjs

@@ -35,6 +35,7 @@ const CONFIG = {
   tempDirName: ".stryker-tmp",
 };
 import { IGNORER, IGNORER_VERSION } from "./stryker-ignorer.mjs";
+import { TS7_TSCONFIG, usesTs7 } from "./stryker-ts7.mjs";
 
 async function exists(p) {
   try {
@@ -69,6 +70,7 @@ export async function scaffoldStryker(projectDir, rel) {
   const kept = [];
   const patched = [];
   const unreadable = [];
+  const ts7 = await usesTs7(dir, projectDir);
 
   // A project that already runs Stryker keeps every setting it chose. The two
   // entries that wire the ignorer in are added, because a config written before
@@ -95,8 +97,9 @@ export async function scaffoldStryker(projectDir, rel) {
       const coverage = cfg.coverageAnalysis;
       const canSetStatic = coverage === undefined || coverage === "perTest";
       const needsStatic = cfg.ignoreStatic !== true && canSetStatic;
+      const needsTsconfig = ts7 && cfg.tsconfigFile === undefined;
 
-      if (!needsIgnorer && !needsStatic) {
+      if (!needsIgnorer && !needsStatic && !needsTsconfig) {
         kept.push(`${at}stryker.config.json`);
       } else {
         if (needsIgnorer) {
@@ -107,12 +110,14 @@ export async function scaffoldStryker(projectDir, rel) {
           cfg.coverageAnalysis = "perTest";
           cfg.ignoreStatic = true;
         }
+        if (needsTsconfig) cfg.tsconfigFile = TS7_TSCONFIG;
         await writeFile(configPath, JSON.stringify(cfg, null, 2) + "\n");
         patched.push(`${at}stryker.config.json`);
       }
     }
   } else {
-    await writeFile(configPath, JSON.stringify(CONFIG, null, 2) + "\n");
+    const config = ts7 ? { ...CONFIG, tsconfigFile: TS7_TSCONFIG } : CONFIG;
+    await writeFile(configPath, JSON.stringify(config, null, 2) + "\n");
     wrote.push(`${at}stryker.config.json`);
   }
 

package/lib/stryker-ts7.mjs

@@ -0,0 +1,26 @@
+import { readFile } from "node:fs/promises";
+import { join } from "node:path";
+
+// Stryker rewrites tsconfig through TypeScript's JS API, which 7 no longer has
+// (stryker-js#6111). Naming a file that does not exist skips the rewrite.
+export const TS7_TSCONFIG = "no-tsconfig-rewrite-typescript-7-has-no-compiler-api.json";
+
+async function readJson(p) {
+  try {
+    return JSON.parse(await readFile(p, "utf-8"));
+  } catch {
+    return null;
+  }
+}
+
+// A range like "catalog:" or "workspace:*" carries no number, so the installed copy wins.
+export async function usesTs7(...dirs) {
+  for (const d of dirs) {
+    const installed = await readJson(join(d, "node_modules", "typescript", "package.json"));
+    const pkg = await readJson(join(d, "package.json"));
+    const version = installed?.version ?? pkg?.devDependencies?.typescript ?? pkg?.dependencies?.typescript;
+    const major = Number(version?.match(/\d+/)?.[0]);
+    if (major) return major >= 7;
+  }
+  return false;
+}

package/package.json

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