markdownlint 0.41.0 → 0.42.0

package/lib/md060.mjs

@@ -4,21 +4,35 @@ import { filterByTypes } from "../helpers/micromark-helpers.cjs";
 import { filterByTypesCached } from "./cache.mjs";
 import stringWidth from "string-width";
 
-/** @typedef {import("micromark-extension-gfm-table")} */
+/** @typedef {typeof import("micromark-extension-gfm-table")} _ambient_ */
 /** @typedef {import("markdownlint").MicromarkToken} MicromarkToken */
 /** @typedef {import("markdownlint").RuleOnErrorInfo} RuleOnErrorInfo */
 /** @typedef {import("markdownlint").RuleOnErrorFixInfo} RuleOnErrorFixInfo */
 
+// Regular expression to match text that could represent an unordered list item prefix
+const unorderedListPrefix = /^[>\s]*[-+*]$/;
+
 /**
  * Adds a RuleOnErrorInfo object to a list of RuleOnErrorInfo objects.
  *
  * @param {RuleOnErrorInfo[]} errors List of errors.
+ * @param {readonly string[]} lines File/string lines.
  * @param {number} lineNumber Line number.
  * @param {number} column Column number.
  * @param {string} detail Detail message.
  * @param {RuleOnErrorFixInfo} [fixInfo] Fix information.
+ * @returns {void}
  */
