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-organized, dense instructional skill: every rule is a specific directive rather than background explanation, and the workflow is clearly sequenced with a build-validation habit. The body would be slightly stronger if the Checklist and Common Smells sections were consolidated and the build checkpoint were made an explicit workflow step with a fix-and-retry loop.
Suggestions
Embed the validation loop directly in the Workflow (e.g., 'Build after each major split; if errors appear, fix locally before continuing') rather than leaving it only in the Refactor Checklist.
Merge or trim the overlapping 'Refactor Checklist' and 'Common Smells' sections, which largely restate the Core Guidelines, to reduce redundancy.
Add one short before/after code sketch of a bloated root view versus the split result to ground guidelines 2b and 3 concretely.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean, assumes Claude's competence (never explains what SwiftUI or AppKit is), and every guideline is actionable prose, but "Common Smells" and "Refactor Checklist" partially restate the Core Guidelines, leaving minor tightening opportunities. | 4 / 5 |
Actionability | Guidance is concrete and executable: an exact member-ordering list, specific file paths (App/<AppName>App.swift, Views/SidebarView.swift, Stores/, Services/), a numeric threshold ("roughly under 50 lines"), and named property wrappers. Minor gaps remain — e.g., no illustrative before/after snippet for the worst smells. | 4 / 5 |
Workflow Clarity | The 7-step Workflow is clearly sequenced and includes a behavior-preservation guard (step 7), and the Checklist contains a validation checkpoint ("Build after each major split so compile errors stay local"), but that build-verify loop is not embedded as an explicit workflow step, so it falls short of anchor 5's feedback-loop structure. | 4 / 5 |
Progressive Disclosure | No bundle files exist and the single-file body is well-sectioned (Overview, Core Guidelines, Workflow, Checklist, Smells, Notes), so nothing that belongs in a separate file is inlined. It misses anchor 5 because at ~105 lines it exceeds the simple-skill exception and carries minor redundancy across sections. | 4 / 5 |
Total | 16 / 20 Passed |