fix: don't globSync redundantly

Author: avivkeller

npm-shrinkwrap.json

@@ -75,6 +75,7 @@
     "rolldown": "^1.0.0-beta.53",
     "semver": "^7.7.3",
     "shiki": "^3.19.0",
+    "to-vfile": "^8.0.0",
     "unified": "^11.0.5",
     "unist-builder": "^4.0.0",
     "unist-util-find-after": "^5.0.0",

package.json

@@ -1,13 +1,9 @@
 import { extname } from 'node:path';
 
 import { globSync } from 'glob';
+import { read } from 'to-vfile';
 
-import createJsLoader from '../../loaders/javascript.mjs';
-import createJsParser from '../../parsers/javascript.mjs';
-
-const { loadFiles } = createJsLoader();
-
-const { parseJsSource } = createJsParser();
+import { parseJsSource } from '../../parsers/javascript.mjs';
 
 /**
  * This generator parses Javascript sources passed into the generator's input
@@ -40,19 +36,9 @@ export default {
   async processChunk(inputSlice, itemIndices) {
     const filePaths = itemIndices.map(idx => inputSlice[idx]);
 
-    const vfilesPromises = loadFiles(filePaths);
-
-    const results = [];
-
-    for (const vfilePromise of vfilesPromises) {
-      const vfile = await vfilePromise;
-
-      const parsed = await parseJsSource(vfile);
-
-      results.push(parsed);
-    }
-
-    return results;
+    return Promise.all(
+      filePaths.map(async path => parseJsSource(await read(path)))
+    );
   },
 
   /**

src/generators/ast-js/index.mjs

@@ -3,12 +3,10 @@
 import { extname } from 'node:path';
 
 import { globSync } from 'glob';
+import { read } from 'to-vfile';
 
-import createLoader from '../../loaders/markdown.mjs';
 import { getRemark } from '../../utils/remark.mjs';
 
-const { loadFiles } = createLoader();
-
 const remarkProcessor = getRemark();
 
 /**
@@ -38,20 +36,16 @@ export default {
   async processChunk(inputSlice, itemIndices) {
     const filePaths = itemIndices.map(idx => inputSlice[idx]);
 
-    const vfilesPromises = loadFiles(filePaths);
-
-    const results = [];
-
-    for (const vfilePromise of vfilesPromises) {
-      const vfile = await vfilePromise;
-
-      results.push({
-        tree: remarkProcessor.parse(vfile),
-        file: { stem: vfile.stem, basename: vfile.basename },
-      });
-    }
+    return Promise.all(
+      filePaths.map(async path => {
+        const vfile = await read(path);
 
-    return results;
+        return {
+          tree: remarkProcessor.parse(vfile),
+          file: { stem: vfile.stem, basename: vfile.basename },
+        };
+      })
+    );
   },
 
   /**

src/generators/ast/index.mjs

@@ -3,50 +3,27 @@
 import * as acorn from 'acorn';
 
 /**
- * Creates a Javascript source parser for a given source file
+ * Parses a given JavaScript file into an ESTree AST representation of it
+ *
+ * @param {import('vfile').VFile | Promise<import('vfile').VFile>} sourceFile
+ * @returns {Promise<JsProgram>}
  */
-const createParser = () => {
-  /**
-   * Parses a given JavaScript file into an ESTree AST representation of it
-   *
-   * @param {import('vfile').VFile | Promise<import('vfile').VFile>} sourceFile
-   * @returns {Promise<JsProgram>}
-   */
-  const parseJsSource = async sourceFile => {
-    // We allow the API doc VFile to be a Promise of a VFile also,
-    // hence we want to ensure that it first resolves before we pass it to the parser
-    const resolvedSourceFile = await Promise.resolve(sourceFile);
-
-    if (typeof resolvedSourceFile.value !== 'string') {
-      throw new TypeError(
-        `expected resolvedSourceFile.value to be string but got ${typeof resolvedSourceFile.value}`
-      );
-    }
-
-    const res = acorn.parse(resolvedSourceFile.value, {
-      allowReturnOutsideFunction: true,
-      ecmaVersion: 'latest',
-      locations: true,
-    });
-
-    return { ...res, path: resolvedSourceFile.path };
-  };
-
-  /**
-   * Parses multiple JavaScript files into ESTree ASTs by wrapping parseJsSource
-   *
-   * @param {Array<import('vfile').VFile | Promise<import('vfile').VFile>>} apiDocs List of API doc files to be parsed
-   * @returns {Promise<Array<JsProgram>>}
-   */
-  const parseJsSources = async apiDocs => {
-    // We do a Promise.all, to ensure that each API doc is resolved asynchronously
-    // but all need to be resolved first before we return the result to the caller
-    const resolvedApiDocEntries = await Promise.all(apiDocs.map(parseJsSource));
-
-    return resolvedApiDocEntries;
-  };
-
-  return { parseJsSource, parseJsSources };
+export const parseJsSource = async sourceFile => {
+  // We allow the API doc VFile to be a Promise of a VFile also,
+  // hence we want to ensure that it first resolves before we pass it to the parser
+  const resolvedSourceFile = await Promise.resolve(sourceFile);
+
+  if (typeof resolvedSourceFile.value !== 'string') {
+    throw new TypeError(
+      `expected resolvedSourceFile.value to be string but got ${typeof resolvedSourceFile.value}`
+    );
+  }
+
+  const res = acorn.parse(resolvedSourceFile.value, {
+    allowReturnOutsideFunction: true,
+    ecmaVersion: 'latest',
+    locations: true,
+  });
+
+  return { ...res, path: resolvedSourceFile.path };
 };
-
-export default createParser;