reverse-exports: fix matching pattern to use single asterisk as catch…

…-all

packages/reverse-exports/package.json

@@ -9,11 +9,7 @@
   "keywords": [],
   "author": "",
   "license": "ISC",
-  "devDependencies": {
-    "@types/minimatch": "^3.0.4"
-  },
   "dependencies": {
-    "minimatch": "^3.0.4",
     "resolve.exports": "^2.0.2"
   }
 }

packages/reverse-exports/src/index.ts

@@ -1,5 +1,4 @@
 import { posix } from 'path';
-import minimatch from 'minimatch';
 import { exports as resolveExports } from 'resolve.exports';
 
 type Exports = string | string[] | { [key: string]: Exports };
@@ -82,12 +81,9 @@ export default function reversePackageExports(
   }
 
   const maybeKeyValuePair = _findPathRecursively(exportsObj, candidate => {
-    // miminatch does not treat directories as full of content without glob
-    if (candidate.endsWith('/')) {
-      candidate += '**';
-    }
+    const regex = new RegExp(_prepareStringForRegex(candidate));
 
-    return minimatch(relativePath, candidate);
+    return regex.test(relativePath);
   });
 
   if (!maybeKeyValuePair) {
@@ -114,3 +110,16 @@ export default function reversePackageExports(
 
   return resolvedPath.replace(/^./, name);
 }
+
+export function _prepareStringForRegex(input: string): string {
+  let result = input
+    .split('*')
+    .map(substr => substr.replace(/[/\-\\^$*+?.()|[\]{}]/g, '\\$&'))
+    .join('.*');
+
+  if (result.endsWith('/')) {
+    result += '.*';
+  }
+
+  return `^${result}$`;
+}

packages/reverse-exports/tests/reverse-exports.test.ts

@@ -1,4 +1,4 @@
-import reversePackageExports, { _findPathRecursively } from '../src';
+import reversePackageExports, { _findPathRecursively, _prepareStringForRegex } from '../src';
 
 describe('reverse exports', function () {
   it('exports is missing', function () {
@@ -141,7 +141,7 @@ describe('reverse exports', function () {
     );
   });
 
-  it('breaks TODO rename this test to something better', function () {
+  it('conditional exports: using a single asterisk as glob for nested path', function () {
     const packageJson = {
       name: 'my-v2-addon',
       exports: {
@@ -278,3 +278,16 @@ describe('_findKeyRecursively', function () {
     expect(_findPathRecursively(exports, str => str === './missing-path.js')).toBe(undefined);
   });
 });
+
+describe('_prepareStringForRegex', function () {
+  [
+    { input: './foo', expected: '^\\.\\/foo$' },
+    { input: './foo.js', expected: '^\\.\\/foo\\.js$' },
+    { input: './foo/*.js', expected: '^\\.\\/foo\\/.*\\.js$' },
+    { input: './foo/', expected: '^\\.\\/foo\\/.*$' },
+  ].forEach(({ input, expected }) => {
+    it(input, function () {
+      expect(_prepareStringForRegex(input)).toStrictEqual(expected);
+    });
+  });
+});