From a1227aa0f97f64b92ded28f59be5a324ef274b9d Mon Sep 17 00:00:00 2001 From: bymyself Date: Sat, 31 Jan 2026 15:54:21 -0800 Subject: [PATCH] docs: add skill maintenance reminder to browser_tests/AGENTS.md - Remind to update skill docs when modifying test infrastructure - Add mapping table from fixture files to skill doc files - Expand directory structure section Amp-Thread-ID: https://ampcode.com/threads/T-019c165d-cebf-7393-847a-914e5858bd9a Co-authored-by: Amp --- browser_tests/AGENTS.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/browser_tests/AGENTS.md b/browser_tests/AGENTS.md index b5cbe18c4..839471f35 100644 --- a/browser_tests/AGENTS.md +++ b/browser_tests/AGENTS.md @@ -5,6 +5,8 @@ See `@docs/guidance/playwright.md` for Playwright best practices (auto-loaded fo ## Directory Structure - `assets/` - Test data (JSON workflows, fixtures) +- `fixtures/` - ComfyPage, ComfyMouse, and component fixtures +- `helpers/` - Shared test utilities - Tests use premade JSON workflows to load desired graph state ## After Making Changes @@ -12,3 +14,9 @@ See `@docs/guidance/playwright.md` for Playwright best practices (auto-loaded fo - Run `pnpm typecheck:browser` after modifying TypeScript files in this directory - Run `pnpm exec eslint browser_tests/path/to/file.ts` to lint specific files - Run `pnpm exec oxlint browser_tests/path/to/file.ts` to check with oxlint + +## Skill Documentation + +A Playwright test-writing skill exists at `.claude/skills/writing-playwright-tests/SKILL.md`. + +The skill documents **meta-level guidance only** (gotchas, anti-patterns, decision guides). It does **not** duplicate fixture APIs - agents should read the fixture code directly in `browser_tests/fixtures/`.