Skip to content

Sharpen pest-plugin-ai skill against observed failure modes#3

Merged
nunomaduro merged 2 commits into4.xfrom
skill/pest-plugin-ai-pitfalls
May 4, 2026
Merged

Sharpen pest-plugin-ai skill against observed failure modes#3
nunomaduro merged 2 commits into4.xfrom
skill/pest-plugin-ai-pitfalls

Conversation

@pushpak1300
Copy link
Copy Markdown
Collaborator

@pushpak1300 pushpak1300 commented May 4, 2026

Agents using the pest-plugin-ai skill kept hitting the same predictable pitfalls — passing natural-language prompts as PHP, inventing browser methods like onMobile(), treating screenshot()'s first arg as a filename, and rendering empty-state pages because the test DB starts empty. This tightens the skill so those failures stop being repeatable and adds the missing pieces around seeding, waits, and debugging.

Add critical rules covering natural-language snippets, invented browser
methods, and the screenshot() signature trap. Link the Pest browser-testing
docs so agents can fetch the full API when something isn't covered here.
Document the empty-DB seeding pattern for visual review of feed/list
pages, the HasWaitCapabilities helpers for SPA/Inertia transitions, the
click() timeout splitting recipe, the screenshot signature with
fullPage caveat, the resize() vs on()->mobile() precedence, and
script() return-value debugging. Correct the screenshot output path to
tests/Browser/Screenshots/ and soften the blanket delete-screenshots
rule for design-review workflows.
@pushpak1300 pushpak1300 force-pushed the skill/pest-plugin-ai-pitfalls branch from ab2711b to ffc30d9 Compare May 4, 2026 18:21
@nunomaduro nunomaduro marked this pull request as ready for review May 4, 2026 20:33
@nunomaduro nunomaduro merged commit 55e495e into 4.x May 4, 2026
7 of 20 checks passed
@nunomaduro nunomaduro deleted the skill/pest-plugin-ai-pitfalls branch May 4, 2026 20:34
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.

2 participants