PHPStan level 7 (#1045)

Co-authored-by: Simon Podlipsky <[email protected]>

Author: spawnia

CHANGELOG.md

@@ -17,7 +17,7 @@ You can find and compare releases at the [GitHub release page](https://github.co
 - Use native PHP types for properties of `Type` and its subclasses
 - Throw `SerializationError` over client safe `Error` when failing to serialize leaf types
 - Move debug entries in errors under `extensions` key
-- Use native PHP types for `Schema` and `SchemaConfig`
+- Use native PHP types wherever possible
 - Always throw `RequestError` with useful message when clients provide an invalid JSON body
 - Move class `BlockString` from namespace `GraphQL\Utils` to `GraphQL\Language`
 - Return string-keyed arrays from `GraphQL::getStandardDirectives()`, `GraphQL::getStandardTypes()` and `GraphQL::getStandardValidationRules()`
@@ -32,6 +32,11 @@ You can find and compare releases at the [GitHub release page](https://github.co
 - Always convert recursively when calling `Node::toArray()`
 - Make `Directive::$config['args']` use the same definition style as `FieldDefinition::$config['args']`
 - Rename `FieldArgument` to `Argument`
+- Make errors when parsing scalar literals more precise
+- Change expected `QueryPlan` options from `['group-implementor-fields']` to `['groupImplementorFields' => true]` in `ResolveInfo::lookAhead()`
+- Always convert promises through `PromiseAdapter::convertThenable()` before calling `->then()` on them
+- Use `JSON_THROW_ON_ERROR` in `json_encode()`
+- Validate some internal invariants through `assert()`
 
 ### Added
 
@@ -97,6 +102,7 @@ You can find and compare releases at the [GitHub release page](https://github.co
 - Remove `ListOfType::$ofType`, `ListOfType::getOfType()` and `NonNull::getOfType()`
 - Remove option `commentDescriptions` from `BuildSchema::buildAST()`, `BuildSchema::build()` and `Printer::doPrint()`
 - Remove parameter `$options` from `ASTDefinitionBuilder`
+- Remove `FieldDefinition::create()` in favor of `new FieldDefinition()`
 
 ## 14.11.3
 

benchmarks/Utils/SchemaGenerator.php

@@ -89,7 +89,7 @@ protected function createType(int $nestingLevel, ?string $typeName = null): Obje
     }
 
     /**
-     * @return array{0: ObjectType, 1: string}
+     * @return array{0: Type, 1: string}
      */
     protected function getFieldTypeAndName(int $nestingLevel, int $fieldIndex): array
     {

composer.json

@@ -29,7 +29,8 @@
     "psr/http-message": "^1",
     "react/promise": "^2",
     "symfony/polyfill-php81": "^1.23",
-    "symfony/var-exporter": "^5.3"
+    "symfony/var-exporter": "^5.3",
+    "thecodingmachine/safe": "^1.3"
   },
   "suggest": {
     "psr/http-message": "To use standard GraphQL server",

docs/class-reference.md

@@ -2346,7 +2346,7 @@ static function toArray(GraphQL\Language\AST\Node $node): array
  * | null          | NullValue            |
  *
  * @param Type|mixed|null                                     $value
- * @param ScalarType|EnumType|InputObjectType|ListOfType<Type &InputType>|NonNull $type
+ * @param ScalarType|EnumType|InputObjectType|ListOfType<Type&InputType>|NonNull $type
  *
  * @return ObjectValueNode|ListValueNode|BooleanValueNode|IntValueNode|FloatValueNode|EnumValueNode|StringValueNode|NullValueNode|null
  *

examples/00-hello-world/graphql.php

@@ -58,6 +58,10 @@
     ]);
 
     $rawInput = file_get_contents('php://input');
+    if (false === $rawInput) {
+        throw new RuntimeException('Failed to get php://input');
+    }
+
     $input = json_decode($rawInput, true);
     $query = $input['query'];
     $variableValues = $input['variables'] ?? null;

examples/01-blog/Blog/Type/NodeType.php

@@ -5,13 +5,13 @@
 namespace GraphQL\Examples\Blog\Type;
 
 use Exception;
-use function get_class;
 use GraphQL\Examples\Blog\Data\Image;
 use GraphQL\Examples\Blog\Data\Story;
 use GraphQL\Examples\Blog\Data\User;
 use GraphQL\Examples\Blog\Types;
 use GraphQL\Type\Definition\InterfaceType;
-use GraphQL\Type\Definition\Type;
+use GraphQL\Type\Definition\ObjectType;
+use GraphQL\Utils\Utils;
 
 class NodeType extends InterfaceType
 {
@@ -26,7 +26,12 @@ public function __construct()
         ]);
     }
 
