Content
14%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill reads as a dumped API reference rather than an actionable diagnostic guide. It is excessively verbose, listing every method and property without prioritization or workflow structure. It lacks a clear troubleshooting sequence with validation checkpoints, and all content is crammed into a single file with no progressive disclosure.
Suggestions
Add a clear diagnostic workflow at the top: 'Run /doctor quick → identify failing components → run /doctor <component> → apply fix from Common Issues → re-run to verify', with explicit validation steps.
Move the full TypeScript API reference to a separate REFERENCE.md and keep only a quick-start example (createDoctorService + runDiagnostics) in the main skill.
Remove redundant console.log examples for each check method — a single pattern showing how to interpret check results is sufficient; Claude can generalize to other methods.
Cut the Best Practices section entirely (generic advice Claude already knows) and trim the status/result tables to a single concise reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The content is extremely verbose at ~200+ lines, serving as a full API reference document. It exhaustively lists every method, every property, every check result with console.log examples that are largely redundant. Much of this (status levels, check result meanings, best practices like 'run regularly') is information Claude already knows or could infer. | 1 / 3 |
Actionability | The code examples are concrete and appear executable with specific imports and method calls. However, they depend on a 'clodds/doctor' package whose existence and API are unverifiable, and many examples are just console.log wrappers around returned properties rather than demonstrating meaningful usage patterns or decision-making. | 2 / 3 |
Workflow Clarity | There is no clear diagnostic workflow or troubleshooting sequence. The content lists individual checks and common issues but never sequences them into a coherent troubleshooting process (e.g., 'run quick check first, then drill into failing components, then apply fixes and re-verify'). No validation or feedback loops are present for a domain where they matter. | 1 / 3 |
Progressive Disclosure | This is a monolithic wall of content with no references to external files. The full API reference, CLI commands, common issues, and best practices are all inlined. The API reference section alone could be a separate REFERENCE.md, and common issues could be a TROUBLESHOOTING.md. | 1 / 3 |
Total | 5 / 12 Passed |