Content
77%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 is highly actionable with an exemplary validated workflow — executable commands everywhere, dry-run/authorization checkpoints, and documented exit-code semantics. Its weaknesses are a monolithic single-file structure with no progressive disclosure offloading (the inlined resolver script and remediation dialogs belong in scripts/ and references/) and localized verbosity, including a duplicated remediation-confirmation sentence.
Suggestions
Move the 40-line plugin-root resolver into scripts/resolve-root.sh and reference it with a one-line invocation, eliminating the repeated 'reuse the resolver from Step 1' instructions and the inlined boilerplate.
Move the AskUserQuestion remediation dialogs (RTK install, RTK hook, missing providers) into a references/remediation.md file, keeping only the decision rules (when to offer, batching with multiSelect, dry-run before apply) in SKILL.md.
Remove the duplicated sentence 'Cache cleanup, stale PID cleanup, login flows, package installation, and plugin updates... require explicit confirmation' (appears in both the Step 5 preamble and the post-repair paragraph), keeping it in one place.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense with skill-specific facts (exit-code contracts, repair rules, JSON schema), but it is noticeably padded in places: 'Cache cleanup, stale PID cleanup, login flows, package installation, and plugin updates... require explicit confirmation' appears nearly verbatim twice (Steps 1's remediation preamble and Step 5), and the 40-line resolver is inlined where a one-line reference to a script would do. Fits 'mostly efficient but could be tightened', not level 2 (no widespread explanation of concepts Claude already knows) nor level 4 (the duplication and inlined boilerplate are more than minor). | 3 / 5 |
Actionability | Everything is copy-paste executable: a complete resolver bash script, exact per-category doctor commands, the JSON outer contract, ready-to-use AskUserQuestion dialogs, and a concrete issue/fix table. This matches 'fully executable; copy-paste ready code or commands; specific examples cover the common cases'; level 4 would leave gaps in the common cases, and there are none. | 5 / 5 |
Workflow Clarity | Five clearly sequenced steps with explicit validation checkpoints and feedback loops: 'repair --dry-run' shown before 'repair --apply', explicit authorization required, 'After repair, rerun only the affected category first, then offer a full scan', and documented exit-code semantics with 'do not retry them as full scans' for usage errors. The destructive/batch cap does not apply because validation is present, so this matches the top anchor rather than the level-4 anchor (which tolerates minor validation gaps). | 5 / 5 |
Progressive Disclosure | There are no bundle files at all — the SKILL.md is a monolithic ~430-line file with good headers and tables, but content that clearly belongs in separate files is inlined: the 40-line resolver script (referenced repeatedly as 'reuse the resolver') belongs in scripts/, and the AskUserQuestion remediation dialogs and legacy-profile summary tables belong in references/. This matches 'some structure but... content that should be separate is inline'; not level 2 (structure is present and navigation is possible) and not level 4 (nothing is split out at all). | 3 / 5 |
Total | 16 / 20 Passed |