Content
75%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 highly actionable, expert-level reference: nearly every instruction is an executable command enriched with hard-won edge-case knowledge (silent exit-0 failures, flag differences between tools, multi-simulator pitfalls) plus explicit validation gates for crash detection and UI assertions. Its weaknesses are redundancy across sections, inline version-sensitive details, and a body long enough that some material arguably belongs in the referenced files.
Suggestions
Merge 'Comprehensive Diagnostics (simctl diagnose)' into section 11 'App Inventory & Diagnostics' and consolidate the duplicated 'Unable to boot' and `xcui doctor` explanations into single locations.
Collect the Xcode-version-specific notes (Xcode 26/26.6+/27, DeviceHub naming, flag differences) into one clearly labeled compatibility section so version churn doesn't age the main instructions.
Move the full flag tables and dispatch rules of sections 8, 13, and 15 into the already-referenced files (device-control-ref.md, xcui-ref.md) to shorten the SKILL.md body, and define `$SIM` (or unify on `$UDID`) in section 8.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and command-first with no basic-concept padding, but it duplicates content — 'Comprehensive Diagnostics' repeats section 11's `simctl diagnose`, the 'Unable to boot' fix appears in both 'Common fix' and the error table, `xcui doctor` is explained in sections 13 and 15, and the toxiproxy preflight appears in Mandatory First Steps and section 16 — and inline time-sensitive version specifics (Xcode 26/26.6+/27, DeviceHub) are not quarantined in a deprecated/old-patterns section. This matches 'mostly efficient but could be tightened' rather than the minor-trimming anchor 4. | 3 / 5 |
Actionability | Nearly every section delivers copy-paste-ready commands with exact flags, complete jq filters, and real failure-mode semantics (e.g. 'a tap prints ✓ whether or not anything happened, so assert the result with xcui wait/xcui assert'), fully covering the common cases per the top anchor. The lone `$SIM` variable in section 8 (undefined; elsewhere `$UDID`) is a trivial naming alias rather than a guidance gap. | 5 / 5 |
Workflow Clarity | The Test Workflow gives a clear 7-step sequence, and explicit validation exists — a dedicated Crash Detection gate before reporting failure (with a hang-vs-crash branch), assert-by-readback for status bar overrides, and an error quick-reference table with recovery fixes. However, these checkpoints live in separate sections rather than being sequenced inline in the workflow itself, which is a minor validation gap that keeps it below anchor 5. | 4 / 5 |
Progressive Disclosure | The body uses well-organized numbered sections and defers exhaustive detail to clearly signaled one-level-deep references (axiom-tools skills/xcui-ref.md, device-control-ref.md, xcsym-ref.md; axiom-testing skills/ui-testing.md); no bundle files ship with this skill, so structure and references alone are judged. The ~495-line body still inlines substantial reference-style material across 16 capability sections, which is a minor organization gap matching anchor 4 rather than the cleanly split anchor 5. | 4 / 5 |
Total | 16 / 20 Passed |