Content
88%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, lean, and actionable audit workflow with explicit validation feedback loops. The main gap is actionability depth (template URLs and named-fix patterns rather than copy-paste recipes) and some inline content that could be split into a reference file.
Suggestions
Replace the template recipe URL with one or two concrete resolved example URLs so Claude can fetch a real recipe without guessing the plugin/rule segments.
Move the visual-audit rubric table and the code-level cause lists into a references/ file (e.g. visual-rubric.md) and link to it, keeping SKILL.md as a tighter overview.
For the highest-impact fixes (allocation inside useFrame, missing disposal), include a short copy-paste code snippet showing the before/after pattern.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence, with no padding explaining what Three.js, useFrame, or R3F are; every section (hot-path map, triage lists, visual rubric table, cause lists) earns its place. | 5 / 5 |
Actionability | Provides concrete commands (npx react-doctor@latest --verbose, --scope changed), per-finding fix patterns, and a fail-condition rubric table, but the recipe URL is a template (<plugin>/<rule>.md) and fixes are named patterns rather than copy-paste code, leaving minor gaps below the fully-executable top anchor. | 4 / 5 |
Workflow Clarity | Six clearly sequenced steps (Recon, Scan, Triage, Visual audit, Fix, Validate) with explicit validation checkpoints in Step 6 (re-run --scope changed, confirm no regression, re-check failed rows, watch memory profile) and a validate-fix-retry feedback loop, matching the top anchor. | 5 / 5 |
Progressive Disclosure | Well-organized with clean section headers and only a one-level external recipe URL reference, but the dense visual-rubric table and dual cause lists are content that could plausibly live in a separate reference file, and the body exceeds 50 lines so the simple-skill 5 exception does not cleanly apply. | 4 / 5 |
Total | 18 / 20 Passed |