GraphQL: Fix formatting for directives in fields

changelog_unreleased/graphql/pr-9116.md

@@ -0,0 +1,28 @@
+#### Fix formatting for directives in fields (#9116 by @sosukesuzuki)
+
+<!-- prettier-ignore -->
+```graphql
+# Input
+type Query {
+  someQuery(id: ID!, someOtherData: String!): String! @deprecated @isAuthenticated
+  versions: Versions!
+}
+
+
+# Prettier stable
+type Query {
+  someQuery(id: ID!, someOtherData: String!): String!
+  @deprecated
+  @isAuthenticated
+  versions: Versions!
+}
+
+# Prettier master
+type Query {
+  someQuery(id: ID!, someOtherData: String!): String!
+    @deprecated
+    @isAuthenticated
+  versions: Versions!
+}
+
+```

src/language-graphql/printer-graphql.js

@@ -611,7 +611,24 @@ function printDirectives(path, print, n) {
     return "";
   }
 
-  return group(concat([line, join(line, path.map(print, "directives"))]));
+  if (n.kind === "FragmentDefinition" || n.kind === "OperationDefinition") {
+    return group(concat([line, join(line, path.map(print, "directives"))]));
+  }
+
+  return concat([
+    " ",
+    group(
+      indent(
+        concat([
+          softline,
+          join(
+            concat([ifBreak("", " "), softline]),
+            path.map(print, "directives")
+          ),
+        ])
+      )
+    ),
+  ]);
 }
 
 function printSequence(sequencePath, options, print) {

tests/graphql/directives/__snapshots__/jsfmt.spec.js.snap

@@ -32,7 +32,7 @@ query MyQuery @directive(arg: 5) {
   field @skip(if: true) @nope
   otherField
   ...fragmentSpread
-  @include(if: ["this isn't even a boolean", "wow, that's really odd"])
+    @include(if: ["this isn't even a boolean", "wow, that's really odd"])
 }
 
 fragment YouCanHaveDirectivesHereToo on SomeType @yesReally(what: "yes") {
@@ -45,10 +45,10 @@ fragment YouCanHaveDirectivesHereToo on SomeType @yesReally(what: "yes") {
   }
 
   thisFieldHasALotOfDirectives
-  @thisIsthefirst
-  @thisIsTheSecond
-  @thisIsTheThird
-  @thisIstheFourthWillBeTooLongForSure(and: "it has arguments as well")
+    @thisIsthefirst
+    @thisIsTheSecond
+    @thisIsTheThird
+    @thisIstheFourthWillBeTooLongForSure(and: "it has arguments as well")
 }
 
 query QueryWVars($x: String) @directive {

tests/graphql/newline/__snapshots__/jsfmt.spec.js.snap

@@ -63,16 +63,16 @@ query MyQuery
   arg2: 10
 ) {
   field
-  @skip(
-    if: true
+    @skip(
+      if: true
 
-    # comment
-    cursor: 10
-  )
-  @nope
+      # comment
+      cursor: 10
+    )
+    @nope
   otherField
   ...fragmentSpread
-  @include(if: ["this isn't even a boolean", "wow, that's really odd"])
+    @include(if: ["this isn't even a boolean", "wow, that's really odd"])
 }
 
 ================================================================================