remap eslint/prefer-spread to unicorn/prefer-spread

[Sysix]

In this PR, unicorn/prefer-spread rule was migrated to eslint:prefer-spread.
So should we update status of unicorn/prefer-spread? (It seems to be not implemented status)

Originally posted by @tbashiyy in #684 (comment)

Needs the same mapping like jest ot vitest

and then update the doc tasks too:

https://github.com/oxc-project/oxc/blob/main/tasks/lint_rules/src/oxlint-rules.cjs#L181-L198

[camc314]

imo, it would be easiest just to split the rules.
rules are now allowed to be the same name, which bypasses the original reason for why it was merged

[Sysix]

imo, it would be easiest just to split the rules.

Should we do the same for all remapped rules in https://github.com/oxc-project/oxc/blob/main/crates/oxc_linter/src/utils/mod.rs ?

[camc314]

so my reasoning for spliting them is, iirc, eslint/prefer-spread and unicorn/prefer-spread are completly differnt rules

• eslint
• unicorn

Whereas the typescript ones as usually (i haven't checked this) the same as the eslint rule just with TS support

Vitest is basically the same - its 99% the same code across jest/vitest for rules such as no identical title so no need to split

I'm happy to be disagreed with though 🙂

[Boshen]

This is my fault. You're correct that these are completely different rules.

Let's revert that decision and separate these two rules.