Content
81%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, highly actionable SKILL.md with a clear sequenced workflow, explicit validation and error-recovery loops, and well-signaled one-level-deep references. Weaknesses are modest: duplicated content (Workflow vs. Methods, two reference tables, error table vs. troubleshooting.md) and example commands that rely on test fixtures not shipped in the bundle.
Suggestions
Merge the trailing 'Reference Files' table into 'When to Read External Files' and consolidate the 'Methods' section with 'Workflow' — the 50% filter, ≥11-sample strata, and mean/median fallback rules are currently stated three times.
Ship the tests/data fixtures (sample_expression_matrix.csv, sample_groups.csv) with the skill, or rewrite examples to use placeholder user paths so the copy-paste commands are runnable.
Trim the inline error table to the few most common codes and defer the full catalog to references/troubleshooting.md to reduce duplication with that file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes Claude's competence (no explanation of what KNN or CSV is), but contains redundancy: the "When to Use" section restates the description, the "Methods" section restates the "Workflow" section's filter/strata/fallback rules nearly verbatim ("Genes with at least 50% missing values are removed first... strata of 10 samples or fewer..."), and the reference-file table appears twice. This fits level 4 (efficient, minor instances that could be trimmed) rather than level 5 (every token earns its place). | 4 / 5 |
Actionability | Concrete, copy-paste-ready Rscript invocations with every flag, a full argument table with defaults and types, executable examples, and local-validation commands. The gap keeping it from level 5: all example commands depend on `tests/data/sample_expression_matrix.csv` and `tests/data/sample_groups.csv` fixtures that are not present in the skill bundle (no tests/data directory exists), so the examples are not runnable as-is. | 4 / 5 |
Workflow Clarity | A clear five-step sequence (Validate Input → Filter Genes → Build Strata → Run Imputation → Save Results) with explicit validation in Step 1, a stop condition ("Stop if all genes are removed by this filter"), a complete error-code table with causes and solutions, and an explicit escalation feedback loop ("IF error persists, READ: references/troubleshooting.md") plus output-verification commands in Local Validation. This matches the level-5 anchor (explicit validation steps, feedback loops for error recovery). | 5 / 5 |
Progressive Disclosure | Good structure: a situation-to-file table routes to references/algorithm.md, scripts/main.R, references/troubleshooting.md, and references/cli-guide.md — all real files, verified one level deep (reference files do not cross-reference each other), and implementation scripts (functions.R, utils.R, etc.) are appropriately kept out of SKILL.md. Minor organization gaps keep it at level 4 rather than 5: the reference file table is duplicated verbatim at the bottom ("Reference Files" repeats "When to Read External Files"), and the inline error table substantially duplicates references/troubleshooting.md. | 4 / 5 |
Total | 17 / 20 Passed |