🐛 ts/useShorthandArrayType is not taking into consideration the need for parenthesis

[xunilrj]

Environment information

Running the analyzer from the main branch.

What happened?

The lint rule ts/useShorthandArrayType is not taking into consideration the need for parenthesis when creating the suggested fix.

Example:

let v = [] as Array<(...args: E) => void>;
console.log(v);

https://play.rome.tools/?code=let+v+%3D+%5B%5D+as+Array%3C%28...args%3A+E%29+%3D%3E+void%3E%3B%0Aconsole.log%28v%29%3B&lineWidth=80&indentStyle=tab&quoteStyle=double&indentWidth=2&sourceType=module&typescript=true&jsx=false#bABlAHQAIAB2ACAAPQAgAFsAXQAgAGEAcwAgAEEAcgByAGEAeQA8ACgALgAuAC4AYQByAGcAcwA6ACAARQApACAAPQA+ACAAdgBvAGkAZAA+ADsACgBjAG8AbgBzAG8AbABlAC4AbABvAGcAKAB2ACkAOwA=

Suggested fix here is:

error[[ts/useShorthandArrayType](https://rome.tools/docs/lint/rules/useShorthandArrayType/)]: Use shorthand T[] syntax instead of Array<T> syntax.
  ┌─ main.ts:1:15
  │
1 │ let v = [] as Array<(...args: E) => void>;
  │               ^^^^^^^^^^^^^^^^^^^^^^^^^^^

Suggested fix: Use shorthand T[] syntax to replace
    | @@ -1,2 +1,2 @@
0   | - let v = [] as Array<(...args: E) => void>;
  0 | + let v = [] as (...args: E) => void[];
1 1 |   console.log(v);

Expected result

I was expecting the suggested fix to be

let v = [] as ((...args: E) => void)[]; // <- see wrapping parenthesis

Code of Conduct

• I agree to follow Rome's Code of Conduct