fix exports matching

This fixes the usage of `esmock` with the latest version of
`@modelcontextprotocol/sdk`, which introduced new exports entries.

See: modelcontextprotocol/typescript-sdk#851

Author: matz3

resolvewithplus.js

@@ -150,11 +150,19 @@ const gettargetnameandspecifier = target =>
 // the right hand side target pattern as a ** glob against the list of files
 // within the package.
 //
+// Paths need to match exactly. Partial matches are only valid when using the
+// star glob pattern.
+//
 // './lib/*' './lib/index' -> true
+// './lib/feature', './lib/feature' -> true
 // './lib/feature', './lib/index' -> false
+// './lib/feature', './lib/feature/something' -> false
 const ispathesmmatch = (pathesm, pathlocal) => {
   const isesmkeymatchRe = new RegExp(
-    pathesm.replace(esmStrPathCharRe, '\\$1').replace(esmStrGlobRe, '.*'))
+    '^' +
+    pathesm.replace(esmStrPathCharRe, '\\$1').replace(esmStrGlobRe, '.*') +
+    '$'
+  )
 
   return isesmkeymatchRe.test(pathlocal)
 }

tests/tests-basic/nodejsexample_02_exports/feature/stuff.js

@@ -0,0 +1 @@
+export default 'feature-stuff'

tests/tests-basic/nodejsexample_02_exports/lib/stuff.js

@@ -0,0 +1 @@
+export default 'lib-stuff'

tests/tests-basic/nodejsexample_15_exports/package.json

@@ -0,0 +1,19 @@
+{
+  "name": "nodejsexample_15_exports",
+  "type": "module",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/iambumblehead/resolvewithplus.git"
+  },
+  "exports": {
+    ".": {
+      "import": "./src/index.js"
+    },
+    "./mystuff": {
+      "import": "./src/mystuff/index.js"
+    },
+    "./*": {
+      "import": "./src/*"
+    }
+  }
+}

tests/tests-basic/nodejsexample_15_exports/src/index.js

@@ -0,0 +1 @@
+export default 'mystuff'

tests/tests-basic/nodejsexample_15_exports/src/mystuff/feature.js

@@ -0,0 +1 @@
+export default 'mystuff'

tests/tests-basic/nodejsexample_15_exports/src/mystuff/index.js

@@ -20,6 +20,7 @@
     "nodejsexample_12_exports": "file:nodejsexample_12_exports",
     "nodejsexample_13_exports": "file:nodejsexample_13_exports",
     "nodejsexample_14_exports": "file:nodejsexample_14_exports",
+    "nodejsexample_15_exports": "file:nodejsexample_15_exports",
     "resolvewithplus": "file:.."
   },
   "workspaces": [

tests/tests-basic/package.json

@@ -39,17 +39,25 @@ test('should return matched export paths', () => {
     .getesmkeyvalmatch(key, o[key], path)
 
   assert.strictEqual(
-    getmatch(exports, '.', './lib/index.test.js'),
+    getmatch(exports, '.', '.'),
     './lib/index.test.js')
 
   assert.strictEqual(
-    getmatch(exports, './lib', './lib/index.test.js'),
+    getmatch(exports, './lib', './lib'),
     './lib/index.test.js')
 
   assert.strictEqual(
     getmatch(exports, './lib/*', './lib/index.test.js'),
     './lib/index.test.js')
 
+  assert.strictEqual(
+    getmatch(exports, './lib/*', './lib/feature.js'),
+    './lib/feature.js')
+
+  assert.strictEqual(
+    getmatch(exports, './lib/*', './lib/feature'),
+    './lib/feature.js')
+
   assert.strictEqual(
     getmatch(exports, './submodule.js', './submodule.js'),
     './src/submodule.js')

tests/tests-basic/tests-basic.test.js

@@ -81,8 +81,12 @@ test('should mock all exports from nodejsexample_01_exports', () => {
 test('should mock all exports from nodejsexample_02_exports', () => {
   const noderesolvedlibindex = toresolvefileurl(
     './nodejsexample_02_exports/lib/index.test.js')
+  const noderesolvedlibstuff = toresolvefileurl(
+    './nodejsexample_02_exports/lib/stuff.js')
   const noderesolvedfeatureindex = toresolvefileurl(
     './nodejsexample_02_exports/feature/index.test.js')
+  const noderesolvedfeaturestuff = toresolvefileurl(
+    './nodejsexample_02_exports/feature/stuff.js')
   const noderesolvedpackagejson = toresolvefileurl(
     './nodejsexample_02_exports/package.json')
 
@@ -95,24 +99,24 @@ test('should mock all exports from nodejsexample_02_exports', () => {
     noderesolvedlibindex)
 
   assert.strictEqual(
-    resolvewithplus('nodejsexample_02_exports/lib/index'),
-    noderesolvedlibindex)
+    resolvewithplus('nodejsexample_02_exports/lib/stuff'),
+    noderesolvedlibstuff)
 
   assert.strictEqual(
-    resolvewithplus('nodejsexample_02_exports/lib/index.js'),
-    noderesolvedlibindex)
+    resolvewithplus('nodejsexample_02_exports/lib/stuff.js'),
+    noderesolvedlibstuff)
 
   assert.strictEqual(
     resolvewithplus('nodejsexample_02_exports/feature'),
     noderesolvedfeatureindex)
 
   assert.strictEqual(
-    resolvewithplus('nodejsexample_02_exports/feature/index'),
-    noderesolvedfeatureindex)
+    resolvewithplus('nodejsexample_02_exports/feature/stuff'),
+    noderesolvedfeaturestuff)
 
   assert.strictEqual(
-    resolvewithplus('nodejsexample_02_exports/feature/index.js'),
-    noderesolvedfeatureindex)
+    resolvewithplus('nodejsexample_02_exports/feature/stuff.js'),
+    noderesolvedfeaturestuff)
 
   assert.strictEqual(
     resolvewithplus('nodejsexample_02_exports/package.json'),
@@ -313,6 +317,32 @@ test('should mock exports from nodejsexample_14_exports, asterisk dir', () => {
     noderesolvedindex)
 })
 
+// "exports": {
+//   ".": {
+//     "import": "./src/index.js"
+//   },
+//   "./mystuff": {
+//     "import": "./src/mystuff/index.js"
+//   },
+//   "./*": {
+//     "import": "./src/*"
+//   }
+// }
+test('should mock exports from nodejsexample_15_exports, partial match', () => {
+  const noderesolvedmystuff = toresolvefileurl(
+    './nodejsexample_15_exports/src/mystuff/index.js')
+  const noderesolvedmystufffeature = toresolvefileurl(
+    './nodejsexample_15_exports/src/mystuff/feature.js')
+
+  assert.strictEqual(
+    resolvewithplus('nodejsexample_15_exports/mystuff'),
+    noderesolvedmystuff)
+
+  assert.strictEqual(
+    resolvewithplus('nodejsexample_15_exports/mystuff/feature.js'),
+    noderesolvedmystufffeature)
+})
+
 // "exports": './lib/index.js',
 // "exports": { "import": "./lib/index.js" },
 // "exports": { ".": "./lib/index.js" },