refactor: reuse exports from src/node.ts

pi0 · pi0 · commit a6e602d57f13 · 2023-08-22T19:19:40.000+02:00
diff --git a/src/_utils.ts b/src/_utils.ts
diff --git a/src/index.ts b/src/index.ts
@@ -1,18 +1,25 @@
-import _fetch, {
+import {
+  fetch as _fetch,
+  AbortController as _AbortController,
   Blob as _Blob,
   File as _File,
   FormData as _FormData,
   Headers as _Headers,
   Request as _Request,
   Response as _Response,
-} from "node-fetch";
+} from "./node";
 
-import _AbortController from "abort-controller";
+export {
+  AbortError,
+  FetchError,
+  blobFrom,
+  blobFromSync,
+  fileFrom,
+  fileFromSync,
+  isRedirect,
+} from "./node";
 
-const _forceNodeFetch =
-  typeof process !== undefined &&
-  typeof process.env !== undefined &&
-  process.env.FORCE_NODE_FETCH;
+const _forceNodeFetch = !!globalThis.process?.env?.FORCE_NODE_FETCH;
 
 function _getFetch() {
   if (!_forceNodeFetch && globalThis.fetch) {
@@ -32,13 +39,3 @@ export const Request = (!_forceNodeFetch && globalThis.Request) || _Request;
 export const Response = (!_forceNodeFetch && globalThis.Response) || _Response;
 export const AbortController =
   (!_forceNodeFetch && globalThis.AbortController) || _AbortController;
-
-export {
-  AbortError,
-  FetchError,
-  blobFrom,
-  blobFromSync,
-  fileFrom,
-  fileFromSync,
-  isRedirect,
-} from "node-fetch";
diff --git a/src/native.ts b/src/native.ts
@@ -9,6 +9,8 @@ export const AbortController = globalThis.AbortController;
 export const fetch =
   globalThis.fetch ||
   (() => {
-    throw new Error("global fetch is not available!");
+    throw new Error(
+      "[node-fetch-native] Failed to fetch: `globalThis.fetch` is not available!",
+    );
   });
 export default fetch;
diff --git a/src/node.ts b/src/node.ts
@@ -1,11 +1,8 @@
 import _fetch from "node-fetch";
-import { checkNodeEnvironment } from "./_utils";
 
 export { Blob, File, FormData, Headers, Request, Response } from "node-fetch";
 export { default as AbortController } from "abort-controller";
 
-checkNodeEnvironment();
-
 export const fetch = _fetch;
 export default fetch;
 
@@ -18,3 +15,16 @@ export {
   fileFromSync,
   isRedirect,
 } from "node-fetch";
+
+checkNodeEnvironment();
+
+function checkNodeEnvironment() {
+  if (
+    !globalThis.process?.versions?.node &&
+    !globalThis.process?.env.DISABLE_NODE_FETCH_NATIVE_WARN
+  ) {
+    throw new Error(
+      "Node.js compatible build of `node-fetch-native` is being used in a non-Node.js environment. Please make sure you are using proper export conditions or report this issue to https://github.com/unjs/node-fetch-native. You can set `process.env.DISABLE_NODE_FETCH_NATIVE_WARN` to disable this warning.",
+    );
+  }
+}
diff --git a/src/polyfill.ts b/src/polyfill.ts
@@ -1,17 +1,13 @@
-import _fetch, {
+import {
+  fetch as _fetch,
   Blob as _Blob,
   File as _File,
   FormData as _FormData,
   Headers as _Headers,
   Request as _Request,
   Response as _Response,
-} from "node-fetch";
-
-import _AbortController from "abort-controller";
-
-import { checkNodeEnvironment } from "./_utils";
-
-checkNodeEnvironment();
+  AbortController as _AbortController,
+} from "./node";
 
 function polyfill(name: string, impl: any) {
   if (!(name in globalThis)) {
