-
-
Notifications
You must be signed in to change notification settings - Fork 174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG]: Uninstalling the SecHealthUI fails, despite the app being installed. #195
Comments
Hi @haiku-balls,thank you for the quality issue with all context and good suggestion. I will fix this in a patch release soon. I looked for official sources and did tests to see these service statuses. Here are my results:
So As we have official documentation for both services, the script should attempt to uninstall both services with some special logic:
|
This commit fixes the issue of Windows Security app not being removed in Windows 11. It addresses the problem by extending the app uninstallation process to cover the new app package specific to Windows 11. It improves the overall design of templated functions for store app removal to implement the fix. - Improve Windows Security removal script: - Add support for removing `Microsoft.SecHealthUI` in Windows 11. - Revise script documentation for clarity and correct typos. - Redesign uninstallion of Store apps: - Change `UninstallSystemApp` to `UninstallNonRemovableStoreApp` for wider usage. This change is due to `Microsoft.SecHealthUI` being non-removable yet not a system app. - Refactor app data cleanup into two distinct functions (`ClearStoreAppDataBeforeUninstallation` and `ClearStoreAppDataAfterUninstallation`) for better clarity and maintainability. This also helps in testing by allowing easier reordering of operations. - Seperate between simple non-removable app uninstallation and uninstallation with cleanup in separate functions, highlighting that the latter is more invasive and should be used cautiously. This addresses permission issues encountered with `SecHealthUI` app removal during cleanup on Windows 11. - Separate uninstalling app and uninstalling app with cleanup to different functions, document that cleanup should no longer be prefered as it's invasive and too aggresive. Cleanup logic introduces permission issues/errors for `SecHealthUI` in Windows 11. - Extend app soft-deletion to include the default Windows app folder, this ensures that the cleanup covers any kind of Store apps (not only system apps).
Windows 11 app removal is added in 0.12.9 🚀 |
Description
When toggling the Uninstall Windows Store Apps > Uninstall Sys. Apps > People Hub > Windows Security GUI (Sec Health UI) App, it reports 'Not Installed' despite the app being there. The reason is the app isn't "Microsoft.Windows.SecHealthUI" it's "Microsoft.SecHealthUI". I'm not entirely sure if it's due to me using a development build.
OS
Windows 22H2 (Build 25276.1; Dev)
Reproduction steps
Toggle the option. Run the script with that option, reports 'not installed'.
Scripts
The text was updated successfully, but these errors were encountered: