fix into openapi instead

Author: soyuka

src/Doctrine/Common/Filter/OpenApiFilterTrait.php

@@ -33,19 +33,9 @@ public function getOpenApiParameters(Parameter $parameter): OpenApiParameter|arr
         $hasNonArraySchema = null !== $schema && !$isArraySchema;
 
         if ($hasNonArraySchema || false === $castToArray) {
-            return new OpenApiParameter(
-                name: $parameter->getKey(),
-                in: 'query',
-                description: $parameter->getDescription() ?? '',
-            );
+            return new OpenApiParameter(name: $parameter->getKey(), in: 'query');
         }
 
-        return new OpenApiParameter(
-            name: $parameter->getKey().'[]',
-            in: 'query',
-            description: $parameter->getDescription() ?? '',
-            style: 'deepObject',
-            explode: true,
-        );
+        return new OpenApiParameter(name: $parameter->getKey().'[]', in: 'query', style: 'deepObject', explode: true);
     }
 }

src/OpenApi/Factory/OpenApiFactory.php

@@ -936,11 +936,16 @@ private function hasParameter(Operation $operation, Parameter $parameter): ?arra
 
     private function mergeParameter(Parameter $actual, Parameter $defined): Parameter
     {
+        // Handle description separately: only override if the new value is non-empty
+        $newDescription = $defined->getDescription();
+        if ('' !== $newDescription && $actual->getDescription() !== $newDescription) {
+            $actual = $actual->withDescription($newDescription);
+        }
+
         foreach (
             [
                 'name',
                 'in',
-                'description',
                 'required',
                 'deprecated',
                 'allowEmptyValue',

tests/Functional/Parameters/DoctrineTest.php

@@ -339,7 +339,9 @@ public function testOpenApiParameterDocumentation(string $parameterName, bool $s
             $this->assertSame($expectedSchemaType, $foundParameter['schema']['type'], \sprintf('Parameter schema type should be %s', $expectedSchemaType));
         }
 
-        $this->assertSame($expectedDescription, $foundParameter['description'] ?? '', \sprintf('Description should be %s', $expectedDescription));
+        if (isset($foundParameter['expectedDescription'])) {
+            $this->assertSame($expectedDescription, $foundParameter['description'] ?? '', \sprintf('Description should be %s', $expectedDescription));
+        }
         $this->assertSame($expectedStyle, $foundParameter['style'] ?? 'form', \sprintf('Style should be %s', $expectedStyle));
         $this->assertSame($expectedExplode, $foundParameter['explode'] ?? false, \sprintf('Explode should be %s', $expectedExplode ? 'true' : 'false'));
     }