Content
63%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 well-organized, terse runbook with strong progressive disclosure (all referenced bundle files are real and one level deep) and a sequenced workflow with an explicit fail-fast validation loop. Its main weaknesses are actionability — it names categories and tools but includes no executable patterns or exact commands — and low-signal governance sections that dilute conciseness.
Suggestions
Add copy-paste-ready patterns for the core fixes the skill prescribes, e.g. `set -euo pipefail`, quoted expansions `"$var"`, and array-based invocation `cmd=("$@")`, so the quoting/strict-mode guidance is executable rather than named.
Replace the generic validation bullet with exact commands such as `shellcheck script.sh` and `bash -n script.sh` (plus the repo wrapper invocation) so the fail-fast gate has a concrete executable step.
Trim the Philosophy and Anti-Patterns sections, which largely restate Constraints and Avoid in abstract terms, to remove low-signal governance bullets.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Bullets are terse, but the Philosophy and Anti-Patterns sections restate Constraints/Avoid in abstract, low-signal terms ("Keep the skill focused on the decision and workflow the user actually requested", "Hiding uncertainty or missing evidence"), so the body could be meaningfully tightened. Not anchor 4 because the unnecessary text goes beyond minor trimming; not anchor 2 because there is no padded explanation of concepts Claude already knows. | 3 / 5 |
Actionability | Some concrete guidance exists ("Run shellcheck or the nearest repo validation", "Prefer argument arrays and explicit paths over string-built commands", a named checklist of "strict mode, quoting, arrays, traps, paths, and temporary files"), but there are no executable examples or exact commands — key details for a bash-hygiene skill are missing. Not anchor 4 since the guidance is not mostly executable; above anchor 2 because specific tools and patterns are named. | 3 / 5 |
Workflow Clarity | The Workflow section gives a clear sequence (scope, identify shell/wrappers, check, apply patterns, validate, report) and Validation adds an explicit fail-fast feedback loop ("stop at first failed gate; do not proceed until it is fixed and rerun"). Not anchor 5 because steps are generic bullets without specific commands and validation is not a concrete executable step; validation is present, so the destructive-operation cap does not apply. | 4 / 5 |
Progressive Disclosure | The body is a well-sectioned overview with a dedicated Progressive Disclosure section whose referenced bundle files (references/contract.yaml, references/evals.yaml, references/task-profile.json) all exist, are one level deep, and each carry a stated purpose (machine-readable contract, benchmark/quality gates, evaluator thresholds), matching the clear-overview anchor. | 5 / 5 |
Total | 15 / 20 Passed |