Content
50%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 contains genuinely useful, executable usage and example content for tree styling, but it is buried under large amounts of generic boilerplate, contradictory/time-sensitive metadata, and vague reference signaling. Structure and token efficiency are the main weaknesses.
Suggestions
Remove the generic boilerplate sections (Risk Assessment, Security Checklist, Evaluation Criteria, Lifecycle Status, Response Template, Output Contract, Validation and Safety Rules) that are not specific to phylogenetic tree styling.
Resolve the contradiction between Dependencies (ete3, matplotlib, numpy, pandas) and Prerequisites ("No additional Python packages required"), and drop the time-sensitive "Next Review Date".
Replace generic "references/" mentions with an explicit link to references/runtime_checklist.md, and fix the inverted "See `## Features` above" self-references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is padded with many generic boilerplate sections (Risk Assessment, Security Checklist, Evaluation Criteria, Lifecycle Status, Output Requirements, Response Template, Output Contract, Validation and Safety Rules) that teach nothing tree-specific, and includes a time-sensitive date ("Next Review Date: 2026-03-06") and a contradictory Prerequisites section, so it is noticeably verbose with several unnecessary sections. | 2 / 5 |
Actionability | It provides concrete executable invocations with real flags (e.g. `python3 scripts/main.py -i tree.nwk -o tree_bootstrap.png --show-bootstrap --bootstrap-threshold 70`), a full parameter table, taxonomy CSV format, and a Newick example, with only minor gaps such as the off-topic medical sample string in an example invocation. | 4 / 5 |
Workflow Clarity | A 5-step Workflow and a py_compile Quick Check are present, but the steps are generic process boilerplate with only a parse-check validation and no output validation checkpoint or validate-fix-retry feedback loop for the actual styling task. | 3 / 5 |
Progressive Disclosure | Bundle files exist (references/runtime_checklist.md, scripts/main.py) but the body only gestures at "references/" generically and never links the specific file, while nearly all content is inlined in one 275-line SKILL.md alongside confusing self-references like "See `## Features` above". | 3 / 5 |
Total | 12 / 20 Passed |