eslint-plugin-markdown-preferences 0.5.0 → 0.6.0

package/README.md

@@ -94,6 +94,7 @@ The rules with the following star ⭐ are included in the configs.
 | [markdown-preferences/no-text-backslash-linebreak](https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/no-text-backslash-linebreak.html) | disallow text backslash at the end of a line. |  | ⭐ |
 | [markdown-preferences/no-trailing-spaces](https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/no-trailing-spaces.html) | disallow trailing whitespace at the end of lines in Markdown files. | 🔧 |  |
 | [markdown-preferences/prefer-inline-code-words](https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/prefer-inline-code-words.html) | enforce the use of inline code for specific words. | 🔧 |  |
+| [markdown-preferences/prefer-link-reference-definitions](https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/prefer-link-reference-definitions.html) | enforce using link reference definitions instead of inline links | 🔧 |  |
 | [markdown-preferences/prefer-linked-words](https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/prefer-linked-words.html) | enforce the specified word to be a link. | 🔧 |  |
 
 <!--RULES_TABLE_END-->

package/lib/index.d.ts

@@ -30,6 +30,11 @@ interface RuleOptions {
    * @see https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/prefer-inline-code-words.html
    */
   'markdown-preferences/prefer-inline-code-words'?: Linter.RuleEntry<MarkdownPreferencesPreferInlineCodeWords>;
+  /**
+   * enforce using link reference definitions instead of inline links
+   * @see https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/prefer-link-reference-definitions.html
+   */
+  'markdown-preferences/prefer-link-reference-definitions'?: Linter.RuleEntry<MarkdownPreferencesPreferLinkReferenceDefinitions>;
   /**
    * enforce the specified word to be a link.
    * @see https://ota-meshi.github.io/eslint-plugin-markdown-preferences/rules/prefer-linked-words.html
@@ -54,6 +59,9 @@ type MarkdownPreferencesPreferInlineCodeWords = [] | [{
   }[];
   [k: string]: unknown | undefined;
 }];
+type MarkdownPreferencesPreferLinkReferenceDefinitions = [] | [{
+  minLinks?: number;
+}];
 type MarkdownPreferencesPreferLinkedWords = [] | [{
   words: ({
     [k: string]: (string | null);
@@ -82,7 +90,7 @@ declare namespace meta_d_exports {
   export { name, version };
 }
 declare const name: "eslint-plugin-markdown-preferences";
-declare const version: "0.5.0";
+declare const version: "0.6.0";
 //#endregion
 //#region src/index.d.ts
 declare const configs: {

package/lib/index.js

@@ -245,7 +245,7 @@ var no_trailing_spaces_default = createRule("no-trailing-spaces", {
 
 //#endregion
 //#region src/utils/search-words.ts
-const RE_BOUNDARY = /^[\s\p{Letter_Number}\p{Modifier_Letter}\p{Modifier_Symbol}\p{Nonspacing_Mark}\p{Other_Letter}\p{Other_Symbol}\p{Script=Han}!"#$%&'(),./:;<=>?\\{|}~\u{2ffc}-\u{303d}\u{30a0}-\u{30fb}\u{3192}-\u{32bf}\u{fe10}-\u{fe1f}\u{fe30}-\u{fe6f}\u{ff00}-\u{ffef}\u{2ebf0}-\u{2ee5d}]*$/u;
+const RE_BOUNDARY = /^[\s\p{Letter_Number}\p{Modifier_Letter}\p{Modifier_Symbol}\p{Nonspacing_Mark}\p{Other_Letter}\p{Other_Symbol}\p{Script=Han}!"#$%&'()*+,./:;<=>?\\{|}~\u{2ffc}-\u{303d}\u{30a0}-\u{30fb}\u{3192}-\u{32bf}\u{fe10}-\u{fe1f}\u{fe30}-\u{fe6f}\u{ff00}-\u{ffef}\u{2ebf0}-\u{2ee5d}]*$/u;
 /**
 * Iterate through words in a text node that match the specified words.
 */
@@ -403,6 +403,140 @@ var prefer_inline_code_words_default = createRule("prefer-inline-code-words", {
 	}
 });
 
