Content
65%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 a model of conciseness with a precise output contract and genuinely useful gotchas, but it is hollowed out by two dangling references: the entire generation procedure lives in style-guide.md and template.md, neither of which is present in the skill's bundle. As shipped, the skill tells Claude exactly where to put the file but not how to build it.
Suggestions
Ship the referenced bundle: create references/style-guide.md and references/template.md (or inline a minimal HTML skeleton in SKILL.md) so the 'How to Generate' step is executable.
Inline the critical values the gotchas depend on — the current html2canvas SRI hash and the toBlob null-check pattern — instead of deferring them to the missing template.md.
Add a verification checkpoint after generation, e.g. screenshot the rendered HTML with Playwright (using an absolute path) and confirm the diagram is not truncated before reporting completion.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence: no space is spent explaining what HTML, SRI, or Playwright are — every section (Output, Required Sections, Section Selection, Gotchas) delivers non-obvious, project-specific information. It matches the anchor 'Lean and efficient; assumes Claude's intelligence; every token earns its place'; anchor 4 would require some over-explanation that could be trimmed, and none is evident. | 5 / 5 |
Actionability | The output specification is fully concrete ("{MAIN_REPO_ROOT}/diagrams/opik-{TICKET_NUMBER}-diagram.html" with git rev-parse resolution, copy-as-image button, max-4-sections rule), but the core executable content is delegated to "Follow the style guide in style-guide.md and use the HTML template in template.md" — and neither file exists in the bundle. The template, the SRI hash, and the toBlob copy script are all unavailable, so the skill cannot actually be executed from what ships. This lands at anchor 3 ('Some concrete guidance but incomplete; missing key details') — anchor 4 requires mostly executable guidance, which the dangling references break. | 3 / 5 |
Workflow Clarity | The flow (pick sections by change type -> follow style guide/template -> emit file at fixed path) is sequenced and the Section Selection mapping is a useful decision guide, but the critical step 'How to Generate' points to the missing files, and there is no validation checkpoint (e.g. open the HTML, screenshot it, verify the diagram is not truncated). This matches anchor 3 ('Steps listed but validation gaps; sequence present but checkpoints missing or implicit') rather than 4, whose sequence must be executable end-to-end with at most minor gaps. | 3 / 5 |
Progressive Disclosure | The in-file structure is good — a dedicated 'Reference Files' section with one-line descriptions of each reference, links one level deep, no nested references. However, per the actual bundle listing, style-guide.md and template.md do not exist in references/ (the directory is absent), so navigation to the detailed material fails. A well-signaled pointer to a nonexistent file is a broken structure, not the anchor-5 'Clear overview with well-signaled one-level-deep references'; anchor 4's 'minor organization gaps' understates a fully missing bundle, so it sits at 3. | 3 / 5 |
Total | 14 / 20 Passed |