Content
75%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, lean instructional body that assumes Claude's competence and gives unusually concrete file-layout and attribute-level rules. The main gaps are the absence of any illustrative before/after example and validation appearing as a checklist item rather than an explicit step in the workflow with a fix-retry loop.
Suggestions
Add one short before/after SwiftUI snippet under guideline 3 or 4 to make "stabilize the root layout" and "extract subviews" concrete rather than judgment-based.
Move "Build after each major split" into the numbered Workflow as an explicit step with a feedback loop (e.g., "If the build fails, fix locally before the next split") so checkpoints live inside the sequence, not only in the checklist.
Consider moving the directory-layout rules of guideline 2b into a small references/ file (e.g., FILE-LAYOUT.md) and keeping a one-line summary in the body, tightening SKILL.md further.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is directive and dense — bullet-style rules with no padding, no explanation of what SwiftUI or AppKit is, and no restating of Claude's existing knowledge. Minor trimming is possible (the Overview repeats the description, and "Common Smells" partially restates the Core Guidelines), which keeps it at 4 rather than the fully lean 5. | 4 / 5 |
Actionability | As an instruction-only skill it gives concrete, executable guidance: an exact member ordering for files, explicit directory targets ("Views/ContentView.swift", "Models/*.swift", "Stores/*.swift", "Services/*.swift", "Support/*.swift"), and specific attribute rules (@SceneStorage vs @AppStorage, @Observable stored as @State). It stays at 4 because there are no before/after snippets illustrating a split, so a few guidelines (e.g., "stabilize the root layout") rely on judgment rather than a shown pattern. | 4 / 5 |
Workflow Clarity | The Workflow section gives a clear 7-step sequence, and the checklist includes the checkpoint "Build after each major split so compile errors stay local", plus step 7 guards behavior preservation. It misses a 5 because the build checkpoint lives in the checklist rather than the workflow sequence, and there is no explicit error-recovery loop (build fails → fix locally before proceeding). | 4 / 5 |
Progressive Disclosure | The single-file body (no references/, scripts/, or assets/ exist) is well organized into Overview, numbered Core Guidelines, Workflow, Checklist, Smells, and Notes, with pointers to sibling skills ("use macos-swiftui-patterns", "use macos-appkit-interop") for adjacent problems. It fits the 4 anchor — good structure with everything appropriately placed in one file — but at ~105 lines some guideline detail (e.g., the file-layout rules in 2b) could be offloaded to a reference file to reach the clearest overview-plus-references shape. | 4 / 5 |
Total | 16 / 20 Passed |