Content
42%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill provides reasonable structure and appropriate progressive disclosure but suffers from lack of actionability. The instructions are abstract guidance rather than concrete, executable steps. Without code examples, specific refactoring patterns, or clear decision criteria, Claude would struggle to apply this skill consistently.
Suggestions
Add concrete code examples showing before/after refactoring for common patterns (e.g., extracting methods, removing duplication, applying SOLID principles)
Replace vague instructions like 'Identify high-impact refactor candidates' with specific criteria or heuristics (e.g., 'Functions over 50 lines, classes with more than 5 dependencies, duplicated code blocks')
Add explicit validation checkpoints with specific commands or checks (e.g., 'Run test suite: `npm test`, verify no regressions before proceeding')
Remove the redundant 'Context' section and the persona statement at the top to improve conciseness
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient but includes some unnecessary framing ('You are a code refactoring expert...') and the 'Context' section restates what's already clear. The 'Use this skill when' and 'Do not use this skill when' sections add value but could be tighter. | 2 / 3 |
Actionability | Instructions are vague and abstract ('Identify high-impact refactor candidates', 'Apply changes with a focus on readability') with no concrete code examples, specific commands, or executable guidance. Describes what to do rather than showing how. | 1 / 3 |
Workflow Clarity | Steps are listed in a logical sequence (identify, break down, apply, validate) but lack explicit validation checkpoints, specific criteria for 'high-impact candidates', or concrete feedback loops for error recovery. | 2 / 3 |
Progressive Disclosure | Appropriately structured with a concise overview and clear one-level-deep reference to 'resources/implementation-playbook.md' for detailed patterns. Content is well-organized with distinct sections. | 3 / 3 |
Total | 8 / 12 Passed |