Content
93%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.
An exemplary contract-style skill body: it locks down mutation points, credentials, and the OSS entitlement seam with exact names and commands, and delegates setup detail to external docs instead of restating it. The only gap is the absence of error-recovery guidance around its single validation gate (the E2E suite).
Suggestions
Add one line of recovery guidance to step 5 of 'Working on billing': what to check or fix when the E2E suite refuses to start (e.g., where the test-mode assertion fails and which env vars/channel config to correct) so the validation gate has an explicit feedback loop.
Make the entitlement seam concrete with a short example of gating through `getEntitlement` (a typed wrapper call site), so the 'New gates go through this function' rule is actionable rather than only named.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean throughout — it explicitly delegates known material ("Setup itself is documented; cite it, don't restate it") and states only repo-specific contracts and rules without explaining what Stripe, Metronome, or webhooks are; every token earns its place, matching anchor 5. | 5 / 5 |
Actionability | Fully executable guidance: copy-paste commands (`pnpm tsx editor/scripts/billing/cli.ts setup:stripe`, `pnpm --filter editor vitest run lib/billing/__tests__/e2e`), exact function and module names (`fn_billing_apply_stripe_event`, `getEntitlement` from `editor/lib/billing/metronome.ts`), and concrete rules covering the common cases, matching anchor 5. | 5 / 5 |
Workflow Clarity | The six-step "Working on billing" sequence is clearly ordered and includes an explicit validation gate (the E2E suite that "refuses to start unless every channel is demonstrably test-mode"), but there is no guidance on what to do when that gate or the suite fails — the error-recovery feedback loop of anchor 5 is only implied, so anchor 4 fits. | 4 / 5 |
Progressive Disclosure | A single-file skill with well-organized sections and clearly signaled, one-level-deep references (grida.co docs for setup/design/known-issues, `editor/scripts/billing/README.md`, and the `ee`/`security`/`database` sibling skills); nothing is nested deeper or buried, matching anchor 5. | 5 / 5 |
Total | 19 / 20 Passed |