Move recommended rules to their own list

Author: benjie

src/index.ts

@@ -355,6 +355,7 @@ export {
   ValidationContext,
   // All validation rules in the GraphQL Specification.
   specifiedRules,
+  recommendedRules,
   // Individual validation rules.
   ExecutableDefinitionsRule,
   FieldsOnCorrectTypeRule,

src/validation/index.ts

@@ -4,7 +4,7 @@ export { ValidationContext } from './ValidationContext.js';
 export type { ValidationRule } from './ValidationContext.js';
 
 // All validation rules in the GraphQL Specification.
-export { specifiedRules } from './specifiedRules.js';
+export { specifiedRules, recommendedRules } from './specifiedRules.js';
 
 // Spec Section: "Defer And Stream Directive Labels Are Unique"
 export { DeferStreamDirectiveLabelRule } from './rules/DeferStreamDirectiveLabelRule.js';

src/validation/specifiedRules.ts

@@ -25,6 +25,8 @@ import { KnownTypeNamesRule } from './rules/KnownTypeNamesRule.js';
 import { LoneAnonymousOperationRule } from './rules/LoneAnonymousOperationRule.js';
 // SDL-specific validation rules
 import { LoneSchemaDefinitionRule } from './rules/LoneSchemaDefinitionRule.js';
+// TODO: Spec Section
+import { MaxIntrospectionDepthRule } from './rules/MaxIntrospectionDepthRule.js';
 // Spec Section: "Fragments must not form cycles"
 import { NoFragmentCyclesRule } from './rules/NoFragmentCyclesRule.js';
 // Spec Section: "All Variable Used Defined"
@@ -75,8 +77,11 @@ import { VariablesAreInputTypesRule } from './rules/VariablesAreInputTypesRule.j
 import { VariablesInAllowedPositionRule } from './rules/VariablesInAllowedPositionRule.js';
 import type { SDLValidationRule, ValidationRule } from './ValidationContext.js';
 
-// TODO: Spec Section
-import { MaxIntrospectionDepthRule } from './rules/MaxIntrospectionDepthRule.js';
+/**
+ * Technically these aren't part of the spec but they are strongly encouraged
+ * validation rules.
+ */
+export const recommendedRules = Object.freeze([MaxIntrospectionDepthRule]);
 
 /**
  * This set includes all validation rules defined by the GraphQL spec.
@@ -115,8 +120,7 @@ export const specifiedRules: ReadonlyArray<ValidationRule> = Object.freeze([
   VariablesInAllowedPositionRule,
   OverlappingFieldsCanBeMergedRule,
   UniqueInputFieldNamesRule,
-  // Technically this isn't part of the spec but it's a strongly encouraged validation rule.
-  MaxIntrospectionDepthRule,
+  ...recommendedRules,
 ]);
 
 /**