More powerful `new-for-builtins`

[fisker]

Currently, new-for-builtins enforce builtins used either call or new.

My idea is to extend this rule to check all builtins.

• Objects like JSON/Math/Reflect/Temporal should be neither call nor new
• Objects from global objects like Intl.DateTimeFormat/Temporal.Now... should also be checked
• ~Only fix cases new and call result the same Function('')/new Function(''), Error('')/new Error(''), use suggestion for other cases like new Symbol()(runtime error)~

[dimaMachina]

What’s the reason to disallow new Promise?

[fisker]

Mistake, updated.

[sindresorhus]

use suggestion for other cases like new Symbol()(runtime error)

Why cannot new Symbol() be auto-fixed?

[fisker]

Why cannot new Symbol() be auto-fixed?

I guess I was overthinking.

[sindresorhus]

Accepted