Skip to content

docs(review): append babysit sweep ledger rows #1193

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

Merged
BigSimmo merged 6 commits into from
Jul 25, 2026
+12 −0
Merged

docs(review): append babysit sweep ledger rows #1193

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
71639bf
docs(review): append babysit sweep ledger rows for merged PRs
BigSimmo Jul 25, 2026
1b0c979
merge origin/main into babysit ledger
BigSimmo Jul 25, 2026
756c1da
Merge branch 'main' into codex/pr-babysit-ledger
github-actions[bot] Jul 25, 2026
b72fe95
merge origin/main into ledger PR
BigSimmo Jul 25, 2026
9c12d84
Merge branch 'codex/pr-babysit-ledger' of https://github.com/BigSimmo…
BigSimmo Jul 25, 2026
314ccf0
merge origin/main after #1141
BigSimmo Jul 25, 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
12 changes: 12 additions & 0 deletions docs/branch-review-ledger.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -823,6 +823,18 @@ This file is append-only. Never rewrite or delete an existing review record; app
| 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. |

@coderabbitai coderabbitai Bot Jul 25, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ 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-recommendations-fix (PR #1140) | f4ae0a7217e | Babysit sweep: viewport/keyboard audit — squash-merged after CI green + thread triage | production-ui + pr-required | merged |

@coderabbitai coderabbitai Bot Jul 25, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ 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.

| 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 |
Comment on lines +826 to +831

@coderabbitai coderabbitai Bot Jul 25, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ 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 | cursor-indexing-ignore (PR #1171) | 3a4036580df | Babysit sweep: Cursor indexing ignore rules — squash-merged | pr-required | merged |
| 2026-07-25 | cursor/ledger-009-010-032-041-063-519b (PR #1175) | 87b6b432c19 | Babysit sweep: close ledger #009/#010/#032/#041/#063 — resolved outstanding-issues merge + prettier, squash-merged | static-pr + pr-required | merged |
| 2026-07-25 | canary-comparison-preflight (PR #1180) | 43f261cf229 | Babysit sweep: canary comparison preflight docs — squash-merged | pr-required | merged |
| 2026-07-25 | sitewide-design-review-ledger (PR #1181) | 8284fcd4420 | Babysit sweep: design-review ledger — auto-merged after sync | pr-required | merged |
| 2026-07-25 | codex/complete-all-pending-tasks (PR #1191) | e2488dbb108 | Babysit sweep: scoped-label pagination + order assertion for CodeRabbit thread — squash-merged | unit + pr-required | merged |
| 2026-07-25 | open-pr-babysit-sweep-20260725 | multipass | Babysit continuation: approved action_required workflows; Sources autofocus fix on #1141; ledger dedupe #1157; outstanding-issues merges #1175/#1177; skipped non-trivial conflict clusters #1162/#1185-1190/#1187 draft | gh checks + merge-tree | in-progress |
| 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. prlanded content-diff empty. | Hosted CI PR required SUCCESS on tip 761765a3f; focused vitest keyboard/overlay contracts. No provider-backed checks run. |
| 2026-07-25 | cursor/canary-artifact-comparison-8e05 (PR #1180) | 618d8640fa528de4a94b0d3e2599bcfe0df3f6f5 | PR babysit: retrigger required CI | Empty sync after main advanced; no product change. | No provider-backed checks. |
| 2026-07-25 | cursor/codebase-indexing-optimize-7a2b (PR #1171) | 37a534fb4b89ba504bec00f4c91932a251f0739a | PR babysit: retrigger required CI | Empty sync after main advanced; no product change. | No provider-backed checks. |
Expand Down
Loading