Content
92%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 an exemplary instruction-only guard skill: a crisp no-op policy with concrete forbidden operations, an exact fallback message, precise config/file references, and a clearly gated re-enable procedure. Its only blemish is mild repetition of the 'never commit' prohibition and a rationale section that could be tightened by a line or two.
Suggestions
State the core prohibition once in Behavior and close with the existing bolded 'never commit' — drop the duplicate phrasing in the intro paragraph.
Condense the three-bullet Rationale to a single line (e.g. 'Commit format requires Trello-aware data and user-owned granularity; see task.md') to tighten conciseness toward anchor 5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — no git basics, no concept explanations — but the core prohibition is stated three times (intro, Behavior item 1, closing 'Until then: **never commit.**') and the three-bullet Rationale could be condensed to one line. This matches anchor 4 (efficient, minor over-explanation that could be trimmed), not anchor 3's 'some unnecessary explanation' and not anchor 5's every-token-earns-its-place. | 4 / 5 |
Actionability | For an instruction-only no-op skill the guidance is fully executable: exact forbidden operations ("git add, git commit, or any mutating git operation"), an exact copy-paste output string ('[spec-kit] auto-commit disabled by project policy; commit manually'), exact config keys ("auto_commit.<event>.enabled") and file paths ('.specify/extensions/git/git-config.yml'), plus a concrete misconfiguration response. Anchor 5's 'specific examples cover the common cases' is met. | 5 / 5 |
Workflow Clarity | This is a simple single-purpose skill whose single action (return immediately, optionally emit one informational line) is unambiguous, and it even includes an edge-case checkpoint (hook wiring still calling auto-commit.sh → treat as misconfiguration, report, do not commit). The destructive/batch cap does not apply since the skill exists to prevent mutations, so the simple-skill exception yields 5. | 5 / 5 |
Progressive Disclosure | The body is under 50 lines, has no need for external reference files (none exist in the bundle), and is organized into clean, well-labeled sections (Behavior, Rationale, Execution, Re-enabling). Per the rubric's simple-skill note, this earns 5 with well-organized sections alone. | 5 / 5 |
Total | 19 / 20 Passed |