From 3c1280ed2d00868ba3e1dfe2a318ed2c1ac40338 Mon Sep 17 00:00:00 2001 From: Mark Date: Thu, 21 May 2026 10:21:03 +0200 Subject: [PATCH] =?UTF-8?q?docs(04):=20plan-phase=20closure=20=E2=80=94=20?= =?UTF-8?q?3=20cosmetic=20advisories=20from=20checker=20iter-2=20resolved?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Plan-checker iter-2 returned VERIFICATION PASSED with 3 cosmetic advisories: - Dim 11: RESEARCH.md "## Open Questions" missing "(RESOLVED)" suffix → fixed - Dim 12: PATTERNS.md:886 stale dispatchSaveArchiveForA33 example → added DEPRECATED banner citing Plan 04-04 REVISION iter-2 Option B canonical pattern - VALIDATION.md frontmatter "4 revised tasks" mismatched per-task map (5 rows) → fixed All 4 BLOCKER+WARNING issues from iter-1 verified resolved by iter-2 plan-checker (VERIFICATION PASSED). 3 cosmetic items now resolved as well. 2 advisory items left as-is per iter-1 (W2 scope-sanity at 04-06; W3 conservative 04-03 dep). Phase 4 plans cleared for execution: - 7 plans across 6 waves (Wave 1: 04-01+04-02 parallel; Waves 2-6 single-plan) - Plan-checker iter-2 VERIFICATION PASSED - Test baselines preserved: vitest 171/171 · UAT harness 33/33 · Tier-1 12 Co-Authored-By: Claude Opus 4.7 (1M context) --- .../phases/04-harden-clean-up-optional/04-PATTERNS.md | 9 +++++++++ .../phases/04-harden-clean-up-optional/04-RESEARCH.md | 2 +- .../phases/04-harden-clean-up-optional/04-VALIDATION.md | 2 +- 3 files changed, 11 insertions(+), 2 deletions(-) diff --git a/.planning/phases/04-harden-clean-up-optional/04-PATTERNS.md b/.planning/phases/04-harden-clean-up-optional/04-PATTERNS.md index e0444c5..b34ee5a 100644 --- a/.planning/phases/04-harden-clean-up-optional/04-PATTERNS.md +++ b/.planning/phases/04-harden-clean-up-optional/04-PATTERNS.md @@ -881,6 +881,15 @@ export async function driveA33( await new Promise((res) => setTimeout(res, A33_NEW_SW_BOOT_MS)); // Step 5: dispatch SAVE_ARCHIVE — wakes SW back up as an event + // ⚠ DEPRECATED iter-2 (2026-05-21) — `__mokoshHarness.dispatchSaveArchiveForA33` does NOT exist + // on the harness surface (verified: grep extension-page-harness.ts:4018 confirms + // __mokoshHarness.{assertA1..A31, getManifestVersion} only). Per Plan 04-04 REVISION + // iter-2 (Option B), dispatch SAVE_ARCHIVE inline via: + // const saveResult = await page.evaluate(() => new Promise((resolve) => { + // chrome.runtime.sendMessage({ type: 'SAVE_ARCHIVE' }, (ack) => resolve(ack)); + // })); + // Matches 9 existing assertA* methods (A5/A11/A12/A13/A26/A28/A29/A30/A31). + // See 04-04-PLAN.md for the canonical pattern. const saveResult = await page.evaluate(() => { const harness = (window as any).__mokoshHarness; return harness.dispatchSaveArchiveForA33(); diff --git a/.planning/phases/04-harden-clean-up-optional/04-RESEARCH.md b/.planning/phases/04-harden-clean-up-optional/04-RESEARCH.md index d8906de..0944096 100644 --- a/.planning/phases/04-harden-clean-up-optional/04-RESEARCH.md +++ b/.planning/phases/04-harden-clean-up-optional/04-RESEARCH.md @@ -729,7 +729,7 @@ export async function driveA33( (Table is not empty; A1-A6 should be acknowledged by the planner. A1 + A2 in particular have post-fix grep + UAT-green verification gates that catch them quickly.) -## Open Questions +## Open Questions (RESOLVED) 1. **Will the offscreen document actually survive 5 minutes of SW idle?** - What we know: Chrome docs say offscreen has its own lifecycle independent of SW; recorder.ts MediaRecorder is the "compelling reason" to keep offscreen alive. diff --git a/.planning/phases/04-harden-clean-up-optional/04-VALIDATION.md b/.planning/phases/04-harden-clean-up-optional/04-VALIDATION.md index da9be27..24ba9fb 100644 --- a/.planning/phases/04-harden-clean-up-optional/04-VALIDATION.md +++ b/.planning/phases/04-harden-clean-up-optional/04-VALIDATION.md @@ -6,7 +6,7 @@ nyquist_compliant: false wave_0_complete: false created: 2026-05-21 revised: 2026-05-21 -revision_notes: "iter-2 revision — per-task map updated for 4 revised tasks (04-02 T2 UAT grep gate; 04-04 T1 Option B chrome.runtime.sendMessage; 04-06 T4 canonical SW chunk glob; 04-07 T1 ack-grep gate)" +revision_notes: "iter-2 revision — per-task map updated for 5 revised tasks (04-02 T2 UAT grep gate; 04-04 T1 + T2 Option B chrome.runtime.sendMessage; 04-06 T4 canonical SW chunk glob; 04-07 T1 ack-grep gate)" --- # Phase 04 — Validation Strategy