Content
67%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-organized router/index body that routes symptoms to specific reference files and auditor agents with clear decision logic and a critical-migration gate. Its main weakness is redundancy: the same routing mappings are repeated across the Quick Reference, Decision Tree, and Example Invocations sections.
Suggestions
Consolidate the Quick Reference table, Decision Tree, and Example Invocations into a single routing structure — the same ~30 topic→file mappings appear three times; pick one primary format (e.g. the table) and link the others to it.
Trim the Example Invocations section, which largely restates the Quick Reference; keep only the few examples that illustrate non-obvious routing (e.g. the tvOS iCloud-first case or errSecDuplicateItem).
Normalize reference notation so all pointers use the same form; mixing `See skills/foo.md`, `See axiom-security (skills/foo.md)`, and `→ Read: skills/foo.md` makes the navigation surface inconsistent.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient tables/pointers, but the Quick Reference table, Decision Tree, and Example Invocations sections each re-encode the same ~30 topic→file mappings in three formats, and the ~50-line Example Invocations largely restates the Quick Reference — material that could be consolidated. | 3 / 5 |
Actionability | Gives concrete, executable routing — explicit file paths to read, named agents to launch ("Launch `core-data-auditor` agent"), and slash commands ("/axiom:audit core-data") — with minor gaps since the referenced skills/*.md files are not present in this skill's bundle. | 4 / 5 |
Workflow Clarity | The numbered Decision Tree is a clear sequenced routing flow, and the "Critical Pattern: Migrations" callout ("ALWAYS read `skills/database-migration.md`... prevents data loss") acts as an explicit gate on the destructive path, though the router itself delegates validation to the downstream migration skill. | 4 / 5 |
Progressive Disclosure | Well-structured overview pointing to one-level-deep references with clear navigation (tables + decision tree), but the bundle directories are empty so the many `skills/*.md` references resolve to sibling-suite files rather than this skill's own bundle, and cross-suite hops ("See axiom-security (skills/keychain.md)") slightly muddy the one-level-deep ideal. | 4 / 5 |
Total | 15 / 20 Passed |