Content
73%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, executable skill body with excellent workflow gating, validation loops, and a verified, well-organized one-level reference bundle. Its weaknesses are duplicated test/parameter tables, an oversized inline browser-smoke section that belongs in a reference, and an undefined `smoke test` entry command.
Suggestions
Merge the "Test Options (--test)" table into the "Mandatory Test Checklist" table (they map the same options to the same reference files) to remove the duplication.
Move the "Local Studio Browser Smoke" routes/evidence table and report template into a reference file (e.g., references/browser-smoke.md), keeping only the when-to-run rules and the dev-server health-check inline.
Define the `smoke test` entry command — state what tool or script provides it (or replace it with the actual commands to run) so the Usage examples are fully executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — tables, concrete commands, and templates rather than prose — but there is real duplication: the "Test Options (--test)" table repeats the "Mandatory Test Checklist" table's option/reference mapping, and "Multi-Environment Support" restates what the Usage examples already show. Not 2 because there is no explanatory padding of concepts Claude already knows; not 4 because the duplicated tables and the ~70-line inline browser section could be meaningfully tightened. | 3 / 5 |
Actionability | Highly executable throughout: copy-paste `gh pr view`/`gh pr list` queries with expected output shape, a curl dev-server restart-and-poll loop, route tables with pass criteria, and report templates. Not 5 because the `smoke test` entry command itself is never defined or sourced — where it comes from (a script, CLI, or harness) is unstated, leaving a gap in the most important command. | 4 / 5 |
Workflow Clarity | Clear sequenced workflow with explicit validation checkpoints and feedback loops: verify the dev server is alive before opening the browser, restart-and-poll for readiness, a fallback when networkidle times out, gating rules ("Do not create the alpha smoke-test project until the automatic alpha publish workflow has completed"), a partial-publish recovery branch, and a mandatory checklist with per-test read-execute-mark progression and explicit blocker/reporting rules. Matches the top anchor including checklist and error-recovery loops. | 5 / 5 |
Progressive Disclosure | Verified against the actual bundle: all referenced paths exist (all 12 references/tests/*.md, every references/*.md named in the body and References table, and the 3 scripts/*.sh), references are one level deep, and the body instructs staying in SKILL.md until the workflow branches. Not 5 because the "Local Studio Browser Smoke" section (routes table, expectations, report template) is detailed inline content that belongs in a reference file, adding ~70 lines to the overview. | 4 / 5 |
Total | 16 / 20 Passed |