docs: track review-ready documentation units

docs/execution/approved-issue-plans-ledger.md

@@ -34,16 +34,16 @@ state changes are outside scope.
 | Unit | Issues | Status / current validity | Owner | Worktree / branch | Base → head | Dependencies | Evidence | Tests | PR | Next action | Blockers |
 | --- | --- | --- | --- | --- | --- | --- | --- | --- | --- | --- | --- |
 | 01 account login/recovery | #3166, #3164, #3078, #3046, #2897, #2837 | blocked | audit `task_855fbee0a803` | `codex-audit-backend-01-06` | `7a98f6662` → no source change | none | zero drift; live issues remain split; #3046/#3078 have merged #3095 candidate only | auth 42, email 6, focused/full API/server suites and affected typechecks/boundaries passed in audit | — | select exact auth/mail/API boundary only after current sanitized trace | current cloud digest, provider/account method, request/status trace, controlled mailbox |
-| 02 hosted v4 recovery | #3181, #2760 | implementing | review fix `task_5fe67c42d856` / `ctx_28ed501db631` | `codex/issue-3181-recovery-procedure` | `7a98f6662` → `3f53deca8` + pending | #2760 identity branch depends on 01 evidence | reviewer reproduced false no-op after lossy schema fallback, contradictory target state, and overstated raw-v4 wording | implementation gates green; independent direct probes prove three findings | — | fix with new RED/GREEN tests, re-review, then publish | real recovery still needs verified owner, snapshot, mapping, private delivery |
+| 02 hosted v4 recovery | #3181, #2760 | reviewing | re-review `task_5bffa8e386f2` / `ctx_57fe2ac671f8` | `codex/issue-3181-recovery-procedure` | `7a98f6662` → `6af21121b` | #2760 identity branch depends on 01 evidence | strict validation, target-state invariant, v5-only wording, and hash-limit corrections committed with new regressions | focused tooling/API/auth/typechecks/boundaries/Biome/Markdown/diff green; independent re-review active | — | publish after clean re-review and fresh verification | real recovery still needs verified owner, snapshot, mapping, private delivery |
 | 03 MCP registration | #3398, #3153 | no-change | audit `task_855fbee0a803` | `codex-audit-backend-01-06` | `7a98f6662` → no source change | coordinate DB/startup with 05 | merged #3421 (`fe9b59e`) present on main; current schema/startup/auth contracts match plan | #3421 hosted checks successful; audit auth/server/API/DB tests, typechecks, boundaries passed | #3421 merged before run | deployment verification only | deployed digest/log correlation and exact Codex/Claude DCR/consent/PKCE/MCP retest unavailable |
 | 04 AI provider compatibility | #2732, #2766, #2723, #2708 | blocked | audit `task_855fbee0a803` | `codex-audit-backend-01-06` | `7a98f6662` → no source change | serialize edits to AI service | zero drift; historical #2708 Responses path and #2766 structured Test path absent; remaining tuples differ | focused API 96 and full API suites/typecheck/boundaries passed in audit | — | select wire/state/import/base-path unit only after exact current tuple fails | provider/model/base URL/path/version/action/error tuple and allowed endpoint unavailable |
 | 05 AI provider migrations | #3152 | blocked | audit `task_855fbee0a803` | `codex-audit-backend-01-06` | `7a98f6662` → no source change | coordinate DB/startup with 03 | zero drift; table schema, migration, startup ordering, Docker copy path coherent; `42P01` historical cause unresolved | audit API/server/DB tests, typechecks, boundaries passed; no real PostgreSQL migration fixture | — | choose config/packaging/startup/migration repair only after disposable reproduction | affected image layout/digest, working directory, DB schema/search path/journal, fresh+upgrade DB fixture |
 | 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 | implementing | review fix `task_8aa48854e7da` / `ctx_5aa2007ec136` | `codex/issue-2705-git-backup-docs` | `7a98f6662` → `d4ef67113` + pending | none | reviewer found embedded-cover-letter scope contradiction and incorrect independent template-checkpoint claim | functional/doc gates green; reviewer independently reproduced source semantics | — | fix two factual statements, re-review, publish unmerged PR | DB integration/E2E unavailable; no sync/remote/whole-account restore |
+| 09 external version backup | #2705 | reviewing | re-review `task_1bd82b008a77` / `ctx_ca2a0176b52f` | `codex/issue-2705-git-backup-docs` | `7a98f6662` → `6931d2cc4` | none | JSON cover-letter scope and throttled template snapshot wording corrected | 152 focused tests, Markdown/diff/scope green; independent re-review active | — | publish after clean re-review and fresh verification | 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 | implementing | `task_58d76423d364` / `ctx_20d4adf43908` | `issue-3010-jsearch-docs`; target `codex/issue-3010-jsearch-docs` | `7a98f6662` → pending | none | history proves JSearch removed in v5.1.0; current provider-native search and supplied-description tailoring verified | implementation verification pending | — | make minimal three-file factual docs correction, then independent review | removal motive unknown; JSearch not restored and live search remains provider/model-dependent |
+| 11 job search policy | #3010 | reviewing | implementation `task_58d76423d364`; review `task_7d5d4c1417a9` / `ctx_17ee05faf9e7` | `codex/issue-3010-jsearch-docs` | `7a98f6662` → `cb011841c` | none | three docs committed with v5.1.0 history, supplied-description workflow, and provider-native capability boundary | implementer reports 16 focused tests, Markdown/diff/scope green; independent review active | — | resolve findings, 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 |
 | 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 |
