CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

test-all

Parallel test orchestrator. Runs all 9 test suites concurrently via Task sub-agents and the iwsdk CLI. Handles build, example setup, dev servers, agent launch, polling, retries, and result aggregation.

Quality

77%

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 ./.claude/skills/test-all/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

75%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 strongly actionable, well-sequenced orchestration skill with copy-paste commands, explicit stop-on-failure checkpoints, timeouts, and a bounded retry policy. Its weaknesses are redundancy (the dynamic-port point is made four times, and Key Design Decisions duplicates phase content) and one genuine workflow contradiction between the no-polling instruction and the file-polling design note.

Suggestions

State the dynamic-port/no-port-passing rule once (in Phase 3) and delete the repetitions in the Test Map note and Key Design Decisions to reclaim tokens.

Resolve the polling contradiction: pick one policy (wait for completion notifications only, reading output_file solely on the 20-minute timeout) and align the Key Design Decisions section with it.

Move the "Boolean values must be JSON booleans" note into the per-suite test skills where test assertions are actually written, since the orchestrator never sets components itself.

DimensionReasoningScore

Conciseness

The "ports are dynamic / sub-agents don't need ports" point is repeated four times (Test Map note, Phase 3, and twice under Key Design Decisions), the Key Design Decisions section restates Phase 3–4 content, and "Boolean values must be JSON booleans" is a test-authoring detail that belongs in the sub-skills. Mostly efficient, but several sections could be tightened — matching anchor 3 rather than 4, where over-explanation would be only minor.

3 / 5

Actionability

Every phase supplies copy-paste-ready commands ("node scripts/test-prep.mjs install", "node scripts/test-servers.mjs start/ports/stop", "pnpm build:tgz"), a verbatim sub-agent prompt template with substitution placeholders, and a concrete retry policy with explicit limits. Fully executable guidance covering the common cases; not 4, as there are no real gaps in executable detail.

5 / 5

Workflow Clarity

The Phase 1–7 sequence is clearly ordered with explicit checkpoints ("All must succeed. **Stop on failure.**", 60s server-readiness timeout, 20-minute hard timeout, retry limits, troubleshooting section). However, Phase 5's "**Do NOT poll, do NOT run bash commands to check output files**" directly contradicts Key Design Decisions' "Use `Read` tool on the output_file to check progress", and Phase 5 says to "proceed to Phase 7" without routing through Phase 6's retry decision — matching anchor 4 (minor gaps), not 5.

4 / 5

Progressive Disclosure

The skill is a single self-contained file with well-organized sections (Test Map, phases, design decisions, troubleshooting), and its references to the per-test SKILL.md files are clearly signaled in the Test Map table. Good structure with minor organization gaps — at ~250 lines, content like the troubleshooting section and sub-agent prompt template could arguably live in separate files; not 5, but comfortably above anchor 3's 'content that should be separate is inline'.

4 / 5

Total

16

/

20

Passed

Description

71%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.

A highly specific, third-person description that enumerates the full orchestration lifecycle (build, prep, servers, launch, polling, retries, aggregation). Its main weakness is the complete absence of a "Use when..." trigger clause, which caps completeness, and it has minor overlap risk with the sibling per-suite test skills.

Suggestions

Add an explicit trigger clause, e.g. "Use when the user asks to run all tests, the full test suite, or all 9 IWSDK test suites in parallel."

Include natural trigger variations users might say — "run all tests", "full test run", "regression run" — to improve trigger-term coverage.

Differentiate from sibling per-suite skills by noting this is the umbrella orchestrator (e.g., "one command to run every suite; use test-* skills for a single suite").

DimensionReasoningScore

Specificity

"Runs all 9 test suites concurrently via Task sub-agents and the iwsdk CLI. Handles build, example setup, dev servers, agent launch, polling, retries, and result aggregation" lists seven concrete actions plus the execution mechanism, matching the comprehensive-coverage anchor. Not 4: there are no minor coverage gaps — the full orchestration lifecycle is enumerated.

5 / 5

Completeness

The "what" is clear ("Parallel test orchestrator. Runs all 9 test suites concurrently... Handles build, example setup, dev servers..."), but there is no "Use when..." clause or equivalent explicit trigger guidance, which the guidelines cap at 3. Not 4: the "when" is entirely absent rather than merely under-specified.

3 / 5

Trigger Term Quality

Natural keywords like "test", "test suites", "build", "dev servers", "retries", and "result aggregation" give good coverage of what a user would say. Not 5: it lacks common synonyms and variations ("run all tests", "regression", "CI"); not 3: keyword coverage is good, not merely 'some relevant keywords'.

4 / 5

Distinctiveness Conflict Risk

"Parallel test orchestrator... all 9 test suites... iwsdk CLI" carves a distinct orchestrator niche, but it overlaps with the closely related per-suite test-* skills it dispatches to, which share the testing trigger space. Not 5: minor overlap risk with those closely related skills; not 3: the orchestrator role is clearly differentiated from running a single suite.

4 / 5

Total

16

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

referenced_paths_exist

Referenced path issues: 10 missing

Warning

Total

14

/

16

Passed

Repository

facebook/immersive-web-sdk

Path

.claude/skills/test-all/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.