Content
56%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.
The skill body delivers a well-sequenced, actionable technical-debt workflow with concrete thresholds and honest labeling of hypothetical figures. Its main weaknesses are verbosity (long taxonomies and templates Claude largely knows) and a monolithic structure with no progressive disclosure into reference files.
Suggestions
Cut the debt-type taxonomy to a compact checklist (type + quantify metric only) and drop the concepts Claude already knows, such as definitions of cyclomatic complexity or feature envy.
Move the metrics dashboard YAML, communication-plan markdown, and team-allocation templates into a references/ file (e.g., references/templates.md) and link to them from SKILL.md.
Replace illustrative stub code (the pass-body PaymentService) with either complete runnable snippets or an explicit note that the pattern is a template to adapt, and add per-phase validation checkpoints (e.g., verify metrics against the repo before impact scoring).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~395-line body extensively catalogs concepts Claude already knows ("High cyclomatic complexity (>10)", "Feature envy", "God classes", "Brittle tests") and inlines long hypothetical dashboards, team-allocation YAML, and stakeholder-report templates, matching the 2 anchor (noticeably verbose, several padded sections). Not 3 because the padding spans many sections rather than being occasional. | 2 / 5 |
Actionability | Concrete thresholds (">500 lines, >20 methods", "max 5% duplication"), quantification prompts per debt type, and an executable facade/feature-flag migration pattern in Python give mostly executable guidance. Not 5 because some examples remain illustrative stubs (e.g., PaymentService.process_payment is a pass body) and no runnable commands for scanning a repo are given. | 4 / 5 |
Workflow Clarity | The eight numbered phases (Inventory through Success Metrics) plus a defined Output Format give a clear sequence, and Requirements supplies checkpoints ("Report missing cost/usage inputs as unknown", "Stop and ask for clarification if required inputs... are missing"). Not 5 because there are no explicit validate-then-proceed feedback loops between analysis phases; not 3 because sequence and key checkpoints are present. | 4 / 5 |
Progressive Disclosure | Sections are clearly headed, but there are no bundle files at all; content that clearly belongs in separate references (metrics dashboard template, communication-plan templates, the full debt taxonomy) is inlined in a monolithic ~400-line file. This matches the 3 anchor (some structure, should-be-separate content inline); not 4 because no one-level-deep references exist to signal. | 3 / 5 |
Total | 13 / 20 Passed |