fix: do not include node-fetch in runtime bundle

SpellcraftAI · Jun 23, 2023 · 5970b66 · 5970b66
1 parent d8fdf04
commit 5970b66
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 13 deletions.
diff --git a/package.json b/package.json
@@ -47,13 +47,13 @@
     "ava": "^5.2.0",
     "dotenv": "^16.0.3",
     "eslint": "^8.41.0",
+    "node-fetch": "^3.3.1",
     "openai": "^3.3.0",
     "typedoc": "^0.24.7",
     "typescript": "^5.0.4"
   },
   "dependencies": {
     "eventsource-parser": "^1.0.0",
-    "node-fetch": "^3.3.1",
     "yield-stream": "^3.0.0"
   },
   "keywords": [

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/lib/backoff.ts b/src/lib/backoff.ts
@@ -12,15 +12,14 @@ export interface BackoffOptions {
 export const fetchWithBackoff = async (
   input: RequestInfo & NodeFetchRequestInfo,
   init?: RequestInit & NodeFetchRequestInit,
-  fetch?: Fetch,
+  fetch: Fetch = globalThis.fetch,
   { delay, maxRetries }: BackoffOptions = {
     delay: 500,
     maxRetries: 7
   }
 ) => {
   if (!fetch) {
-    const { default: nodeFetch } = await import("node-fetch");
-    fetch = nodeFetch;
+    throw new Error("No fetch implementation found.");
   }
 
   for (let i = 0; i <= maxRetries; i++) {