wip

eslint.config.mjs

@@ -26,9 +26,6 @@ export default [
     ]
   },
   {
-    "languageOptions": {
-      "sourceType": "commonjs"
-    },
     "linterOptions": {
       "reportUnusedDisableDirectives": true
     },
@@ -68,10 +65,20 @@ export default [
   },
   {
     "files": [
-      "**/*.mjs"
+      "**/*-formatter-*.js",
+      "webworker/*.js"
+    ],
+    "ignores": [
+      "webworker/fs-virtual.js"
     ],
     "languageOptions": {
-      "sourceType": "module"
+      "sourceType": "commonjs",
+      "globals": {
+        "__dirname": "readonly",
+        "__filename": "readonly",
+        "module": "readonly",
+        "require": "readonly"
+      }
     }
   }
 ];

export-markdownlint-helpers.js

@@ -1,4 +1,3 @@
 // @ts-check
 
-import helpers from "markdownlint/helpers";
-export default helpers;
+export { default } from "markdownlint/helpers";

export-markdownlint.js

@@ -1,4 +1,3 @@
 // @ts-check
 
-import markdownlint from "markdownlint";
-export default markdownlint;
+export { default } from "markdownlint";

markdownlint-cli2.js

@@ -4,7 +4,7 @@
 
 // Requires
 import fsx from "node:fs";
-import { createRequire } from 'node:module';
+import { createRequire } from "node:module";
 const dynamicRequire = createRequire(import.meta.url);
 import os from "node:os";
 import pathDefault from "node:path";
@@ -211,12 +211,12 @@ const readOptionsOrConfig = async (configPath, fs, noRequire) => {
 };
 
 // Filter a list of files to ignore by glob
-const removeIgnoredFiles = (dir, files, ignores) => {
-  return micromatch(
+const removeIgnoredFiles = (dir, files, ignores) => (
+  micromatch(
     files.map((file) => pathPosix.relative(dir, file)),
     ignores
-  ).map((file) => pathPosix.join(dir, file));
-};
+  ).map((file) => pathPosix.join(dir, file))
+);
 
 // Process/normalize command-line arguments and return glob patterns
 const processArgv = (argv) => {
@@ -886,6 +886,7 @@ const outputSummary = async (
     const dirs = [ dir, ...modulePaths ];
     const formattersAndParams = outputFormatters
       ? await importOrRequireIdsAndParams(dirs, outputFormatters, noRequire)
+      // eslint-disable-next-line unicorn/no-await-expression-member
       : [ [ (await import("markdownlint-cli2-formatter-default")).default ] ];
     await Promise.all(formattersAndParams.map((formatterAndParams) => {
       const [ formatter, ...formatterParams ] = formatterAndParams;

package.json

@@ -95,7 +95,6 @@
     "eslint-plugin-jsdoc": "50.5.0",
     "eslint-plugin-n": "17.13.2",
     "eslint-plugin-unicorn": "56.0.0",
-    "nano-spawn": "0.2.0",
     "markdown-it-emoji": "3.0.0",
     "markdown-it-for-inline": "2.0.1",
     "markdownlint-cli2-formatter-codequality": "0.0.5",
@@ -106,6 +105,7 @@
     "markdownlint-cli2-formatter-summarize": "0.0.7",
     "markdownlint-cli2-formatter-template": "0.0.2",
     "markdownlint-rule-extended-ascii": "0.1.0",
+    "nano-spawn": "0.2.0",
     "npm-run-all": "4.1.5"
   },
   "keywords": [

test/fs-mock.js

@@ -3,9 +3,9 @@
 import fs from "node:fs";
 import nodePath from "node:path";
 
-const mapPath = (base, mockPath) => {
-  return nodePath.resolve(base, nodePath.relative("/mock", mockPath));
-};
+const mapPath = (base, mockPath) => (
+  nodePath.resolve(base, nodePath.relative("/mock", mockPath))
+);
 
 class fsMock {
   constructor(base, symbolicLinks) {

test/import-with-type-json.mjs

@@ -4,4 +4,9 @@ import fs from "node:fs/promises";
 import path from "node:path";
 
 // Avoids "ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time"
-export default async (file) => JSON.parse(await fs.readFile(path.resolve(import.meta.dirname, file), "utf8"));
+const importWithTypeJson = async (file) => (
+  // @ts-ignore
+  JSON.parse(await fs.readFile(path.resolve(import.meta.dirname, file)))
+);
+
+export default importWithTypeJson;

test/markdownlint-cli2-test-exports.js

@@ -4,9 +4,6 @@ import test from "ava";
 import importWithTypeJson from "./import-with-type-json.mjs";
 const packageJson = await importWithTypeJson("../package.json");
 
-import { createRequire } from "node:module";
-const require = createRequire(import.meta.url);
-
 const exportMappings = new Map([
   [ ".", "../markdownlint-cli2.js" ],
   [ "./markdownlint", "markdownlint" ],
@@ -23,9 +20,11 @@ test("exportMappings", (t) => {
   );
 });
 
+const commonJsRe = /.js$/u;
+
 for (const [ exportName, exportPath ] of exportMappings) {
   test(exportName, async (t) => {
-    const commonJs = !/.js$/.test(exportPath);
+    const commonJs = !commonJsRe.test(exportPath);
     const importExportName = await import(exportName.replace(/^\./u, packageJson.name));
     const importExportPath = await import(exportPath);
     t.is(

test/resolve-and-require-test.js

@@ -7,8 +7,6 @@ import resolveAndRequire from "../resolve-and-require.js";
 import { createRequire } from "node:module";
 const require = createRequire(import.meta.url);
 
-/* eslint-disable n/no-missing-require */
-
 test("built-in module", (t) => {
   t.plan(1);
   t.deepEqual(