CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

pw-review

Review Playwright tests for quality. Use when user says "review tests", "check test quality", "audit tests", "improve tests", "test code review", or "playwright best practices check".

60

Quality

76%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.gemini/skills/pw-review/SKILL.md

The canonical home for this skill is pw-review in alirezarezvani/claude-skills

SKILL.md
Quality
Evals
Security

Quality

Content

71%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A well-structured, highly actionable review procedure with concrete anti-pattern checks, scoring criteria, and a report template. Its two real defects are the dangling anti-patterns.md reference (the file is absent and its content is duplicated inline) and missing post-fix verification in the workflow.

Suggestions

Resolve the anti-patterns.md discrepancy: either ship the file (with details/fixes per anti-pattern) and trim the inline list to the names, or delete the "Load anti-patterns.md" instruction and keep the inline list as the source of truth.

Add a verification step after applying fixes, e.g., re-run the affected test file with `npx playwright test <file>` and confirm the critical issues are resolved before reporting completion.

Qualify or remove the "/batch" recommendation in step 5 (it may not exist in the user's environment) and add a checkpoint to validate/normalize sub-agent results before aggregating the summary table.

DimensionReasoningScore

Conciseness

The body is lean with no concept over-explanation — every section (anti-pattern list, scoring rubric, report template) is actionable. Not anchor 5: step 2 says "Load anti-patterns.md ... Check for all 20 anti-patterns" and then inlines all 20 anyway, duplicating content that could be trimmed to one representation.

4 / 5

Actionability

Concrete per-line fix examples ("Line 15: waitForTimeout(2000) → use expect(locator).toBeVisible()"), a 1-10 scoring rubric with criteria, and a copy-ready report template make the guidance mostly executable. Not anchor 5: the instruction "Load anti-patterns.md from this skill directory" targets a file that does not exist in the skill bundle, and "use /batch" is asserted without verifying that command exists.

4 / 5

Workflow Clarity

Six clearly sequenced steps with a user confirmation gate ("Apply these fixes? [Yes/No]") before destructive edits. Not anchor 5: no verification after fixes are applied (e.g., re-run the affected tests) and no validation checkpoint for sub-agent results in the project-wide batch path — anchor 4's 'most checkpoints present, minor validation gaps'.

4 / 5

Progressive Disclosure

The body is well-sectioned and the core reference content (the 20 anti-patterns) is inlined, but the single external reference — "Load anti-patterns.md from this skill directory" — is a dangling pointer to a nonexistent file whose content duplicates the inline list. Reference signaling is broken rather than merely having minor gaps, fitting anchor 3 better than anchor 4.

3 / 5

Total

15

/

20

Passed

Description

70%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description has excellent, explicit trigger guidance and a clear niche, but the capability statement is underdeveloped — it says only "review for quality" without naming the concrete outputs (anti-pattern checks, scores, fix list) the skill delivers. The main improvement is enriching the "what" with 2-3 concrete actions.

Suggestions

Expand the capability statement with concrete actions, e.g., "Checks Playwright test files against 20 anti-patterns, scores each file 1-10, and generates an actionable fix list."

Add file-extension trigger cues such as ".spec.ts" or a standalone "playwright" trigger so users who just name the tool or the file type still match.

Qualify generic triggers like "review tests" with "review Playwright tests" to reduce overlap with general test-review or code-review skills.

DimensionReasoningScore

Specificity

"Review Playwright tests for quality" names the domain and a single action; the description never mentions anti-pattern detection, per-file scoring, or report generation that the skill actually performs. This matches anchor 3 (1-2 concrete actions, not comprehensive) and falls short of anchor 4, which expects several specific actions listed.

3 / 5

Completeness

Both "what" ("Review Playwright tests for quality") and an explicit "when" with concrete trigger phrases are present. Not anchor 5 because the "what" is a single thin action, unlike the multi-action anchor-5 example; it exceeds anchor 4's typical weakness since the "when" clause is fully explicit.

4 / 5

Trigger Term Quality

Six natural phrases users would plausibly say ("review tests", "check test quality", "audit tests", "improve tests", "test code review", "playwright best practices check") with good synonym coverage. Not anchor 5: no file-extension cues (e.g., ".spec.ts") and no standalone "playwright" trigger for users who just name the tool.

4 / 5

Distinctiveness Conflict Risk

"Playwright tests" carves a clear niche with a tool-specific trigger ("playwright best practices check"). Minor overlap risk remains with generic test-review skills because triggers like "review tests" or "audit tests" are not tool-qualified — anchor 4, not 5.

4 / 5

Total

15

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

alirezarezvani/claude-skills

Path

.gemini/skills/pw-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.