diff --git a/.planning/ROADMAP.md b/.planning/ROADMAP.md index 796520e..acd9f26 100644 --- a/.planning/ROADMAP.md +++ b/.planning/ROADMAP.md @@ -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) diff --git a/.planning/phases/08-gitea-ci/08-CONTEXT.md b/.planning/phases/08-gitea-ci/08-CONTEXT.md index a6b351b..a17c034 100644 --- a/.planning/phases/08-gitea-ci/08-CONTEXT.md +++ b/.planning/phases/08-gitea-ci/08-CONTEXT.md @@ -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. diff --git a/.planning/phases/999.15-desktop-e2e-coverage-for-the-pwa-harness/.gitkeep b/.planning/phases/999.15-desktop-e2e-coverage-for-the-pwa-harness/.gitkeep new file mode 100644 index 0000000..e69de29