Streamline deno shims since not all are needed for distribution

- Deno.test and Deno.env are only used in tests
- Deno.version is only available for deno environments and does not need to be shimmed
- DOMException is only used in tests

Author: sebastianquek

CHANGELOG.md

@@ -12,6 +12,9 @@ and this project adheres to
 
 ### Changed
 
+- Exclude Deno and DOMException from distribution shim.
+- Refine retrieval of Node/Deno version.
+
 ### Fixed
 
 ### Removed

scripts/build_npm.ts

@@ -8,8 +8,15 @@ await build({
   rootTestDir: "./tests",
   outDir: "./npm",
   shims: {
-    deno: true, // Required for `Deno.test`, `Deno.env`, etc.
-    domException: true, // https://deno.land/std/async/delay.ts relies on DOMException
+    // Tests require `Deno.test` and `Deno.env`.
+    // Although `Deno.version` is used, it doesn't need to be shimmed since
+    // it won't be used when the user runs the module in Node.
+    deno: "dev",
+
+    // https://deno.land/std/async/delay.ts relies on DOMException.
+    // This is only used in tests.
+    domException: "dev", //  Only used in tests.
+
     undici: true, // Required for `fetch`
   },
   package: {

src/utils.ts

@@ -77,15 +77,23 @@ function getSource() {
   const moduleSource = `serpapi@${version}`;
   try {
     // Check if running in Node.js
+    // dnt-shim-ignore
     // deno-lint-ignore no-explicit-any
-    if ((globalThis as any)?.process?.version) {
-      // deno-lint-ignore no-explicit-any
-      const nodeVersion = (globalThis as any).process.version.replace("v", "");
+    const nodeVersion = (globalThis as any).process?.versions?.node;
+    if (nodeVersion) {
       return `nodejs@${nodeVersion},${moduleSource}`;
     }
 
-    // Assumes running in Deno instead
-    return `deno@${Deno.version.deno},${moduleSource}`;
+    // Assumes running in Deno instead. https://deno.land/api?s=Deno.version
+    // Deno.version is not shimmed since it's not used when ran in a Node env.
+    // dnt-shim-ignore
+    // deno-lint-ignore no-explicit-any
+    const denoVersion = (globalThis as any).Deno?.version?.deno;
+    if (denoVersion) {
+      return `deno@${denoVersion},${moduleSource}`;
+    }
+
+    return `nodejs,${moduleSource}`;
   } catch {
     // If something unexpectedly occurs, revert to "nodejs".
     return `nodejs,${moduleSource}`;