Content
85%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill body is well-structured, highly actionable, and uses progressive disclosure effectively with verified one-level-deep references and explicit validation in its workflows. Its main weakness is conciseness: several prescriptive rules are duplicated across the General Rules and Anti-Patterns sections.
Suggestions
Deduplicate guidance repeated across sections: the sidebar-row density rules, the MenuBarExtra 30-char cap, the WindowGroup(...,id:)-vs-Window launch advice, and the opaque custom sidebar-background warning each appear in both "General Rules" and "Anti-Patterns" — state each once and cross-reference the other section.
Merge or shrink the "Pre-Edit Checklist For New App Scaffolds", which largely restates the "New app scaffolding" bullets, into a short pre-edit gate to avoid restating the same points twice.
Move the detailed sidebar row-density and background-paint guidance out of the inline "General Rules" list into references/split-inspectors.md (already referenced) and keep only the rule statement in the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body gives prescriptive SwiftUI guidance rather than explaining concepts Claude already knows, but several rules are stated twice — sidebar-row density, the 30-char MenuBarExtra cap, WindowGroup-vs-Window launch advice, and opaque custom sidebar backgrounds each appear in both "General Rules" and "Anti-Patterns" — so not every token earns its place. | 2 / 3 |
Actionability | Guidance names exact APIs (@SceneStorage, @AppStorage, @NSApplicationDelegateAdaptor, NSApp.setActivationPolicy(.regular)), concrete commands (git rev-parse --is-inside-work-tree, git init), and a copy-ready App/Views/Models/Stores/Services file layout, which is highly actionable for an instruction skill. | 3 / 3 |
Workflow Clarity | The "Workflow For A New macOS Scene Or View" is a clear 7-step sequence whose final step is an explicit validation checkpoint ("Validate the flow with a build and a quick usability pass"), complemented by a dedicated Pre-Edit Checklist for scaffolds. | 3 / 3 |
Progressive Disclosure | The body is a clear overview with a "Cross-Cutting References" section signaling six one-level-deep reference files (all verified present), using components-index.md as an entry point and deferring detailed examples to references/split-inspectors.md. | 3 / 3 |
Total | 11 / 12 Passed |