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.
The body is well-structured and highly actionable, with executable verification code in two languages and clear reference pointers. The main gaps are a few non-essential promotional/attribution sections and references to example directories that are not present in the bundle.
Suggestions
Remove or relocate the examples/express, examples/nextjs, and examples/fastapi links (or add the directories) so no referenced paths dangle from the bundle.
Trim the six-entry 'Related Skills' promotional list and the 'Attribution' boilerplate to reduce tokens that do not serve the core webhook task.
Consider a short numbered verify-then-parse-then-handle sequence with an explicit checkpoint (e.g., 'only proceed when signature is valid') to elevate workflow clarity.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Prose and code are tight and assume competence, but the attribution boilerplate and six-entry related-skills promotional list add tokens that are not strictly necessary for the task; not the 5 anchor because those sections prevent 'every token earns its place', and not the 3 anchor because no basic-concept padding is present. | 4 / 5 |
Actionability | Provides copy-paste-ready Node and Python verification functions, a concrete hookdeck-cli command, an env-var spec, and event/header tables covering the common cases; not the 4 anchor because the examples are fully executable rather than having minor gaps. | 5 / 5 |
Workflow Clarity | The verify sequence (parse signature → HMAC over raw body → timing-safe compare → return false on failure) is clear with built-in error handling, but it is prose-and-code rather than an explicit numbered checkpoint flow; not the 5 anchor due to absent explicit validation/retry checkpoints, and not the 3 anchor because the sequence and failure handling are unambiguous. | 4 / 5 |
Progressive Disclosure | SKILL.md is an overview pointing one level deep to real reference files (overview.md, setup.md, verification.md), but the body also references examples/express/, examples/nextjs/, and examples/fastapi/ paths that do not exist in the bundle; not the 5 anchor because of the dangling example paths, and not the 3 anchor because the real references are clearly signaled and well-organized. | 4 / 5 |
Total | 17 / 20 Passed |