fixup!

Author: avivkeller

src/generators/ast-js/index.mjs

@@ -37,7 +37,7 @@ export default {
     const filePaths = itemIndices.map(idx => inputSlice[idx]);
 
     return Promise.all(
-      filePaths.map(async path => parseJsSource(await read(path)))
+      filePaths.map(async path => parseJsSource(await read(path, 'utf-8')))
     );
   },
 

src/generators/ast/index.mjs

@@ -5,8 +5,8 @@ import { extname } from 'node:path';
 import { globSync } from 'glob';
 import { read } from 'to-vfile';
 
+import createQueries from '../../utils/queries/index.mjs';
 import { getRemark } from '../../utils/remark.mjs';
-import createQueries from '../utils/queries/index.mjs';
 
 const { updateStabilityPrefixToLink } = createQueries();
 const remarkProcessor = getRemark();
@@ -40,7 +40,7 @@ export default {
 
     return Promise.all(
       filePaths.map(async path => {
-        const vfile = await read(path);
+        const vfile = await read(path, 'utf-8');
 
         updateStabilityPrefixToLink(vfile);
 

src/parsers/javascript.mjs

@@ -5,25 +5,21 @@ import * as acorn from 'acorn';
 /**
  * Parses a given JavaScript file into an ESTree AST representation of it
  *
- * @param {import('vfile').VFile | Promise<import('vfile').VFile>} sourceFile
+ * @param {import('vfile').VFile} sourceFile
  * @returns {Promise<JsProgram>}
  */
 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') {
+  if (typeof sourceFile.value !== 'string') {
     throw new TypeError(
-      `expected resolvedSourceFile.value to be string but got ${typeof resolvedSourceFile.value}`
+      `expected sourceFile.value to be string but got ${typeof sourceFile.value}`
     );
   }
 
-  const res = acorn.parse(resolvedSourceFile.value, {
+  const res = acorn.parse(sourceFile.value, {
     allowReturnOutsideFunction: true,
     ecmaVersion: 'latest',
     locations: true,
   });
 
-  return { ...res, path: resolvedSourceFile.path };
+  return { ...res, path: sourceFile.path };
 };