feat!: Uses specified rules by default

Adjusts argument order

Author: NeedleInAJayStack

MIGRATION.md

@@ -20,6 +20,25 @@ With the conversion from NIO to Swift Concurrency, types used across async bound
 
 The `queryStrategy`, `mutationStrategy`, and `subscriptionStrategy` arguments have been removed from `graphql` and `graphqlSubscribe`. Instead Queries and Subscriptions are executed in parallel and Mutations are executed serially, [as required by the spec](https://spec.graphql.org/October2021/#sec-Mutation).
 
+### `validationRules` argument reorder
+
+The `validationRules` argument has been moved from the beginning of  `graphql` and `graphqlSubscribe` to the end to better reflect its relative importance:
+
+
+```swift
+// Before
+let result = try await graphql(
+    validationRules: [ruleABC],
+    schema: schema,
+    ...
+)
+// After
+let result = try await graphql(
+    schema: schema,
+    ...
+    validationRules: [ruleABC]
+)
+```
 
 ### EventStream removal
 

Sources/GraphQL/GraphQL.swift

@@ -80,13 +80,13 @@ public struct GraphQLErrors: Error, Sendable {
 /// and there will be an error inside `errors` specifying the reason for the failure and the path of
 /// the failed field.
 public func graphql(
-    validationRules: [@Sendable (ValidationContext) -> Visitor] = [],
     schema: GraphQLSchema,
     request: String,
     rootValue: (any Sendable) = (),
     context: (any Sendable) = (),
     variableValues: [String: Map] = [:],
-    operationName: String? = nil
+    operationName: String? = nil,
+    validationRules: [@Sendable (ValidationContext) -> Visitor] = specifiedRules
 ) async throws -> GraphQLResult {
     let source = Source(body: request, name: "GraphQL request")
     let documentAST = try parse(source: source)
@@ -192,13 +192,13 @@ public func graphql<Retrieval: PersistedQueryRetrieval>(
 /// will be an error inside `errors` specifying the reason for the failure and the path of the
 /// failed field.
 public func graphqlSubscribe(
-    validationRules: [@Sendable (ValidationContext) -> Visitor] = [],
     schema: GraphQLSchema,
     request: String,
     rootValue: (any Sendable) = (),
     context: (any Sendable) = (),
     variableValues: [String: Map] = [:],
-    operationName: String? = nil
+    operationName: String? = nil,
+    validationRules: [@Sendable (ValidationContext) -> Visitor] = specifiedRules
 ) async throws -> Result<AsyncThrowingStream<GraphQLResult, Error>, GraphQLErrors> {
     let source = Source(body: request, name: "GraphQL Subscription request")
     let documentAST = try parse(source: source)

Sources/GraphQL/Validation/Validate.swift

@@ -1,20 +1,3 @@
-/// Implements the "Validation" section of the spec.
-///
-/// Validation runs synchronously, returning an array of encountered errors, or
-/// an empty array if no errors were encountered and the document is valid.
-///
-/// - Parameters:
-/// execution, and field resolution stages.
-///   - schema:          The GraphQL type system to use when validating and executing a query.
-///   - ast:             A GraphQL document representing the requested operation.
-/// - Returns: zero or more errors
-public func validate(
-    schema: GraphQLSchema,
-    ast: Document
-) -> [GraphQLError] {
-    return validate(schema: schema, ast: ast, rules: [])
-}
-
 /**
  * Implements the "Validation" section of the spec.
  *
@@ -31,7 +14,7 @@ public func validate(
 public func validate(
     schema: GraphQLSchema,
     ast: Document,
-    rules: [@Sendable (ValidationContext) -> Visitor]
+    rules: [@Sendable (ValidationContext) -> Visitor] = specifiedRules
 ) -> [GraphQLError] {
     let typeInfo = TypeInfo(schema: schema)
     let rules = rules.isEmpty ? specifiedRules : rules