From the round-5 review of #216 (follows #245).
Where: .claude/skills/update-xmech-page/SKILL.md: the Procedure preamble, step 4 and step 11's guard.
The preamble keeps "only the snapshot and its logs" in $SNAP, and step 11 says it removes "the scan logs". But the guard allows only names matching *.log. A stage's output sent to $SNAP/subsets87.out, as the #120 run did, stops step 11 as "not snapshot data", and the snapshot leaks. Nothing tells the refresh to name logs *.log.
Fix: say that logs in $SNAP are named *.log, and have step 4 write stage output to $SNAP/<stage>.log.
From the round-5 review of #216 (follows #245).
Where:
.claude/skills/update-xmech-page/SKILL.md: the Procedure preamble, step 4 and step 11's guard.The preamble keeps "only the snapshot and its logs" in
$SNAP, and step 11 says it removes "the scan logs". But the guard allows only names matching*.log. A stage's output sent to$SNAP/subsets87.out, as the #120 run did, stops step 11 as "not snapshot data", and the snapshot leaks. Nothing tells the refresh to name logs*.log.Fix: say that logs in
$SNAPare named*.log, and have step 4 write stage output to$SNAP/<stage>.log.