Content
63%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 strongly actionable, well-sequenced Xcode build audit workflow with exact commands, expected setting values, and good failure handling. Its weaknesses are redundancy (mission restated, UI walkthroughs, invocation examples), an unexplained $SETTINGS_JSON input, and a monolithic body that offloads nothing to reference files.
Suggestions
Collapse the mission statements (intro, "## Your Mission", "## Scope") into one section and delete the Invocation Examples block, which duplicates the description's triggers.
Define how $SETTINGS_JSON is produced (or replace it with a concrete capture command) so every listed command is actually runnable.
Move the build-settings catalog and Build Timeline walkthrough into a references/ file (e.g. references/settings-catalog.md) and keep SKILL.md as a scan-process overview with one-level-deep pointers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense, useful checklists and commands, but there is real redundancy and padding: the mission is stated three times (intro "Your mission is to scan the project...", "## Your Mission", "## Scope"), the Invocation Examples section restates triggers already in the description, and the Build Timeline section walks through Xcode UI navigation (Report Navigator, Cmd+9, Assistant editor) that Claude already knows. It fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than 4, where over-explanation would be only minor. | 3 / 5 |
Actionability | Guidance is largely executable: exact `xcproject settings`/`inspect` commands with all flags, exact setting names and expected values (e.g. `SWIFT_COMPILATION_MODE` = `singlefile` for Debug), a copy-paste timing-build snippet with exit-code handling, and a concrete ❌/✅ before/after fix. It stops short of 5 because `$SETTINGS_JSON` is passed to commands but never created or explained anywhere, and the `xcproject` fallback chain (plugin-root resolution, npx) is convoluted enough that a fresh run could stall. | 4 / 5 |
Workflow Clarity | The Scan Process is a clearly sequenced 5-step workflow with specific commands per step, and validation checkpoints are present: fresh authorized capture vs. declared values, "do not infer missing settings as optimal defaults", pgrep inventory before running a build, exit-code-preserving failure handling, and "Always measure before and after". It does not reach 5 because there is no explicit feedback/re-validation loop after recommending fixes, and per-step output checks are implicit rather than a checklist. | 4 / 5 |
Progressive Disclosure | The bundle contains no reference files at all (no references/, scripts/, assets/), and the ~215-line body inlines everything: a full build-settings catalog, phase-script red flags, timeline analysis, and output-format spec are all inline with only headers for navigation. The one cross-reference, `axiom-build (skills/build-performance.md)`, points to an external skill outside this bundle. This fits 'some structure but could be better organized; content that should be separate is inline' — good section structure keeps it above 2, but nothing is offloaded into one-level-deep reference files, so it cannot score 4. | 3 / 5 |
Total | 14 / 20 Passed |