fix: remove duplicate uri of workspace paths sent by VS (#585)

* fix: remove duplicate uri of workspace paths sent by VS

* fix: add tests for deduplication filter

Author: XiaoxuanLu

runtimes/runtimes/lsp/router/lspRouter.test.ts

@@ -678,6 +678,31 @@ describe('LspRouter', () => {
             assert.deepStrictEqual(lspRouter.getAllWorkspaceFolders(), expectedFolders)
             assert(didChangeWorkspaceFoldersSpy.calledWith({ event }))
         })
+
+        it('should filter out duplicate workspace folder URIs when adding', () => {
+            const initialFolders = [
+                { name: 'existing', uri: 'file:///existing' },
+                { name: 'another', uri: 'file:///another' },
+            ]
+            lspRouter['workspaceFolders'] = initialFolders
+
+            const event = {
+                added: [
+                    { name: 'new', uri: 'file:///new' },
+                    { name: 'duplicate', uri: 'file:///existing' }, // duplicate URI
+                ],
+                removed: [],
+            }
+
+            lspRouter.didChangeWorkspaceFolders(event)
+
+            const expectedFolders = [
+                { name: 'existing', uri: 'file:///existing' },
+                { name: 'another', uri: 'file:///another' },
+                { name: 'new', uri: 'file:///new' },
+            ]
+            assert.deepStrictEqual(lspRouter.getAllWorkspaceFolders(), expectedFolders)
+        })
     })
 
     describe('didCreateFiles', () => {

runtimes/runtimes/lsp/router/lspRouter.ts

@@ -144,7 +144,9 @@ ${JSON.stringify({ ...result.capabilities, ...result.awsServerCapabilities })}`
         this.workspaceFolders = this.workspaceFolders.filter(
             folder => !event.removed.some(removed => removed.uri === folder.uri)
         )
-        this.workspaceFolders.push(...event.added)
+        this.workspaceFolders.push(
+            ...event.added.filter(added => !this.workspaceFolders.some(existing => existing.uri === added.uri))
+        )
         const params: DidChangeWorkspaceFoldersParams = { event }
 
         this.routeNotificationToAllServers((server, params) => {