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 strong, high-signal body: the verification section delivers complete executable code with the exact non-obvious details (header variants, prefix, raw body, per-product secrets), and detail is correctly pushed into three well-organized reference files. The main deductions are the cross-promotional Related Skills/Attribution sections and dangling example-directory links.
Suggestions
Trim the 11-link "Related Skills" section to the 2-3 most relevant siblings (e.g., webhook-handler-patterns and one comparable provider skill) and drop or compress the Attribution boilerplate to save ~30 lines of tokens.
Fix the dangling examples/express/, examples/nextjs/, and examples/fastapi/ links — either include those directories in the bundle or remove/replace the pointer so navigation stays one level deep and every referenced path resolves.
Replace the version-pinned aside ("@aftership/tracking-sdk 17.0.0 has no webhook-verification helper") with a version-agnostic statement so the guidance does not age.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Core sections are dense and information-rich — every fact is non-obvious (three header variants, prefix stripping, secret-as-UTF-8, no timestamp/replay window, the 14-retry schedule). However, ~40 lines of "Related Skills" cross-promotion (11 sibling links), the Attribution boilerplate, and the version-specific aside ("@aftership/tracking-sdk 17.0.0") are minor trimmable padding, which fits the 4 anchor rather than 5. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready code in both Node and Python (fail-closed, timing-safe, header-order handling), plus the exact hookdeck-cli tunnel command, env var, port constraints, and retry math — concrete guidance covering the common cases end to end. | 5 / 5 |
Workflow Clarity | Verify-first ordering is explicit ("Verify first, parse second, handle idempotently third"), the code fails closed, and gotchas (raw body, per-product secrets, respond 2xx) are flagged as checkpoints. It falls short of 5 because the overall flow (tunnel → register endpoint → verify → respond) is organized by topic rather than an explicit ordered sequence with feedback loops. | 4 / 5 |
Progressive Disclosure | A clear overview with well-signaled, one-level-deep references: the Reference Materials section names each of the three real bundle files (overview.md, setup.md, verification.md) with its purpose, and they exist and match. Not 5: the examples/express/, examples/nextjs/, and examples/fastapi/ links point at directories absent from the bundle, and four companion-skill references are external GitHub URLs — minor navigation gaps. | 4 / 5 |
Total | 17 / 20 Passed |