Skip to content

Reduce the Accessibility timeout when fetching all windows. Closes #219 - #222

Merged
cmsj merged 1 commit into
mainfrom
allWindows-ax-timeout
Sep 12, 2026
Merged

cmsj merged 1 commit into
mainfrom
allWindows-ax-timeout

Conversation

@cmsj

@cmsj cmsj commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Closes #219

@cmsj cmsj self-assigned this Sep 12, 2026
@greptile-apps

greptile-apps Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Temporarily reduces the Accessibility messaging timeout to 250 ms while fetching all windows, limiting delays from unresponsive applications.

  • Restores the default timeout when enumeration exits.
  • Preserves existing permission checks, application filtering, and query-error handling.
  • No actionable issues identified.

Confidence Score: 5/5

This PR appears safe to merge.

The timeout is scoped to synchronous window enumeration and restored on exit; no blocking behavioral regressions or repository-rule violations were identified.

Important Files Changed

Filename Overview
Hammerspoon 2/Modules/hs.window/HSWindowModule.swift Adds a temporary Accessibility timeout around allWindows enumeration and restores the default with defer; no actionable defects identified.

Reviews (1): Last reviewed commit: "Reduce the Accessibility timeout when fe..." | Re-trigger Greptile

@cmsj
cmsj merged commit 0436c3d into main Sep 12, 2026
3 checks passed
@cmsj
cmsj deleted the allWindows-ax-timeout branch September 12, 2026 20:11
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.

hs.window.allWindows() waits 1.5s for HammerspoonOSAScriptHelper, which has no windows

1 participant