Content
82%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, highly actionable reference skill: complete dual-language verification code, useful tables, and real bundle references. Main weaknesses are dead examples/ links in the bundle, an inline promo for a separate skill, and the absence of an explicitly sequenced handler workflow with validation checkpoints.
Suggestions
Either include the examples/express, examples/nextjs, and examples/fastapi directories in the bundle or remove/repoint those links, since they currently resolve to nothing.
Trim the Attribution and Related Skills sections (or move them to a single reference file) to cut non-operational tokens.
Add a short numbered handler sequence inline (receive raw body -> verify signature -> check X-Event-Key -> dispatch idempotently) with an explicit validation checkpoint, rather than delegating it to an external skill's docs.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and useful — two tight code samples, compact tables, and no explanation of concepts Claude already knows. However, the 'Attribution' section and the 7-item 'Related Skills' list are promotional overhead that earns no operational value, keeping it just below the 'every token earns its place' 5 anchor. | 4 / 5 |
Actionability | Both verification functions are complete, executable, copy-paste ready (including edge-case handling such as the missing-signature and timingSafeEqual-throw paths), and are backed by a concrete env var and a runnable tunnel command ('npx hookdeck-cli listen 3000 bitbucket --path /webhooks/bitbucket'). This matches the 'fully executable, covers common cases' 5 anchor. | 5 / 5 |
Workflow Clarity | The core action (verify the raw body timing-safely before anything else) is unambiguous and the handler sequence is explicitly signaled via the pointer 'Verify first, parse second, handle idempotently third'. The multi-step setup flow (create webhook, set secret, wire route) is delegated to references/setup.md rather than sequenced inline, and there is no inline validation checkpoint beyond the verify function itself — a minor gap versus the 5 anchor's explicit feedback loops. | 4 / 5 |
Progressive Disclosure | Good structure: three one-level-deep, well-signaled reference files (references/overview.md, setup.md, verification.md) that all exist in the bundle, each with a one-line description. However, the body links to examples/express/, examples/nextjs/, and examples/fastapi/ which do not exist in the bundle, and the 'Recommended' section links four files inside a different skill's GitHub tree — dead or out-of-bundle references that keep it below the 5 anchor's 'easy navigation'. | 4 / 5 |
Total | 17 / 20 Passed |