| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652 |
- /**
- * @fileoverview enforce a particular style for multiline comments
- * @author Teddy Katz
- * @deprecated in ESLint v9.3.0
- */
- "use strict";
- const astUtils = require("./utils/ast-utils");
- //------------------------------------------------------------------------------
- // 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: "9.3.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: "multiline-comment-style",
- url: "https://eslint.style/rules/multiline-comment-style",
- },
- },
- ],
- },
- type: "suggestion",
- docs: {
- description: "Enforce a particular style for multiline comments",
- recommended: false,
- url: "https://eslint.org/docs/latest/rules/multiline-comment-style",
- },
- fixable: "whitespace",
- schema: {
- anyOf: [
- {
- type: "array",
- items: [
- {
- enum: ["starred-block", "bare-block"],
- },
- ],
- additionalItems: false,
- },
- {
- type: "array",
- items: [
- {
- enum: ["separate-lines"],
- },
- {
- type: "object",
- properties: {
- checkJSDoc: {
- type: "boolean",
- },
- },
- additionalProperties: false,
- },
- ],
- additionalItems: false,
- },
- ],
- },
- messages: {
- expectedBlock:
- "Expected a block comment instead of consecutive line comments.",
- expectedBareBlock:
- "Expected a block comment without padding stars.",
- startNewline: "Expected a linebreak after '/*'.",
- endNewline: "Expected a linebreak before '*/'.",
- missingStar: "Expected a '*' at the start of this line.",
- alignment:
- "Expected this line to be aligned with the start of the comment.",
- expectedLines:
- "Expected multiple line comments instead of a block comment.",
- },
- },
- create(context) {
- const sourceCode = context.sourceCode;
- const option = context.options[0] || "starred-block";
- const params = context.options[1] || {};
- const checkJSDoc = !!params.checkJSDoc;
- //----------------------------------------------------------------------
- // Helpers
- //----------------------------------------------------------------------
- /**
- * Checks if a comment line is starred.
- * @param {string} line A string representing a comment line.
- * @returns {boolean} Whether or not the comment line is starred.
- */
- function isStarredCommentLine(line) {
- return /^\s*\*/u.test(line);
- }
- /**
- * Checks if a comment group is in starred-block form.
- * @param {Token[]} commentGroup A group of comments, containing either multiple line comments or a single block comment.
- * @returns {boolean} Whether or not the comment group is in starred block form.
- */
- function isStarredBlockComment([firstComment]) {
- if (firstComment.type !== "Block") {
- return false;
- }
- const lines = firstComment.value.split(astUtils.LINEBREAK_MATCHER);
- // The first and last lines can only contain whitespace.
- return (
- lines.length > 0 &&
- lines.every((line, i) =>
- (i === 0 || i === lines.length - 1
- ? /^\s*$/u
- : /^\s*\*/u
- ).test(line),
- )
- );
- }
- /**
- * Checks if a comment group is in JSDoc form.
- * @param {Token[]} commentGroup A group of comments, containing either multiple line comments or a single block comment.
- * @returns {boolean} Whether or not the comment group is in JSDoc form.
- */
- function isJSDocComment([firstComment]) {
- if (firstComment.type !== "Block") {
- return false;
- }
- const lines = firstComment.value.split(astUtils.LINEBREAK_MATCHER);
- return (
- /^\*\s*$/u.test(lines[0]) &&
- lines.slice(1, -1).every(line => /^\s* /u.test(line)) &&
- /^\s*$/u.test(lines.at(-1))
- );
- }
- /**
- * Processes a comment group that is currently in separate-line form, calculating the offset for each line.
- * @param {Token[]} commentGroup A group of comments containing multiple line comments.
- * @returns {string[]} An array of the processed lines.
- */
- function processSeparateLineComments(commentGroup) {
- const allLinesHaveLeadingSpace = commentGroup
- .map(({ value }) => value)
- .filter(line => line.trim().length)
- .every(line => line.startsWith(" "));
- return commentGroup.map(({ value }) =>
- allLinesHaveLeadingSpace ? value.replace(/^ /u, "") : value,
- );
- }
- /**
- * Processes a comment group that is currently in starred-block form, calculating the offset for each line.
- * @param {Token} comment A single block comment token in starred-block form.
- * @returns {string[]} An array of the processed lines.
- */
- function processStarredBlockComment(comment) {
- const lines = comment.value
- .split(astUtils.LINEBREAK_MATCHER)
- .filter(
- (line, i, linesArr) =>
- !(i === 0 || i === linesArr.length - 1),
- )
- .map(line => line.replace(/^\s*$/u, ""));
- const allLinesHaveLeadingSpace = lines
- .map(line => line.replace(/\s*\*/u, ""))
- .filter(line => line.trim().length)
- .every(line => line.startsWith(" "));
- return lines.map(line =>
- line.replace(
- allLinesHaveLeadingSpace ? /\s*\* ?/u : /\s*\*/u,
- "",
- ),
- );
- }
- /**
- * Processes a comment group that is currently in bare-block form, calculating the offset for each line.
- * @param {Token} comment A single block comment token in bare-block form.
- * @returns {string[]} An array of the processed lines.
- */
- function processBareBlockComment(comment) {
- const lines = comment.value
- .split(astUtils.LINEBREAK_MATCHER)
- .map(line => line.replace(/^\s*$/u, ""));
- const leadingWhitespace = `${sourceCode.text.slice(comment.range[0] - comment.loc.start.column, comment.range[0])} `;
- let offset = "";
- /*
- * Calculate the offset of the least indented line and use that as the basis for offsetting all the lines.
- * The first line should not be checked because it is inline with the opening block comment delimiter.
- */
- for (const [i, line] of lines.entries()) {
- if (!line.trim().length || i === 0) {
- continue;
- }
- const [, lineOffset] = line.match(/^(\s*\*?\s*)/u);
- if (lineOffset.length < leadingWhitespace.length) {
- const newOffset = leadingWhitespace.slice(
- lineOffset.length - leadingWhitespace.length,
- );
- if (newOffset.length > offset.length) {
- offset = newOffset;
- }
- }
- }
- return lines.map(line => {
- const match = line.match(/^(\s*\*?\s*)(.*)/u);
- const [, lineOffset, lineContents] = match;
- if (lineOffset.length > leadingWhitespace.length) {
- return `${lineOffset.slice(leadingWhitespace.length - (offset.length + lineOffset.length))}${lineContents}`;
- }
- if (lineOffset.length < leadingWhitespace.length) {
- return `${lineOffset.slice(leadingWhitespace.length)}${lineContents}`;
- }
- return lineContents;
- });
- }
- /**
- * Gets a list of comment lines in a group, formatting leading whitespace as necessary.
- * @param {Token[]} commentGroup A group of comments containing either multiple line comments or a single block comment.
- * @returns {string[]} A list of comment lines.
- */
- function getCommentLines(commentGroup) {
- const [firstComment] = commentGroup;
- if (firstComment.type === "Line") {
- return processSeparateLineComments(commentGroup);
- }
- if (isStarredBlockComment(commentGroup)) {
- return processStarredBlockComment(firstComment);
- }
- return processBareBlockComment(firstComment);
- }
- /**
- * Gets the initial offset (whitespace) from the beginning of a line to a given comment token.
- * @param {Token} comment The token to check.
- * @returns {string} The offset from the beginning of a line to the token.
- */
- function getInitialOffset(comment) {
- return sourceCode.text.slice(
- comment.range[0] - comment.loc.start.column,
- comment.range[0],
- );
- }
- /**
- * Converts a comment into starred-block form
- * @param {Token} firstComment The first comment of the group being converted
- * @param {string[]} commentLinesList A list of lines to appear in the new starred-block comment
- * @returns {string} A representation of the comment value in starred-block form, excluding start and end markers
- */
- function convertToStarredBlock(firstComment, commentLinesList) {
- const initialOffset = getInitialOffset(firstComment);
- return `/*\n${commentLinesList.map(line => `${initialOffset} * ${line}`).join("\n")}\n${initialOffset} */`;
- }
- /**
- * Converts a comment into separate-line form
- * @param {Token} firstComment The first comment of the group being converted
- * @param {string[]} commentLinesList A list of lines to appear in the new starred-block comment
- * @returns {string} A representation of the comment value in separate-line form
- */
- function convertToSeparateLines(firstComment, commentLinesList) {
- return commentLinesList
- .map(line => `// ${line}`)
- .join(`\n${getInitialOffset(firstComment)}`);
- }
- /**
- * Converts a comment into bare-block form
- * @param {Token} firstComment The first comment of the group being converted
- * @param {string[]} commentLinesList A list of lines to appear in the new starred-block comment
- * @returns {string} A representation of the comment value in bare-block form
- */
- function convertToBlock(firstComment, commentLinesList) {
- return `/* ${commentLinesList.join(`\n${getInitialOffset(firstComment)} `)} */`;
- }
- /**
- * Each method checks a group of comments to see if it's valid according to the given option.
- * @param {Token[]} commentGroup A list of comments that appear together. This will either contain a single
- * block comment or multiple line comments.
- * @returns {void}
- */
- const commentGroupCheckers = {
- "starred-block"(commentGroup) {
- const [firstComment] = commentGroup;
- const commentLines = getCommentLines(commentGroup);
- if (commentLines.some(value => value.includes("*/"))) {
- return;
- }
- if (commentGroup.length > 1) {
- context.report({
- loc: {
- start: firstComment.loc.start,
- end: commentGroup.at(-1).loc.end,
- },
- messageId: "expectedBlock",
- fix(fixer) {
- const range = [
- firstComment.range[0],
- commentGroup.at(-1).range[1],
- ];
- return commentLines.some(value =>
- value.startsWith("/"),
- )
- ? null
- : fixer.replaceTextRange(
- range,
- convertToStarredBlock(
- firstComment,
- commentLines,
- ),
- );
- },
- });
- } else {
- const lines = firstComment.value.split(
- astUtils.LINEBREAK_MATCHER,
- );
- const expectedLeadingWhitespace =
- getInitialOffset(firstComment);
- const expectedLinePrefix = `${expectedLeadingWhitespace} *`;
- if (!/^\*?\s*$/u.test(lines[0])) {
- const start = firstComment.value.startsWith("*")
- ? firstComment.range[0] + 1
- : firstComment.range[0];
- context.report({
- loc: {
- start: firstComment.loc.start,
- end: {
- line: firstComment.loc.start.line,
- column: firstComment.loc.start.column + 2,
- },
- },
- messageId: "startNewline",
- fix: fixer =>
- fixer.insertTextAfterRange(
- [start, start + 2],
- `\n${expectedLinePrefix}`,
- ),
- });
- }
- if (!/^\s*$/u.test(lines.at(-1))) {
- context.report({
- loc: {
- start: {
- line: firstComment.loc.end.line,
- column: firstComment.loc.end.column - 2,
- },
- end: firstComment.loc.end,
- },
- messageId: "endNewline",
- fix: fixer =>
- fixer.replaceTextRange(
- [
- firstComment.range[1] - 2,
- firstComment.range[1],
- ],
- `\n${expectedLinePrefix}/`,
- ),
- });
- }
- for (
- let lineNumber = firstComment.loc.start.line + 1;
- lineNumber <= firstComment.loc.end.line;
- lineNumber++
- ) {
- const lineText = sourceCode.lines[lineNumber - 1];
- const errorType = isStarredCommentLine(lineText)
- ? "alignment"
- : "missingStar";
- if (!lineText.startsWith(expectedLinePrefix)) {
- context.report({
- loc: {
- start: { line: lineNumber, column: 0 },
- end: {
- line: lineNumber,
- column: lineText.length,
- },
- },
- messageId: errorType,
- fix(fixer) {
- const lineStartIndex =
- sourceCode.getIndexFromLoc({
- line: lineNumber,
- column: 0,
- });
- if (errorType === "alignment") {
- const [, commentTextPrefix = ""] =
- lineText.match(/^(\s*\*)/u) || [];
- const commentTextStartIndex =
- lineStartIndex +
- commentTextPrefix.length;
- return fixer.replaceTextRange(
- [
- lineStartIndex,
- commentTextStartIndex,
- ],
- expectedLinePrefix,
- );
- }
- const [, commentTextPrefix = ""] =
- lineText.match(/^(\s*)/u) || [];
- const commentTextStartIndex =
- lineStartIndex +
- commentTextPrefix.length;
- let offset;
- for (const [idx, line] of lines.entries()) {
- if (!/\S+/u.test(line)) {
- continue;
- }
- const lineTextToAlignWith =
- sourceCode.lines[
- firstComment.loc.start.line -
- 1 +
- idx
- ];
- const [
- ,
- prefix = "",
- initialOffset = "",
- ] =
- lineTextToAlignWith.match(
- /^(\s*(?:\/?\*)?(\s*))/u,
- ) || [];
- offset = `${commentTextPrefix.slice(prefix.length)}${initialOffset}`;
- if (
- /^\s*\//u.test(lineText) &&
- offset.length === 0
- ) {
- offset += " ";
- }
- break;
- }
- return fixer.replaceTextRange(
- [lineStartIndex, commentTextStartIndex],
- `${expectedLinePrefix}${offset}`,
- );
- },
- });
- }
- }
- }
- },
- "separate-lines"(commentGroup) {
- const [firstComment] = commentGroup;
- const isJSDoc = isJSDocComment(commentGroup);
- if (firstComment.type !== "Block" || (!checkJSDoc && isJSDoc)) {
- return;
- }
- let commentLines = getCommentLines(commentGroup);
- if (isJSDoc) {
- commentLines = commentLines.slice(
- 1,
- commentLines.length - 1,
- );
- }
- const tokenAfter = sourceCode.getTokenAfter(firstComment, {
- includeComments: true,
- });
- if (
- tokenAfter &&
- firstComment.loc.end.line === tokenAfter.loc.start.line
- ) {
- return;
- }
- context.report({
- loc: {
- start: firstComment.loc.start,
- end: {
- line: firstComment.loc.start.line,
- column: firstComment.loc.start.column + 2,
- },
- },
- messageId: "expectedLines",
- fix(fixer) {
- return fixer.replaceText(
- firstComment,
- convertToSeparateLines(firstComment, commentLines),
- );
- },
- });
- },
- "bare-block"(commentGroup) {
- if (isJSDocComment(commentGroup)) {
- return;
- }
- const [firstComment] = commentGroup;
- const commentLines = getCommentLines(commentGroup);
- // Disallows consecutive line comments in favor of using a block comment.
- if (
- firstComment.type === "Line" &&
- commentLines.length > 1 &&
- !commentLines.some(value => value.includes("*/"))
- ) {
- context.report({
- loc: {
- start: firstComment.loc.start,
- end: commentGroup.at(-1).loc.end,
- },
- messageId: "expectedBlock",
- fix(fixer) {
- return fixer.replaceTextRange(
- [
- firstComment.range[0],
- commentGroup.at(-1).range[1],
- ],
- convertToBlock(firstComment, commentLines),
- );
- },
- });
- }
- // Prohibits block comments from having a * at the beginning of each line.
- if (isStarredBlockComment(commentGroup)) {
- context.report({
- loc: {
- start: firstComment.loc.start,
- end: {
- line: firstComment.loc.start.line,
- column: firstComment.loc.start.column + 2,
- },
- },
- messageId: "expectedBareBlock",
- fix(fixer) {
- return fixer.replaceText(
- firstComment,
- convertToBlock(firstComment, commentLines),
- );
- },
- });
- }
- },
- };
- //----------------------------------------------------------------------
- // Public
- //----------------------------------------------------------------------
- return {
- Program() {
- return sourceCode
- .getAllComments()
- .filter(comment => comment.type !== "Shebang")
- .filter(
- comment =>
- !astUtils.COMMENTS_IGNORE_PATTERN.test(
- comment.value,
- ),
- )
- .filter(comment => {
- const tokenBefore = sourceCode.getTokenBefore(comment, {
- includeComments: true,
- });
- return (
- !tokenBefore ||
- tokenBefore.loc.end.line < comment.loc.start.line
- );
- })
- .reduce((commentGroups, comment, index, commentList) => {
- const tokenBefore = sourceCode.getTokenBefore(comment, {
- includeComments: true,
- });
- if (
- comment.type === "Line" &&
- index &&
- commentList[index - 1].type === "Line" &&
- tokenBefore &&
- tokenBefore.loc.end.line ===
- comment.loc.start.line - 1 &&
- tokenBefore === commentList[index - 1]
- ) {
- commentGroups.at(-1).push(comment);
- } else {
- commentGroups.push([comment]);
- }
- return commentGroups;
- }, [])
- .filter(
- commentGroup =>
- !(
- commentGroup.length === 1 &&
- commentGroup[0].loc.start.line ===
- commentGroup[0].loc.end.line
- ),
- )
- .forEach(commentGroupCheckers[option]);
- },
- };
- },
- };
|