Skip to content

fix(ally-guard): a re-review of one head supersedes, it is not a duplicate (BLO-25764) #1996

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
allyblockcast wants to merge 5 commits into
base: master
Choose a base branch
from
Open

fix(ally-guard): a re-review of one head supersedes, it is not a duplicate (BLO-25764) #1996

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
e3e84e2
fix(ally-guard): a re-review of one head supersedes, it is not a dupl…
Oct 1, 2026
fb0bf2a
fix(ally-guard): count a blocker's findings the way the merge gate do…
kkroo Oct 2, 2026
bb48535
fix(ally-guard): retire a ledger entry only on a whole retiring verb …
kkroo Oct 2, 2026
930d31b
fix(ally-guard): read the disposition ledger from fence-stripped text…
kkroo Oct 2, 2026
f3a0b40
fix(ally-guard): read still-present from the raw body, as the gate's …
kkroo Oct 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion scripts/ally-review-consistency-baseline.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@
"Empty as of 2026-09-20 (PEN-2847). The six founding entries (#1525 x2, #1360, #1316 x2,",
"#1304) had all self-expired by head movement, exactly as the fingerprint design intends,",
"and the guard warned on every hourly run asking for their removal. An empty list is not a",
"disabled ratchet: with no entries, every finding on a mergeable PR fails the run."
"disabled ratchet: with no entries, every finding on a mergeable PR fails the run.",
"Separately (BLO-25764): the I1 shape four of them suppressed is no longer a violation when",
"the bodies differ - a re-review supersedes. That exemption is I1-only and does not reach",
"I3, which asserts one canonical body and a matching attestation."
],
"entries": []
}
Loading
Loading