Skip to content

Governance: record dirty-work consolidation reviews #1363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
BigSimmo merged 2 commits into from
Jul 28, 2026
Merged

Governance: record dirty-work consolidation reviews #1363

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9cffcd9
docs: record dirty-work consolidation reviews
BigSimmo Jul 28, 2026
fe33ca7
Merge origin/main into codex/consolidate-ledger-governance
cursoragent Jul 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions docs/branch-review-ledger.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1201,5 +1201,8 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie
| 2026-07-28 | PR #1316 / `claude/top-search-design-mockups-w53znc` | `7b968d695c4545e1677c2e7f136172ef686d0012` | CI/review closeout: loadError split + adoption mode homes | FIXED new Codex P2s. Separated account loadError from mutation error; expanded band adoption to mode href pages + 2-hop reach; prior #024/favourites-counts/therapy-retry threads already resolved. Merge-tree clean vs main (0 behind). Bugbot earlier pass had no P0/P1 on prior WIP. | vitest favourites-account-retry + adoption + hub (10); typecheck; full suite 4232/4 prior tip |
| 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. |
Comment thread
coderabbitai[bot] marked this conversation as resolved.
| 2026-07-28 | PR-1295 | 2d474c973828736bc199c59136018220debb9907 | PR #1295 final remediation vs current origin/main | APPROVE after exact-head CI; redundant workflows and stale audit artifacts removed, tablet action cards aligned | Clean merge-tree; ledger guard PASS; diff check PASS; verify:pr-local dry-run selected runtime, lock parity, format, lint, typecheck, full test, build, and RAG fixture checks; local execution unavailable because node_modules is absent |
| 2026-07-28 | fix/audit-remediation-from-main | 88deecfb988da030d806b1d8c0a4c8349502a5f8 | stale-checkout P0 regression discovery | P0 regressions found in stale file checkouts | Confirmed affected worker/main.ts and tests/reconciliation-preflight.test.ts; superseded by later remediation and current final review |
Loading