Remove definition iteration from 'ExecutableDefinitions' rule

Author: IvanGoncharov

src/validation/rules/ExecutableDefinitions.js

@@ -10,7 +10,10 @@
 import type { ASTValidationContext } from '../ValidationContext';
 import { GraphQLError } from '../../error/GraphQLError';
 import { Kind } from '../../language/kinds';
-import { isExecutableDefinitionNode } from '../../language/predicates';
+import {
+  isDefinitionNode,
+  isExecutableDefinitionNode,
+} from '../../language/predicates';
 import type { ASTVisitor } from '../../language/visitor';
 
 export function nonExecutableDefinitionMessage(defName: string): string {
@@ -27,23 +30,24 @@ export function ExecutableDefinitions(
   context: ASTValidationContext,
 ): ASTVisitor {
   return {
-    Document(node) {
-      for (const definition of node.definitions) {
-        if (!isExecutableDefinitionNode(definition)) {
+    enter(node) {
+      if (isDefinitionNode(node)) {
+        if (!isExecutableDefinitionNode(node)) {
+          const isSchemaNode =
+            node.kind === Kind.SCHEMA_DEFINITION ||
+            node.kind === Kind.SCHEMA_EXTENSION;
+
           context.reportError(
             new GraphQLError(
               nonExecutableDefinitionMessage(
-                definition.kind === Kind.SCHEMA_DEFINITION ||
-                definition.kind === Kind.SCHEMA_EXTENSION
-                  ? 'schema'
-                  : definition.name.value,
+                isSchemaNode ? 'schema' : node.name.value,
               ),
-              [definition],
+              node,
             ),
           );
         }
+        return false;
       }
-      return false;
     },
   };
 }