Fix duplicate project nodes in Project Outline on rename (#4820)

* Initial plan

* Fix duplicate project nodes in Project Outline on rename (#4343)

When a CMake project is renamed in CMakeLists.txt, the Project Outline
now replaces the existing node instead of creating a duplicate. The fix
clears stale entries keyed by the old project name before inserting the
new ProjectNode.

Co-authored-by: Omotola <[email protected]>

* Remove accidentally committed .npmrc.bak.old

Co-authored-by: Omotola <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Omotola <[email protected]>
Co-authored-by: Omotola <[email protected]>

Author: Copilot

CHANGELOG.md

@@ -31,6 +31,7 @@ Improvements:
 - Honor `debugger.workingDirectory` from the CMake File API when debugging a target, so that the `DEBUGGER_WORKING_DIRECTORY` target property is used as the debugger working directory. [#4595](https://github.com/microsoft/vscode-cmake-tools/issues/4595)
 
 Bug Fixes:
+- Fix renaming a CMake project creating a duplicate node in the Project Outline instead of replacing the existing one. [#4343](https://github.com/microsoft/vscode-cmake-tools/issues/4343)
 - Remove internal stack traces from the Output pane when a subprocess fails. Error messages now show only human-readable information; stack traces are still available in debug-level logging. [#4807](https://github.com/microsoft/vscode-cmake-tools/issues/4807)
 - Fix duplicate launch configurations in the ctest debug picker when both `.vscode/launch.json` and a `.code-workspace` file exist. [#4586](https://github.com/microsoft/vscode-cmake-tools/issues/4586)
 - Users can now debug with or without launch configurations from the test explorer. `cmake.ctest.neverDebugTestsWithLaunchConfiguration` can be set to `true` to bypass the quick pick menu where the launch configuration is selected. [#4790](https://github.com/microsoft/vscode-cmake-tools/issues/4790)

src/ui/projectOutline/projectOutline.ts

@@ -747,6 +747,9 @@ export class WorkspaceFolderNode extends BaseNode {
         const rootProject = projectOutlineModel.project;
         let item = this.getNode(cmakeProject, rootProject.name);
         if (!item) {
+            // Remove stale entries for this cmakeProject so that a renamed
+            // project replaces the old node instead of appearing alongside it.
+            this.removeNodes(cmakeProject);
             item = new ProjectNode(rootProject.name, this.wsFolder, cmakeProject.folderPath);
             this.setNode(cmakeProject, rootProject.name, item);
         }