Content
48%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 dominated by generic template scaffolding (risk tables, security checklists, lifecycle status, self-referential cross-references) that crowds out the small amount of domain content (use cases, parameters, returns). The runnable commands are real but documented parameters contradict the packaged script's actual interface, and no reference files exist despite one being cited.
Suggestions
Cut the duplicated sections: the description appears twice verbatim, Audit-Ready Commands repeats Quick Check and Example Usage, and 'See ## X above' placeholders in Dependencies/Implementation Details should be replaced by the single authoritative section.
Align documentation with the actual script: document --ab1/--reference/--demo as they exist in scripts/main.py, remove the nonexistent CONFIG block reference, and add one real usage example (e.g. python scripts/main.py --ab1 sample.ab1 --reference ATCG...) once the script supports AB1 parsing.
Delete or drastically compress the generic Risk Assessment, Security Checklist, Evaluation Criteria, and Lifecycle boilerplate — none of it is skill-specific knowledge — and either ship a real requirements.txt or drop the reference to it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body duplicates itself repeatedly: the frontmatter description is pasted verbatim twice ("Use this skill when the task needs Use sanger chromatogram qa for..." under When to Use, and again under Key Features); "Audit-Ready Commands" repeats the exact commands already in Quick Check and Example Usage; "Dependencies" and "Implementation Details" are placeholder cross-references ("See ## Prerequisites above", "See ## Workflow above"); and the Risk Assessment, Security Checklist, Evaluation Criteria, and Lifecycle sections are generic template boilerplate. This is 'noticeably verbose; several unnecessary explanations or padded sections' (2) — not 1, since it never explains concepts Claude already knows at length, but well below the 'mostly efficient' (3) anchor. | 2 / 5 |
Actionability | There are concrete, runnable commands ("python -m py_compile scripts/main.py", "python scripts/main.py --help") and scripts/main.py exists, but key details are wrong: the documented parameters ("ab1_file", "expected_seq", "variant_pos") do not match the script's actual CLI (--ab1, --reference, --demo); "Edit the in-file CONFIG block" points to a block that does not exist in main.py; and the example cd path ("20260318/scientific-skills/Data Analytics/sanger-chromatogram-qa") is a workspace-specific artifact. The mismatches go beyond anchor 4's 'minor gaps', matching anchor 3's 'concrete guidance but incomplete... missing key details'. | 3 / 5 |
Workflow Clarity | The Workflow section gives a clear 5-step sequence with an explicit validation checkpoint ("Use this command to verify that the packaged script entry point can be parsed before deeper execution") and an error-recovery path ("If scripts/main.py fails, report the failure point, summarize what still can be completed safely, and provide a manual fallback"). This meets anchor 4 ('clear sequence with most checkpoints present; minor validation gaps') but not 5, because the workflow is generic process scaffolding (confirm inputs, run script, return structured result) rather than domain-specific steps with a validate-fix-retry loop over the chromatogram analysis itself. | 4 / 5 |
Progressive Disclosure | The actual bundle is scripts/main.py only (no references/ or assets/ directories); the body's reference to scripts/main.py is accurate, but "Declared in requirements.txt" points at a file that does not exist in the bundle. All content lives inline in a ~190-line SKILL.md whose bulk is template boilerplate that serves no navigational purpose — 'content that should be separate is inline' and a dangling reference match anchor 3 rather than the 'good structure; references mostly clear' of anchor 4. | 3 / 5 |
Total | 12 / 20 Passed |