Content
63%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 delivers strong domain-specific reasoning, thresholds, and a well-sequenced 8-phase workflow with QC validation gates, but it is undermined by a broken reference layer: all five files in 'See Also' and the .gmt bundle are missing from the bundle, and the Quick Start relies on helper functions defined nowhere. Fixing the bundle references and making the entry-point code executable would lift actionability and progressive disclosure substantially.
Suggestions
Add the missing referenced files (ANALYSIS_DETAILS.md, USE_CASES.md, EXAMPLES.md, QUICK_START.md, FALLBACK_PATCH.md and the .gmt bundle) to the bundle, or remove the 'See Also' section and inline the minimal needed content — currently every detail-level reference is dangling.
Make the Quick Start executable: define or import the helper functions (load_sgrna_counts, filter_low_count_sgrnas, normalize_counts, calculate_lfc, mageck_gene_scoring, enrich_essential_genes, prioritize_drug_targets, generate_crispr_report) in a bundled script, or point to where they live.
Trim the redundant 'Overview' section, whose bullet list duplicates the 'Core Workflow' phase headings, and drop the generic opening paragraph after RULE ZERO.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is high-value, non-obvious domain content (QC thresholds, evidence-grading criteria, expected-aggregation-level gotchas like 'the expected GT is almost always the sgRNA-level Spearman'). Minor trimming is possible: the 'Overview' bullet list duplicates the 'Core Workflow' phases, and the opening 'Comprehensive skill for analyzing...' paragraph is filler — 'efficient; minor instances of over-explanation that could be trimmed', not the noticeably padded level 3. | 4 / 5 |
Actionability | Concrete guidance exists (tool names with parameters, exact tu run commands, thresholds like 'LFC < -0.5' and 'recovery_rate >~0.8'), but the Quick Start code calls helper functions — load_sgrna_counts, filter_low_count_sgrnas, mageck_gene_scoring, prioritize_drug_targets — that are defined nowhere in the bundle (the ANALYSIS_DETAILS.md that would hold them is missing), making it effectively pseudocode. This matches anchor 3: 'some concrete guidance but incomplete; pseudocode instead of executable code'. | 3 / 5 |
Workflow Clarity | The 8-phase workflow is clearly numbered and sequenced, and RULE ZERO plus QC gates (core-essential recovery check, 'their absence from the hit list indicates a QC problem') provide real validation checkpoints — above anchor 3. It falls short of anchor 5 because recovery actions after a failed check are implied rather than given as explicit validate→fix→retry loops, and the fallback-pattern file referenced for API failures is absent. | 4 / 5 |
Progressive Disclosure | Section structure is good and the scripts/ bundle files referenced in the body (reference_gene_sets.py, CEGv2/NEGv1 sets) are real, but the entire 'See Also' layer — ANALYSIS_DETAILS.md, USE_CASES.md, EXAMPLES.md, QUICK_START.md, FALLBACK_PATCH.md — plus the mentioned '.gmt' bundle do not exist in the bundle. The primary detail references are clearly signaled but broken, so navigation does not hold up; this is anchor 3 ('some structure but could be better organized; references present but not clearly signaled') rather than 4's 'minor organization gaps', since 5 of the referenced files are missing. | 3 / 5 |
Total | 14 / 20 Passed |