fix: suppress eslint no-unused-vars warnings by renaming unused params/vars with underscore

Author: aheizi

src/services/mcp/McpHub.ts

@@ -19,7 +19,7 @@ export class McpHub {
 
 		this.configManager = new ConfigManager()
 
-		const connectionFactory = new ConnectionFactory(this.configManager, provider, (server: McpServer) =>
+		const connectionFactory = new ConnectionFactory(this.configManager, provider, (_server: McpServer) =>
 			this.notifyServersChanged(),
 		)
 		connectionFactory.registerHandler(new StdioHandler())

src/services/mcp/config/ConfigManager.ts

@@ -118,7 +118,7 @@ export class ConfigManager {
 	 * @param serverName Optional server name for error messages
 	 * @returns Validated server configuration
 	 */
-	public validateServerConfig(config: unknown, serverName?: string): ServerConfig {
+	public validateServerConfig(config: unknown, _serverName?: string): ServerConfig {
 		try {
 			const configCopy = { ...(config as Record<string, unknown>) }
 			const result = safeParseServerConfig(configCopy)

src/services/mcp/connection/ConnectionManager.ts

@@ -125,7 +125,7 @@ export class ConnectionManager {
 
 					const stripNonConnectionFields = (configObj: any) => {
 						// Exclude alwaysAllow and timeout, timeout changes do not trigger reconnection
-						const { alwaysAllow, timeout, ...rest } = configObj
+						const { alwaysAllow: _alwaysAllow, timeout: _timeout, ...rest } = configObj
 						return rest
 					}