Content
85%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-structured, highly actionable QA-orchestration skill: executable commands, explicit validation gates and feedback loops, and clean one-level-deep reference split. Its main weakness is length and redundancy — the cookie-extraction and multi-user-seeding procedures are described twice — which keeps conciseness from the top level.
Suggestions
Collapse the duplicated guidance: keep the detailed "Extracting the session cookie for curl (auth on)" and "Seeding non-owner skills" sections, and shorten the corresponding Execution-flow steps 3 and 4 to one-line pointers so the procedures appear once.
Trim the verbose repeated env-handling rationale — the `.env`-is-source-of-truth point is made in "How `mastra dev` reads env", "Auth modes", "Known rough edges", and the `.env` policy note; consolidate to one place and cross-reference.
Consider moving the large parameter table and scope-shortcut table into a `references/usage.md` (or `references/cli.md`) so the SKILL.md overview stays lean, leaving only the mandatory checklist and execution flow inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and largely operational (error-code tables, env handling, design decisions) rather than padded with concepts Claude knows, but it is long and repeats material — the session-cookie and non-owner-seeding procedures are summarized in "Execution flow" and then re-stated verbatim in dedicated sections — so it could be tightened. | 2 / 3 |
Actionability | Provides fully executable bash (exact script paths like `bash .claude/skills/builder-smoke-test/scripts/preflight.sh --expect off`, `curl -H "Cookie: $COOKIE"`, `export BASE=…`) plus a per-error-code remediation table, matching the copy-paste-ready anchor. | 3 / 3 |
Workflow Clarity | Multi-step process is clearly sequenced with explicit validation checkpoints — the Mandatory Test Checklist table, numbered Execution flow, Canonical order rationale, and preflight/wait-for-server gating — and includes feedback loops (error code → remediation → re-run preflight; "Verify before filing") for destructive/batch operations. | 3 / 3 |
Progressive Disclosure | SKILL.md acts as an overview whose section table maps each of 15 sections to a one-level-deep `references/<section>.md` file (all 18 reference files and 4 scripts exist on disk), with a Required/Standard/Extended tiering scheme and a final References index for easy navigation. | 3 / 3 |
Total | 11 / 12 Passed |