Content
85%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 lean, well-sequenced hook spec with genuine validation gates and feedback loops — the workflow engineering is excellent. However, all three references point to a HOOKS-REFERENCE.md that does not exist in the bundle, and markdown-escaped pipes corrupt several copy-paste commands in the on-session-start table.
Suggestions
Ship HOOKS-REFERENCE.md in the skill bundle (or inline the extended startup checks and per-check example commands) — the body points to it three times but the file does not exist, so readers hit a dead end when they need the detailed commands.
Fix the pipe escaping in the on-session-start table: `rg -n "ERROR\|FAIL" .opencastle/AGENT-FAILURES.md \|\| true` renders with literal backslashes when copied. Move commands out of tables into fenced code blocks, or restructure the table so pipes don't need escaping.
Replace the `opencastle log --type session ...` ellipsis with one complete, realistic invocation (including the fields the Pre-Response Quality Gate expects) so the logging command is copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is tables of bare commands with no concept explanations, no padding, and no restating of things Claude already knows ("Conventions (not auto-triggers) agents execute at specific lifecycle points" is the only prose). Every token earns its place — the 5 anchor. | 5 / 5 |
Actionability | Mostly concrete, executable commands (`rg -n "keyword" .opencastle/LESSONS-LEARNED.md`, `opencastle doctor`, `gh issue view TAS-XX --json body -q '.body'`), but three gaps keep it from 5: pipes in table cells are markdown-escaped (`rg -n "ERROR\|FAIL" ... \|\| true`) so the commands are corrupted when copied, `opencastle log --type session ...` trails off with a literal ellipsis, and "keyword"/"TAS-XX" placeholders are unexplained. This is the 4 anchor (concrete commands, minor gaps) rather than 3 because the commands are real, not pseudocode — the defects are fixable transcription artifacts. | 4 / 5 |
Workflow Clarity | The lifecycle is explicitly sequenced with an ASCII diagram, on-pre-delegate has a hard gate ("All 6 must pass before the sub-agent is dispatched"), and on-post-delegate has verification with an error-recovery loop ("Move to Done or re-delegate; 3rd failure → .opencastle/AGENT-FAILURES.md") plus per-AC verification. Clear sequence, explicit validation steps, feedback loops, and checklists — the 5 anchor; validation is present throughout, so the missing-validation cap does not apply. | 5 / 5 |
Progressive Disclosure | The body cites "See [HOOKS-REFERENCE.md](HOOKS-REFERENCE.md)" three times for extended startup checks, example commands per check, and detailed verification commands — but no bundle directory exists and HOOKS-REFERENCE.md is nowhere in the skill, so the promised detail layer is missing entirely and navigation dead-ends exactly where a reader needs the extended commands. References are clearly signaled (4-anchor trait) but broken, and the body (~74 lines) exceeds the under-50-line simple-skill exception, so it cannot score 5 on sections alone. | 3 / 5 |
Total | 17 / 20 Passed |