123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308 |
- /**
- * @fileoverview Disallows or enforces spaces inside of object literals.
- * @author Jamund Ferguson
- */
- "use strict";
- const astUtils = require("./utils/ast-utils");
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- module.exports = {
- meta: {
- type: "layout",
- docs: {
- description: "enforce consistent spacing inside braces",
- category: "Stylistic Issues",
- recommended: false,
- url: "https://eslint.org/docs/rules/object-curly-spacing"
- },
- fixable: "whitespace",
- schema: [
- {
- enum: ["always", "never"]
- },
- {
- type: "object",
- properties: {
- arraysInObjects: {
- type: "boolean"
- },
- objectsInObjects: {
- type: "boolean"
- }
- },
- additionalProperties: false
- }
- ],
- messages: {
- requireSpaceBefore: "A space is required before '{{token}}'.",
- requireSpaceAfter: "A space is required after '{{token}}'.",
- unexpectedSpaceBefore: "There should be no space before '{{token}}'.",
- unexpectedSpaceAfter: "There should be no space after '{{token}}'."
- }
- },
- create(context) {
- const spaced = context.options[0] === "always",
- sourceCode = context.getSourceCode();
- /**
- * Determines whether an option is set, relative to the spacing option.
- * If spaced is "always", then check whether option is set to false.
- * If spaced is "never", then check whether option is set to true.
- * @param {Object} option The option to exclude.
- * @returns {boolean} Whether or not the property is excluded.
- */
- function isOptionSet(option) {
- return context.options[1] ? context.options[1][option] === !spaced : false;
- }
- const options = {
- spaced,
- arraysInObjectsException: isOptionSet("arraysInObjects"),
- objectsInObjectsException: isOptionSet("objectsInObjects")
- };
- //--------------------------------------------------------------------------
- // Helpers
- //--------------------------------------------------------------------------
- /**
- * Reports that there shouldn't be a space after the first token
- * @param {ASTNode} node The node to report in the event of an error.
- * @param {Token} token The token to use for the report.
- * @returns {void}
- */
- function reportNoBeginningSpace(node, token) {
- const nextToken = context.getSourceCode().getTokenAfter(token, { includeComments: true });
- context.report({
- node,
- loc: { start: token.loc.end, end: nextToken.loc.start },
- messageId: "unexpectedSpaceAfter",
- data: {
- token: token.value
- },
- fix(fixer) {
- return fixer.removeRange([token.range[1], nextToken.range[0]]);
- }
- });
- }
- /**
- * Reports that there shouldn't be a space before the last token
- * @param {ASTNode} node The node to report in the event of an error.
- * @param {Token} token The token to use for the report.
- * @returns {void}
- */
- function reportNoEndingSpace(node, token) {
- const previousToken = context.getSourceCode().getTokenBefore(token, { includeComments: true });
- context.report({
- node,
- loc: { start: previousToken.loc.end, end: token.loc.start },
- messageId: "unexpectedSpaceBefore",
- data: {
- token: token.value
- },
- fix(fixer) {
- return fixer.removeRange([previousToken.range[1], token.range[0]]);
- }
- });
- }
- /**
- * Reports that there should be a space after the first token
- * @param {ASTNode} node The node to report in the event of an error.
- * @param {Token} token The token to use for the report.
- * @returns {void}
- */
- function reportRequiredBeginningSpace(node, token) {
- context.report({
- node,
- loc: token.loc,
- messageId: "requireSpaceAfter",
- data: {
- token: token.value
- },
- fix(fixer) {
- return fixer.insertTextAfter(token, " ");
- }
- });
- }
- /**
- * Reports that there should be a space before the last token
- * @param {ASTNode} node The node to report in the event of an error.
- * @param {Token} token The token to use for the report.
- * @returns {void}
- */
- function reportRequiredEndingSpace(node, token) {
- context.report({
- node,
- loc: token.loc,
- messageId: "requireSpaceBefore",
- data: {
- token: token.value
- },
- fix(fixer) {
- return fixer.insertTextBefore(token, " ");
- }
- });
- }
- /**
- * Determines if spacing in curly braces is valid.
- * @param {ASTNode} node The AST node to check.
- * @param {Token} first The first token to check (should be the opening brace)
- * @param {Token} second The second token to check (should be first after the opening brace)
- * @param {Token} penultimate The penultimate token to check (should be last before closing brace)
- * @param {Token} last The last token to check (should be closing brace)
- * @returns {void}
- */
- function validateBraceSpacing(node, first, second, penultimate, last) {
- if (astUtils.isTokenOnSameLine(first, second)) {
- const firstSpaced = sourceCode.isSpaceBetweenTokens(first, second);
- if (options.spaced && !firstSpaced) {
- reportRequiredBeginningSpace(node, first);
- }
- if (!options.spaced && firstSpaced && second.type !== "Line") {
- reportNoBeginningSpace(node, first);
- }
- }
- if (astUtils.isTokenOnSameLine(penultimate, last)) {
- const shouldCheckPenultimate = (
- options.arraysInObjectsException && astUtils.isClosingBracketToken(penultimate) ||
- options.objectsInObjectsException && astUtils.isClosingBraceToken(penultimate)
- );
- const penultimateType = shouldCheckPenultimate && sourceCode.getNodeByRangeIndex(penultimate.range[0]).type;
- const closingCurlyBraceMustBeSpaced = (
- options.arraysInObjectsException && penultimateType === "ArrayExpression" ||
- options.objectsInObjectsException && (penultimateType === "ObjectExpression" || penultimateType === "ObjectPattern")
- ) ? !options.spaced : options.spaced;
- const lastSpaced = sourceCode.isSpaceBetweenTokens(penultimate, last);
- if (closingCurlyBraceMustBeSpaced && !lastSpaced) {
- reportRequiredEndingSpace(node, last);
- }
- if (!closingCurlyBraceMustBeSpaced && lastSpaced) {
- reportNoEndingSpace(node, last);
- }
- }
- }
- /**
- * Gets '}' token of an object node.
- *
- * Because the last token of object patterns might be a type annotation,
- * this traverses tokens preceded by the last property, then returns the
- * first '}' token.
- * @param {ASTNode} node The node to get. This node is an
- * ObjectExpression or an ObjectPattern. And this node has one or
- * more properties.
- * @returns {Token} '}' token.
- */
- function getClosingBraceOfObject(node) {
- const lastProperty = node.properties[node.properties.length - 1];
- return sourceCode.getTokenAfter(lastProperty, astUtils.isClosingBraceToken);
- }
- /**
- * Reports a given object node if spacing in curly braces is invalid.
- * @param {ASTNode} node An ObjectExpression or ObjectPattern node to check.
- * @returns {void}
- */
- function checkForObject(node) {
- if (node.properties.length === 0) {
- return;
- }
- const first = sourceCode.getFirstToken(node),
- last = getClosingBraceOfObject(node),
- second = sourceCode.getTokenAfter(first, { includeComments: true }),
- penultimate = sourceCode.getTokenBefore(last, { includeComments: true });
- validateBraceSpacing(node, first, second, penultimate, last);
- }
- /**
- * Reports a given import node if spacing in curly braces is invalid.
- * @param {ASTNode} node An ImportDeclaration node to check.
- * @returns {void}
- */
- function checkForImport(node) {
- if (node.specifiers.length === 0) {
- return;
- }
- let firstSpecifier = node.specifiers[0];
- const lastSpecifier = node.specifiers[node.specifiers.length - 1];
- if (lastSpecifier.type !== "ImportSpecifier") {
- return;
- }
- if (firstSpecifier.type !== "ImportSpecifier") {
- firstSpecifier = node.specifiers[1];
- }
- const first = sourceCode.getTokenBefore(firstSpecifier),
- last = sourceCode.getTokenAfter(lastSpecifier, astUtils.isNotCommaToken),
- second = sourceCode.getTokenAfter(first, { includeComments: true }),
- penultimate = sourceCode.getTokenBefore(last, { includeComments: true });
- validateBraceSpacing(node, first, second, penultimate, last);
- }
- /**
- * Reports a given export node if spacing in curly braces is invalid.
- * @param {ASTNode} node An ExportNamedDeclaration node to check.
- * @returns {void}
- */
- function checkForExport(node) {
- if (node.specifiers.length === 0) {
- return;
- }
- const firstSpecifier = node.specifiers[0],
- lastSpecifier = node.specifiers[node.specifiers.length - 1],
- first = sourceCode.getTokenBefore(firstSpecifier),
- last = sourceCode.getTokenAfter(lastSpecifier, astUtils.isNotCommaToken),
- second = sourceCode.getTokenAfter(first, { includeComments: true }),
- penultimate = sourceCode.getTokenBefore(last, { includeComments: true });
- validateBraceSpacing(node, first, second, penultimate, last);
- }
- //--------------------------------------------------------------------------
- // Public
- //--------------------------------------------------------------------------
- return {
- // var {x} = y;
- ObjectPattern: checkForObject,
- // var y = {x: 'y'}
- ObjectExpression: checkForObject,
- // import {y} from 'x';
- ImportDeclaration: checkForImport,
- // export {name} from 'yo';
- ExportNamedDeclaration: checkForExport
- };
- }
- };
|