review-test-code

Review Playwright test code for stability, correctness, and convention adherence.

4|1|Updated Feb 3, 2026
One-click install
npx skills add https://github.com/lespaceman/athena-workflow-marketplace --skill review-test-code
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review-test-code
Source: https://github.com/lespaceman/athena-workflow-marketplace/tree/main/plugins/playwright-automation/skills/review-test-code
Command: npx skills add https://github.com/lespaceman/athena-workflow-marketplace --skill review-test-code

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviews Playwright test code for stability, correctness, and adherence to project conventions before final execution signoff.

Core Features & Use Cases

  • Provides an independent, fresh-subagent review by checking test files and project conventions (e.g., playwright.config.*) to establish patterns and expectations.
  • Detects locator quality, timing strategies, assertion coverage, test isolation, and TC-ID traceability to prevent flaky or non-reproducible tests.
  • Supports guidance on anti-patterns and ensures tests align with project conventions, data handling, and setup/teardown practices.

Quick Start

Provide the test file path or directory to review, for example: /review-test-code tests/e2e/login.spec.ts.

Frequently Asked Questions about review-test-code

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review Playwright tests for flakiness before running them?

Review Playwright tests for flakiness by checking locator quality, waiting strategies, and assertion coverage before execution. This process detects timing issues and anti-patterns in e2e suites, outputting blockers and warnings to prevent non-reproducible test runs.

What is test traceability and how does it apply to Playwright e2e suites?

Test traceability in Playwright suites involves mapping test cases to unique TC-IDs for verification. It ensures every test file aligns with reference cases and project conventions, providing clear tracking of what each e2e test validates across the suite.

How do I check if my Playwright test code follows project conventions?

Check Playwright test code conventions by extracting patterns from the playwright.config file and reference test cases. Compare your test files against these extracted expectations for locator strategies, setup, teardown, and data handling to ensure adherence.

Can I review an entire Playwright e2e test directory at once?

Yes, you can review an entire Playwright e2e test directory by providing the folder path instead of a single test file. The review checks test isolation, locator quality, and traceability across all files, outputting aggregated blockers and recommendations.

What is the best way to prevent flaky Playwright locators and timing issues?

The best way to prevent flaky Playwright locators and timing issues is enforcing a rigorous checklist for locator quality and waiting strategies before test execution. This catches unstable selectors and improper waits, providing warnings and recommendations for correction.

Why does my Playwright test review report blockers for test isolation?

Playwright test review reports blockers for test isolation when tests improperly share state or lack proper setup and teardown. The review verifies isolation against project conventions, flagging tests that may cause non-reproducible execution or data contamination.