feat: always polyfill fetch (#1)

Node 18+ introduces the fetch API globally but it does not yet support all our use-cases.

Author: jacob-ebey

packages/fetch/src/lib.node.js

@@ -1,15 +1,15 @@
 import * as WebFetch from "./fetch.js"
 
 export { ReadableStream, Blob, FormData  } from './package.js';
-// Electron-renderer should get the browser implementation instead of node
-// Browser configuration is not enough
+// Node 18+ introduces fetch API globally and it doesn't support our use-cases yet.
+// For now we always use the polyfill.
 
 // Marking export as a DOM File object instead of custom class.
 export const fetch = /** @type {typeof globalThis.fetch} */
-  (typeof globalThis.fetch === "function" ? globalThis.fetch : WebFetch.fetch)
+  WebFetch.fetch
 
-export const Headers = globalThis.Headers || WebFetch.Headers
-export const Request = globalThis.Request || WebFetch.Request
-export const Response = global.Response || WebFetch.Response
+export const Headers = WebFetch.Headers
+export const Request = WebFetch.Request
+export const Response = WebFetch.Response
 
 export default fetch

packages/file/src/lib.node.js

@@ -8,6 +8,6 @@ import { File as WebFile } from "./file.js"
 
 // Marking export as a DOM File object instead of custom class.
 /** @type {typeof globalThis.File} */
-const File = typeof globalThis.File === "function" ? globalThis.File : WebFile
+const File = WebFile
 
 export { File, Blob }

packages/form-data/src/lib.node.js

@@ -3,9 +3,4 @@
 
 import * as polyfill from "./form-data.js"
 
-// Electron-renderer should get the browser implementation instead of node
-// which is why we check global first.
-export const FormData =
-  typeof globalThis.FormData === "function"
-    ? globalThis.FormData
-    : polyfill.FormData
+export const FormData = polyfill.FormData