Skip to content

#2251: add uninstall support for global tools windows - #2301

Open
cap-juan wants to merge 2 commits into
devonfw:mainfrom
cap-juan:feature/2251-uninstall-global-tools-windows
Open

#2251: add uninstall support for global tools windows#2301
cap-juan wants to merge 2 commits into
devonfw:mainfrom
cap-juan:feature/2251-uninstall-global-tools-windows

Conversation

@cap-juan

@cap-juan cap-juan commented Aug 10, 2026

Copy link
Copy Markdown

This PR fixes #2251

Implemented changes:

  • Use getWindowsRegistryAppName() instead of the generic tool name when uninstalling global tools on Windows.
  • This ensures that the same registry application name used for installation detection is also used to locate the correct uninstall entry.

Testing instructions

  1. Build IDEasy locally while executing ./build-local-dev.sh.
  2. Install a supported global tool on Windows e.g. ide install pgAdmin.
  3. Run ide -d uninstall pgadmin.
  4. Verify that IDEasy detects the installed pgAdmin application via the Windows registry and starts the uninstall process instead of reporting "Couldn't uninstall pgadmin because we could not find an installation"

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat and not feature/921 fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summaries what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labelled
    with internal
  • You have not changed any dependency in pom.xml files or otherwise if runtime dependencies changed, you have updated our LICENSE.asciidoc
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

Checklist for tool commandlets

Have you added a new «tool» as commandlet? There are the following additional checks:

  • The tool can be installed automatically (during setup via settings) or via the commandlet call
  • The tool is isolated in its IDEasy project, see Sandbox Principle
  • The new tool is added to the table of tools in LICENSE.asciidoc
  • The new commandlet is a command-wrapper for «tool»
  • Proper help texts for all supported languages are added here
  • The new commandlet installs potential dependencies automatically
  • The variables «TOOL»_VERSION and «TOOL»_EDITION are honored by your commandlet
  • The new commandlet is tested on all platforms it is available for or tested on all platforms that are in scope of the linked issue

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board Aug 10, 2026
@cap-juan
cap-juan force-pushed the feature/2251-uninstall-global-tools-windows branch 2 times, most recently from 86cac90 to 15b6180 Compare August 10, 2026 12:52
@cap-juan cap-juan changed the title Feature/2251 uninstall global tools windows Feature/2251 add uninstall support for global tools windows Aug 10, 2026
@cap-juan cap-juan moved this from 🆕 New to Team Review in IDEasy board Aug 10, 2026
@cap-juan cap-juan self-assigned this Aug 10, 2026
@cap-juan cap-juan added enhancement New feature or request windows specific for Microsoft Windows OS uninstall uninstall tools or IDEasy itself labels Aug 10, 2026
@cap-juan
cap-juan force-pushed the feature/2251-uninstall-global-tools-windows branch from 15b6180 to cbae6d3 Compare August 10, 2026 12:58
@coveralls

coveralls commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 31470005434

Coverage decreased (-0.01%) to 72.87%

Details

  • Coverage decreased (-0.01%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 17 coverage regressions across 3 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

17 previously-covered lines in 3 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/os/WindowsHelperImpl.java 14 72.8%
com/devonfw/tools/ide/tool/GlobalToolCommandlet.java 2 8.57%
com/devonfw/tools/ide/version/VersionSegment.java 1 90.55%

Coverage Stats

Coverage Status
Relevant Lines: 17563
Covered Lines: 13349
Line Coverage: 76.01%
Relevant Branches: 7763
Covered Branches: 5106
Branch Coverage: 65.77%
Branches in Coverage %: Yes
Coverage Strength: 3.23 hits per line

💛 - Coveralls

@hohwille hohwille changed the title Feature/2251 add uninstall support for global tools windows #2251: add uninstall support for global tools windows Aug 11, 2026
@cap-juan
cap-juan force-pushed the feature/2251-uninstall-global-tools-windows branch from cbae6d3 to bab534c Compare August 11, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request uninstall uninstall tools or IDEasy itself windows specific for Microsoft Windows OS

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Provide generic uninstall support for globally installed tools (Windows)

2 participants