refactor: work synchronously

Author: brettz9

.babelrc.json

@@ -1,6 +1,7 @@
 {
   "plugins": [
-    "add-module-exports"
+    "add-module-exports",
+    "babel-plugin-transform-import-meta"
   ],
   "presets": [
     [

package.json

@@ -11,9 +11,10 @@
     "debug": "^4.3.4",
     "escape-string-regexp": "^4.0.0",
     "esquery": "^1.5.0",
-    "parse-imports": "^2.0.0",
+    "parse-imports": "^2.1.0",
     "semver": "^7.6.2",
-    "spdx-expression-parse": "^4.0.0"
+    "spdx-expression-parse": "^4.0.0",
+    "synckit": "^0.9.0"
   },
   "description": "JSDoc linting rules for ESLint.",
   "devDependencies": {
@@ -44,6 +45,7 @@
     "@typescript-eslint/parser": "^7.11.0",
     "babel-plugin-add-module-exports": "^1.0.4",
     "babel-plugin-istanbul": "^6.1.1",
+    "babel-plugin-transform-import-meta": "^2.2.1",
     "c8": "^9.1.0",
     "camelcase": "^6.3.0",
     "chai": "^4.3.10",
@@ -126,7 +128,7 @@
   "scripts": {
     "tsc": "tsc",
     "tsc-build": "tsc -p tsconfig-prod.json",
-    "build": "rimraf ./dist && cross-env NODE_ENV=production babel ./src --out-file-extension .cjs --out-dir ./dist --copy-files --source-maps --ignore ./src/bin/*.js --no-copy-ignored && replace 'require\\(\"\\.(.*?)\\.[^.]*?\"\\)' 'require(\".$1.cjs\")' 'dist' -r --include=\"*.cjs\" && pnpm tsc-build",
+    "build": "rimraf ./dist && cross-env NODE_ENV=production babel ./src --out-file-extension .cjs --out-dir ./dist --copy-files --source-maps --ignore ./src/bin/*.js --no-copy-ignored && replace 'require\\(\"\\.(.*?)\\.[^.]*?\"\\)' 'require(\".$1.cjs\")' 'dist' -r --include=\"*.cjs\" && cp src/import-worker.mjs dist/import-worker.mjs && pnpm tsc-build",
     "check-docs": "babel-node ./src/bin/generateDocs.js --check",
     "create-docs": "npm run create-options && babel-node ./src/bin/generateDocs.js",
     "create-rule": "babel-node ./src/bin/generateRule.js",

pnpm-lock.yaml

@@ -0,0 +1,11 @@
+import { runAsWorker } from 'synckit'
+
+runAsWorker(async (imprt) => {
+  const { parseImports } = await import('parse-imports');
+  try {
+    // ESLint doesn't support async rules
+    return [...await parseImports(imprt)];
+  } catch (err) {
+    return false;
+  }
+})

src/import-worker.mjs

@@ -1,8 +1,14 @@
-import { parseImportsSync } from 'parse-imports';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+import { createSyncFn } from 'synckit';
 import semver from 'semver';
 import spdxExpressionParse from 'spdx-expression-parse';
 import iterateJsdoc from '../iterateJsdoc.js';
 
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const pathName = join(__dirname, '../import-worker.mjs');
+
 const allowedKinds = new Set([
   'class',
   'constant',
@@ -169,11 +175,8 @@ export default iterateJsdoc(({
         ? `${typePart}${name} ${description}`
         : `${typePart}${name}`);
 
-      try {
-        // Should technically await non-sync, but ESLint doesn't support async rules;
-        //  thankfully, the Wasm load time is safely fast
-        parseImportsSync(imprt);
-      } catch (err) {
+      const getImports = createSyncFn(pathName);
+      if (!getImports(imprt)) {
         report(
           `Bad @import tag`,
           null,

src/rules/checkValues.js

@@ -1,4 +1,7 @@
-import { parseImportsSync } from 'parse-imports';
+import { dirname, join } from 'path';
+import { fileURLToPath } from 'url';
+
+import { createSyncFn } from 'synckit';
 import {
   getJSDocComment,
   parse as parseType,
@@ -9,6 +12,9 @@ import iterateJsdoc, {
   parseComment,
 } from '../iterateJsdoc.js';
 
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const pathName = join(__dirname, '../import-worker.mjs');
+
 const extraTypes = [
   'null', 'undefined', 'void', 'string', 'boolean', 'object',
   'function', 'symbol',
@@ -146,16 +152,13 @@ export default iterateJsdoc(({
       ? `${typePart}${name} ${description}`
       : `${typePart}${name}`);
 
-    let imports;
-    try {
-      // Should technically await non-sync, but ESLint doesn't support async rules;
-      //  thankfully, the Wasm load time is safely fast
-      imports = parseImportsSync(imprt);
-    } catch (err) {
+    const getImports = createSyncFn(pathName);
+    const imports = /** @type {import('parse-imports').Import[]} */ (getImports(imprt));
+    if (!imports) {
       return null;
     }
 
-    return [...imports].flatMap(({importClause}) => {
+    return imports.flatMap(({importClause}) => {
       /* c8 ignore next */
       const {default: dflt, named, namespace} = importClause || {};
       const types = [];

src/rules/noUndefinedTypes.js

@@ -17,7 +17,8 @@
     "src/**/*.js",
     "test/**/*.js",
     "typings/gitdown.d.ts",
-    "typings/babel__eslint-parser.d.ts"
+    "typings/babel__eslint-parser.d.ts",
+    "src/import-worker.mjs"
   ],
   "exclude": ["node_modules"]
 }