Files
club-3090/scripts/lib
noonghunnaandClaude Opus 4.7 49d9bb4313 v0.8.0 [review] pre-tag fixes: scrub internal-path leaks from shipped source + make .pull-captures-corpus tests CI-safe (skip-when-absent)
Pre-tag full-branch review caught two classes the per-STEP verification
missed (each gated by transient dev-rig state, not visible per-commit):

1. Internal-path leaks in COMMITTED source (8 files): F-series sub-agent
   docstrings/comments cited the internal locked-brief / design / on-rig
   F8-log absolute paths. Non-functional but ship internal paths in a
   public repo. Scrubbed to non-leaky grounding (which CONTRACT / §;
   point to in-repo docs/LOOP.md). The test-pullemit-capture.sh
   redaction-canary `/opt/ai` strings are deliberately LEFT (they test
   that redaction strips them).

2. CI-robustness: test-pullemit-capture.sh (F6 G1 gate) and test-dedup.sh
   (F5 real-data block) HARD-asserted `>=2 real .pull-captures/ bundles`.
   `.pull-captures/` is gitignored runtime state — populated only after a
   real on-rig pull, ALWAYS absent on a fresh clone / in CI. These passed
   on the dev rig only because on-rig E5/F8 left captures behind; they
   would RED the v0.8.0 tag's CI. Converted to skip-when-absent /
   verify-when-present (the real invariant is the serialization-format /
   round-trip of any captures present, not that the corpus exists).

Verified: full 14-suite run with .pull-captures ABSENT (the exact CI
condition) all RC=0; kv-calc --calibration 22/22; v0.8.0-changeset leak
sweep clean (only legit redaction canaries remain). Comment/docstring +
test-skip logic only — zero production decision-logic change.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
2026-05-17 12:01:25 +00:00
..