docs(review): append babysit sweep ledger rows#1193
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
📝 WalkthroughWalkthroughAdded 2026-07-25 entries to the branch review ledger documenting PR babysit sweeps, merge outcomes, updated ledger SHAs, and an in-progress multipass continuation. ChangesReview ledger
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…/Database into codex/pr-babysit-ledger
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 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/branch-review-ledger.md`:
- Line 827: Update the ledger row for implement-audit-recommendations-fix to use
PR `#1141` instead of PR `#1140`, and replace the malformed replacement character in
the squash-merged description with the intended text while preserving the rest
of the row.
- Around line 826-831: The branch-review ledger contains bare branch labels that
parseLedgerBranches cannot extract; update every affected row at
docs/branch-review-ledger.md lines 826-831 and 834-835 to use the exact
namespace-qualified branch/ref, replacing all six bare branch names while
preserving the existing row details.
- Line 826: Remove the duplicate ledger row for branch
implement-audit-viewport-fixes, keeping the canonical entry at Line 838;
alternatively, clearly mark this row as a correction or supersession rather than
another sweep record. Preserve the append-only ledger’s unique event history and
avoid retaining two equivalent entries.
🪄 Autofix (Beta)
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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 2d100e49-1b89-4e07-9278-0a68a30ebca3
📒 Files selected for processing (1)
docs/branch-review-ledger.md
| | 2026-07-24 | fix-physics-animation-audit (PR #1142) | 11776696ddaa4e3d41f395ea35c22075e3144e7e | Run PR re-sync sweep | Before: CONFLICTING. After: merged origin/main clean. | merge origin/main and/or conflict re-check only; no provider-backed checks run | | ||
| | 2026-07-25 | fix-physics-animation-audit (PR #1142) | bdfe81e15c57d376ff74ddb611a8959b0ae94cc9 | Open-PR maintenance: review fix + drift | Before: 24 commits behind and 1 unresolved P2 thread; CSS changed phone reserve timing without pinning the timing in static/phone-scroll coverage. After: current main is merged; static coverage pins 200/240ms transitions and the motion-enabled phone-scroll sweep asserts the active 200ms reserve transition before geometry checks. | Prettier check pass; `git diff --check` pass; focused Vitest/Playwright not run because repository heavyweight lock is owned by worktree 6314; hosted CI will exercise the updated tests; no provider-backed checks run. | | ||
| | 2026-07-25 | fix-physics-animation-audit (PR #1142) | c88c4516476cae3246e1975ce648dff0f3ecb3f7 | Ledger append-only placement fix | CORRECTION: relocated the five PR #1142-unique ledger rows that had been inserted below the table header / among older entries so they append after the final existing record, without rewriting any other rows' content. Restores the append-only contract called out in the Codex P1. | `npm run check:branch-review-ledger`; no provider-backed checks run | | ||
| | 2026-07-25 | implement-audit-viewport-fixes (PR #1140) | f4ae0a7217e513b893d14400fcfd49f31a3dd090 | PR babysit sweep: sync + threads + CI fix + squash merge | Before: CONFLICTING/DIRTY (stale), threads open, behind main. After: merged origin/main, resolved Codex/CodeRabbit threads, fixed document-viewer keyboard lift + baseline reset + Sources focus restore; Production UI green; squash-merged. | Hosted CI PR required SUCCESS on tip 761765a3f; focused vitest keyboard/overlay contracts. No provider-backed checks run. | |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Avoid recording the same sweep twice.
This row repeats the same branch, HEAD, scope, and outcome already recorded at Line 838, differing only by the trailing prlanded content-diff empty. note. Keep one canonical record or mark the later entry explicitly as a correction/supersession; otherwise the append-only ledger contains duplicate event history that the exact-row validator will not detect.
🤖 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/branch-review-ledger.md` at line 826, Remove the duplicate ledger row
for branch implement-audit-viewport-fixes, keeping the canonical entry at Line
838; alternatively, clearly mark this row as a correction or supersession rather
than another sweep record. Preserve the append-only ledger’s unique event
history and avoid retaining two equivalent entries.
| | 2026-07-25 | implement-audit-viewport-fixes (PR #1140) | f4ae0a7217e513b893d14400fcfd49f31a3dd090 | PR babysit sweep: sync + threads + CI fix + squash merge | Before: CONFLICTING/DIRTY (stale), threads open, behind main. After: merged origin/main, resolved Codex/CodeRabbit threads, fixed document-viewer keyboard lift + baseline reset + Sources focus restore; Production UI green; squash-merged. | Hosted CI PR required SUCCESS on tip 761765a3f; focused vitest keyboard/overlay contracts. No provider-backed checks run. | | ||
| | 2026-07-25 | implement-audit-recommendations-fix (PR #1140) | f4ae0a7217e | Babysit sweep: viewport/keyboard audit � squash-merged after CI green + thread triage | production-ui + pr-required | merged | | ||
| | 2026-07-25 | fix-physics-animation-audit (PR #1142) | e966b5aa972 | Babysit sweep: spring physics / reduced-motion � auto-merged after main sync | pr-required | merged | | ||
| | 2026-07-25 | information-page-shell (PR #1148) | 5b9574af480 | Babysit sweep: unify information-page structure � squash-merged | pr-required | merged | | ||
| | 2026-07-25 | mobile-ergonomics-fixes (PR #1156) | de1a82b4936 | Babysit sweep: mobile touch ergonomics � squash-merged | pr-required | merged | | ||
| | 2026-07-25 | automated-audit-remediations (PR #1158) | aa745922f00 | Babysit sweep: automated audit remediations � squash-merged | pr-required | merged | |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Use slash-qualified branch refs for sweep automation. These ledger rows use bare branch labels that parseLedgerBranches cannot extract.
docs/branch-review-ledger.md#L826-L831: replace each bare branch name with its exact namespace-qualified branch/ref.docs/branch-review-ledger.md#L834-L835: replace both bare branch names with their exact namespace-qualified refs.
📍 Affects 1 file
docs/branch-review-ledger.md#L826-L831(this comment)docs/branch-review-ledger.md#L834-L835
🤖 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/branch-review-ledger.md` around lines 826 - 831, The branch-review
ledger contains bare branch labels that parseLedgerBranches cannot extract;
update every affected row at docs/branch-review-ledger.md lines 826-831 and
834-835 to use the exact namespace-qualified branch/ref, replacing all six bare
branch names while preserving the existing row details.
| | 2026-07-25 | fix-physics-animation-audit (PR #1142) | bdfe81e15c57d376ff74ddb611a8959b0ae94cc9 | Open-PR maintenance: review fix + drift | Before: 24 commits behind and 1 unresolved P2 thread; CSS changed phone reserve timing without pinning the timing in static/phone-scroll coverage. After: current main is merged; static coverage pins 200/240ms transitions and the motion-enabled phone-scroll sweep asserts the active 200ms reserve transition before geometry checks. | Prettier check pass; `git diff --check` pass; focused Vitest/Playwright not run because repository heavyweight lock is owned by worktree 6314; hosted CI will exercise the updated tests; no provider-backed checks run. | | ||
| | 2026-07-25 | fix-physics-animation-audit (PR #1142) | c88c4516476cae3246e1975ce648dff0f3ecb3f7 | Ledger append-only placement fix | CORRECTION: relocated the five PR #1142-unique ledger rows that had been inserted below the table header / among older entries so they append after the final existing record, without rewriting any other rows' content. Restores the append-only contract called out in the Codex P1. | `npm run check:branch-review-ledger`; no provider-backed checks run | | ||
| | 2026-07-25 | implement-audit-viewport-fixes (PR #1140) | f4ae0a7217e513b893d14400fcfd49f31a3dd090 | PR babysit sweep: sync + threads + CI fix + squash merge | Before: CONFLICTING/DIRTY (stale), threads open, behind main. After: merged origin/main, resolved Codex/CodeRabbit threads, fixed document-viewer keyboard lift + baseline reset + Sources focus restore; Production UI green; squash-merged. | Hosted CI PR required SUCCESS on tip 761765a3f; focused vitest keyboard/overlay contracts. No provider-backed checks run. | | ||
| | 2026-07-25 | implement-audit-recommendations-fix (PR #1140) | f4ae0a7217e | Babysit sweep: viewport/keyboard audit � squash-merged after CI green + thread triage | production-ui + pr-required | merged | |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Correct the PR number and malformed text.
implement-audit-recommendations-fix is identified as PR #1141 elsewhere in this ledger, not PR #1140. This row currently misattributes the review record; replace it with the correct PR number and remove the replacement character in � squash-merged.
🤖 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/branch-review-ledger.md` at line 827, Update the ledger row for
implement-audit-recommendations-fix to use PR `#1141` instead of PR `#1140`, and
replace the malformed replacement character in the squash-merged description
with the intended text while preserving the rest of the row.
Summary
docs/branch-review-ledger.mdrows for PRs merged during the 2026-07-25 open-PR babysit sweep (fix(viewport): implement mobile keyboard and touch target audit fixes #1140, fix(ui): remediate spring physics animations, GPU layer compositing, and reduced-motion presets #1142, Unify information-page basic structure across modes #1148, Fix: Implement mobile touch ergonomics audit recommendations #1156, fix: implement automated audit remediations #1158, Optimize Cursor codebase indexing ignore rules #1171, docs: close ledger #009 #010 #032 #041 #063 #1175, docs: record canary comparison preflight #1180, Sitewide design/UX review ledger + design-review subagent #1181, Fix scoped-label pagination and stabilize reconciliation preflight test #1191) plus an in-progress sweep note.Test plan
npm run check:branch-review-ledgerlocally against the appended fileNotes
RAG impact: no retrieval behaviour change — ledger documentation only.
Made with Cursor
Summary by CodeRabbit