Content
72%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 highly actionable with excellent executable verification code and well-organized real references, but it loses points for referencing non-existent example directories and for an implied rather than explicitly sequenced handler workflow with in-body validation checkpoints.
Suggestions
Create or remove the referenced examples/express/, examples/nextjs/, and examples/fastapi/ directories — they are cited as the source for 'complete handlers with route wiring, event dispatch, and tests' but do not exist, breaking navigation.
Add an explicit numbered receive→verify→handle workflow with a validation checkpoint (e.g., verify signature before parsing, handle idempotently via webhook id) rather than deferring the entire handler sequence to the external webhook-handler-patterns skill.
Trim the deprecated-types prose (the 'observed absence, not vendor-announced removal' rationale) to a concise note, since the deprecation status is already clear from the table.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense and largely free of basic-concept padding, but the deprecated-types prose ('observed absence, not a vendor-announced removal... no successor event is claimed') and the four-link webhook-handler-patterns block are somewhat over-explained and could be trimmed. | 4 / 5 |
Actionability | Provides a complete, copy-paste-ready HMAC verification function with timingSafeEqual and raw-body handling, plus concrete env vars, an npx tunnel command, and specific egress IPs — fully executable guidance covering the core case. | 5 / 5 |
Workflow Clarity | The verification core has a guard (try/catch on length mismatch) but the broader receive→verify→dispatch workflow is implied rather than an explicit numbered sequence, and idempotency/retry validation is deferred to an external skill rather than sequenced in-body. | 3 / 5 |
Progressive Disclosure | Structure and the three real reference files (overview.md, setup.md, verification.md) are well signaled one level deep, but the body points to examples/express/, examples/nextjs/, and examples/fastapi/ as 'complete handlers with route wiring, event dispatch, and tests' — none of these directories exist, a navigation failure that prevents the top score. | 3 / 5 |
Total | 15 / 20 Passed |