Content
92%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 expert-aimed decision guide with excellent actionability and a rare explicit per-state verification section with expected outputs. Its only weakness is repetition: the off-state cost calculus, the stale-gate rationale, and the use-offline example each appear more than once and could be consolidated without losing information.
Suggestions
Merge 'Is the off-state run worth its cost?' into the directive-choice cases 3–4, which already make the same points ('browser e2e time is considerable, so skip the run'), keeping the cost analysis in one place.
State the stale-gate rationale once (the intro's "fails the suite the day the 'known failure' starts passing") and reference it from the Pattern section instead of restating it as "the suite fails the day they start passing".
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with repo-specific, non-inferable facts (condition tier lists, build-skipping semantics, pragma-attachment hard errors) and explains nothing Claude already knows, but the off-state cost calculus is stated three times — "browser e2e time is considerable, so skip the run" in item 3, item 4, and again in "Is the off-state run worth its cost?" ("For a new API the calculus flips: … so the off state costs almost nothing"), and the use-offline example and stale-gate semantics are each repeated. That is more than the 5 anchor's 'every token earns its place' but is trimmable redundancy rather than padding, so 4 rather than 3 or 5. | 4 / 5 |
Actionability | Guidance is fully executable: copy-paste-ready verification commands with env vars ("__NEXT_TEST_AXIS=A NEXT_SKIP_ISOLATE=1 pnpm test-start-webpack test/e2e/app-dir/<suite>/<suite>.test.ts"), an executable fixture config snippet, concrete pragma examples, exact condition names, and real working-example directories ("test/e2e/app-dir/use-offline/", "test/e2e/app-dir/concurrent-router-queue/"). Specific examples cover the common cases, matching the 5 anchor. | 5 / 5 |
Workflow Clarity | The decision sequence is clearly laid out (numbered directive-choice cases → condition tiers → fixture pattern → pitfalls) and the "Verify a gated suite in every state it can run in" section provides explicit validation checkpoints with expected outcomes per state ("expect `○ skipped` at collection, no fixture boot", "expect `⚠ gated test failed as expected`"), plus error-recovery guidance in Pitfalls. This is not a destructive or batch operation, so no cap applies, and it matches the explicit-validation 5 anchor rather than the minor-gaps 4 anchor. | 5 / 5 |
Progressive Disclosure | No bundle files exist, and the body opens with a clearly signaled one-level reference — "Full reference: [`test/lib/gate/README.md`](../../../test/lib/gate/README.md). This skill is the decision guide" — keeping only the decision guide inline and appropriately deferring full detail. Sections are well-organized with no nested references and easy navigation, matching the clear-overview 5 anchor. | 5 / 5 |
Total | 19 / 20 Passed |