- Notifications
You must be signed in to change notification settings - Fork 0
docs: correct Gate E PR state and backfill missing review records#2248
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.
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
Merged
+3
−1
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading. Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading. Please reload this page.
Diff view
Diff view
There are no files selected for viewing
1 change: 1 addition & 0 deletions
1 ...ords/d021712c69d85d2e42499fadabae91ada5396e3de2eb4652d748b09fae84ba8c.record.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| | 2026-08-21 | claude/issues-reconcile-post-2206 | 02d1bcaab01f933b8af3eefa3a7f085e95aac661 | docs(issues): reconcile the queued inbox batch into the canonical ledger (#2218) | Backfilled record — PR #2218 merged 2026-08-21 (squash 02d1bcaab01f933b8af3eefa3a7f085e95aac661) with no review record captured at the time. Diff inspected post hoc: 29 immutable inbox requests applied verbatim into docs/outstanding-issues.md plus 2 closures (#VZN8G3 dup of #BSBE9B, #Q5JHBJ archived) verified against the ledger by the PR author; docs-only, classifyPullRequestFiles clinicalRisk:false, no RAG/src/worker/supabase paths touched. Approved by content. | PR-reported: reconciler applied 29/29 requests cleanly; pre-push guard ran check:ledger-write-discipline on pushed commit 42dc0e0f4 with no rejection; Prettier clean on changed markdown; CI static-pr required checks green (merged). Not independently re-run in this session — docs-truth backfill only. | | ||
1 change: 1 addition & 0 deletions
1 ...ords/f97d90d561a026c4d126c174fc5ab7882423a4f351a6158170f0ad2fa240c3c3.record.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| | 2026-08-21 | claude/shadow-runbook-operator-answers | 1fe3c60c94822f61a60e5b61858114274fb8b340 | docs(worker): answer two of the four B4 operator unknowns from measured evidence (#2228) | Backfilled record — PR #2228 merged 2026-08-21 (squash 1fe3c60c94822f61a60e5b61858114274fb8b340) with no review record captured at the time. Diff inspected post hoc: docs/worker-deploy-runbook.md updated with measured Railway variable-change and memory-headroom evidence (read-only lookups only, no mutation), plus a one-line worker/main.ts startup log-message correction (rollback now states set-then-deploy, no behaviour change) and one queued issues:update inbox request. classifyPullRequestFiles clinicalRisk:false; shadow extraction remains default-off everywhere. Approved by content. | PR-reported: npm run docs:check-links passed (2063 references resolve); Prettier clean on both changed files; CI static-pr required checks green (merged). verify:pr-local full and provider-backed gates explicitly not run per PR body (docs/log-message scope). Not independently re-run in this session — docs-truth backfill only. | |
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
Oops, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When
ledger:lookupis run for either feature branch, it resolves the branch's pre-merge PR head and requires both the ref and HEAD to match; these two backfills instead pair the branch refs with merge squash SHAs (02d1bca…and1fe3c60…). Reproducing the documented lookup therefore returnsNOT REVIEWEDand reports the record only as belonging to a different HEAD, defeating review throttling. Append superseding records using each PR's actual reviewed head SHA rather than its squash commit.AGENTS.md reference: AGENTS.md:L147-L147
Useful? React with 👍 / 👎.