docs(evidence): re-verify duplicate heading-hierarchy PR review item (PR #22 conflict-locked, superseded by #28) against current GitHub state - #89
Conversation
…against current GitHub state (2026-08-11)
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
📝 WalkthroughWalkthroughThe PR adds a dated evidence receipt for the duplicate heading review item. It verifies PR status, the merged fix, current checks, regression tests, live heading outlines, closure criteria, and supersession of the previous receipt. ChangesHeading review closeout
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/evidence/duplicate-heading-prs-2026-08-11.md`:
- Around line 3-6: Update the verification evidence in
duplicate-heading-prs-2026-08-11.md so it does not claim checks occurred after
the current review date: either defer the receipt until August 11, 2026 when the
checks complete, or replace every verification date, including the referenced
sections, with the actual completed-check date while preserving the evidence
scope.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: de0476fb-cdae-46c1-968c-940bade97bd1
📒 Files selected for processing (1)
docs/evidence/duplicate-heading-prs-2026-08-11.md
| Date: 2026-08-11 | ||
| Scope: the review item "Two open PRs carry the same heading-hierarchy fix — | ||
| PR #22 is conflict-locked and superseded" (review queue, unreviewed-by-grok), | ||
| re-verified against the GitHub repository state on 2026-08-11. |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Do not publish future-dated verification evidence.
This receipt states that GitHub and live-site checks ran on August 11, 2026. The current review date is August 10, 2026. Defer this receipt until August 11, 2026 after the checks complete, or replace every verification date with the actual date of completed checks.
Also applies to: 80-83, 102-113
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/evidence/duplicate-heading-prs-2026-08-11.md` around lines 3 - 6, Update
the verification evidence in duplicate-heading-prs-2026-08-11.md so it does not
claim checks occurred after the current review date: either defer the receipt
until August 11, 2026 when the checks complete, or replace every verification
date, including the referenced sections, with the actual completed-check date
while preserving the evidence scope.
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
1 participant
Closes the review item "Two open PRs carry the same heading-hierarchy fix —
PR #22 is conflict-locked and superseded" (review queue, unreviewed-by-grok),
re-verified against the current GitHub state on 2026-08-11.
Verified facts (2026-08-11):
mergeStateStatus: DIRTY(conflict-locked) at close — the item's description was accurate.
sibling duplicate fix(public): remove heading-level skips on home and sibling pages #27 is also CLOSED.
<h1>–<h6>changes in anyHTML and zero changes to
scripts/test-heading-hierarchy.mjsor thetest:headingswiring — no open PR duplicates the heading fix.served pages;
npm run checkpasses andtest:headingspasses 6/6 oncurrent origin/main (d70bea0).
This receipt supersedes the stale 2026-08-09 closeout
(
docs/evidence/duplicate-heading-prs-2026-08-09.md, PR #51), whoseopen-PR snapshot ("five open PRs") no longer reflects the repository.
Docs-only change; no code, no deployment.
Summary by CodeRabbit