Content
71%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A highly actionable body with executable commands, complete configs, and strong before/after examples. Its weaknesses are padding (re-explained mutation basics, version-sensitive details) and a monolithic structure: runner-specific deep detail is inlined rather than split into clearly signaled reference files.
Suggestions
Delete or compress the "Core Concept" and "Common Mutation Types" sections; Claude already knows what mutants, killed/survived, and arithmetic/logical operator mutations are — keep only the project-specific scoring thresholds.
Move the Bun-native-runner section (requirements, config, key behaviors) into a `references/stryker-bun-runner.md` file and link it from a short pointer in SKILL.md, so the main file stays a lean overview.
Add an explicit validation checkpoint to the Workflow (e.g., "Re-run until the mutation score meets the configured threshold; confirm `bunx stryker run` exits 0 against `break: 60`") to close the feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient (terse install/config/run commands), but the "Core Concept" section and "Common Mutation Types" re-explain mutation-testing basics Claude already knows, and unpinned version requirements ("Bun >= 1.3.7", "@stryker-mutator/core ^9.6.1") are time-sensitive details not confined to a deprecation section. Not 4 because these padded sections are a clear, trimmable overhead beyond minor instances. | 3 / 5 |
Actionability | Install commands, complete copy-paste-ready `stryker.config.mjs`/`stryker.conf.mjs` objects, concrete run commands (`bunx stryker run --incremental`), and before/after weak-vs-strong test examples fully cover the common TypeScript and Python cases. It matches anchor 5's fully executable standard. | 5 / 5 |
Workflow Clarity | The Workflow section sequences coverage check → run → check report → fix survived mutants → re-run incrementally, including verification steps (coverage gate, report inspection, incremental re-run loop), so the batch-operation cap at 3 does not apply. It stops short of anchor 5 because there is no explicit validation checkpoint such as verifying the mutation score meets the threshold before finishing. | 4 / 5 |
Progressive Disclosure | The body is well-sectioned but everything lives inline in a ~230-line SKILL.md with no reference files; niche detail (the entire Bun-native-runner requirements/behaviors section and both full configs) is the kind of content anchor 3 describes as could-be-separate. Not 4 because there are no clearly signaled external references at all, and the file is well beyond the simple-skill 50-line exception. | 3 / 5 |
Total | 15 / 20 Passed |