Skip to content

Commit 3afd5e1

Browse files
committed
feat(mcp-server-integration): TF-28514: Removed: disposables from mcp server provider definition
1 parent 27599ea commit 3afd5e1

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

src/features/mcpServer.ts

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,6 @@ interface McpServerDefinition {
1919
}
2020

2121
export class McpServerFeature {
22-
private disposables: vscode.Disposable[] = [];
23-
private infoMessageShown = false;
24-
2522
constructor(
2623
private context: vscode.ExtensionContext,
2724
private reporter: TelemetryReporter,
@@ -38,7 +35,6 @@ export class McpServerFeature {
3835
const provider = this.registerMcpServerProvider();
3936
if (provider) {
4037
this.context.subscriptions.push(provider);
41-
this.disposables.push(provider);
4238
}
4339
} catch (error) {
4440
console.error('Failed to register MCP server definition provider:', error);
@@ -160,10 +156,7 @@ export class McpServerFeature {
160156
});
161157
}
162158

163-
public dispose(): void {
164-
this.disposables.forEach((d) => {
165-
d.dispose();
166-
});
167-
this.disposables = [];
159+
dispose(): void {
160+
// context.subscriptions will be disposed by the extension, so any explicit code should not be required.
168161
}
169162
}

0 commit comments

Comments
 (0)