Content
52%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 well-structured with a useful decision tree, concrete commands, and a verification-backed redlining workflow, but its cross-file references are broken: it cites ooxml.md, docx-js.md, and ooxml/scripts/*.py that are absent from the bundle while ignoring the actual Document library in scripts/document.py. This undermines both actionability and progressive disclosure.
Suggestions
Fix the script paths and references: point to the actual bundle (scripts/unpack.py, scripts/pack.py, scripts/document.py) instead of the nonexistent ooxml/scripts/ and ooxml.md, or add those files to the bundle.
Inline a minimal Document-library usage example (e.g. get_node + doc.save() + add_comment from scripts/document.py) so the core editing flow is executable without relying on a missing reference file.
De-duplicate the batch-organization guidance (currently described in both step 2 and step 4) and consolidate the repeated "MANDATORY - READ ENTIRE FILE" instructions to tighten token use.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with a decision tree and terse commands, but batch-organization guidance is duplicated (step 2 "Batch organization" and step 4 "Suggested batch groupings") and the "MANDATORY - READ ENTIRE FILE" phrasing repeats three times — padding that could be tightened, matching the score-3 anchor. | 3 / 5 |
Actionability | Concrete commands (pandoc, soffice, pdftoppm) and a detailed BAD/GOOD redlining XML example are present, but the core editing API (get_node, doc.save(), add_comment) is deferred entirely to ooxml.md — which does not exist in the bundle — and the cited ooxml/scripts/*.py paths do not match the actual scripts/ bundle, leaving key execution details missing. | 3 / 5 |
Workflow Clarity | The redlining workflow has a clear numbered sequence with a final verification step (grep checks for applied/unintended changes) and batch-testing guidance, so verification is not missing; it sits just below score 5 because per-batch validation is described only as "test each batch" without a concrete checkpoint command. | 4 / 5 |
Progressive Disclosure | The body points to one-level-deep references (docx-js.md, ooxml.md), but scored against the actual bundle those referenced files do not exist, while the real bundle (scripts/document.py, scripts/utilities.py, scripts/templates/*.xml) is never referenced — navigation from SKILL.md to the detailed material is broken. | 2 / 5 |
Total | 12 / 20 Passed |