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 well-structured, highly actionable body: complete executable verification code for two languages, precise signature semantics, and a clean split between inline essentials and three real reference files. The main gaps are mild token overhead from the long related-skills cross-promotion list, an implied rather than explicit step ordering, and links to examples/ directories absent from the bundle.
Suggestions
Trim or collapse the 'Related Skills' list (10 entries) and the four external webhook-handler-patterns links into a short 'See also' line to reclaim tokens.
Make the handler workflow explicit as a short ordered sequence (verify signature → dedupe on event id → branch on type → respond 2xx within 10s) so the ordering is not only implied by section layout.
Either include the examples/express/, examples/nextjs/, and examples/fastapi/ directories in the bundle or remove/qualify those links so all referenced paths resolve.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core sections are lean and dense with non-obvious gotchas (whole whsec_ key, not Standard Webhooks, replay window off by default) with no re-teaching of known concepts, matching 'efficient; minor instances of over-explanation'. It falls short of 5 because the ten-entry 'Related Skills' list and the four external GitHub links in 'Recommended: webhook-handler-patterns' are peripheral tokens that could be trimmed or consolidated. | 4 / 5 |
Actionability | Two complete, copy-paste-ready verification functions (Node.js and Python) with security-conscious details (timingSafeEqual, hex-length check), exact env vars, a runnable tunnel command, and specific API endpoints for secret retrieval. This fully matches 'fully executable; copy-paste ready code or commands; specific examples cover the common cases'. | 5 / 5 |
Workflow Clarity | The sequence is clear — verify signature first (code provided), then parse/dedupe on event id, respond 2xx within 10 seconds — with validation built into the verify functions and idempotency guidance. It is not 5 because the ordering is implied by section layout rather than an explicit ordered checklist, and there is no explicit verify-then-fix feedback loop for the setup/debug path. | 4 / 5 |
Progressive Disclosure | The body is a well-signaled overview with one-level-deep references (references/overview.md, setup.md, verification.md) that all exist and are annotated at point of use and in a 'Reference Materials' list. It misses 5 because the 'For complete handlers with tests, see examples/express/, examples/nextjs/, examples/fastapi/' links point to directories that do not exist in this bundle, a minor navigation gap. | 4 / 5 |
Total | 17 / 20 Passed |