Content
86%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-built reference skill: executable examples, concrete rules, correct progressive disclosure with all referenced files present and real. The body is mostly lean for the breadth covered; its only real flaws are minor duplication between the SwiftUI Essentials and Apple Concurrency Patterns sections.
Suggestions
Remove the duplicated guidance: the SettingsViewModel example in Apple Concurrency Patterns repeats the UserViewModel pattern from SwiftUI Essentials, and the '.task cancels automatically' rule appears in both sections plus the mistakes table — keep one canonical statement and cross-reference it.
Consider consolidating the scattered decision rules (greenfield vs. legacy, check deployment target, respect existing architecture) into one short ordered checklist at the top so the application sequence is explicit.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, rule-driven, and assumes Claude's competence — no padding or explanations of basic concepts. Minor trimming opportunities keep it below the lean anchor: the `.task`-cancels rule appears twice (SwiftUI Essentials and Apple Concurrency Patterns) and again in the mistakes table, and the short `@MainActor @Observable` ViewModel example in Apple Concurrency Patterns substantially repeats the UserViewModel example in SwiftUI Essentials. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready Swift for the common cases: design tokens and theming, `@Observable` ViewModel + View, the complete MVVM pattern including dependency injection, and a realistic TCA alternative. Concrete rules ("never raw values", "body exceeds ~30 lines", "suffix `View`") and a mistakes table with specific fixes round it out. | 5 / 5 |
Workflow Clarity | Decision guidance is clear and ordered — check project context first (deployment target, Swift version, existing architecture), then greenfield vs. legacy rules ("existing UIKit screens should not be partially rewritten"), then per-platform reference routing via tables. It stays at 4 rather than 5 because the application flow is distributed across rule bullets rather than given as an explicit ordered sequence with checkpoints, and the duplicated `.task` guidance slightly muddies the rules. | 4 / 5 |
Progressive Disclosure | Exemplary structure: SKILL.md is a true overview, all 20 referenced files exist on disk with substantial content (verified), each is one level deep with a one-line scope summary, and the platform/cross-platform tables plus inline pointers (e.g., "See `references/localization.md` for String Catalog patterns") make navigation easy. | 5 / 5 |
Total | 18 / 20 Passed |