Skip to content

fix: terminal launch failure (-1743)#69

Open
lynicis wants to merge 2 commits into
andrew-waters:mainfrom
lynicis:fix/issue-64
Open

fix: terminal launch failure (-1743)#69
lynicis wants to merge 2 commits into
andrew-waters:mainfrom
lynicis:fix/issue-64

Conversation

@lynicis

@lynicis lynicis commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes terminal launch failures (-1743 errAEEventNotPermitted) caused by missing Apple Events permissions on macOS Catalina and later. Hardened runtime applications require the com.apple.security.automation.apple-events entitlement and NSAppleEventsUsageDescription string in Info.plist to send AppleEvents to other applications (like Terminal and iTerm2). Since both were missing, macOS automatically denied the AppleEvent request without showing a prompt.

This PR adds:

  1. com.apple.security.automation.apple-events entitlement to Orchard/Orchard.entitlements.
  2. NSAppleEventsUsageDescription key to Orchard/Info.plist with a clear explanation.

Related issues

Closes #64

Screenshots / recording

N/A

Checklist

  • The project builds and runs (Orchard scheme)
  • Tests pass (⌘U / xcodebuild test)
  • I've added an entry to CHANGELOG.md (Added / Changed / Fixed)
  • The change is focused on a single logical concern

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Orchard declares its need to control Terminal through an Apple Events usage description and automation entitlement, with the changelog documenting the related Terminal and iTerm2 launch fix.

Changes

Terminal automation permissions

Layer / File(s) Summary
Terminal automation permission configuration
Orchard/Info.plist, Orchard/Orchard.entitlements, CHANGELOG.md
Adds the NSAppleEventsUsageDescription message, enables com.apple.security.automation.apple-events, and documents the resolved -1743 launch failure.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The entitlement and Info.plist updates address issue #64 by allowing Orchard to send Apple Events to Terminal.
Out of Scope Changes check ✅ Passed The changes stay focused on the Apple Events permission fix and the changelog note.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and concisely describes the main fix for Terminal launch failures caused by error -1743.
Description check ✅ Passed The description matches the template and includes the change summary, related issue, screenshots status, and checklist items.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

Fixes andrew-waters#64 by adding the com.apple.security.automation.apple-events entitlement and NSAppleEventsUsageDescription to Info.plist. This resolves the -1743 (errAEEventNotPermitted) error when trying to launch Terminal or iTerm2 using AppleScript.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@Orchard/Info.plist`:
- Around line 12-13: Update the NSAppleEventsUsageDescription value in
Info.plist to mention both Terminal and iTerm2, or use wording that clearly
covers terminal applications, while preserving the existing explanation of
launching container shells.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 03c4a63d-4bcb-4df1-b137-f18f5e17571e

📥 Commits

Reviewing files that changed from the base of the PR and between f4cd837 and 610ee7a.

📒 Files selected for processing (2)
  • Orchard/Info.plist
  • Orchard/Orchard.entitlements

Comment thread Orchard/Info.plist Outdated
@lynicis lynicis changed the title Fix terminal launch failure (-1743) #64 fix: terminal launch failure (-1743) #64 Jul 12, 2026
@lynicis lynicis changed the title fix: terminal launch failure (-1743) #64 fix: terminal launch failure (-1743) Jul 12, 2026
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.

Terminal to container fail to open

1 participant