diff --git a/docs/execution/approved-issue-plans-ledger.md b/docs/execution/approved-issue-plans-ledger.md index f15138402..0725d1870 100644 --- a/docs/execution/approved-issue-plans-ledger.md +++ b/docs/execution/approved-issue-plans-ledger.md @@ -10,8 +10,8 @@ pass. Issue comments and state changes are recorded when explicitly directed by - Coordinator branch: `codex/issue-execution-ledger` - Started: 2026-09-05, Europe/Berlin - Merge policy changed: 2026-09-06; maintainer authorized immediate merge of complete, mergeable PRs -- Current integrated main evidence: `10eb3bdbc77a903a792f1e6bfa7cba0db9dea130` after twenty-seven approved merges, - including Plan 31 HTML accessibility PR #3481 and Plan 30 ATS extraction diagnostic PR #3482 +- Current integrated main evidence: `11d619d3d9c7da87bb38463147a7aac4bc35b092` after twenty-eight approved merges, + including Plan 30 ATS extraction diagnostic PR #3482 and Plan 18 geometry diagnostic PR #3483 - Status values: `pending`, `diagnosing`, `implementing`, `reviewing`, `published`, `merged`, `no-change`, `blocked`, `declined`, `skipped` - Evidence rule: each terminal disposition needs current source/GitHub proof, focused tests or reproduction evidence, and @@ -54,7 +54,7 @@ pass. Issue comments and state changes are recorded when explicitly directed by | 15 picture fitting/style | #3168, #3088, #2794, #2782 | 15A merged; other causes blocked | implementation/review/finalization chain complete | `codex/issue-2782-picture-fit` | `7a98f6662` → `6e071ebcd` → merge `ab67831e4` | coordinate 06/25; schema/PDF owner exclusion | clean independent exact-head rereview; five threads resolved; fresh CodeRabbit approval; schema/retry fixes valid; rejected requests documented | focused tests/typechecks/boundaries/build/Biome green; exact-head hosted E2E 35/35 and all code checks green; Mintlify parser failure proven pre-existing; normal merge required no bypass | [#3461](https://github.com/amruthpillai/reactive-resume/pull/3461) | partial unit complete | delivery/centering issues still lack source revisions/assets/expected crop | | 16 imported table borders | #3196 | merged | implementation/review chain complete | `codex/issue-3196-editable-tables` | `7a98f6662` → `b170290e9` → merge `999cd618c` | rich-text owner; Plan 19 stacked from reviewed head | table grammar/grid and link semantics now fail closed when Tiptap cannot round-trip; borderless PDF test is color-independent; final focused rereview clean | focused/broad web/PDF tests, typechecks, boundaries, build, narrow Biome, authenticated E2E, independent review, and hosted checks green | [#3464](https://github.com/amruthpillai/reactive-resume/pull/3464) | complete | historical visual equivalence lacks reporter HTML/JSON but feature contract is approved | | 17 list/skill pagination | #2751, #3040 | blocked corrective work; diagnostic-ready | audit `task_1c6582ccdeae` | `codex-audit-rendering-12-19` | `7a98f6662` → no source change | renderer owner shared with 13/14/18 | merged ordered-marker/wrap/level-gap fixes present; current residual not reproduced | audit focused PDF suites, typecheck, boundaries, build passed | — | retain controls; add fix only from exact failing list/skill fixture | #2751 source absent; #3040 current fixture no longer proves residual clipping | -| 18 preview/export geometry | #2683 | remediation active | implementation `task_938999fe6ae1`; review `task_a71aca971d4c`; remediation `task_4ddf17ed5fea` / `ctx_23031cf90f43` | `codex/issue-2683-preview-export-geometry` | `cdb7bdd2f` → `ce372b54b` → integrated `69d2a35cd` | share output-boundary harness with 12; viewer owner exclusion | dedicated PostgreSQL/production-server synthetic matrix passed, but review found diagnostic safety/false-pass/tolerance/source-settling gaps and disputed zoom-scale invariants; no runtime fix | coordinator 64 focused tests, two typechecks, collection, Biome, boundaries 1,445, and diff green before remediation; findings being fixed and full matrix must rerun | — | publish bounded negative evidence only after focused remediation verification | exact reporter JSON/PDF/browser/DPR/zoom/page settings absent | +| 18 preview/export geometry | #2683 | diagnostic merged; runtime fix blocked | implementation/review/remediation/rereview chain complete | `codex/issue-2683-preview-export-geometry` | `cdb7bdd2f` → `ce372b54b` → `5f5dca844` → merge `11d619d3d` | share output-boundary harness with 12; viewer owner exclusion | opt-in production-path E2E verifies exact disposable DB identity, persisted source/revision and export target, positive pages/ink/sentinels, measured zoom scale, stable transform/clip geometry, all formats/margins/fit/overflow, and DPR1/2; synthetic matrix did not reproduce mismatch | dedicated PostgreSQL/production-server E2E 2/2; web 949, PDF 1,073, two typechecks, collection, Biome, boundaries 1,452, diff hygiene; seven review findings remediated and focused rereview clean | [#3483](https://github.com/amruthpillai/reactive-resume/pull/3483) | retain #2683 pending original JSON/browser fixture; no runtime change from negative synthetic evidence | exact reporter JSON/PDF/browser/DPR/zoom/page settings absent; no measured first divergence | | 19 literal rich-text whitespace | #3397 | merged | implementation/review/remediation chain complete | `codex/issue-3397-literal-whitespace` | `38832014b` → `2f6942fb7` → merge `ea97de5ec` | Plan 16 merged; coordinate DOCX/PDF/editor with 31 | literal whitespace preserved through editor transforms, clipboard/table normalization, PDF layout/textkit geometry, and DOCX; final bare `TD`/`TH` gap fixed | post-main web 102, PDF 106, DOCX 20, three typechecks, Biome, diff green; dedicated Chromium E2E 2/2 | [#3472](https://github.com/amruthpillai/reactive-resume/pull/3472) | complete | direct Word/LibreOffice visual unavailable; XML/four-space contract verified | | 20 section restoration | #3378, #3265, #2921 | 20A merged for #2921; residuals split | implementation/review chain complete | `codex/issue-2921-hidden-section-recovery` | `7a98f6662` → `792d7f8e2` → merge `5850230f8` | 20A before 21/31; placement semantics before 29 | collapsed navigation reopens real accordion before deferred row focus/scroll; final independent review found no issues | focused tests, typechecks, boundaries, build, PO checks, diff gates, authenticated E2E, and all hosted checks green; approved/mergeable/clean | [#3462](https://github.com/amruthpillai/reactive-resume/pull/3462) | begin dependent Plan 21 when owner slot permits | #3378/#3265 need version/action/sanitized JSON; defensive fix cannot close them | | 21 section heading visibility | #3060 | merged | implementation/review chain complete | `codex/issue-3060-section-heading-visibility` | `2a4a1583b` → `3ce4321cb` → merge `368858a56` | 20A and 34 merged; unblocks 23B and 31 | Q1–Q3 implemented across schema, builder menus, Move-to defaults, PDF/DOCX visual omission, accessibility labels, generated references, and recovery fixtures | full 19-task suite; post-main schema 103, PDF 125, DOCX 15, web 30; four typechecks, boundaries, Biome, Markdown, diff; independent review clean | [#3477](https://github.com/amruthpillai/reactive-resume/pull/3477) | complete; issue closed by merge | no remaining approved scope | @@ -174,7 +174,7 @@ pass. Issue comments and state changes are recorded when explicitly directed by | Plan 24A date characterization | `task_45f004efc2bc`, review `task_1b18f7c40ea3`, remediation `task_69caca11a49e` | `codex/issue-3155-date-layout-characterization` | complete; workers released; PR #3480 merged | executable deterministic JSON/PNG baselines; exact Meowth exception; raw PDFs omitted; merge `cdb7bdd2f` | | Plan 31 accessibility | implementation `task_f03773b6531d`, review `task_02b2a71db0f6`, remediation `task_75034a30305e` | `codex/issue-2844-accessibility` | complete; workers released; PR #3481 merged | H3/H4 plus safe rich-text semantics; 1,155 reviewer-emitted JS artifacts removed; public/export/manual gates retained; merge `3e62a1d60` | | Plan 30 export evaluation | implementation `task_8ccb73b62e79`; review `task_62e7e0f678f3`; remediation `task_328f1a7caa2c`; rereview `task_27bfa7e77312`; final fix `task_57d3967893fb` | `codex/issue-2845-ats-export-evaluation` | complete; all workers released; PR #3482 merged | commits `f89873f08`, `0e5994f24`, `d17e188b0`; 108 tests and focused rereview evidence; merge `10eb3bdbc` | -| Plan 18 geometry diagnostic | implementation `task_938999fe6ae1`; review `task_a71aca971d4c`; remediation `task_4ddf17ed5fea` / `ctx_23031cf90f43` | `codex/issue-2683-preview-export-geometry` | remediating; implementer/reviewer released | commit `ce372b54b`, current-main integration `69d2a35cd`; harden opt-in, presence, scale, tolerance, persistence/target, clip, and settling evidence before publication | +| Plan 18 geometry diagnostic | implementation `task_938999fe6ae1`; review `task_a71aca971d4c`; remediation `task_4ddf17ed5fea`; rereview `task_e6b96ad69cd7` | `codex/issue-2683-preview-export-geometry` | complete; all workers released; PR #3483 merged | commits `ce372b54b`, `5f5dca844`, integration `25e044c86`; full dedicated matrix 2/2 and focused rereview clean; merge `11d619d3d` | ## Existing PR and residual accounting @@ -204,6 +204,7 @@ pass. Issue comments and state changes are recorded when explicitly directed by | PR #3480 | merged 2026-09-06 | `77a549988` → merge `cdb7bdd2f` | 14 focused tests, PDF typecheck, Biome, boundaries, Markdown, diff; independent artifact findings remediated | characterization complete; keep #3155/#2841 open pending numeric visual geometry | | PR #3481 | merged 2026-09-06 | `acd2a9cfe` → merge `3e62a1d60` | focused 15, full web 949, typecheck, Biome, boundaries; independent heading findings remediated; hosted E2E rerun attempt 2 failed unrelated baseline `picture-rendering` Contain persistence | HTML residual complete; keep #2844 open for public/PDF/DOCX/manual evidence | | PR #3482 | merged 2026-09-06 | `d17e188b0` → merge `10eb3bdbc` | evaluator 108/108, four affected typechecks, Biome, boundaries 1,117; five initial review gaps and one focused-rereview label gap remediated | diagnostic complete; keep #2845 open pending vendor-side evidence; no ATS preset justified | +| PR #3483 | merged 2026-09-06 | `25e044c86` → merge `11d619d3d` | dedicated PostgreSQL/production-server geometry E2E 2/2; web 949, PDF 1,073, typechecks, collection, Biome, boundaries 1,452; seven review findings remediated and focused rereview clean | diagnostic complete; keep #2683 open pending original fixture/browser evidence; no runtime fix warranted | | Residual #2828 | pending product direction | — | stale whole-document concurrent-tab overwrite reproduced in Chromium/PostgreSQL; excluded from 63 | account for separately; do not implement conflict UI until product policy selected | ## Rulings and blockers log @@ -237,7 +238,7 @@ pass. Issue comments and state changes are recorded when explicitly directed by #3470 (`744eaa902`), #3471 (`a6057abd7`), #3472 (`ea97de5ec`), #3473 (`2a4a1583b`), and #3474 (`97f34b7cc`), #3475 (`578cb496a`), #3476 (`66c25efe1`), #3477 (`368858a56`), and #3478 (`5e8284e49`), #3479 (`f783908b0`), #3480 (`cdb7bdd2f`), #3481 (`3e62a1d60`), and - #3482 (`10eb3bdbc`). + #3482 (`10eb3bdbc`) and #3483 (`11d619d3d`). - Plan 10: PR [#3463](https://github.com/amruthpillai/reactive-resume/pull/3463), closed unmerged by maintainer direction; issue [#2836 comment](https://github.com/amruthpillai/reactive-resume/issues/2836#issuecomment-5556080394) records not-planned rationale. - Plan 16: PR [#3464](https://github.com/amruthpillai/reactive-resume/pull/3464), merged as @@ -279,3 +280,5 @@ pass. Issue comments and state changes are recorded when explicitly directed by `3e62a1d604041e3769a7fc8f5b2ec8224d0663a4`; #2844 remains open for public/export/manual accessibility evidence. - Plan 30 diagnostic: PR [#3482](https://github.com/amruthpillai/reactive-resume/pull/3482), merged as `10eb3bdbc77a903a792f1e6bfa7cba0db9dea130`; #2845 remains open pending vendor-side evidence, and current measurements do not justify a preset. +- Plan 18 diagnostic: PR [#3483](https://github.com/amruthpillai/reactive-resume/pull/3483), merged as + `11d619d3d9c7da87bb38463147a7aac4bc35b092`; #2683 remains open pending original JSON/browser evidence, and no synthetic mismatch justified a runtime change.