Fix code style

Author: chr-hertel

examples/toolbox/firecrawl-crawl.php

@@ -18,7 +18,7 @@
 use Symfony\AI\Platform\Message\Message;
 use Symfony\AI\Platform\Message\MessageBag;
 
-require_once dirname(__DIR__) . '/bootstrap.php';
+require_once dirname(__DIR__).'/bootstrap.php';
 
 $platform = PlatformFactory::create(env('OPENAI_API_KEY'), http_client());
 $model = new Gpt(Gpt::GPT_4O_MINI);
@@ -37,4 +37,4 @@
 $messages = new MessageBag(Message::ofUser('Crawl the following URL: https://symfony.com/doc/current/setup.html then resume it in less than 200 words.'));
 $result = $agent->call($messages);
 
-echo $result->getContent() . \PHP_EOL;
+echo $result->getContent().\PHP_EOL;

examples/toolbox/firecrawl-map.php

@@ -18,7 +18,7 @@
 use Symfony\AI\Platform\Message\Message;
 use Symfony\AI\Platform\Message\MessageBag;
 
-require_once dirname(__DIR__) . '/bootstrap.php';
+require_once dirname(__DIR__).'/bootstrap.php';
 
 $platform = PlatformFactory::create(env('OPENAI_API_KEY'), http_client());
 $model = new Gpt(Gpt::GPT_4O_MINI);
@@ -37,4 +37,4 @@
 $messages = new MessageBag(Message::ofUser('Retrieve all the links from https://symfony.com then list only the ones related to the Messenger component.'));
 $result = $agent->call($messages);
 
-echo $result->getContent() . \PHP_EOL;
+echo $result->getContent().\PHP_EOL;

examples/toolbox/firecrawl-scrape.php

@@ -18,7 +18,7 @@
 use Symfony\AI\Platform\Message\Message;
 use Symfony\AI\Platform\Message\MessageBag;
 
-require_once dirname(__DIR__) . '/bootstrap.php';
+require_once dirname(__DIR__).'/bootstrap.php';
 
 $platform = PlatformFactory::create(env('OPENAI_API_KEY'), http_client());
 $model = new Gpt(Gpt::GPT_4O_MINI);
@@ -37,4 +37,4 @@
 $messages = new MessageBag(Message::ofUser('Scrape the following URL: https://symfony.com/doc/current/setup.html then resume it in less than 200 words.'));
 $result = $agent->call($messages);
 
-echo $result->getContent() . \PHP_EOL;
+echo $result->getContent().\PHP_EOL;

fixtures/SomeStructure.php

@@ -13,12 +13,12 @@
 
 final class SomeStructure
 {
+    public string $some;
+
     public function __construct(?string $some = null)
     {
         if (null !== $some) {
             $this->some = $some;
         }
     }
-
-    public string $some;
 }

src/ai-bundle/tests/Security/IsGrantedToolAttributeListenerTest.php

@@ -41,14 +41,6 @@ class IsGrantedToolAttributeListenerTest extends TestCase
     private EventDispatcherInterface $dispatcher;
     private AuthorizationCheckerInterface&MockObject $authChecker;
 
-    #[Before]
-    protected function setupTool(): void
-    {
-        $this->dispatcher = new EventDispatcher();
-        $this->authChecker = $this->createMock(AuthorizationCheckerInterface::class);
-        $this->dispatcher->addListener(ToolCallArgumentsResolved::class, new IsGrantedToolAttributeListener($this->authChecker));
-    }
-
     #[TestWith([new ToolWithIsGrantedOnMethod(), new Tool(new ExecutionReference(ToolWithIsGrantedOnMethod::class, 'simple'), 'simple', '')])]
     #[TestWith([new ToolWithIsGrantedOnMethod(), new Tool(new ExecutionReference(ToolWithIsGrantedOnMethod::class, 'expressionAsSubject'), 'expressionAsSubject', '')])]
     #[TestWith([new ToolWithIsGrantedOnClass(), new Tool(new ExecutionReference(ToolWithIsGrantedOnClass::class, '__invoke'), 'ToolWithIsGrantedOnClass', '')])]
