Content
92%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 well-crafted skill body: a tight, correctly ordered procedure with genuine validation checkpoints, failure-mode recovery, and clean one-level-deep disclosure into real reference files and a bundled inspection script. The only gap keeping actionability from a top score is that a few common-case commands (baseline generation, a minimal neon snippet) live only in the reference files.
Suggestions
Add the exact baseline generation command (e.g. `vendor/bin/phpstan analyse --generate-baseline`) to step 5 so the most common baseline workflow is copy-paste ready without opening references.
Include a 5-line minimal `phpstan.neon` example or point explicitly to the template's location in references/configuration.md from step 2 so users land on it directly.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — it never explains what PHPStan or static analysis is, and every line is procedural ("Keep `paths` focused on first-party code", "Do not 'baseline' newly introduced errors"). The one-liner "use as a migration tool, not a trash bin" is a memorable rule, not padding, so the content matches the anchor-5 'every token earns its place' example. | 5 / 5 |
Actionability | Mostly executable guidance: an exact inspect command (`node skills/wp-phpstan/scripts/phpstan_inspect.mjs`), run commands (`composer run phpstan`, `vendor/bin/phpstan analyse`), concrete package names (`szepeviktor/phpstan-wordpress`, `php-stubs/woocommerce-stubs`), and specific typing patterns (`WP_REST_Request<...>`, array shapes for hook callbacks). It falls short of anchor 5 because copy-paste-ready details like a baseline-generation command or an inline `phpstan.neon` snippet are deferred to references rather than covered for the common cases. | 4 / 5 |
Workflow Clarity | A clearly sequenced procedure (steps 0–5: discover entrypoints → stubs → config → fixes → third-party classes → baseline) with an explicit Verification section (run the discovered command, confirm the baseline "didn't grow unexpectedly", "re-run after changing `ignoreErrors`") plus Failure modes and Escalation sections that provide feedback loops and recovery guidance. This matches the anchor-5 pattern of explicit validation steps with error-recovery loops. | 5 / 5 |
Progressive Disclosure | The body is a concise overview with well-signaled, one-level-deep references — `references/configuration.md`, `references/third-party-classes.md`, `references/wordpress-annotations.md` (all present in the bundle) — plus a real bundled script (`scripts/phpstan_inspect.mjs`). Content is appropriately split: rules of thumb inline, templates and details in references, exactly matching the anchor-5 structure. | 5 / 5 |
Total | 19 / 20 Passed |