firewall_gate: check PyAutoHands out now its leg has landed - #208
Merged
Conversation
PyAutoHands#237 cleared the last of issue #198's 9 tenant-firewall findings and added the `--only` gate to Hands' own PR CI. The Mind-side gate deliberately omitted the Hands checkout while Hands main still carried that finding — including it earlier would have reddened this workflow on drift it could not fix. That reason is now gone. With this, the gate verifies the checker against all three organ mains, so an allowlist over-grant or a check that stopped finding real drift fails the PR that authors it — for every organ, not two of three. Verified against a four-organ root before merging Hands' leg: the check reported the single Hands mismatch, and reported OK once the leg was applied. Negative probe re-run on the cleared tree: a bogus manifest name in the genericised file is still flagged. Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_013xP1c7XRVQJzxoWAoH92GQ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PyAutoLabs/PyAutoHands#237 cleared the last of issue #198's 9 tenant-firewall
findings and added the
--onlygate to Hands' own PR CI. The Mind-side gatedeliberately omitted the Hands checkout while Hands
mainstill carried thatfinding — including it earlier would have reddened this workflow on drift it
could not fix. That reason is now gone.
With this, the gate verifies the checker against all three organ mains, so an
allowlist over-grant or a check that stopped finding real drift fails the PR
that authors it — for every organ, not two of three.
Changes
PyAutoLabs/PyAutoHandscheckout tofirewall_gate.ymljoined
Verification
Run against a four-organ root (Mind + Brain + Heart + Hands) before merging
Hands' leg, the check reported the single Hands mismatch:
With the leg applied it reports
OK, exit 0. Negative probe re-run on thecleared tree: a bogus manifest name in the genericised file is still flagged,
so the check is not weakened.
Note on issue #198
Both remaining checklist items are now resolved or unblocked. They were not
independent: Heart's
manifest_drift.pyshells out torepos_sync.py --check,so on a full local workspace the tenant-firewall YELLOW reason could not drop
while this Hands finding stood. It can now.
Generated by Claude Code