Content
81%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-engineered audit workflow: explicit tool-use mandates, verbatim search patterns, per-finding verification, compound-severity rules, and honest measurement caveats. The sequence and validation loops are exemplary; the residual gaps are one-line fix directions instead of code examples and some redundancy across phases and closing sections.
Suggestions
Add compact before/after Swift snippets for the two or three CRITICAL patterns (formatter caching, async file loading) so fixes are copy-paste ready rather than prose directions.
Trim the Invocation Examples, Scope, and parts of the Output Format template that restate Phase content, cutting roughly 30-40 lines without losing guidance.
Consider moving the 11 pattern definitions into a referenced file (e.g., patterns.md) to slim SKILL.md toward an overview plus phase workflow.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with earned content — measured costs ("~135 µs per call", "+95 MB over an 800-row scroll"), exact grep patterns, false-positive lists, and per-pattern verification rules — and mostly avoids teaching concepts Claude already knows. Not 5 because there is trimming available: the long Output Format template, Invocation Examples, and Scope sections partly restate content, and several rules are explained twice across Phases 2-4; not 3 because the padding is minor relative to genuinely non-obvious detail. | 4 / 5 |
Actionability | Every detection pattern ships a verbatim grep string ("DateFormatter()", ".first(where:", "preparingThumbnail"), a verification step, and a concrete fix direction ("Use `.task` with async loading, store in @State"), and exclusions are given as literal glob paths. Not 5 because fixes are stated as one-line directions rather than the copy-paste code examples the output format itself demands ("Fix: Code example showing the fix"), leaving the agent to author each fix from scratch. | 4 / 5 |
Workflow Clarity | Five phases are explicitly sequenced (map hierarchy → detect anti-patterns → reason about context → cross-reference → score), with a required intermediate artifact ("Present this map in the output before proceeding"), per-match validation ("use Read to verify the surrounding context before reporting"), a false-positive rejection list, and output limits for overflow. This matches the anchor for a clear sequence with explicit validation steps and error-recovery feedback loops; the read-verify-discard loop on every grep hit is exactly that. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned by phase and severity, and cross-skill references ("axiom-swiftui (skills/layout-ref.md)", "axiom-performance (skills/memory-debugging.md)") are clearly signaled in a Related section. No bundle files exist, so all content lives inline; not 5 because ~280 lines of 11 detection patterns plus context tables is a natural candidate for splitting into a patterns reference file, and the external skill references are not resolvable within this skill's bundle; not 3 because what is inline is well-organized and navigable rather than a wall of text. | 4 / 5 |
Total | 17 / 20 Passed |