repremium/mokosh
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

wip: phase-04 paused — 7 plans validated iter-2 PASSED, ready for execution .planning/phases/04-harden-clean-up-optional/.continue-here.md .planning/HANDOFF.json

Browse Source
This commit is contained in:
Mark
2026-05-21 10:25:29 +02:00
parent 55369b80ad
commit dbcf4827f6
2 changed files with 236 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

124
.planning/phases/04-harden-clean-up-optional/.continue-here.md Normal file
View File

@@ -0,0 +1,124 @@
---
context: phase
phase: 04-harden-clean-up-optional
plan_count: 7
status: paused
last_updated: 2026-05-21T08:22:59.958Z
---
# BLOCKING CONSTRAINTS — None
> No blocking constraints introduced this session. Three **advisory** anti-patterns logged below (do not gate resumption).
## Critical Anti-Patterns
| Pattern | Description | Severity | Prevention Mechanism |
|---------|-------------|----------|---------------------|
| state.record-session CLI bug — recurring | gsd-sdk query state.record-session flips STATE.md `status` to `completed` when total_plans/completed_plans ratio triggers it. Recurred at 8+ state.record-session calls this session. | advisory | NEXT-SESSION: manually fix STATE.md `status` field after every state.record-session call. Upstream GSD framework concern; not Mokosh-side. |
| /gsd-plan-phase UI gate auto-exits on manual invocation | Manual /gsd-plan-phase cannot nested-Skill-spawn /gsd-ui-phase due to AskUserQuestion-in-subcontext issue #1009. Workflow correctly exits + tells user to run /gsd-ui-phase separately. | advisory | Pattern used twice this session (Phase 3 + Phase 4); preserve gate answers in .plan-phase-preferences.md (auto-deleted on next plan-phase consumption). Standard pattern; not a defect. |
| Plan-checker line-range disjointness check missing | Plan-checker only validates file-level overlap; doesn't check line-range disjointness for potential parallelization wins. | advisory | NEXT-SESSION: when reviewing wave structure, manually check line-range disjointness if files_modified overlap is present. |
---
## Current State
**Phase 4 (Harden + clean up — optional) PLANNING COMPLETE.** Ready for execution. This is the final v1 milestone phase.
- Branch: `gsd/phase-03-spec-10-smoke-verification-dom-event-log-verification` at HEAD `55369b8` (Phase 4 branch NOT yet created — happens in `/gsd-execute-phase 4` per branching_strategy=phase)
- 7 plans validated + plan-checker iter-2 PASSED + 3 cosmetic advisories fixed
- Test baselines: vitest 171/171 GREEN · UAT 33/33 GREEN · Tier-1 FORBIDDEN_HOOK_STRINGS 12 · pre-checkpoint bundle gates 6/6 PASS
- Working tree: clean
## Plans Ready for Execution
| Plan | Wave | Type | Scope | Closes |
|---|---|---|---|---|
| **04-01** | 1 | tdd | Audit P1 polish (RED→GREEN 3 unit tests in src/content/index.ts:147 + 99-109 + 36-42) | Audit P1 #11/#14/#15 |
| **04-02** | 1 (parallel) | tdd | Build/CSP hygiene — setimmediate polyfill Option a + dead-code grep + generate-icons.cjs | ROADMAP SC #3 + #4 |
| **04-03** | 2 | auto | A29 cs-injection-world rewrite (strict-sentinel filter for IncrementalSource.Mutation) | ~1/3 flake from Phase 3 |
| **04-04** | 3 | spike→auto | A33 SW state persistence — **Wave 0 SPIKE FIRST**; chrome.runtime.sendMessage Option B | ROADMAP SC #1 |
| **04-05** | 4 | auto | A34 fetch+XHR network_error empirical (validates 04-01 P1#11 end-to-end) | ROADMAP SC #2 |
| **04-06** | 5 | auto + human-verify | Visual polish — dark-logo currentColor (5 UI-SPEC edits) + cursor verification + operator empirical | D-P4-03 |
| **04-07** | 6 | auto | VERIFICATION.md aggregator + ROADMAP backfill + REQ marker flips + milestone v1 close prep | D-P4-05 + Phase 4 closure |
**Wave 1 parallelization:** 04-01 + 04-02 are file-disjoint (04-01 src/content/* + 04-02 vite/SW/build-tests) — safe parallel.
**Waves 2-6 sequential:** file-overlap on tests/uat/* harness files.
## Recent commit chain (most recent first)
- `55369b8` docs(state): record phase 4 plan-phase closure — iter-2 PASSED + 3 cosmetic advisories fixed
- `3c1280e` docs(04): plan-phase closure — 3 cosmetic advisories from checker iter-2 resolved
- `76fffb3` fix(04): revise plans per checker iter-1 — 2 BLOCKERS + 2 WARNINGS fixed
- `526ac78` docs(04): create phase plan — 7 plans for Phase 4 hardening
- `f012c8c` docs(04): pattern map — ~30 anticipated files mapped
- `7178d14` docs(phase-04): add validation strategy
- `d1f6767` docs(04): research phase domain — 5 findings incl. cursor:'always' ALREADY shipped
- `61caf04` docs(state): record phase 4 UI-SPEC session — APPROVED 5/6 + 1 FLAG
- `266aa95` docs(04): UI-SPEC.md status approved (5/6 PASS + 1 FLAG non-blocking)
- `55cefba` docs(04): UI design contract — thin scope (dark-logo currentColor strategy)
- (… Phase 3 closure + Phase 2 closure earlier)
## Required Reading (in order)
1. `.planning/HANDOFF.json` — machine-readable handoff (sibling)
2. `.planning/STATE.md` — current progress (Phase 3 CLOSED; Phase 4 ready)
3. `.planning/ROADMAP.md` — 4-phase milestone structure; Phase 4 optional
4. `.planning/REQUIREMENTS.md` — all v1 REQs verified (Phase 4 has no new REQs)
5. `.planning/PROJECT.md` — Validated section split by phase
6. `.planning/phases/04-harden-clean-up-optional/04-CONTEXT.md` — 5 D-P4-* locked decisions
7. `.planning/phases/04-harden-clean-up-optional/04-UI-SPEC.md` — thin-design-surface dark-logo (Option A currentColor; 5 prescribed file edits)
8. `.planning/phases/04-harden-clean-up-optional/04-RESEARCH.md` — 5 findings; **CRITICAL finding 4: cursor:'always' ALREADY SHIPPED at src/offscreen/recorder.ts:285** (Plan 04-06 cursor task = verification only)
9. `.planning/phases/04-harden-clean-up-optional/04-PATTERNS.md` — ~30 files mapped + DEPRECATED banner at line 886 (citing Plan 04-04 Option B)
10. `.planning/phases/04-harden-clean-up-optional/04-VALIDATION.md` — 16-row per-task verification map; iter-2 revised
11. `.planning/phases/04-harden-clean-up-optional/04-0{1..7}-PLAN.md` — 7 plans ready
12. Auto-memory at `/home/parf/.claude/projects/-home-parf-projects-work-repremium/memory/MEMORY.md` — 4 feedback constraints from prior sessions
## Decisions Locked This Session (Phase 4)
- **D-P4-01:** Full Phase 4 scope (all 4 ROADMAP SC + ~10 of 12 deferred items)
- **D-P4-02:** Audit P1 all three (#11 fetch + #14 nav URL + #15 rrweb timestamps)
- **D-P4-03:** Both visual polish items (cursor + dark-logo)
- **D-P4-04:** Alpha tester integration user-handled out-of-band
- **D-P4-05:** ROADMAP backfill in scope (docs hygiene)
- **UI-SPEC Option A:** currentColor SVG + ?raw import + DOMParser inline (5 file edits)
- **Plan 04-04 Option B:** chrome.runtime.sendMessage inline for SAVE_ARCHIVE dispatch (BLOCKER 2 fix — no dispatchSaveArchive helper)
- **Plan 04-06 Task 4 glob:** `dist/assets/index.ts-*.js` (BLOCKER 1 fix + glob-existence pre-gate)
- **§10 #9 RAM closure:** user explicit ack 2026-05-20 on harness scaffolding + alpha distribution coverage path
## Next-Session Order of Operations
1. **`/gsd-resume-work`** to load this `.continue-here.md` + `.planning/HANDOFF.json`
2. **Acknowledge any blocking constraints** (none this session; 3 advisory anti-patterns above)
3. **`/gsd-execute-phase 4`** to start Phase 4 execution (~6-8h foreground; canonical Skill)
- Alternative: `/gsd-execute-phase 4 --wave 1` lands Wave 1 (04-01 + 04-02 parallel) only (~30-60 min)
4. After Phase 4 closes: milestone v1 close prep (fresh alpha re-distribution + v1.0 tag + release notes per D-P4-04)
## Infrastructure State
- Branch: `gsd/phase-03-spec-10-smoke-verification-dom-event-log-verification` at HEAD `55369b8` — Phase 4 branch NOT yet created (happens in /gsd-execute-phase 4)
- Working tree clean
- Test baselines preserved: vitest 171/171 GREEN · UAT 33/33 GREEN · Tier-1 12 · bundle gates 6/6 PASS
- Saved memories applied throughout: feedback-no-unilateral-scope-reduction + feedback-gsd-ceremony-for-fixes + feedback-pre-checkpoint-bundle-gates + feedback-trust-harness-over-manual-uat
## Phase 4 Expected Outcomes
- vitest: 171 → ~180 GREEN (+~10 Wave 0 unit tests: P1 #11/#14/#15 + 2 build gates + welcome inline-svg)
- UAT harness: 33 → 35 GREEN (+A33 SW persistence + A34 fetch/XHR network_error)
- Tier-1 FORBIDDEN_HOOK_STRINGS: 12 unchanged (A33+A34 ride production surfaces)
- Bundle gates 6/6 PASS — Gate 2 (SW CSP-safety `new Function`) flips from 1 documented exception to **0 hits** after Plan 04-02 setimmediate polyfill replacement
- ROADMAP success criteria #1-#4 all verified
- Plan 04-06 Task 4 operator empirical UAT (dark-logo aesthetic judgment in OS dark mode)
- Milestone v1 ready for tag + release notes after Phase 4 closure
## Pending Human Action (non-blocking, deferred to Phase 4 execution)
- **Plan 04-06 Task 4 operator empirical UAT** — dark-logo aesthetic judgment in OS dark mode (UI-SPEC AC #6; canonical Phase 4 human-verify case)
- All other Phase 4 surfaces are harness-covered or grep-verifiable
## Context
Massive session arc: 2026-05-20 → 2026-05-21 spanning Phase 2 execution + closure, Phase 3 full lifecycle (discuss + UI + plan + execute + closure), Phase 4 full pre-execution lifecycle (discuss + UI + research + pattern + plan + 2-iteration plan-checker revision). Three phases closed end-to-end; Phase 4 fully planned + validated; ready for execution. User explicitly chose canonical GSD ceremony throughout (consistent pattern across all phases). Test baseline growth this session: vitest 153→171 GREEN (+18); UAT 24→33 GREEN (+9 incl. A24-A32). 4 saved memories cited throughout. Phase 4 is the final v1 milestone phase; after closure, milestone v1 is ready for release.
## Next action
Start with: `/gsd-resume-work` to load this handoff, then `/gsd-execute-phase 4` to start Phase 4 execution.