+//#endregion
+//#region src/rules/prefer-link-reference-definitions.ts
+var prefer_link_reference_definitions_default = createRule("prefer-link-reference-definitions", {
+	meta: {
+		type: "layout",
+		docs: {
+			description: "enforce using link reference definitions instead of inline links",
+			categories: []
+		},
+		fixable: "code",
+		hasSuggestions: false,
+		schema: [{
+			type: "object",
+			properties: { minLinks: {
+				type: "number",
+				description: "minimum number of links to trigger the rule (default: 2)",
+				default: 2,
+				minimum: 1
+			} },
+			additionalProperties: false
+		}],
+		messages: { useLinkReferenceDefinitions: "Use link reference definitions instead of inline links." }
+	},
+	create(context) {
+		const sourceCode = context.sourceCode;
+		const options = context.options[0] || {};
+		const minLinks = options.minLinks ?? 2;
+		const definitions = [];
+		const links = [];
+		const linkReferences = [];
+		/**
+		* Verify links.
+		*/
+		function verify() {
+			const resourceToNodes = /* @__PURE__ */ new Map();
+			for (const link of links) getResourceNodes(link).links.push(link);
+			for (const linkReference of linkReferences) {
+				const definition = definitions.find((def) => def.identifier === linkReference.identifier);
+				if (definition) getResourceNodes(definition).linkReferences.push(linkReference);
+			}
+			for (const definition of definitions) getResourceNodes(definition).definitions.push(definition);
+			for (const map of resourceToNodes.values()) for (const nodes of map.values()) {
+				if (nodes.links.length === 0 || nodes.links.length + nodes.linkReferences.length < minLinks) continue;
+				for (const link of nodes.links) {
+					const linkInfo = getLinkInfo(link);
+					if (linkInfo.label === "") continue;
+					context.report({
+						node: link,
+						messageId: "useLinkReferenceDefinitions",
+						*fix(fixer) {
+							const definition = nodes.definitions[0];
+							let identifier;
+							if (definition) identifier = definition.label ?? definition.identifier;
+							else identifier = linkInfo.label.replaceAll(/\]/g, "-");
+							yield fixer.replaceText(link, `${sourceCode.text.slice(...linkInfo.labelRange)}${identifier === linkInfo.label ? "" : `[${identifier}]`}`);
+							if (!definition) {
+								const linkRange = sourceCode.getRange(link);
+								const reLineBreaks = /\n#{1,6}\s/gu;
+								reLineBreaks.lastIndex = linkRange[1];
+								const nextSectionMatch = reLineBreaks.exec(sourceCode.text);
+								const insertIndex = !nextSectionMatch ? sourceCode.text.trimEnd().length : nextSectionMatch.index;
+								yield fixer.insertTextAfterRange([insertIndex, insertIndex], `${sourceCode.text[insertIndex - 1] === "\n" ? "" : "\n"}\n[${identifier}]: ${sourceCode.text.slice(linkInfo.urlAndTitleRange[0] + 1, linkInfo.urlAndTitleRange[1] - 1).trim()}${nextSectionMatch ? "\n" : ""}`);
+							}
+						}
+					});
+				}
+			}
+			/**
+			* Get the resource nodes for a link or definition.
+			*/
+			function getResourceNodes(resource) {
+				const url = resource.url;
+				const title = resource.title ?? null;
+				let map = resourceToNodes.get(url);
+				if (!map) {
+					map = /* @__PURE__ */ new Map();
+					resourceToNodes.set(url, map);
+				}
+				let nodes = map.get(title);
+				if (!nodes) {
+					nodes = {
+						links: [],
+						linkReferences: [],
+						definitions: []
+					};
+					map.set(title, nodes);
+				}
+				return nodes;
+			}
+		}
+		return {
+			link(node) {
+				links.push(node);
+			},
+			linkReference(node) {
+				linkReferences.push(node);
+			},
+			definition(node) {
+				definitions.push(node);
+			},
+			"root:exit"() {
+				verify();
+			}
+		};
+		/**
+		* Get the range of the link label.
+		*/
+		function getLinkInfo(link) {
+			const range = sourceCode.getRange(link);
+			const linkLabelRange = getLinkLabelRange();
+			const linkLabelWithBracketsText = sourceCode.text.slice(...linkLabelRange);
+			const linkLabelText = linkLabelWithBracketsText.slice(1, -1).trim();
+			const urlStartIndex = sourceCode.text.indexOf("(", linkLabelRange[1]);
+			return {
+				label: linkLabelText,
+				labelRange: linkLabelRange,
+				urlAndTitleRange: [urlStartIndex, range[1]]
+			};
+			/**
+			* Get the range of the link label.
+			*/
+			function getLinkLabelRange() {
+				if (link.children.length === 0) {
+					const index$1 = sourceCode.text.indexOf("]", range[0] + 1);
+					return [range[0], index$1 + 1];
+				}
+				const lastRange = sourceCode.getRange(link.children[link.children.length - 1]);
+				const index = sourceCode.text.indexOf("]", lastRange[0] + 1);
+				return [range[0], index + 1];
+			}
+		}
+	}
+});
+
 //#endregion
 //#region src/rules/prefer-linked-words.ts
 var prefer_linked_words_default = createRule("prefer-linked-words", {
@@ -518,6 +652,7 @@ const rules$1 = [
 	no_text_backslash_linebreak_default,
 	no_trailing_spaces_default,
 	prefer_inline_code_words_default,
+	prefer_link_reference_definitions_default,
 	prefer_linked_words_default
 ];
 
@@ -553,7 +688,7 @@ __export(meta_exports, {
 	version: () => version
 });
 const name = "eslint-plugin-markdown-preferences";
-const version = "0.5.0";
+const version = "0.6.0";
 
 //#endregion
 //#region src/index.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-markdown-preferences",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "ESLint plugin that enforces our markdown preferences",
   "type": "module",
   "exports": {
@@ -71,6 +71,7 @@
     "@types/eslint-utils": "^3.0.5",
     "@types/estree": "^1.0.6",
     "@types/json-schema": "^7.0.15",
+    "@types/mdast": "^4.0.4",
     "@types/mocha": "^10.0.10",
     "@types/node": "^22.13.10",
     "@types/semver": "^7.5.8",