Content
92%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 an excellent repo-workguide: lean, symbol-accurate, with a prerequisites gate, an explicit file checklist, a component contract, a state machine, and a verification section. Its one real defect is structural: the single external reference (reference.md, holding the full template) is dangling — the file is absent from the bundle, so the progressive-disclosure path breaks and the agent must reconstruct the template from the closest sibling instead.
Suggestions
Add the missing reference.md (the full <Channel>SetupGuide template the body promises) to the bundle, or point the link at an existing file — right now the skill's only progressive-disclosure path dead-ends.
If reference.md is not added, inline a minimal component skeleton in the 'File checklist' section so the 'copy the closest sibling' instruction has a fallback the agent can execute without the missing template.
The verify step could name the concrete type-check command (e.g. the tsc/diagnostic invocation to run) instead of relying on 'Cursor diagnostics', which assumes a particular editor.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and lean: a compact ASCII component tree, a symbol table instead of prose, a file checklist, and a 3-step recipe. There is no explanation of concepts Claude already knows (no 'what is a stepper' or 'how React state works') and every section delivers repo-specific facts (symbols, query keys, prop contracts, conventions). It matches the 'lean and efficient; every token earns its place' anchor rather than the minor-over-explanation anchor (4), which would require some trimmable passage — none stands out. | 5 / 5 |
Actionability | Guidance is fully concrete for an instruction-only skill: exact paths ('apps/dashboard/src/components/agents/<channel>-setup-guide.tsx'), the exact props contract, the precise integration-resolution predicate ('_id === integrationId && providerId === ChatProviderIdEnum.<Channel>'), named utilities with signatures (deriveStepStatus, hasIntegrationCredentials), and the query key to invalidate. It never describes without instructing, so it sits at the top anchor rather than the 'minor gaps' anchor (4). | 5 / 5 |
Workflow Clarity | The sequence is explicit and gated: 'Step 0 — Prerequisites' (ChatProviderIdEnum exists, integrationId available), a CREATE/EDIT file checklist, a numbered 6-point state machine, a 3-step recipe, and a 'Build & verify' section with concrete checkpoints (steps advance on credential save, drawer opens/saves, 'Listening…' flips to 'Connected' with confetti). Checklists and validation are both present, matching the top anchor; this is not a destructive or batch operation, so the validation cap does not apply. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned (~90 lines) and correctly pushes the full template out to a clearly signaled one-level reference — 'Full template: see [reference.md](reference.md)' — but that referenced file does not exist in the bundle (no references/, scripts/, or assets/ directories, and no reference.md anywhere). Per the guideline to score against the actual bundle structure, the disclosure chain is broken at its single link: an agent following it dead-ends. This lands at 'references present but the organization cannot fully serve navigation' (3) rather than 'minor organization gaps' (4), because the gap is a missing file, not a placement choice — and above 2 because the inline content is otherwise appropriately structured. | 3 / 5 |
Total | 18 / 20 Passed |