fix(node): Include "node" condition during CJS re-export analysis

ext/node_resolver/analyze.rs

@@ -196,7 +196,7 @@ impl<TCjsCodeAnalyzer: CjsCodeAnalyzer, TNodeResolverEnv: NodeResolverEnv>
             &referrer,
             // FIXME(bartlomieju): check if these conditions are okay, probably
             // should be `deno-require`, because `deno` is already used in `esm_resolver.rs`
-            &["deno", "require", "default"],
+            &["deno", "node", "require", "default"],
             NodeResolutionMode::Execution,
           );
           let reexport_specifier = match result {

tests/registry/npm/@denotest/conditional-exports-node/1.0.0/bad.js

@@ -0,0 +1,3 @@
+module.exports = {
+  hello: "bad"
+};

tests/registry/npm/@denotest/conditional-exports-node/1.0.0/good.cjs

@@ -0,0 +1,3 @@
+module.exports = {
+    hello: "from node"
+};

tests/registry/npm/@denotest/conditional-exports-node/1.0.0/index.js

@@ -0,0 +1,3 @@
+export default {
+  hello: "default export"
+}

tests/registry/npm/@denotest/conditional-exports-node/1.0.0/package.json

@@ -0,0 +1,13 @@
+{
+  "name": "@denotest/conditional-exports",
+  "version": "1.0.0",
+  "type": "module",
+  "exports": {
+    ".": {
+      "node": "./good.cjs",
+      "require": "./bad.js",
+      "default": "./index.js"
+    },
+    "./*": "./*"
+  }
+}

tests/registry/npm/@denotest/mjs-reexport-cjs/1.0.0/dist/package.cjs.js

@@ -0,0 +1,7 @@
+Object.defineProperty(exports, '__esModule', { value: true });
+
+const pkg = require("@denotest/conditional-exports-node");
+
+Object.keys(pkg).forEach(function (k) {
+  if (k !== 'default' && !Object.prototype.hasOwnProperty.call(exports, k)) exports[k] = pkg[k];
+});

tests/registry/npm/@denotest/mjs-reexport-cjs/1.0.0/index.js

@@ -0,0 +1 @@
+module.exports = require('./dist/package.cjs.js')

tests/registry/npm/@denotest/mjs-reexport-cjs/1.0.0/index.mjs

@@ -0,0 +1 @@
+export * from "./index.js";

tests/registry/npm/@denotest/mjs-reexport-cjs/1.0.0/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@denotest/mjs-reexport-cjs",
+  "version": "1.0.0",
+  "dependencies": {
+    "@denotest/conditional-exports-node": "*"
+  },
+  "exports": {
+    "node": "./index.mjs",
+    "default": "./index.js"
+  }
+}

tests/specs/node/cjs_reexport_node_condition/__test__.jsonc

@@ -0,0 +1,13 @@
+{
+  "tempDir": true,
+  "steps": [
+    {
+      "args": "install",
+      "output": "[WILDCARD]"
+    },
+    {
+      "args": "run -A ./main.ts",
+      "output": "from node\n"
+    }
+  ]
+}

tests/specs/node/cjs_reexport_node_condition/deno.json

@@ -0,0 +1,3 @@
+{
+  "nodeModulesDir": "manual"
+}

tests/specs/node/cjs_reexport_node_condition/main.ts

@@ -0,0 +1,3 @@
+import { hello } from "@denotest/mjs-reexport-cjs";
+
+console.log(hello);

tests/specs/node/cjs_reexport_node_condition/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies": {
+    "@denotest/mjs-reexport-cjs": "*"
+  }
+}