@@ -82,4 +74,12 @@ public function testItWillEvaluateSubjectExpression(object $tool, Tool $metadata
         $this->authChecker->expects($this->once())->method('isGranted')->with('test:permission', 44)->willReturn(true);
         $this->dispatcher->dispatch(new ToolCallArgumentsResolved($tool, $metadata, ['itemId' => 44]));
     }
+
+    #[Before]
+    protected function setupTool(): void
+    {
+        $this->dispatcher = new EventDispatcher();
+        $this->authChecker = $this->createMock(AuthorizationCheckerInterface::class);
+        $this->dispatcher->addListener(ToolCallArgumentsResolved::class, new IsGrantedToolAttributeListener($this->authChecker));
+    }
 }

src/mcp-sdk/src/Message/Notification.php

@@ -22,6 +22,11 @@ public function __construct(
     ) {
     }
 
+    public function __toString(): string
+    {
+        return \sprintf('%s', $this->method);
+    }
+
     /**
      * @param array{method: string, params?: array<string, mixed>} $data
      */
@@ -44,9 +49,4 @@ public function jsonSerialize(): array
             'params' => $this->params,
         ];
     }
-
-    public function __toString(): string
-    {
-        return \sprintf('%s', $this->method);
-    }
 }

src/mcp-sdk/src/Message/Request.php

@@ -23,6 +23,11 @@ public function __construct(
     ) {
     }
 
+    public function __toString(): string
+    {
+        return \sprintf('%s: %s', $this->id, $this->method);
+    }
+
     /**
      * @param array{id: string|int, method: string, params?: array<string, mixed>} $data
      */
@@ -47,9 +52,4 @@ public function jsonSerialize(): array
             'params' => $this->params,
         ];
     }
-
-    public function __toString(): string
-    {
-        return \sprintf('%s: %s', $this->id, $this->method);
-    }
 }

src/mcp-sdk/src/Server/NotificationHandler/InitializedHandler.php

@@ -15,12 +15,12 @@
 
 final class InitializedHandler extends BaseNotificationHandler
 {
-    protected function supportedNotification(): string
+    public function handle(Notification $notification): void
     {
-        return 'initialized';
     }
 
-    public function handle(Notification $notification): void
+    protected function supportedNotification(): string
     {
+        return 'initialized';
     }
 }

src/mcp-sdk/tests/Message/FactoryTest.php

@@ -30,18 +30,6 @@ protected function setUp(): void
         $this->factory = new Factory();
     }
 
-    /**
-     * @param iterable<mixed> $items
-     */
-    private function first(iterable $items): mixed
-    {
-        foreach ($items as $item) {
-            return $item;
-        }
-
-        return null;
-    }
-
     public function testCreateRequest()
     {
         $json = '{"jsonrpc": "2.0", "method": "test_method", "params": {"foo": "bar"}, "id": 123}';
@@ -91,4 +79,16 @@ public function testBatchMissingMethod()
         $result = array_shift($results);
         $this->assertInstanceOf(Notification::class, $result);
     }
+
+    /**
+     * @param iterable<mixed> $items
+     */
+    private function first(iterable $items): mixed
+    {
+        foreach ($items as $item) {
+            return $item;
+        }
+
+        return null;
+    }
 }

src/platform/src/Bridge/Anthropic/Contract/DocumentNormalizer.php

@@ -21,16 +21,6 @@
  */
 class DocumentNormalizer extends ModelContractNormalizer
 {
-    protected function supportedDataClass(): string
-    {
-        return Document::class;
-    }
-
-    protected function supportsModel(Model $model): bool
-    {
-        return $model instanceof Claude;
-    }
-
     /**
      * @param Document $data
      *
@@ -47,4 +37,14 @@ public function normalize(mixed $data, ?string $format = null, array $context =
             ],
         ];
     }
+
+    protected function supportedDataClass(): string
+    {
+        return Document::class;
+    }
+
+    protected function supportsModel(Model $model): bool
+    {
+        return $model instanceof Claude;
+    }
 }