RFC: __typename is not valid at subscription root (#776)

* __typename is not valid at subscription root * Update spec edits based on Lee's comments * Simplify * Clarify * Typo * Update Section 5 -- Validation.md Co-authored-by: Lee Byron <lee@leebyron.com>
graphql · Apr 15, 2021 · 413b689 · 413b689
1 parent 3958da9
commit 413b689
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/spec/Section 4 -- Introspection.md b/spec/Section 4 -- Introspection.md
@@ -101,6 +101,8 @@ identify which actual Object type of the possible types has been returned.
 As a meta-field, `__typename` is implicit and does not appear in the fields list
 in any defined type.
 
+Note: `__typename` may not be included as a root field in a subscription
+operation.
 
 ## Schema Introspection
 

diff --git a/spec/Section 5 -- Validation.md b/spec/Section 5 -- Validation.md
@@ -255,7 +255,8 @@ query getName {
 * Let {variableValues} be the empty set.
 * Let {groupedFieldSet} be the result of
   {CollectFields(subscriptionType, selectionSet, variableValues)}.
-* {groupedFieldSet} must have exactly one entry.
+* {groupedFieldSet} must have exactly one entry, which must not be an 
+  introspection field.
 
 **Explanatory Text**
 
@@ -311,14 +312,11 @@ fragment multipleSubscriptions on Subscription {
 }
 ```
 
-Introspection fields are counted. The following example is also invalid:
+The root field of a subscription operation must not be an introspection field.
+The following example is also invalid:
 
 ```graphql counter-example
 subscription sub {
-  newMessage {
-    body
-    sender
-  }
   __typename
 }
 ```