Skip to content

Further tweaks to process checks#3090

Merged
pjrobertson merged 1 commit intomainfrom
pjrobertson/process-tweaks
Jan 30, 2026
Merged

Further tweaks to process checks#3090
pjrobertson merged 1 commit intomainfrom
pjrobertson/process-tweaks

Conversation

@pjrobertson
Copy link
Member

This adds a few extra tweaks to the previous PR. We don't only want to show NSBundle type apps (because command line apps are legitimate 'foreground apps' - especially if they have GUIs. But we also don't want to show background apps (esp the safari xpc apps). So I've added an additional check for XPC here and removed the previous check that only allowed NSBundles through.

shows HelloQS doesn't show safari XPC apps
Screen Region 2026-01-30 at 16 42 27 Screen Region 2026-01-30 at 16 42 51

@n8henrie
Copy link
Member

Seems reasonable. It's possible someone out there is using QS to relaunch hung XPC processes or kill Microsoft / Chrome AutoUpdater or what have you, but seems pretty unlikely; I don't see any issues mentioning "xpc" that don't seem to be about crashes or hangs, so this seems like a net win.

If there are complaints, I imagine we could add a setting to reenable this, but I think the default should be off (80/20 rule).

Thanks as always for quick action!

@pjrobertson
Copy link
Member Author

Thanks! Sounds good. I think if people really want to relaunch XPC apps, then they can just enable 'show background processes'. No need for an extra setting ;-)

@pjrobertson pjrobertson merged commit 537259c into main Jan 30, 2026
2 checks passed
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.

2 participants