Content
81%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 content is a highly actionable, rigorously validated 8-step workflow with executable bash for every step and explicit failure handling — top marks for actionability and workflow clarity. The weaknesses are token efficiency (a verbatim ~30-line bash block duplicated across Steps 3 and 5, plus an extended narration-rules section) and bundling (the duplicated resolution logic would fit better as a shared script, keeping SKILL.md leaner).
Suggestions
Factor the duplicated valid_root / plugin-root resolution bash block (Steps 3 and 5) into a small shared script under scripts/ or have Step 5 reuse the PLUGIN_ROOT resolved in Step 3, cutting ~30 lines.
Compress Step 0's narration-rules contract (violation/normal example blocks) to one-line rules each, or move the examples into references/cursor-cli-only.md alongside the other governance content.
Make the TASK_SUMMARY export an explicit, concrete step (a one-line export command after Step 1) instead of describing it retrospectively in Step 6.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly operational (commands, not concept explanations), but it can be tightened: the ~30-line valid_root/plugin-root resolution bash block is duplicated verbatim in Step 3 and Step 5, and Step 0 spends ~40 lines on narration-rule contract with violation/normal examples that could be compressed. This fits the anchor 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the level-4 anchor, where over-explanation would be only minor. | 3 / 5 |
Actionability | Every step ships a fully executable bash block with explicit exit-code handling, retry limits, error markers (ERROR/WARN/ABORT/CHERRY_PICK_CONFLICT), and fallbacks (e.g. TASK_SUMMARY fallback message, --no-verify rationale). This is copy-paste-ready guidance covering the common cases, matching the top anchor; the only placeholders (<task-description>, <task>) are inherent to a parameterized workflow. | 5 / 5 |
Workflow Clarity | Steps 0-8 are clearly sequenced with validation checkpoints throughout: parallel pre-checks (Step 2), plugin-root/backend/model resolution guards (Step 3), dirty-changeset auto-commit/amend before review (Step 6), contract-gate scripts run in the worktree, Plans.md clean-precondition check before cherry-pick, and explicit failure/abort/retry branches (1 retry, then REQUEST_CHANGES or ABORT). This matches the top anchor with explicit validation steps, feedback loops, and error-recovery paths — critical given the destructive cherry-pick-to-main and force worktree removal. | 5 / 5 |
Progressive Disclosure | Structure is good: the single bundle reference (references/cursor-cli-only.md, verified to exist) is clearly signaled twice with a ${CLAUDE_SKILL_DIR} link and one level deep, and the inline bash blocks belong inline as executable workflow. It falls short of the top anchor because the body is a ~400-line monolith where the duplicated resolution logic (used in two steps) arguably belongs in a shared script under scripts/, and the narration contract could live in a reference file. | 4 / 5 |
Total | 17 / 20 Passed |