Content
85%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 operational skill body: concrete commands, a tight drive loop with built-in validation, hard safety rules, and a genuinely useful error-recovery table, all in under 60 lines. The minor deductions come from version pins that will age, one redundant boundary sentence, and step 2's bridge wiring being described rather than shown.
Suggestions
Make step 2 as copy-paste ready as the daemon and mint commands: include the exact devicectl launch invocation and a minimal DebugBridge SPM dependency/wiring snippet, or explicitly name the README section that contains them.
Trim the boundary sentence 'No simulator, no XCTest, no WebDriverAgent' (it duplicates the frontmatter description) and move the tool version pins ('swift >= 5.9, bun >= 1.1') into the referenced runtime README so the body does not go stale.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational with no explanation of concepts Claude already knows, but it carries time-sensitive version pins ("swift >= 5.9, bun >= 1.1") outside any old-patterns/deprecated section and repeats the frontmatter boundary ("No simulator, no XCTest, no WebDriverAgent"). This places it at 'efficient with minor instances that could be trimmed' rather than 'every token earns its place'. | 4 / 5 |
Actionability | Mostly executable: copy-paste commands like "bun run gen-accessors", "bin/ios-agent-qa-daemon", "bin/ios-agent-qa-mint grant --remote <id>", and full endpoint paths ("GET /screenshot", "POST /session/acquire", "/tap"). Not 5 because step 2 gives direction rather than code ("Add the `DebugBridge` SPM dependency... wire it from the `@main` App init") and the exact devicectl launch invocation is unspecified. | 4 / 5 |
Workflow Clarity | Four clearly sequenced phases with an explicit in-loop validation checkpoint ("re-screenshot → compare" after every state-changing action, with screenshots required before and after) and a five-row failure table mapping specific error codes (403, 409, 429, 503) to concrete recovery actions — explicit validation plus feedback loops for error recovery. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), and the body stays a lean overview: it delegates setup, capability tiers, and ACL detail to the runtime repo's README ("Read that repo's README.md first; it is the source of truth for setup, capability tiers, and the Tailscale ACL shape") — a well-signaled, one-level-deep reference with clear section navigation (Runtime, Loop, Rules, Failure modes). | 5 / 5 |
Total | 18 / 20 Passed |