Content
61%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 highly actionable, well-sectioned body whose code is executable and unit-aware, but it functions as a monolith: it duplicates its own examples, lacks validation checkpoints in batch imaging workflows, and completely ignores the scripts/ bundle it ships with.
Suggestions
Replace the inlined implementations with short usage snippets and one-level-deep pointers to the existing scripts (e.g., "**ADC maps**: See [scripts/compute_adc.py](scripts/compute_adc.py) for the full CLI implementation"), cutting SKILL.md to an overview.
Add validation checkpoints to the workflows (e.g., visually verify the Otsu bone mask or ADC map against the source volume before reporting statistics, and re-check if the mask includes soft tissue).
Remove the duplicated cosinor example — keep it in either Quick Start or section 4, and have Workflow 3 reference that instead of repeating it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The prose is lean, but the body duplicates itself: the Quick Start re-implements the cosinor example that appears again in section 4 and a third time in Workflow 3, and Workflow 1 just re-invokes section 1's function. It matches the 3 anchor (mostly efficient but could be tightened); not 4 because the redundant example blocks are clearly trimmable. | 3 / 5 |
Actionability | Every capability ships as a complete, executable function with real defaults, docstrings, units, and printed outputs (e.g., compute_adc_map, bone_morphometry with voxel_size_um, cosinor_analysis with significance testing), plus an install command — copy-paste ready and covering the common cases. Only cosmetic warts (an unused idx loop variable) keep it from being flawless, which the 5 anchor does not penalize. | 5 / 5 |
Workflow Clarity | "Typical Workflows" are single unvalidated calls (load → analyze → print) with no verification of intermediate results (e.g., inspect the segmentation before reporting BV/TV or plaque counts). These are batch analyses over whole volumes/stacks, and the Troubleshooting section only partially compensates as an error-recovery loop, so the batch-operation cap of 3 applies. | 3 / 5 |
Progressive Disclosure | The bundle contains five standalone scripts (compute_adc.py, cosinor_analysis.py, etc.) that duplicate the body's code, yet the ~600-line SKILL.md never references any of them — all seven full implementations are inlined and the bundle is undiscoverable. This matches the 2 anchor (content that clearly belongs in separate files is inlined; references buried), not 3, because no reference to the bundle exists at all. | 2 / 5 |
Total | 13 / 20 Passed |