modify rule to skip null undefined

Author: margaretjgu

validator/rules/no-inline-unions.js

@@ -39,14 +39,31 @@ export default createRule({
           return;
         }
 
-        // skip simple nullable patterns (Type | null or Type | undefined)
         if (node.types.length === 2) {
-          const hasNullOrUndefined = node.types.some(t => 
-            t.type === 'TSNullKeyword' || t.type === 'TSUndefinedKeyword'
-          );
+          const [first, second] = node.types;
+          
+          // skip Type | null or Type | undefined
+          const hasNullOrUndefined = 
+            first.type === 'TSNullKeyword' || first.type === 'TSUndefinedKeyword' ||
+            second.type === 'TSNullKeyword' || second.type === 'TSUndefinedKeyword';
           if (hasNullOrUndefined) {
             return;
           }
+          
+          // skip Type | Type[] pattern
+          if (second.type === 'TSArrayType') {
+            if (first.type === second.elementType?.type) {
+              if (first.type === 'TSTypeReference' && second.elementType.type === 'TSTypeReference') {
+                // for reference types, check names match
+                if (first.typeName?.name === second.elementType.typeName?.name) {
+                  return;
+                }
+              } else if (first.type === second.elementType.type) {
+                // for primitive types (string, number, etc.), types already match
+                return;
+              }
+            }
+          }
         }
 
         context.report({ 

validator/test/no-inline-unions.test.js

@@ -58,20 +58,14 @@ ruleTester.run('no-inline-unions', rule, {
     `interface Config {
        value: integer | undefined
      }`,
+    `class MyClass {
+      filter: QueryContainer | QueryContainer[]
+    }`,
+    `interface MyInterface {
+      input: string | string[]
+    }`,
   ],
   invalid: [
-    {
-      code: `class MyClass {
-        filter: QueryContainer | QueryContainer[]
-      }`,
-      errors: [{ messageId: 'noInlineUnion' }]
-    },
-    {
-      code: `interface MyInterface {
-        input: string | string[]
-      }`,
-      errors: [{ messageId: 'noInlineUnion' }]
-    },
     {
       code: `export class Request {
         value: string | number | boolean