Parse comments in the front of definitions

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to
 ## Fixed
 - Location of a parse error is returned in a singleton array with key
   `locations`.
+- Parsing comments in the front of definitions.
 
 ## Added
 - `AST` reexports `AST.Parser`.

src/Language/GraphQL/AST/Parser.hs

@@ -6,7 +6,7 @@ module Language.GraphQL.AST.Parser
     ( document
     ) where
 
-import Control.Applicative (Alternative(..), optional)
+import Control.Applicative (Alternative(..), liftA2, optional)
 import Control.Applicative.Combinators (sepBy1)
 import qualified Control.Applicative.Combinators.NonEmpty as NonEmpty
 import Data.List.NonEmpty (NonEmpty(..))
@@ -24,8 +24,8 @@ import Text.Megaparsec (lookAhead, option, try, (<?>))
 -- | Parser for the GraphQL documents.
 document :: Parser Document
 document = unicodeBOM
-    >> spaceConsumer
+    *> spaceConsumer
-    >> lexeme (NonEmpty.some definition)
+    *> lexeme (NonEmpty.some definition)
 
 definition :: Parser Definition
 definition = ExecutableDefinition <$> executableDefinition
@@ -40,19 +40,22 @@ executableDefinition = DefinitionOperation <$> operationDefinition
 
 typeSystemDefinition :: Parser TypeSystemDefinition
 typeSystemDefinition = schemaDefinition
-    <|> TypeDefinition <$> typeDefinition
+    <|> typeSystemDefinitionWithDescription
-    <|> directiveDefinition
     <?> "TypeSystemDefinition"
+  where
+    typeSystemDefinitionWithDescription = description
+        >>= liftA2 (<|>) typeDefinition' directiveDefinition
+    typeDefinition' description' = TypeDefinition
+        <$> typeDefinition description'
 
 typeSystemExtension :: Parser TypeSystemExtension
 typeSystemExtension = SchemaExtension <$> schemaExtension
     <|> TypeExtension <$> typeExtension
     <?> "TypeSystemExtension"
 
-directiveDefinition :: Parser TypeSystemDefinition
+directiveDefinition :: Description -> Parser TypeSystemDefinition
-directiveDefinition = DirectiveDefinition
+directiveDefinition description' = DirectiveDefinition description'
-    <$> description
+    <$ symbol "directive"
-    <* symbol "directive"
     <* at
     <*> name
     <*> argumentsDefinition
@@ -91,13 +94,13 @@ typeSystemDirectiveLocation = Directive.Schema <$ symbol "SCHEMA"
     <|> Directive.InputObject <$ symbol "INPUT_OBJECT"
     <|> Directive.InputFieldDefinition <$ symbol "INPUT_FIELD_DEFINITION"
 
-typeDefinition :: Parser TypeDefinition
+typeDefinition :: Description -> Parser TypeDefinition
-typeDefinition = scalarTypeDefinition
+typeDefinition description' = scalarTypeDefinition description'
-    <|> objectTypeDefinition
+    <|> objectTypeDefinition description'
-    <|> interfaceTypeDefinition
+    <|> interfaceTypeDefinition description'
-    <|> unionTypeDefinition
+    <|> unionTypeDefinition description'
-    <|> enumTypeDefinition
+    <|> enumTypeDefinition description'
-    <|> inputObjectTypeDefinition
+    <|> inputObjectTypeDefinition description'
     <?> "TypeDefinition"
 
 typeExtension :: Parser TypeExtension
@@ -109,10 +112,9 @@ typeExtension = scalarTypeExtension
     <|> inputObjectTypeExtension
     <?> "TypeExtension"
 
-scalarTypeDefinition :: Parser TypeDefinition
+scalarTypeDefinition :: Description -> Parser TypeDefinition
-scalarTypeDefinition = ScalarTypeDefinition
+scalarTypeDefinition description' = ScalarTypeDefinition description'
-    <$> description
+    <$ symbol "scalar"
-    <* symbol "scalar"
     <*> name
     <*> directives
     <?> "ScalarTypeDefinition"
@@ -121,10 +123,9 @@ scalarTypeExtension :: Parser TypeExtension
 scalarTypeExtension = extend "scalar" "ScalarTypeExtension"
     $ (ScalarTypeExtension <$> name <*> NonEmpty.some directive) :| []
 
-objectTypeDefinition :: Parser TypeDefinition
+objectTypeDefinition :: Description -> Parser TypeDefinition
-objectTypeDefinition = ObjectTypeDefinition
+objectTypeDefinition description' = ObjectTypeDefinition description'
-    <$> description
+    <$ symbol "type"
-    <* symbol "type"
     <*> name
     <*> option (ImplementsInterfaces []) (implementsInterfaces sepBy1)
     <*> directives
@@ -153,13 +154,12 @@ objectTypeExtension = extend "type" "ObjectTypeExtension"
 
 description :: Parser Description
 description = Description
-    <$> optional (string <|> blockString)
+    <$> optional (blockString <|> string)
     <?> "Description"
 
-unionTypeDefinition :: Parser TypeDefinition
+unionTypeDefinition :: Description -> Parser TypeDefinition
-unionTypeDefinition = UnionTypeDefinition
+unionTypeDefinition description' = UnionTypeDefinition description'
-    <$> description
+    <$ symbol "union"
-    <* symbol "union"
     <*> name
     <*> directives
     <*> option (UnionMemberTypes []) (unionMemberTypes sepBy1)
@@ -187,10 +187,9 @@ unionMemberTypes sepBy' = UnionMemberTypes
     <*> name `sepBy'` pipe
     <?> "UnionMemberTypes"
 
-interfaceTypeDefinition :: Parser TypeDefinition
+interfaceTypeDefinition :: Description -> Parser TypeDefinition
-interfaceTypeDefinition = InterfaceTypeDefinition
+interfaceTypeDefinition description' = InterfaceTypeDefinition description'
-    <$> description
+    <$ symbol "interface"
-    <* symbol "interface"
     <*> name
     <*> directives
     <*> braces (many fieldDefinition)
@@ -208,10 +207,9 @@ interfaceTypeExtension = extend "interface" "InterfaceTypeExtension"
         <$> name
         <*> NonEmpty.some directive
 
-enumTypeDefinition :: Parser TypeDefinition
+enumTypeDefinition :: Description -> Parser TypeDefinition
-enumTypeDefinition = EnumTypeDefinition
+enumTypeDefinition description' = EnumTypeDefinition description'
-    <$> description
+    <$ symbol "enum"
-    <* symbol "enum"
     <*> name
     <*> directives
     <*> listOptIn braces enumValueDefinition
@@ -229,10 +227,9 @@ enumTypeExtension = extend "enum" "EnumTypeExtension"
         <$> name
         <*> NonEmpty.some directive
 
-inputObjectTypeDefinition :: Parser TypeDefinition
+inputObjectTypeDefinition :: Description -> Parser TypeDefinition
-inputObjectTypeDefinition = InputObjectTypeDefinition
+inputObjectTypeDefinition description' = InputObjectTypeDefinition description'
-    <$> description
+    <$ symbol "input"
-    <* symbol "input"
     <*> name
     <*> directives
     <*> listOptIn braces inputValueDefinition

tests/Language/GraphQL/AST/ParserSpec.hs

@@ -149,3 +149,13 @@ spec = describe "Parser" $ do
               title
             }
         |]
+
+    it "parses documents beginning with a comment" $
+        parse document "" `shouldSucceedOn` [r|
+            """
+            Query
+            """
+            type Query {
+                queryField: String
+            }
+        |]