From 78e16e419531cc6cd4e1c1a788bc2a67585ccfdb Mon Sep 17 00:00:00 2001 From: Amruth Pillai Date: Sun, 6 Sep 2026 04:25:38 +0200 Subject: [PATCH] docs: record active review lanes --- docs/execution/approved-issue-plans-ledger.md | 23 +++++++++++-------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/docs/execution/approved-issue-plans-ledger.md b/docs/execution/approved-issue-plans-ledger.md index 953099f60..6d07c4cae 100644 --- a/docs/execution/approved-issue-plans-ledger.md +++ b/docs/execution/approved-issue-plans-ledger.md @@ -55,16 +55,16 @@ pass. Issue comments and state changes are recorded when explicitly directed by | 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 | blocked corrective work; diagnostic-ready | audit `task_1c6582ccdeae` | `codex-audit-rendering-12-19` | `7a98f6662` → no source change | share output-boundary harness with 12; viewer owner exclusion | current viewport/page-size/margin baselines pass; video alone cannot locate cause | audit page/preview tests, affected typechecks, boundaries, build passed | — | compare identical preview/download bytes, MediaBoxes, ink bounds, viewport, DPR | exact JSON/PDF/browser/DPR/zoom/page settings absent | -| 19 literal rich-text whitespace | #3397 | independent review active | implementation `task_d9237449193e`; review `task_a138df5923b5` / `ctx_d1537f68e074` | `codex/issue-3397-literal-whitespace` | `b85d285b6` → `6d9341425` | Plan 16 merged; coordinate DOCX/PDF/editor with 31 | marked whitespace contract implemented across web/PDF/DOCX; current main integrated with three conflicts resolved by retaining already-reviewed Plan 16 behavior | focused web 83, PDF 118, DOCX 20; full Turbo 19 tasks; typechecks, boundaries, build, frozen install, Biome, PostgreSQL E2E 1/1 | — | publish immediately if independent review is clean | direct Word/LibreOffice visual unavailable; XML/four-space contract verified | +| 19 literal rich-text whitespace | #3397 | review remediation active | implementation `task_d9237449193e`; review `task_a138df5923b5`; remediation `task_5afe88028c7c` / `ctx_a1a01a86bac7` | `codex/issue-3397-literal-whitespace` | `b85d285b6` → `6d9341425` → active | Plan 16 merged; coordinate DOCX/PDF/editor with 31 | independent review found one P1 PDF leading-whitespace geometry defect and four P2 editor transform/clipboard/tab-display defects | 232 focused tests plus typechecks, frozen install, Biome, boundaries, and diff passed but direct probes exposed missing acceptance coverage | — | fix five proven defects, focused rereview, then publish | 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 | pending after 20A | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | 20A → 21 → 31 | runtime PDF seam exists; persisted schema/menu/DOCX/accessibility contract absent; Q1–Q3 make unit ready | audit schema/web/PDF/DOCX/resume suites and typechecks/boundaries passed | — | add migration-safe flag, menu, PDF/DOCX visual behavior, retained semantic label | serialize shared section/schema files | -| 22 skill keyword presentation | #2785 | implementing | `task_69bd20897491` / `ctx_ea85ede4cf66` | `codex/issue-2785-skill-keyword-layout` | `695cdb851` → active | must precede/serialize 34 | array currently comma-joined; inline/list contract covers built-in and custom Skills; invalid field-level `display` fixture rejected as unsupported and replaced with valid semantic property coverage | audit schema/web/PDF/DOCX suites and typechecks/boundaries passed; implementation worker active | — | finish and independently review inline compatibility plus semantic list mode | no product gate | +| 22 skill keyword presentation | #2785 | independent review active | implementation `task_69bd20897491`; review `task_546ed601aa62` / `ctx_e423055f28ee` | `codex/issue-2785-skill-keyword-layout` | `b85d285b6` → `d87176ef4` | must precede/serialize 34 | built-in/custom inline/list schema, menus, shared PDF, DOCX, accessible HTML, import, and locales implemented; invalid field-level `display` fixture correctly excluded | 1,280 affected tests, five typechecks, build, boundaries, Biome, and diff green | — | publish immediately if independent review is clean | no product gate | | 23 pagination controls | #3350, #3090 | 23A published; 23B pending; 23C blocked | implementation/review chain complete for 23A | `codex/issue-3090-authored-page-guidance` | `b85d285b6` → `412da288f` | 23B → possible 23C | 23A explains authored layout pages versus renderer-created physical overflow; hosted pagination assertion strengthened; 14 review threads answered and resolved; current main integrated | builder/PDF focused tests, affected typechecks, individual catalog validation, authenticated E2E, Biome/boundaries/build/diff gates passed; replacement hosted E2E and bot rereview active | [#3467](https://github.com/amruthpillai/reactive-resume/pull/3467) | merge after replacement hosted checks and approval; then build 23B matrix | safe oversized-item fallback and widow/orphan UI remain unresolved | | 24 date layout | #3155, #2841 | design-ready only | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | 24A → 24B; mutual exclusion with 32, not dependency | free-text date/period sources and current location-first order verified; geometry contract absent | audit schema/PDF suites and typechecks/boundaries passed | — | run all-template 24A geometry characterization and select width/bounds/default | physical width, bounds, alignment, default; DOCX scope not yet confirmed | | 25 experience company logos | #3379 | blocked implementation; contract-design-ready | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | 25A → 25B; coordinate 06/15 | current generic uploader has eager deletion and no reference counting/ownership validation; unsafe to reuse unchanged | audit relevant package suites/typechecks/boundaries passed | — | define asset ownership, accepted types, retention/orphan/copy/undo contract and tests | storage ownership/lifetime contract requires new decision | | 26 secondary color | #3373 | inventory-ready; implementation blocked | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | 26A → 26B; serialize semantic docs with 28 | theme has three roles; primary consumers are not classified decorative vs semantic | audit schema/PDF suites and typechecks/boundaries passed | — | enumerate primary consumers and approve decorative truth table | consumer classification/visual contract not selected | | 27 offline fonts | #3377 | evidence-ready; implementation blocked | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | 27A → 27B; informs 30A; serialize 13/14 | remote catalog/PDF/web fetch paths verified; no local resolver or complete licensed manifest | audit font/PDF/web suites and typechecks/boundaries passed | — | inventory cold-network requests and build asset/license manifest | licenses, all fallback assets, size, deployment path unknown | -| 28 basics custom styles | #3137 | published; review remediation active | implementation/review complete; remediation `task_5fa47e8b49ca` / `ctx_eee1a308efa3` | `codex/issue-3137-semantic-css-diagnostics` | `b85d285b6` → `d6751831d` | serialize semantic docs/tests with 26 | gradients intentionally unsupported; partial diagnostics/docs/rendered-host unit published after clean independent review; two hosted test-quality findings accepted | resume 1,372, PDF 1,015, web 876 after main integration; typechecks/diff clean; replacement hosted E2E active | [#3468](https://github.com/amruthpillai/reactive-resume/pull/3468) | fix gradient false positives and direct-LINK assertions, then merge when hosted approval/checks pass | reporter has not named remaining failure; keep issue open | +| 28 basics custom styles | #3137 | published; hosted E2E active | implementation/review/remediation complete | `codex/issue-3137-semantic-css-diagnostics` | `b85d285b6` → `28c933b55` | serialize semantic docs/tests with 26 | gradients remain unsupported; false-positive recognition and direct-LINK assertions fixed; both threads resolved and CodeRabbit approved | main-integrated package gates plus remediation resume 47/47, PDF 10/10, typechecks, Biome, and diff green | [#3468](https://github.com/amruthpillai/reactive-resume/pull/3468) | merge immediately when hosted E2E passes | reporter has not named remaining failure; keep issue open | | 29 Onyx profile header | #2812 | blocked | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | placement semantics 20; accessibility order 31 | historical Onyx-only PR obsolete; no persisted generic placement owner exists | audit relevant suites/typechecks/boundaries passed | — | choose generic placement capability and produce one-page/overflow visual contract | new persisted-interface choice plus visual contract required | | 30 ATS export evaluation | #2845 | blocked pending measurement prerequisites | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | renderer baselines 12–19 and font assumptions → 30A → possible 30B | current DOCX mirrors authored layout; internal ATS/extraction analyzers exist; no preset/projection | audit PDF/DOCX/tooling suites and typechecks/boundaries passed | — | run synthetic measurement matrix, make no vendor claims; only then consider pure projection | renderer baselines; material benefit and adapter contract unproved | | 31 document accessibility | #2844 | diagnostic-ready after prerequisites | audit `task_47ed7eb02d48` | `codex-audit-builder-20-34` | `7a98f6662` → no source change | 20 → 21; rich text 16 → 19; renderer baselines → 31A | builder mirror has H1/H2 but strips rich structure; public viewer differs; cross-output audit absent | audit accessible/PDF/DOCX suites and typechecks/boundaries passed | — | audit DOM/PDF/DOCX/public viewer and manual screen reader before remediation | duplicate-announcement risk and reporter evidence; no assumed tagged-PDF fix | @@ -146,12 +146,15 @@ pass. Issue comments and state changes are recorded when explicitly directed by | Plan 32 independent review | `task_0903351a51d2` / `ctx_537efdb21d35` | same Plan 32 worktree | complete; no findings; worker released | exact final head approved after focused behavior/typecheck/catalog/diff review; PR #3465 merged as `b85d285b6` | | Plan 23A implementation/review | `task_fd704c2e7a5d`, `task_2cb223b6dbb9` | `codex/issue-3090-authored-page-guidance` | complete; PR #3467 active | review-fix `322bed31c`; latest-main integration head `412da288f`; all 14 hosted threads resolved; E2E and bot rereview running | | Plan 19 implementation | `task_d9237449193e` / `ctx_656a3cd4360a` | `codex/issue-3397-literal-whitespace` | complete; worker released | exact integrated head `6d9341425`; full local gates and PostgreSQL E2E green | -| Plan 19 independent review | `task_a138df5923b5` / `ctx_d1537f68e074` | same Plan 19 worktree | reviewing | focused Standards/Spec review at exact integrated head; publish immediately if clean | -| Plan 22 implementation | `task_69bd20897491` / `ctx_ea85ede4cf66` | `codex/issue-2785-skill-keyword-layout` | implementing | issue/branch corrected from accidental #3060 label before publication; valid semantic property fixture retained | +| Plan 19 independent review | `task_a138df5923b5` / `ctx_d1537f68e074` | same Plan 19 worktree | complete; changes requested | one P1 PDF leading-whitespace failure plus four P2 editor transform, clipboard, and tab-display failures | +| Plan 19 review remediation | `task_5afe88028c7c` / `ctx_a1a01a86bac7` | same Plan 19 worktree | implementing | fix one P1 PDF geometry and four P2 editor transform/clipboard/tab-display findings | +| Plan 22 implementation | `task_69bd20897491` | `codex/issue-2785-skill-keyword-layout` | complete | exact head `d87176ef4`; 75 files including 55 catalogs; 1,280 affected tests and full build green | +| Plan 22 independent review | `task_546ed601aa62` / `ctx_e423055f28ee` | same Plan 22 worktree | reviewing | focused full-diff Standards/Spec review; no broad suite repeat | | Plan 28 diagnostics/review | `task_e8b11d71f5ec`, `task_30f17b58c95a` | `codex/issue-3137-semantic-css-diagnostics` | complete; PR #3468 active | independent review clean; current-main integration head `d6751831d`; hosted review found two focused test-quality defects | -| Plan 28 hosted remediation | `task_5fa47e8b49ca` / `ctx_eee1a308efa3` | same Plan 28 worktree | implementing | narrow gradient-recognition and direct-LINK assertion fixes; push/reply without another broad review cycle | +| Plan 28 hosted remediation | `task_5fa47e8b49ca` | same Plan 28 worktree | complete | head `28c933b55`; narrow gradient-recognition and direct-LINK assertion fixes pushed; threads resolved; approved | | Plan 08 implementation | `task_3aecece76616` / `ctx_db909cf711e4` | `codex/issue-2669-root-public-resume` | implementing | corrected root public-resume contract; focused static shell/canonical/noindex seam included | | Plan 35 reproduction | `task_6efe013f350a` / `ctx_5f174f7a941d` | `codex/issue-2768-import-reproduction` | diagnosing | controlled synthetic import matrix; no production fix unless deterministic failure repeats three times | +| Plan 33A research | `task_0ed3877760c1` / `ctx_7e91c8f1d077` | `codex/issue-2689-europass-research` | researching | authoritative Europass sources, ResumeData mapping, and reviewable one-page/overflow proposal only; template code blocked on visual approval | ## Existing PR and residual accounting @@ -165,8 +168,8 @@ pass. Issue comments and state changes are recorded when explicitly directed by | PR #3464 | merged 2026-09-06 | `b170290e96b411bce46873b64b345e7d5b8e2fa1` → merge `999cd618c` | independent review and exact-head hosted checks green | complete | | PR #3465 | merged 2026-09-06 | `5c07409ae035549cd743b702b99b2959cb25e183` → merge `b85d285b6` | independent review; refreshed tooling 97/97, resume 75/75, web 29/29; hosted checks green and approved | complete | | PR #3466 | merged 2026-09-06 | `43e6de7e4591b87f211f9ace28363204bbbe30f8` → merge `695cdb851` | four stale recovery hashes independently recomputed after picture-fit default changed canonical bytes; focused 83/83, tooling 97/97, typecheck/Biome/diff green | complete; monitor hosted E2E | -| PR #3467 | open; hosted checks and bot rereview active | `412da288f0164be8c52d7ca51404d0bd2a5880b7` | Plan 23A independent review clean; pagination assertion fixed; 14 threads resolved; current main and focused combined gates green | merge after replacement E2E and approval; keep #3350 out of scope | -| PR #3468 | open; hosted remediation active | `d6751831daaed16bf67b7862c220a344309e4f97` | clean prepublication review and current-main package gates; two CodeRabbit test-quality findings accepted | push focused fixes, then merge after E2E and approval; keep #3137 open | +| PR #3467 | open; approved; E2E rerun active | `412da288f0164be8c52d7ca51404d0bd2a5880b7` | pagination assertion fixed; 14 threads resolved; current main and focused gates green; first replacement E2E failed only unrelated picture/recovery baselines | merge after one failed-job rerun passes; keep #3350 out of scope | +| PR #3468 | open; approved; E2E active | `28c933b555de7aeed9c18b11745b10f2b108eca9` | prepublication review plus focused hosted remediation; two threads resolved; CodeRabbit/Codacy/Greptile/autofix green | merge after E2E passes; keep #3137 open | | 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 @@ -203,5 +206,5 @@ pass. Issue comments and state changes are recorded when explicitly directed by `412da288f0164be8c52d7ca51404d0bd2a5880b7`; current main integrated, hosted assertions fixed, all 14 threads resolved, and replacement checks active. - Plan 28: PR [#3468](https://github.com/amruthpillai/reactive-resume/pull/3468), open at - `d6751831daaed16bf67b7862c220a344309e4f97`; gradients remain unsupported and two focused hosted - review findings are under remediation. + `28c933b555de7aeed9c18b11745b10f2b108eca9`; gradients remain unsupported, both focused hosted + review findings are fixed/resolved, CodeRabbit approved, and E2E remains active.