Skip to content

Fix event tap recovery and add menu shortcuts - #204

Open
alexj11324 wants to merge 2 commits into
pilotmoon:masterfrom
alexj11324:codex/fix-reverser-stability-shortcuts
Open

Fix event tap recovery and add menu shortcuts#204
alexj11324 wants to merge 2 commits into
pilotmoon:masterfrom
alexj11324:codex/fix-reverser-stability-shortcuts

Conversation

@alexj11324

Copy link
Copy Markdown

Summary

  • fix event-tap recovery under memory pressure by removing per-event permission polling and immediately re-enabling disabled taps
  • bound deferred debug logging and stop logging when the debug window closes
  • add a stable Apple Development-signed Scroll Reverser Dev.app build/install workflow
  • add ⌘, for Settings and ⌘Q for Quit in both the status menu and application main menu
  • rename the Settings localization key and update all 26 supported locales

Root cause

The event-tap callback refreshed permissions for every gesture/scroll event. Under memory pressure, callback work and an unbounded deferred debug-log queue could delay the tap long enough for macOS to disable it.

Validation

  • ./script/build_and_run.sh --verify
  • strict deep code-signing validation for built, staged, and installed app bundles
  • runtime identity: accessibilityTrusted=1, stableLaunchPathMatched=1
  • event-tap recovery probe: disabledThenReenabled=1
  • real UI checks: ⌘, opened Settings; ⌘Q terminated the app
  • all 26 .strings files passed plutil -lint
  • MainMenu.xib passed xmllint and ibtool validation

@alexj11324
alexj11324 marked this pull request as ready for review July 31, 2026 15:17
Copilot AI review requested due to automatic review settings July 31, 2026 15:17

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@pilotmoon

Copy link
Copy Markdown
Owner

Thanks for this -- looks really handy. May explain the instability that is sometimes reported. I'll take a proper look soon.

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.

3 participants