Enhance API response handling by inferring status codes for Post and Delete routes

Author: yceruto

src/Controller/Decorator/ApiResponseDecorator.php

@@ -5,6 +5,8 @@
 namespace OpenSolid\Api\Controller\Decorator;
 
 use OpenSolid\Api\Controller\Model\ResponseOptions;
+use OpenSolid\Api\Routing\Attribute\Delete;
+use OpenSolid\Api\Routing\Attribute\Post;
 use OpenSolid\CallableInvoker\CallableMetadata;
 use OpenSolid\CallableInvoker\Decorator\CallableClosure;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -39,10 +41,15 @@ public function decorate(CallableClosure $callable, CallableMetadata $metadata):
         $request = $metadata->context['request'];
         /** @var Type $returnType */
         $type ??= $metadata->getAttribute('return_type');
+        $statusCode ??= $request->attributes->getInt('_api_status_code', match ($request->attributes->getInt('_api_route_class')) {
+            Post::class => 201,
+            Delete::class => 204,
+            default => 200,
+        });
 
         return new StreamedResponse(
             callbackOrChunks: $this->jsonStreamWriter->write($response, $type),
-            status: $statusCode ?? $request->attributes->getInt('_api_status_code', 200),
+            status: $statusCode,
             headers: $headers ?? ['Content-Type' => 'application/json'],
         );
     }

src/Routing/Attribute/ApiRoute.php

@@ -28,17 +28,14 @@ public function __construct(
         public array $tags = [],
         public bool $deprecated = false,
     ) {
-        if (null === $statusCode && $this instanceof Post) {
-            $statusCode = 201;
-        }
-
         parent::__construct(
             path: $path,
             name: $name,
             requirements: $requirements,
             options: $options,
             defaults: $defaults + array_filter([
                 '_api_controller' => true,
+                '_api_route_class' => $this::class,
                 '_api_status_code' => $statusCode,
             ], fn ($v) => null !== $v),
             host: $host,