fix: remove optional chaining

Author: developer-bandi

Diff for: packages/eslint-plugin/src/rules/no-array-constructor.ts

@@ -2,7 +2,7 @@ import type { TSESTree } from '@typescript-eslint/utils';
 
 import { AST_NODE_TYPES } from '@typescript-eslint/utils';
 
-import { createRule, isOptionalCallExpression } from '../util';
+import { createRule } from '../util';
 
 export default createRule({
   name: 'no-array-constructor',
@@ -32,8 +32,7 @@ export default createRule({
         node.arguments.length !== 1 &&
         node.callee.type === AST_NODE_TYPES.Identifier &&
         node.callee.name === 'Array' &&
-        !node.typeArguments &&
-        !isOptionalCallExpression(node)
+        !node.typeArguments
       ) {
         context.report({
           node,
@@ -43,7 +42,10 @@ export default createRule({
               return fixer.replaceText(node, '[]');
             }
             const fullText = context.sourceCode.getText(node);
-            const preambleLength = node.callee.range[1] - node.range[0];
+            const preambleLength =
+              node.parent.type === AST_NODE_TYPES.ChainExpression
+                ? node.callee.range[1] + 2 - node.range[0]
+                : node.callee.range[1] - node.range[0];
 
             return fixer.replaceText(
               node,

Diff for: packages/eslint-plugin/tests/rules/no-array-constructor.test.ts

@@ -33,8 +33,6 @@ ruleTester.run('no-array-constructor', rule, {
     'Array.foo?.();',
     'Array?.<Foo>(1, 2, 3);',
     'Array?.<Foo>();',
-    'Array?.(0, 1, 2);',
-    'Array?.(x, y);',
   ],
 
   invalid: [
@@ -58,6 +56,16 @@ ruleTester.run('no-array-constructor', rule, {
       ],
       output: '[];',
     },
+    {
+      code: 'Array?.();',
+      errors: [
+        {
+          messageId,
+          type: AST_NODE_TYPES.CallExpression,
+        },
+      ],
+      output: '[];',
+    },
     {
       code: 'new Array(x, y);',
       errors: [
@@ -78,6 +86,16 @@ ruleTester.run('no-array-constructor', rule, {
       ],
       output: '[x, y];',
     },
+    {
+      code: 'Array?.(x, y);',
+      errors: [
+        {
+          messageId,
+          type: AST_NODE_TYPES.CallExpression,
+        },
+      ],
+      output: '[x, y];',
+    },
     {
       code: 'new Array(0, 1, 2);',
       errors: [
@@ -98,6 +116,16 @@ ruleTester.run('no-array-constructor', rule, {
       ],
       output: '[0, 1, 2];',
     },
+    {
+      code: 'Array?.(0, 1, 2);',
+      errors: [
+        {
+          messageId,
+          type: AST_NODE_TYPES.CallExpression,
+        },
+      ],
+      output: '[0, 1, 2];',
+    },
     {
       code: `
 new Array(0, 1, 2);