Skip to content

#2172: add getIdeMetadataPath() for $IDE_HOME/.ide/<ide>/<workspace>#2175

Open
quando632 wants to merge 5 commits into
devonfw:mainfrom
quando632:feature/2172-ide-metadata-path
Open

#2172: add getIdeMetadataPath() for $IDE_HOME/.ide/<ide>/<workspace>#2175
quando632 wants to merge 5 commits into
devonfw:mainfrom
quando632:feature/2172-ide-metadata-path

Conversation

@quando632

@quando632 quando632 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

This PR fixes #2172 (part of #2142)

Foundation step for moving IDE-specific metadata out of the workspace. No behavior change yet.

Implemented changes:

The follow-up subtasks switch the actual metadata writers onto this getter: #2173 (JetBrains) and #2174 (VSCode).


Testing instructions

  1. cd cli && mvn -Dtest=IdeToolCommandletTest test both tests pass.
  2. Verify no existing IDE launch behavior changed (getter is not yet wired into any caller).

Checklist for this PR

  • When running mvn clean test locally all tests pass and build is successful (ran IdeToolCommandletTest)
  • PR title is of the form #«issue-id»: «brief summary»
  • PR top-level comment summarizes 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
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

@coveralls

coveralls commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 29916981277

Coverage increased (+0.05%) to 72.542%

Details

  • Coverage increased (+0.05%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 6 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

6 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/tool/ide/IdeToolCommandlet.java 6 81.97%

Coverage Stats

Coverage Status
Relevant Lines: 16959
Covered Lines: 12826
Line Coverage: 75.63%
Relevant Branches: 7584
Covered Branches: 4978
Branch Coverage: 65.64%
Branches in Coverage %: Yes
Coverage Strength: 3.2 hits per line

💛 - Coveralls

@quando632 quando632 added enhancement New feature or request workspace workspaces sub-folder to manage sub-projects labels Jul 21, 2026
@quando632 quando632 moved this from 🆕 New to Team Review in IDEasy board Jul 21, 2026
@quando632 quando632 moved this from Team Review to 🏗 In progress in IDEasy board Jul 21, 2026
Comment thread CHANGELOG.adoc Outdated
@vivu001

vivu001 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Looks good 👍 @quando632: the getIdeMetadataPath() is added as a shared foundation for moving IDE metadata to $IDE_HOME/.ide/«ide»/«workspace», and it’s not wired into any launch path yet, so behavior stays unchanged.

The new unit test covers the path resolution, and the change is ready for the follow-up IDE-specific wiring PRs.

@quando632
quando632 marked this pull request as ready for review July 22, 2026 11:42
@vivu001 vivu001 moved this from Team Review to 👀 In review in IDEasy board Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request workspace workspaces sub-folder to manage sub-projects

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

Foundation: introduce getIdeMetadataPath() for $IDE_HOME/.ide/<ide>/<workspace>

3 participants