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, actionable patterns skill with strong progressive disclosure and concrete guidance backed by a verified reference bundle. The main weakness is cross-section redundancy that inflates length without adding information.
Suggestions
Consolidate the repeated sidebar opaque-background, Window-vs-WindowGroup launch, and 30-char menu-cap guidance into one canonical location (e.g., General Rules) and have Anti-Patterns reference it rather than restating it, to reduce redundancy and improve conciseness.
Add an explicit error-recovery feedback loop to the scene/view workflow (e.g., 'If the build or usability pass fails, fix the reported issue and re-run before proceeding') to push workflow_clarity to the top anchor.
Include one short assembled example in the body or in a referenced file (e.g., a minimal WindowGroup+Settings+MenuBarExtra App scaffold) to close the actionability gap where only individual APIs are named but no complete usage is shown.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The guidance is genuinely opinionated and beyond what Claude already knows, but several points are restated across sections (sidebar opaque-background guidance, Window-vs-WindowGroup launch behavior, and the 30-char menu cap each appear two to three times in Quick Start, General Rules, and Anti-Patterns), so it could be tightened. | 3 / 5 |
Actionability | Concrete and executable guidance throughout — exact commands ("git rev-parse --is-inside-work-tree", "git init"), specific APIs ("NSApp.setActivationPolicy(.regular)", "@NSApplicationDelegateAdaptor"), a 30-character threshold, and a State Ownership decision table — with the minor gap that no fully assembled SwiftUI scene example appears inline (deferred to references). | 4 / 5 |
Workflow Clarity | A clear 7-step "Workflow For A New macOS Scene Or View" plus a "Pre-Edit Checklist" and an explicit validation step ("Validate the flow with a build and a quick usability pass") are present; the gap is the absence of an explicit error-recovery feedback loop (validate -> fix -> re-validate). | 4 / 5 |
Progressive Disclosure | The body is a well-organized overview with a dedicated "Cross-Cutting References" section and inline pointers to six verified, one-level-deep reference files (components-index, windowing, settings, commands-menus, split-inspectors, menu-bar-extra), with detail appropriately split out and easy navigation. | 5 / 5 |
Total | 16 / 20 Passed |