Skip to content

Commit b922e27

Browse files
authored
mcp: include correct scope when creating server install events (microsoft#254532)
Refs microsoft#254520
1 parent c8441f2 commit b922e27

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/vs/workbench/services/mcp/common/mcpWorkbenchManagementService.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ export class WorkbenchMcpManagementService extends Disposable implements IWorkbe
147147
}));
148148

149149
this._register(this.workspaceMcpManagementService.onDidInstallMcpServers(async e => {
150-
const { mcpServerInstallResult } = this.createInstallMcpServerResultsFromEvent(e);
150+
const { mcpServerInstallResult } = this.createInstallMcpServerResultsFromEvent(e, LocalMcpServerScope.Workspace);
151151
this._onDidInstallMcpServersInCurrentProfile.fire(mcpServerInstallResult);
152152
this._onDidInstallMcpServersInCurrentProfile.fire(mcpServerInstallResult);
153153
}));
@@ -163,7 +163,7 @@ export class WorkbenchMcpManagementService extends Disposable implements IWorkbe
163163
}));
164164

165165
this._register(this.workspaceMcpManagementService.onDidUpdateMcpServers(e => {
166-
const { mcpServerInstallResult } = this.createInstallMcpServerResultsFromEvent(e);
166+
const { mcpServerInstallResult } = this.createInstallMcpServerResultsFromEvent(e, LocalMcpServerScope.Workspace);
167167
this._onDidUpdateMcpServersInCurrentProfile.fire(mcpServerInstallResult);
168168
this._onDidUpdateMcpServersInCurrentProfile.fire(mcpServerInstallResult);
169169
}));
@@ -204,13 +204,13 @@ export class WorkbenchMcpManagementService extends Disposable implements IWorkbe
204204
}));
205205
}
206206

207-
private createInstallMcpServerResultsFromEvent(e: readonly InstallMcpServerResult[]) {
207+
private createInstallMcpServerResultsFromEvent(e: readonly InstallMcpServerResult[], scope: LocalMcpServerScope) {
208208
const mcpServerInstallResult: IWorkbenchMcpServerInstallResult[] = [];
209209
const mcpServerInstallResultInCurrentProfile: IWorkbenchMcpServerInstallResult[] = [];
210210
for (const result of e) {
211211
const workbenchResult = {
212212
...result,
213-
local: result.local ? this.toWorkspaceMcpServer(result.local, LocalMcpServerScope.User) : undefined
213+
local: result.local ? this.toWorkspaceMcpServer(result.local, scope) : undefined
214214
};
215215
mcpServerInstallResult.push(workbenchResult);
216216
if (this.uriIdentityService.extUri.isEqual(result.mcpResource, this.userDataProfileService.currentProfile.mcpResource)) {
@@ -222,7 +222,7 @@ export class WorkbenchMcpManagementService extends Disposable implements IWorkbe
222222
}
223223

224224
private handleInstallMcpServerResultsFromEvent(e: readonly InstallMcpServerResult[], emitter: Emitter<readonly InstallMcpServerResult[]>, currentProfileEmitter: Emitter<readonly InstallMcpServerResult[]>): void {
225-
const { mcpServerInstallResult, mcpServerInstallResultInCurrentProfile } = this.createInstallMcpServerResultsFromEvent(e);
225+
const { mcpServerInstallResult, mcpServerInstallResultInCurrentProfile } = this.createInstallMcpServerResultsFromEvent(e, LocalMcpServerScope.User);
226226
emitter.fire(mcpServerInstallResult);
227227
if (mcpServerInstallResultInCurrentProfile.length) {
228228
currentProfileEmitter.fire(mcpServerInstallResultInCurrentProfile);

0 commit comments

Comments
 (0)