Skip to content

[ISSUE-495]: Fix access data permission prompt and Core Data crash during setup#500

Merged
ptsochantaris merged 2 commits intoptsochantaris:mainfrom
luca-borrione:lb/495-access-data-permission
Aug 23, 2025
Merged

[ISSUE-495]: Fix access data permission prompt and Core Data crash during setup#500
ptsochantaris merged 2 commits intoptsochantaris:mainfrom
luca-borrione:lb/495-access-data-permission

Conversation

@luca-borrione
Copy link
Copy Markdown
Contributor

@luca-borrione luca-borrione commented Aug 18, 2025

Fixes #495

Primary Fix: Access Data Permission Prompt

The "access data" permission prompt was appearing because:

  • Trailer uses Apple Events to communicate between the main app and its launcher helper
  • macOS requires explicit permission declarations for Apple Events access

Without proper NSAppleEventsUsageDescription entries, the system shows a generic "access data" prompt.
These changes provide clear, user-friendly explanations for why the app needs Apple Events access, replacing the confusing generic permission prompt with proper context.

The launcher also needs com.apple.security.automation.apple-events entitlement for automation permissions

Secondary Fix: Core Data Crash in SetupAssistant

While testing the permission fix, discovered and fixed a crash in SetupAssistant.swift that occurred during server setup:

The app crashed with assert(!objectID.isTemporaryID) when trying to set auth tokens on newly created ApiServer objects.

Solution:

Ensures the server has a permanent Core Data object ID before setting the auth token, preventing the assertion failure.

Test

Started from Trailer 1.8.14 (current main a9d2353) on Mac Sequoia 15.6

Copy link
Copy Markdown
Owner

@ptsochantaris ptsochantaris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for this! That permission requirement was extremely well hidden, as a Trailer user I'm definitely grateful for not having that noise anymore.

@ptsochantaris ptsochantaris merged commit 6f8f023 into ptsochantaris:main Aug 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trailer asks to access data from other apps

2 participants