From ceb3f63fa2e1cf8887beffc815f11d33abf3c6ad Mon Sep 17 00:00:00 2001 From: Bastien Chanot Date: Mon, 6 Jul 2026 19:24:01 +0200 Subject: [PATCH] job4: SPEC-11 prune-suite-repo-skill-source MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit skills/prune-memory/tests/run-deterministic.sh:11 default changed from $HOME/.claude/skills/prune-memory/SKILL.md to $HERE/../SKILL.md (kept the ${SKILL:-…} env override; reordered HERE's definition before it, since the new default references $HERE). Closes J4-11 (FIXTURE-DRIFT): the suite sourced the INSTALLED path, safe today only because ~/.claude/skills/prune-memory is a symlinked directory back to this repo — if an install ever materializes real copies instead of symlinking, the suite would silently test the wrong (stale) artifact while the shipped SKILL.md drifts unnoticed. Behavior identical today (verified: symlink resolves to the same inode, `diff` confirms byte-identical content). GREEN: real repo, suite still all GREEN (RED-1/2/5/6/7). Red demo (lean scratch copy — skills/prune-memory/{SKILL.md,tests/ run-deterministic.sh} only): moved $HERE/../SKILL.md away → loud `grep`/`awk: cannot open ... No such file or directory` errors, exit 1, RED-2/RED-5 flip status — proves the new default is genuinely what gets read, not a silent fallback. --- skills/prune-memory/tests/run-deterministic.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/skills/prune-memory/tests/run-deterministic.sh b/skills/prune-memory/tests/run-deterministic.sh index d8ad2ce..f6264a5 100644 --- a/skills/prune-memory/tests/run-deterministic.sh +++ b/skills/prune-memory/tests/run-deterministic.sh @@ -8,8 +8,8 @@ # Usage: bash run-deterministic.sh (exit 0 = all green, 1 = >=1 red) set -uo pipefail -SKILL="${SKILL:-$HOME/.claude/skills/prune-memory/SKILL.md}" HERE="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" +SKILL="${SKILL:-$HERE/../SKILL.md}" SANDBOX="$(mktemp -d "${TMPDIR:-/tmp}/prune-red.XXXXXX")" trap 'rm -rf "$SANDBOX"' EXIT