Content
56%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 richly detailed, well-sectioned design spec with a clear reproduction-first workflow, but its two structural weaknesses pull it down: the canonical example.html the entire protocol depends on is missing from the bundle, and significant content (the connector mapping table, the do/don't restatement of it) is inlined and duplicated where a single reference file would serve. Adding a closing validation checkpoint would complete the workflow.
Suggestions
Ship the referenced example.html (or inline the structural skeleton it defines) — Step 1 of the protocol is currently impossible to execute, which caps both actionability and progressive disclosure.
Merge the "Implementation constraints" do/don't table into the connector→UI mapping (it largely restates it) and move the 15-family mapping plus fallback heuristics into a references/ file linked one level deep from SKILL.md.
Add an explicit final verification step to the protocol, e.g. 'Step 4: Before delivering, diff your output against example.html section-by-section — section count, module order, connector list, and footer string must match exactly.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense (exact hex tokens, pixel sizes, fonts, section order), but there is real redundancy: the "Implementation constraints" do/don't table restates the connector→UI mapping table nearly verbatim (e.g. Sentry's "big red number + 7-day heatmap" appears twice), and the design-system policy is asserted three times ("⚠️ Design system policy", the bullet list, and "This is a hard constraint"). It could be tightened without losing any guidance — the anchor-3 fit — while it clearly exceeds anchor 2's padded explanatory style. | 3 / 5 |
Actionability | The spec level is very concrete (exact tokens, Cormorant/Inter stacks, section heights, per-family UI forms, fallback heuristics), but the protocol's Step 1 — "Open and read the shipped `example.html` in this folder … your job is to reproduce it" — is non-executable because example.html is not present in the bundle (no references/, scripts/, or assets/ directories exist). The central instruction hinges on a missing key artifact, which is the anchor-3 'missing key details' fit rather than anchor 4's 'minor gaps'. | 3 / 5 |
Workflow Clarity | The "Source-of-truth protocol" gives a clear numbered 3-step sequence (read example → mirror 1:1 with an explicit checklist of what must match → freshen data values only) followed by identity-guard and hard-constraint rules — a coherent, well-sequenced workflow. It falls short of anchor 5 because there is no final validation/check step (e.g. 'compare your output section-by-section against example.html before delivering'), leaving a minor checkpoint gap. | 4 / 5 |
Progressive Disclosure | Sections are well organized and the one external reference (example.html) is clearly signaled and prominent. However, scored against the actual bundle structure, that referenced path does not exist, and a large operational table (the 15-family connector→UI mapping) that would sit naturally in a reference file is fully inlined in a ~200-line single file. That is anchor 3's 'content that should be separate is inline / references present but not resolvable' rather than anchor 4's 'minor organization gaps'. | 3 / 5 |
Total | 13 / 20 Passed |