#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 8 commits into
Conversation
…kspace>/config + migration
Coverage Report for CI Build 30438912015Coverage increased (+0.03%) to 72.626%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions9 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 |
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.