ClawKit Logo
ClawKitReliability Toolkit
Back to Registry
Official Verified

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.

skill-install — Terminal

Install via CLI (Recommended)

clawhub install openclaw/skills/skills/anderskev/app-intents-code-review
Or

App Intents Code Review

Quick Reference

Issue TypeReference
AppIntent protocol, perform(), return typesreferences/intent-structure.md
AppEntity, EntityQuery, identifiersreferences/entities.md
AppShortcutsProvider, phrases, discoveryreferences/shortcuts.md
@Parameter, validation, dynamic optionsreferences/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

  1. 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?
Read Full Documentation on GitHub

Metadata

Author@anderskev
Stars4473
Views1
Updated2026-05-01
View Author Profile
AI Skill Finder

Not sure this is the right skill?

Describe what you want to build — we'll match you to the best skill from 16,000+ options.

Find the right skill
Add to Configuration

Paste this into your clawhub.json to enable this plugin.

{
  "plugins": {
    "official-anderskev-app-intents-code-review": {
      "enabled": true,
      "auto_update": true
    }
  }
}
Safety NoteClawKit audits metadata but not runtime behavior. Use with caution.

Related Skills

explanation-docs

Explanation documentation patterns for understanding-oriented content - conceptual guides that explain why things work the way they do

anderskev 4473

tutorial-docs

Tutorial patterns for documentation - learning-oriented guides that teach through guided doing

anderskev 4473

pytest-code-review

Reviews pytest test code for async patterns, fixtures, parametrize, and mocking. Use when reviewing test_*.py files, checking async test functions, fixture usage, or mock patterns.

anderskev 4473

reference-docs

Reference documentation patterns for API and symbol documentation. Use when writing reference docs, API docs, parameter tables, or technical specifications. Triggers on reference docs, API reference, function reference, parameters table, symbol documentation.

anderskev 4473

serde-code-review

Reviews serde serialization code for derive patterns, enum representations, custom implementations, and common serialization bugs. Use when reviewing Rust code that uses serde, serde_json, toml, or any serde-based serialization format. Covers attribute macros, field renaming, and format-specific pitfalls.

anderskev 4473