| 1 | | Earlier finding | Disposition | |
| 2 | |---|---| |
| 3 | | R1.1 — Manual purge omits settlement | Fixed: settlement precedes purge prerequisites and teardown. | |
| 4 | | R1.2 — Destructive handlers retain pre-lock decisions | Fixed: existence and protection are rechecked under the lock. | |
| 5 | | R1.3 — `ErrNotTrashed` tests bypass recovery | Fixed: marker removal occurs between reads; recovery execution is asserted. | |
| 6 | | R1.4 — Static-instance test accepts unrelated failures | Fixed: requires 201 and verifies the stored record. | |
| 7 | | R1.5 — Empty `--ttl` selects ordinary deployment | Fixed: explicit flag presence is checked and tested. | |
| 8 | | R2.1 — Required implementation file omitted from commit | Fixed: Task 5 stages `svcprincipal.go`. | |
| 9 | | R2.2 — Settlement breaks trash retries | Fixed: an existing marker permits idempotent success. | |
| 10 | | R2.3 — Refusal wording breaks existing assertions | Fixed: existing refusal wording is retained. | |
| 11 | | R3.1 — Delete confirmation promises 30 days | Fixed: confirmation selects three or 30 days, with tests. | |
| 12 | | R3.2 — Refusal test decodes plain-text 404 as JSON | Fixed: uses `rawReq`, checks the canonical 404 and unchanged state. | |
| 13 | | R4.1 — Restore guidance names a forbidden command | Fixed in the cited CLI/API/browser responses, with a provisioned-site regression. A separate documentation inconsistency remains below. | |
| 14 | |
| 15 | Reviewed `8d850c17`. Static review only; no tests run or files changed. |
| 16 | |
| 17 | 1. **P2 — Task 11, Step 3: CLI documentation promises that restored temporary sites remain unrouted.** |
| 18 | The replacement documentation at [plan:6079](/home/paulca/amadan.net/carlos/platform/.claude/worktrees/temporary-sites/docs/superpowers/plans/2026-09-26-temporary-static-sites.md:6079) explicitly includes temporary apps, then says restore returns “no routes at all, not even the platform apex” and instructs users to add them again. This contradicts Task 7’s revised guidance at [plan:3526](/home/paulca/amadan.net/carlos/platform/.claude/worktrees/temporary-sites/docs/superpowers/plans/2026-09-26-temporary-static-sites.md:3526): retained instance records cause routes to return automatically. That behavior is exercised by [reconcile_test.go:1382](/home/paulca/amadan.net/carlos/platform/.claude/worktrees/temporary-sites/internal/provision/reconcile_test.go:1382). Someone following the documentation could restore a site expecting it to remain offline. |
| 19 | |
| 20 | **Fix:** update Task 11’s restore paragraph to explain automatic route recreation from retained instance records for temporary sites. Include the correction in Task 0’s `C11.26` copy review. |
| 21 | |
| 22 | **Verdict: revise.** |
| 23 | |
| 24 | --- |
| 25 | |
| 26 | **FIRST PROMPT:** |
| 27 | Review the temporary-sites plan at `8d850c17`, round 5, dispositioning earlier findings first. |
| 28 | |
| 29 | **WHAT:** |
| 30 | Check plan fidelity, correctness, integration and tests. |
| 31 | |
| 32 | **STATUS:** |
| 33 | Review complete: one new P2 documentation finding. |
| 34 | |
| 35 | **NEEDED:** |
| 36 | Nothing. |
| 37 | |
| 38 | **RECOMMENDED:** |
| 39 | Correct Task 11’s restore documentation. |
| 40 | |
| 41 | **SESSION:** |
| 42 | Safe to close. No processes started or files changed; existing untracked files remain. Handoff file not written because the workspace is read-only. |
amadan