diff --git a/docs/branch-review-ledger.md b/docs/branch-review-ledger.md index bb52230af5..7cc0fcc298 100644 --- a/docs/branch-review-ledger.md +++ b/docs/branch-review-ledger.md @@ -1138,6 +1138,8 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1291 / `claude/issues-upload-limit-sync-123366` | `1607558188283d3497683f1067835d96f1031d3c` | CI babysit + merge conflict + Bugbot | FIXED. GitHub CONFLICTING/DIRTY was a real content conflict in `docs/outstanding-issues.md`: main had claimed `#084` for completed per-result grading evidence, colliding with this PR's upload-limit capture. Resolved by keeping main's ledger, renumbering the upload-limit recommendation to `#085`, and bumping `issues:next-id` to `086`. Synced again when main advanced with #1300. CodeRabbit date thread already resolved. Bugbot: zero `cursor[bot]` findings. Required CI green (PR required SUCCESS). | merge-tree CLEAN; prettier + docs:check-links PASS; hosted Change scope/Static/PR required SUCCESS; no provider-backed checks. | | 2026-07-28 | PR #1291 / `claude/issues-upload-limit-sync-123366` | `af140d11d5ca23dee0d8705d9933db967fc8c404` | Babysit closeout tip | Supersedes prior #1291 row at `16075581` after appending the conflict/Bugbot ledger record. Product delta vs main unchanged: `#085` upload-limit capture only. merge-tree CLEAN; awaiting exact-head required checks. | ledger append + check:branch-review-ledger PASS; prior tip hosted PR required SUCCESS. | | 2026-07-28 | PR #1302 / `claude/maturity-ledger-entry` | `64da2c1b34ae101590b8676af12ec6b49c14f0ad` | CI/conflict babysit + Codex threads + Bugbot | FIXED. Real content conflict with main: `#085` already claimed by upload-limit rec (#1291). Merged origin/main; renumbered maturity backlog to `#086`, bumped `issues:next-id` to `087`, added recommended-queue order 29 with go-ahead/RAG/provider stop rules. X7/M1 work orders arrived via main #1299. Codex P2 threads replied + resolved. Bugbot: zero cursor[bot] findings. CircleCI stub from main clears prior "no configuration" status error. | merge-tree CLEAN; prettier + docs:check-links + docs:check-scripts PASS; awaiting exact-head hosted CI; no provider-backed checks. | +| 2026-07-28 | PR #1304 / `fix-test-run-lock` | `352eedfeb4bcec2665201188c1113fceab7d565d` | CI/conflict babysit + Bugbot | FIXED. Real content conflicts vs main (9 files). Merged origin/main; took main for superseded test-run-lock rewrite (lease/heartbeat already present), phone chrome CSS/tests, document-top-navigation mockups, ui-primitives forced-colors, and ultra-review prompts. Kept knip.json cleanup removing unused `ignoreDependencies: ["tailwindcss"]` (only unique product delta). Bugbot: zero `cursor[bot]` findings; 0 review threads. Local `verify:cheap` PASS (4114 tests). Hosted CI re-running on merge tip; mergeable=MERGEABLE. | merge-tree then manual resolve; verify:cheap PASS; Bugbot triage; no provider-backed checks. | +| 2026-07-28 | PR #1304 / `fix-test-run-lock` | `6300b0218b2911fcdd6bc09c51db34dc9bed765b` | Babysit closeout tip | MERGE-READY for knip-only product delta. Supersedes prior #1304 row at `352eedfe` after ledger append. Hosted PR required SUCCESS on exact tip; mergeable=MERGEABLE; 0 review threads; Bugbot empty. Unique vs main: knip.json drops unused `ignoreDependencies: ["tailwindcss"]`. Original test-run-lock/phone-chrome work superseded by main during conflict resolve. | Hosted CI run 30328907503 PR required SUCCESS; verify:cheap PASS earlier; no provider-backed checks. | | 2026-07-28 | PR #1305 / `execute-audit-remediation-fixes` | `7662c94cf85fac19925debb567035c2e3717a20f` | pr-bugbot proactive (zero cursor[bot] Bugbot threads) | FIXED P1 phone-chrome regression: reverted ClinicalDashboard `@container`/`@max-sm:fixed`/`@md:` migration to viewport `sm:`/`md:`/`max-sm:` (search-chrome #20 + reserve contracts). FIXED P2 privacy notice stacking: restored `z-[5]` and allowed rung 5 in z-index ladder. Validated merge-sensitive: trustGatedAnswer clears `answer:""`; upload has no `canonicalAuthority`; SettingsStateProvider wired; no conflict markers. | Vitest chrome/clinical 31/31; eslint touched files clean; no provider-backed checks. | | 2026-07-28 | PR #1305 / `execute-audit-remediation-fixes` | `6c089f2fa5ce5a0e6057dc99ff0199fce136d105` | CI/conflict babysit + Bugbot + PR policy body | FIXED. Merged origin/main (was CONFLICTING); restored fail-closed clinical-notes `answer:""` wipe; fixed upload merge hazard (undefined canonicalAuthority → 500); restored phone chrome viewport breakpoints; kept ClinicalDashboard under 4140-line budget via settingsState handle + memoized provider; updated account-access source contract; completed Clinical Governance Preflight in PR body. Zero unresolved review threads; zero cursor[bot] Bugbot findings. | `npm run verify:cheap` PASS (4115 tests); local pr-policy evaluate ok; no provider-backed checks. | | 2026-07-28 | PR #1305 / `execute-audit-remediation-fixes` | `b488485d075912bc54d514ea65a1cf73c3d9b413` | PR policy body sync closeout | SUPERSEDES prior #1305 babysit tip. Synced complete Summary/Verification/Risk/governance via temporary `PR_POLICY_BODY.md` (gh/API cannot edit PR body — 403), then deleted template. PR policy PASS; mergeable CLEAN. | Sync PR policy body SUCCESS; PR policy PASS; `verify:cheap` prior tip green. | @@ -1157,5 +1159,9 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1310 / claude/branch-review-ledger-fixes-42575f | 7c870c139211a419fe8b4dfacae3195a7a7caa2b | PR babysit: CI + Codex P2s + Bugbot-equivalent | Hosted PR required SUCCESS on 7c870c13. Fixed 3 Codex P2s (exact scope match, supersede mints distinct scope, verify full SHAs via git rev-parse) plus n/a-embedded hex and parenthetical ref-token false matches. 3 review threads replied+resolved. Mergeable; 0 behind main. Hosted Cursor Bugbot check not produced — bot-authored bugbot run/cursor review comments ignored; local Bugbot-style review done and defects fixed. | check:branch-review-ledger PASS; vitest repo-hygiene 25/25; lint; typecheck; full vitest 4133 pass; hosted Static/Unit/Build/Safety/PR-required SUCCESS. No provider-backed gates. | | 2026-07-28 | PR #1305 / execute-audit-remediation-fixes | 71ed5083616acff5b820069f2fd9835eab92ec88 | CI/conflict babysit + Bugbot + ledger merge residue | FIXED. Merged origin/main (#1310 ledger repair); converted 4 merge-residue heading records into table form (unique 2026-07-26 review kept as six-cell row). PR MERGEABLE + PR required PASS on this tip. Unresolved review threads 0. Bugbot-equivalent product-diff review: no P0/P1/P2; @cursor review requested. Trust gating + upload fail-closed + SettingsStateProvider + phone chrome viewport breakpoints retained. | check:branch-review-ledger PASS; focused vitest 41/41; hosted PR policy/Static/Build/Unit/Safety/Advisory/Production UI/PR required PASS; no provider-backed checks. | | 2026-07-28 | codex/universal-search-live-test-fix | 9f5994069cddb8a58308d9d5acb9403948a7f617 | live universal-search owner test handoff | APPROVE. Test-only fix aligns live owner coverage with the intentional federated and focused document timeout contract; no production behavior changed. | Node TypeScript syntax PASS; git diff --check PASS; full local gates blocked by an active exclusive repository lease; hosted required checks pending; no live provider tests run. | +| 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 ', 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 | 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. | +| 2026-07-28 | PR #1304 / `fix-test-run-lock` | `463e5c0adc77fe722e20376666f5991db3e288d9` | CI babysit closeout | MERGE-READY. Hosted PR required SUCCESS on exact tip; mergeable=MERGEABLE; 0 behind main; merge-tree CLEAN. Unique product delta: knip.json removes unused tailwindcss ignoreDependencies. Prior GitHub DIRTY labels during babysit were main-churn only. | Hosted CI run success on 463e5c0a; no provider-backed checks. | +| 2026-07-28 | PR #1304 / fix-test-run-lock | 7cc32c053c752bef19f3de408a1376428e54af74 | CI babysit: sync main | SUPERSEDED (documenting stale-CI ledger error). Prior row for this HEAD incorrectly treated hosted required-CI SUCCESS on earlier tip e6b826ed9150f312c2e7a957f715019e73a7f0be as verification of this later merge commit 7cc32c05. No hosted required-CI result exists for this exact SHA. This ref has since advanced; the later row at 463e5c0adc77fe722e20376666f5991db3e288d9 recorded exact-tip hosted CI SUCCESS, so this commit's status is historical/superseded. | No hosted CI run on this exact HEAD; prior row reused results from e6b826ed; corrective ledger entry only. | diff --git a/knip.json b/knip.json index 556d1e1552..3443ae7bda 100644 --- a/knip.json +++ b/knip.json @@ -9,7 +9,6 @@ ], "project": ["src/**/*.{ts,tsx}", "scripts/**/*.{ts,mjs,cjs}", "worker/**/*.ts", "eslint-rules/**/*.mjs"], "ignore": ["src/lib/supabase/database.types.ts", "**/*mockup*", "supabase/functions/**"], - "ignoreDependencies": ["tailwindcss"], "vitest": { "config": ["vitest.config.mts"] }, "playwright": false }