Content
86%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 well-structured, highly actionable skill body with copy-paste-ready code across every step and clean progressive disclosure that offloads the macOS delta to a real reference file. Minor room to tighten the blockquotes and add an explicit retry/recovery loop for failed runs.
Suggestions
Replace or trim the repeated Apple-documentation blockquotes; one cited statement of the three-class pattern and the continueAfterFailure default is enough.
Add a short 'if a test fails' recovery note in Step 7/8 (re-run with -only-testing, inspect xcresult attachments) to close the feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient body built around tight code blocks with only minimal framing and no padding of concepts Claude already knows, but the repeated Apple-doc blockquotes and a few explanatory asides could be trimmed slightly. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready Swift code for query/synthesize/assert steps, bash commands for xcodebuild destinations, an xcresulttool command, and a CI yaml workflow - all covering the common cases concretely. | 5 / 5 |
Workflow Clarity | An explicit Step 1-8 sequence with a dedicated assert/verify step (Step 5) and an Anti-patterns table acting as a checklist, but it lacks an explicit if-fail-then-retry feedback loop; the destructive/batch cap does not apply since UI testing is neither. | 4 / 5 |
Progressive Disclosure | Clear overview with a well-signaled, one-level-deep reference to the real references/macos.md for the macOS delta, the rest appropriately kept inline, and a References section listing sibling skills for easy navigation. | 5 / 5 |
Total | 18 / 20 Passed |