Skip to content

docs(ledger): open-PR usefulness triage #1367

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
cursor merged 6 commits into from
Jul 28, 2026
Merged

docs(ledger): open-PR usefulness triage #1367

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
841e845
docs(ledger): record open-PR usefulness triage
cursoragent Jul 28, 2026
aca0944
Merge origin/main into cursor/open-pr-usefulness-triage-7f78
cursoragent Jul 28, 2026
4b8c560
Merge origin/main into cursor/open-pr-usefulness-triage-7f78
cursor[bot] Jul 28, 2026
914928b
Merge origin/main into cursor/open-pr-usefulness-triage-7f78
cursor[bot] Jul 28, 2026
048fefa
Merge origin/main into cursor/open-pr-usefulness-triage-7f78
cursoragent Jul 28, 2026
2998f05
Merge origin/main into cursor/open-pr-usefulness-triage-7f78
cursoragent Jul 28, 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
1 change: 1 addition & 0 deletions docs/branch-review-ledger.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1214,6 +1214,7 @@ 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 |
Comment thread
cursor[bot] marked this conversation as resolved.
| 2026-07-28 | PR-1298 | 56536a4d00a5fc799025522d2dbe98377721837e | PR #1298 final remediation vs current origin/main | APPROVE after exact-head CI; unvalidated retrieval behaviour removed, remaining changes are UI/test hardening | Protected clinical-search and retrieval-variant production files match origin/main byte-for-byte; merge-tree clean; ledger guard PASS; diff check PASS; verify:pr-local dry-run selected full local gate; local execution unavailable because node_modules is absent; RAG impact no retrieval behaviour change |
| 2026-07-28 | claude/navigation-pane-mockups-0600af | 8fb9867483104a5cc89eec5cfb512e1ca8718029 | PR #1311 CI/review fix | Fixed maintainability budget (DocumentViewer extract), sticky-header anchors/rail, lg-only section card, section reading order + non-collapsible source-text; dispositioned CodeRabbit mockup wiring as exempt; 0 unresolved threads; Bugbot none | verify:cheap PASS (419 files/4256 tests); maintainability 1633/1734; vitest section+account-access; eslint/typecheck; sitemap:check; Bugbot none |
| 2026-07-28 | claude/navigation-pane-mockups-0600af | 29372bb5a53df26902bef53f8a1cb5dd4be5b239 | PR #1311 CI/review fix | Supersedes prior head: Prettier format:check was the remaining Static PR / CircleCI blocker; formatted 8 files. Product fixes unchanged; 0 unresolved threads. | format:check PASS; maintainability budgets; prior verify:cheap retained |
Expand Down