fix: address code review feedback for ToolPipe MCP Server integration

- Fix MCP API types: use vscode.McpHttpServerDefinition and vscode.McpStdioServerDefinition
- Remove unused dead code (promptConfigureToolPipe function)
- Implement dynamic Markdown fence escaping to prevent backtick-related breakage
- Replace ephemeral tunnel URLs with example placeholders
- Change default remoteUrl to empty string with user guidance
- Add extension to build compilation list (gulpfile.extensions.ts)
- Update tsconfig.json with skipLibCheck and proper glob pattern
- Update engine requirement to ^1.97.0 when MCP APIs were introduced
- Remove trailing whitespace

Author: rupeshkumar-555

build/gulpfile.extensions.ts

@@ -85,6 +85,7 @@ const compilations = [
 	'extensions/references-view/tsconfig.json',
 	'extensions/search-result/tsconfig.json',
 	'extensions/simple-browser/tsconfig.json',
+	'extensions/toolpipe-mcp-server/tsconfig.json',
 	'extensions/tunnel-forwarding/tsconfig.json',
 	'extensions/typescript-language-features/web/tsconfig.json',
 	'extensions/typescript-language-features/tsconfig.json',

extensions/toolpipe-mcp-server/README.md

@@ -52,18 +52,20 @@ ToolPipe MCP Server provides comprehensive developer tools organized into five c
 
 ## Configuration
 
-### Quick Start (Remote Mode - Recommended)
+### Quick Start (Remote Mode)
 
-By default, ToolPipe uses the **remote cloud-hosted server** which requires no additional setup:
+To use ToolPipe with a remote server, configure the URL of your MCP server:
 
 ```json
 {
   "toolpipeMcpServer.enabled": true,
   "toolpipeMcpServer.mode": "remote",
-  "toolpipeMcpServer.remoteUrl": "https://troops-submission-what-stays.trycloudflare.com/mcp"
+  "toolpipeMcpServer.remoteUrl": "https://example.com/mcp"
 }
 ```
 
+Replace `https://example.com/mcp` with your actual ToolPipe server URL.
+
 ### Advanced Configuration
 
 #### Using Local Server

extensions/toolpipe-mcp-server/package.json

@@ -5,9 +5,9 @@
   "version": "1.0.0",
   "publisher": "vscode",
   "license": "MIT",
-  "engines": {
-    "vscode": "^1.96.0"
-  },
+	"engines": {
+		"vscode": "^1.97.0"
+	},
   "categories": [
     "Other",
     "Developer Tools"
@@ -57,11 +57,11 @@
             "default": "remote",
             "description": "ToolPipe connection mode: 'remote' for cloud-hosted server, 'local' for npm-based server"
           },
-          "toolpipeMcpServer.remoteUrl": {
-            "type": "string",
-            "default": "https://troops-submission-what-stays.trycloudflare.com/mcp",
-            "description": "Remote URL for ToolPipe MCP Server (HTTP endpoint)"
-          },
+  				"toolpipeMcpServer.remoteUrl": {
+					"type": "string",
+					"default": "",
+					"description": "Remote URL for ToolPipe MCP Server (HTTPS endpoint). Provide your own server URL, for example: https://example.com/mcp"
+				},
           "toolpipeMcpServer.localCommand": {
             "type": "string",
             "default": "npx",

extensions/toolpipe-mcp-server/src/extension.ts

@@ -7,7 +7,7 @@ import * as vscode from 'vscode';
 
 /**
  * ToolPipe MCP Server Extension
- * 
+ *
  * Provides integration with ToolPipe MCP Server, which offers 120+ developer utilities:
  * - Code tools: Review, minification, formatting (JS/TS/Python/SQL/CSS/HTML)
  * - Data tools: JSON/CSV/XML/YAML conversion, Base64, UUID generation
@@ -16,9 +16,6 @@ import * as vscode from 'vscode';
  * - DevOps: Docker Compose generation, GitHub Actions workflows, Nginx configs
  */
 
-// Store for MCP Server definitions
-const mcpDefinitions = new Map<string, vscode.lm.McpServerDefinition>();
-
 export async function activate(context: vscode.ExtensionContext) {
 	const config = vscode.workspace.getConfiguration('toolpipeMcpServer');
 	const enabled = config.get('enabled', true);
@@ -30,7 +27,7 @@ export async function activate(context: vscode.ExtensionContext) {
 
 	// Register MCP Server Definition Provider
 	const provider = {
-		provideMcpServerDefinitions: async (): Promise<vscode.lm.McpServerDefinition[]> => {
+		provideMcpServerDefinitions: async (): Promise<vscode.McpServerDefinition[]> => {
 			return getMcpServerDefinitions();
 		}
 	};
@@ -45,92 +42,36 @@ export async function activate(context: vscode.ExtensionContext) {
 /**
  * Generates MCP Server definitions based on current configuration
  */
-function getMcpServerDefinitions(): vscode.lm.McpServerDefinition[] {
+function getMcpServerDefinitions(): vscode.McpServerDefinition[] {
 	const config = vscode.workspace.getConfiguration('toolpipeMcpServer');
 	const mode = config.get<string>('mode', 'remote');
 
-	const definitions: vscode.lm.McpServerDefinition[] = [];
+	const definitions: vscode.McpServerDefinition[] = [];
 
 	if (mode === 'remote') {
-		const remoteUrl = config.get<string>('remoteUrl', 'https://troops-submission-what-stays.trycloudflare.com/mcp');
-		definitions.push({
-			name: 'toolpipe',
-			displayName: 'ToolPipe Developer Tools',
-			description: 'Access 120+ developer utilities for code, data, security, API, and DevOps tasks',
-			type: 'http',
-			url: remoteUrl,
-			capabilities: {
-				tools: {},
-				resources: {}
-			}
-		} as any);
+		const remoteUrl = config.get<string>('remoteUrl', '');
+		if (remoteUrl) {
+			definitions.push(
+				new vscode.McpHttpServerDefinition(
+					'ToolPipe Developer Tools',
+					vscode.Uri.parse(remoteUrl)
+				)
+			);
+		}
 	} else if (mode === 'local') {
 		const command = config.get<string>('localCommand', 'npx');
 		const args = config.get<string[]>('localArgs', ['@cosai-labs/toolpipe-mcp-server']);
-		
-		definitions.push({
-			name: 'toolpipe',
-			displayName: 'ToolPipe Developer Tools (Local)',
-			description: 'Access 120+ developer utilities via locally-hosted ToolPipe server',
-			type: 'stdio',
-			command,
-			args,
-			capabilities: {
-				tools: {},
-				resources: {}
-			}
-		} as any);
-	}
-
-	return definitions;
-}
 
-/**
- * Prompts user to configure ToolPipe if not already configured
- */
-async function promptConfigureToolPipe(): Promise<void> {
-	const config = vscode.workspace.getConfiguration('toolpipeMcpServer');
-	
-	if (config.get('configured')) {
-		return;
+		definitions.push(
+			new vscode.McpStdioServerDefinition(
+				'ToolPipe Developer Tools (Local)',
+				command,
+				args
+			)
+		);
 	}
 
-	const choice = await vscode.window.showQuickPick(
-		[
-			{
-				label: 'Use Remote Server (Cloud-hosted)',
-				description: 'Connect to ToolPipe cloud server (no setup required)',
-				value: 'remote'
-			},
-			{
-				label: 'Use Local Server (npm-based)',
-				description: 'Run ToolPipe locally: npx @cosai-labs/toolpipe-mcp-server',
-				value: 'local'
-			},
-			{
-				label: 'Skip Configuration',
-				description: 'Configure later',
-				value: 'skip'
-			}
-		],
-		{ placeHolder: 'Choose ToolPipe server mode' }
-	);
-
-	if (!choice) {
-		return;
-	}
-
-	if (choice.value === 'skip') {
-		return;
-	}
-
-	// Update configuration
-	await config.update('mode', choice.value, vscode.ConfigurationTarget.Global);
-	await config.update('configured', true, vscode.ConfigurationTarget.Global);
-
-	vscode.window.showInformationMessage(
-		`ToolPipe MCP Server configured to use ${choice.value} mode. Settings can be changed in preferences.`
-	);
+	return definitions;
 }
 
 export function deactivate() {

extensions/toolpipe-mcp-server/tsconfig.json

@@ -1,18 +1,17 @@
 {
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "commonjs",
-    "lib": ["ES2022"],
-    "outDir": "./out",
-    "rootDir": "./src",
-    "declaration": true,
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "resolveJsonModule": true,
-    "moduleResolution": "node"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "out"]
+	"extends": "../tsconfig.base.json",
+	"compilerOptions": {
+		"outDir": "./out",
+		"rootDir": "./src",
+		"skipLibCheck": true
+	},
+	"include": [
+		"src/**/*",
+		"../../src/vscode-dts/vscode.d.ts",
+		"../../src/vscode-dts/vscode.proposed.mcpServerDefinitions.d.ts"
+	],
+	"exclude": [
+		"node_modules",
+		"out"
+	]
 }

src/vs/workbench/contrib/chat/common/widget/annotations.ts

@@ -109,11 +109,14 @@ export function annotateSpecialMarkdownContent(response: Iterable<IChatProgressR
 				const refId = refIdPool++;
 				const printUri = URI.parse(contentRefUrl).with({ path: String(refId) });
 				let markdownText = `[${label}](${printUri.toString()})`;
-				
+
 				// If the inline reference includes a code snippet, append it as a code block
 				if (item.snippet) {
 					const languageId = item.languageId || inferLanguageFromLabel(label) || '';
-					markdownText += `\n\`\`\`${languageId}\n${item.snippet}\n\`\`\``;
+					// Use a fence length that exceeds any backtick run in the snippet
+					const fenceLength = Math.max(3, (item.snippet.match(/`+/g)?.reduce((max, run) => Math.max(max, run.length), 0) ?? 0) + 1);
+					const fence = '`'.repeat(fenceLength);
+					markdownText += `\n${fence}${languageId}\n${item.snippet}\n${fence}`;
 				}
 
 				const annotationMetadata = { [refId]: item };

src/vs/workbench/contrib/mcp/common/mcpConfiguration.ts

@@ -81,7 +81,7 @@ const httpSchemaExamples = {
 	},
 	'toolpipe-remote': {
 		type: 'http',
-		url: 'https://troops-submission-what-stays.trycloudflare.com/mcp',
+		url: 'https://example.com/mcp',
 	}
 };