diff --git a/docs/execution/approved-issue-plans-ledger.md b/docs/execution/approved-issue-plans-ledger.md index 43d3a6538..346e0079a 100644 --- a/docs/execution/approved-issue-plans-ledger.md +++ b/docs/execution/approved-issue-plans-ledger.md @@ -47,7 +47,7 @@ state changes are outside scope. | 12 preview/export failures | #3323, #3290, #3033, #3007, #2609 | blocked corrective work; diagnostic-ready | audit `task_1c6582ccdeae` | `codex-audit-rendering-12-19` | `7a98f6662` → no source change | renderer ownership; share observation harness with 18 | zero drift; staged-preview/error-boundary baselines remain; reports cross persistence/font/PDF/viewer/deployment boundaries | audit focused web/PDF suites, affected typechecks, boundaries, build passed | — | isolate PT Sans, browser/config, and template-selection boundaries; no shared fix | exact JSON/output/browser/config/current reproduction missing | | 13 font/glyph/spacing | #3249, #3159, #3147, #3093, #3089, #2988 | blocked corrective work; diagnostic-ready | audit `task_1c6582ccdeae` | `codex-audit-rendering-12-19` | `7a98f6662` → no source change | serialize font sources with 14/27 | merged metrics/cache/Unicode fixes present and passing; residuals require per-font/per-symptom fixtures | audit focused PDF/font suites, affected typechecks, boundaries, build passed | — | retain regressions; create residual unit only from exact failing font fixture | missing Ropa Sans/source strings/font hashes/before-after artifacts | | 14 RTL export layout | #3275 | blocked corrective work; diagnostic-ready | audit `task_1c6582ccdeae` | `codex-audit-rendering-12-19` | `7a98f6662` → no source change | serialize renderer/fonts with 13/27 | merged canvas-direction fix remains; broader shaping/bidi cause unproved | audit RTL/preview suites, affected typechecks, boundaries, build passed | — | run controlled same-bytes export matrix against approved shaping oracle | exact JSON/font/version and known-good reference absent | -| 15 picture fitting/style | #3168, #3088, #2794, #2782 | 15A published; two hosted findings fixing; other causes blocked | `task_855387612fac` / `ctx_3e9f9c80147d` | `codex/issue-2782-picture-fit` | `7a98f6662` → `6145d5a59`, fix active | coordinate 06/25; schema/PDF owner exclusion | hosted review accepted published input-schema optionality and failed same-file retry; rejected metadata flag, map/crop, and docstring requests as out of scope or non-defects | all exact-head hosted checks green before fix; independent review reran focused suites, affected typechecks, boundaries, and narrow Biome | [#3461](https://github.com/amruthpillai/reactive-resume/pull/3461) | fix two proven defects, independently rereview exact new head, then reply and resolve all threads | delivery/centering issues still lack source revisions/assets/expected crop | +| 15 picture fitting/style | #3168, #3088, #2794, #2782 | 15A published; hosted rereview active; other causes blocked | `task_1116a06f2543` / `ctx_636533e5113e` | `codex/issue-2782-picture-fit` | `7a98f6662` → `6e071ebcd` | coordinate 06/25; schema/PDF owner exclusion | published input-schema optionality and failed same-file retry fixed; metadata flag, map/crop, and docstring requests remain rejected as out of scope or non-defects | 172 focused tests, five typechecks, boundaries, build, narrow Biome, and authenticated E2E green; hosted rerun partly pending | [#3461](https://github.com/amruthpillai/reactive-resume/pull/3461) | independently rereview exact head, resolve remaining dispositions, obtain terminal hosted state | delivery/centering issues still lack source revisions/assets/expected crop | | 16 imported table borders | #3196 | final rereview active | review `task_fac7a8d1cf90` / `ctx_b5d8f7f10380` | `codex/issue-3196-editable-tables` | `7a98f6662` → `d0cca949f` | rich-text owner; must precede 19/31 | fail-closed unsupported grammar, conversion removal, and unrelated-save persistence proof committed; historical fixture still absent | fix gates: web 811, PDF 1,004, focused tests, typechecks, boundaries, build, narrow Biome, and authenticated E2E green | — | independently rereview exact head; fix findings or publish reviewed branch | 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 | @@ -114,7 +114,8 @@ state changes are outside scope. | Plan 15A review fix | `task_5a4afecb7d0d` / `ctx_43faac4f25eb` | same Plan 15A worktree | complete; worker released | commit `6145d5a59`; all three findings fixed; full tests/build and authenticated raster E2E green | | Plan 15A final rereview | `task_2b45ae3b4e98` / `ctx_695a139cd0b9` | same Plan 15A worktree | complete; no findings; worker released | exact head `6145d5a59`; full 70-file behavior, mutation-sensitive geometry, computed CSS, compatibility, scope, full suites, build, and E2E approved before PR #3461 | | Plan 15A hosted review | `task_06bedd03c57a` / `ctx_f3a9297b252e` | same Plan 15A worktree | complete; two changes required; terminal transferred | accepted published-schema optionality and failed same-file retry; rejected metadata flag semantics and Codacy/style warnings; full report at `.orchestration/plan-15a-hosted-review.md` | -| Plan 15A hosted review fix | `task_855387612fac` / `ctx_3e9f9c80147d` | same Plan 15A worktree | implementing | bounded schema/OpenAPI compatibility and same-file retry corrections; commit/push, then independent rereview before any thread mutation | +| Plan 15A hosted review fix | `task_855387612fac` / `ctx_3e9f9c80147d` | same Plan 15A worktree | complete; pushed; worker released | commit `6e071ebcd`; public schema/OpenAPI omit required `picture.fit` while runtime output remains required/defaulted; failed Contain upload clears input for same-file retry; accepted threads auto-resolved | +| Plan 15A hosted rereview | `task_1116a06f2543` / `ctx_636533e5113e` | same Plan 15A worktree | reviewing | fresh independent exact-head/full-diff/thread review, including Mintlify failure causality; no edits or hosted mutation | | Plan 07 hosted review follow-up | `task_6e9bc84ca2d4` / `ctx_e31d0e345240` | `codex/issue-2722-postgres-docs` | no findings; pushed; worker released | commit `a7b8c4c`; five valid comments fixed, `--no-deps` removal rejected; six replies and zero unresolved threads | | Plan 07 late hosted fix | `task_6d0545ee43b1` / `ctx_1d6a1e36def3` | same Plan 07 worktree | complete; worker released | commit `171637526`; separate image/repository build and log paths; Compose/docs gates green | | Plan 07 late fix rereview | `task_8ff818f3e95a` / `ctx_b73be2273e01` | same Plan 07 worktree | complete; no findings; pushed; worker released | exact head `171637526`; full-diff/Compose review and fresh coordinator gates green; hosted checks rerunning | @@ -132,7 +133,7 @@ state changes are outside scope. | --- | --- | --- | --- | --- | | PR #3453 | open, non-draft, mergeable, approved | `ccd111da894cf7d44cc3dee06c937f70d91fef24` | live 2026-09-05: E2E/autofix/Codacy/CodeRabbit/Greptile successful on exact head | monitor only; keep unmerged | | PR #3454 | open, non-draft, mergeable, approved | `80b0d3ab02cc4292f8a4514db8c2516adc1f9dc3` | live 2026-09-05: E2E/autofix/Codacy/CodeRabbit/Greptile successful on exact head | monitor only; keep unmerged | -| PR #3461 | open, non-draft, mergeable; changes requested | `6145d5a5925373287b87dfaa30ab675ebc84e105` | all hosted checks green; five unresolved threads; independent hosted review proved two defects and rejected three requested changes | bounded fix active; independently rereview new head, rerun hosted gates, reply and resolve; keep unmerged | +| PR #3461 | open, non-draft; changes requested | `6e071ebcd60c5e31a5b7bcc48f8a24c7c491e787` | two accepted threads auto-resolved; three rejected threads remain; autofix/Codacy/Greptile green, E2E/CodeRabbit pending, Mintlify deployment parser failure under investigation | independent rereview active; obtain terminal hosted state, reply/resolve remaining threads with evidence; keep unmerged | | 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