@@ -79,15 +79,18 @@ state changes are outside scope.
 | Plans 20–34 | `task_47ed7eb02d48` / `ctx_50d8257459ab` | `codex-audit-builder-20-34` | complete; worker release pending | ready: 20A, 21, 22, 23A, 28 diagnostics, 32, 34; remaining plans split at evidence/design gates |
 | Plan 07 implementation | `task_aee702e37eae` / `ctx_ed134b1d79ce` | `codex/issue-2722-postgres-docs` | implementation complete; worker released | commit `e37b73566`; two-file docs change; implementation report complete |
 | Plan 09 implementation | `task_e450ea143bfa` / `ctx_3b575cf1d5c7` | `codex/issue-2705-git-backup-docs` | implementation complete; worker released | commit `d4ef67113`; two-file docs change and synthetic local-Git validation |
-| Plan 11 implementation | `task_58d76423d364` / `ctx_20d4adf43908` | `issue-3010-jsearch-docs`; target `codex/issue-3010-jsearch-docs` | implementing | three-file docs change, source/test validation, commit, independent review |
+| Plan 11 implementation | `task_58d76423d364` / `ctx_20d4adf43908` | `codex/issue-3010-jsearch-docs` | implementation complete; worker released | commit `cb011841c`; three-file docs change and validation report |
 | Plan 02 synthetic recovery | `task_bae016c4d1f2` / `ctx_303f5d1f1031` | `codex/issue-3181-recovery-procedure` | implementation complete; worker released | commit `3f53deca8`; pure comparator, safeguards docs, TDD evidence |
 | Plan 02 independent review | `task_46be9a1a4d82` / `ctx_e9f651d38a65` | same Plan 02 worktree | complete; changes required | false no-op, target-state contradiction, v4 wording overclaim |
 | Plan 07 independent review | `task_b29de2cd974c` / `ctx_46d7f32205a1` | same Plan 07 worktree | complete; changes required | high: all-services pull could cross PostgreSQL major version |
 | Plan 07 review fix | `task_8ea6cfc76ef4` / `ctx_0184e1101bca` | same Plan 07 worktree | complete | commit `b61ca1609`; app-only update recipe and separately pinned database upgrade path |
-| Plan 02 review fix | `task_5fe67c42d856` / `ctx_28ed501db631` | same Plan 02 worktree | implementing | strict validation, target invariant, exact v5-only documentation |
-| Plan 07 re-review | `task_90a9a6b63782` / `ctx_2f1def9119d6` | same Plan 07 worktree | reviewing | full-diff verification after update-safety fix |
+| Plan 02 review fix | `task_5fe67c42d856` / `ctx_28ed501db631` | same Plan 02 worktree | complete | commit `6af21121b`; strict validation, target invariant, exact v5-only documentation |
+| Plan 07 re-review | `task_90a9a6b63782` / `ctx_2f1def9119d6` | same Plan 07 worktree | complete; no findings | full-diff verification after update-safety fix |
 | Plan 09 independent review | `task_129b49e32bc7` / `ctx_68b13c76cc72` | same Plan 09 worktree | complete; changes required | embedded-letter JSON contradiction; template-checkpoint overclaim |
-| Plan 09 review fix | `task_8aa48854e7da` / `ctx_5aa2007ec136` | same Plan 09 worktree | implementing | correct rendered-export scope and throttled template snapshot wording |
+| Plan 09 review fix | `task_8aa48854e7da` / `ctx_5aa2007ec136` | same Plan 09 worktree | complete | commit `6931d2cc4`; correct rendered-export scope and throttled template snapshot wording |
+| Plan 02 re-review | `task_5bffa8e386f2` / `ctx_57fe2ac671f8` | same Plan 02 worktree | reviewing | reproduce prior findings and review full fixed diff |
+| Plan 09 re-review | `task_1bd82b008a77` / `ctx_ca2a0176b52f` | same Plan 09 worktree | reviewing | verify both factual corrections and full diff |
+| Plan 11 independent review | `task_7d5d4c1417a9` / `ctx_17ee05faf9e7` | same Plan 11 worktree | reviewing | history/runtime/workflow accuracy and independent validation |
 
 ## Existing PR and residual accounting
 

docs/execution/briefs/review-plan-11.md

@@ -0,0 +1,23 @@
+# Independent review: plan 11 JSearch/current tailoring documentation
+
+Review only; do not edit, commit, push, open PR, mutate issues, or spawn subagents. Read current `AGENTS.md`, RTK,
+issue/domain guidance, code-review/documentation skills, `.orchestration/plan-11-implementation.md`, and approved plan 11.
+Trust local planning checkout only when HEAD equals `a2557b2ad40e06e1e63eb655f286e6a78fe6bf0d`; otherwise use pinned `git show`
+for `plans/11-job-search-policy.md`.
+
+Fetch current `origin/main` and live issue/PR state. Inspect complete diff and verify every historical/runtime/UI claim against
+Git history and current source/tests:
+
+- v5.1.0 removal timing of JSearch/RapidAPI Job Listings is factual; removal motive remains unknown.
+- Legacy settings redirect, current Integrations/provider setup, supplied job-description tailoring, attachments, review edits,
+  patch inspection, and Restore labels/workflow match current UI/source.
+- Provider-native live web search is clearly capability/provider/model-dependent and not equated with structured JSearch
+  results. Unsupported setups can use pasted/attached content without implied live search.
+- No stale model list, paid API restoration, credentials, quotas, unsupported attachment promise, unverified motive, or issue
+  closing claim. Prose minimal and non-duplicative. Diff exactly three docs.
+
+Rerun agent/capability tests, Git history/source checks, Markdown lint, links, diff/scope gates. Optional real provider/browser
+flow remains a named skip. Findings first, severity-ordered with exact file/line/evidence; otherwise `No findings` plus risks.
+
+Write `.orchestration/plan-11-review.md` with reviewed SHA/base, findings, commands/results, skipped gates, risks, and verdict
+`ready for publication` or `changes required`. Final response at most ten lines.