Content
88%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 strong, highly actionable checklist skill: concrete file paths, real failure signatures, explicit validation steps, and a required output report. The main weaknesses are minor redundancy between the Guardrails section and the checklist, and a ~130-line body that could offload some sections to reference files for better progressive disclosure.
Suggestions
Trim the Guardrails section — its four bullets restate checklist items 1, 5, and 6; keep only the Zod/TS and centralize-vs-duplicate bullets that add new information.
Move 'SDK Upgrade Hygiene' and 'Common Failure Modes' into a reference file (e.g. references/upgrade-hygiene.md) and keep one-line summaries with clear links in SKILL.md, bringing the always-loaded body closer to a lean overview.
Convert inline doc path mentions (e.g. 'docs/developer/adding-a-provider.md') into clearly-signaled links under a 'References' heading so navigation to deeper material is unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and project-specific — checklists, a file table, concrete failure signatures — with almost no explanation of concepts Claude already knows. However, the 'Guardrails' section largely restates checklist items ('Do not ship a fix in one adapter without verifying the sibling', 'Do not verify env changes with probes only' repeat sections 1 and 6), which is minor padding that could be trimmed. It sits between the level 5 'every token earns its place' anchor and level 4 'minor instances of over-explanation', closer to 4 due to this repetition. | 4 / 5 |
Actionability | Guidance is fully executable for an instruction-only skill: exact file paths ('electron/main/ipc/schemas.ts', 'src/lib/providers/schemas.ts'), a concrete command ('bun run typecheck'), specific grep targets, exact failure output ('env: node: No such file or directory'), and precise verification surfaces (GUI launch, Settings → Providers). Matches the 'copy-paste ready ... specific examples cover the common cases' anchor; level 4's 'minor gaps' does not fit. | 5 / 5 |
Workflow Clarity | The process is clearly sequenced: a seven-section symmetry checklist ordered by concern, followed by a verification section with explicit validation steps (typecheck always, real turns with each provider, GUI launch for env changes, probe-surface check for tooling-status changes) and a required output report. This matches the 'clear sequence with explicit validation steps; checklists for complex processes' anchor and exceeds level 4. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), so the ~130-line body must stand alone; it is well organized with clear section headers, a file table, and inline pointers to repo docs ('docs/developer/adding-a-provider.md', 'docs/providers/codex-upgrade-checklist.md'). This is 'good structure; most content appropriately placed' (level 4) rather than 5: at this length, sections like 'SDK upgrade hygiene' and 'Common Failure Modes' could be split into reference files to slim the always-loaded body, and the doc pointers are inline code paths rather than clearly-signaled references. | 4 / 5 |
Total | 18 / 20 Passed |