ios-animation-code-review — independently scanned and version-tracked by SaferSkills.
SaferSkills independently audited ios-animation-code-review (Agent Skill) and scored it 100/100 (green). The audit ran 55 deterministic rules across Security, Supply Chain, Maintenance, Transparency, and Community; it found 0 high-severity and 0 lower-severity findings. The full rule-by-rule trace and per-finding evidence are below. Free, methodology-open.
Findings & checks · 0 flagged
Every scanned point with the score it earned and what moved between them.
First recorded scan — no prior version to compare against.
The primary manifest — the file an agent reads to learn what this artifact does.
| Issue Type | Reference |
|---|---|
| Spring parameters, withAnimation misuse, phase/keyframe bugs | references/swiftui-animation-patterns.md |
| Frame drops, offscreen rendering, main thread blocking | references/performance.md |
| Reduce Motion, VoiceOver, motion sensitivity | references/accessibility.md |
| Transition protocol, matchedGeometryEffect, navigation transitions | references/transitions.md |
Complete in order for the files in scope. If a step fails, omit the finding, re-anchor, or downgrade to a question—do not ship accusations without meeting the pass condition.
| Step | What you do | Pass condition (objective) |
|---|---|---|
| 1. Inventory | List each file under review and where animation APIs appear (line ranges or symbol names: withAnimation, .animation, matchedGeometryEffect, PhaseAnimator, UIKit/CA animators, etc.). | A written list exists; files with no animation APIs are explicitly marked out of scope. |
| 2. Anchor | Re-read the cited region in the current file or diff hunk before naming an issue. | Each [FILE:LINE] still shows the behavior; stale line numbers are fixed or the finding is dropped. |
| 3. Evidence | For framework-specific claims (spring curves, Transition conformance, Reduce Motion), cross-check the matching row in Quick Reference against references/*.md. | The finding’s detail names the reference file used, or states inline-only (structural/readability with no framework rule). |
| 4. Report | Emit findings using Output Format. | Headers match [FILE:LINE] ISSUE_TITLE; checklist items below are applied only where gates 1–2 covered that code. |
Report each finding as:
[FILE:LINE] ISSUE_TITLEExample: [AnimatedCard.swift:42] Missing Reduce Motion fallback for spring animation
All details, code suggestions, and rationale follow after the header line.
@Environment(\.accessibilityReduceMotion) checked — animations have Reduce Motion fallback.sensoryFeedback) or audioduration/bounce parameters (not raw mass/stiffness/damping unless UIKit/CA).animation() without value: parameterwithAnimation wraps state changes, not view declarationsmatchedGeometryEffect IDs are stable and unique within the namespacegeometryGroup() used when parent geometry animates with child views appearingPhaseAnimator, symbolEffect) have finite phases or appropriate triggerCATransaction.setAnimationDuration() in UIView-backed layers (use UIView.animate instead)shadowPath (avoids per-frame recalculation).id() modifier (destroys view identity — use transition or matchedGeometryEffect instead)withAnimation scope issues → swiftui-animation-patterns.mdmatchedGeometryEffect glitches or custom Transition bugs → transitions.mdwithAnimation scoped to the minimal state change needed, or is it wrapping unrelated mutations?matchedGeometryEffect — are source and destination using the same ID and namespace, and is only one visible at a time?~30 seconds. Free. No account. Every finding cites a rule and a line of evidence.