Content
71%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 router/index skill: exact paths, filename tables, and a decision tree make it highly actionable, with an explicit fallback path when Xcode is absent. Its main defects are the duplicated decision-tree layer and hard-coded file counts that will drift, an ls command that only works when AXIOM_XCODE_PATH is set, and two references to a nonexistent skills/apple-docs-research.md file.
Suggestions
Fix or remove the dangling reference to skills/apple-docs-research.md — it is cited twice (fallback and research methodology sections) but does not exist in the bundle; either ship the file or inline the one or two essential instructions.
Make the fallback `ls` command work for default installs, e.g., `ls /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/share/doc/swift/diagnostics/` with a note to substitute $AXIOM_XCODE_PATH when set.
Drop the hard-coded counts ("20 files", "46 files") and consider trimming the Routing Decision Tree, since it duplicates routing information already derivable from the tables and adds tokens that must be maintained in sync.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is almost entirely routing data (path tables, an index, a decision tree) with no explanation of concepts Claude already knows, so it is broadly efficient. Minor tightening opportunities exist: the Routing Decision Tree re-states targets already enumerable from the tables, and hard-coded counts ("20 files", "46 files") duplicate what a directory listing shows and will drift with Xcode versions. Not level 5 because of this duplicated routing layer. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: exact base directories, a copy-paste example Read invocation, and per-topic filename tables. Falls short of level 5 because the fallback `ls $AXIOM_XCODE_PATH/...` command assumes an env variable that is only documented for Xcode-beta setups, so it fails on a default install where the fixed default path should be used instead. | 4 / 5 |
Workflow Clarity | The read flow is clearly sequenced: session-start hook resolves the base directory, Read the specific file, follow the decision tree, and a three-step fallback when Xcode is unavailable with an explicit "Do not silently fail" instruction. This is a read-only routing skill, so the destructive/batch validation cap does not apply. Not level 5 because there is no checkpoint for handling a hook that never ran or a stale session-context path — the reader must infer to verify the path exists first. | 4 / 5 |
Progressive Disclosure | Sections are well organized (When to Use, How to Read, per-category tables, decision tree, fallback), but the only external references — "see skills/apple-docs-research.md" (lines 203 and 215) — point to a file that does not exist in this bundle, a dangling reference that breaks navigation. Per the judging guideline to score against the actual bundle structure, this lands at level 3 ('references present but not clearly/signaled-valid') rather than 4. | 3 / 5 |
Total | 15 / 20 Passed |