chore(commonjs): switch to fdir for fewer dependencies (#1741)

BREAKING CHANGES: directories listed in dynamicRequireTargets now require a trailing slash Co-authored-by: shellscape <andrew@shellscape.org>
rollup · Sep 23, 2024 · c2a05eb · c2a05eb
1 parent 05af988
commit c2a05eb
Show file tree

Hide file tree

Showing 8 changed files with 51 additions and 131 deletions.
diff --git a/packages/commonjs/package.json b/packages/commonjs/package.json
@@ -65,9 +65,10 @@
     "@rollup/pluginutils": "^5.0.1",
     "commondir": "^1.0.1",
     "estree-walker": "^2.0.2",
-    "glob": "^10.4.1",
+    "fdir": "^6.1.1",
     "is-reference": "1.2.1",
-    "magic-string": "^0.30.3"
+    "magic-string": "^0.30.3",
+    "picomatch": "^2.3.1"
   },
   "devDependencies": {
     "@rollup/plugin-json": "^5.0.0",

diff --git a/packages/commonjs/src/dynamic-modules.js b/packages/commonjs/src/dynamic-modules.js
@@ -3,7 +3,7 @@ import { join, resolve, dirname } from 'path';
 
 import getCommonDir from 'commondir';
 
-import { glob } from 'glob';
+import { fdir } from 'fdir';
 
 import { getVirtualPathForDynamicRequirePath, normalizePathSlashes } from './utils';
 
@@ -41,8 +41,13 @@ export function getDynamicRequireModules(patterns, dynamicRequireRoot) {
       isNegated
         ? dynamicRequireModules.delete(targetPath)
         : dynamicRequireModules.set(targetPath, resolvedPath);
-    for (const path of glob
-      .sync(isNegated ? pattern.substr(1) : pattern)
+    // eslint-disable-next-line new-cap
+    for (const path of new fdir()
+      .withBasePath()
+      .withDirs()
+      .glob(isNegated ? pattern.substr(1) : pattern)
+      .crawl()
+      .sync()
       .sort((a, b) => a.localeCompare(b, 'en'))) {
       const resolvedPath = resolve(path);
       const requirePath = normalizePathSlashes(resolvedPath);

diff --git a/packages/commonjs/test/fixtures/function/dynamic-require-instances/_config.js b/packages/commonjs/test/fixtures/function/dynamic-require-instances/_config.js
@@ -7,9 +7,9 @@ module.exports = {
   },
   pluginOptions: {
     dynamicRequireTargets: [
-      'fixtures/function/dynamic-require-instances/direct',
+      'fixtures/function/dynamic-require-instances/direct/',
       'fixtures/function/dynamic-require-instances/direct/index.js',
-      'fixtures/function/dynamic-require-instances/package',
+      'fixtures/function/dynamic-require-instances/package/',
       'fixtures/function/dynamic-require-instances/package/main.js'
     ]
   }

diff --git a/packages/commonjs/test/fixtures/function/dynamic-require-package/_config.js b/packages/commonjs/test/fixtures/function/dynamic-require-package/_config.js
@@ -8,9 +8,9 @@ module.exports = {
   },
   pluginOptions: {
     dynamicRequireTargets: [
-      'fixtures/function/dynamic-require-package',
-      'fixtures/function/dynamic-require-package/sub',
-      'fixtures/function/dynamic-require-package/node_modules/custom-module'
+      'fixtures/function/dynamic-require-package/',
+      'fixtures/function/dynamic-require-package/sub/',
+      'fixtures/function/dynamic-require-package/node_modules/custom-module/'
     ]
   }
 };
diff --git a/packages/commonjs/test/fixtures/function/dynamic-require-resolve-index/_config.js b/packages/commonjs/test/fixtures/function/dynamic-require-resolve-index/_config.js
@@ -7,9 +7,9 @@ module.exports = {
   },
   pluginOptions: {
     dynamicRequireTargets: [
-      'fixtures/function/dynamic-require-resolve-index',
-      'fixtures/function/dynamic-require-resolve-index/sub',
-      'fixtures/function/dynamic-require-resolve-index/node_modules/custom-module'
+      'fixtures/function/dynamic-require-resolve-index/',
+      'fixtures/function/dynamic-require-resolve-index/sub/',
+      'fixtures/function/dynamic-require-resolve-index/node_modules/custom-module/'
     ]
   }
 };
diff --git a/packages/commonjs/test/fixtures/function/dynamic-require-resolve-reference/_config.js b/packages/commonjs/test/fixtures/function/dynamic-require-resolve-reference/_config.js
@@ -7,7 +7,7 @@ module.exports = {
   },
   pluginOptions: {
     dynamicRequireTargets: [
-      'fixtures/function/dynamic-require-resolve-reference/node_modules/{custom-module,custom-module2}{,/*.js}'
+      'fixtures/function/dynamic-require-resolve-reference/node_modules/{custom-module,custom-module2}{/,/*.js}'
     ]
   }
 };
diff --git a/packages/commonjs/test/fixtures/function/dynamic-require-slash-access/_config.js b/packages/commonjs/test/fixtures/function/dynamic-require-slash-access/_config.js
@@ -7,9 +7,9 @@ module.exports = {
   },
   pluginOptions: {
     dynamicRequireTargets: [
-      'fixtures/function/dynamic-require-slash-access',
-      'fixtures/function/dynamic-require-slash-access/sub',
-      'fixtures/function/dynamic-require-slash-access/node_modules/custom-module',
+      'fixtures/function/dynamic-require-slash-access/',
+      'fixtures/function/dynamic-require-slash-access/sub/',
+      'fixtures/function/dynamic-require-slash-access/node_modules/custom-module/',
       'fixtures/function/dynamic-require-slash-access/node_modules/custom-module2/*.js'
     ]
   }