markdownlint 0.40.0 → 0.41.0

package/lib/md060.mjs

@@ -7,6 +7,7 @@ import stringWidth from "string-width";
 /** @typedef {import("micromark-extension-gfm-table")} */
 /** @typedef {import("markdownlint").MicromarkToken} MicromarkToken */
 /** @typedef {import("markdownlint").RuleOnErrorInfo} RuleOnErrorInfo */
+/** @typedef {import("markdownlint").RuleOnErrorFixInfo} RuleOnErrorFixInfo */
 
 /**
  * Adds a RuleOnErrorInfo object to a list of RuleOnErrorInfo objects.
@@ -15,12 +16,14 @@ import stringWidth from "string-width";
  * @param {number} lineNumber Line number.
  * @param {number} column Column number.
  * @param {string} detail Detail message.
+ * @param {RuleOnErrorFixInfo} [fixInfo] Fix information.
  */
-function addError(errors, lineNumber, column, detail) {
+function addError(errors, lineNumber, column, detail, fixInfo) {
   errors.push({
     lineNumber,
     detail,
-    "range": [ column, 1 ]
+    "range": [ column, 1 ],
+    fixInfo
   });
 }
 
@@ -123,11 +126,29 @@ export default {
               if (previous) {
                 if (previous.type === "whitespace") {
                   if (previous.text.length !== 1) {
-                    addError(errorsIfCompact, startLine, startColumn, "Table pipe has extra space to the left for style \"compact\"");
+                    addError(
+                      errorsIfCompact,
+                      startLine,
+                      startColumn,
+                      "Table pipe has extra space to the left for style \"compact\"",
+                      { "editColumn": previous.startColumn, "deleteCount": previous.text.length - 1 }
+                    );
                   }
-                  addError(errorsIfTight, startLine, startColumn, "Table pipe has space to the left for style \"tight\"");
+                  addError(
+                    errorsIfTight,
+                    startLine,
+                    startColumn,
+                    "Table pipe has space to the left for style \"tight\"",
+                    { "editColumn": previous.startColumn, "deleteCount": previous.text.length }
+                  );
                 } else {
-                  addError(errorsIfCompact, startLine, startColumn, "Table pipe is missing space to the left for style \"compact\"");
+                  addError(
+                    errorsIfCompact,
+                    startLine,
+                    startColumn,
+                    "Table pipe is missing space to the left for style \"compact\"",
+                    { "editColumn": previous.endColumn, "insertText": " " }
+                  );
                 }
               }
               const next = tokensOfInterest[i + 1];
@@ -135,12 +156,30 @@ export default {
                 if (next.type === "whitespace") {
                   if (next.endColumn !== row.endColumn) {
                     if (next.text.length !== 1) {
-                      addError(errorsIfCompact, startLine, startColumn, "Table pipe has extra space to the right for style \"compact\"");
+                      addError(
+                        errorsIfCompact,
+                        startLine,
+                        startColumn,
+                        "Table pipe has extra space to the right for style \"compact\"",
+                        { "editColumn": next.startColumn, "deleteCount": next.text.length - 1 }
+                      );
                     }
-                    addError(errorsIfTight, startLine, startColumn, "Table pipe has space to the right for style \"tight\"");
+                    addError(
+                      errorsIfTight,
+                      startLine,
+                      startColumn,
+                      "Table pipe has space to the right for style \"tight\"",
+                      { "editColumn": next.startColumn, "deleteCount": next.text.length }
+                    );
                   }
                 } else {
-                  addError(errorsIfCompact, startLine, startColumn, "Table pipe is missing space to the right for style \"compact\"");
+                  addError(
+                    errorsIfCompact,
+                    startLine,
+                    startColumn,
+                    "Table pipe is missing space to the right for style \"compact\"",
+                    { "editColumn": next.startColumn, "insertText": " " }
+                  );
                 }
               }
             }

package/lib/micromark-parse.mjs

@@ -10,9 +10,10 @@ import { parse as micromarkParse, postprocess as micromarkPostprocess, preproces
 // Unfortunately, omitting this dependency from package.json breaks strict dependency resolution (e.g., pnpm)
 import { labelEnd } from "micromark-core-commonmark";
 import { isHtmlFlowComment } from "../helpers/micromark-helpers.cjs";
-import { flatTokensSymbol, htmlFlowSymbol, newLineRe } from "../helpers/shared.cjs";
+import { flatTokensSymbol, htmlFlowSymbol, newlineRe } from "../helpers/shared.cjs";
 
 /** @typedef {import("micromark-util-types").Event} Event */
+/** @typedef {import("micromark-util-types").Extension} Extension */
 /** @typedef {import("micromark-util-types").ParseOptions} MicromarkParseOptions */
 /** @typedef {import("micromark-util-types").State} State */
 /** @typedef {import("micromark-util-types").Token} Token */
@@ -20,6 +21,20 @@ import { flatTokensSymbol, htmlFlowSymbol, newLineRe } from "../helpers/shared.c
 /** @typedef {import("markdownlint").MicromarkToken} MicromarkToken */
 /** @typedef {import("./micromark-types.d.mts")} */
 
+/**
+ * Gets a modified micromark-extension-directive without support for inline directives.
+ *
+ * @see {@link https://github.com/DavidAnson/markdownlint/issues/2080}
+ * @returns {Extension} Modified implementation.
+ */
+function directiveNoInline() {
+  const extension = {
+    ...directive()
+  };
+  delete extension.text;
+  return extension;
+}
+
 /**
  * Gets the Markdown text for a Micromark token.
  *
@@ -51,7 +66,7 @@ export function getEvents(
 ) {
   // Customize extensions list to add useful extensions
   const extensions = [
-    directive(),
+    directiveNoInline(),
     gfmAutolinkLiteral(),
     gfmFootnote(),
     gfmTable(),
@@ -66,7 +81,7 @@ export function getEvents(
 
   /** @type {Tokenizer} */
   function tokenizeShim(effects, okOriginal, nokOriginal) {
-    // eslint-disable-next-line consistent-this, unicorn/no-this-assignment, no-invalid-this
+    // eslint-disable-next-line consistent-this, no-invalid-this, unicorn/no-this-assignment, unicorn/no-this-outside-of-class
     const tokenizeContext = this;
     const events = tokenizeContext.events;
 
@@ -269,7 +284,7 @@ function parseInternal(
               }
             ]
           };
-          lines = markdown.split(newLineRe);
+          lines = markdown.split(newlineRe);
         }
         const reparseMarkdown = lines
           .slice(current.startLine - 1, current.endLine)

