Skip to content

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

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

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

Conversation

@noahgift

Copy link
Copy Markdown
Member

Fixes #6069 (epic #5510). Stacked on #6057 (base is its branch, so the arm script refuses it until #6057 lands and this is retargeted to main). The #6057 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. The || true sit at lines 17 and 26 of this file, not 170/179.

🤖 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 85b3dac. 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