fix: resolve 4 CodeQL alerts

[laurentiu021]

Summary

Fixes 4 actionable CodeQL alerts in source code. Remaining 26 alerts are in generated code (obj/Release/*.g.cs) and 1 P/Invoke call (acceptable).

Alerts fixed

• Uninstaller - improve the uninstall capabilityies in order to uninstall also local applications #236 cs/catch-of-all-exceptions — PingMonitorService.Stop() bare catch → AggregateException + ObjectDisposedException
• Speed test, save last speedtest scans under each scan type in order to have visibility of service degradation #237 cs/catch-of-all-exceptions — TracerouteMonitorService.Stop() same fix
• About SysManager - adding option to redirect the user to the git hub change list in order to have full visibility #232, Drivers - adding option to hide windows specific drivers / system drivers #234 cs/simplifiable-boolean-expression — BoolInverterConverter simplified to pattern matching
• SysManager 0.35.8 released #231 cs/cast-from-abstract-to-concrete-collection — LogsViewModel safe as-cast with fallback

Remaining alerts (26)

All in generated code (RegexGenerator.g.cs, *.g.cs view files) — cannot be fixed in source.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...sManager/SysManager/Services/PingMonitorService.cs	0.00%	3 Missing ⚠️
...er/SysManager/Services/TracerouteMonitorService.cs	0.00%	3 Missing ⚠️
...r/SysManager/Helpers/OutputKindToBrushConverter.cs	0.00%	2 Missing ⚠️
SysManager/SysManager/ViewModels/LogsViewModel.cs	0.00%	0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR consolidates five small defensive coding improvements: background task shutdown methods now handle specific exceptions instead of broad catch-all blocks, a boolean converter is refactored to pattern matching, a view model cast is made safe with fallback logic, and all changes are documented in the changelog.

Changes

Code Robustness Improvements

Layer / File(s)	Summary
Service shutdown exception handling refinement SysManager/SysManager/Services/PingMonitorService.cs, SysManager/SysManager/Services/TracerouteMonitorService.cs	Stop() methods in both services replace broad exception swallowing with targeted handling for AggregateException and ObjectDisposedException, with bounded timeouts (1500ms and 3000ms respectively).
Boolean converter logic refactoring SysManager/SysManager/Helpers/OutputKindToBrushConverter.cs	BoolInverterConverter's Convert and ConvertBack methods are simplified from conditional bool checking to direct pattern matching using value is not true.
View model safe casting for collection view SysManager/SysManager/ViewModels/LogsViewModel.cs	UpdateVisibleCount replaces an unsafe direct cast ((CollectionView)EntriesView).Count with safe as-casting and a fallback count operation via EntriesView.Cast<object>().Count().
Release notes documentation CHANGELOG.md	New 0.48.6 release section (2026-05-14) documents fixes for both services, the boolean converter, and the view model casting improvement.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Five little fixes, all snug and tight,
Exception handling done just right,
Casting safely, patterns clean,
Changelog records what was seen,
Robustness hopping toward the light! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: resolve 4 CodeQL alerts' directly and accurately describes the primary change—fixing four CodeQL security alerts across multiple files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/codeql-catch-all

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}