Content
70%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.
An unusually rigorous process skill: drift signals, recovery limits, exit protocol, and handoff template are concrete and fully sequenced with explicit validation gates. Its weaknesses are length — metaphor and interop material inflate the always-loaded body — and no use of separate reference files despite clearly splittable content.
Suggestions
Cut the Mental Model metaphor and Principles sections to a couple of lines each; they are motivational padding in an always-loaded file.
Move the intent-framed-agent relationship/cadence rules, pipeline interop tables, and Entire CLI integration details into a references/ file, keeping a one-line pointer plus the precedence rule in SKILL.md.
Supply the cold-context reviewer brief as a copy-paste-ready prompt template instead of a description of what it should contain, matching the quality of the handoff and notification templates.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The operational core (drift signal lists, recovery protocol, exit protocol, handoff template) is concrete and earns its tokens, but the extended surfing metaphor ("Paddling in", "The peak", "The shoulder", "The close-out"), the motivational Principles section ("Ride the peak, not the whole ocean"), and repeated anchor listings (in Activation, the Full/Partial/Standalone bullets, and Project Context Files) are trimmable padding across ~420 lines. It fits anchor 3 — mostly efficient with some unnecessary explanation that could be tightened — rather than anchor 2 because the bulk is specific non-obvious guidance, not generic concepts Claude already knows. | 3 / 5 |
Actionability | The body provides executable, copy-paste-ready artifacts: the `entire status 2>/dev/null` detection command, a complete handoff file template with exact naming pattern, the exact hooks JSON for `.claude/settings.json`, a verbatim user-notification message, and the `ls .context-surfing/handoff-*.md` fallback check. It falls short of anchor 5 because the cold-context reviewer brief in Recovery Protocol Step 2 is described rather than supplied as ready-to-send text, and directives like "maintain a running status in your output" lack a concrete format. | 4 / 5 |
Workflow Clarity | The lifecycle is explicitly sequenced (activation → riding with pre-commit anchor checks → drift detection → recovery protocol → exit protocol → handoff/next session → close) with genuine validation checkpoints and feedback loops: the pre-commit anchor check gates every side-effecting action, re-anchor limits cap recovery attempts ("at most one local re-anchor and one cold-context check"), escalation paths branch on reviewer verdicts, and the handoff-consumed marker is set "only after re-entry succeeds". The handoff template doubles as a checklist, matching anchor 5's clear sequence, explicit validation steps, and error-recovery loops. | 5 / 5 |
Progressive Disclosure | Section headers are clear and the one bundle file (`scripts/handoff-checker.sh`) is referenced correctly with a real path, but the ~420-line body inlines substantial material that could live in separate reference files — the cross-skill relationship/cadence rules, the pipeline interop tables, and detailed Entire CLI integration notes — with no use of a references/ directory at all. This fits anchor 3 (some structure, but content that should be separate is inline) rather than anchor 4's appropriately-placed split. | 3 / 5 |
Total | 15 / 20 Passed |