fix: use pretty-printing for .roo/mcp.json

Support user-serviceable .roo/mcp.json file with pretty-printing

- Remove safeWriteJson in favor of direct JSON.stringify to pretty-print
  user serviceable MCP configuration files

Cc: @mrubens

Signed-off-by: Eric Wheeler <[email protected]>

Author: Eric Wheeler

src/services/mcp/McpHub.ts

@@ -1,4 +1,3 @@
-import { safeWriteJson } from "../../utils/safeWriteJson"
 import { Client } from "@modelcontextprotocol/sdk/client/index.js"
 import { StdioClientTransport, getDefaultEnvironment } from "@modelcontextprotocol/sdk/client/stdio.js"
 import { SSEClientTransport } from "@modelcontextprotocol/sdk/client/sse.js"
@@ -1345,7 +1344,7 @@ export class McpHub {
 			mcpServers: config.mcpServers,
 		}
 
-		await safeWriteJson(configPath, updatedConfig)
+		await fs.writeFile(configPath, JSON.stringify(updatedConfig, null, 2))
 	}
 
 	public async updateServerTimeout(
@@ -1423,7 +1422,7 @@ export class McpHub {
 					mcpServers: config.mcpServers,
 				}
 
-				await safeWriteJson(configPath, updatedConfig)
+				await fs.writeFile(configPath, JSON.stringify(updatedConfig, null, 2))
 
 				// Update server connections with the correct source
 				await this.updateServerConnections(config.mcpServers, serverSource)
@@ -1568,7 +1567,7 @@ export class McpHub {
 			targetList.splice(toolIndex, 1)
 		}
 
-		await safeWriteJson(normalizedPath, config)
+		await fs.writeFile(normalizedPath, JSON.stringify(config, null, 2))
 
 		if (connection) {
 			connection.server.tools = await this.fetchToolsList(serverName, source)