new-for-builtins: Correct fix for Date()

[fisker]

typeof new Date()
'object'
typeof Date()
'string'

[axetroy]

Why no fix this to

const foo = String(new Date(/*comment*/));

It should be safe

[fisker]

It's suggestion, see L1455

[fisker]

Sorry, I misunderstood, we should try not remove comments, and it's hard to fix to your version.

[fisker]

The correct fix seems like

		function * fix(fixer) {
			yield * fixSpaceAroundKeyword(fixer, node, sourceCode);
			yield fixer.insertTextBefore(node, 'String(');
			yield fixer.insertTextAfter(node, ')');
			yield * switchCallExpressionToNewExpression(node, sourceCode, fixer, /* fixSpaces */ false);
			yield * removeArguments();
		}

Just not worth the effort

[fisker]

Maybe you wonder why switchCallExpressionToNewExpression is necessary, something like

Array(globalThis)[0].Date(/*comment*/);

is not easy to fix correctly, but switchCallExpressionToNewExpression already considered this.