docs: add backlog item 999.15 — desktop e2e coverage; drop from phase 8 context

This commit is contained in:
Lucas Berger
2026-06-11 09:36:53 -04:00
parent 41b0b60291
commit 4a510f1d79
3 changed files with 22 additions and 1 deletions
+21
View File
@@ -389,3 +389,24 @@ Plans:
Plans:
- [ ] TBD (promote with /gsd-review-backlog when ready)
### Phase 999.15: Desktop e2e coverage — add a Desktop Playwright profile + desktop-safe specs (BACKLOG)
**Goal:** [Captured for future planning] The Playwright harness (`apps/pwa/playwright.config.ts`) defines only **mobile** device profiles — `iphone` (iPhone 14 / WebKit) and `pixel` (Pixel 7 / Chromium), both with touch and a mobile viewport. The Phase 8 CI regression gate runs `pnpm test:e2e`, so it currently validates the **mobile experience only**. Add desktop coverage so the regression gate exercises the desktop layout/flows as well.
**Options / decisions to make when picking this up:**
- **Add a Desktop profile:** a new `desktop` project in `playwright.config.ts` (e.g. `devices['Desktop Chrome']`, no `hasTouch`, wide viewport). Optionally a Desktop WebKit/Safari profile too — but the family's Apple member is already covered on mobile Safari via `iphone`; Desktop Chrome is likely sufficient for a shared/wall browser.
- **Spec-compat pass (the real work):** the existing e2e specs were authored for mobile — they may assume touch gestures, a mobile nav/drawer, or mobile-only layout. Each spec needs review/adjustment so it passes (or is appropriately skipped) on a no-touch, wide-viewport desktop. This is harness/spec work, not CI plumbing.
- **Gating choice:** decide whether desktop runs block the merge immediately, or run advisory (non-blocking) until the specs are confirmed desktop-safe.
**Boundary:** Phase 8 deliberately reused the Phase 7 harness **unchanged** (CI owns only stack bring-up + readiness waits, not spec content), which is why this was deferred. Once a Desktop project is added to the config, Phase 8 CI picks it up automatically via `pnpm test:e2e` — no CI changes needed beyond whatever runtime/wait the desktop profile requires.
**Context:** Deferred from Phase 8 (Gitea CI) planning, 2026-06-11 — user wants both mobile and desktop validated, but desktop needs a config addition + spec review that is out of Phase 8's CI-plumbing scope. Tags: testing, playwright, e2e, desktop, harness, ci.
**Requirements:** TBD
**Plans:** 0 plans
Plans:
- [ ] TBD (promote with /gsd-review-backlog when ready)
+1 -1
View File
@@ -102,7 +102,7 @@ This phase owns only the CI plumbing: workflow files, dev-stack bring-up + readi
## Deferred Ideas
- **ROADMAP status fix:** ROADMAP.md line 29 marks Phase 8 "completed 2026-06-11" while line 204 says "Not started" and no Phase 8 artifacts exist. This is a bookkeeping error to correct (Phase 8 is being started now) — a docs/roadmap cleanup, not Phase 8 scope.
- **Desktop e2e coverage (decided 2026-06-11):** The Phase 7 harness defines only mobile profiles (`iphone`/WebKit + `pixel`/Chromium), so Phase 8 CI gates **mobile only**. Desktop validation is wanted but deferred to a **dedicated harness follow-up phase** — it requires editing `apps/pwa/playwright.config.ts` to add a Desktop profile **and** a spec-compat pass (the existing specs assume touch + mobile viewport/layout). Phase 8 deliberately reuses the Phase 7 harness unchanged; CI will pick up the desktop profile automatically via `pnpm test:e2e` once the follow-up adds it. Track as a backlog phase.
- **Desktop e2e coverage:** moved to backlog **Phase 999.15** Phase 8 CI gates mobile only; adding a Desktop Playwright profile + desktop-safe specs is out of CI-plumbing scope.
</deferred>