Add attributes targetTypeQuery and targetTypeMutation on `@Provider

Author: Vincz

docs/annotations/annotations-reference.md

@@ -381,7 +381,13 @@ class SecretArea {
 ## @Mutation
 
 This annotation applies on methods for classes tagged with the `@Provider` annotation. It indicates that the method on this class will resolve a Mutation field.  
-The resulting field is added to the root Mutation type (defined in configuration at key `overblog_graphql.definitions.schema.mutation`).  
+The corresponding GraphQL field is added to the GraphQL type(s) following the logic :
+- The type(s) specified in the `targetType` attribute of the `@Mutation` annotation if it defined.  
+  or
+- The type(s) specified in the `targetTypeMutation` attribute of the `@Provider` annotation if it is defined.  
+  or
+- The root Query type of the default schema (define in configuration at key `overblog_graphql.definitions.schema.mutation` or `overblog_graphql.definitions.schema.default.mutation`).  
+  
 The class exposing the mutation(s) must be declared as a [service](https://symfony.com/doc/current/service_container.html).
 
 Optional attributes:
@@ -427,11 +433,19 @@ You can use `@Access` and/or `@IsPublic` on a provider class to add default acce
 Optional attributes:
 
 -   **prefix** : A prefix to apply to all field names from this provider
+-   **targetTypeQuery** : The default GraphQL type(s) to attach the provider `@Query` to
+-   **targetTypeMutation** : The default GraphQL type(s) to attach the provider `@Mutation` to
 
 ## @Query
 
 This annotation applies on methods for classes tagged with the `@Provider` annotation. It indicates that on this class a method will resolve a Query field.  
-By default, the resulting field is added to the root Query type (define in configuration at key `overblog_graphql.definitions.schema.query`).  
+The corresponding GraphQL field is added to the GraphQL type(s) following the logic :
+- The type(s) specified in the `targetType` attribute of the `@Query` annotation if it defined.  
+  or
+- The type(s) specified in the `targetTypeQuery` attribute of the `@Provider` annotation if it is defined.  
+  or
+- The root Query type of the default schema (define in configuration at key `overblog_graphql.definitions.schema.query` or `overblog_graphql.definitions.schema.default.query`).  
+  
 The class exposing the query(ies) must be declared as a [service](https://symfony.com/doc/current/service_container.html).
 
 Optional attributes:

src/Annotation/Provider.php

@@ -14,8 +14,20 @@ final class Provider implements Annotation
 {
     /**
      * Optionnal prefix for provider fields.
-     *
-     * @var string
      */
     public string $prefix;
+
+    /**
+     * The default target types to attach the provider queries to.
+     *
+     * @var array<string>
+     */
+    public array $targetTypeQuery;
+
+    /**
+     * The default target types to attach the provider mutations to.
+     *
+     * @var array<string>
+     */
+    public array $targetTypeMutation;
 }

src/Config/Parser/AnnotationParser.php

@@ -712,6 +712,15 @@ private static function getGraphQLFieldsFromProviders(GraphClass $graphClass, st
 
                 $annotationTargets = $annotation->targetType ?? null;
 
+                if (null === $annotationTargets) {
+                    if ($annotation instanceof GQL\Mutation && isset($providerAnnotation->targetTypeMutation)) {
+                        $annotationTargets = $providerAnnotation->targetTypeMutation;
+                    }
+                    if ($annotation instanceof GQL\Query && isset($providerAnnotation->targetTypeQuery)) {
+                        $annotationTargets = $providerAnnotation->targetTypeQuery;
+                    }
+                }
+
                 if (null === $annotationTargets) {
                     if ($isDefaultTarget) {
                         $annotationTargets = [$targetType];
@@ -728,7 +737,7 @@ private static function getGraphQLFieldsFromProviders(GraphClass $graphClass, st
                 }
 
                 if (!$annotation instanceof $expectedAnnotation) {
-                    if (GQL\Mutation::class == $expectedAnnotation) {
+                    if (GQL\Mutation::class === $expectedAnnotation) {
                         $message = sprintf('The provider "%s" try to add a query field on type "%s" (through @Query on method "%s") but "%s" is a mutation.', $providerMetadata->getName(), $targetType, $method->getName(), $targetType);
                     } else {
                         $message = sprintf('The provider "%s" try to add a mutation on type "%s" (through @Mutation on method "%s") but "%s" is not a mutation.', $providerMetadata->getName(), $targetType, $method->getName(), $targetType);

tests/Config/Parser/AnnotationParserTest.php

@@ -276,6 +276,10 @@ public function testProviders(): void
                     'access' => '@=default_access',
                     'public' => '@=default_public',
                 ],
+                'countSecretWeapons' => [
+                    'type' => 'Int!',
+                    'resolve' => "@=call(service('Overblog\\\\GraphQLBundle\\\\Tests\\\\Config\\\\Parser\\\\fixtures\\\\annotations\\\\Repository\\\\WeaponRepository').countSecretWeapons, arguments({}, args))",
+                ],
             ],
         ]);
 
@@ -316,6 +320,10 @@ public function testProvidersMultischema(): void
                     'access' => '@=default_access',
                     'public' => '@=default_public',
                 ],
+                'hasSecretWeapons' => [
+                    'type' => 'Boolean!',
+                    'resolve' => "@=call(service('Overblog\\\\GraphQLBundle\\\\Tests\\\\Config\\\\Parser\\\\fixtures\\\\annotations\\\\Repository\\\\WeaponRepository').hasSecretWeapons, arguments({}, args))",
+                ],
             ],
         ]);
 
@@ -335,6 +343,10 @@ public function testProvidersMultischema(): void
                     'access' => '@=default_access',
                     'public' => '@=default_public',
                 ],
+                'createLightsaber' => [
+                    'type' => 'Boolean!',
+                    'resolve' => "@=call(service('Overblog\\\\GraphQLBundle\\\\Tests\\\\Config\\\\Parser\\\\fixtures\\\\annotations\\\\Repository\\\\WeaponRepository').createLightsaber, arguments({}, args))",
+                ],
             ],
         ]);
     }

tests/Config/Parser/fixtures/annotations/Repository/WeaponRepository.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Overblog\GraphQLBundle\Tests\Config\Parser\fixtures\annotations\Repository;
+
+use Overblog\GraphQLBundle\Annotation as GQL;
+
+/**
+ * @GQL\Provider(targetTypeQuery={"RootQuery2"}, targetTypeMutation="RootMutation2")
+ */
+class WeaponRepository
+{
+    /**
+     * @GQL\Query
+     */
+    public function hasSecretWeapons(): bool
+    {
+        return true;
+    }
+
+    /**
+     * @GQL\Query(targetType="RootQuery")
+     */
+    public function countSecretWeapons(): int
+    {
+        return 2;
+    }
+
+    /**
+     * @GQL\Mutation
+     */
+    public function createLightsaber(): bool
+    {
+        return true;
+    }
+}