Fixed comments

Author: xperiandri

src/FSharp.Data.GraphQL.Server/Executor.fs

@@ -100,7 +100,7 @@ type Executor<'Root>(schema: ISchema<'Root>, middlewares : IExecutorMiddleware s
         match Validation.Types.validateTypeMap schema.TypeMap with
         | Success -> ()
         | ValidationError errors -> raise (GQLMessageException (System.String.Join("\n", errors)))
-    
+
     // Compute schema ID once after middleware has run and cache it for the lifetime of this Executor instance
     let schemaId = schema.Introspected.GetHashCode()
 
@@ -189,7 +189,7 @@ type Executor<'Root>(schema: ISchema<'Root>, middlewares : IExecutorMiddleware s
     /// Asynchronously executes a provided execution plan. In case of repetitive queries, execution plan may be preprocessed
     /// and cached using `documentId` as an identifier.
     /// Returned value is a readonly dictionary consisting of following top level entries:
-    /// 'documentId' (unique identifier of current document's AST, it can be used as a key/identifier of ExecutionPlan as well),
+    /// 'documentId' (unique identifier of the current document's AST, it can be used as a key/identifier of ExecutionPlan as well),
     /// 'data' (GraphQL response matching the structure provided in GraphQL query string), and
     /// 'errors' (optional, contains a list of errors that occurred while executing a GraphQL operation).
     /// </summary>
@@ -202,7 +202,7 @@ type Executor<'Root>(schema: ISchema<'Root>, middlewares : IExecutorMiddleware s
 
     /// <summary>
     /// Asynchronously executes parsed GraphQL query AST. Returned value is a readonly dictionary consisting of following top level entries:
-    /// 'documentId' (unique identifier of current document's AST, it can be used as a key/identifier of ExecutionPlan as well),
+    /// 'documentId' (unique identifier of the current document's AST, it can be used as a key/identifier of ExecutionPlan as well),
     /// 'data' (GraphQL response matching the structure provided in GraphQL query string), and
     /// 'errors' (optional, contains a list of errors that occurred while executing a GraphQL operation).
     /// </summary>
@@ -220,7 +220,7 @@ type Executor<'Root>(schema: ISchema<'Root>, middlewares : IExecutorMiddleware s
 
     /// <summary>
     /// Asynchronously executes unparsed GraphQL query AST. Returned value is a readonly dictionary consisting of following top level entries:
-    /// 'documentId' (unique identifier of current document's AST, it can be used as a key/identifier of ExecutionPlan as well),
+    /// 'documentId' (unique identifier of the current document's AST, it can be used as a key/identifier of ExecutionPlan as well),
     /// 'data' (GraphQL response matching the structure provided in GraphQL query string), and
     /// 'errors' (optional, contains a list of errors that occurred while executing a GraphQL operation).
     /// </summary>

src/FSharp.Data.GraphQL.Shared/AstExtensions.fs

@@ -108,7 +108,7 @@ type Document with
     member x.ToQueryString ([<Optional; DefaultParameterValue(QueryStringPrintingOptions.None)>] options : QueryStringPrintingOptions) =
         let sb = PaddedStringBuilder ()
         let escapeGraphQLString (s : string) =
-            let escaped = StringBuilder (s.Length + 2)
+            let escaped = StringBuilder (s.Length + s.Length / 4 + 2)
             escaped.Append ('"') |> ignore
             for c in s do
                 let appendStr =
@@ -328,7 +328,7 @@ type Document with
             | None -> failwithf "Can not get information about fragment \"%s\". Fragment spread definition was not found in the query." name
         let operations =
             this.Definitions
-            |> List.choose (function
+            |> List_choose (function
                 | FragmentDefinition _ -> None
                 | OperationDefinition def -> Some def)
             |> List.map (fun operation -> operation.Name, operation)

src/FSharp.Data.GraphQL.Shared/TypeSystem.fs

@@ -671,6 +671,7 @@ and PlanningContext = {
     RootDef : ObjectDef
     Document : Document
     Operation : OperationDefinition
+    /// Unique identifier of the current document's AST.
     DocumentId : string
     Metadata : Metadata
 }
@@ -887,7 +888,7 @@ and SchemaCompileContext = {
 /// A planning of an execution phase.
 /// It is used by the execution process to execute an operation.
 and ExecutionPlan = {
-    /// Unique identifier of the current execution plan.
+    /// Unique identifier of the current document's AST.
     DocumentId : string
     /// AST definition of current operation.
     Operation : OperationDefinition