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.
The body is an exceptionally disciplined, domain-expert workflow: executable commands, precise formats, explicit validation checkpoints, and hard-won caveats that are not inferable. Its only real costs are density in the main file — fixture-test evidence and deep flag-semantics detail that would sit better in a reference — keeping conciseness and progressive disclosure just short of the top anchors.
Suggestions
Move the synthetic-fixture test-evidence paragraph ("The FASTQ command structure was exercised with a synthetic two-guide library... not real-screen statistical calibration") into references/runtime.md or a short references/validation.md, leaving one summary sentence in SKILL.md.
Move the deep report.json flag-semantics detail (the --gene-test-fdr-threshold vs --fdr distinction, RRA null changes, and FDR-family caveat) into a reference file, keeping only the report.json field list and the key behavioral note inline.
Tighten step 5 by consolidating the numeric fallback conditions (zero median / 45% zero guides, control-guide subset evaluation) into runtime.md alongside the other source-review findings, keeping the decision guidance (median vs control normalization) in the main file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious, empirically derived caveats Claude would not know ("Numeric sample names are unsafe", "The helper uses a raw-count population Gini; MAGeCK's native count-summary Gini uses log(count + 1)", "MAGeCK 0.5.9.5 switches median normalization to total-count scaling for a zero median or more than 45% zero guides") and explains no basic concepts, matching 'Efficient; minor instances of over-explanation that could be trimmed'. Not 5: the synthetic-fixture evidence paragraph ("counts of 30 and 12 were recovered exactly... recovered 15, 7, and 0 reads") and parts of the report.json flag-semantics paragraph are provenance detail that could be trimmed or moved; not 3 because essentially all content is skill-specific knowledge rather than padding. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands: "mageck count -l library.tsv --fastq c1.fastq.gz ... --sample-label c1,c2,t1,t2 --trim-5 0 --norm-method none -n counts" and "python scripts/screen_analysis.py qc --counts counts.count.txt --library library.tsv --control c1 c2 --treatment t1 t2 --output qc.json", with exact input formats ("headerless TSV library containing guide ID, DNA sequence, gene"; header beginning "sgRNA, Gene"), exact flags and thresholds. Matches the anchor 'Fully executable; copy-paste ready code or commands; specific examples cover the common cases'. | 5 / 5 |
Workflow Clarity | Seven clearly sequenced steps (design establishment → library validation → counting → QC → normalization → test → interpretation) with explicit validation checkpoints: "Verify both mageck --version and mageck test --help before an analysis", "Run QC before statistical testing", "Check the report's applied method, size factors, and warnings", and helper-enforced rejections (mismatched IDs, ambiguous sequences, reused results directories) with recovery guidance ("rename them before analysis", "resolve intentional multi-target guides explicitly upstream"). Matches the anchor 'Clear sequence with explicit validation steps; feedback loops for error recovery' — no validation cap applies. | 5 / 5 |
Progressive Disclosure | Structure is good: installation/runtime detail and multi-factor MLE design are split into real, clearly signaled one-level-deep files ("The tested source installation and external-runtime caveat are in [references/runtime.md](references/runtime.md)", "For a multi-factor design, see [references/design.md](references/design.md)") that exist in the bundle, and scripts/screen_analysis.py is present as referenced; neither reference nests further local files. Matches 'Good structure; most content is appropriately placed; references mostly clear; minor organization gaps'. Not 5: the main body still inlines substantial detail that could live one level down (the test-fixture evidence paragraph, the deep report.json flag-semantics and RRA-null discussion); not 3 because the core overview plus targeted offload pattern is clearly present and navigable. | 4 / 5 |
Total | 18 / 20 Passed |