docs(phase-27): add validation strategy

This commit is contained in:
2026-04-10 23:27:04 +02:00
parent 9bb8f8faa2
commit 078694c124

View File

@@ -0,0 +1,80 @@
---
phase: 27
slug: top-nav-restructure-and-search-bar-rethink
status: draft
nyquist_compliant: false
wave_0_complete: false
created: 2026-04-10
---
# Phase 27 — Validation Strategy
> Per-phase validation contract for feedback sampling during execution.
---
## Test Infrastructure
| Property | Value |
|----------|-------|
| **Framework** | Playwright (E2E) + Bun test runner (unit) |
| **Config file** | `playwright.config.ts` (E2E), built-in Bun test runner |
| **Quick run command** | `bun test tests/` |
| **Full suite command** | `bun run test:e2e` |
| **Estimated runtime** | ~30 seconds (E2E), ~5 seconds (unit) |
---
## Sampling Rate
- **After every task commit:** Run `bun test tests/`
- **After every plan wave:** Run `bun run test:e2e`
- **Before `/gsd:verify-work`:** Full suite must be green
- **Max feedback latency:** 30 seconds
---
## Per-Task Verification Map
| Task ID | Plan | Wave | Requirement | Test Type | Automated Command | File Exists | Status |
|---------|------|------|-------------|-----------|-------------------|-------------|--------|
| 27-01-01 | 01 | 1 | SC-1 | E2E | `bunx playwright test e2e/dashboard.spec.ts` | Partial — needs update | ⬜ pending |
| 27-01-02 | 01 | 1 | SC-2 | E2E | `bunx playwright test e2e/dashboard.spec.ts` | ❌ W0 | ⬜ pending |
| 27-01-03 | 01 | 1 | SC-3 | E2E (mobile) | `bunx playwright test e2e/dashboard.spec.ts` | ❌ W0 | ⬜ pending |
| 27-02-01 | 02 | 1 | SC-4 | E2E | `bunx playwright test e2e/dashboard.spec.ts` | Partial — needs update | ⬜ pending |
| 27-02-02 | 02 | 1 | SC-5 | E2E | `bunx playwright test e2e/collection.spec.ts` | ❌ W0 | ⬜ pending |
| 27-02-03 | 02 | 1 | SC-5 | E2E | `bunx playwright test e2e/collection.spec.ts` | ❌ needs update | ⬜ pending |
*Status: ⬜ pending · ✅ green · ❌ red · ⚠️ flaky*
---
## Wave 0 Requirements
- [ ] `e2e/dashboard.spec.ts` — update existing tests: remove assertions about hero heading "Discover Gear", add assertion for top nav presence, update "GearBox heading" to check nav bar (not h1)
- [ ] `e2e/dashboard.spec.ts` — add: anon user clicking Collection nav triggers auth modal
- [ ] `e2e/dashboard.spec.ts` — add: mobile viewport bottom tab bar test (use Playwright `page.setViewportSize`)
- [ ] `e2e/collection.spec.ts` — update: remove Setups tab assertions, add /setups route navigation test
---
## Manual-Only Verifications
| Behavior | Requirement | Why Manual | Test Instructions |
|----------|-------------|------------|-------------------|
| Nav bar visual alignment and spacing | SC-1 | Visual pixel-level layout | Inspect desktop nav at 1280px: logo left, links center, search+avatar right |
| Bottom tab bar touch targets | SC-3 | Touch interaction on real device | Tap each tab bar icon on mobile viewport, verify navigation and overlay trigger |
| Search bar expand/collapse animation | Claude discretion | Animation smoothness is subjective | Click search icon on desktop, verify expand animation is smooth |
---
## Validation Sign-Off
- [ ] All tasks have `<automated>` verify or Wave 0 dependencies
- [ ] Sampling continuity: no 3 consecutive tasks without automated verify
- [ ] Wave 0 covers all MISSING references
- [ ] No watch-mode flags
- [ ] Feedback latency < 30s
- [ ] `nyquist_compliant: true` set in frontmatter
**Approval:** pending