Skip to content

docs: keep PR bodies full-prose under caveman output styles#1335

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

docs: keep PR bodies full-prose under caveman output styles #1335

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e449ecc
docs: keep PR bodies full-prose under caveman output styles
cursoragent Jul 28, 2026
c0fe859
docs(ledger): record six-PR consolidation review outcomes
cursoragent Jul 28, 2026
5270dc9
docs(ledger): pin #1335 tip after consolidation ledger commit
cursoragent Jul 28, 2026
3df6945
merge origin/main into caveman PR-policy docs (ledger fold)
cursoragent Jul 28, 2026
91e2405
merge origin/main into #1335
cursoragent Jul 28, 2026
1b1dc92
docs(ledger): use resolvable branch refs for six-pr consolidation rows
cursoragent Jul 28, 2026
4dcf7ef
merge origin/main into #1335 after #1371
cursoragent Jul 28, 2026
2f75e57
merge origin/main into #1335 after #1341
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
9 changes: 9 additions & 0 deletions docs/branch-review-ledger.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -1195,6 +1195,15 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie
| 2026-07-28 | PR #1304 / `fix-test-run-lock` | `7cc32c053c752bef19f3de408a1376428e54af74` | CI babysit: sync main | FIXED. GitHub CONFLICTING/DIRTY was staleness only (`git merge-tree` CLEAN; 7 behind). Merged origin/main. Required CI was already SUCCESS on prior tip `e6b826ed`; no product conflicts. Unique vs main remains knip.json (+ ledger). Bugbot/review threads: none unresolved. | merge-tree CLEAN; merge origin/main; no provider-backed checks. |
| 2026-07-28 | PR #1310 / `claude/branch-review-ledger-fixes-42575f` (merged) | 422e43d86a69c88368454065c2b117f5982a43d6 | prlanded | LANDED as squash 422e43d86. Ledger repair + lookup/append tooling + hardened guard all present on main; guard PASS at 1107 records and repo-hygiene 25/25. Review improved the branch before merge and main is ahead of the authoring branch: findReviews now compares scope exactly (the original substring match would have let a branch-cleanup-deletion-pending row satisfy a branch-cleanup lookup and skip a branch that still needed cleanup), refTokens no longer false-hits on bare parenthetical prose, headMatches accepts an annotated 'sha (squash)' cell and rejects 'n/a - see <sha>', and resolveHead now verifies full-length hex so a mistyped 40-char string cannot become an unmatchable HEAD. Authoring branch was deleted at merge; its unpushed local ledger-record commit was superseded by this row rather than pushed. | npm run check:branch-review-ledger PASS (1107 records) and vitest tests/repo-hygiene.test.ts 25/25 PASS, both run against origin/main after the merge. Pre-merge npm run verify:pr-local PASS on the merged tree (405 files / 4126 tests, build 3.7min). No provider-backed checks run. |
| 2026-07-28 | PR #1305 / execute-audit-remediation-fixes | b101b69631edfe51bcbbc8f6c07e47157fe2c4e8 | CI green closeout after main re-sync | APPROVE for merge by human. Hosted PR required + Production UI PASS on tip after merging origin/main (#1320). MERGEABLE. Unresolved review threads 0. Bugbot-equivalent: no P0/P1/P2 on unique product delta; @cursor review requested. Product delta retained: clinical-notes trust gating answer wipe, SettingsStateProvider wiring, z-index ladder, OverlayProvider/card fixes, phone chrome viewport breakpoints. | Hosted PR policy/Static/Build/Unit/Safety/Advisory/Production UI/PR required PASS on b101b696; check:branch-review-ledger PASS; no provider-backed checks. |
| 2026-07-28 | cursor/search-correctness-030-075-6273 | 134b006b7ff42412e727292c6c03fa4c404252ae | six-pr-consolidation | close-superseded: already on main via #1177/#1204/#1215 | diff-vs-main,merge-tree,gh-merged-history |
| 2026-07-28 | codex/chat-mode-page-navigation-2328 | db7fee69d13239de92800ff83e85610884bdde7d | six-pr-consolidation | ported-to-#1336; tip closed | diff-vs-main,merge-tree,port-review |
| 2026-07-28 | claude/pr-policy-caveman-fix-6875bb | 29c58693d6e63f8174386e6a8cdc94e39f0f8343 | six-pr-consolidation | ported-to-#1335; tip closed | diff-vs-main,content-proofread |
| 2026-07-28 | codex/chat-top-nav-mockups-b3ce | b8f4c658412d8f546f47b64b77e7f274a11018ff | six-pr-consolidation | close-superseded: mockups via #1278 | diff-vs-main,gh-merged-history |
| 2026-07-28 | codex/backlog-diagnostics-reconciliation | c0e8a126bfdba0e7ee22ba8da236c452592524b7 | six-pr-consolidation | close-superseded: canary/eval already on main | diff-vs-main,merge-tree |
| 2026-07-28 | codex/comparison-alias-distinctness | 9f93b1a56fa557e15fa1df8526d28d0b10bd1d21 | six-pr-consolidation | close-superseded: matcher evolved on main | diff-vs-main,merge-tree |
| 2026-07-28 | cursor/pr-policy-caveman-docs-dc4e | e449eccc7e050736519588a89d2c35b2348e4380 | pr-1335 | docs-ready: caveman×pr-policy guidance | content-proofread |
| 2026-07-28 | cursor/mode-secondary-navigation-dc4e | 3712edca0a4ea0638668d316c8696a695d180256 | pr-1336 | nav-port-ready; UI chrome gate pending | vitest-51+58+4186,tsc,eslint |
| 2026-07-28 | cursor/pr-policy-caveman-docs-dc4e | c0fe85980b998a7790e99e4a0ab4a04c01934536 | pr-1335 | docs-ready: caveman×pr-policy + consolidation ledger | content-proofread,check:branch-review-ledger |
| 2026-07-28 | PR #1295 / `fix/audit-remediation-from-main` | `3ff686e4957f5261c3d23d48e7a8195a60a8c800` | Main sync + container-images RAM guard | FIXED. GitHub CONFLICTING/DIRTY after main advanced 9 commits; real conflict only in `scripts/guard-next-build.mjs`. Kept main's `ALLOW_LOW_RAM_BUILD` / `evaluateNextBuildRamGuard` (+ Dockerfile/docker-image.yml wiring) which unblocks the container-images failure (Docker build lacked GITHUB_ACTIONS so the prior GITHUB_ACTIONS-only soften still hard-failed at 7.8 GiB). | Focused vitest guard-next-build+container-ci+therapy-compass 18/18; check:github-actions + ledger PASS; merge-tree CLEAN post-resolve; no provider checks. |
| 2026-07-28 | PR-1297 | 5c7c4ff8cc92a3af1cd0a65898067272f332809e | PR #1297 full diff vs origin/main | APPROVE after main sync; no high-confidence P0-P2 defects | Local diff review and merge-tree clean; prior exact-head PR required, build, unit coverage, and Production UI passed; new exact-head CI pending |
| 2026-07-28 | PR #1304 / `fix-test-run-lock` | `2abe39506f9f0ceaf8cf638bfa2b6dc37dc0ed2c` | CI babysit: resync main | FIXED. After green PR required on `f7d86c27`, main advanced by 1 commit (#1305); GitHub DIRTY again but `git merge-tree` CLEAN. Merged origin/main. No product conflicts; unique delta still knip.json (+ ledger). | merge-tree CLEAN; hosted PR required SUCCESS on prior tip; no provider-backed checks. |
Expand Down