Content
88%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-engineered CLI skill body: executable commands, complete argument and output documentation, explicit validation checkpoints, and clean reference routing. The two deductions are the duplicated scope sections ('When to Use' vs 'Input Validation') and references to a tests/ directory that does not exist in the bundle, including the default signature matrix path.
Suggestions
Merge the 'Input Validation' section's scope bullets and refusal script into the existing 'When to Use' / 'When Not to Use' sections to remove the near-verbatim duplication.
Bundle the referenced tests/ files (tests/run_tests.R, tests/data/LM22.txt, tests/data/expression_matrix.csv, tests/data/group_info.csv) or update the 'When to Read External Files' table, Testing section, and --signature_file default to paths that exist.
Consider moving the full 20-row Arguments table to references/cli-guide.md and keeping only the required flags inline, trimming SKILL.md token weight.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and token-efficient — argument, output, and error tables with no explanation of concepts Claude already knows — but the 'Input Validation' section (~lines 154-172) restates the 'When to Use'/'When Not to Use' scope almost verbatim, which is duplicative padding that could be trimmed. Not level 5 because of this redundant scope restatement; not level 3 because the rest of the document is lean with every table earning its place. | 4 / 5 |
Actionability | Fully executable guidance: a complete copy-paste Rscript invocation with all flags, a full argument table with types and defaults, concrete input-format examples, error codes with solutions, and a validated test-path command. Not level 4 because the commands cover the common cases end-to-end with no missing key details. | 5 / 5 |
Workflow Clarity | The 4-step workflow includes an explicit validation checkpoint ('Inspect run_record.txt and output_manifest.txt after each run, including failed validation attempts'), and the Error Handling section provides a feedback loop (error code → solution → 'If the error persists, READ: references/troubleshooting.md'). Not level 4 because checkpoints and error-recovery loops are explicit, not merely implied; the destructive/batch cap does not apply since payload preservation and failure recording are validated. | 5 / 5 |
Progressive Disclosure | Good structure overall: a 'When to Read External Files' table clearly signals one-level-deep references (algorithm.md, troubleshooting.md, cli-guide.md — all real files) with purposes, and the body stays an overview. Not level 5 because the body, the argument defaults, and the Testing section reference tests/data/ and tests/run_tests.R paths that are not present in the actual bundle, so key referenced navigation targets are broken. | 4 / 5 |
Total | 18 / 20 Passed |