App Intents Code Review

Reviews App Intents code for intent structure, entities, shortcuts, and parameters. Use when reviewing code with import AppIntents, @AppIntent, AppEntity, AppShortcutsProvider, or @Parameter. Use when this capability is needed.

tomevault-io 1f0362a 2 files · 3.1 KB Updated Sep 17, 2026

Coding & Dev Tools

Install

Press C to copy

$ npx skillmds@latest add tomevault-io/app-intents-code-review

Installs into ~/.claude/skills (or .claude/skills inside a project). Restart Claude Code to pick it up.

＋ Add to collection

README badge

[![SkillMD](https://skillmd.com/badge/tomevault-io/app-intents-code-review.svg)](https://skillmd.com/skills/tomevault-io/app-intents-code-review)

Trust & safety

2 of 3 audits passed

Last audited Sep 18, 2026 · commit 1f0362a

Works with: Claude Code Claude.ai OpenAI Codex +

File contents

App Intents Code Review

Quick Reference

Issue Type	Reference
AppIntent protocol, perform(), return types	references/intent-structure.md
AppEntity, EntityQuery, identifiers	references/entities.md
AppShortcutsProvider, phrases, discovery	references/shortcuts.md
@Parameter, validation, dynamic options	references/parameters.md

Review Checklist

perform() marked with @MainActor if accessing UI/main thread resources
perform() completes within 30-second timeout (no heavy downloads/processing)
Custom errors conform to CustomLocalizedStringResourceConvertible
EntityQuery.entities(for:) handles missing identifiers gracefully
EntityStringQuery used if Siri voice input needed (not plain EntityQuery)
suggestedEntities() returns reasonable defaults for disambiguation
AppShortcut phrases include .applicationName parameter
Non-optional @Parameter has sensible defaults or uses requestValue()
@IntentParameterDependency not used on iOS 16 targets (crashes)
Phrases localized in AppShortcuts.strings, not Localizable.strings
App Intents defined in app bundle, not Swift Package (pre-iOS 17)
isDiscoverable = false for internal/widget-only intents

When to Load References

AppIntent protocol implementation -> intent-structure.md
Entity queries, identifiers, Spotlight -> entities.md
App Shortcuts, phrases, discovery -> shortcuts.md
Parameter validation, dynamic options -> parameters.md

Review Questions

Does perform() handle timeout limits for long-running operations?
Are entity queries self-contained (no @Dependency injection in Siri context)?
Do phrases read naturally and include the app name?
Are SwiftData models passed by persistentModelID, not directly?
Would migrating from SiriKit break existing user shortcuts?

Converted and distributed by TomeVault — claim your Tome and manage your conversions.

---name: app-intents-code-reviewdescription: Reviews App Intents code for intent structure, entities, shortcuts, and parameters. Use when reviewing code with import AppIntents, @AppIntent, AppEntity, AppShortcutsProvider, or @Parameter. Use when this capability is needed.---# App Intents Code Review## Quick Reference| Issue Type | Reference ||------------|-----------|| AppIntent protocol, perform(), return types | [references/intent-structure.md](references/intent-structure.md) || AppEntity, EntityQuery, identifiers | [references/entities.md](references/entities.md) || AppShortcutsProvider, phrases, discovery | [references/shortcuts.md](references/shortcuts.md) || @Parameter, validation, dynamic options | [references/parameters.md](references/parameters.md) |## Review Checklist- [ ] `perform()` marked with `@MainActor` if accessing UI/main thread resources- [ ] `perform()` completes within 30-second timeout (no heavy downloads/processing)- [ ] Custom errors conform to `CustomLocalizedStringResourceConvertible`- [ ] `EntityQuery.entities(for:)` handles missing identifiers gracefully- [ ] `EntityStringQuery` used if Siri voice input needed (not plain `EntityQuery`)- [ ] `suggestedEntities()` returns reasonable defaults for disambiguation- [ ] `AppShortcut` phrases include `.applicationName` parameter- [ ] Non-optional `@Parameter` has sensible defaults or uses `requestValue()`- [ ] `@IntentParameterDependency` not used on iOS 16 targets (crashes)- [ ] Phrases localized in `AppShortcuts.strings`, not `Localizable.strings`- [ ] App Intents defined in app bundle, not Swift Package (pre-iOS 17)- [ ] `isDiscoverable = false` for internal/widget-only intents## When to Load References- AppIntent protocol implementation -> intent-structure.md- Entity queries, identifiers, Spotlight -> entities.md- App Shortcuts, phrases, discovery -> shortcuts.md- Parameter validation, dynamic options -> parameters.md## Review Questions1. Does `perform()` handle timeout limits for long-running operations?2. Are entity queries self-contained (no `@Dependency` injection in Siri context)?3. Do phrases read naturally and include the app name?4. Are SwiftData models passed by `persistentModelID`, not directly?5. Would migrating from SiriKit break existing user shortcuts?---> Converted and distributed by [TomeVault](https://tomevault.io/claim/existential-birds) — claim your Tome and manage your conversions.

tomevault-io/skills-registry/tree/main/existential-birds--beagle--app-intents-code-review commit 1f0362adb3

Frequently asked questions

npx skillmds@latest add tomevault-io/app-intents-code-review