Content
85%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.
An exceptionally rigorous and executable skill body: pinned environment, exact commands, explicit validation with rejection cases and error recovery, and a clean one-level reference structure verified against the actual bundle. Its one weakness is token efficiency — several provenance/testimony paragraphs (Zarr pin history, fixture test details, platform notes) read as evidence for human reviewers rather than instructions for the executing agent.
Suggestions
Move the extended Zarr-pin backstory (the codec_registry failure, macOS ARM64 run, and 'not claims about latest releases' caveat) into references/upstream-review.md, keeping only one inline sentence: 'Keep both Zarr pins even for HDF5-only conversions; see upstream-review.md.'
Trim the real-library test paragraph to the facts that change execution decisions (rejection cases, the TIFF (time,y,x) → NWB (time,x,y) mapping, and the retained Inspector orientation finding), and drop fixture tallies like 'eight non-square uint16 images' and 'four position samples'.
Convert the multi-sentence output/provenance notes in the Execute section into short bullets so the refusal behaviors (no overwrite of existing NWB/report, fresh path after failure) stand out for the executing agent.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body never explains concepts Claude already knows and is dense with domain-specific constraints, but it carries several testimonial passages that could be tightened or moved: the five-sentence Zarr-pin backstory ("the tested unconstrained Zarr 3.4.0 installation failed on `zarr.codec_registry`... ran the real conversion... on macOS ARM64... These are compatibility pins, not claims that..."), the fixture-level test description ("converts eight non-square uint16 images with irregular frame timing plus four position samples... recovers a known 1000-ppm clock drift"), and the platform note. This is more than the 'minor instances' of anchor 4, so anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened') is the best fit; not anchor 2 because none of it is generic padding — it is all skill-specific. | 3 / 5 |
Actionability | Guidance is fully executable: a copy-paste install command with exact pins, a copy-paste execution command (`nwb-env/bin/python scripts/convert_session.py /path/to/session.json --output /path/to/session.nwb`), a concrete config path (copy `assets/session-template.json`, replace synthetic values, contract in `references/input-contract.md`), and precise input requirements (finite strictly increasing timestamps, matching image/timestamp counts, one channel/plane declared). The common case is covered end-to-end. | 5 / 5 |
Workflow Clarity | The six-step workflow is clearly sequenced (inventory → config → timebase → convert → validate → deliver) with explicit validation checkpoints: step 5 reads `.validation.json`, the script "exits with an error for schema failures", rejection cases are enumerated (duplicate timestamps, mismatched frame counts, absent clock evidence, nonlinear pulse disagreement, missing timezone), and error recovery is specified ("retain the error and use a fresh output path after correcting the cause"). Feedback loops for this batch/write operation are explicit, matching anchor 5; anchor 4's 'minor validation gaps' does not apply. | 5 / 5 |
Progressive Disclosure | The body is a well-organized overview (supported-streams table, install, workflow, execute, references) that delegates details one level deep with clearly signaled links: input contract and pulse-pair variant to `references/input-contract.md`, pin rationale and interface checks to `references/upstream-review.md`, config shape to `assets/session-template.json`, execution to `scripts/convert_session.py`. All referenced bundle files exist and none nest further. Navigation is easy and the split is appropriate. | 5 / 5 |
Total | 18 / 20 Passed |