Merge remote-tracking branch 'origin/master'

Author: klapaudius

.github/workflows/coverage.yml

@@ -17,7 +17,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Validate composer.json and composer.lock
         run: composer validate --strict

.github/workflows/fix-php-code-style-issues.yml

@@ -17,7 +17,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           ref: ${{ github.head_ref }}
 

.github/workflows/tests.yml

@@ -25,7 +25,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Validate composer.json and composer.lock
         run: composer validate --strict

src/Services/PromptService/Examples/ExamplePromptProvider.php

@@ -31,7 +31,6 @@
  *
  * ALTERNATIVE APPROACH: Return prompt class names.
  * WARNING: This requires prompts to be registered as public services.
- *
  * @example Returning class names (NOT RECOMMENDED - requires public services):
  * ```php
  * public function getPrompts(): iterable
@@ -54,7 +53,6 @@
  * - ✅ Better performance (prompts instantiated once during compilation)
  * - ✅ Type safety with constructor injection
  * - ✅ Follows Symfony best practices
- *
  * @example Registration in services.yaml:
  * ```yaml
  * services:

src/Services/PromptService/PromptRepository.php

@@ -31,15 +31,15 @@ class PromptRepository
     /**
      * The logger instance.
      */
-    protected LoggerInterface|null $logger;
+    protected ?LoggerInterface $logger;
 
     /**
      * Constructor.
      *
      * @param  ContainerInterface  $container  The Symfony service container.
      * @param  LoggerInterface|null  $logger  Optional logger instance for debugging prompt registration.
      */
-    public function __construct(ContainerInterface $container, LoggerInterface|null $logger = null)
+    public function __construct(ContainerInterface $container, ?LoggerInterface $logger = null)
     {
         $this->container = $container;
         $this->logger = $logger;

src/Services/ResourceService/Examples/ExampleResourceProvider.php

@@ -31,7 +31,6 @@
  *
  * ALTERNATIVE APPROACH: Return resource class names.
  * WARNING: This requires resources to be registered as public services.
- *
  * @example Returning class names (NOT RECOMMENDED - requires public services):
  * ```php
  * public function getResources(): iterable
@@ -54,7 +53,6 @@
  * - ✅ Better performance (resources instantiated once during compilation)
  * - ✅ Type safety with constructor injection
  * - ✅ Follows Symfony best practices
- *
  * @example Registration in services.yaml:
  * ```yaml
  * services:

src/Services/ResourceService/ResourceRepository.php

@@ -37,15 +37,15 @@ class ResourceRepository
     /**
      * The logger instance.
      */
-    protected LoggerInterface|null $logger;
+    protected ?LoggerInterface $logger;
 
     /**
      * Constructor.
      *
      * @param  ContainerInterface  $container  The Symfony service container. If null, it resolves from the facade.
      * @param  LoggerInterface|null  $logger  Optional logger instance for debugging resource registration.
      */
-    public function __construct(ContainerInterface $container, LoggerInterface|null $logger = null)
+    public function __construct(ContainerInterface $container, ?LoggerInterface $logger = null)
     {
         $this->container = $container;
         $this->logger = $logger;

tests/Server/MCPServerTest.php

@@ -655,7 +655,7 @@ public function test_register_sampling_response_handler_with_client(): void
 
         $mockProtocol->expects($this->once())
             ->method('registerResponseHandler')
-            ->with($this->callback(function ($arg) use ($mockSamplingClient, $mockLogger) {
+            ->with($this->callback(function ($arg) {
                 $this->assertInstanceOf(SamplingResponseHandler::class, $arg);
 
                 return true;