From 8a9ebe10e40d934401635160d6f9b6c97d55b528 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 14:13:22 +0000 Subject: [PATCH 1/5] Add branch review record for PR #2342 sweep Records the Run-PR-style sweep check on PR #2342 (Improve Therapy best-match visibility): review comments already fixed, branch synced from main, CI green on completed checks. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1 --- ...5134c9a0698807da115b718c898168870bbb21c991acc5e39a4.record.md | 1 + 1 file changed, 1 insertion(+) create mode 100644 docs/branch-review-records/539859dde22645134c9a0698807da115b718c898168870bbb21c991acc5e39a4.record.md diff --git a/docs/branch-review-records/539859dde22645134c9a0698807da115b718c898168870bbb21c991acc5e39a4.record.md b/docs/branch-review-records/539859dde22645134c9a0698807da115b718c898168870bbb21c991acc5e39a4.record.md new file mode 100644 index 0000000000..37a4dc1b3e --- /dev/null +++ b/docs/branch-review-records/539859dde22645134c9a0698807da115b718c898168870bbb21c991acc5e39a4.record.md @@ -0,0 +1 @@ +| 2026-08-24 | codex/therapy-best-match-20260824 (PR #2342) | c5701bdf739fa7d56ad5cd06b15e1065e747987a | Run PR sweep: branch drift only | No unresolved review threads on current head (Cursor Bugbot / Codex findings were already resolved before this sweep). No CI failures found. PR was behind main only; synced origin/main into the branch via update_pull_request_branch (clean merge-tree confirmed offline first with git merge-tree). New CI run in progress at sweep end with zero failures in completed jobs (Change scope, Safety and config checks, Build, and most of Static PR checks all green); Production UI x3, Production UI critical, Unit coverage, and Lighthouse budget still running and deferred per the 30-minute dormant-observation cap. | No local gates run (sync only, no code changes). Hosted CI observed only: Change scope=success, Safety and config checks=success, Build=success, Static PR checks green through Lint step; remaining jobs unsettled at sweep end. No provider-backed checks run. | From 4a853f6de5c126c4435796beb30f6191f03778b2 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 14:44:11 +0000 Subject: [PATCH 2/5] docs(ledger): record Run PR sweep of Dependabot PRs #2296, #2297, #2325, #2326 Immutable review records for the four-PR dependency sweep: CI fix on #2325, clean main-syncs on #2296/#2297, and diagnosis-only on #2326 (Node 26 Docker bump incompatible with the engine-strict Node 24 pin). Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1 --- ...ec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md | 1 + ...435a2a3b2d3c6813cf359094d0830024bf6ce81eec0c235a2e2.record.md | 1 + ...993b56b7a99572409e956423257a31bc73a9df056c6e9a86a00.record.md | 1 + ...d0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md | 1 + 4 files changed, 4 insertions(+) create mode 100644 docs/branch-review-records/0f01b1cc3970eec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md create mode 100644 docs/branch-review-records/2e396899ac12a435a2a3b2d3c6813cf359094d0830024bf6ce81eec0c235a2e2.record.md create mode 100644 docs/branch-review-records/5cf197814565d993b56b7a99572409e956423257a31bc73a9df056c6e9a86a00.record.md create mode 100644 docs/branch-review-records/f02d1a581a75dd0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md diff --git a/docs/branch-review-records/0f01b1cc3970eec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md b/docs/branch-review-records/0f01b1cc3970eec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md new file mode 100644 index 0000000000..35dfcf950e --- /dev/null +++ b/docs/branch-review-records/0f01b1cc3970eec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md @@ -0,0 +1 @@ +| 2026-08-24 | dependabot/npm_and_yarn/npm-development-2db4774c1b (PR #2297) | f6f700fa1d8a4756fd9f162d40f8d9e0f20b4a51 | Run PR sweep: main sync | before: CI already green on prior head; branch behind main only (mergeable_state: behind), confirmed clean via git merge-tree. No fix needed. Synced via mcp__github__update_pull_request_branch (authenticated human identity). No unresolved review threads. CI re-running on new head. | no local gate re-run needed (prior head was fully green); no provider-backed checks run | diff --git a/docs/branch-review-records/2e396899ac12a435a2a3b2d3c6813cf359094d0830024bf6ce81eec0c235a2e2.record.md b/docs/branch-review-records/2e396899ac12a435a2a3b2d3c6813cf359094d0830024bf6ce81eec0c235a2e2.record.md new file mode 100644 index 0000000000..9eb56c8677 --- /dev/null +++ b/docs/branch-review-records/2e396899ac12a435a2a3b2d3c6813cf359094d0830024bf6ce81eec0c235a2e2.record.md @@ -0,0 +1 @@ +| 2026-08-24 | dependabot/npm_and_yarn/npm-production-9d7c78ff3c (PR #2296) | 953bfc6c80325c5d873619c690f779a14e543bf2 | Run PR sweep: main sync | before: CI already green on prior head; branch behind main only (mergeable_state: behind), confirmed clean via git merge-tree. No fix needed. Synced via mcp__github__update_pull_request_branch (authenticated human identity). No unresolved review threads. CI re-running on new head. | no local gate re-run needed (prior head was fully green); no provider-backed checks run | diff --git a/docs/branch-review-records/5cf197814565d993b56b7a99572409e956423257a31bc73a9df056c6e9a86a00.record.md b/docs/branch-review-records/5cf197814565d993b56b7a99572409e956423257a31bc73a9df056c6e9a86a00.record.md new file mode 100644 index 0000000000..7c43836afb --- /dev/null +++ b/docs/branch-review-records/5cf197814565d993b56b7a99572409e956423257a31bc73a9df056c6e9a86a00.record.md @@ -0,0 +1 @@ +| 2026-08-24 | dependabot/github_actions/github-actions-a0271f4b22 (PR #2325) | b41957ce29f79c6d8881607b00233e2c035f5fa6 | Run PR sweep: CI fix | before: PR required failing (Static PR checks: check:github-actions pin-allowlist rejected 4 new reviewed SHAs; Unit coverage: tests/codex-run-pr-operator-workflow.test.ts hardcoded old openai/codex-action SHA). Fixed by adding reviewed-pin allowlist entries with release-note review comments and updating the test's expected SHA; merged origin/main in (clean, no conflicts). No unresolved review threads. CI re-running on new head. | node scripts/check-github-action-pins.mjs (passed); npx vitest run tests/codex-run-pr-operator-workflow.test.ts (10 passed); npx eslint on both changed files (clean); no provider-backed checks run | diff --git a/docs/branch-review-records/f02d1a581a75dd0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md b/docs/branch-review-records/f02d1a581a75dd0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md new file mode 100644 index 0000000000..1b12526493 --- /dev/null +++ b/docs/branch-review-records/f02d1a581a75dd0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md @@ -0,0 +1 @@ +| 2026-08-24 | dependabot/docker/docker-images-263a700181 (PR #2326) | 7d57ac89d4f96fd7bb8d2044b41962c6a39d457f | Run PR sweep: diagnosis only | before: PR required failing. Root cause confirmed via job logs: Docker image bump from node:24-bookworm-slim to node:26-bookworm-slim breaks the repo's engine-strict Node 24 pin (package.json engines >=24.15.0 <25) -- npm ci fails with EBADENGINE (Actual node v26.7.0). This is a genuine incompatibility, not a fixable CI flake: bumping past Node 24 needs a coordinated change across package.json engines, CI runner Node version, and setup scripts, which is out of scope for an automated dependency-bump sweep. No fix attempted per explicit task instruction; recommend the PR owner close or defer this PR until the repo is ready to move off Node 24. No unresolved review threads. No branch drift action taken (mergeable_state was 'behind' but fixing it would not change the outcome). | diagnosed via mcp__github__get_job_logs on the failing Container images / build-and-verify job; no local reproduction attempted (would require building a node:26 image against this repo's Node-24-pinned toolchain, which is the exact incompatibility being reported, not verification); no provider-backed checks run | From 5930124f5ff4c6620fb2f1c2514276b8e3b6a950 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 15:04:34 +0000 Subject: [PATCH 3/5] Add branch review record for PR #2339 sweep Records the Run-PR-style sweep check on PR #2339 (therapy comparison mobile design mockups): already fully green, only needed a main sync. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1 --- ...61b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md | 1 + 1 file changed, 1 insertion(+) create mode 100644 docs/branch-review-records/018d666bbf82061b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md diff --git a/docs/branch-review-records/018d666bbf82061b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md b/docs/branch-review-records/018d666bbf82061b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md new file mode 100644 index 0000000000..c3f4eae4c7 --- /dev/null +++ b/docs/branch-review-records/018d666bbf82061b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md @@ -0,0 +1 @@ +| 2026-08-24 | claude/therapy-comparison-mobile-design-z0dagr (PR #2339) | 6ef5b3617956ba8e0a4fa69a1953ec51cba1336f | Run PR sweep: branch sync | Already fully green (PR required success) before sweep; only action was syncing origin/main in (clean merge-tree, no conflicts) via update_pull_request_branch. All 5 review threads were already resolved by the author. Post-sync CI reconfirmed green (PR required success at run 32741127269). | No local gates run — no code change, only a merge-from-main sync; CI (Static PR checks, Lint, Typecheck, Unit coverage, Build, Production UI x3, Production UI critical, Lighthouse budget, PR policy, PR required) reran green on GitHub. No provider-backed checks run. | From 4a689c533d4704e7924e9df517ad689122cbc96d Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 15:05:43 +0000 Subject: [PATCH 4/5] Add branch review records for PR #2341, #2347 sweep Records the Run-PR-style sweep checks on #2341 (dictionary filter band, fixed via main sync, unrelated flake confirmed) and #2347 (browser test gate handoff, fixed stale generated file + doc-link allowlist entries; owner closed the PR mid-sweep for unrelated reasons). Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1 --- ...c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md | 1 + ...2d2c35aeccb703dedcdf0855e198da9ced10731e889b52e1db5.record.md | 1 + 2 files changed, 2 insertions(+) create mode 100644 docs/branch-review-records/82e2fac319f19c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md create mode 100644 docs/branch-review-records/8630ee29b3ed12d2c35aeccb703dedcdf0855e198da9ced10731e889b52e1db5.record.md diff --git a/docs/branch-review-records/82e2fac319f19c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md b/docs/branch-review-records/82e2fac319f19c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md new file mode 100644 index 0000000000..8345b4423a --- /dev/null +++ b/docs/branch-review-records/82e2fac319f19c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md @@ -0,0 +1 @@ +| 2026-08-24 | claude/browser-test-gate-handoff-d5c1db (PR #2347) | d3f5edc26b319f5feccebd628fd537855321f33c | Run PR sweep: CI fixes (owner closed mid-sweep) | Before: PR required failing (Static PR checks: stale outstanding-issues snapshot + 3 broken docs-links; PR policy: Clinical Governance Preflight unchecked) and behind main. Fixed: (1) regenerated data/outstanding-issues-snapshot.json via its generator script; (2) added 3 scoped docs-link-checker allowlist entries in scripts/check-docs-links.mjs for historical/quoted-diagnostic path references in the Phase 2B SDD archive docs (verified locally, not a broken link). Synced origin/main in (clean merge-tree) both times. Static PR checks went green after both fixes. PR policy remained red (Clinical Governance Preflight checkboxes require a PR-body edit, outside this sweep's authorization). The PR owner (BigSimmo) then closed the PR at 14:42 UTC for unrelated reasons (auto-merge armed on incomplete work; a known-false workspace-coverage claim; explained in PR comment) — not an action taken by this sweep. Both fix commits remain on the branch per the owner's own comment (nothing lost). | node scripts/generate-outstanding-issues-snapshot.mjs + node scripts/check-outstanding-issues-snapshot.mjs (pass); node scripts/check-docs-links.mjs (pass, 3322 refs resolve); prettier --check scripts/check-docs-links.mjs (pass). No provider-backed checks run. | diff --git a/docs/branch-review-records/8630ee29b3ed12d2c35aeccb703dedcdf0855e198da9ced10731e889b52e1db5.record.md b/docs/branch-review-records/8630ee29b3ed12d2c35aeccb703dedcdf0855e198da9ced10731e889b52e1db5.record.md new file mode 100644 index 0000000000..421f4a116c --- /dev/null +++ b/docs/branch-review-records/8630ee29b3ed12d2c35aeccb703dedcdf0855e198da9ced10731e889b52e1db5.record.md @@ -0,0 +1 @@ +| 2026-08-24 | cursor/dictionary-persistent-filter-band-4ee0 (PR #2341) | cc99fef611bcb3ed4d7bb2b79434a136b8f61c99 | Run PR sweep: CI diagnosis + drift sync | Before: PR required failing (Production UI shard 3 red on an unrelated differentials-compare-queue test); behind main. Diagnosed via job log: failure was in tests/ui-tools.spec.ts differentials compare queue test, unrelated to this PR's dictionary-only diff. Re-ran the failed job on the pre-sync head — it passed, confirming flake not regression. Synced origin/main in (clean merge-tree) via update_pull_request_branch. Post-sync full CI green (PR required success at run 32741127269). All 4 review threads were already resolved by the author before the sweep; none required action. | No local code changes and no local gates run for this PR — root cause was CI-side (rerun) and branch drift (sync), both diagnosed/fixed via GitHub. No provider-backed checks run. | From 6f8fd91f79d06b1ff2db529ac96ea2ad3f79c68b Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 15:12:12 +0000 Subject: [PATCH 5/5] Add branch review records for PR #2338, #2337, #2333 sweep Records the Run-PR-style sweep checks: #2337 fixed a design-token ratchet failure, #2333 fixed a tap-target size regression and resolved a concurrent-push merge, #2338 was only a main sync. #2333 and #2338 still have an open PR-policy failure (missing Clinical Governance Preflight section) left for the PR author to fill in. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1 --- ...6b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md | 1 + ...ec1da3871de945351009d6a31809d32e80c7108d5b7d5a06c9d.record.md | 1 + ...145f50f09fae92928bfbfa0a5b546a8af0732fc674927c67c0c.record.md | 1 + 3 files changed, 3 insertions(+) create mode 100644 docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md create mode 100644 docs/branch-review-records/72c5a9775b63dec1da3871de945351009d6a31809d32e80c7108d5b7d5a06c9d.record.md create mode 100644 docs/branch-review-records/ba504bd5ee15c145f50f09fae92928bfbfa0a5b546a8af0732fc674927c67c0c.record.md diff --git a/docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md b/docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md new file mode 100644 index 0000000000..8133319fa3 --- /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 0000000000..eb1f3deda6 --- /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 0000000000..ac2b29f81a --- /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 |