Content
77%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: clearly phased, verification-heavy, and highly actionable, with strong false-positive guards and a defined output contract. The main gaps are the total absence of file-level content splitting in a 273-line document and a few prose-only search instructions that are not directly executable.
Suggestions
Move the Phase 2 pattern catalog details and/or the Field Crash Correlation section into a references/ file (e.g. references/patterns.md), keeping SKILL.md as a lean overview of the five phases with one-level-deep, clearly signaled pointers.
Convert prose search instructions into exact grep patterns or explicitly mark them as judgment-based checks — e.g. replace "Task { stored in properties" and "Classes without clear deallocation point" with concrete regexes like "var \w+: Task<" and a Read-based inspection step.
Trim the redundant "## Scope" and "Invocation Examples" sections (or fold them into one short paragraph), since both restate the frontmatter description and trigger conditions already covered elsewhere.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and prescriptive — grep patterns, verification commands, severity tables — and assumes Claude's competence rather than explaining what memory leaks or retain cycles are. Minor over-explanation exists: the "## Scope" section largely restates the frontmatter description, and the Invocation Examples section is partially redundant with the trigger clause. | 4 / 5 |
Actionability | Mostly executable guidance: concrete grep regexes ("Timer\.scheduledTimer.*repeats.*true", "NotificationCenter.default.addObserver"), an exact command ("xcsym crash --format=summary <path-to-ips>"), and specific fixes ("Add `timer?.invalidate()` in `deinit`"). Falls short of 5 because some searches are prose rather than greppable patterns ("Task { stored in properties", "Classes without clear deallocation point") and fixes are one-line descriptions rather than shown code, as its own output template even requests ("Fix: Code example showing the fix"). | 4 / 5 |
Workflow Clarity | Five phases are clearly sequenced with explicit validation checkpoints: pair counting with mandatory contextual verification ("For every grep match, use Read to verify the surrounding context before reporting"), a Phase 1 gate ("Present this map in the output before proceeding"), a false-positives checklist, a cross-referencing phase for compound findings, and output limits for oversized results. This batch-scan workflow has its verification steps fully covered. | 5 / 5 |
Progressive Disclosure | The skill ships as a single 273-line SKILL.md with no bundle files, and the Phase 2 pattern catalog, Field Crash Correlation table, and output templates are substantial content that could live in separate reference files. In-file structure is good (clear headers, well-organized phases), which keeps it above anchor 3's disorganized example, but nothing is split across files and the only file references point to other skills' files ("axiom-performance (skills/memory-debugging.md)"), not to a bundle of its own. | 3 / 5 |
Total | 16 / 20 Passed |