| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447 |
- /**
- * @fileoverview Source code for spaced-comments rule
- * @author Gyandeep Singh
- * @deprecated in ESLint v8.53.0
- */
- "use strict";
- const escapeRegExp = require("escape-string-regexp");
- const astUtils = require("./utils/ast-utils");
- //------------------------------------------------------------------------------
- // Helpers
- //------------------------------------------------------------------------------
- /**
- * Escapes the control characters of a given string.
- * @param {string} s A string to escape.
- * @returns {string} An escaped string.
- */
- function escape(s) {
- return `(?:${escapeRegExp(s)})`;
- }
- /**
- * Escapes the control characters of a given string.
- * And adds a repeat flag.
- * @param {string} s A string to escape.
- * @returns {string} An escaped string.
- */
- function escapeAndRepeat(s) {
- return `${escape(s)}+`;
- }
- /**
- * Parses `markers` option.
- * If markers don't include `"*"`, this adds `"*"` to allow JSDoc comments.
- * @param {string[]} [markers] A marker list.
- * @returns {string[]} A marker list.
- */
- function parseMarkersOption(markers) {
- // `*` is a marker for JSDoc comments.
- if (!markers.includes("*")) {
- return markers.concat("*");
- }
- return markers;
- }
- /**
- * Creates string pattern for exceptions.
- * Generated pattern:
- *
- * 1. A space or an exception pattern sequence.
- * @param {string[]} exceptions An exception pattern list.
- * @returns {string} A regular expression string for exceptions.
- */
- function createExceptionsPattern(exceptions) {
- let pattern = "";
- /*
- * A space or an exception pattern sequence.
- * [] ==> "\s"
- * ["-"] ==> "(?:\s|\-+$)"
- * ["-", "="] ==> "(?:\s|(?:\-+|=+)$)"
- * ["-", "=", "--=="] ==> "(?:\s|(?:\-+|=+|(?:\-\-==)+)$)" ==> https://jex.im/regulex/#!embed=false&flags=&re=(%3F%3A%5Cs%7C(%3F%3A%5C-%2B%7C%3D%2B%7C(%3F%3A%5C-%5C-%3D%3D)%2B)%24)
- */
- if (exceptions.length === 0) {
- // a space.
- pattern += "\\s";
- } else {
- // a space or...
- pattern += "(?:\\s|";
- if (exceptions.length === 1) {
- // a sequence of the exception pattern.
- pattern += escapeAndRepeat(exceptions[0]);
- } else {
- // a sequence of one of the exception patterns.
- pattern += "(?:";
- pattern += exceptions.map(escapeAndRepeat).join("|");
- pattern += ")";
- }
- pattern += `(?:$|[${Array.from(astUtils.LINEBREAKS).join("")}]))`;
- }
- return pattern;
- }
- /**
- * Creates RegExp object for `always` mode.
- * Generated pattern for beginning of comment:
- *
- * 1. First, a marker or nothing.
- * 2. Next, a space or an exception pattern sequence.
- * @param {string[]} markers A marker list.
- * @param {string[]} exceptions An exception pattern list.
- * @returns {RegExp} A RegExp object for the beginning of a comment in `always` mode.
- */
- function createAlwaysStylePattern(markers, exceptions) {
- let pattern = "^";
- /*
- * A marker or nothing.
- * ["*"] ==> "\*?"
- * ["*", "!"] ==> "(?:\*|!)?"
- * ["*", "/", "!<"] ==> "(?:\*|\/|(?:!<))?" ==> https://jex.im/regulex/#!embed=false&flags=&re=(%3F%3A%5C*%7C%5C%2F%7C(%3F%3A!%3C))%3F
- */
- if (markers.length === 1) {
- // the marker.
- pattern += escape(markers[0]);
- } else {
- // one of markers.
- pattern += "(?:";
- pattern += markers.map(escape).join("|");
- pattern += ")";
- }
- pattern += "?"; // or nothing.
- pattern += createExceptionsPattern(exceptions);
- return new RegExp(pattern, "u");
- }
- /**
- * Creates RegExp object for `never` mode.
- * Generated pattern for beginning of comment:
- *
- * 1. First, a marker or nothing (captured).
- * 2. Next, a space or a tab.
- * @param {string[]} markers A marker list.
- * @returns {RegExp} A RegExp object for `never` mode.
- */
- function createNeverStylePattern(markers) {
- const pattern = `^(${markers.map(escape).join("|")})?[ \t]+`;
- return new RegExp(pattern, "u");
- }
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- /** @type {import('../types').Rule.RuleModule} */
- module.exports = {
- meta: {
- deprecated: {
- message: "Formatting rules are being moved out of ESLint core.",
- url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
- deprecatedSince: "8.53.0",
- availableUntil: "11.0.0",
- replacedBy: [
- {
- message:
- "ESLint Stylistic now maintains deprecated stylistic core rules.",
- url: "https://eslint.style/guide/migration",
- plugin: {
- name: "@stylistic/eslint-plugin",
- url: "https://eslint.style",
- },
- rule: {
- name: "spaced-comment",
- url: "https://eslint.style/rules/spaced-comment",
- },
- },
- ],
- },
- type: "suggestion",
- docs: {
- description:
- "Enforce consistent spacing after the `//` or `/*` in a comment",
- recommended: false,
- url: "https://eslint.org/docs/latest/rules/spaced-comment",
- },
- fixable: "whitespace",
- schema: [
- {
- enum: ["always", "never"],
- },
- {
- type: "object",
- properties: {
- exceptions: {
- type: "array",
- items: {
- type: "string",
- },
- },
- markers: {
- type: "array",
- items: {
- type: "string",
- },
- },
- line: {
- type: "object",
- properties: {
- exceptions: {
- type: "array",
- items: {
- type: "string",
- },
- },
- markers: {
- type: "array",
- items: {
- type: "string",
- },
- },
- },
- additionalProperties: false,
- },
- block: {
- type: "object",
- properties: {
- exceptions: {
- type: "array",
- items: {
- type: "string",
- },
- },
- markers: {
- type: "array",
- items: {
- type: "string",
- },
- },
- balanced: {
- type: "boolean",
- default: false,
- },
- },
- additionalProperties: false,
- },
- },
- additionalProperties: false,
- },
- ],
- messages: {
- unexpectedSpaceAfterMarker:
- "Unexpected space or tab after marker ({{refChar}}) in comment.",
- expectedExceptionAfter:
- "Expected exception block, space or tab after '{{refChar}}' in comment.",
- unexpectedSpaceBefore:
- "Unexpected space or tab before '*/' in comment.",
- unexpectedSpaceAfter:
- "Unexpected space or tab after '{{refChar}}' in comment.",
- expectedSpaceBefore:
- "Expected space or tab before '*/' in comment.",
- expectedSpaceAfter:
- "Expected space or tab after '{{refChar}}' in comment.",
- },
- },
- create(context) {
- const sourceCode = context.sourceCode;
- // Unless the first option is never, require a space
- const requireSpace = context.options[0] !== "never";
- /*
- * Parse the second options.
- * If markers don't include `"*"`, it's added automatically for JSDoc
- * comments.
- */
- const config = context.options[1] || {};
- const balanced = config.block && config.block.balanced;
- const styleRules = ["block", "line"].reduce((rule, type) => {
- const markers = parseMarkersOption(
- (config[type] && config[type].markers) || config.markers || [],
- );
- const exceptions =
- (config[type] && config[type].exceptions) ||
- config.exceptions ||
- [];
- const endNeverPattern = "[ \t]+$";
- // Create RegExp object for valid patterns.
- rule[type] = {
- beginRegex: requireSpace
- ? createAlwaysStylePattern(markers, exceptions)
- : createNeverStylePattern(markers),
- endRegex:
- balanced && requireSpace
- ? new RegExp(
- `${createExceptionsPattern(exceptions)}$`,
- "u",
- )
- : new RegExp(endNeverPattern, "u"),
- hasExceptions: exceptions.length > 0,
- captureMarker: new RegExp(
- `^(${markers.map(escape).join("|")})`,
- "u",
- ),
- markers: new Set(markers),
- };
- return rule;
- }, {});
- /**
- * Reports a beginning spacing error with an appropriate message.
- * @param {ASTNode} node A comment node to check.
- * @param {string} messageId An error message to report.
- * @param {Array} match An array of match results for markers.
- * @param {string} refChar Character used for reference in the error message.
- * @returns {void}
- */
- function reportBegin(node, messageId, match, refChar) {
- const type = node.type.toLowerCase(),
- commentIdentifier = type === "block" ? "/*" : "//";
- context.report({
- node,
- fix(fixer) {
- const start = node.range[0];
- let end = start + 2;
- if (requireSpace) {
- if (match) {
- end += match[0].length;
- }
- return fixer.insertTextAfterRange([start, end], " ");
- }
- end += match[0].length;
- return fixer.replaceTextRange(
- [start, end],
- commentIdentifier + (match[1] ? match[1] : ""),
- );
- },
- messageId,
- data: { refChar },
- });
- }
- /**
- * Reports an ending spacing error with an appropriate message.
- * @param {ASTNode} node A comment node to check.
- * @param {string} messageId An error message to report.
- * @param {string} match An array of the matched whitespace characters.
- * @returns {void}
- */
- function reportEnd(node, messageId, match) {
- context.report({
- node,
- fix(fixer) {
- if (requireSpace) {
- return fixer.insertTextAfterRange(
- [node.range[0], node.range[1] - 2],
- " ",
- );
- }
- const end = node.range[1] - 2,
- start = end - match[0].length;
- return fixer.replaceTextRange([start, end], "");
- },
- messageId,
- });
- }
- /**
- * Reports a given comment if it's invalid.
- * @param {ASTNode} node a comment node to check.
- * @returns {void}
- */
- function checkCommentForSpace(node) {
- const type = node.type.toLowerCase(),
- rule = styleRules[type],
- commentIdentifier = type === "block" ? "/*" : "//";
- // Ignores empty comments and comments that consist only of a marker.
- if (node.value.length === 0 || rule.markers.has(node.value)) {
- return;
- }
- const beginMatch = rule.beginRegex.exec(node.value);
- const endMatch = rule.endRegex.exec(node.value);
- // Checks.
- if (requireSpace) {
- if (!beginMatch) {
- const hasMarker = rule.captureMarker.exec(node.value);
- const marker = hasMarker
- ? commentIdentifier + hasMarker[0]
- : commentIdentifier;
- if (rule.hasExceptions) {
- reportBegin(
- node,
- "expectedExceptionAfter",
- hasMarker,
- marker,
- );
- } else {
- reportBegin(
- node,
- "expectedSpaceAfter",
- hasMarker,
- marker,
- );
- }
- }
- if (balanced && type === "block" && !endMatch) {
- reportEnd(node, "expectedSpaceBefore");
- }
- } else {
- if (beginMatch) {
- if (!beginMatch[1]) {
- reportBegin(
- node,
- "unexpectedSpaceAfter",
- beginMatch,
- commentIdentifier,
- );
- } else {
- reportBegin(
- node,
- "unexpectedSpaceAfterMarker",
- beginMatch,
- beginMatch[1],
- );
- }
- }
- if (balanced && type === "block" && endMatch) {
- reportEnd(node, "unexpectedSpaceBefore", endMatch);
- }
- }
- }
- return {
- Program() {
- const comments = sourceCode.getAllComments();
- comments
- .filter(token => token.type !== "Shebang")
- .forEach(checkCommentForSpace);
- },
- };
- },
- };
|