Content
78%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, actionability-focused skill: sections are organized by concern, exact API modifiers are named, concrete code is properly offloaded to a real one-level reference, and the workflow ends with a verification step and checklist. Weaknesses are mild: some guidance is repeated across sections/guardrails, and the reference snippets rely on undefined helper functions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes competence — no padding explaining what SwiftUI or windows are, and bullets are concrete directives. Minor trimming is possible: the Review Checklist and Guardrails restate several section bullets (e.g., drag-region and restoration rules appear three times), and a few soft-advisory lines ("Keep one clear path back to regular window management...") could be tightened, keeping it below the lean 5 anchor. | 4 / 5 |
Actionability | Guidance is concrete throughout, naming exact modifiers (`.toolbar(removing: .title)`, `.windowMinimizeBehavior(.disabled)`, `.defaultWindowPlacement { content, context in ... }`, `content.sizeThatFits(.unspecified)`, `context.defaultDisplay.visibleRect`) and deferring executable examples to the reference file. Not a 5 because the reference snippets call undefined helpers (`clampToDisplay`, `zoomToFit`, `centeredPosition`), so the code is not fully copy-paste ready. | 4 / 5 |
Workflow Clarity | The 7-step workflow is clearly sequenced, includes an explicit verification checkpoint ("Build and launch the app with `build-run-debug` to verify the result"), a conditional fallback (step 7 → `appkit-interop`), and a Review Checklist. It stops short of the 5 anchor because there is no explicit fix-and-retry loop if verification reveals a problem (e.g., what to check when drag or placement misbehaves). | 4 / 5 |
Progressive Disclosure | The body is a well-sectioned overview and all executable code lives in one clearly signaled, one-level-deep reference ("For concrete window modifier examples, read `references/api-snippets.md`") that exists in the bundle. No nested references and no inline API dump, matching the clear-split 5 anchor. | 5 / 5 |
Total | 17 / 20 Passed |