UNPKG

eslint-plugin-sonarjs

Version:
107 lines (106 loc) 4.92 kB
"use strict"; /* * SonarQube JavaScript Plugin * Copyright (C) 2011-2025 SonarSource SA * mailto:info AT sonarsource DOT com * * This program is free software; you can redistribute it and/or * modify it under the terms of the Sonar Source-Available License Version 1, as published by SonarSource SA. * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. * See the Sonar Source-Available License for more details. * * You should have received a copy of the Sonar Source-Available License * along with this program; if not, see https://sonarsource.com/license/ssal/ */ // https://sonarsource.github.io/rspec/#/rspec/S3524/javascript Object.defineProperty(exports, "__esModule", { value: true }); exports.rule = void 0; const index_js_1 = require("../helpers/index.js"); const meta_js_1 = require("./meta.js"); const MESSAGE_ADD_PARAMETER = 'Add parentheses around the parameter of this arrow function.'; const MESSAGE_REMOVE_PARAMETER = 'Remove parentheses around the parameter of this arrow function.'; const MESSAGE_ADD_BODY = 'Add curly braces and "return" to this arrow function body.'; const MESSAGE_REMOVE_BODY = 'Remove curly braces and "return" from this arrow function body.'; const DEFAULT_OPTIONS = { requireParameterParentheses: false, requireBodyBraces: false, }; exports.rule = { meta: (0, index_js_1.generateMeta)(meta_js_1.meta, { schema: meta_js_1.schema }), create(context) { const { requireParameterParentheses, requireBodyBraces } = { ...DEFAULT_OPTIONS, ...context.options[0], }; return { ArrowFunctionExpression(node) { const arrowFunction = node; checkParameters(context, requireParameterParentheses, arrowFunction); checkBody(context, requireBodyBraces, arrowFunction); }, }; }, }; function checkParameters(context, requireParameterParentheses, arrowFunction) { if (arrowFunction.params.length !== 1) { return; } const parameter = arrowFunction.params[0]; // Looking at the closing parenthesis after the parameter to avoid problems with cases like // `functionTakingCallbacks(x => {...})` where the opening parenthesis before `x` isn't part // of the function literal const tokenAfterParameter = context.sourceCode.getTokenAfter(parameter); const hasParameterParentheses = tokenAfterParameter && tokenAfterParameter.value === ')'; if (requireParameterParentheses && !hasParameterParentheses) { context.report({ node: parameter, message: MESSAGE_ADD_PARAMETER }); } else if (!requireParameterParentheses && !hasGeneric(context, arrowFunction) && hasParameterParentheses) { const arrowFunctionComments = context.sourceCode.getCommentsInside(arrowFunction); const arrowFunctionBodyComments = context.sourceCode.getCommentsInside(arrowFunction.body); // parameters comments inside parentheses are not available, so use the following subtraction: const hasArrowFunctionParamsComments = arrowFunctionComments.filter(comment => !arrowFunctionBodyComments.includes(comment)).length > 0; if (parameter.type === 'Identifier' && !hasArrowFunctionParamsComments && !parameter.typeAnnotation && !arrowFunction.returnType) { context.report({ node: parameter, message: MESSAGE_REMOVE_PARAMETER }); } } } function hasGeneric(context, arrowFunction) { const offset = arrowFunction.async ? 1 : 0; const firstTokenIgnoreAsync = context.sourceCode.getFirstToken(arrowFunction, offset); return firstTokenIgnoreAsync && firstTokenIgnoreAsync.value === '<'; } function checkBody(context, requireBodyBraces, arrowFunction) { const hasBodyBraces = arrowFunction.body.type === 'BlockStatement'; if (requireBodyBraces && !hasBodyBraces) { context.report({ node: arrowFunction.body, message: MESSAGE_ADD_BODY }); } else if (!requireBodyBraces && hasBodyBraces) { const statements = arrowFunction.body.body; if (statements.length === 1) { const statement = statements[0]; if (isRemovableReturn(statement)) { context.report({ node: arrowFunction.body, message: MESSAGE_REMOVE_BODY }); } } } } function isRemovableReturn(statement) { if (statement.type === 'ReturnStatement') { const returnStatement = statement; const returnExpression = returnStatement.argument; if (returnExpression && returnExpression.type !== 'ObjectExpression') { const location = returnExpression.loc; return location && location.start.line === location.end.line; } } return false; }