Content
92%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 strong, dense reference-style body: executable examples for every major SwiftUI animation API, a triage workflow with explicit verification steps, correct progressive disclosure into two deep-dive reference files, and useful common-mistake guidance. The only weakness is minor redundancy across the triage, Common Mistakes, and Review Checklist sections and a slightly long matchedGeometryEffect example.
Suggestions
Collapse the redundant guidance between Triage Step 3, Common Mistakes, and the Review Checklist into one place (e.g., keep the checklist and have Step 3 link to it) to save tokens.
Shorten the matchedGeometryEffect example by hoisting the duplicated withAnimation(.spring(duration: 0.4, bounce: 0.2)) toggle into a single helper or by showing one branch with a comment for the other.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is code-forward with no basic-concept padding (e.g., no explanation of what SwiftUI or springs are), but a few spots could be trimmed: the matchedGeometryEffect example duplicates the wrapped-button structure across both branches, and the same checks (Reduce Motion, expensive closures, single matched-geometry source) are restated across Triage Step 3, Common Mistakes, and the Review Checklist. This fits the 4 anchor (efficient, minor instances that could be trimmed) rather than the 5 anchor (every token earns its place). | 4 / 5 |
Actionability | Nearly every section provides a complete, copy-paste-ready Swift view struct or modifier chain covering the common case (withAnimation with completion, PhaseAnimator with per-phase curves, KeyframeAnimator with multi-track keyframes, navigation zoom pairing, symbolEffect variants, ContentTransition pairing). The few elisions ("// ..." in the PhaseAnimator trigger variant, the placeholder path(in:) in the @Animatable example) reference complete code shown above or are justified flexibility, so the 4 anchor's "minor gaps" does not apply. | 5 / 5 |
Workflow Clarity | The Triage Workflow gives a clear sequence (identify category → choose curve → apply and verify) with explicit validation checkpoints ("Test with Accessibility > Reduce Motion enabled", "Verify no expensive work runs inside animation content closures") and a 7-item Review Checklist for complex reviews. This matches the 5 anchor (clear sequence, explicit validation steps, checklist); feedback loops are not needed since no destructive or batch operations are involved. | 5 / 5 |
Progressive Disclosure | The body keeps one canonical example per topic while exhaustive catalogs are appropriately split into two verified, well-signaled, one-level-deep reference files (animation-advanced.md for CustomAnimation/Spring variants/Transition types, core-animation-bridge.md for CA bridging), with in-context pointers like "Read references/core-animation-bridge.md before advising" and a References section. Referenced anchors match real headings in the bundle files. This matches the 5 anchor (clear overview, well-signaled one-level-deep references, appropriate split) rather than 4. | 5 / 5 |
Total | 19 / 20 Passed |