Skip to content

(CODUX-131) Add docs page E2E smoke - #2784

Merged
numnx merged 1 commit into
feature/sprint-COWIN-1-fix-electron-docs-page-implementationfrom
task/feature-sprint-c-t03-codex-6bfad8b3-mrdqg0a7
Jul 9, 2026
Merged

(CODUX-131) Add docs page E2E smoke#2784
numnx merged 1 commit into
feature/sprint-COWIN-1-fix-electron-docs-page-implementationfrom
task/feature-sprint-c-t03-codex-6bfad8b3-mrdqg0a7

Conversation

@numnx

@numnx numnx commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

🧩 Task: Add docs page E2E smoke

Automated task PR opened by Code UX for T03.

📋 Summary

Sprint: Sprint 1 · Fix Electron docs page · Task: T03

Fix the Electron Docs page 404 by ensuring packaged runtime docs assets are available, and add CI-covered smoke checks for the main Docs route plus representative documentation pages.

🤖 Model & Provider

Provider Codex CLI
Model gpt-5.5

⏱️ Timing

Started 2026-07-09 16:40:08 UTC
Finished 2026-07-09 16:45:41 UTC
Duration 5m 33s

📊 Token Usage

Metric Value
Input tokens 65,790
Cached input tokens 791,808
Output tokens 6,954
Total tokens 864,552
Invocations 1
Tool calls 41
Active time 3m 45s

✅ QA Review

⏳ QA review runs after this PR is opened. Check the task card in the dashboard for the outcome once it lands.

📝 Full Task Prompt
## Objective
Add CI-covered browser smoke coverage for the dashboard Docs page so the main docs index and representative docs pages cannot silently regress to 404.

## Scope
- `tests/e2e/navigation/docs-page.spec.ts` to create
- Existing helpers from `tests/e2e/helpers/prepare-app.ts` or `tests/e2e/helpers/e2e-fixtures.ts`
- Existing Playwright `navigation` project in `playwright.config.ts`
- Routes to cover: `/docs`, `/docs/docs-overview`, `/docs/user-dashboard-overview`, `/docs/user-quickstart`, `/docs/developer-mcp-tools`, `/docs/architecture-system-overview`

## Implementation Requirements
1. Create a Playwright spec under `tests/e2e/navigation/` so it is automatically included in the existing `navigation` CI project.
2. Complete onboarding and suppress the dashboard tour using existing helpers before loading docs routes.
3. Capture page errors, console errors, and HTTP responses with status `>= 400`, then fail the test with a useful message for any unexpected error.
4. For each route, assert the main content landmark is visible, `data-testid="docs-web-page-root"` is visible, and the rendered article shows an expected heading or source-path marker for that document.
5. Keep the spec focused on the main docs page plus the requested representative samples; do not add exhaustive subpage coverage.

## Constraints
- Do not modify `.github/workflows/playwright.yml` unless the new spec is not picked up by the existing `navigation` project.
- Do not rely on visual styling selectors except stable `data-testid` roots and accessible landmarks/headings.
- Do not require provider credentials, Docker provider startup, or a real external project.
- Keep the test deterministic against the compiled local server used by Playwright.

## Verification
- Run `pnpm run build` before the E2E check if `dist/` may be stale.
- Run `pnpm exec playwright test --project=navigation tests/e2e/navigation/docs-page.spec.ts`.
- Success means all listed docs routes render content and no captured HTTP, console, or page errors occur.
🌿 Branch Info

Base: feature/sprint-COWIN-1-fix-electron-docs-page-implementation
Head: task/feature-sprint-c-t03-codex-6bfad8b3-mrdqg0a7


🤖 Generated by Code UX

@numnx
numnx merged commit 8531478 into feature/sprint-COWIN-1-fix-electron-docs-page-implementation Jul 9, 2026
6 checks passed
@numnx
numnx deleted the task/feature-sprint-c-t03-codex-6bfad8b3-mrdqg0a7 branch July 9, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant