Content
82%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 strong, command-dense dispatcher body with excellent actionability and a clearly gated workflow from input confirmation through preflight, test run, real run, and downstream DE. Weaknesses are minor: duplicated local-runner invocation, some harness-internal phrasing, and no explicit post-run output-validation checkpoint.
Suggestions
Collapse the duplicated local-runner invocation: keep the full 'Plugin-Owned Local Paths' command and drop or shorten the partial repeat under 'Local execution run'.
Add an explicit post-run validation checkpoint, e.g., 'Check validation/ and MultiQC summaries in the run envelope before starting differential expression.'
Drop or rewrite the eval-harness sentence in user-facing terms, or move it to a reference doc if it only matters to plugin maintainers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean: terse input checklists, command blocks, and gating sentences with no concept explanations Claude already knows. Not anchor 5 because the local runner invocation appears twice (full 'Plugin-Owned Local Paths' version and a partial repeat under 'Local execution run'), and 'Do not depend on development-only eval harness paths in a shared package' is harness-internal phrasing that earns little. | 4 / 5 |
Actionability | Commands are fully executable and copy-paste ready for the common cases: the two local runner invocations with explicit flags, four preflight commands, a test-profile kickoff, and a real-run skeleton with concrete parameter values. Placeholders (samplesheet.csv, GRCh38) are appropriately generic. | 5 / 5 |
Workflow Clarity | The sequence is clear and gated: confirm inputs, dispatch, preflight, test kickoff, real run, then DE only after 'biological replicates, design formula, and contrasts' are confirmed, plus strandedness inference before final counts. Not anchor 5 because this batch pipeline lacks an explicit output-validation checkpoint (e.g., inspect MultiQC/validation artifacts before proceeding to DE), keeping a minor validation gap. | 4 / 5 |
Progressive Disclosure | Well-organized single-file skill with clean sections, no nested references, and no inlined bulk that belongs in separate files; the skill exceeds 50 lines, so the simple-skill exception to anchor 5 does not apply. The external script paths ('plugins/ngs-analysis/scripts/...') are not bundled in references/ or scripts/, so they cannot be verified from the skill directory. | 4 / 5 |
Total | 17 / 20 Passed |