Content
86%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 exemplary instruction-only skill body: lean, project-specific, and dense with exact identifiers, paths, commands, and behavioral invariants that only this codebase's author could know. The only meaningful gap is the absence of a minimal executable pattern for the settled-turn/stream-resume checks and explicit validation feedback loops in the verification workflow.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative bullets with zero padding — no explanations of concepts Claude already knows, every line a project-specific invariant, command, or file path (e.g. "Upsert events by local `eventIndex`; on final snapshot, delete rows beyond the final event length"). It clearly matches the level-5 anchor 'Lean and efficient; assumes Claude's competence; every token earns its place'; it is not level 4 because there is no over-explanation to trim. | 5 / 5 |
Actionability | Guidance is highly concrete: a copy-paste ripgrep command, exact route paths ("POST /eve/v1/session/:sessionId"), exact file conventions ("agent/tools/<snake_case>.ts"), and verification commands ("pnpm typecheck", "pnpm build"). Per the code_vs_instruction note the absence of code is not penalized, but a few directives remain judgment-level rather than executable — e.g. "Consider a turn settled on an event `endsTurn` from `eve/client` accepts" names the events without a minimal code pattern for the check — which keeps it below the fully copy-paste-ready level-5 anchor. | 4 / 5 |
Workflow Clarity | The sections impose a clear order — First Pass (read docs/search before code), then Core Model, Persistence, Streaming, Connections, and a Verification section with explicit checkpoints ("Run `pnpm typecheck` ... `pnpm build` before opening a PR", plus a manual-flow checklist). It falls short of the level-5 anchor because there are no explicit feedback loops (validate → fix → re-run) and the manual verification list lacks pass/fail criteria, matching 'Clear sequence with most checkpoints present; minor validation gaps'. | 4 / 5 |
Progressive Disclosure | With no bundle files present, the ~55-line body is appropriately self-contained: well-organized sections, each a compact set of invariants, with clearly signaled one-level-deep pointers to the project's deeper docs ("Read `AGENTS.md` first", "Read `docs/how-the-chatbot-works.md`", "docs/setup-and-deploy.md"). This matches the level-5 anchor's clear overview with well-signaled references and easy navigation; splitting this little content into separate files would add indirection, not value. | 5 / 5 |
Total | 18 / 20 Passed |