Content
42%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 with useful use/don't-use and safety sections, but it provides almost no executable guidance — every concrete pattern is deferred to an external playbook file that does not exist in the bundle. Workflow steps lack validation checkpoints despite the skill covering destructive operations.
Suggestions
Inline at least one copy-paste-ready defensive snippet (e.g. `set -euo pipefail` with an `ERR`/`EXIT` trap and a logging function) so the skill is actionable without the external file.
Add an explicit validation/verification checkpoint to the workflow (e.g. 'After enabling strict mode, run `bash -n script.sh` and execute against a dry-run input before real execution').
Fix the broken reference: either create `resources/implementation-playbook.md` (or move it under `references/`) or remove the reference and inline the key patterns; de-duplicate the single inline 'Refer to...' line versus the '## Resources' section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with clear sections, but the padded marketing opener ('Comprehensive guidance... to prevent common pitfalls and ensure reliability'), eight overlapping 'Use when' bullets, and a duplicated resource reference (stated both inline and again under '## Resources') could be tightened; not 4 because the redundancy is noticeable, not 2 because no concept-explanation padding is present. | 3 / 5 |
Actionability | The Instructions are high-level hints only ('Enable strict mode and safe defaults', 'Validate inputs, quote variables', 'Add logging, error traps') with no executable code, concrete commands, or snippets (e.g. no `set -euo pipefail`, no trap example); all concrete guidance is deferred to an external file, matching the anchor for minimal concrete guidance missing the specific steps to execute. | 2 / 5 |
Workflow Clarity | A four-step sequence is present (confirm environment → enable strict mode → validate/quote → add logging/traps/tests), but there are no validation or verification checkpoints, and the Safety section flags destructive operations — per the rubric, a destructive/batch skill without validation checkpoints is capped at 3. | 3 / 5 |
Progressive Disclosure | Sections are well-organized and the reference is one level deep and clearly signaled, but the referenced path `resources/implementation-playbook.md` does not exist (no `resources/`, `references/`, `scripts/`, or `assets/` bundle directory is present), so navigation is broken; capped at 3 because a clearly signaled reference points at a missing file. | 3 / 5 |
Total | 11 / 20 Passed |