feat(linter): Implement eslint/no-object-constructor

[azihsoyn]

Hello, again!

I added eslint rule eslint/no-object-constructor
https://eslint.org/docs/latest/rules/no-object-constructor

Notes

A test case for new Object() has been commented out:

This is due to the test configuration specifying globals: { Object: "off" }.
This approach follows the example set by the no_new_wrappers rule. Reference Code

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[codspeed-hq]

CodSpeed Performance Report

Merging #7345 will not alter performance

_{Comparing azihsoyn:feat/eslint-no-object-constructor (69ba5d4) with main (bf839c1)}

Summary

✅ 30 untouched benchmarks

[no-yan]

Thank you for your contribution!
Since the team seem to be focusing on other tasks, I’ve added comments on smaller points to help keep things moving efficiently.

[no-yan]

I noticed this could be simplified. Can we use this instead?

Suggested change

	ctx.diagnostic(
	crate::rules::eslint::no_object_constructor::no_object_constructor_diagnostic(span),
	);
	ctx.diagnostic(no_object_constructor_diagnostic(span));

[no-yan]

It seems that the fix test cases were not scaffolded properly.

Could you add the test cases in a commented-out form? It makes it easier for implementers to notice and use that cases.

[azihsoyn]

Thanks!