-    public function resolveNodeType(object $object): Type
+    /**
+     * @param mixed $object
+     *
+     * @return callable(): ObjectType
+     */
+    public function resolveNodeType($object)
     {
         if ($object instanceof User) {
             return Types::user();
@@ -40,6 +45,6 @@ public function resolveNodeType(object $object): Type
             return Types::story();
         }
 
-        throw new Exception('Unknown type: ' . get_class($object));
+        throw new Exception('Unknown type: ' . Utils::printSafe($object));
     }
 }

examples/01-blog/Blog/Type/Scalar/UrlType.php

@@ -57,6 +57,6 @@ public function parseLiteral(Node $valueNode, ?array $variables = null): string
     private function isUrl($value): bool
     {
         return is_string($value)
-            && filter_var($value, FILTER_VALIDATE_URL);
+            && false !== filter_var($value, FILTER_VALIDATE_URL);
     }
 }

examples/01-blog/Blog/Types.php

@@ -4,7 +4,6 @@
 
 namespace GraphQL\Examples\Blog;
 
-use function class_exists;
 use Closure;
 use function count;
 use Exception;
@@ -94,35 +93,29 @@ public static function url(): callable
     }
 
     /**
+     * @param class-string<Type> $classname
+     *
      * @return Closure(): Type
      */
     private static function get(string $classname): Closure
     {
         return static fn () => self::byClassName($classname);
     }
 
+    /**
+     * @param class-string<Type> $classname
+     */
     private static function byClassName(string $classname): Type
     {
         $parts = explode('\\', $classname);
 
         $cacheName = strtolower(preg_replace('~Type$~', '', $parts[count($parts) - 1]));
-        $type = null;
 
         if (! isset(self::$types[$cacheName])) {
-            if (class_exists($classname)) {
-                $type = new $classname();
-            }
-
-            self::$types[$cacheName] = $type;
+            return self::$types[$cacheName] = new $classname();
         }
 
-        $type = self::$types[$cacheName];
-
-        if (! $type) {
-            throw new Exception('Unknown graphql type: ' . $classname);
-        }
-
-        return $type;
+        return self::$types[$cacheName];
     }
 
     public static function byTypeName(string $shortName): Type

examples/02-schema-definition-language/graphql.php

@@ -33,6 +33,10 @@
     ];
 
     $rawInput = file_get_contents('php://input');
+    if (false === $rawInput) {
+        throw new RuntimeException('Failed to get php://input');
+    }
+
     $input = json_decode($rawInput, true);
     $query = $input['query'];
     $variableValues = $input['variables'] ?? null;

generate-class-reference.php

@@ -174,12 +174,9 @@ function renderProp(ReflectionProperty $prop): string
     return unpadDocblock($prop->getDocComment()) . "\n" . $signature;
 }
 
-/**
- * @param string|false $docBlock
- */
-function unwrapDocblock($docBlock): string
+function unwrapDocblock(string $docBlock): string
 {
-    if (! $docBlock) {
+    if ('' === $docBlock) {
         return '';
     }
 
@@ -196,7 +193,7 @@ function unwrapDocblock($docBlock): string
  */
 function unpadDocblock($docBlock): string
 {
-    if (! $docBlock) {
+    if (false === $docBlock) {
         return '';
     }
 
@@ -215,7 +212,7 @@ function unpadDocblock($docBlock): string
 function isApi(Reflector $reflector): bool
 {
     $comment = $reflector->getDocComment();
-    if (! $comment) {
+    if (false === $comment) {
         return false;
     }