Content
81%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 strong, immediately usable skill: complete verified handlers in two languages with correct Standard Webhooks details and a well-organized reference bundle. The main costs are padded cross-promotion sections that dilute the token budget and three dead examples/ links that break navigation, plus partial duplication of the reference material inline.
Suggestions
Remove or drastically trim the 'Related Skills' section (10 external links) and fold the 'Recommended: webhook-handler-patterns' links into the Reference Materials section — they consume context without aiding the task.
Fix or remove the broken examples/express/, examples/nextjs/, and examples/fastapi/ links, since those paths do not exist in the skill bundle and will fail when followed.
Trim the in-body event tables to the few most common event types and point to references/overview.md for the full list, since that file already claims to carry the complete event reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core is dense and information-rich (raw-body capture, signed-content format, data.type vs top-level type), but the 'When to Use This Skill' section duplicates the frontmatter description, and the 'Related Skills' link farm (10 external links), 'Recommended' section, and 'Attribution' add no operational value and could be trimmed. | 3 / 5 |
Actionability | Two fully executable handlers (Express and FastAPI) with signature verification, replay window, timing-safe comparison, raw-body middleware, and event dispatch are copy-paste ready, plus a concrete SDK alternative, env vars, and a runnable tunnel command. The only gap is the FastAPI handler's '# Handle event.data.type ...' placeholder, but the dispatch pattern is fully shown in the Express version. | 5 / 5 |
Workflow Clarity | The single task (implement a verified webhook handler) is unambiguous: verify signature with timestamp window → reject with 400 → parse → dispatch → 200, with the validation checkpoint embedded in both implementations and the verify-first ordering stated in prose. Idempotency/retry semantics are explicitly deferred to the webhook-handler-patterns skill with pointed links. | 5 / 5 |
Progressive Disclosure | Three real, clearly annotated one-level-deep references (overview.md, setup.md, verification.md) with good overview structure. However, the 'For complete working examples' links point to examples/express/, examples/nextjs/, and examples/fastapi/ which do not exist in the bundle, and the body's event tables duplicate the 'full event list' that references/overview.md already provides. | 4 / 5 |
Total | 17 / 20 Passed |