Content
66%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 body is a thorough, well-sequenced refactoring procedure with strong validation and recovery loops, but it is a monolithic wall of text with no progressive disclosure and a broken script reference.
Suggestions
Move the REFACTOR_TEMPLATE and the Team Mode addendum into separate reference files (e.g. references/refactor-template.md, references/team-mode.md) and link to them from SKILL.md.
Either bundle scripts/ast_grep_helper.py or replace the invocations with the plain `sg` CLI commands already shown so no referenced file is missing.
Trim the repeated 'this section wins' override sentences to a single statement near the compatibility table.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~770-line body is mostly efficient procedural guidance rather than concept padding, but it is long and repeats itself (the 'if a code block below conflicts, this section wins' override appears three times) and could be tightened. | 3 / 5 |
Actionability | It gives concrete LSP calls, ast-grep/sg commands, TodoWrite lists, git commit snippets, and agent-spawn examples; minor gaps are that the core examples use OpenCode-only tools flagged 'do not call literally' and reference a non-existent scripts/ast_grep_helper.py. | 4 / 5 |
Workflow Clarity | Phases 0-6 are explicitly sequenced with an intent-gate checklist, per-step lsp_diagnostics/test/typecheck verification, a STOP-REVERT-DIAGNOSE failure recovery protocol, and abort conditions — full feedback loops for a destructive operation. | 5 / 5 |
Progressive Disclosure | There are no bundle files (references/, scripts/, assets/ absent) yet the body inlines large blocks that belong in separate files (the REFACTOR_TEMPLATE and team-mode addendum) and references scripts/ast_grep_helper.py, which does not exist. | 2 / 5 |
Total | 14 / 20 Passed |