Skip to content

fix(gate): check_affected_readers.sh fails closed on grep error / missing dir (PMAT-6069) - #6084

Closed
noahgift wants to merge 1 commit into
gate/affected-set-readers-only-v5from
gate/affected-readers-failclosed-6069-v2
Closed

noahgift wants to merge 1 commit into
gate/affected-set-readers-only-v5from
gate/affected-readers-failclosed-6069-v2

Conversation

@noahgift

Copy link
Copy Markdown
Member

Replaces #6072 (linear rebuild: gate/affected-readers-failclosed-6069-v2 @d01bf4f226, one commit on gate/affected-set-readers-only-v5 @47e8f11783). Fixes #6069 (epic #5510). Stacked: base is the v5 branch, so the arm script refuses it until that base lands and this is retargeted to main. The earlier quorum flagged the || true as non-blocking; the check has no consumer yet. Must-fail rows: missing crates dir, grep rc 2 (shim). Mutant: making g() lenient turns the shim row red.

keep-open: #5510 epic
keep-open: #6072 is closed by wsweep as superseded.

🤖 Generated with Claude Code

…a missing crates dir (#6069)

hits() and build_inputs() ended in `|| true`, so an unreadable or missing crates
dir, or any grep rc>=2, read as "no readers" and the check printed ok. A new g()
treats only rc 1 (no match) as empty; check() is RED on a missing dir and on a
failed scan. Self-test rows: missing dir and a grep shim exiting 2 must be RED,
a tree with no reader forms stays green; the shim row dies when g() is made lenient.

PMAT-Ticket: PMAT-6069
PMAT-Batch: BATCH-aprender-readers-6069
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Member Author

Snapshot: head d01bf4f. Superseded by newer open #6091 on the same topic (PR-explosion cleanup, cop order aprender-57; EMG-7 tickets verified on main or in the successor). Branch kept; closing.

@noahgift noahgift closed this Oct 23, 2026
@noahgift noahgift reopened this Oct 23, 2026
@noahgift

Copy link
Copy Markdown
Member Author

superseded by #6091 (same ticket, newer PR; class A per OPERATOR 10:32Z ruling via aprender-b9). Branch kept.

@noahgift noahgift closed this Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant