fix: Add Node 16+ compatibility with node-fetch

facundofarias · claude · facundofarias · commit cf2051e4642e · 2025-11-11T08:25:42.000+01:00
Replace undici with node-fetch v2 to support Node.js 16+, which is commonly used by Claude Desktop/Code installations. Changes: - Replace undici with node-fetch@2 for broader Node.js compatibility - Update engine requirement from >=20.0.0 to >=16.0.0 - Add @types/node-fetch for TypeScript support - Update tsconfig.json to include node types - Simplify fetch implementation using node-fetch types - Update documentation to reflect Node 16+ requirement This fixes the "fetch is not defined" and "ReadableStream is not defined" errors encountered when Claude Desktop/Code uses Node v16. Tested with Node v16.20.2 (Claude Desktop default) and Node v24.5.0. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -70,7 +70,7 @@ Restart Claude Desktop to test.
 **Shared Core:**
 - `src/mcp-server.ts`: Factory function `createMCPServer(username, password, account)` that returns a configured MCP Server instance. Used by both stdio and hosted modes.
 - `src/tools.ts`: Defines 6 MCP tools (list_projects, get_project, list_servers, list_deployments, get_deployment, create_deployment) with Zod validation schemas
-- `src/api-client.ts`: DeployHQClient class for HTTP Basic Auth API calls to DeployHQ REST API
+- `src/api-client.ts`: DeployHQClient class for HTTP Basic Auth API calls to DeployHQ REST API. Uses node-fetch for Node 16+ compatibility.
 
 **Transport Handlers (hosted mode only):**
 - `src/transports/sse-handler.ts`: Server-Sent Events transport
@@ -159,4 +159,4 @@ For stdio testing, use absolute path to `dist/stdio.js` in client configs to avo
 
 ## Important: Node Version
 
-Requires Node.js >=20.0.0 (specified in package.json engines). The stdio transport uses modern Node.js features.
+Requires Node.js >=16.0.0 (specified in package.json engines). The server uses node-fetch for HTTP requests to ensure compatibility with Node 16+, which is commonly used by Claude Desktop/Code installations.
diff --git a/README.md b/README.md
@@ -132,9 +132,11 @@ Once configured, you can ask Claude to interact with DeployHQ:
 
 ## &#128230; Prerequisites
 
-- Node.js 20+
+- **Node.js 16+** (Node 20+ recommended)
 - DeployHQ account with API access
 
+**Note**: The server uses `node-fetch` for HTTP requests, providing compatibility with Node.js 16 and above.
+
 ## &#128295; Local Development
 
 ### 1. Clone the repository
diff --git a/docs/USER_GUIDE.md b/docs/USER_GUIDE.md
@@ -27,7 +27,7 @@ This guide will help you configure and use the DeployHQ MCP Server with Claude D
 
 - **Claude Desktop** (for GUI) or **Claude Code CLI** (for terminal) installed
 - DeployHQ account with API access
-- Node.js 20 or higher installed
+- Node.js 16 or higher installed (Node 20+ recommended)
 - For development: Git and npm/npx
 
 ### Getting Your DeployHQ Credentials
@@ -423,12 +423,12 @@ Choose the scope that best fits your needs. For most users, `--scope user` or us
 
 **Possible Causes**:
 - Node.js not installed
-- Node.js version too old (requires v20+)
+- Node.js version too old (requires v16+, v20+ recommended)
 - PATH not configured correctly
 
 **Solution**:
-1. Install Node.js 20 or higher: `brew install node` (macOS) or download from nodejs.org
-2. Verify installation: `node --version`
+1. Install Node.js 16 or higher: `brew install node` (macOS) or download from nodejs.org
+2. Verify installation: `node --version` (should show v16.0.0 or higher)
 3. Restart Claude Desktop/Code after installing Node.js
 4. For development builds, ensure you ran `npm run build`
 
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -8,7 +8,7 @@
   },
   "type": "module",
   "engines": {
-    "node": ">=20.0.0"
+    "node": ">=16.0.0"
   },
   "scripts": {
     "dev": "tsx --watch src/index.ts",
@@ -29,13 +29,15 @@
   "license": "MIT",
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.4",
-    "express": "^4.21.1",
     "dotenv": "^16.4.5",
+    "express": "^4.21.1",
+    "node-fetch": "^2.7.0",
     "zod": "^3.23.8"
   },
   "devDependencies": {
     "@types/express": "^5.0.0",
     "@types/node": "^22.9.0",
+    "@types/node-fetch": "^2.6.13",
     "@typescript-eslint/eslint-plugin": "^8.13.0",
     "@typescript-eslint/parser": "^8.13.0",
     "eslint": "^9.14.0",
diff --git a/src/api-client.ts b/src/api-client.ts
@@ -3,6 +3,9 @@
  * Provides type-safe access to DeployHQ API endpoints
  */
 
+// Import fetch for Node 16+ compatibility
+import fetch, { RequestInit as NodeFetchRequestInit } from 'node-fetch';
+
 /**
  * Custom error class for DeployHQ API errors
  */
@@ -153,23 +156,24 @@ export class DeployHQClient {
    * @param options - Fetch options
    * @returns Parsed JSON response
    */
-  private async request<T>(path: string, options: RequestInit = {}): Promise<T> {
+  private async request<T>(path: string, options: NodeFetchRequestInit = {}): Promise<T> {
     const url = `${this.baseUrl}${path}`;
 
     const controller = new AbortController();
     const timeoutId = setTimeout(() => controller.abort(), this.timeout);
 
     try {
       const response = await fetch(url, {
-        ...options,
+        method: options.method || 'GET',
         headers: {
           'Authorization': this.authHeader,
           'Accept': 'application/json',
           'Content-Type': 'application/json',
-          ...options.headers,
+          ...(options.headers as Record<string, string> || {}),
         },
+        body: options.body,
         signal: controller.signal,
-      });
+      } as NodeFetchRequestInit);
 
       clearTimeout(timeoutId);
 
diff --git a/tsconfig.json b/tsconfig.json
@@ -4,6 +4,7 @@
     "module": "NodeNext",
     "moduleResolution": "NodeNext",
     "lib": ["ES2022"],
+    "types": ["node"],
     "outDir": "./dist",
     "rootDir": "./src",
     "strict": true,
