Content
77%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 dense, command-rich playbook: nearly every workflow ships copy-paste-ready bash with explicit destructive-action gating, validation checkpoints, a decision tree, and escalation paths. Its weaknesses are length and repetition — duplicated CI/SPM blocks, a ~35-line runtime-download catalogue inlined into the main file, and an unusually dense axbuild prose section — plus the absence of any progressive-disclosure bundle.
Suggestions
Deduplicate: the CI-detection snippet and the resolvePackageDependencies retry block each appear twice; keep one canonical copy and cross-reference it from the other location.
Create a references/ bundle and move the simulator-runtime download catalogue (~35 lines), the error-pattern table, and the invocation examples into it, linking them from SKILL.md to cut the main file roughly in half.
Break the 'Capture Build and Test Diagnostics' wall of prose into a short bulleted checklist (resolve AXBUILD path, run pgrep pre-check, inspect command vs collection separately, read artifacts.report for full records) so the axbuild contract is scannable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and command-focused, but with real duplication: the CI-detection snippet appears both in Mandatory First Steps ("echo \"CI env: ${CI:-not set}...\"") and again as a full if/else in "Detecting CI/CD Context"; the resolvePackageDependencies retry block is repeated verbatim in §3 and the CI/CD fixes; and the closing Scope section restates the mission. The dense axbuild prose paragraph ("Inspect command (native outcome) and collection (evidence completeness) separately...") also needs multiple readings. Not the 4 anchor because the repetition is more than 'minor instances of over-explanation'. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready bash throughout: pgrep with exit-code interpretation, jq JSON filters for simulator state, sed extraction of BUILD_DIR with a case guard scoping the rm -rf, mktemp-wrapped result-bundle capture, and a saved-log fallback. Placeholders like <ACTUAL_SCHEME_NAME> are explicitly paired with their discovery command (xcodebuild -list), and the error-pattern table maps concrete error strings to specific fixes, covering the common cases. | 5 / 5 |
Workflow Clarity | "Your Mission" gives a numbered five-step sequence with an explicit verification step and an approval gate before destructive actions; the decision tree routes each symptom to a fix section; "When to Stop and Report" and Audit Guidelines (a 9-item checklist including "Verify fixes worked" and "If fix doesn't work - escalate") provide checkpoints and error-recovery feedback loops. Destructive operations are properly validated (ownership/state inspection, pgrep before every invocation, scoped-removal guard), so the destructive-cap at 3 does not apply. | 5 / 5 |
Progressive Disclosure | No references/, scripts/, or assets/ bundle exists — 475 lines live in a single file. Section headers and § cross-references give it real structure, but reference-like content is inlined: the ~35-line simulator-runtime download catalogue, the error-pattern table, and six invocation examples clearly belong in separate files. This matches 'some structure but could be better organized; content that should be separate is inline' rather than the 4 anchor's 'most content is appropriately placed'. | 3 / 5 |
Total | 16 / 20 Passed |