eslint-plugin-unicorn/rules/fix/switch-new-expression-to-call-expression.js

More than 100 powerful ESLint rules

import isNewExpressionWithParentheses from '../utils/is-new-expression-with-parentheses.js';
import {isParenthesized} from '../utils/parentheses.js';
import isOnSameLine from '../utils/is-on-same-line.js';
import addParenthesizesToReturnOrThrowExpression from './add-parenthesizes-to-return-or-throw-expression.js';
import removeSpaceAfter from './remove-spaces-after.js';

export default function * switchNewExpressionToCallExpression(newExpression, sourceCode, fixer) {
	const newToken = sourceCode.getFirstToken(newExpression);
	yield fixer.remove(newToken);
	yield removeSpaceAfter(newToken, sourceCode, fixer);

	if (!isNewExpressionWithParentheses(newExpression, sourceCode)) {
		yield fixer.insertTextAfter(newExpression, '()');
	}

	/*
		Remove `new` from this code will makes the function return `undefined`

		```js
			() => {
				return new // comment
					Foo()
			}
		```
	*/
	if (!isOnSameLine(newToken, newExpression.callee) && !isParenthesized(newExpression, sourceCode)) {
		// Ideally, we should use first parenthesis of the `callee`, and should check spaces after the `new` token
		// But adding extra parentheses is harmless, no need to be too complicated
		yield * addParenthesizesToReturnOrThrowExpression(fixer, newExpression.parent, sourceCode);
	}
}