From 841e8450a4541dd98d1e3f89e2bd3ab53f9ea30b Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Tue, 28 Jul 2026 13:05:07 +0000 Subject: [PATCH] docs(ledger): record open-PR usefulness triage Inventory of 19 open PRs: close throwaway/superseded/stale audit continuation; keep product/dependabot work; fold ledger-only PRs. Co-authored-by: BigSimmo --- docs/branch-review-ledger.md | 1 + 1 file changed, 1 insertion(+) diff --git a/docs/branch-review-ledger.md b/docs/branch-review-ledger.md index f8cd26bea9..b5f07ec734 100644 --- a/docs/branch-review-ledger.md +++ b/docs/branch-review-ledger.md @@ -1196,3 +1196,4 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1316 / `claude/top-search-design-mockups-w53znc` | `7b968d695c4545e1677c2e7f136172ef686d0012` | CI/review closeout: loadError split + adoption mode homes | FIXED new Codex P2s. Separated account loadError from mutation error; expanded band adoption to mode href pages + 2-hop reach; prior #024/favourites-counts/therapy-retry threads already resolved. Merge-tree clean vs main (0 behind). Bugbot earlier pass had no P0/P1 on prior WIP. | vitest favourites-account-retry + adoption + hub (10); typecheck; full suite 4232/4 prior tip | | 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `84d846d8d0d168ca2babcc6d699e0a88bb0379c0` | Inspect closeout: sync main + forced-colors scope | FIXED. GitHub CONFLICTING was unpushed main sync (local merge-tree CLEAN, 3 behind on remote tip). Pushed merge. Bugbot P2: removed broad forced-colors `!important` wipe on `.edge-glass-header`/`[aria-selected=true]`/`.surface-raised` (token remap retained; header Canvas fill already earlier). Mockup board `z-[2147483647]` → ladder `z-[100]`. Prior CodeRabbit/Codex threads remain resolved. | lint/typecheck/format:check PASS; vitest 4197; local build PASS; no provider checks. | | 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `f7c34fd7` | Sync main #1297 CSS/motion audit | FIXED. GitHub CONFLICTING after main landed #1297; `git merge origin/main` clean (globals.css + ledger auto-merge). Retained content-auto, scoped forced-colors token remap, mockup z-[100]. Prior tip hosted PR required SUCCESS. | merge-tree CLEAN; focused nav test PASS; no provider checks. | +| 2026-07-28 | open-prs@2026-07-28 | ac786b3553252df48bd13ef414afbcc62e1b41e9 | open-pr-usefulness | TRIAGE of 19 open PRs vs origin/main ac786b35: CLOSE #1364 (author throwaway), #1322 (superseded by #1335), #1352 (stale continuation of merged #1305; 595 behind; real merge-tree conflicts; deletes private-access tests / diverges answer+upload). KEEP product: #1366 Claude settings diagnostic, #1365 issues #096/#097, #1335 caveman×pr-policy docs, #1360 npm-ci CI, #1361 sheet max-h, #1362 calculator/therapy mockups, #1311 doc-nav mockups+viewer, #1351 error/a11y audit, #1298 sheet-focus/P2 tests, #1295 tablet mode-home, #1268/#1269/#1267 dependabot. FOLD ledger-only #1353+#1341+#1363 into one hygiene PR then close the extras. Order: land #1335 before depending on caveman guidance; sync #1361 vs #1298 (same sheet.tsx, different deltas); refresh #1267 after #1366; do not land #096's seven unrun follow-ups piecemeal. | gh-pr-list; git-fetch-prune; ahead/behind+cherry-pick+merge-tree per tip; three-dot file inventory; overlap matrix; #1305 MERGED proof; content compare SettingsStateProvider/Overlay/z-ladder already on main; #1335 body supersedes #1322; no provider checks; no closes performed |