Content
78%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 genuinely actionable, with strong progressive disclosure that keeps the overview lean while routing detail to verified reference files. The main gaps are minor: restated safety language, --help-only run examples, and no explicit error-recovery loop in verification.
Suggestions
Consolidate the repeated safety-boundary statements into the Hard Safety Boundary section and reference it once from downstream sections to trim redundant tokens.
Replace at least one "--help" line in Run Locally with a full end-to-end example (e.g., `python3 scripts/cohort_table_generator.py assets/aggregate_cohort_table_template.json -o out/`) so users see the actual invocation pattern.
Add an explicit feedback loop to the Verification section (e.g., "If tests or AST parse fail, fix the script and re-run until both pass before proceeding").
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence—no padding with basic concept explanations—and uses compact tables and bullets; not a 5 because the safety boundary is restated across multiple sections (Hard Safety Boundary, Data Gate, Required Artifact Header, per-topic "Do not..." lines), adding tokens that could be consolidated. | 4 / 5 |
Actionability | Provides a concrete need→asset→script selection table, an executable Verification block ("python3 -m unittest discover...", "python3 -c \"import ast...\""), and a Data Gate checklist; not a 5 because the Run Locally section shows only "--help" invocations rather than a full example of running a script with real input to produce output. | 4 / 5 |
Workflow Clarity | A clear four-step Workflow (Frame → Select → Run → Human Review) with a前置 Data Gate validation checkpoint, a Required Artifact Header checklist, and a Verification section; not a 5 because the Verification block lacks an explicit validate→fix→retry feedback loop for the batch/destructive-adjacent operations. | 4 / 5 |
Progressive Disclosure | SKILL.md acts as an overview with each topic ending in a well-signaled one-level-deep "See `references/X.md`" pointer, a terminal Reference Map for navigation, and a selection table; all referenced references/scripts/assets files resolve to real bundle files. | 5 / 5 |
Total | 17 / 20 Passed |