Content
73%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 exceptionally actionable, with executable commands, complete YAML, and well-sequenced workflows with real validation loops for the risky (fix) mode. Its two weaknesses are a missing bundle file — the repeatedly referenced references/zizmor.md does not exist, breaking the skill's core progressive-disclosure design — and systematic restatement of the same rules across sections.
Suggestions
Ship the missing references/zizmor.md (audit catalogue, triage decision tree, zizmor.yml schema notes, CI templates) or inline its essential content — the body instructs "Read references/zizmor.md before responding" and cites named sections of it, but no references/ directory exists in the bundle.
Collapse the three overlapping handoff tables (ownership boundary, Layer-1 wizard list, intent-classification table) into one authoritative routing table referenced from the other two spots.
State the GH_TOKEN/offline rule and the pinning-policy recommendation once each; each is currently repeated across 3-4 sections (Layer 2, scan mode, Never/Always lists, and config mode).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and free of explaining-known-concepts, but systematic redundancy could be tightened: the same handoff routing appears in three tables (ownership boundary, Layer-1 wizard, intent classification), and the token/offline rule ("zizmor reads GH_TOKEN, GITHUB_TOKEN or ZIZMOR_GITHUB_TOKEN from the environment itself") and the pinning-policy recommendation are each restated in 3-4 sections. The duplication goes beyond the 4 anchor's "minor instances that could be trimmed", fitting the 3 anchor's "could be tightened". | 3 / 5 |
Actionability | Fully copy-paste ready: exact CLI invocations ("zizmor --strict-collection --offline .", "uvx zizmor@1.29.0", the git dirty-tree guard one-liner), complete YAML for all three pinning-policy shapes and the pre-commit hook, and exact suppression syntax ("# zizmor: ignore[id] <reason>") and triage outcome mechanisms. Not 4: there are no gaps in executable coverage across all eight modes. | 5 / 5 |
Workflow Clarity | Clear mode dispatch and a three-layer wizard with explicit validation checkpoints and feedback loops for the file-editing (fix) mode: refuse on a dirty tree ("git status --porcelain | grep -q . && { echo 'commit or stash first'; exit 1; }"), show the diff, "run the audit again and report what remains", plus rollback ("git checkout -- ."). Validation is present for the in-place-editing operation, so the destructive/batch cap does not apply; not 4 because every checkpoint for every mode is explicit, including the SARIF exit-code trap. | 5 / 5 |
Progressive Disclosure | The body's load-bearing reference is dangling: it opens with "Read `references/zizmor.md` before responding" and cites named sections of it ("Triage decision tree", "Repository patterns", "Worked baseline config", "CI integration", the 41-audit catalogue), but no references/ directory exists anywhere in the bundle, so the catalogue, decision tree, and CI templates the skill depends on are unreachable. Combined with a ~390-line body, this matches the 2 anchor (minimal effective structure / referenced material not actually available) rather than the 3 anchor's "references present". | 2 / 5 |
Total | 15 / 20 Passed |