Content
75%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 dense, high-quality body: executable verification code, precise delivery semantics, and a smart event_id dedup insight, all with clean reference structure. The main defects are the missing examples/ directories referenced for complete handlers, the absence of a dedup code example, and some trimmable promotional sections.
Suggestions
Remove or repair the references to examples/express/, examples/nextjs/, and examples/fastapi/ — these directories are not in the bundle, so the 'complete handlers with tests' pointer leads nowhere. Either ship the example handlers or replace the links with a minimal inline handler skeleton.
Add a short executable deduplication snippet (e.g., a keyed cache/DB check on event_id returning early on conflict) to match the concreteness of the verification section, since dedup is presented as a core concern.
Trim the 'When to Use This Skill' question list (it duplicates the frontmatter description) and condense the 13-item 'Related Skills' list to the few most relevant entries to reclaim token budget.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The technical core is lean and packed with non-obvious facts (svix 2.x return-value gotcha, retry schedule, 15-second window, event_id vs svix-id) with zero basic-concept padding, but the 6-bullet 'When to Use' section duplicates the description and the 13-item 'Related Skills' list plus attribution block could be trimmed. | 4 / 5 |
Actionability | Mostly copy-paste ready: a complete JS verification function with error paths, a Python one-liner, a concrete hookdeck-cli command, and the env var with its retrieval path. The gap is deduplication — the skill's second core task — which is explained but lacks any executable snippet. | 4 / 5 |
Workflow Clarity | The handler flow is well defined per section and the sequence is stated ('Verify first, parse second, handle idempotently third') with validation checkpoints in the verification code (fail closed → 500, WebhookVerificationError → 400), but the end-to-end sequence is never laid out in the body as one workflow — it is delegated to an external skill that may not be installed. | 4 / 5 |
Progressive Disclosure | Three one-level-deep references (overview.md, setup.md, verification.md) are clearly signaled with per-file descriptions, all verified present, and bulk content is correctly split out. However, the body points to examples/express/, examples/nextjs/, and examples/fastapi/ for 'complete handlers with tests', and none of these directories exist in the bundle — broken navigation paths. | 4 / 5 |
Total | 16 / 20 Passed |