markdownlint 0.41.0 → 0.41.1

package/lib/constants.mjs

@@ -11,4 +11,4 @@ export const fixableRuleNames = [
   "MD054", "MD058", "MD060"
 ];
 export const homepage = "https://github.com/DavidAnson/markdownlint";
-export const version = "0.41.0";
+export const version = "0.41.1";

package/lib/markdownlint.mjs

@@ -93,6 +93,7 @@ function validateRuleList(ruleList, synchronous) {
       }
       for (const tag of rule.tags) {
         const tagUpper = tag.toUpperCase();
+        // eslint-disable-next-line unicorn/no-computed-property-existence-check
         if (!result && allIds[tagUpper]) {
           result = new Error("Tag '" + tag + "' of custom rule at index " +
             customIndex + " is already used as a name.");
@@ -439,6 +440,7 @@ function lintContent(
   // eslint-disable-next-line jsdoc/reject-any-type
   const callbackError = (/** @type {any} */ error) => callback(error instanceof Error ? error : new Error(error));
   // Remove UTF-8 byte order marker (if present)
+  // eslint-disable-next-line unicorn/prefer-unicode-code-point-escapes
   content = content.replace(/^\uFEFF/, "");
   // Remove front matter
   const removeFrontMatterResult = removeFrontMatter(content, frontMatter);
@@ -771,8 +773,8 @@ function lintFile(
  */
 function lintInput(options, synchronous, callback) {
   // Normalize inputs
-  options = options || {};
-  callback = callback || function noop() {};
+  options ||= {};
+  callback ||= () => {};
   /** @type {Rule[]} */
   // @ts-ignore
   const customRuleList =
@@ -1209,6 +1211,7 @@ export function applyFix(line, fixInfo, lineEnding = "\n") {
   const editIndex = editColumn - 1;
   return (deleteCount === -1) ?
     null :
+    // eslint-disable-next-line unicorn/no-unsafe-string-replacement
     line.slice(0, editIndex) + insertText.replace(/\n/g, lineEnding) + line.slice(editIndex + deleteCount);
 }
 

package/lib/md004.mjs

@@ -40,6 +40,7 @@ export default {
       for (const listItemMarker of listItemMarkers) {
         const itemStyle = markerToStyle(listItemMarker.text);
         if (style === "sublist") {
+          // eslint-disable-next-line unicorn/no-computed-property-existence-check
           if (!nestingStyles[nesting]) {
             nestingStyles[nesting] =
               (itemStyle === nestingStyles[nesting - 1]) ?

package/lib/md005.mjs

@@ -34,7 +34,7 @@ export default {
         } else {
           const markerLength = listItemPrefix.text.trim().length;
           const actualEnd = listItemPrefix.startColumn + markerLength - 1;
-          expectedEnd = expectedEnd || actualEnd;
+          expectedEnd ||= actualEnd;
           if ((expectedIndent !== actualIndent) || endMatching) {
             if (expectedEnd === actualEnd) {
               endMatching = true;

package/lib/md019-md021.mjs

@@ -16,6 +16,7 @@ function validateHeadingSpaces(onError, heading, delta) {
   const { children, startLine, text } = heading;
   let index = (delta > 0) ? 0 : (children.length - 1);
   while (
+    // eslint-disable-next-line unicorn/no-computed-property-existence-check
     children[index] &&
     (children[index].type !== "atxHeadingSequence")
   ) {

package/lib/md022.mjs

@@ -11,6 +11,7 @@ const defaultLines = 1;
 // eslint-disable-next-line jsdoc/reject-any-type
 const getLinesFunction = (/** @type {any} */ linesParam) => {
   if (Array.isArray(linesParam)) {
+    // eslint-disable-next-line unicorn/no-unreadable-new-expression
     const linesArray = new Array(6).fill(defaultLines);
     for (const [ index, value ] of [ ...linesParam.entries() ].slice(0, 6)) {
       linesArray[index] = value;

package/lib/md024.mjs

@@ -11,7 +11,7 @@ export default {
   "tags": [ "headings" ],
   "parser": "micromark",
   "function": function MD024(params, onError) {
-    const siblingsOnly = !!params.config.siblings_only || false;
+    const siblingsOnly = !!params.config.siblings_only;
     const knownContents = [ null, [] ];
     let lastLevel = 1;
     let knownContent = knownContents[lastLevel];

package/lib/md029.mjs

@@ -15,13 +15,16 @@ const listStyles = Object.keys(listStyleExamples);
  * Gets the column and text of an ordered list item prefix token.
  *
  * @param {import("markdownlint").MicromarkToken} listItemPrefix List item prefix token.
- * @returns {{column: number, value: number}} List item value column and text.
+ * @returns {{startColumn: number, endColumn: number, text: string, number: number}} List item value column and text.
  */
 function getOrderedListItemValue(listItemPrefix) {
   const listItemValue = getDescendantsByType(listItemPrefix, [ "listItemValue" ])[0];
+  const { startColumn, endColumn, text } = listItemValue;
   return {
-    "column": listItemValue.startColumn,
-    "value": Number(listItemValue.text)
+    startColumn,
+    endColumn,
+    text,
+    "number": Number(text)
   };
 }
 
@@ -35,40 +38,49 @@ export default {
     const style = String(params.config.style);
     for (const listOrdered of filterByTypesCached([ "listOrdered" ])) {
       const listItemPrefixes = getDescendantsByType(listOrdered, [ "listItemPrefix" ]);
-      let expected = 1;
+      let expectedNumber = 1;
       let incrementing = false;
-      // Check for incrementing number pattern 1/2/3 or 0/1/2
-      if (listItemPrefixes.length >= 2) {
-        const first = getOrderedListItemValue(listItemPrefixes[0]);
-        const second = getOrderedListItemValue(listItemPrefixes[1]);
-        if ((second.value !== 1) || (first.value === 0)) {
-          incrementing = true;
-          if (first.value === 0) {
-            expected = 0;
+      let rightAligned = false;
+      let zeroPrefixed = false;
+      if (listItemPrefixes.length > 0) {
+        // Check for right alignment
+        const firstPrefixEndColumn = listItemPrefixes[0].endColumn;
+        rightAligned = listItemPrefixes.every((prefix) => prefix.endColumn === firstPrefixEndColumn);
+        if (listItemPrefixes.length > 1) {
+          // Check for incrementing number pattern 1/2/3 or 0/1/2
+          const firstValue = getOrderedListItemValue(listItemPrefixes[0]);
+          const secondValue = getOrderedListItemValue(listItemPrefixes[1]);
+          if ((secondValue.number !== 1) || (firstValue.number === 0)) {
+            incrementing = true;
+            if (firstValue.number === 0) {
+              expectedNumber = 0;
+            }
           }
+          zeroPrefixed = firstValue.text.startsWith("0");
         }
       }
       // Determine effective style
       const listStyle = listStyles.includes(style) ? style : (incrementing ? "ordered" : "one");
       if (listStyle === "zero") {
-        expected = 0;
+        expectedNumber = 0;
       } else if (listStyle === "one") {
-        expected = 1;
+        expectedNumber = 1;
       }
       // Validate each list item marker
       for (const listItemPrefix of listItemPrefixes) {
-        const orderedListItemValue = getOrderedListItemValue(listItemPrefix);
-        const actual = orderedListItemValue.value;
+        const expectedText = expectedNumber.toString();
+        const { endColumn, startColumn, "number": actualNumber } = getOrderedListItemValue(listItemPrefix);
+        const length = endColumn - startColumn;
         const fixInfo = {
-          "editColumn": orderedListItemValue.column,
-          "deleteCount": orderedListItemValue.value.toString().length,
-          "insertText": expected.toString()
+          "editColumn": startColumn,
+          "deleteCount": length,
+          "insertText": rightAligned ? expectedText.padStart(length, (zeroPrefixed ? "0" : " ")) : expectedText
         };
         addErrorDetailIf(
           onError,
           listItemPrefix.startLine,
-          expected,
-          actual,
+          expectedNumber,
+          actualNumber,
           // @ts-ignore
           "Style: " + listStyleExamples[listStyle],
           undefined,
@@ -76,7 +88,7 @@ export default {
           fixInfo
         );
         if (listStyle === "ordered") {
-          expected++;
+          expectedNumber++;
         }
       }
     }

package/lib/md032.mjs

@@ -52,10 +52,11 @@ export default {
         (token) => !nonContentTokens.has(token.type),
         (token) => nonContentTokens.has(token.type) ? [] : token.children
       );
-      let endLine = list.endLine;
-      if (flattenedChildren.length > 0) {
-        endLine = flattenedChildren[flattenedChildren.length - 1].endLine;
-      }
+      const endLine = (
+        (flattenedChildren.length > 0) ?
+          flattenedChildren[flattenedChildren.length - 1] :
+          list
+      ).endLine;
 
       // Look for a blank line below the list
       const lastLineNumber = endLine;

package/lib/md037.mjs

@@ -38,8 +38,7 @@ export default {
       }
 
       // Process bare tokens for each emphasis marker type
-      for (const entry of emphasisTokensByMarker.entries()) {
-        const [ marker, emphasisTokens ] = entry;
+      for (const [ marker, emphasisTokens ] of emphasisTokensByMarker) {
         for (let i = 0; i + 1 < emphasisTokens.length; i += 2) {
 
           // Process start token of start/end pair

package/lib/md051.mjs

@@ -141,6 +141,7 @@ export default {
               };
             }
             const textLower = text.toLowerCase();
+            // eslint-disable-next-line unicorn/prefer-iterator-helpers
             const mixedCaseKey = [ ...fragments.keys() ]
               .find((key) => textLower === key.toLowerCase());
             if (mixedCaseKey) {

package/lib/micromark-parse.mjs

@@ -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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "markdownlint",
-  "version": "0.41.0",
+  "version": "0.41.1",
   "description": "A Node.js style checker and lint tool for Markdown/CommonMark files.",
   "type": "module",
   "exports": {
@@ -39,7 +39,7 @@
     "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": "concurrently npm:build-demo npm:lint npm:serial-config-docs npm:serial-declaration npm:test && git diff --exit-code && echo PASS",
+    "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-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",
@@ -62,6 +62,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"
@@ -87,28 +88,29 @@
     "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",
+    "eslint": "10.6.0",
+    "eslint-node-test": "0.1.0",
+    "eslint-plugin-jsdoc": "63.0.10",
+    "eslint-plugin-n": "18.2.1",
+    "eslint-plugin-regexp": "3.1.1",
+    "eslint-plugin-unicorn": "70.0.0",
     "gemoji": "8.1.0",
     "globby": "16.2.0",
-    "js-yaml": "4.2.0",
+    "js-yaml": "5.2.1",
     "json-schema-to-typescript": "15.0.4",
     "json-stringify-safe": "5.0.1",
     "jsonc-parser": "3.3.1",
-    "markdown-it": "14.2.0",
+    "markdown-it": "14.3.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.1.0",
     "terser-webpack-plugin": "5.6.1",
-    "toml": "4.1.1",
+    "toml": "4.1.2",
     "typescript": "6.0.3",
-    "webpack": "5.107.2",
-    "webpack-cli": "7.0.3"
+    "webpack": "5.108.3",
+    "webpack-cli": "7.1.0"
   },
   "keywords": [
     "markdown",