mirror of
https://github.com/AmruthPillai/Reactive-Resume.git
synced 2026-10-02 17:54:22 +10:00
docs: track plan 09 hosted review follow-up
This commit is contained in:
@@ -41,7 +41,7 @@ state changes are outside scope.
|
||||
| 06 image storage delivery | #2684, #2778 | blocked | audit `task_855fbee0a803` | `codex-audit-backend-01-06` | `7a98f6662` → no source change | coordinate 12/15/25 renderer/image fixtures | #2684 ACL cause fixed by merged #3432 (`35cecf9`); #2778 Browserless path obsolete; current topology unproved | #3432 hosted checks successful; audit storage/upload/PDF tests, typechecks, boundaries passed | #3432 merged before run | deployment retest plus shared real-encoding/backend/render fixture before repair | deployed digest; disposable S3/Garage/SeaweedFS/MinIO/proxy and browser/server raster matrix unavailable |
|
||||
| 07 AIO deployment | #2722 | published | implementation/review chain complete | `codex/issue-2722-postgres-docs` | `7a98f6662` → `b61ca1609` | none | two-doc supported-topology guidance; review-found PostgreSQL update hazard fixed; clean independent re-review | fresh Compose config, Markdown lint, runtime claims, links, diff/scope passed at published head | [#3457](https://github.com/amruthpillai/reactive-resume/pull/3457) | monitor hosted checks/review; keep unmerged | no AIO implementation; optional Unraid host smoke unavailable |
|
||||
| 08 root public resume | #2669 | plan amendment required before implementation | audit `task_9f27829ca50f` | planned `codex/issue-2669-root-resume` | `7a98f6662` → — | none | current password redirect rejects `/`; public view is slug-hook-bound; SSR canonical cannot import server env; expanded auth/canonical contract identified | audit focused API/web suites, typechecks, boundaries passed | — | amend ownership for identity-vs-return auth flow and server-derived canonical root, then dispatch coherent feature | engineering scope correction; feature remains narrower than arbitrary domain/TLS registry |
|
||||
| 09 external version backup | #2705 | published | implementation/review chain complete | `codex/issue-2705-git-backup-docs` | `7a98f6662` → `6931d2cc4` | none | JSON cover-letter scope and throttled template snapshot wording corrected; clean independent re-review | fresh 152 focused tests, Markdown/diff/exact-base/scope green | [#3458](https://github.com/amruthpillai/reactive-resume/pull/3458) | monitor hosted checks/review; keep unmerged | DB integration/E2E unavailable; no sync/remote/whole-account restore |
|
||||
| 09 external version backup | #2705 | published; hosted correction under review | hosted re-review queued `task_a44e374822b4` | `codex/issue-2705-git-backup-docs` | `7a98f6662` → published `6931d2cc4`; local `2d3ca10` | none | clean pre-publication review; hosted bots correctly found `HEAD` could not select earlier revision; selected-commit commands committed locally | 152 focused tests plus Markdown/diff/base/scope green; hosted correction Markdown/diff green | [#3458](https://github.com/amruthpillai/reactive-resume/pull/3458) | independently review correction, push, resolve duplicate threads | DB integration/E2E unavailable; no sync/remote/whole-account restore |
|
||||
| 10 legacy link routing | #2836 | pending implementation after brief correction | audit `task_9f27829ca50f` | planned `codex/issue-2836-retired-slug-notices` | `7a98f6662` → — | none | prospective same-username slug-attempt notice remains coherent; exact DB/API/UI/E2E owner set verified | audit focused API/web suites, DB/API/web typechecks, boundaries passed | — | correct final E2E target to `public-sharing.spec.ts`, then implement migration/API/owner UI atomically | historical cause absent; no redirects/emails/recovery; prospective partial coverage only |
|
||||
| 11 job search policy | #3010 | reviewing | review `task_86a66b578544` / `ctx_3bb8017e8676` | `codex/issue-3010-jsearch-docs` | `7a98f6662` → `e05977007` | none | attachment overclaim removed and restore guidance deduplicated in follow-up commit | Markdown/link/diff/scope green; independent re-review active | — | clean re-review, fresh verify, publish unmerged PR | real provider/browser optional gate unavailable; removal motive unknown |
|
||||
| 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 |
|
||||
@@ -92,6 +92,7 @@ state changes are outside scope.
|
||||
| Plan 02 second review fix | `task_eb3b8c09aef2` / `ctx_3d3e5f7d2496` | same Plan 02 worktree | complete; worker released | commit `8607a5138`; original-form validation plus boxed-string/custom-`toJSON` regressions |
|
||||
| Plan 02 final re-review | `task_ae1f7e591826` | same Plan 02 worktree | ready; awaiting slot | all prior findings and full fixed diff |
|
||||
| Plan 09 re-review | `task_1bd82b008a77` / `ctx_ca2a0176b52f` | same Plan 09 worktree | complete; no findings; worker released | full diff and both factual corrections verified before PR #3458 |
|
||||
| Plan 09 hosted review follow-up | `task_a44e374822b4` | same Plan 09 worktree | local fix committed; review queued | commit `2d3ca10`; select earlier revision with path-filtered log plus chosen commit reference |
|
||||
| Plan 11 independent review | `task_7d5d4c1417a9` / `ctx_17ee05faf9e7` | same Plan 11 worktree | complete; changes required | attachment-format overclaim and duplicated patch/restore guidance |
|
||||
| Plan 11 review fix | `task_351a96e90b69` / `ctx_e398c9128731` | same Plan 11 worktree | complete; worker released | commit `e05977007`; attachment promise removed and adjacent guidance deduplicated |
|
||||
| Plan 11 re-review | `task_86a66b578544` / `ctx_3bb8017e8676` | same Plan 11 worktree | reviewing | full fixed diff and both prior findings |
|
||||
|
||||
@@ -0,0 +1,15 @@
|
||||
# Independently re-review plan 09 hosted-review correction
|
||||
|
||||
Review only. Read pinned approved plan 09, current `AGENTS.md`, RTK, applicable code-review skill, prior Plan 09 review/fix/
|
||||
rereview reports, and both inline comments on PR #3458. Review complete `origin/main...HEAD` diff at current head.
|
||||
|
||||
Verify duplicated hosted finding is resolved: instructions list commits affecting `resume.json`, use a user-selected commit
|
||||
reference rather than `HEAD`, and still recover by saving selected JSON then importing as a new resume. In a disposable
|
||||
local repository with at least two committed resume versions, prove `git log --oneline -- resume.json` identifies both and
|
||||
`git show <selected>:resume.json` returns chosen earlier content. Ensure user-facing commands remain plain Git with no RTK,
|
||||
remote, credentials, push, global config, sync, destructive replacement, or whole-account restore promise.
|
||||
|
||||
Revalidate all previously corrected export/cover-letter/version-history claims, exact two-doc scope, fresh base/live PR
|
||||
state, focused tests, Markdown lint, diff/scope. Write `.orchestration/plan-09-hosted-review-rereview.md` with findings
|
||||
first, exact head, commands/results, skipped gates, risks, and push/thread-resolution verdict. Do not edit tracked files,
|
||||
push, merge, resolve threads, mutate issues, or spawn subagents. Final response at most ten lines.
|
||||
Reference in New Issue
Block a user