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.
The body is a well-engineered, highly actionable seven-phase workflow with explicit validation and error-recovery loops, naming exact MCP tools and parameters. Its main weaknesses are minor verbosity in the principles rationale and the absence of any progressive disclosure into reference files for a skill of this length.
Suggestions
Tighten the Core Principles and Troubleshooting prose to the essential rule plus one concrete example, trimming rationale that does not change behavior.
Consider extracting the detailed convention-inference bullets (Phase 2 step 4) or the Troubleshooting section into a references/ file, keeping SKILL.md a concise overview with a one-level-deep pointer.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense with skill-specific actionable specifics and avoids explaining generic concepts Claude already knows, but the Core Principles section and a few troubleshooting entries carry rationale prose that could be trimmed. It sits above the 'mostly efficient' anchor (3) but short of fully lean (5). | 4 / 5 |
Actionability | Names exact MCP tools and parameters (get_amplitude_context, get_events paginated at 500/page, get_properties with propertyType, create_branch/create_events/create_properties, approvalWF: "Required"), a git diff command, a spec-table template, and explicit judgment criteria — fully actionable for an instruction-only skill. | 5 / 5 |
Workflow Clarity | Seven clearly sequenced phases with explicit validation checkpoints (stop if no user-facing change, stop if no SDK, run typecheck/lint/tests and fix), error-recovery feedback loops (403 → report-only fallback), and a spec-table checklist for a complex process. Validation is present, so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | Well-organized into Core Principles, seven phase sections, and Troubleshooting with clear headers, but it is a single ~118-line file with no bundle files or one-level-deep references; the over-50-line simple-skill exception does not apply and anchor 5 requires well-signaled external references. | 4 / 5 |
Total | 18 / 20 Passed |