Made the introspection capabilities check more robust. (#8957)

Author: michaelstaib

src/HotChocolate/Utilities/src/Utilities.Introspection/CapabilityInspector.cs

@@ -71,6 +71,7 @@ private async Task InspectArgumentDeprecationAsync()
 
         if (result.Data.ValueKind is JsonValueKind.Object
             && result.Data.TryGetProperty("__type", out var type)
+            && type.ValueKind is JsonValueKind.Object
             && type.TryGetProperty("fields", out var fields))
         {
             foreach (var field in fields.EnumerateArray())
@@ -129,9 +130,18 @@ private async Task InspectDirectiveTypeAsync()
         using var response = await _client.SendAsync(request, _cancellationToken).ConfigureAwait(false);
         using var result = await response.ReadAsResultAsync(_cancellationToken).ConfigureAwait(false);
 
-        if (result.Data.ValueKind is JsonValueKind.Object
-            && result.Data.TryGetProperty("__type", out var type)
-            && type.TryGetProperty("fields", out var fields))
+        if (result.Data.ValueKind is not JsonValueKind.Object
+            || !result.Data.TryGetProperty("__type", out var type)
+            || type.ValueKind is not JsonValueKind.Object
+            || !type.TryGetProperty("fields", out var fields))
+        {
+            // if we cannot detect features because __type does return null,
+            // we will assume `locations` exists but assume all other
+            // directive related features do not exist.
+            _features.HasDirectiveLocations = true;
+            _features.HasRepeatableDirectives = false;
+        }
+        else
         {
             var locations = false;
             var isRepeatable = false;
@@ -203,6 +213,7 @@ private async Task InspectDirectivesAsync()
 
         if (result.Data.ValueKind is JsonValueKind.Object
             && result.Data.TryGetProperty("__schema", out var schema)
+            && schema.ValueKind is JsonValueKind.Object
             && schema.TryGetProperty("directives", out var directives))
         {
             var defer = false;
@@ -278,6 +289,7 @@ private async Task InspectSchemaAsync()
 
         if (result.Data.ValueKind is JsonValueKind.Object
             && result.Data.TryGetProperty("__type", out var type)
+            && type.ValueKind is JsonValueKind.Object
             && type.TryGetProperty("fields", out var fields))
         {
             var description = false;

src/HotChocolate/Utilities/src/Utilities.Introspection/Queries/inspect_argument_deprecation.graphql

@@ -1,10 +1,10 @@
 query IntrospectionQuery {
-    __type(name: "__Field") {
-        fields {
-            name
-            args {
-                name
-            }
-        }
+  __type(name: "__Field") {
+    fields {
+      name
+      args {
+        name
+      }
     }
+  }
 }

src/HotChocolate/Utilities/src/Utilities.Introspection/Queries/inspect_directives.graphql

@@ -1,7 +1,7 @@
 query IntrospectionQuery {
-    __schema {
-        directives {
-            name
-        }
+  __schema {
+    directives {
+      name
     }
+  }
 }

src/HotChocolate/Utilities/src/Utilities.Introspection/Queries/inspect_schema.graphql

@@ -1,7 +1,7 @@
 query IntrospectionQuery {
-    __type(name: "__Schema") {
-        fields {
-            name
-        }
+  __type(name: "__Schema") {
+    fields {
+      name
     }
+  }
 }