@spardutti/claude-skills 2.56.0 → 2.56.2

package/README.md

@@ -36,7 +36,7 @@ npx @spardutti/claude-skills
 ```
 
 ```text
-  Claude Skills Installer v2.56.0
+  Claude Skills Installer v2.56.2
 
   ── Frontend ──────────────────────────────
   ◉ react              ◯ tanstack-query

package/hooks/ship-log-report.mjs

@@ -6,7 +6,9 @@ function records(dir) {
   return [...read(dir), ...read(join(dir, "open"))].flatMap((f) => {
     try {
       const r = JSON.parse(readFileSync(f, "utf8"));
-      return r?.v === 1 && Array.isArray(r.gate) && Array.isArray(r.findings) ? [r] : [];
+      const log = r?.v === 1 && Array.isArray(r.gate) && Array.isArray(r.findings);
+      // A /ship that only merged opens a log and never fills it.
+      return log && (r.gate.length || r.findings.length) ? [r] : [];
     } catch { return []; }
   });
 }

package/hooks/ship-log.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { spawnSync } from "node:child_process";
-import { existsSync, mkdirSync, readFileSync, renameSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readdirSync, readFileSync, renameSync, writeFileSync } from "node:fs";
 import { basename, dirname, join, resolve } from "node:path";
 import { git, parseReport, prSize } from "./ship-log-parse.mjs";
 import { printReport, tally } from "./ship-log-report.mjs";
@@ -45,8 +45,8 @@ function summarize(record) {
   return { minutes: Math.round(minutes * 10) / 10, gateRuns: record.gate.length, gateSeconds: record.gate.reduce((n, r) => n + (r.seconds ?? 0), 0), findings: by };
 }
 
-function finalize(dir, path, record, tag, status, reason) {
-  Object.assign(record, { status, reason: reason ?? null, finished: now() });
+function finalize(dir, path, record, tag, status, reason, at) {
+  Object.assign(record, { status, reason: reason ?? null, finished: at ?? now() });
   record.summary = summarize(record);
   const stem = `${record.app}_${record.finished.slice(0, 10)}_${tag}`;
   let out = join(dir, `${stem}.json`);
@@ -56,8 +56,25 @@ function finalize(dir, path, record, tag, status, reason) {
   console.log(`ship-log: saved ${out}`);
 }
 
+function prOf(branch) {
+  const r = spawnSync("gh", ["pr", "list", "--head", branch, "--state", "all", "--json", "number,createdAt", "--limit", "1"], { encoding: "utf8" });
+  try { return r.status === 0 ? JSON.parse(r.stdout)[0] : undefined; } catch { return undefined; }
+}
+
+// Agents skip the `pr` call, so a log whose branch has a PR by now is closed at that PR.
+function closeOpened(dir, app) {
+  const open = join(dir, "open");
+  for (const f of existsSync(open) ? readdirSync(open).filter((n) => n.startsWith(`${app}__`)) : []) {
+    const path = join(open, f), record = JSON.parse(readFileSync(path, "utf8"));
+    if (!record.gate.length && !record.findings.length) continue;
+    const pr = prOf(record.branch);
+    if (pr) finalize(dir, path, Object.assign(record, { pr: pr.number }), `pr${pr.number}`, "pr-opened", null, pr.createdAt);
+  }
+}
+
 // A ship that never reached pr or end was abandoned; keep it, but never fold it into the next one.
 function restart(dir, path, record, id) {
+  closeOpened(dir, id.app);
   if (existsSync(path)) finalize(dir, path, record, id.branch.replace(/[^\w.-]+/g, "-"), "abandoned");
   save(path, load(path, id));
 }

package/hooks/ship-mark.sh

@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
+HERE="$(cd "$(dirname "$0")" && pwd)"
 cd "${CLAUDE_PROJECT_DIR:-$PWD}" 2>/dev/null || exit 1
+NO_LOG=""; [ "${2:-}" = "--no-log" ] && { NO_LOG=1; set -- "$1" "${@:3}"; }
 # The shared .git dir, so a worktree's mark is found from the main folder.
 COMMON=$(cd "$(git rev-parse --git-common-dir 2>/dev/null)" 2>/dev/null && pwd) || exit 1
 BRANCH=${2:-$(git branch --show-current 2>/dev/null)}
@@ -8,7 +10,11 @@ MARK="/tmp/claude-ship-$(printf '%s %s' "$COMMON" "$BRANCH" | cksum | cut -d' '
 
 # A mark lasts 12 hours, so a branch reused later (dev, for a release) needs /ship again.
 case "${1:-}" in
-  begin) touch "$MARK" ;;
+  begin)
+    touch "$MARK"
+    # The log opens here, not in ship.md: agents skipped that step. A failed log never fails the mark.
+    [ -z "$NO_LOG" ] && command -v node >/dev/null && node "$HERE/ship-log.mjs" start >/dev/null 2>&1
+    true ;;
   check) [ -n "$(find "$MARK" -mmin -720 2>/dev/null)" ] ;;
-  *) echo "usage: ship-mark.sh begin | check [branch]" >&2; exit 2 ;;
+  *) echo "usage: ship-mark.sh begin [--no-log] | check [branch]" >&2; exit 2 ;;
 esac

package/package.json

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