Revert "feat: Add browser compatibility for HTTP-only MCP clients"

This reverts commit ccbb0c3.

Author: tonxxd

index.ts

@@ -1,8 +1,10 @@
 import { MCPAgent } from './src/agents/mcp_agent.js'
 import { RemoteAgent } from './src/agents/remote.js'
 import { MCPClient } from './src/client.js'
+import { loadConfigFile } from './src/config.js'
 import { BaseConnector } from './src/connectors/base.js'
 import { HttpConnector } from './src/connectors/http.js'
+import { StdioConnector } from './src/connectors/stdio.js'
 import { WebSocketConnector } from './src/connectors/websocket.js'
 
 import { Logger, logger } from './src/logging.js'
@@ -24,18 +26,4 @@ export { AIMessage, BaseMessage, HumanMessage, SystemMessage, ToolMessage } from
 // Re-export StreamEvent type from LangChain for convenience
 export type { StreamEvent } from '@langchain/core/tracers/log_stream'
 
-export { BaseConnector, HttpConnector, Logger, logger, MCPAgent, MCPClient, MCPSession, RemoteAgent, WebSocketConnector }
-
-// Conditionally export Node.js-specific functionality
-// Check if we're in a Node.js environment
-const isNodeJS = typeof process !== 'undefined'
-  && process.versions
-  && process.versions.node
-  && typeof window === 'undefined'
-
-// Only export these in Node.js environments to prevent browser import errors
-export const StdioConnector = isNodeJS ? undefined : undefined
-export const loadConfigFile = isNodeJS ? undefined : undefined
-
-// Note: The actual Node.js implementations are loaded by the config.ts module
-// when needed, avoiding import-time errors in browser environments
+export { BaseConnector, HttpConnector, loadConfigFile, Logger, logger, MCPAgent, MCPClient, MCPSession, RemoteAgent, StdioConnector, WebSocketConnector }

package.json

@@ -28,18 +28,8 @@
   ],
   "exports": {
     ".": {
-      "browser": {
-        "types": "./dist/browser.d.ts",
-        "import": "./dist/browser.js"
-      },
-      "node": {
-        "types": "./dist/index.d.ts",
-        "import": "./dist/index.js"
-      },
-      "default": {
-        "types": "./dist/index.d.ts",
-        "import": "./dist/index.js"
-      }
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js"
     }
   },
   "main": "./dist/index.js",

src/browser.ts

@@ -1,40 +1,10 @@
 import type { BaseConnector } from './connectors/base.js'
+import { readFileSync } from 'node:fs'
 import { HttpConnector } from './connectors/http.js'
+import { StdioConnector } from './connectors/stdio.js'
 import { WebSocketConnector } from './connectors/websocket.js'
 
-// Check if we're in a Node.js environment
-const isNodeJS = typeof process !== 'undefined'
-  && process.versions
-  && process.versions.node
-  && typeof window === 'undefined'
-
-// Conditionally import Node.js-specific modules
-let StdioConnector: any
-let readFileSync: any
-
-if (isNodeJS) {
-  try {
-    // Dynamic import for Node.js-specific modules
-    Promise.all([
-      import('./connectors/stdio.js'),
-      import('node:fs'),
-    ]).then(([stdioModule, fsModule]) => {
-      StdioConnector = stdioModule.StdioConnector
-      readFileSync = fsModule.readFileSync
-    }).catch((error) => {
-      console.warn('Failed to load Node.js modules:', error)
-    })
-  }
-  catch (error) {
-    console.warn('Failed to load Node.js modules:', error)
-  }
-}
-
 export function loadConfigFile(filepath: string): Record<string, any> {
-  if (!isNodeJS || !readFileSync) {
-    throw new Error('loadConfigFile is only available in Node.js environments')
-  }
-
   const raw = readFileSync(filepath, 'utf-8')
   return JSON.parse(raw)
 }
@@ -43,10 +13,6 @@ export function createConnectorFromConfig(
   serverConfig: Record<string, any>,
 ): BaseConnector {
   if ('command' in serverConfig && 'args' in serverConfig) {
-    if (!isNodeJS || !StdioConnector) {
-      throw new Error('StdioConnector is not available in browser environments. Use HTTP or WebSocket connectors instead.')
-    }
-
     return new StdioConnector({
       command: serverConfig.command,
       args: serverConfig.args,
@@ -73,5 +39,5 @@ export function createConnectorFromConfig(
     })
   }
 
-  throw new Error('Cannot determine connector type from config. Browser environments only support HTTP and WebSocket connectors.')
+  throw new Error('Cannot determine connector type from config')
 }