Make AST constructors parametrized

.github/labeler.yml

@@ -1,6 +1,6 @@
 test:
-  - src/GraphQLParser.ApiTests/**/*
   - src/GraphQLParser.Tests/**/*
+  - src/GraphQLParser.ApiTests/**/*.cs
 
 CI:
   - .github/workflows/**/*

src/GraphQLParser/AST/ASTListNode.cs

@@ -7,10 +7,23 @@ namespace GraphQLParser.AST;
 /// </summary>
 public abstract class ASTListNode<TNode> : ASTNode, IReadOnlyList<TNode>
 {
+    internal ASTListNode()
+    {
+        Items = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="ASTListNode{TNode}"/>.
+    /// </summary>
+    protected ASTListNode(List<TNode> items)
+    {
+        Items = items;
+    }
+
     /// <summary>
     /// A list of nested AST nodes.
     /// </summary>
-    public List<TNode> Items { get; set; } = null!;
+    public List<TNode> Items { get; set; }
 
     /// <summary>
     /// Get the number of AST nodes in the list.

src/GraphQLParser/AST/Definitions/GraphQLArgumentsDefinition.cs

@@ -5,6 +5,18 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLArgumentsDefinition : ASTListNode<GraphQLInputValueDefinition>
 {
+    internal GraphQLArgumentsDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLArgumentsDefinition"/>.
+    /// </summary>
+    public GraphQLArgumentsDefinition(List<GraphQLInputValueDefinition> items)
+        : base(items)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.ArgumentsDefinition;
 }

src/GraphQLParser/AST/Definitions/GraphQLDirectiveDefinition.cs

@@ -8,6 +8,20 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLDirectiveDefinition: {Name}")]
 public class GraphQLDirectiveDefinition : GraphQLTypeDefinition, IHasArgumentsDefinitionNode
 {
+    internal GraphQLDirectiveDefinition()
+    {
+        Locations = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLDirectiveDefinition"/>.
+    /// </summary>
+    public GraphQLDirectiveDefinition(GraphQLName name, GraphQLDirectiveLocations locations)
+        : base(name)
+    {
+        Locations = locations;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.DirectiveDefinition;
 
@@ -29,7 +43,7 @@ public class GraphQLDirectiveDefinition : GraphQLTypeDefinition, IHasArgumentsDe
     /// <summary>
     /// Returns a node with a list of locations representing the valid locations this directive may be placed.
     /// </summary>
-    public GraphQLDirectiveLocations Locations { get; set; } = null!;
+    public GraphQLDirectiveLocations Locations { get; set; }
 }
 
 internal sealed class GraphQLDirectiveDefinitionWithLocation : GraphQLDirectiveDefinition

src/GraphQLParser/AST/Definitions/GraphQLEnumTypeDefinition.cs

@@ -8,6 +8,18 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLEnumTypeDefinition: {Name}")]
 public class GraphQLEnumTypeDefinition : GraphQLTypeDefinition, IHasDirectivesNode
 {
+    internal GraphQLEnumTypeDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLEnumTypeDefinition"/>.
+    /// </summary>
+    public GraphQLEnumTypeDefinition(GraphQLName name)
+        : base(name)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.EnumTypeDefinition;
 

src/GraphQLParser/AST/Definitions/GraphQLEnumValueDefinition.cs

@@ -8,6 +8,20 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLEnumValueDefinition: {EnumValue}")]
 public class GraphQLEnumValueDefinition : GraphQLTypeDefinition, IHasDirectivesNode
 {
+    internal GraphQLEnumValueDefinition()
+    {
+        EnumValue = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLEnumValueDefinition"/>.
+    /// </summary>
+    public GraphQLEnumValueDefinition(GraphQLName name, GraphQLEnumValue enumValue)
+        : base(name)
+    {
+        EnumValue = enumValue;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.EnumValueDefinition;
 
@@ -16,7 +30,7 @@ public class GraphQLEnumValueDefinition : GraphQLTypeDefinition, IHasDirectivesN
     /// <see cref="GraphQLTypeDefinition.Name"/> property holds almost
     /// the same data and should be set as well.
     /// </summary>
-    public GraphQLEnumValue EnumValue { get; set; } = null!;
+    public GraphQLEnumValue EnumValue { get; set; }
 
     /// <inheritdoc/>
     public GraphQLDirectives? Directives { get; set; }

src/GraphQLParser/AST/Definitions/GraphQLEnumValuesDefinition.cs

@@ -5,6 +5,18 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLEnumValuesDefinition : ASTListNode<GraphQLEnumValueDefinition>
 {
+    internal GraphQLEnumValuesDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLEnumValuesDefinition"/>.
+    /// </summary>
+    public GraphQLEnumValuesDefinition(List<GraphQLEnumValueDefinition> items)
+        : base(items)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.EnumValuesDefinition;
 }

src/GraphQLParser/AST/Definitions/GraphQLExecutableDefinition.cs

@@ -7,11 +7,24 @@ namespace GraphQLParser.AST;
 /// </summary>
 public abstract class GraphQLExecutableDefinition : ASTNode, IHasSelectionSetNode, IHasDirectivesNode
 {
+    internal GraphQLExecutableDefinition()
+    {
+        SelectionSet = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLExecutableDefinition"/>.
+    /// </summary>
+    protected GraphQLExecutableDefinition(GraphQLSelectionSet selectionSet)
+    {
+        SelectionSet = selectionSet;
+    }
+
     /// <inheritdoc/>
     public GraphQLDirectives? Directives { get; set; }
 
     /// <inheritdoc/>
 #pragma warning disable CS8767 // Nullability of reference types in return type doesn't match implicitly implemented member (possibly because of nullability attributes).
-    public GraphQLSelectionSet SelectionSet { get; set; } = null!;
+    public GraphQLSelectionSet SelectionSet { get; set; }
 #pragma warning restore CS8767
 }

src/GraphQLParser/AST/Definitions/GraphQLFieldDefinition.cs

@@ -8,6 +8,20 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLFieldDefinition: {Name}")]
 public class GraphQLFieldDefinition : GraphQLTypeDefinition, IHasDirectivesNode, IHasArgumentsDefinitionNode
 {
+    internal GraphQLFieldDefinition()
+    {
+        Type = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLFieldDefinition"/>.
+    /// </summary>
+    public GraphQLFieldDefinition(GraphQLName name, GraphQLType type)
+        : base(name)
+    {
+        Type = type;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.FieldDefinition;
 
@@ -19,7 +33,7 @@ public class GraphQLFieldDefinition : GraphQLTypeDefinition, IHasDirectivesNode,
     /// <summary>
     /// Nested <see cref="GraphQLType"/> AST node with field type.
     /// </summary>
-    public GraphQLType Type { get; set; } = null!;
+    public GraphQLType Type { get; set; }
 
     /// <inheritdoc/>
     public GraphQLDirectives? Directives { get; set; }

src/GraphQLParser/AST/Definitions/GraphQLFieldsDefinition.cs

@@ -5,6 +5,18 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLFieldsDefinition : ASTListNode<GraphQLFieldDefinition>
 {
+    internal GraphQLFieldsDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLFieldsDefinition"/>.
+    /// </summary>
+    public GraphQLFieldsDefinition(List<GraphQLFieldDefinition> items)
+        : base(items)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.FieldsDefinition;
 }

src/GraphQLParser/AST/Definitions/GraphQLFragmentDefinition.cs

@@ -8,18 +8,34 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLFragmentDefinition: {FragmentName.Name.StringValue}")]
 public class GraphQLFragmentDefinition : GraphQLExecutableDefinition
 {
+    internal GraphQLFragmentDefinition()
+    {
+        FragmentName = null!;
+        TypeCondition = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLFragmentDefinition"/>.
+    /// </summary>
+    public GraphQLFragmentDefinition(GraphQLFragmentName name, GraphQLTypeCondition typeCondition, GraphQLSelectionSet selectionSet)
+        : base(selectionSet)
+    {
+        FragmentName = name;
+        TypeCondition = typeCondition;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.FragmentDefinition;
 
     /// <summary>
     /// Fragment name represented as a nested AST node.
     /// </summary>
-    public GraphQLFragmentName FragmentName { get; set; } = null!;
+    public GraphQLFragmentName FragmentName { get; set; }
 
     /// <summary>
     /// Nested <see cref="GraphQLTypeCondition"/> AST node with type condition of this fragment.
     /// </summary>
-    public GraphQLTypeCondition TypeCondition { get; set; } = null!;
+    public GraphQLTypeCondition TypeCondition { get; set; }
 }
 
 internal sealed class GraphQLFragmentDefinitionWithLocation : GraphQLFragmentDefinition

src/GraphQLParser/AST/Definitions/GraphQLInputFieldsDefinition.cs

@@ -5,6 +5,18 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLInputFieldsDefinition : ASTListNode<GraphQLInputValueDefinition>
 {
+    internal GraphQLInputFieldsDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLInputFieldsDefinition"/>.
+    /// </summary>
+    public GraphQLInputFieldsDefinition(List<GraphQLInputValueDefinition> items)
+        : base(items)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.InputFieldsDefinition;
 }

src/GraphQLParser/AST/Definitions/GraphQLInputObjectTypeDefinition.cs

@@ -8,6 +8,18 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLInputObjectTypeDefinition: {Name}")]
 public class GraphQLInputObjectTypeDefinition : GraphQLTypeDefinition, IHasDirectivesNode
 {
+    internal GraphQLInputObjectTypeDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLInputObjectTypeDefinition"/>.
+    /// </summary>
+    public GraphQLInputObjectTypeDefinition(GraphQLName name)
+        : base(name)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.InputObjectTypeDefinition;
 

src/GraphQLParser/AST/Definitions/GraphQLInputValueDefinition.cs

@@ -5,13 +5,27 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLInputValueDefinition : GraphQLTypeDefinition, IHasDirectivesNode, IHasDefaultValueNode
 {
+    internal GraphQLInputValueDefinition()
+    {
+        Type = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLInputValueDefinition"/>.
+    /// </summary>
+    public GraphQLInputValueDefinition(GraphQLName name, GraphQLType type)
+        : base(name)
+    {
+        Type = type;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.InputValueDefinition;
 
     /// <summary>
     /// Nested <see cref="GraphQLType"/> AST node with input value type.
     /// </summary>
-    public GraphQLType Type { get; set; } = null!;
+    public GraphQLType Type { get; set; }
 
     /// <inheritdoc />
     public GraphQLValue? DefaultValue { get; set; }

src/GraphQLParser/AST/Definitions/GraphQLInterfaceTypeDefinition.cs

@@ -8,6 +8,18 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLInterfaceTypeDefinition: {Name}")]
 public class GraphQLInterfaceTypeDefinition : GraphQLTypeDefinition, IHasDirectivesNode, IHasInterfacesNode, IHasFieldsDefinitionNode
 {
+    internal GraphQLInterfaceTypeDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLInterfaceTypeDefinition"/>.
+    /// </summary>
+    public GraphQLInterfaceTypeDefinition(GraphQLName name)
+        : base(name)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.InterfaceTypeDefinition;
 

src/GraphQLParser/AST/Definitions/GraphQLObjectTypeDefinition.cs

@@ -8,6 +8,18 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLObjectTypeDefinition: {Name}")]
 public class GraphQLObjectTypeDefinition : GraphQLTypeDefinition, IHasDirectivesNode, IHasInterfacesNode, IHasFieldsDefinitionNode
 {
+    internal GraphQLObjectTypeDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLObjectTypeDefinition"/>.
+    /// </summary>
+    public GraphQLObjectTypeDefinition(GraphQLName name)
+        : base(name)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.ObjectTypeDefinition;
 

src/GraphQLParser/AST/Definitions/GraphQLOperationDefinition.cs

@@ -8,6 +8,18 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLOperationDefinition: {Operation}")]
 public class GraphQLOperationDefinition : GraphQLExecutableDefinition, INamedNode
 {
+    internal GraphQLOperationDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLOperationDefinition"/>.
+    /// </summary>
+    public GraphQLOperationDefinition(GraphQLSelectionSet selectionSet)
+        : base(selectionSet)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.OperationDefinition;