From 9cffcd9708171ef2005f3417b8b0389bb12fa4ad Mon Sep 17 00:00:00 2001 From: BigSimmo <87357024+BigSimmo@users.noreply.github.com> Date: Tue, 28 Jul 2026 19:41:06 +0800 Subject: [PATCH] docs: record dirty-work consolidation reviews --- docs/branch-review-ledger.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/branch-review-ledger.md b/docs/branch-review-ledger.md index 29f9bf0f39..78f708bc74 100644 --- a/docs/branch-review-ledger.md +++ b/docs/branch-review-ledger.md @@ -1190,3 +1190,6 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | cursor/mode-secondary-navigation-dc4e | e33997dbdd4a62689ff17ff1f536d9e830ae22ad | pr-1336-babysit | fixed Codex P1 horizontal scroll + P2 service section ids; merged main | vitest-62,eslint,tsc | | 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `84d846d8d0d168ca2babcc6d699e0a88bb0379c0` | Inspect closeout: sync main + forced-colors scope | FIXED. GitHub CONFLICTING was unpushed main sync (local merge-tree CLEAN, 3 behind on remote tip). Pushed merge. Bugbot P2: removed broad forced-colors `!important` wipe on `.edge-glass-header`/`[aria-selected=true]`/`.surface-raised` (token remap retained; header Canvas fill already earlier). Mockup board `z-[2147483647]` → ladder `z-[100]`. Prior CodeRabbit/Codex threads remain resolved. | lint/typecheck/format:check PASS; vitest 4197; local build PASS; no provider checks. | | 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `f7c34fd7` | Sync main #1297 CSS/motion audit | FIXED. GitHub CONFLICTING after main landed #1297; `git merge origin/main` clean (globals.css + ledger auto-merge). Retained content-auto, scoped forced-colors token remap, mockup z-[100]. Prior tip hosted PR required SUCCESS. | merge-tree CLEAN; focused nav test PASS; no provider checks. | +| 2026-07-28 | codex/consolidate-platform-reliability | aa178d6465ff9aeb92b02766f4accff522c0f8a8 | dirty-work consolidation review | APPROVE. Retained reproducible npm-cache installs, explicit dev dependency installs, toolchain parity coverage, and corrected ops preflight wording. Rejected contaminated image work, fail-open audit behavior, duplicated assets, visual-regression machinery, and the unsafe primary-checkout stale-lease recovery. | git diff --check; focused Vitest 8/8; check:github-actions; check:installed-lock-parity; verify:cheap PASS (412 files, 4199 passed, 3 skipped); no provider checks. | +| 2026-07-28 | codex/consolidate-navigation-overlays | 2e30b2bb39a99a48fc95fd76374520ce592f5854 | dirty-work consolidation review | APPROVE. Retained only current-main-compatible responsive sheet max-height precedence. Rejected stale sidebar/routes, calculator removals, broken overlay imports, and focus/inert regressions. | git diff --check; focused sheet DOM Vitest 6/6; verify:cheap PASS (412 files, 4199 passed, 3 skipped); no provider checks. | +| 2026-07-28 | codex/consolidate-design-mockups | d120dbcca5ab4c5caab85176aace052788320405 | dirty-work consolidation review | APPROVE. Isolated calculator and therapy-navigation experiments under mockup routes, kept production calculators/navigation unchanged, and fixed the final decorative-icon accessibility lint finding. | boundary Vitest 3/3; design-system contract PASS; sitemap PASS; mockup Playwright 16/16; verify:cheap PASS (413 files, 4201 passed, 3 skipped); no provider checks. |