#2174: store VSCode user-data under $IDE_HOME/.ide/vscode/<workspace>/config - #2201
#2174: store VSCode user-data under $IDE_HOME/.ide/vscode/<workspace>/config#2201quando632 wants to merge 11 commits into
Conversation
…kspace>/config + migration
Coverage Report for CI Build 33606129092Coverage increased (+0.01%) to 73.626%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions4 previously-covered lines in 2 files lost coverage.
Coverage Stats💛 - Coveralls |
|
Update: the JetBrains PR #2208 no longer adds a migration (the vmoptions cleanup migration was dropped as unnecessary — vmoptions regenerate at the new location). So there is no migration-version coordination needed anymore; this PR's |
|
Core implementation is correct and well-tested. Merge-ready after rebase |
|
@quando632 and @hohwille : Should PyCharm, AndroidStudio, and Eclipse Move Metadata Like VSCode? |
|
We decided not to move the |
…-metadata # Conflicts: # CHANGELOG.adoc
hohwille
left a comment
There was a problem hiding this comment.
@quando632 thanks for your PR. Your change and migration looks good. Nice job 👍
Please fix the build:
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /home/runner/work/IDEasy/IDEasy/cli/src/test/java/com/devonfw/tools/ide/tool/vscode/VscodeTest.java:[170,4] error: cannot find symbol
symbol: class Path
location: class VscodeTest
And we need to update to the recent version due to my late review...
|
Thanks @hohwille, all points addressed in b432f9e:
Ready for re-check. |
This PR fixes #2174 (part of #2142)
Moves VSCode user-data out of the workspace into the dedicated
$IDE_HOME/.ide/vscode/«workspace»/configfolder so workspaces stay clean and independent of the IDE.Implemented changes:
Vscode.configureToolArgsnow points--user-data-dirtogetIdeMetadataPath().resolve("config")instead of«workspace»/.vscode/.userdata. The folder VSCode opens (getWorkspacePath()) is unchanged.Mig202608001(target2026.08.001) moves any existing.vscode/.userdataof every workspace to the new location. It is idempotent: skips workspaces already migrated and skips (with a warning, no overwrite) if a target already exists. Registered inIdeMigrator.VscodeTest.testConfigureToolArgsUsesIdeMetadataPathForUserData,Mig202608001Test(move, no-op, and target-exists cases).Testing instructions
cd cli && mvn -Dtest=VscodeTest,Mig202608001Test,IdeMigratorTest test— all pass.$IDE_HOME/workspaces/main/.vscode/.userdata/state.json, runide update, verify it moved to$IDE_HOME/.ide/vscode/main/config/state.jsonand the workspace no longer contains.vscode/.userdata.Checklist for this PR
mvn clean testlocally all tests pass and build is successful (ran the affected tests)#«issue-id»: «brief summary»In Progressand assigned to youCoordination note
IdeMigratorrequires strictly ascending migration versions. The JetBrains subtask (#2173) also adds a migration — whichever merges second must adjust its version to stay ascending.