diff --git a/docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md b/docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md new file mode 100644 index 000000000..8133319fa --- /dev/null +++ b/docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md @@ -0,0 +1 @@ +| 2026-08-24 | cursor/calculators-show-all-mockups-7ce3 (PR #2337) | ff09c24260e2616b7b7c7bb12e19572aa03fba85 | Run PR sweep: CI fix | Fixed real CI failure: Static PR checks failed on check:design-system-contract arbitraryTracking ratchet — tracking-[-0.01em] in show-all-chip.tsx switched to the sanctioned tracking-[var(--tracking-display)] token (folds per globals.css letterspacing scale doc). Branch was not behind main (0 behind). Sole open review thread already resolved; no unresolved threads found. PR title/body/head changed mid-sweep via a concurrent author push (Cursor cloud agent) before this fix; re-verified against the latest head before pushing. | npm run check:design-system-contract — pass (arbitrary tracking 0); npx vitest run tests/calculators-mode.dom.test.tsx tests/consolidated-mode-home-redirect.test.ts tests/favourites-auth-gate.dom.test.tsx tests/calculators-show-all-mockups.dom.test.tsx — 48 passed; npx eslint on changed file — clean; no provider-backed checks run | diff --git a/docs/branch-review-records/72c5a9775b63dec1da3871de945351009d6a31809d32e80c7108d5b7d5a06c9d.record.md b/docs/branch-review-records/72c5a9775b63dec1da3871de945351009d6a31809d32e80c7108d5b7d5a06c9d.record.md new file mode 100644 index 000000000..eb1f3deda --- /dev/null +++ b/docs/branch-review-records/72c5a9775b63dec1da3871de945351009d6a31809d32e80c7108d5b7d5a06c9d.record.md @@ -0,0 +1 @@ +| 2026-08-24 | cursor/factsheets-topics-page-ec19 (PR #2333) | 7fe94b6198fb8b78674b0b423be57851704f86c7 | Run PR sweep: CI fix + drift sync | Fixed real CI failure: legacyTapClasses (h-11/w-11) in factsheets-topics-browse.tsx bumped to h-12/w-12; merged origin/main (clean); a concurrent upstream push (b2c23389d) then rewrote the same component, resolved by merge taking upstream's version (no more -11 class) and regenerating COMPONENTS.md. 4 review threads already resolved, none new. Remaining CI failure: PR policy blocks on missing Clinical Governance Preflight section (src/lib/mode-secondary-navigation.ts triggers clinicalRisk) — left open, PR body edits are out of scope for this sweep. | npm run check:design-system-contract (incl. design-system-adoption, design-sync-contract) — pass; npx vitest run tests/factsheets-topics-page.dom.test.tsx tests/factsheets-topics-phone-mockups.test.ts tests/factsheets-data.test.ts tests/design-system-adoption.test.ts tests/mode-secondary-navigation.test.ts — 109 passed; npx eslint on changed file — clean; no provider-backed checks run | diff --git a/docs/branch-review-records/ba504bd5ee15c145f50f09fae92928bfbfa0a5b546a8af0732fc674927c67c0c.record.md b/docs/branch-review-records/ba504bd5ee15c145f50f09fae92928bfbfa0a5b546a8af0732fc674927c67c0c.record.md new file mode 100644 index 000000000..ac2b29f81 --- /dev/null +++ b/docs/branch-review-records/ba504bd5ee15c145f50f09fae92928bfbfa0a5b546a8af0732fc674927c67c0c.record.md @@ -0,0 +1 @@ +| 2026-08-24 | claude/review-queue-header-design-of7baq (PR #2338) | c934541ceca611481073c4b761da98b99545b05a | Run PR sweep: drift sync | No code fix needed; branch was only behind origin/main (clean merge-tree). Merged origin/main and pushed. Both existing review threads already resolved. Remaining CI failure: PR policy blocks on missing Clinical Governance Preflight section (src/lib/mode-secondary-navigation.ts change triggers clinicalRisk) — left open, PR body edits are out of scope for this sweep. | npx vitest run tests/therapy-compass-pathways.test.ts tests/route-reachability.test.ts tests/therapy-compass-mode-wiring.test.ts tests/therapy-review-regressions.test.ts tests/mode-nav-contract.test.ts tests/mode-secondary-navigation.test.ts tests/therapy-compass-responsive-contract.test.ts — 113 passed; no provider-backed checks run |