Content
71%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 well-structured and appropriately brief for a conventions skill: it defines repository boundaries, a validation-gated workflow with a feedback loop, and a single clearly signaled verification script. Its main weakness is actionability — the workflow steps reference checks and applies without giving the concrete commands to run.
Suggestions
Make the workflow steps executable by naming the commands, e.g. step 1: "Run `bash scripts/check.sh`", step 3: "Run `terraform plan` to review, then `terraform apply`", instead of "check that everything is still working" and "apply your changes".
State the check mechanism inline in the workflow ("1. Run @scripts/check.sh") rather than deferring it to the trailing "How to verify infrastructure code" section, so each step is self-contained.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence (e.g. "Never add IAM bindings or service account changes to wheel-of-meeting here"), with only minor throat-clearing padding ("it's important to maintain a consistent style", "It is important to keep terraform valid and working") that could be trimmed. It is not a 5 because of those sentences, and not a 3 because nothing over-explains concepts Claude already knows. | 4 / 5 |
Actionability | Concrete pointers exist (fresh branch, @scripts/check.sh as the verification entry point, commit, push), but the workflow steps themselves are not executable: "check that everything is still working" and "apply your changes" give no actual commands (no terraform validate/plan/apply). This matches the 'some concrete guidance but incomplete; missing key details' anchor rather than the mostly-executable level 4. | 3 / 5 |
Workflow Clarity | A clear numbered sequence is present with explicit validation checkpoints (step 1 check, step 3 re-verify), a fix-and-retry feedback loop ("If something breaks, fix it and try again"), and a stop-and-ask condition ("if step 1 fails, DO NOT continue"), so the destructive-operation cap does not apply. It falls short of 5 because the validation mechanism is only revealed in a later section and "apply your changes" is ambiguous about what to actually run. | 4 / 5 |
Progressive Disclosure | The skill is under 50 lines with no need for external reference files, and per the simple-skill guidance well-organized sections suffice: Repository setup, Workflow, and How to verify are cleanly separated, with one clearly signaled one-level-deep reference (@scripts/check.sh). It is not a 4 because there are no organization gaps to point to. | 5 / 5 |
Total | 16 / 20 Passed |