Content
96%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.
Excellent operational content: executable commands, copy-paste snippets, explicit validation checkpoints, a troubleshooting table, and a fully specified mock-agent-host flow with expected log output. The only structural gap is that some long-tail detail (mobile emulation, mock-agent seeding) is inlined in a fairly long single file rather than split into reference files.
Suggestions
Move the mobile emulation section (CDP snippets, UA override, standalone-Playwright note) into a references file (e.g. references/mobile-emulation.md) and keep a one-line pointer plus the key caveat in SKILL.md.
Split the mock agent host deep detail (session seeding env var, scripted prompt list, protocol-version tag requirement) into references/mock-agent-host.md, leaving the start command, URL, and expected log sequence inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every section conveys non-obvious operational knowledge (sibling-repo layout, 'npm run dev' must run from vscode-dev, first-start crash behavior, service-worker cache purge snippet, known-noise console list, mock-agent protocol-version requirement) with no conceptual padding Claude would already know. Lean and efficient, matching the 5 anchor rather than 4, since there is no over-explanation left to trim. | 5 / 5 |
Actionability | Fully executable throughout: 'npm run dev' with an absolute-pushd variant, a curl readiness check returning 200, exact URLs, copy-paste Playwright/CDP snippets, a precise mock-agent-host launch command with flags and env-var seeding, and a symptom/cause/fix troubleshooting table. Covers the common cases, matching the 5 anchor. | 5 / 5 |
Workflow Clarity | The process is clearly sequenced — layout assumption → start dev server → curl-based readiness check → URLs → browser interaction patterns → troubleshooting → optional mock agent host with prerequisite check ('If out/... is missing, start that task first') and verification via the expected ordered log sequence. Validation checkpoints and error-recovery loops are explicit, matching the 5 anchor; no destructive/batch cap applies. | 5 / 5 |
Progressive Disclosure | Good structure with clear section headers and easy navigation, but the skill is a single ~135-line file with no bundle files; deeper detail such as the mobile CDP emulation snippet and the mock-agent session/prompt seeding reference would fit better in a separate reference file. Anchor 4 fits ('good structure, minor organization gaps') rather than 5, since the under-50-lines exception does not apply and some content that could be split is inline. | 4 / 5 |
Total | 19 / 20 Passed |