WidgetKit Code Review
Quick Reference
| Issue Type | Reference | |------------|-----------| | TimelineProvider, entries, reload policies | references/timeline.md | | Widget families, containerBackground, deep linking | references/views.md | | AppIntentConfiguration, EntityQuery, @Parameter | references/intents.md | | Refresh budget, memory limits, caching | references/performance.md |
Review Checklist
- [ ]
placeholder(in:)returns immediately without async work - [ ] Timeline entries spaced at least 5 minutes apart
- [ ]
getSnapshotcheckscontext.isPreviewfor gallery previews - [ ]
containerBackground(for:)used for iOS 17+ compatibility - [ ]
widgetURLused for systemSmall (not Link) - [ ] No Button views (use Link or widgetURL)
- [ ] No AsyncImage or UIViewRepresentable in widget views
- [ ] Images downsampled to widget display size (~30MB limit)
- [ ] App Groups configured for data sharing between app and widget
- [ ] EntityQuery implements
defaultResult()for non-optional parameters - [ ] New intent parameters handle nil for existing widgets after updates
- [ ]
reloadTimelinescalled strategically (not on every data change)
When to Load References
- TimelineProvider implementation or refresh issues -> timeline.md
- Widget sizes, Lock Screen, containerBackground -> views.md
- Configurable widgets, AppIntent migration -> intents.md
- Memory issues, caching, budget management -> performance.md
Review Questions
- Does the widget provide fallback entries for when system delays refresh?
- Are Lock Screen families (accessoryCircular/Rectangular/Inline) handled appropriately?
- Would migrating from IntentConfiguration break existing user widgets?
- Is timeline populated with future entries or does it rely on frequent refreshes?
- Is data cached via App Groups for widget access?
Hard gates (before reporting)
Complete in order for each finding you intend to report. Do not advance until the pass condition is satisfied.
- Location artifact — The finding includes
[FILE:LINE](or a line range) copied from the current file contents; the path resolves in this repo. - Scope read — You read the full surrounding implementation: the
TimelineProvider(includingplaceholder,getSnapshot, andgetTimelinewhen relevant), the@mainWidget/ widget bundle, or the configurable widget’sAppIntentConfiguration/ intent types—not only a diff hunk or snippet. - Platform or system claim (only if the finding depends on refresh budget, ~30MB memory guidance, Lock Screen accessory families, iOS 17+
containerBackground, App Groups data sharing, or migration fromIntentConfigurationtoAppIntentConfiguration) — You name one concrete artifact you inspected (for example.entitlements/ App Group id in project,WidgetFamilyhandling in source,IPHONEOS_DEPLOYMENT_TARGET, or the exact reference subsection you used) or you drop or downgrade the finding to an open question. - Protocol — Pre-report steps in review-verification-protocol are satisfied for this item (no finding if they are not).
Use the issue format [FILE:LINE] ISSUE_TITLE for each reported finding. Hard gate 4 is the full pre-report checklist for this skill’s review type.