Content
90%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 hub-style SKILL.md: a tight task-to-reference routing table plus dense, executable operational rules with real validation gates (evidence-required checks, preflight suites, backend-log cross-checks). The two residual gaps are the absence of a single ordered quickstart sequence and one orphaned bundle file that is invisible from the routing table.
Suggestions
Add a routing-table entry (e.g., 'All-tool acceptance run: read references/skill-all-tool-acceptance.md') so the orphaned 10 KB bundle file is discoverable from SKILL.md and navigation covers the full bundle.
Add a short ordered quickstart sequence at the top of Rules (read ../.env → confirm services running → bin/lbs fixture check → run case → bin/lbs test result --evidence) so a newcomer gets the canonical step order without assembling it from scattered rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean routing table plus operational rules with zero concept explanation — no preamble about what LangBot or e2e testing is; every line is a directive ("Read `../.env` first and use `LANGBOT_FRONTEND_URL`...", "Do not expose API keys..."). It assumes Claude's competence and every token carries operational information, matching the 'lean and efficient' anchor; not 4 because there is no over-explanation to trim. | 5 / 5 |
Actionability | Guidance is fully concrete and executable: exact commands ("bin/lbs suite start <suite-id>", "bin/lbs fixture check", "bin/lbs test run agent-runner-release-preflight"), exact env vars (`LANGBOT_FRONTEND_URL`, `automation_pipeline_url_env`), exact file paths per task, and an exact MCP endpoint pattern. Per the rubric's instruction-skill note, absence of code is not penalized when guidance is this actionable, and the named commands cover the common cases — the anchor-5 fit; not 4 because there are no missing key details in what the body itself sets out to do. | 5 / 5 |
Workflow Clarity | Ordering and validation gates are explicit via before/until language: "Confirm the backend and frontend are actually running before testing", "run ... agent-runner-release-preflight before the full agent-runner-release-gate suite", the suite start → test result → suite report chain, and completion gates ("A WebUI test is not complete until the visible UI result is checked against backend logs", "Do not mark a case pass until `test result --evidence` covers every value in ... `evidence_required`"). It stops short of 5 because there is no canonical ordered sequence for a typical run — a newcomer must assemble the order from scattered rules, and feedback loops on failure are delegated to troubleshooting.md rather than stated inline. | 4 / 5 |
Progressive Disclosure | The Routing section clearly maps each task to a real one-level-deep file, and all 15 cited paths exist in references/, matching 'clear overview with well-signaled references'. However, scoring against the actual bundle: `references/skill-all-tool-acceptance.md` (10 KB) is orphaned — referenced neither in SKILL.md nor by any other reference file — so it is unreachable via navigation, a minor organization gap that holds this at anchor 4 rather than 5. | 4 / 5 |
Total | 18 / 20 Passed |