Content
67%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-structured, navigable skill body with a clear procedure, verification checklist, and a genuine one-level-deep reference bundle. The main deductions are duplicated version-sensitive apiVersion content between SKILL.md and references/block-json.md, sparse inline executable examples, and two script paths that don't resolve against the actual bundle layout.
Suggestions
Collapse step 2's apiVersion detail into a 2-3 line summary plus a pointer to references/block-json.md, removing the duplicated rationale and migration notes from the body.
Fix script paths to match the bundle layout (scripts/list_blocks.mjs) and note that the wp-project-triage script is an external sibling-skill dependency, or vendor it.
Add one small inline example for the most fragile operation — a minimal `deprecated` entry with `save` and `migrate` — so step 9 is executable without a file jump.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly tight bullet-style guidance, but step 2 spends ~15 lines on version-sensitive detail ("WordPress 6.9 enforces apiVersion: 3", "WordPress 7.0 will run the post editor in an iframe", style isolation/viewport rationale) that substantially duplicates references/block-json.md, which repeats the same apiVersion requirements, rationale, and migration checklist. This is more than minor trimming and the version-numbered content is not confined to a deprecated/old-patterns section, so it does not reach anchor 4. | 3 / 5 |
Actionability | Guidance names concrete commands (`node skills/wp-block-development/scripts/list_blocks.mjs`) and exact APIs (`useBlockProps()`, `useBlockProps.save()`, `get_block_wrapper_attributes()`, `useInnerBlocksProps()`), with specific pitfalls ("changing `name` breaks compatibility", "avoid the deprecated `meta` attribute source"). It is not a 5 because several steps defer entirely to reference files without a single inline executable snippet (e.g., no example `deprecated` entry or registration call), leaving minor gaps. | 4 / 5 |
Workflow Clarity | A clearly sequenced procedure (triage → scaffold → model choice → metadata → registration → patterns → serialization → deprecations → tooling) with a dedicated Verification checklist ("Saving + reloading does not create 'Invalid block'") and a Failure modes section that routes errors to specific references. It falls short of anchor 5 because intermediate checkpoints are sparse (only step 1 has an explicit confirm step) and the verification items lack the concrete commands a full feedback loop would use. | 4 / 5 |
Progressive Disclosure | The body is a lean overview with well-signaled, one-level-deep 'Read: references/*.md' pointers, and all 10 referenced reference files exist in the bundle with real content. However, scoring against the actual bundle: the referenced path `skills/wp-block-development/scripts/list_blocks.mjs` resolves only to `scripts/list_blocks.mjs` in the bundle layout, and `skills/wp-project-triage/scripts/detect_wp_project.mjs` does not exist in this bundle at all — minor navigation gaps that keep it below the anchor-5 'easy navigation' bar. | 4 / 5 |
Total | 15 / 20 Passed |