eslint-plugin-unicorn-x/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,
		);
	}
}