Fix styling

Author: klapaudius

src/Command/TestMcpToolCommand.php

@@ -30,7 +30,7 @@ public function __construct(private readonly ContainerInterface $container)
     public function displayResult(mixed $result, array $sentNotifications = [], ?StreamableToolInterface $tool = null): void
     {
         // Display progress notifications if any were sent
-        if (!empty($sentNotifications)) {
+        if (! empty($sentNotifications)) {
             $this->io->newLine();
             $this->io->section('Progress Notifications');
             $this->io->text(sprintf('Total notifications sent: %d', count($sentNotifications)));

src/Controllers/SseController.php

@@ -13,6 +13,7 @@ public function __construct(private MCPServerInterface $server) {}
     public function handle(): StreamedResponse
     {
         $this->server->setProtocolVersion(MCPProtocolInterface::PROTOCOL_VERSION_SSE);
+
         return new StreamedResponse(fn () => $this->server->connect(), headers: [
             'Content-Type' => 'text/event-stream',
             'Cache-Control' => 'no-cache, private',

src/Controllers/StreamableHttpController.php

@@ -31,7 +31,7 @@ public function gethandle(): JsonResponse
         return new JsonResponse(
             [
                 'jsonrpc' => 2.0,
-                'error' => 'This endpoint does not support GET requests yet.'
+                'error' => 'This endpoint does not support GET requests yet.',
             ],
             Response::HTTP_METHOD_NOT_ALLOWED);
     }
@@ -48,7 +48,7 @@ public function postHandle(Request $request): StreamedResponse|JsonResponse
             $input = json_decode($request->getContent(), true, flags: JSON_THROW_ON_ERROR);
             $this->logger?->debug('Received message from clientId:'.$clientId, ['message' => $input]);
 
-            $willStream = !isset($input['jsonrpc']) && isset($input[0]['jsonrpc']);
+            $willStream = ! isset($input['jsonrpc']) && isset($input[0]['jsonrpc']);
             $messages = $willStream ? $input : [$input];
 
             return new StreamedResponse(function () use ($clientId, $messages) {
@@ -62,6 +62,7 @@ public function postHandle(Request $request): StreamedResponse|JsonResponse
             ]);
         } catch (JsonException|StreamableHttpTransportException $e) {
             $message = $e instanceof JsonException ? 'Parse error' : $e->getMessage();
+
             return new JsonResponse(['jsonrpc' => 2.0, 'error' => ['code' => -32700, 'message' => $message]], 400);
         }
     }

src/DependencyInjection/CompilerPass/ConditionalRoutePass.php

@@ -6,7 +6,6 @@
 
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\Routing\RouteCollection;
 
 /**
  * Compiler pass to conditionally register routes based on enabled server providers
@@ -15,7 +14,7 @@ class ConditionalRoutePass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void
     {
-        if (!$container->hasParameter('klp_mcp_server.providers')) {
+        if (! $container->hasParameter('klp_mcp_server.providers')) {
             return;
         }
 

src/DependencyInjection/KlpMcpServerExtension.php

@@ -27,7 +27,7 @@ public function load(array $configs, ContainerBuilder $container): void
         }
         if (isset($config['server_provider'])
             && $config['server_provider'] === 'sse'
-            && !count($providers)) {
+            && ! count($providers)) {
             $providers[] = 'klp_mcp_server.provider.sse';
         }
         $container->setParameter('klp_mcp_server.providers', $providers);
@@ -51,15 +51,15 @@ public function load(array $configs, ContainerBuilder $container): void
     private function removeDisabledControllers(ContainerBuilder $container, array $enabledProviders): void
     {
         // Remove SSE controllers if SSE provider is not enabled
-        if (!in_array('klp_mcp_server.provider.sse', $enabledProviders, true)) {
+        if (! in_array('klp_mcp_server.provider.sse', $enabledProviders, true)) {
             $container->removeDefinition('KLP\KlpMcpServer\Controllers\SseController');
             $container->removeAlias('klp_mcp_server.controller.sse');
             $container->removeDefinition('KLP\KlpMcpServer\Controllers\MessageController');
             $container->removeAlias('klp_mcp_server.controller.message');
         }
 
         // Remove StreamableHTTP controller if StreamableHTTP provider is not enabled
-        if (!in_array('klp_mcp_server.provider.streamable_http', $enabledProviders, true)) {
+        if (! in_array('klp_mcp_server.provider.streamable_http', $enabledProviders, true)) {
             $container->removeDefinition('KLP\KlpMcpServer\Controllers\StreamableHttpController');
             $container->removeAlias('klp_mcp_server.controller.streamable_http');
         }

src/KlpMcpServerBundle.php

@@ -2,9 +2,9 @@
 
 namespace KLP\KlpMcpServer;
 
+use KLP\KlpMcpServer\DependencyInjection\CompilerPass\ConditionalRoutePass;
 use KLP\KlpMcpServer\DependencyInjection\CompilerPass\ResourcesDefinitionCompilerPass;
 use KLP\KlpMcpServer\DependencyInjection\CompilerPass\ToolsDefinitionCompilerPass;
-use KLP\KlpMcpServer\DependencyInjection\CompilerPass\ConditionalRoutePass;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\HttpKernel\Bundle\Bundle;
 

src/Protocol/MCPProtocol.php

@@ -25,6 +25,7 @@
  * MCPProtocol
  *
  * @internal
+ *
  * @see https://modelcontextprotocol.io/docs/concepts/architecture
  */
 final class MCPProtocol implements MCPProtocolInterface
@@ -45,9 +46,7 @@ final class MCPProtocol implements MCPProtocolInterface
      * @param  TransportFactoryInterface  $transportFactory  The transport factory to use for creating transports.
      * @return void
      */
-    public function __construct(private readonly TransportFactoryInterface $transportFactory)
-    {
-    }
+    public function __construct(private readonly TransportFactoryInterface $transportFactory) {}
 
     /**
      * Establishes a connection and processes incoming messages from the transport layer.
@@ -259,6 +258,7 @@ public function getResponseResult(string $clientId): array
                 $result[] = json_decode($message);
             }
         }
+
         return $result;
     }
 

src/Protocol/MCPProtocolInterface.php

@@ -10,11 +10,13 @@
  * MCPProtocol
  *
  * @internal
+ *
  * @see https://modelcontextprotocol.io/docs/concepts/architecture
  */
 interface MCPProtocolInterface
 {
     public const PROTOCOL_VERSION_SSE = '2024-11-05';
+
     public const PROTOCOL_VERSION_STREAMABE_HTTP = '2025-03-26';
 
     public function setProtocolVersion(string $version): void;

src/Routing/Loader/McpRouteLoader.php

@@ -15,7 +15,9 @@
 class McpRouteLoader implements LoaderInterface
 {
     private array $enabledProviders = [];
+
     private string $defaultPath = 'mcp';
+
     private bool $loaded = false;
 
     public function setEnabledProviders(array $providers): void
@@ -28,31 +30,31 @@ public function setDefaultPath(string $defaultPath): void
         $this->defaultPath = $defaultPath;
     }
 
-    public function load($resource, string|null $type = null): RouteCollection
+    public function load($resource, ?string $type = null): RouteCollection
     {
         if ($this->loaded) {
             throw new \RuntimeException('MCP routes already loaded');
         }
 
-        $routes = new RouteCollection();
+        $routes = new RouteCollection;
 
         // StreamableHTTP routes
         if ($this->isProviderEnabled('streamable_http')) {
             $routes->add('klp_mcp_server_streamable_http', new Route(
-                '/' . $this->defaultPath,
+                '/'.$this->defaultPath,
                 ['_controller' => 'klp_mcp_server.controller.streamable_http::handle']
             ));
         }
 
         // SSE routes
         if ($this->isProviderEnabled('sse')) {
             $routes->add('klp_mcp_server_sse', new Route(
-                '/' . $this->defaultPath . '/sse',
+                '/'.$this->defaultPath.'/sse',
                 ['_controller' => 'klp_mcp_server.controller.sse::handle']
             ));
 
             $routes->add('klp_mcp_server_sse_message', new Route(
-                '/' . $this->defaultPath . '/messages',
+                '/'.$this->defaultPath.'/messages',
                 ['_controller' => 'klp_mcp_server.controller.message::handle'],
                 [],
                 [],
@@ -63,19 +65,21 @@ public function load($resource, string|null $type = null): RouteCollection
         }
 
         $this->loaded = true;
+
         return $routes;
     }
 
-    public function supports($resource, string|null $type = null): bool
+    public function supports($resource, ?string $type = null): bool
     {
-        return 'mcp' === $type;
+        return $type === 'mcp';
     }
 
     public function getResolver(): LoaderResolverInterface
     {
         // Return a dummy resolver as it's required by the interface
-        return new class implements LoaderResolverInterface {
-            public function resolve($resource, string|null $type = null): LoaderInterface|false
+        return new class implements LoaderResolverInterface
+        {
+            public function resolve($resource, ?string $type = null): LoaderInterface|false
             {
                 return false;
             }
@@ -99,6 +103,6 @@ public function setResolver(LoaderResolverInterface $resolver): void
 
     private function isProviderEnabled(string $provider): bool
     {
-        return in_array('klp_mcp_server.provider.' . $provider, $this->enabledProviders, true);
+        return in_array('klp_mcp_server.provider.'.$provider, $this->enabledProviders, true);
     }
 }

src/Server/MCPServer.php

@@ -110,9 +110,9 @@ public static function create(
     ): self {
         return new self($protocol,
             $progressNotifierRepository, [
-            'name' => $name,
-            'version' => $version,
-        ], $capabilities);
+                'name' => $name,
+                'version' => $version,
+            ], $capabilities);
     }
 
     /**