Skip to content

fix(appintents): stable IDs, Shortcut auto‑activate #7416

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

fix(appintents): stable IDs, Shortcut auto‑activate #7416

wants to merge 3 commits into from
+27 −11

Conversation

@peterneutron
Copy link

Fixes #7412.

osy
osy requested changes Sep 17, 2025
View reviewed changes
@osy osy mentioned this pull request Sep 17, 2025
Open
@peterneutron
Copy link
Author

peterneutron commented Sep 18, 2025 •
edited
Loading

There seem to be more fundamental problems with the current state of AppIntents, I can confirm the above issue #7417 on my end too. I propose changing this into a draft, wdyt?

@peterneutron
fix(appintents): stable IDs, Shortcut auto‑activate, Spotlight VM ind…
1518ae3 
…exing

  - Prevent ActionNotFound by adding static action IDs
  - Activate/foreground UTM when starting a VM via Shortcuts
  - Index VM entities (IndexedEntity) and reindex on launch/changes for Spotlight free‑text resolution

feat(Intents): activate UTM before starting VM via Shortcut

feat(spotlight): index VM entities (IndexedEntity) and reindex on launch/changes to fix Spotlight free‑text resolution
@peterneutron peterneutron changed the title fix(appintents): add stable IDs (static var id) for custom actions to resolve Shortcuts ActionNotFound fix(appintents): stable IDs, Shortcut auto‑activate, Spotlight VM indexing Sep 18, 2025
@peterneutron
Copy link
Author

@osy Here’s a quick summary of what I stacked on top of the OG commit:

  • Kept the stable static var id fix to stop ActionNotFound.
  • Start UTM when starting a VM via Shortcuts (macOS), so runs don’t time out when UTM isn't running.
  • Made VMs discoverable in Spotlight: UTMVirtualMachineEntity now adopts IndexedEntity with a small attribute set.
  • Reindex VMs on app launch and whenever the VM list changes; this fixes Spotlight’s free‑text parameter conversion (e.g. typing “VMName” now resolves).
  • No Core Spotlight fallback, this targets macOS 15+/iOS 18+ only.

This is all preliminary and subject to more rigorous testing of course.

@osy
Copy link
Contributor

osy commented Sep 19, 2025

Please separate the Core Spotlight changes into a different PR. I am ready to merge the id fix but the Spotlight fix will take more consideration. This implementation will impact performance from my initial thoughts. I would want the Spotlight index to only be changed during a VM save.

@peterneutron
revert(spotlight): remove Spotlight indexing from main PR branch; kee…
f5eac52 
…p only AppIntents ID fix and auto-activate on Start (macOS)
@peterneutron peterneutron changed the title fix(appintents): stable IDs, Shortcut auto‑activate, Spotlight VM indexing fix(appintents): stable IDs, Shortcut auto‑activate Sep 20, 2025
@peterneutron
Copy link
Author

Done. I also noticed a typo that's out of scope for this PR as it touches files across the codebase, Disposible should be Disposable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@osy osy osy requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Shortcuts: custom AppIntents fail with LN…ActionNotFound

2 participants

@peterneutron @osy