Resolved built-in scalar type resolution in BaseTypeMapper::mapNameToType (#784)

webonyx/graphql-php v15.31.0 changed Schema::getType() to call the
typeLoader callback before checking built-in scalars. Since the
RootTypeMapper chain throws TypeNotFoundException for unrecognized
type names, built-in scalars like ID, String, Int, Float, and Boolean
were never reached, causing "Unknown type" errors when used in
outputType annotations (e.g. #[Field(outputType: 'ID')]).

Author: oojacoboo

src/Mappers/Root/BaseTypeMapper.php

@@ -107,7 +107,9 @@ public function toGraphQLInputType(Type $type, InputType|null $subType, string $
      *
      * @throws CannotMapTypeException
      */
-    private function mapBaseType(Type $type): BooleanType|FloatType|IDType|IntType|StringType|UploadType|DateTimeType|ScalarType|null
+    private function mapBaseType(
+        Type $type,
+    ): BooleanType|FloatType|IDType|IntType|StringType|UploadType|DateTimeType|ScalarType|null
     {
         if ($type instanceof Integer) {
             return GraphQLType::int();
@@ -180,11 +182,15 @@ private static function getDateTimeType(): DateTimeType
      */
     public function mapNameToType(string $typeName): NamedType&GraphQLType
     {
-        // No need to map base types, only types added by us.
         return match ($typeName) {
-            'Upload'=>self::getUploadType(),
-            'DateTime'=>self::getDateTimeType(),
-            default=>$this->next->mapNameToType($typeName)
+            'ID' => GraphQLType::id(),
+            'String' => GraphQLType::string(),
+            'Int' => GraphQLType::int(),
+            'Float' => GraphQLType::float(),
+            'Boolean' => GraphQLType::boolean(),
+            'Upload' => self::getUploadType(),
+            'DateTime' => self::getDateTimeType(),
+            default => $this->next->mapNameToType($typeName),
         };
     }
 }

tests/Mappers/Root/BaseTypeMapperTest.php

@@ -3,9 +3,11 @@
 namespace TheCodingMachine\GraphQLite\Mappers\Root;
 
 use GraphQL\Type\Definition\BooleanType;
+use GraphQL\Type\Definition\IDType;
 use GraphQL\Type\Definition\IntType;
 use GraphQL\Type\Definition\ListOfType;
 use GraphQL\Type\Definition\NonNull;
+use GraphQL\Type\Definition\StringType;
 use GraphQL\Type\Definition\WrappingType;
 use phpDocumentor\Reflection\DocBlock;
 use phpDocumentor\Reflection\Fqsen;
@@ -81,6 +83,34 @@ public function testOutputGenericIterables(string $phpdocType, string $expectedI
         }
     }
 
+    /**
+     * Built-in scalars (ID, String, Int, Float, Boolean) must be resolved by
+     * mapNameToType so that webonyx/graphql-php's Schema::getType() — which
+     * calls the typeLoader before checking builtInScalars — doesn't throw.
+     */
+    #[DataProvider('builtInScalarNamesProvider')]
+    public function testMapNameToTypeResolvesBuiltInScalars(string $scalarName, string $expectedType): void
+    {
+        $baseTypeMapper = new BaseTypeMapper(
+            new FinalRootTypeMapper($this->getTypeMapper()),
+            $this->getTypeMapper(),
+            $this->getRootTypeMapper(),
+        );
+
+        $result = $baseTypeMapper->mapNameToType($scalarName);
+
+        $this->assertInstanceOf($expectedType, $result);
+    }
+
+    public static function builtInScalarNamesProvider(): iterable
+    {
+        yield 'ID' => ['ID', IDType::class];
+        yield 'String' => ['String', StringType::class];
+        yield 'Int' => ['Int', IntType::class];
+        yield 'Float' => ['Float', \GraphQL\Type\Definition\FloatType::class];
+        yield 'Boolean' => ['Boolean', BooleanType::class];
+    }
+
     public static function genericIterablesProvider(): iterable
     {
         yield '\ArrayIterator with nullable int item' => ['\ArrayIterator<int|null>', IntType::class];