Merge 4.2

Author: soyuka

AGENTS.md

@@ -147,6 +147,7 @@
         "jangregor/phpstan-prophecy": "^2.1.11",
         "justinrainbow/json-schema": "^6.5.2",
         "mcp/sdk": "^0.3.0",
+        "laravel/framework": "^11.0 || ^12.0",
         "orchestra/testbench": "^10.9",
         "phpspec/prophecy-phpunit": "^2.2",
         "phpstan/extension-installer": "^1.1",

composer.json

@@ -0,0 +1,18 @@
+# Context: API Platform Documentation
+
+## Documentation Standards
+
+* **Tone:** Technical, concise, and helpful. Avoid overly marketing language.  
+* **Format:** Markdown (GitHub flavored).  
+* **Links:** Use relative links for internal navigation.
+
+## **Directory Structure**
+
+* guides/: Learning oriented guides (How-to).  
+* adr/: Architectural Decision Records.  
+* src/: API Reference generation source.
+
+## **Contribution**
+
+* Documentation changes for new features are **mandatory**.  
+* Verify links before finalizing content.

docs/AGENTS.md

@@ -81,11 +81,6 @@ final class SchemaFactory implements SchemaFactoryInterface, SchemaFactoryAwareI
         'required' => ['@type'],
     ];
 
-    /**
-     * @var array<string, true>
-     */
-    private array $transformed = [];
-
     /**
      * @param array<string, mixed> $defaultContext
      */
@@ -284,7 +279,7 @@ public function setSchemaFactory(SchemaFactoryInterface $schemaFactory): void
 
     private function decorateItemDefinition(string $definitionName, \ArrayObject $definitions, string $prefix, string $type, ?array $serializerContext): void
     {
-        if (!isset($definitions[$definitionName]) || ($this->transformed[$definitionName] ?? false)) {
+        if (!isset($definitions[$definitionName])) {
             return;
         }
 
@@ -294,6 +289,12 @@ private function decorateItemDefinition(string $definitionName, \ArrayObject $de
             $baseName = self::ITEM_WITHOUT_ID_BASE_SCHEMA_NAME;
         }
 
+        // Check if the definition is already decorated to prevent double wrapping (recursion or cache)
+        $currentDef = $definitions[$definitionName];
+        if (($currentDef['allOf'][0]['$ref'] ?? null) === $prefix.$baseName) {
+            return;
+        }
+
         if (!isset($definitions[$baseName])) {
             $definitions[$baseName] = $hasNoId ? self::ITEM_BASE_SCHEMA_WITHOUT_ID : self::ITEM_BASE_SCHEMA_WITH_ID;
         }
@@ -309,7 +310,5 @@ private function decorateItemDefinition(string $definitionName, \ArrayObject $de
 
         $definitions[$definitionName] = $allOf;
         unset($definitions[$definitionName]['allOf'][1]['description']);
-
-        $this->transformed[$definitionName] = true;
     }
 }

src/Hydra/JsonSchema/SchemaFactory.php

@@ -0,0 +1,37 @@
+# Context: API Platform (Laravel Bridge)
+
+## Laravel-Specific Protocols
+
+You are currently working within the src/Laravel component. This environment differs from the main Symfony-based core.
+
+### 1. Setup & Environment
+
+* Execution Scope: Run all commands from the src/Laravel directory.
+* Dependency Linking: You MUST link the root dependencies before running tests here.  
+* Fixtures live in `workbench/app/ApiResource` or `workbench/app/Models`, `workbench` is a classic laravel project.
+
+```
+composer link ../../
+vendor/bin/testbench workbench:build
+```
+
+If you need a fresh fixtures start:
+
+```
+vendor/bin/testbench workbench:drop-sqlite-db
+```
+
+### 2. Testing Workflow
+
+**Running Tests:**  
+
+```
+vendor/bin/testbench package:test Tests/NoOperationResourceTest.php
+```
+
+**Static Analysis (Laravel):**  
+
+```
+./vendor/bin/phpstan analyse
+```
+

src/Laravel/AGENTS.md

@@ -0,0 +1,41 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <dunglas@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Laravel\Tests;
+
+use Illuminate\Contracts\Config\Repository;
+use Illuminate\Foundation\Application;
+use Orchestra\Testbench\Concerns\WithWorkbench;
+use Orchestra\Testbench\TestCase;
+
+class NoOperationResourceTest extends TestCase
+{
+    use WithWorkbench;
+
+    /**
+     * @param Application $app
+     */
+    protected function defineEnvironment($app): void
+    {
+        tap($app['config'], static function (Repository $config): void {
+            $config->set('api-platform.resources', [app_path('ApiResource')]);
+        });
+    }
+
+    public function testNoRoutesAreGeneratedForNoOperationResource(): void
+    {
+        $response = $this->get('/api/no_operations/notfound', headers: ['accept' => 'application/ld+json']);
+        $response->assertNotFound();
+        $response->assertJson(['detail' => 'This route does not aim to be called.']);
+    }
+}

src/Laravel/Tests/NoOperationResourceTest.php

@@ -51,6 +51,10 @@
 
                 if (!class_exists($controller)) {
                     $controller = $app->make($controller);
+
+                    if (is_callable([$controller, '__invoke'])) {
+                        $controller = '\\'.$controller::class.'@__invoke'; // see Illuminate\Routing\RouteAction
+                    }
                 }
 
                 $route = Route::addRoute($operation->getMethod(), $uriTemplate, ['uses' => $controller, 'prefix' => $operation->getRoutePrefix() ?? ''])

src/Laravel/routes/api.php

@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <dunglas@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace Workbench\App\ApiResource;
+
+use ApiPlatform\Metadata\ApiResource;
+
+#[ApiResource(operations: [])]
+class NoOperation
+{
+    public int $id = 1;
+}

src/Laravel/workbench/app/ApiResource/NoOperation.php

@@ -45,6 +45,7 @@
 use ApiPlatform\Symfony\Routing\SkolemIriConverter;
 use Negotiation\Negotiator;
 use Symfony\Component\Serializer\Mapping\Factory\CacheClassMetadataFactory;
+use Symfony\Component\Serializer\Normalizer\ObjectNormalizer;
 
 return function (ContainerConfigurator $container) {
     $services = $container->services();
@@ -131,6 +132,19 @@
         ])
         ->tag('serializer.normalizer', ['priority' => -895]);
 
+    $services->set('api_platform.normalizer.object', ObjectNormalizer::class)
+        ->args([
+            service('serializer.mapping.class_metadata_factory'),
+            service('api_platform.name_converter')->ignoreOnInvalid(),
+            service('serializer.property_accessor'),
+            service('property_info')->ignoreOnInvalid(),
+            service('serializer.mapping.class_discriminator_resolver')->ignoreOnInvalid(),
+            null,
+            [],
+            service('property_info')->ignoreOnInvalid(),
+        ])
+        ->tag('serializer.normalizer', ['built_in' => true, 'priority' => -1000]);
+
     $services->set('api_platform.serializer.mapping.class_metadata_factory', ClassMetadataFactory::class)
         ->decorate('serializer.mapping.class_metadata_factory', null, -1)
         ->args([service('api_platform.serializer.mapping.class_metadata_factory.inner')]);

src/Symfony/Bundle/Resources/config/api.php

@@ -252,7 +252,7 @@
 
     $services->set('api_platform.graphql.normalizer.object', ObjectNormalizer::class)
         ->args([
-            service('serializer.normalizer.object'),
+            service('api_platform.normalizer.object'),
             service('api_platform.symfony.iri_converter'),
             service('api_platform.api.identifiers_extractor'),
         ])