Content
70%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 strongly actionable, well-sequenced workflow with genuine validation and feedback loops, undermined by monolithic structure and verbosity. Roughly half the file is conditionally-relevant reference material inlined into the always-loaded body, plus injected-looking noise lines and repeated references to "Codex" that don't belong in this skill.
Suggestions
Move the KaTeX math guide, CVPR/ICLR style guide, and architecture-beta layout tips into references/ files (e.g. references/math.md, references/academic-style.md), keeping SKILL.md to the workflow plus one-line pointers loaded per diagram type.
Delete the noise lines ("ASCII alias for automated checks: ...") and replace all "Codex" references with agent-neutral wording; also remove or compress the 23-row diagram-type table, which Claude already knows.
Fix the dangling instructions: either add the promised "Mermaid syntax reference" as a reference file or point "fetch the up-to-date syntax reference" at a concrete source.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core workflow is operational, but the body carries substantial padding Claude does not need: a 23-row diagram-type table, a full KaTeX pattern table, extended academic style/color guidance, and noise lines like "ASCII alias for automated checks: score >= 9." It sits at "mostly efficient but could be tightened" — not 2, because the workflow steps themselves contain little over-explanation of things Claude already knows. | 3 / 5 |
Actionability | Concrete, executable guidance throughout: `mkdir -p figures`, the `mmdc` verification command with an `npx` fallback, a copy-ready review checklist template, kebab-case naming examples, and junction-routing code. Minor gaps keep it below 5: "Read the corresponding Mermaid syntax reference below" points at no actual syntax reference, and "fetch the up-to-date syntax reference" names no source. | 4 / 5 |
Workflow Clarity | Steps 0-6 are explicitly sequenced with a mandatory `mmdc` syntax validation checkpoint, a fix-and-retry loop bounded by MAX_ITERATIONS, a full strict verification checklist, and a defined ACCEPT/FIX verdict gate — exactly the top anchor (explicit validation, feedback loops, checklists). The stray "Codex" naming is an inconsistency, not a sequencing or validation gap. | 5 / 5 |
Progressive Disclosure | No bundle files exist; everything lives inline in one ~380-line SKILL.md, including ~200 lines of conditionally-relevant material (KaTeX math guide, CVPR/ICLR style guide, architecture-beta tips) that belongs in references/ loaded only when needed. Sections are well-headed (above the headerless-inline anchor at 2), but content that clearly should be separate is inline, holding it at 3. | 3 / 5 |
Total | 15 / 20 Passed |