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.
The body is a strong overview skill: fully executable code for all common MapKit/CoreLocation cases, clear multi-workflow structure, and exemplary progressive disclosure to two verified reference files. The only slack is minor token overhead (TOC and some checklist/reference duplication).
Suggestions
Remove the Contents TOC — the section headings are already short and scannable, and the ~10-line index duplicates them for no navigational gain in a file this size.
Tighten the overlap between Common Mistakes, the Review Checklist, and the reference files (e.g., CLMonitor's 20-condition limit and background-session rules live in mapkit-corelocation-patterns.md) by keeping only the review-facing checklist in SKILL.md and pointing to the reference for the rationale.
Add an inline denial-handling step to workflow 2 (track user location) so the sequence carries its own checkpoint, e.g., '5. On denied authorization, stop the task and present the Settings deep-link view from the Authorization Flow section.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is code-dense and assumes Claude's competence with almost no general-concept padding, but the ~10-line Contents TOC duplicates section headings and Common Mistakes/Review Checklist partially restate material also present in the reference files. Matches the 4 anchor (efficient, minor instances that could be trimmed); not 5 because a few tokens (TOC, checklist overlap) do not earn their place, and not 3 because the padding is minor. | 4 / 5 |
Actionability | Every common case has executable, copy-paste-ready Swift: Map view with markers/controls, Annotation, MapPolyline/Polygon/Circle, MapCameraPosition variants, selection binding, CLLocationUpdate.liveUpdates() tracking task with accuracy filtering, CLServiceSession, authorization-denied Settings UI, CLGeocoder, MKGeocodingRequest, MKLocalSearchCompleter/ MKLocalSearch, MKDirections + route drawing, ETA, and PlaceDescriptor. Matches the 5 anchor; placeholder variables (routeCoords, applePark) are standard illustrative names and coverage of common cases is comprehensive. | 5 / 5 |
Workflow Clarity | Five clearly sequenced numbered workflows cover the distinct task types, and the Review Checklist provides a validation pass for the review workflow. Matches the 4 anchor (clear sequence, most checkpoints present, minor gaps): individual build workflows lack inline verification steps (e.g., workflow 2 does not inline the denial-handling step it covers elsewhere). Not 5 because no explicit validate→fix→retry loop is embedded in the workflows, though none is strictly required since these are not destructive/batch operations. | 4 / 5 |
Progressive Disclosure | Verified against the actual bundle: two well-named reference files one level deep, with every cited anchor (references/mapkit-patterns.md#complete-map-view-setup, #map-in-a-list-or-scrollview, #cycling-directions-ios-14) resolving to real headings. The intro's conditional pointers ('Read X when you need...') clearly signal when to load each file, and the References section annotates both files' scope. Matches the 5 anchor (clear overview, well-signaled one-level-deep references, easy navigation). | 5 / 5 |
Total | 18 / 20 Passed |