Content
38%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 a heavily templated stub: it signals concrete commands and a parameter table but points to bundle files (scripts/main.py, references/audit-reference.md) that do not exist, and is padded with irrelevant boilerplate sections. Workflow steps are generic process guidance rather than a validated computational pipeline.
Suggestions
Ship the referenced bundle (scripts/main.py, requirements.txt, references/audit-reference.md) or remove the references; dead links make the skill non-executable and break progressive disclosure.
Cut the templated boilerplate that is irrelevant to a local statistical calculation — Security Checklist items like 'API requests use HTTPS only', the Risk Assessment 'Network Access: External API calls: High' row, Lifecycle Status, and Evaluation Criteria test cases — and remove the circular 'See ## X above' fillers.
Replace the abstract Workflow steps with a concrete computational sequence with checkpoints: choose test type → supply effect/alpha/power → run script → verify achieved power and plausible N → report assumptions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is noticeably padded with templated boilerplate that adds no value (Risk Assessment table, a Security Checklist with irrelevant items like 'API requests use HTTPS only' for a local stats script, Lifecycle Status, Evaluation Criteria test cases, and circular 'See ## Prerequisites above for related details' fillers), matching 'noticeably verbose; several unnecessary ... padded sections'; it is not 1 because it does not extensively explain basic concepts Claude already knows. | 2 / 5 |
Actionability | Concrete guidance is present — real example invocations ('python scripts/main.py --test ttest --effect 0.5 --alpha 0.05 --power 0.8') and a parameter table with specific flags — but the referenced scripts/main.py does not exist in the bundle, so the commands are incomplete and not actually executable as packaged, fitting 'some concrete guidance but incomplete; missing key details' rather than the 'minor gaps' level above. | 3 / 5 |
Workflow Clarity | A 5-step Workflow and a py_compile 'Quick Check' checkpoint plus an Error Handling fallback give a sequenced path, but the workflow is generic project-management boilerplate ('confirm objective', 'validate scope') rather than a validated computational pipeline with checkpoints on the actual calculation; it is not 4 because verification of computed output (e.g., achieved power, plausible N) is implicit or missing. | 3 / 5 |
Progressive Disclosure | Header structure exists, but the body references four bundle files that are absent (scripts/main.py, references/, references/audit-reference.md, requirements.txt), so the signaled references are dead promises, and inline boilerplate (Security Checklist, Risk Assessment, Lifecycle) that belongs elsewhere is kept in SKILL.md; this matches 'minimal structure; content that clearly belongs in separate files is inlined' rather than 3, and is not 1 because it is not a monolithic wall of text. | 2 / 5 |
Total | 10 / 20 Passed |