Content
81%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-engineered instruction skill: a clearly sequenced workflow with explicit validation and revert feedback loops, concrete risk-tier protocols, and clean progressive disclosure into a complete, well-signposted references/ bundle. Minor deductions for boilerplate-heavy Base Framework/Workflow Integration sections and links to shared/ files that don't exist in the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — it never explains basic refactoring concepts, defines refactoring in one line, and puts catalogs in references with 'don't front-load them all'. Anchor 4 ('Efficient; minor instances of over-explanation that could be trimmed') fits because the Base Framework section and Optional Workflow Integration section are boilerplate-heavy and could be tightened; it is well above anchor 3's 'several unnecessary explanations'. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: the risk-tier table with per-tier protocols, the smell→pattern map with named catalog transformations, and precise instructions like 'tests green → apply → tests green → checkpoint/commit'. It does not reach anchor 5 because transformation mechanics and examples are deferred to references/patterns.md rather than covered inline (e.g., no worked characterization-test example), leaving minor gaps. | 4 / 5 |
Workflow Clarity | The workflow is clearly sequenced (Phases A–D, steps 1–11 in order) with explicit validation checkpoints ('If tests go red, revert — don't debug a broken refactoring forward', 'confirm every change is structural'), feedback loops for error recovery, and pointers to per-phase checklists in references/workflow.md — a direct match for anchor 5. | 5 / 5 |
Progressive Disclosure | Excellent one-level-deep structure: every reference mentioned in the body exists in references/ (all 7 files), each step names which file to read when needed, and the Reference Map table gives a 'Read / When' navigation index. It falls just short of anchor 5 because the body links to shared/base/*.md policy files and shared/workflow-contract.md that are not present in the bundle — a minor navigation gap against the actual file listing. | 4 / 5 |
Total | 17 / 20 Passed |