fix(npm): conditional exports with wildcards (#15652)

cli/tests/integration/npm_tests.rs

@@ -68,6 +68,13 @@ itest!(compare_globals {
   http_server: true,
 });
 
+itest!(conditional_exports {
+  args: "run --allow-read --unstable npm/conditional_exports/main.js",
+  output: "npm/conditional_exports/main.out",
+  envs: env_vars(),
+  http_server: true,
+});
+
 itest!(dynamic_import {
   args: "run --allow-read --allow-env --unstable npm/dynamic_import/main.ts",
   output: "npm/dynamic_import/main.out",

cli/tests/testdata/npm/conditional_exports/main.js

@@ -0,0 +1,9 @@
+import mod from "npm:@denotest/conditional-exports";
+import client from "npm:@denotest/conditional-exports/client";
+import clientFoo from "npm:@denotest/conditional-exports/client/foo";
+import clientBar from "npm:@denotest/conditional-exports/client/bar";
+
+console.log(mod);
+console.log(client);
+console.log(clientFoo);
+console.log(clientBar);

cli/tests/testdata/npm/conditional_exports/main.out

@@ -0,0 +1,6 @@
+Download http://localhost:4545/npm/registry/@denotest/conditional-exports
+Download http://localhost:4545/npm/registry/@denotest/conditional-exports/1.0.0.tgz
+{ hello: "from esm" }
+{ hello: "from esm client" }
+{ hello: "from esm client foo" }
+{ hello: "from esm client bar" }

cli/tests/testdata/npm/registry/@denotest/conditional-exports/1.0.0/cjs/index.cjs

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

cli/tests/testdata/npm/registry/@denotest/conditional-exports/1.0.0/esm/client/bar.js

@@ -0,0 +1,3 @@
+export default {
+    hello: "from esm client bar",
+}

cli/tests/testdata/npm/registry/@denotest/conditional-exports/1.0.0/esm/client/foo.js

@@ -0,0 +1,3 @@
+export default {
+    hello: "from esm client foo",
+}

cli/tests/testdata/npm/registry/@denotest/conditional-exports/1.0.0/esm/client/index.js

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

cli/tests/testdata/npm/registry/@denotest/conditional-exports/1.0.0/esm/index.js

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

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

@@ -0,0 +1,20 @@
+{
+  "name": "@denotest/conditional-exports",
+  "version": "1.0.0",
+  "type": "module",
+  "exports": {
+    ".": {
+      "types": "./types/src/index.d.ts",
+      "require": "./cjs/index.cjs",
+      "import": "./esm/index.js"
+    },
+    "./client": {
+      "types": "./types/src/client/index.d.ts",
+      "import": "./esm/client/index.js"
+    },
+    "./client/*": {
+      "types": "./types/src/client/*.d.ts",
+      "import": "./esm/client/*.js"
+    }
+  }
+}

ext/node/resolution.rs

@@ -451,7 +451,7 @@ pub fn package_exports_resolve(
   for key in package_exports.keys() {
     let pattern_index = key.find('*');
     if let Some(pattern_index) = pattern_index {
-      let key_sub = &key[0..=pattern_index];
+      let key_sub = &key[0..pattern_index];
       if package_subpath.starts_with(key_sub) {
         // When this reaches EOL, this can throw at the top of the whole function:
         //
@@ -472,7 +472,7 @@ pub fn package_exports_resolve(
           best_match = key;
           best_match_subpath = Some(
             package_subpath
-              [pattern_index..=(package_subpath.len() - pattern_trailer.len())]
+              [pattern_index..(package_subpath.len() - pattern_trailer.len())]
               .to_string(),
           );
         }