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.
The body is a highly actionable, well-validated runbook: executable commands cover discovery, execution, parsing, attachment export, and error recovery, with explicit checkpoints for a batch operation. Its weaknesses are token efficiency (duplicated process-management guidance and verbose axbuild internals) and the absence of any progressive disclosure — all reference-worthy material is inlined in one long file. Strong execution guidance undermined modestly by bloat and monolithic structure.
Suggestions
Deduplicate the process-management guidance: the pgrep inventory appears both in the shared section and in Mandatory First Steps, and the detailed description of axbuild's descendant-process cleanup could be condensed to the operational rule (never terminate unrelated processes; inspect ownership).
Move Common Failure Patterns, the Error Quick Reference table, and the Invocation Examples into a references/ file (e.g. references/failure-patterns.md) linked from SKILL.md, keeping the main file as a lean overview with the core run/parse/export workflow.
Tighten the shared axbuild section's multi-environment resolution prose (Claude Code/Codex/Pi/Cursor/MCP cases) into a short decision table or a single fallback rule, since the current paragraph repeats the same verification requirement in several forms.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is command-dense rather than concept-explaining, but the shared axbuild/process-management section is verbose, the pgrep inventory ("pgrep -lx xcodebuild; echo \"pgrep exit=$?\"") appears twice (once in the shared section, once in Mandatory First Steps), and details of axbuild's internal cleanup behavior run longer than an executing agent needs. This matches anchor 3 (mostly efficient with some unnecessary explanation that could be tightened) rather than anchor 4's minor-trim profile. | 3 / 5 |
Actionability | Nearly everything is copy-paste-ready executable bash: scheme discovery via "xcodebuild -list -json" with jq extraction, booted-simulator UDID resolution, fresh mktemp run directories per run, full xcodebuild test invocations (single class/method, skip-testing), and the complete xcresulttool surface (summary, tests, test-details, export attachments with --only-failures, coverage, console logs) plus a manifest-reading command and an error-to-fix table. This matches anchor 5's fully executable, common-case-covering profile. | 5 / 5 |
Workflow Clarity | The sequence (mandatory checks → run → parse → analyze → report) is explicit and paired with genuine validation checkpoints: pgrep exit-code semantics before every Xcode invocation, verifying AXBUILD by running "--help" before use, the booted-simulator check, and rules like "A null failed-test count means uncertainty, not zero" and truncation-marker detection. Error-recovery feedback loops exist (saved-log fallback instead of rebuilding, failure-pattern diagnosis with quick fixes, the error quick reference), so this is a batch operation with validation present and matches anchor 5 rather than anchor 4's minor-validation-gaps profile. | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), so everything — the ~90-line shared axbuild block, Common Failure Patterns, the error quick reference table, and invocation examples — is inlined in a single ~370-line SKILL.md. Section structure is decent, but content that clearly belongs in separate reference files is inline and there are no in-bundle references to signal, matching anchor 3 rather than anchor 4's mostly-appropriate placement with clear references. | 3 / 5 |
Total | 16 / 20 Passed |