diff --git a/docs/branch-review-records/018d666bbf82061b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md b/docs/branch-review-records/018d666bbf82061b551bd8c52246310c88abce21a64e5468114fe97cd744ce40.record.md new file mode 100644 index 000000000..c3f4eae4c --- /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. | diff --git a/docs/branch-review-records/0f01b1cc3970eec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md b/docs/branch-review-records/0f01b1cc3970eec31955bc53d6624ef30e68c315f7df2b4ec09401f6e3391d31.record.md new file mode 100644 index 000000000..35dfcf950 --- /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 000000000..9eb56c867 --- /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 000000000..7c43836af --- /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/82e2fac319f19c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md b/docs/branch-review-records/82e2fac319f19c945709ba44a8882aafde1f18fd49cf471631f517ddb2b4d74e.record.md new file mode 100644 index 000000000..8345b4423 --- /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 000000000..421f4a116 --- /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. | diff --git a/docs/branch-review-records/f02d1a581a75dd0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md b/docs/branch-review-records/f02d1a581a75dd0dfabb583935cde7fc2ed1dc52565410a11b31a087aa3789d9.record.md new file mode 100644 index 000000000..1b1252649 --- /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 |