123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195 |
- /**
- * @fileoverview Comma spacing - validates spacing before and after comma
- * @author Vignesh Anand aka vegetableman.
- */
- "use strict";
- const astUtils = require("./utils/ast-utils");
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- module.exports = {
- meta: {
- type: "layout",
- docs: {
- description: "enforce consistent spacing before and after commas",
- category: "Stylistic Issues",
- recommended: false,
- url: "https://eslint.org/docs/rules/comma-spacing"
- },
- fixable: "whitespace",
- schema: [
- {
- type: "object",
- properties: {
- before: {
- type: "boolean",
- default: false
- },
- after: {
- type: "boolean",
- default: true
- }
- },
- additionalProperties: false
- }
- ],
- messages: {
- missing: "A space is required {{loc}} ','.",
- unexpected: "There should be no space {{loc}} ','."
- }
- },
- create(context) {
- const sourceCode = context.getSourceCode();
- const tokensAndComments = sourceCode.tokensAndComments;
- const options = {
- before: context.options[0] ? context.options[0].before : false,
- after: context.options[0] ? context.options[0].after : true
- };
- //--------------------------------------------------------------------------
- // Helpers
- //--------------------------------------------------------------------------
- // list of comma tokens to ignore for the check of leading whitespace
- const commaTokensToIgnore = [];
- /**
- * Reports a spacing error with an appropriate message.
- * @param {ASTNode} node The binary expression node to report.
- * @param {string} loc Is the error "before" or "after" the comma?
- * @param {ASTNode} otherNode The node at the left or right of `node`
- * @returns {void}
- * @private
- */
- function report(node, loc, otherNode) {
- context.report({
- node,
- fix(fixer) {
- if (options[loc]) {
- if (loc === "before") {
- return fixer.insertTextBefore(node, " ");
- }
- return fixer.insertTextAfter(node, " ");
- }
- let start, end;
- const newText = "";
- if (loc === "before") {
- start = otherNode.range[1];
- end = node.range[0];
- } else {
- start = node.range[1];
- end = otherNode.range[0];
- }
- return fixer.replaceTextRange([start, end], newText);
- },
- messageId: options[loc] ? "missing" : "unexpected",
- data: {
- loc
- }
- });
- }
- /**
- * Validates the spacing around a comma token.
- * @param {Object} tokens The tokens to be validated.
- * @param {Token} tokens.comma The token representing the comma.
- * @param {Token} [tokens.left] The last token before the comma.
- * @param {Token} [tokens.right] The first token after the comma.
- * @param {Token|ASTNode} reportItem The item to use when reporting an error.
- * @returns {void}
- * @private
- */
- function validateCommaItemSpacing(tokens, reportItem) {
- if (tokens.left && astUtils.isTokenOnSameLine(tokens.left, tokens.comma) &&
- (options.before !== sourceCode.isSpaceBetweenTokens(tokens.left, tokens.comma))
- ) {
- report(reportItem, "before", tokens.left);
- }
- if (tokens.right && astUtils.isClosingParenToken(tokens.right)) {
- return;
- }
- if (tokens.right && !options.after && tokens.right.type === "Line") {
- return;
- }
- if (tokens.right && astUtils.isTokenOnSameLine(tokens.comma, tokens.right) &&
- (options.after !== sourceCode.isSpaceBetweenTokens(tokens.comma, tokens.right))
- ) {
- report(reportItem, "after", tokens.right);
- }
- }
- /**
- * Adds null elements of the given ArrayExpression or ArrayPattern node to the ignore list.
- * @param {ASTNode} node An ArrayExpression or ArrayPattern node.
- * @returns {void}
- */
- function addNullElementsToIgnoreList(node) {
- let previousToken = sourceCode.getFirstToken(node);
- node.elements.forEach(element => {
- let token;
- if (element === null) {
- token = sourceCode.getTokenAfter(previousToken);
- if (astUtils.isCommaToken(token)) {
- commaTokensToIgnore.push(token);
- }
- } else {
- token = sourceCode.getTokenAfter(element);
- }
- previousToken = token;
- });
- }
- //--------------------------------------------------------------------------
- // Public
- //--------------------------------------------------------------------------
- return {
- "Program:exit"() {
- tokensAndComments.forEach((token, i) => {
- if (!astUtils.isCommaToken(token)) {
- return;
- }
- if (token && token.type === "JSXText") {
- return;
- }
- const previousToken = tokensAndComments[i - 1];
- const nextToken = tokensAndComments[i + 1];
- validateCommaItemSpacing({
- comma: token,
- left: astUtils.isCommaToken(previousToken) || commaTokensToIgnore.indexOf(token) > -1 ? null : previousToken,
- right: astUtils.isCommaToken(nextToken) ? null : nextToken
- }, token);
- });
- },
- ArrayExpression: addNullElementsToIgnoreList,
- ArrayPattern: addNullElementsToIgnoreList
- };
- }
- };
|