add playwright-triage skill [ci skip]#3333
Merged
Dave Shoup (shouples) merged 3 commits intomainfrom Mar 25, 2026
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new Claude skill to help triage failing Playwright E2E tests in this VS Code extension repo, with guidance for inspecting typical CI artifacts and a helper script to extract an ordered action list from Playwright trace.zip files.
Changes:
- Added a
playwright-triageskill (SKILL.md) documenting a step-by-step workflow for inspecting screenshots, accessibility snapshots, traces, and logs. - Added a
parse-trace.cjsNode script to extract and correlatebefore/aftertrace actions into a readable sequence with errors highlighted.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| .claude/skills/playwright-triage/parse-trace.cjs | CLI utility to unzip and parse trace.trace NDJSON, correlating actions and surfacing failures. |
| .claude/skills/playwright-triage/SKILL.md | Playbook for locating and interpreting Playwright failure artifacts, including use of parse-trace.cjs. |
airlock-confluentinc
Bot
force-pushed
the
djs/playwright-triage-skill
branch
from
62f5081 to
2698140
Compare
|
Dave Shoup (shouples)
changed the title
playwright-triage skillplaywright-triage skill [ci skip]
Lucia Cerchie (Cerchie)
approved these changes
Mar 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds some basic instructions for how to handle
trace.zipfiles and the .gzip CI reports to navigate the test information as it relates to the Electron DOM.Some past experiences showed a lot of attempts trying to unpack the .zip files and navigate them, so this was built after a couple of particularly bumpy sessions to aggregate lessons learned.
gulp e2e) path with failing tests, it was used in order to diagnose the new breaking behavior and suggest the fix that ended up being fix(E2E): save default user settings in test-specific tempdir #3334.getItemByLabel()#3339.