Content
86%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 high-quality, expert-level skill body: dense with non-obvious footguns, fully executable commands, a validated editing workflow, and appropriate offloading of complexity into real bundled scripts. The only dents are a slightly discursive tracked-changes failure-mode discussion and an implicit rather than explicit error-recovery loop.
Suggestions
Tighten the tracked-changes section: the pandoc-vs-accept_changes.py spacer-paragraph exception can be condensed (or moved to a reference) without losing the core gotcha that emptied paragraphs read as stray bullets.
Make the validation feedback loop explicit — after `validate.py out.docx --original doc.docx`, add 'If errors are reported, fix word/document.xml and re-run validate before delivering,' mirroring the anchor-5 validate→fix→retry pattern.
Sharpen the verify-output step by naming what to inspect (e.g., broken table shading, missing TOC entries, stray empty bullets from paragraph deletions) so the render-and-look check is actionable rather than open-ended.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and information-dense throughout — the docx-js footguns and OOXML schema ordering are non-obvious knowledge Claude lacks, so the tokens earn their place. A few asides (the pandoc-vs-accept_changes spacer-paragraph exception) could be trimmed without losing the core gotcha, keeping it just shy of a 5. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout — `python scripts/office/soffice.py --headless --convert-to pdf`, `unzip -q ... -d unpacked/`, `find unpacked -type l -delete`, exact DXA dimensions (12240/15840), and concrete comment.py invocations cover the common cases. | 5 / 5 |
Workflow Clarity | The editing workflow is a clear sequenced procedure (unzip → strip symlinks → merge_runs → edit XML → repack → validate) with a validation checkpoint (`validate.py ... --original`) and a render-and-look verification step, satisfying the destructive-operation cap. The error-recovery loop (fix-and-re-validate) is implicit rather than spelled out, which is the minor gap below a 5. | 4 / 5 |
Progressive Disclosure | SKILL.md is a focused overview of gotchas and usage; the heavy logic lives one level deep in real, correctly-referenced scripts (merge_runs.py, validate.py, accept_changes.py, comment.py, soffice.py — all present in ./scripts/). Sections are well-organized and navigation is easy. | 5 / 5 |
Total | 18 / 20 Passed |