fix

Author: margaretjgu

specification/nodes/info/types.ts

@@ -143,7 +143,6 @@ export class NodeInfoSettingsCluster {
   name: Name
   routing?: IndexRouting
   election: NodeInfoSettingsClusterElection
-  // eslint-disable-next-line es-spec-validator/no-inline-unions -- TODO: create named alias
   initial_master_nodes?: string[] | string
   /**
    * @availability stack since=7.16.0
@@ -188,7 +187,6 @@ export class NodeInfoRepositoriesUrl {
 export class NodeInfoDiscover
   implements AdditionalProperties<string, UserDefinedValue>
 {
-  // eslint-disable-next-line es-spec-validator/no-inline-unions -- TODO: create named alias
   seed_hosts?: string[] | string
   type?: string
   seed_providers?: string[]

specification/security/_types/RoleDescriptor.ts

@@ -55,7 +55,6 @@ export class RoleDescriptor {
    * An object defining global privileges. A global privilege is a form of cluster privilege that is request-aware. Support for global privileges is currently limited to the management of application privileges.
    * @availability stack
    */
-  // eslint-disable-next-line es-spec-validator/no-inline-unions, es-spec-validator/prefer-tagged-variants -- TODO: use tagged variant
   global?: GlobalPrivilege[] | GlobalPrivilege
   /**
    * A list of application privilege entries
@@ -107,7 +106,6 @@ export class RoleDescriptorRead implements OverloadOf<RoleDescriptor> {
   /**
    * An object defining global privileges. A global privilege is a form of cluster privilege that is request-aware. Support for global privileges is currently limited to the management of application privileges.
    */
-  // eslint-disable-next-line es-spec-validator/no-inline-unions, es-spec-validator/prefer-tagged-variants -- TODO: use tagged variant
   global?: GlobalPrivilege[] | GlobalPrivilege
   /**
    * A list of application privilege entries

validator/rules/no-inline-unions.js

@@ -64,6 +64,21 @@ export default createRule({
               }
             }
           }
+          
+          // skip Type[] | Type pattern
+          if (first.type === 'TSArrayType') {
+            if (first.elementType?.type === second.type) {
+              if (first.elementType.type === 'TSTypeReference' && second.type === 'TSTypeReference') {
+                // for reference types, check names match
+                if (first.elementType.typeName?.name === second.typeName?.name) {
+                  return;
+                }
+              } else if (first.elementType.type === second.type) {
+                // for primitive types (string, number, etc.), types already match
+                return;
+              }
+            }
+          }
         }
 
         context.report({ 

validator/rules/prefer-tagged-variants.js

@@ -70,6 +70,8 @@ export default createRule({
         // skip Type | Type[] pattern
         if (node.types.length === 2) {
           const [first, second] = node.types;
+          
+          // check Type | Type[]
           if (second.type === 'TSArrayType' && 
               first.type === 'TSTypeReference' &&
               second.elementType?.type === 'TSTypeReference') {
@@ -79,6 +81,17 @@ export default createRule({
               return;
             }
           }
+          
+          // check Type[] | Type
+          if (first.type === 'TSArrayType' &&
+              first.elementType?.type === 'TSTypeReference' &&
+              second.type === 'TSTypeReference') {
+            const firstName = first.elementType.typeName?.name;
+            const secondName = second.typeName?.name;
+            if (firstName && firstName === secondName) {
+              return;
+            }
+          }
         }
 
         const allMembersAreClasses = node.types.every(typeNode => {

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

@@ -64,6 +64,12 @@ ruleTester.run('no-inline-unions', rule, {
     `interface MyInterface {
       input: string | string[]
     }`,
+    `class MyClass {
+      items: string[] | string
+    }`,
+    `interface Config {
+      nodes: integer[] | integer
+    }`,
   ],
   invalid: [
     {

validator/test/prefer-tagged-variants.test.js

@@ -58,6 +58,12 @@ ruleTester.run('prefer-tagged-variants', rule, {
     `class SearchRequest {
       knn?: KnnSearch | KnnSearch[]
     }`,
+    `class EqlRequest {
+      filter?: QueryContainer[] | QueryContainer
+    }`,
+    `interface MyInterface {
+      data: MyClass[] | MyClass
+    }`,
   ],
   invalid: [
     {