-function addError(errors, lineNumber, column, detail, fixInfo) {
+function addError(errors, lines, lineNumber, column, detail, fixInfo) {
+  if (
+    fixInfo &&
+    (fixInfo.insertText === " ") &&
+    fixInfo.editColumn &&
+    unorderedListPrefix.test(lines[lineNumber - 1].slice(0, fixInfo.editColumn - 1))
+  ) {
+    // Do not apply edits that would change a table cell into an unordered list item
+    fixInfo = undefined;
+  }
   errors.push({
     lineNumber,
     detail,
@@ -70,7 +84,7 @@ function checkStyleAligned(lines, rows, detail) {
     const rowDividerColumns = getTableDividerColumns(lines, row);
     for (const dividerColumn of rowDividerColumns) {
       if ((remainingHeaderDividerColumns.size > 0) && !remainingHeaderDividerColumns.delete(dividerColumn.effective)) {
-        addError(errorInfos, row.startLine, dividerColumn.actual, detail);
+        addError(errorInfos, lines, row.startLine, dividerColumn.actual, detail);
       }
     }
   }
@@ -128,6 +142,7 @@ export default {
                   if (previous.text.length !== 1) {
                     addError(
                       errorsIfCompact,
+                      lines,
                       startLine,
                       startColumn,
                       "Table pipe has extra space to the left for style \"compact\"",
@@ -136,6 +151,7 @@ export default {
                   }
                   addError(
                     errorsIfTight,
+                    lines,
                     startLine,
                     startColumn,
                     "Table pipe has space to the left for style \"tight\"",
@@ -144,6 +160,7 @@ export default {
                 } else {
                   addError(
                     errorsIfCompact,
+                    lines,
                     startLine,
                     startColumn,
                     "Table pipe is missing space to the left for style \"compact\"",
@@ -158,6 +175,7 @@ export default {
                     if (next.text.length !== 1) {
                       addError(
                         errorsIfCompact,
+                        lines,
                         startLine,
                         startColumn,
                         "Table pipe has extra space to the right for style \"compact\"",
@@ -166,6 +184,7 @@ export default {
                     }
                     addError(
                       errorsIfTight,
+                      lines,
                       startLine,
                       startColumn,
                       "Table pipe has space to the right for style \"tight\"",
@@ -175,6 +194,7 @@ export default {
                 } else {
                   addError(
                     errorsIfCompact,
+                    lines,
                     startLine,
                     startColumn,
                     "Table pipe is missing space to the right for style \"compact\"",
@@ -191,7 +211,7 @@ export default {
       let errorInfos = errorsIfAligned;
       if (
         styleCompactAllowed &&
-        ((errorsIfCompact.length < errorInfos.length) || !styleAlignedAllowed)
+        (!styleAlignedAllowed || (errorsIfCompact.length < errorInfos.length))
       ) {
         errorInfos = errorsIfCompact;
       }

package/lib/micromark-parse.mjs

@@ -19,7 +19,7 @@ import { flatTokensSymbol, htmlFlowSymbol, newlineRe } from "../helpers/shared.c
 /** @typedef {import("micromark-util-types").Token} Token */
 /** @typedef {import("micromark-util-types").Tokenizer} Tokenizer */
 /** @typedef {import("markdownlint").MicromarkToken} MicromarkToken */
-/** @typedef {import("./micromark-types.d.mts")} */
+/** @typedef {typeof import("./micromark-types.d.mts")} _ambient_ */
 
 /**
  * Gets a modified micromark-extension-directive without support for inline directives.
@@ -297,9 +297,10 @@ function parseInternal(
           current
         );
         current.children = tokens;
+        // eslint-disable-next-line unicorn/comment-content
         // Avoid stack overflow of Array.push(...spread)
         // @ts-ignore
-        // eslint-disable-next-line unicorn/prefer-spread
+        // eslint-disable-next-line unicorn/no-array-concat-in-loop
         flatTokens = flatTokens.concat(tokens[flatTokensSymbol]);
       }
     } else if (kind === "exit") {

package/lib/resolve-module.cjs

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

package/lib/resolve-module.d.cts

@@ -1,18 +1,15 @@
+declare const _exports: {
+    resolveModule: typeof resolveModule;
+    resolveModuleCustomResolve: typeof resolveModuleCustomResolve;
+};
+export = _exports;
 export type RequireResolveOptions = {
     /**
      * Additional paths to resolve from.
      */
-    paths?: string[] | undefined;
+    paths?: string[];
 };
 export type RequireResolve = (id: string, options: RequireResolveOptions) => string;
-/**
- * Resolves modules according to Node's resolution rules.
- *
- * @param {string} id Module name or path.
- * @param {string[]} [paths] Additional paths to resolve from.
- * @returns {string} Resolved module path.
- */
-export function resolveModule(id: string, paths?: string[]): string;
 /**
  * @typedef RequireResolveOptions
  * @property {string[]} [paths] Additional paths to resolve from.
@@ -31,4 +28,12 @@ export function resolveModule(id: string, paths?: string[]): string;
  * @param {string[]} [paths] Additional paths to resolve from.
  * @returns {string} Resolved module path.
  */
-export function resolveModuleCustomResolve(resolve: RequireResolve, id: string, paths?: string[]): string;
+declare const resolveModuleCustomResolve: (resolve: RequireResolve, id: string, paths?: string[]) => string;
+/**
+ * Resolves modules according to Node's resolution rules.
+ *
+ * @param {string} id Module name or path.
+ * @param {string[]} [paths] Additional paths to resolve from.
+ * @returns {string} Resolved module path.
+ */
+declare const resolveModule: (id: string, paths?: string[]) => string;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "markdownlint",
-  "version": "0.41.0",
-  "description": "A Node.js style checker and lint tool for Markdown/CommonMark files.",
+  "version": "0.42.0",
+  "description": "A Node.js style checker and lint tool for Markdown/CommonMark files",
   "type": "module",
   "exports": {
     ".": "./lib/exports.mjs",
@@ -26,21 +26,16 @@
   "author": "David Anson (https://dlaa.me/)",
   "license": "MIT",
   "homepage": "https://github.com/DavidAnson/markdownlint",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/DavidAnson/markdownlint.git"
-  },
-  "bugs": "https://github.com/DavidAnson/markdownlint/issues",
+  "repository": "github:DavidAnson/markdownlint",
   "funding": "https://github.com/sponsors/DavidAnson",
   "scripts": {
     "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 --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-demo": "node scripts/index.mjs copy node_modules/markdown-it/dist/browser/markdown-it.umd.min.js demo/markdown-it.umd.min.js && cd demo && webpack --no-stats",
     "build-docs": "node doc-build/build-rules.mjs",
-    "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",
+    "ci": "concurrently npm:build-demo npm:lint npm:serial-config-docs npm:serial-declaration npm:test npm:test-webpack && git diff --exit-code && echo PASS",
     "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",
     "clone-test-repos-eslint-eslint": "cd test-repos && git clone https://github.com/eslint/eslint eslint-eslint --depth 1 --no-tags --quiet",
@@ -51,9 +46,9 @@
     "clone-test-repos-v8-v8-dev": "cd test-repos && git clone https://github.com/v8/v8.dev v8-v8-dev --depth 1 --no-tags --quiet",
     "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",
+    "clone-test-repos": "mkdir test-repos && cd test-repos && 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",
     "example": "cd example && node standalone.mjs",
-    "lint": "eslint --max-warnings 0",
+    "lint": "cross-env RECHECK_BACKEND=pure eslint --max-warnings 0",
     "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",
@@ -62,6 +57,7 @@
     "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",
+    "test-webpack": "cd test && node resolve-module-webpack.mjs && webpack --entry ./resolve-module-webpack.mjs --output-path . --output-filename resolve-module-webpack.cjs --stats errors-only && node resolve-module-webpack.cjs",
     "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"
@@ -70,51 +66,54 @@
     "node": ">=22"
   },
   "dependencies": {
-    "micromark": "4.0.2",
-    "micromark-core-commonmark": "2.0.3",
+    "micromark-core-commonmark": "2.0.4",
     "micromark-extension-directive": "4.0.0",
     "micromark-extension-gfm-autolink-literal": "2.1.0",
     "micromark-extension-gfm-footnote": "2.1.0",
-    "micromark-extension-gfm-table": "2.1.1",
+    "micromark-extension-gfm-table": "2.1.2",
     "micromark-extension-math": "3.1.0",
-    "micromark-util-types": "2.0.2",
-    "string-width": "8.2.1"
+    "micromark-util-types": "2.0.3",
+    "micromark": "4.0.3",
+    "re2js": "2.8.6",
+    "string-width": "8.3.0"
   },
   "devDependencies": {
     "@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",
-    "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",
+    "concurrently": "10.0.5",
+    "cross-env": "10.1.0",
+    "eslint-node-test": "1.0.1",
+    "eslint-package-json": "2.0.0",
+    "eslint-plugin-jsdoc": "65.1.0",
+    "eslint-plugin-n": "18.4.1",
+    "eslint-plugin-redos": "4.5.0",
+    "eslint-plugin-regexp": "3.3.1",
+    "eslint-plugin-unicorn": "77.0.0",
+    "eslint": "10.12.0",
     "gemoji": "8.1.0",
-    "globby": "16.2.0",
-    "js-yaml": "4.2.0",
-    "json-schema-to-typescript": "15.0.4",
+    "globby": "16.2.4",
+    "js-yaml": "5.4.2",
+    "json-schema-to-typescript": "16.0.0",
     "json-stringify-safe": "5.0.1",
     "jsonc-parser": "3.3.1",
-    "markdown-it": "14.2.0",
     "markdown-it-for-inline": "2.0.1",
     "markdown-it-sub": "2.0.0",
     "markdown-it-sup": "2.0.0",
+    "markdown-it": "15.0.2",
     "markdownlint-rule-extended-ascii": "0.2.1",
     "nano-spawn": "2.1.0",
+    "smol-toml": "1.9.0",
     "terser-webpack-plugin": "5.6.1",
-    "toml": "4.1.1",
-    "typescript": "6.0.3",
-    "webpack": "5.107.2",
-    "webpack-cli": "7.0.3"
+    "typescript": "7.0.2",
+    "webpack-cli": "7.2.3",
+    "webpack": "5.111.1"
   },
   "keywords": [
     "markdown",
     "lint",
     "md",
-    "CommonMark",
-    "markdownlint"
+    "commonmark"
   ]
 }