Content
81%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.
An exemplary router skill: unambiguous, precedence-ordered routing with concrete API-level corrections and strong conflict resolution, held back by presenting the same routing decision three separate ways and by pointing at suite files that are not included in this bundle. The body adds only routing knowledge Claude could not infer, with essentially no concept padding.
Suggestions
Cut one of the two duplicate decision representations — the graphviz digraph or the numbered list — and trim Example Invocations to cases not already covered by the decision tree, which would remove roughly a third of the body without losing any routing information.
Ship the referenced `skills/*.md` files inside the bundle (e.g. under `references/`) or document where they resolve, so the router's destinations — including `skills/resize-auditor.md` from the inline auditor block — are actually reachable.
Merge any Quick Reference rows whose symptom is already an exact decision-tree condition (e.g. the constraint-conflict rows) to reduce the table to symptoms that only it covers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The same routing map is rendered three times — the graphviz "digraph uikit" block, the numbered decision list ("0. Scene-lifecycle migration... → skills/uikit-modernization.md"), and the Example Invocations section — alongside a Quick Reference table. That is roughly 50 lines of duplication, which fits 'mostly efficient but could be tightened' rather than the 'minor trimming' of anchor 4; no concept explanations or padded prose keep it above anchor 2. | 3 / 5 |
Actionability | Routing guidance is exact and executable: literal error strings ("Unable to simultaneously satisfy constraints"), specific APIs ("UIViewRepresentable", "PKStrokeRecognizer", "UINavigationItem.subtitle"), and concrete corrections such as "Persist `drawing.dataRepresentation()`" and "the canvas must `becomeFirstResponder()` after `setVisible(_:forFirstResponder:)`". As an instruction-only router skill its guidance fully covers the common cases, matching the top anchor. | 5 / 5 |
Workflow Clarity | A single-action skill (classify the task, read the mapped file) whose action is unambiguous: ordered precedence rules ("Try uikit FIRST"), explicit conflict-resolution tie-breaks against swiftui/concurrency/performance/axiom-data, and an anti-rationalization table of known wrong turns. Per the simple-skill exception this warrants 5, and no destructive or batch operations exist to impose a validation cap. | 5 / 5 |
Progressive Disclosure | References are one level deep and clearly signaled ("See `skills/uikit-bridging.md`", "See `skills/pencilkit-paperkit-ref.md`"), but the bundle contains no referenced files at all — none of the `skills/*.md` targets or the inline block's `skills/resize-auditor.md` exist in `references/`, `scripts/`, or `assets/`, so the router's destinations cannot be resolved from this bundle. Good structure with an unverifiable target set fits anchor 4; it is not 5 because navigation is not easy when the targets are absent, and not 3 because references are clearly signaled with nothing inlined that belongs elsewhere. | 4 / 5 |
Total | 17 / 20 Passed |