Skip to content

fix(uninstall): include .diag files in diagnostic report cleanup matching#751

Merged
tw93 merged 1 commit into
tw93:mainfrom
PremPrakashCodes:fix/747-add-diag-diagnostic-reports
Apr 16, 2026
Merged

fix(uninstall): include .diag files in diagnostic report cleanup matching#751
tw93 merged 1 commit into
tw93:mainfrom
PremPrakashCodes:fix/747-add-diag-diagnostic-reports

Conversation

@PremPrakashCodes
Copy link
Copy Markdown
Contributor

Summary

Safety Review

  • Does this change affect cleanup, uninstall, optimize, installer, remove, analyze delete, update, or install behavior?
    • Yes. It affects uninstall discovery behavior by expanding recognized diagnostic report extensions during app-related file lookup.
  • Does this change affect path validation, protected directories, symlink handling, sudo boundaries, or release/install integrity?
    • No changes to path validation, protected directories, symlink handling, sudo boundaries, or release/install integrity.
  • If yes, describe the new boundary or risk change clearly.
    • New boundary: uninstall diagnostic-report discovery now includes .diag files in addition to existing extensions. Risk change is low and scoped to matching logic only.

Tests

  • List the automated tests you ran.
    • bats tests/uninstall.bats --filter "get_diagnostic_report_paths_for_app avoids executable prefix collisions"
    • bats tests/uninstall.bats --filter "find_app_files discovers user-level leftovers|get_diagnostic_report_paths_for_app avoids executable prefix collisions"
  • List any manual checks for high-risk paths or destructive flows.
    • No additional manual destructive-flow checks were run for this patch.

Safety-related changes

  • None.

@PremPrakashCodes PremPrakashCodes requested a review from tw93 as a code owner April 15, 2026 15:40
@PremPrakashCodes PremPrakashCodes changed the title Include .diag files in diagnostic report paths fix(uninstall): include .diag files in diagnostic report cleanup matching Apr 15, 2026
@tw93 tw93 merged commit 1f7e0bb into tw93:main Apr 16, 2026
9 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