Content
82%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 tight, highly actionable reference with copy-paste commands and a genuine preview-then-apply validation checkpoint for batch rewrites. Structure is clear; the main gap is that reference tables could live in bundle files and the rewrite workflow could spell out error recovery.
Suggestions
Trim the "since ast-grep is installed via mise" justification to a bare imperative ("Run all commands through `mise exec --`.") to tighten conciseness.
Add a one-line error-recovery note to the rewrite workflow, e.g. "If the preview does not match the intended sites, refine the pattern and re-preview before applying."
Consider moving the "Key Rust node kinds" table and full YAML rule catalog into a references/ file, keeping SKILL.md as a concise overview with a one-level-deep link.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and dense with non-obvious reference material (Rust node kinds, selector disambiguation, shell escaping), but the "since ast-grep is installed via mise" clause is mild over-explanation that could be trimmed. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready `mise exec -- ast-grep ...` commands covering search, rewrite, scan, JSON, and debug cases, plus complete YAML rule examples. | 5 / 5 |
Workflow Clarity | The rewrite section gives an explicit validation checkpoint ("always preview first, then apply with -U") avoiding the batch-operation cap, but lacks an explicit error-recovery loop if the preview is wrong. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly headed sections (Commands, Pattern syntax, YAML rules, Rust kinds, Shell escaping) with easy navigation, though reference-like material (node kinds, full YAML catalog) is inline rather than split to bundle files. | 4 / 5 |
Total | 17 / 20 Passed |