Content
68%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-structured, actionable skill body with strong bundle hygiene and a clearly sequenced workflow. Its main weakness is missing explicit validation in the step sequence for a batch/destructive operation, plus some body/reference duplication and undefined helpers.
Suggestions
Add an explicit validation step to the Steps sequence (e.g., step 4.5: run scripts/validate-manifest.sh after writing manifest.yaml and only proceed on pass) — the script exists in the bundle but is never sequenced in the workflow.
Deduplicate content between the body and references/reference.md — the token-threshold sizing table and security skip patterns appear in both; keep one-line summaries in the body and the full rules in the reference.
Replace or define the undefined `error` helper in the step-1 bash block with an executable pattern (e.g., `echo "..." >&2; exit 1`) and mention scripts/scan-in-folder.sh in the body since the body's step 2 reimplements its scan with a glob.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence (token-estimation formula, threshold tables, terse steps), but the sizing table and security skip patterns are duplicated between the body and references/reference.md, and the Anti-Patterns section pads some entries with lengthy Why prose, keeping it just below fully lean. | 4 / 5 |
Actionability | Concrete executable guidance throughout (bash prerequisite checks, glob patterns, the priority/token threshold table, summary naming convention, find-based copy command), with minor gaps: the `error` helper in the step-1 bash block is undefined, and `Task(summarize-for-context)` depends on a sub-agent only documented in the reference file. | 4 / 5 |
Workflow Clarity | Steps 1-7 are clearly sequenced, but this is a batch/destructive workflow (--force overwrite, bulk file copy) and the Steps sequence never explicitly runs scripts/validate-manifest.sh or any other validation checkpoint — validation is only implicit via the reference file and an anti-pattern note, so the batch-operation cap applies. | 3 / 5 |
Progressive Disclosure | Good structure: SKILL.md is an overview, references/reference.md is a real, clearly signaled, one-level-deep file, and the two scripts exist in the bundle. Minor gaps: the scripts are surfaced only through reference.md rather than the body, and content duplicated between body and reference (sizing rules, security patterns) blurs the split. | 4 / 5 |
Total | 15 / 20 Passed |