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 dense, genuinely actionable template library with concrete commands, verification steps, and escalation guidance throughout — the actionability is the standout strength. Weaknesses are structural: the body is a ~380-line monolith whose only external reference is a dangling path, and generic instruction boilerplate dilutes an otherwise lean document.
Suggestions
Fix or remove the dangling reference to resources/implementation-playbook.md, which does not exist in the bundle — either create the file or drop the pointer.
Split the two full templates into reference files (e.g., templates/service-outage.md, templates/database-incident.md) and keep SKILL.md as a short overview with well-signaled one-level-deep links, cutting the main body to well under 100 lines.
Delete the generic 'Instructions' filler ('Clarify goals, constraints, and required inputs', 'Apply relevant best practices and validate outcomes') and replace it with the actual workflow for assembling a service-specific runbook from these templates, including how to resolve the $DB_HOST/$DB_USER placeholders and the referenced-but-missing helper scripts.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk of the ~380-line body is lean, high-value template content — severity tables, kubectl/psql commands, escalation matrices, comms templates — with no explanation of concepts Claude already knows. What keeps it below anchor 5 is generic filler: the 'Instructions' bullets ('Clarify goals, constraints, and required inputs. Apply relevant best practices and validate outcomes.') and the boilerplate 'Do not use this skill when' section are vague padding that earns no tokens, plus a doubled code fence artifact around the Resolution Notification template. | 4 / 5 |
Actionability | The templates are overwhelmingly concrete and executable: specific kubectl commands ('kubectl rollout undo deployment/payment-service -n payments'), real SQL against pg_stat_activity, exact alert thresholds ('payment_error_rate > 5%'), and a full NetworkPolicy YAML. It falls short of anchor 5's copy-paste-ready bar due to minor gaps: undefined env vars ($DB_HOST, $DB_USER), helper scripts that exist nowhere in the bundle (./scripts/db-rollback.sh, ./scripts/smoke-test-payments.sh), and a company-internal feature-flag endpoint. | 4 / 5 |
Workflow Clarity | Sequences are clear with checkpoints in the right places: an initial-triage classification table routing symptoms to numbered mitigation sections, per-step 'Verify recovery'/'rollout status' checks, a dedicated Verification Steps block before Rollback Procedures, and a 9-part runbook structure. Validation is present for the destructive operations (pg_terminate_backend, rollout undo, VACUUM FULL) so the destructive-ops cap does not apply; it sits below anchor 5 only because the skill's own meta-workflow for assembling a custom runbook from these templates is left implicit in the vague 'Instructions' section. | 4 / 5 |
Progressive Disclosure | The body has decent section structure but no bundle exists, and its one external pointer — 'If detailed examples are required, open resources/implementation-playbook.md' — references a file that does not exist in the bundle (no resources/ directory). Both full templates (~230 and ~60 lines) are inlined in SKILL.md when they are prime candidates for separate reference files, matching anchor 3's 'content that should be separate is inline' with a buried, broken reference. | 3 / 5 |
Total | 15 / 20 Passed |