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 highly actionable body: copy-paste commands throughout, an enforced dry-run-first discipline with feedback loops for destructive rewrites, and well-organized one-level references. The two deductions are repeated emphasis of the same invariants across three sections, and bundle-referenced install scripts (install.sh/install.ps1) that are not actually present.
Suggestions
Ship or remove the claimed install scripts: the body says the skill "ships ... install.sh (POSIX) and install.ps1 (Windows)", but neither file exists in the bundle — either add them or point install instructions solely at references/install.md.
Consolidate the repeated invariants: the two-pass write trick, dry-run-first, and validate-first rules each appear in 'Three things the agent must internalize', 'Always run dry-run first', and 'Invariants' — stating each once with a cross-reference would trim the body without losing the message.
Trim the tool-availability preamble (the 'OmO native' MCP section) or move it to a reference file, since it describes one specific environment's tool registration rather than guidance every invocation of the skill needs.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with commands, a regex-anti-pattern table, and a decision tree, with almost no explanation of concepts Claude already knows. However, the two-pass write trick, dry-run-first, and validate-first rules each appear three or more times ("Three things the agent must internalize", "Always run dry-run first", "Invariants"), which is more than anchor 4's 'minor' trimming opportunities though well short of anchor 3's padding. | 4 / 5 |
Actionability | Every section gives copy-paste-ready commands with real flags and expected outcomes (e.g., `python3 scripts/ast_grep_helper.py validate '\w+' --lang ts` → "exit 2: regex \w not supported"), covering search, replace, scan, validate, debug, stdin, and inline YAML rules. Fully executable and covering the common cases, matching anchor 5. | 5 / 5 |
Workflow Clarity | For the destructive/batch rewrite operation the skill prescribes a 5-step dry-run→inspect→refine→apply flow with an explicit feedback loop ("If wrong: refine pattern, go back to step 1"), and a prioritized 5-step recovery sequence for 0-match surprises. The missing-validation cap does not apply — checkpoints are explicit. | 5 / 5 |
Progressive Disclosure | Structure is strong: a prioritized "Required reading" list where every referenced path (references/patterns.md, pitfalls.md, recipes.md, cli.md, yaml-rules.md, sgconfig.md, install.md) is a real one-level-deep file, and scripts/ast_grep_helper.py exists. But the body claims (line 10) the skill ships `install.sh` and `install.ps1`, neither of which exists in the bundle — a genuine referenced-path gap that keeps this below anchor 5. | 4 / 5 |
Total | 18 / 20 Passed |