Skip to content

Restart keyboard capture after Input Monitoring grant - #87

Merged
aurorascharff merged 1 commit into
mainfrom
codex/fix-input-monitoring-listener
Oct 5, 2026
Merged

aurorascharff merged 1 commit into
mainfrom
codex/fix-input-monitoring-listener

Conversation

@aurorascharff

Copy link
Copy Markdown
Owner

Summary

  • detect when Input Monitoring becomes trusted after ClickLight returns to the foreground
  • recreate the click and keyboard listeners so live shortcuts start immediately
  • clarify that only Accessibility permission still requires reopening the app

Validation

  • ./build-app.sh

Fixes #85

Recreate ClickLight's event listeners when newly granted Input Monitoring access is detected after the app becomes active again.\n\nFixes #85
Copilot AI balanced review requested due to automatic review settings October 5, 2026 14:01
@vercel

vercel Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
click-light Ready Ready Preview Oct 5, 2026 2:01pm UTC

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 review overview

🟢 Approval recommended

The focused changes correctly recreate listeners after authorization without affecting disabled capture.

Review effort: Balanced
Findings: None

What changed in this PR

Restarts capture listeners after newly granted Input Monitoring permission is detected.

Changes:

  • Tracks Input Monitoring trust state on activation.
  • Adds listener restart support.
  • Clarifies permission restart requirements.
File Description
Sources/​ClickLight/​AppDelegate.swift Detects newly granted permission and restarts capture.
Sources/​ClickLight/​ClickCaptureController.swift Adds conditional listener restart.
README.md Documents permission behavior.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@aurorascharff
aurorascharff merged commit b46c831 into main Oct 5, 2026
6 checks passed
@aurorascharff
aurorascharff deleted the codex/fix-input-monitoring-listener branch October 5, 2026 14:04

This branch was successfully deployed

1 active deployment
Preview — fd1a3d4d Deployed Oct 5, 2026 by vercel[bot]
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.

[Bug]: Show Live Keyboard Shortcuts not working

2 participants