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 strong, instruction-style skill body: a clear seven-step workflow backed by exact commands, real bundle reference, explicit guardrails, and honest output expectations. The main improvement space is removing repetition (the raw-launch warning and simulator caveat each appear multiple times) and moving script-implementation detail fully into the canonical reference.
Suggestions
State the raw-executable-launch rule once (Guardrails) instead of three times across workflow steps 2, 3, and the guardrail section, and fold the duplicate simulator caveat into the same place.
Move the Info.plist key enumeration and flag-by-flag script semantics from workflow step 3 into references/build-run-script.md, keeping only the script's behavioral contract (kill, build, run + optional flags) inline.
Promote process verification from an optional flag to an explicit workflow checkpoint, e.g., after 'Build and run through the script', add 'confirm the process exists with pgrep -x <AppName>'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and prescriptive with no padding or explanation of concepts Claude already knows, fitting 'efficient; minor instances of over-explanation that could be trimmed'. It is not a 5 because the raw-executable-launch warning is stated three times (workflow step 2, step 3 line 59, and the Guardrails section) and the simulator caveat appears twice. | 4 / 5 |
Actionability | Concrete, copy-pasteable commands appear at every stage — 'git rev-parse --is-inside-work-tree', 'xcodebuild -list -workspace <workspace>', './script/build_and_run.sh --debug', 'pgrep -x <AppName>', and the full Info.plist key list — with the executable script body delegated to the real references/build-run-script.md. This is 'mostly executable guidance with minor gaps' rather than a 5, since a few directives stay high-level ('prefer the app-producing scheme', 'identify executable products when possible'). | 4 / 5 |
Workflow Clarity | The seven-step Workflow is clearly sequenced from project discovery through failure classification to debugging, with a feedback path (step 5 'Classify the blocker ... Quote the smallest useful error snippet' feeding step 6 debug) and a validation mode in '--verify' with 'pgrep -x <AppName>'. It misses anchor 5 because validation is offered as an optional flag rather than an explicit checkpoint in the sequence (e.g., the workflow never says to verify the process after launch by default). | 4 / 5 |
Progressive Disclosure | The body is well-sectioned and pushes the full script template to a single real, one-level-deep reference (references/build-run-script.md), clearly signaled in both the References section and workflow step 3 ('Use references/build-run-script.md as the canonical source'). It falls short of a 5 because the body still inlines script-level detail (the Info.plist key list and flag semantics in step 3) that duplicates the canonical reference's territory. | 4 / 5 |
Total | 16 / 20 Passed |