From 9a1a05544a694cf4127ca3c4e132f21390e56155 Mon Sep 17 00:00:00 2001 From: bymyself Date: Sat, 31 Jan 2026 15:17:47 -0800 Subject: [PATCH] docs: promote premade workflow assets guidance to golden rules Amp-Thread-ID: https://ampcode.com/threads/T-019c1640-f128-732b-b621-2621ce8b48d1 Co-authored-by: Amp --- .agents/skills/writing-playwright-tests/SKILL.md | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/.agents/skills/writing-playwright-tests/SKILL.md b/.agents/skills/writing-playwright-tests/SKILL.md index 65b6c972c4..ad080c605f 100644 --- a/.agents/skills/writing-playwright-tests/SKILL.md +++ b/.agents/skills/writing-playwright-tests/SKILL.md @@ -5,9 +5,15 @@ description: 'Writes Playwright e2e tests for ComfyUI_frontend. Use when creatin # Writing Playwright Tests for ComfyUI_frontend -## Golden Rule +## Golden Rules -**ALWAYS look at existing tests first.** Search `browser_tests/` for similar patterns before writing new tests. +1. **ALWAYS look at existing tests first.** Search `browser_tests/` for similar patterns before writing new tests. + +2. **Use premade JSON workflow assets** instead of building workflows programmatically at runtime. + - Assets live in `browser_tests/assets/` + - Load with `await comfyPage.loadWorkflow('feature/my_workflow')` + - More stable, faster, easier to debug + - See [testing/assets.md](testing/assets.md) for details ## Quick Reference