package/lib/resolve-module.cjs

@@ -2,10 +2,10 @@
 
 "use strict";
 
-// @ts-ignore
-// eslint-disable-next-line camelcase, no-inline-comments, no-undef
-const nativeRequire = (typeof __non_webpack_require__ === "undefined") ? require : /* c8 ignore next */ __non_webpack_require__;
 // Captures the native require implementation (even under webpack).
+// @ts-ignore
+// eslint-disable-next-line no-underscore-dangle
+const nativeRequire = globalThis.__non_webpack_require__ ?? require;
 
 /**
  * @typedef RequireResolveOptions

package/lib/resolve-module.d.cts

@@ -2,7 +2,7 @@ export type RequireResolveOptions = {
     /**
      * Additional paths to resolve from.
      */
-    paths?: string[];
+    paths?: string[] | undefined;
 };
 export type RequireResolve = (id: string, options: RequireResolveOptions) => string;
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "markdownlint",
-  "version": "0.40.0",
+  "version": "0.41.0",
   "description": "A Node.js style checker and lint tool for Markdown/CommonMark files.",
   "type": "module",
   "exports": {
@@ -36,10 +36,10 @@
     "build-config": "npm run build-config-schema && npm run build-config-example",
     "build-config-example": "node schema/build-config-example.mjs",
     "build-config-schema": "node schema/build-config-schema.mjs",
-    "build-declaration": "tsc --allowJs --checkJs --declaration --emitDeclarationOnly --module nodenext --outDir dts --rootDir . --target es2015 lib/exports.mjs lib/exports-async.mjs lib/exports-promise.mjs lib/exports-sync.mjs lib/markdownlint.mjs lib/resolve-module.cjs && node scripts/index.mjs copy dts/lib/exports.d.mts lib/exports.d.mts && node scripts/index.mjs copy dts/lib/exports-async.d.mts lib/exports-async.d.mts && node scripts/index.mjs copy dts/lib/exports-promise.d.mts lib/exports-promise.d.mts && node scripts/index.mjs copy dts/lib/exports-sync.d.mts lib/exports-sync.d.mts && node scripts/index.mjs copy dts/lib/markdownlint.d.mts lib/markdownlint.d.mts && node scripts/index.mjs copy dts/lib/resolve-module.d.cts lib/resolve-module.d.cts && node scripts/index.mjs remove dts",
+    "build-declaration": "tsc --project ./declaration/tsconfig.json && node scripts/index.mjs copy dts/lib/exports.d.mts lib/exports.d.mts && node scripts/index.mjs copy dts/lib/exports-async.d.mts lib/exports-async.d.mts && node scripts/index.mjs copy dts/lib/exports-promise.d.mts lib/exports-promise.d.mts && node scripts/index.mjs copy dts/lib/exports-sync.d.mts lib/exports-sync.d.mts && node scripts/index.mjs copy dts/lib/markdownlint.d.mts lib/markdownlint.d.mts && node scripts/index.mjs copy dts/lib/resolve-module.d.cts lib/resolve-module.d.cts && node scripts/index.mjs remove dts",
     "build-demo": "node scripts/index.mjs copy node_modules/markdown-it/dist/markdown-it.min.js demo/markdown-it.min.js && cd demo && webpack --no-stats",
     "build-docs": "node doc-build/build-rules.mjs",
-    "ci": "npm-run-all --continue-on-error --parallel build-demo lint serial-config-docs serial-declaration test-cover && git diff --exit-code",
+    "ci": "concurrently npm:build-demo npm:lint npm:serial-config-docs npm:serial-declaration npm:test && git diff --exit-code && echo PASS",
     "clone-test-repos-apache-airflow": "cd test-repos && git clone https://github.com/apache/airflow apache-airflow --depth 1 --no-tags --quiet",
     "clone-test-repos-dotnet-docs": "cd test-repos && git clone https://github.com/dotnet/docs dotnet-docs --depth 1 --no-tags --quiet",
     "clone-test-repos-electron-electron": "cd test-repos && git clone https://github.com/electron/electron electron-electron --depth 1 --no-tags --quiet && cd electron-electron && node ../../scripts/index.mjs delete package.json && npm install --no-save --ignore-scripts @electron/lint-roller",
@@ -52,25 +52,22 @@
     "clone-test-repos-webhintio-hint": "cd test-repos && git clone https://github.com/webhintio/hint webhintio-hint --depth 1 --no-tags --quiet",
     "clone-test-repos-webpack-webpack-js-org": "cd test-repos && git clone https://github.com/webpack/webpack.js.org webpack-webpack-js-org --depth 1 --no-tags --quiet",
     "clone-test-repos": "mkdir test-repos && cd test-repos && npm run clone-test-repos-apache-airflow && npm run clone-test-repos-dotnet-docs && npm run clone-test-repos-electron-electron && npm run clone-test-repos-eslint-eslint && npm run clone-test-repos-mdn-content && npm run clone-test-repos-mkdocs-mkdocs && npm run clone-test-repos-mochajs-mocha && npm run clone-test-repos-pi-hole-docs && npm run clone-test-repos-v8-v8-dev && npm run clone-test-repos-webhintio-hint && npm run clone-test-repos-webpack-webpack-js-org",
-    "declaration": "npm run build-declaration && npm run test-declaration",
     "example": "cd example && node standalone.mjs",
     "lint": "eslint --max-warnings 0",
-    "lint-test-repos": "ava --timeout=10m test/markdownlint-test-repos-*.mjs",
+    "lint-test-repos": "node --test test/markdownlint-test-repos-*.mjs",
     "serial-config-docs": "npm run build-config && npm run build-docs",
     "serial-declaration": "npm run build-declaration && npm run test-declaration",
-    "test": "ava --timeout=30s test/markdownlint-test.mjs test/markdownlint-test-config.mjs test/markdownlint-test-custom-rules.mjs test/markdownlint-test-exports.mjs test/markdownlint-test-fixes.mjs test/markdownlint-test-helpers.mjs test/markdownlint-test-micromark.mjs test/markdownlint-test-project.mjs test/markdownlint-test-result-object.mjs test/markdownlint-test-scenarios.mjs test/parse-configuration-test.mjs test/resolve-module-test.mjs helpers/test.cjs",
-    "test-cover": "c8 --100 npm test",
-    "test-declaration": "npm-run-all --continue-on-error --parallel test-declaration-cts test-declaration-mts",
-    "test-declaration-cts": "cd example/typescript && node ../../scripts/index.mjs copy type-check.ts type-check-commonjs.cts && tsc --module commonjs --esModuleInterop type-check-commonjs.cts",
-    "test-declaration-mts": "cd example/typescript && node ../../scripts/index.mjs copy type-check.ts type-check-nodenext.mts && tsc --module nodenext                   type-check-nodenext.mts && node type-check-nodenext.mjs",
-    "test-extra": "ava --timeout=10m test/markdownlint-test-extra-parse.mjs test/markdownlint-test-extra-type.mjs",
-    "update-snapshots": "ava --update-snapshots test/markdownlint-test-custom-rules.mjs test/markdownlint-test-exports.mjs test/markdownlint-test-micromark.mjs test/markdownlint-test-scenarios.mjs",
-    "update-snapshots-test-repos": "ava --timeout=10m --update-snapshots test/markdownlint-test-repos-*.mjs",
-    "update-test-repos": "node scripts/index.mjs remove ./test-repos && npm run clone-test-repos && npm run update-snapshots-test-repos",
-    "upgrade": "npx --yes npm-check-updates --upgrade"
+    "test": "node --test --experimental-test-coverage --test-coverage-branches=100 --test-coverage-functions=100 --test-coverage-lines=100 --test-coverage-exclude=test/**/* test/markdownlint-test.mjs test/markdownlint-test-config.mjs test/markdownlint-test-custom-rules.mjs test/markdownlint-test-exports.mjs test/markdownlint-test-fixes.mjs test/markdownlint-test-helpers.mjs test/markdownlint-test-micromark.mjs test/markdownlint-test-project.mjs test/markdownlint-test-result-object.mjs test/markdownlint-test-scenarios.mjs test/parse-configuration-test.mjs test/resolve-module-test.mjs helpers/test.cjs",
+    "test-declaration": "concurrently npm:test-declaration-cts npm:test-declaration-mts",
+    "test-declaration-cts": "cd example/typescript && node ../../scripts/index.mjs copy type-check.ts type-check-commonjs.cts && tsc --project tsconfig.cts.json",
+    "test-declaration-mts": "cd example/typescript && node ../../scripts/index.mjs copy type-check.ts type-check-nodenext.mts && tsc --project tsconfig.mts.json && node type-check-nodenext.mjs",
+    "test-extra": "node --test test/markdownlint-test-extra-parse.mjs test/markdownlint-test-extra-type.mjs",
+    "update-snapshots": "node --test --test-update-snapshots test/markdownlint-test-custom-rules.mjs test/markdownlint-test-exports.mjs test/markdownlint-test-micromark.mjs test/markdownlint-test-result-object.mjs test/markdownlint-test-scenarios.mjs",
+    "update-snapshots-test-repos": "node --test --test-update-snapshots test/markdownlint-test-repos-*.mjs",
+    "update-test-repos": "node scripts/index.mjs remove ./test-repos && npm run clone-test-repos && npm run update-snapshots-test-repos"
   },
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
   "dependencies": {
     "micromark": "4.0.2",
@@ -81,37 +78,37 @@
     "micromark-extension-gfm-table": "2.1.1",
     "micromark-extension-math": "3.1.0",
     "micromark-util-types": "2.0.2",
-    "string-width": "8.1.0"
+    "string-width": "8.2.1"
   },
   "devDependencies": {
-    "@eslint/js": "9.39.1",
-    "@stylistic/eslint-plugin": "5.6.1",
-    "ajv": "8.17.1",
-    "ava": "6.4.1",
-    "c8": "10.1.3",
+    "@eslint/js": "10.0.1",
+    "@stylistic/eslint-plugin": "5.10.0",
+    "@types/markdown-it": "14.1.2",
+    "ajv": "8.20.0",
     "character-entities": "2.0.2",
-    "eslint": "9.39.1",
-    "eslint-plugin-jsdoc": "61.4.1",
-    "eslint-plugin-n": "17.23.1",
-    "eslint-plugin-regexp": "2.10.0",
-    "eslint-plugin-unicorn": "62.0.0",
+    "concurrently": "10.0.3",
+    "eslint": "10.4.1",
+    "eslint-plugin-jsdoc": "63.0.2",
+    "eslint-plugin-n": "18.1.0",
+    "eslint-plugin-regexp": "3.1.0",
+    "eslint-plugin-unicorn": "65.0.1",
     "gemoji": "8.1.0",
-    "globby": "16.0.0",
-    "js-yaml": "4.1.1",
+    "globby": "16.2.0",
+    "js-yaml": "4.2.0",
     "json-schema-to-typescript": "15.0.4",
+    "json-stringify-safe": "5.0.1",
     "jsonc-parser": "3.3.1",
-    "markdown-it": "14.1.0",
+    "markdown-it": "14.2.0",
     "markdown-it-for-inline": "2.0.1",
     "markdown-it-sub": "2.0.0",
     "markdown-it-sup": "2.0.0",
     "markdownlint-rule-extended-ascii": "0.2.1",
-    "nano-spawn": "2.0.0",
-    "npm-run-all": "4.1.5",
-    "terser-webpack-plugin": "5.3.14",
-    "toml": "3.0.0",
-    "typescript": "5.9.3",
-    "webpack": "5.103.0",
-    "webpack-cli": "6.0.1"
+    "nano-spawn": "2.1.0",
+    "terser-webpack-plugin": "5.6.1",
+    "toml": "4.1.1",
+    "typescript": "6.0.3",
+    "webpack": "5.107.2",
+    "webpack-cli": "7.0.3"
   },
   "keywords": [
     "markdown",
@@ -120,4 +117,4 @@
     "CommonMark",
     "markdownlint"
   ]
-}
+}