Content
62%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 skill body delivers a genuinely rigorous, well-sequenced audit workflow with strong validation and concrete templates, but it is bloated: the two optional-flag specifications and their invariants are repeated across at least five sections each, and material that belongs in separate reference files is inlined, producing a 515-line monolith.
Suggestions
State each invariant once (e.g., 'uncited entries are detect-only and never alter the verdict' and 'no .bib mutation under --soft-only') in Key Rules, and have other sections reference it instead of restating it — this alone would remove ~100 lines.
Move the full soft-only mode spec (translation table, augmented JSON/MD schemas, guarantees) and the uncited-detection spec into separate reference files (e.g., references/soft-only-mode.md, references/uncited-detection.md), keeping only a short summary and trigger flags in SKILL.md.
Define the undefined `AUTO_APPLY` constant alongside the other Constants (including how it is set) and either document the `--apply` flag in the Workflow/Output Contract consistently or remove it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 515-line body restates the same invariants many times — "uncited entries never alter the top-level verdict / are not blocking" appears in Step 2, Step 5, 'Why opt-in', 'Effect when enabled', Key Rules, and Submission Artifact Emission, and the --soft-only no-bib-mutation invariant is repeated across Constants, Key Rules, and a ~95-line dedicated section. That is noticeably verbose even though most content is accurate operational contract rather than concepts Claude already knows. | 2 / 5 |
Actionability | Concrete, mostly executable guidance throughout: a copy-paste-ready mcp__codex__codex prompt template with model/config/sandbox, executable `latexmk -C && latexmk -pdf ...` command, full JSON artifact schemas, and an explicit verdict decision table. Minor gaps: `AUTO_APPLY` is referenced in Step 6 but never defined in Constants, `--apply` appears only in the Output Contract, and cite-context extraction (Step 2) has no concrete command. | 4 / 5 |
Workflow Clarity | Steps 1-7 are clearly sequenced with explicit validation checkpoints: per-entry reviewer verdicts (KEEP/FIX/REPLACE/REMOVE), an aggregation step with a decision table, human-approval gating for destructive REPLACE/REMOVE changes, a recompile-and-check-warnings verification step, and defined error-recovery fallbacks (bib enumeration failure → 'unavailable', unreadable bib → BLOCKED, reviewer failure → ERROR). | 5 / 5 |
Progressive Disclosure | References to shared files (reviewer-backends.md, review-tracing.md, assurance-contract.md, citation-discipline.md, etc.) are one level deep and clearly signaled, but none of them exist in this skill's bundle (no references/, scripts/, or assets/ directories, and ../shared-references/ does not resolve). Additionally, large self-contained feature specs — the ~95-line soft-only mode section and the ~60-line uncited-detection section — are inlined in SKILL.md where they clearly belong in separate reference files. | 3 / 5 |
Total | 14 / 20 Passed |