Content
71%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 well-designed router skill with an explicit intent classifier, enforced step ordering, and a copy-paste-ready project probe. Its main weaknesses are two long flag sections that duplicate content belonging in their referenced files, verification phases that name checks without giving commands, and numerous body references to files that are absent from the provided bundle.
Suggestions
Compress the "Migration flag" and "Styling-depth flag" sections to 1-2 sentences each plus the file link, moving the detailed protocol/trap enumerations into sendbird-migration.md and design-matching.md where they belong.
Make the A4/B4 verification phases concrete: give the exact commands or checks to run (e.g., the Gradle sync command, how to confirm ChatClient lifetime) and what to do on failure.
Ensure every referenced path in the body (RULES.md, builder.md, sdk.md, credentials.md, ai-integration.md, design-matching.md, sendbird-migration.md, integration-review.md, ../stream/ai-backend-contract.md) actually exists in the bundle, or note which live in a sibling skill so navigation never hits a missing file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The routing tables, ordering rules, and probe command are lean and assume Claude's competence, but the "Migration flag" and "Styling-depth flag" paragraphs (~550 words combined) restate detail that belongs in sendbird-migration.md and design-matching.md (e.g., the §3.5 gap-handling protocol, composite-slot traps, region checklists). This fits "mostly efficient but includes some unnecessary explanation or could be tightened"; not 2 because the padding is confined to two sections rather than being pervasive. | 3 / 5 |
Actionability | For a router-style instruction skill, the guidance is fully executable: a copy-paste-ready bash probe command, an exact decision table mapping input signals to tracks, explicit once-per-session and skip rules, a verbatim disambiguation question, and a concrete recovery rule ("If a track A/B/D task reaches code work and credentials haven't been collected yet, return to credentials.md"). Specific direction covers the common cases. | 5 / 5 |
Workflow Clarity | The sequence is explicit and enforced (Step 0 classifier -> Step 0.5 credentials -> project signals -> track phases A1-A4/B1-B4) with named verification checkpoints ("Confirm Gradle sync, ChatClient lifetime, auth, and first rendered screen") and a build/compare/iterate loop in the styling flag. Not 5 because the verify phases describe what to confirm without any concrete check command or failure-recovery guidance, leaving validation somewhat implicit. | 4 / 5 |
Progressive Disclosure | The router pattern is good in principle - one-level-deep, clearly signaled links to references/ files that all exist (CHAT-COMPOSE.md, DOCS.md, etc.) with a documented naming scheme - but scoring against the actual bundle, eight-plus referenced paths (RULES.md, builder.md, sdk.md, credentials.md, ai-integration.md, design-matching.md, sendbird-migration.md, integration-review.md, ../stream/ai-backend-contract.md) are not present, and substantial content that belongs in those files is inlined in the migration/styling flag sections. Not 4 because dangling referenced paths plus inlined detail exceed "minor organization gaps". | 3 / 5 |
Total | 15 / 20 Passed |