refactor(Protocol): streamline capability checks for server features

[skip ci]

Author: CodeWithKyrian

src/Protocol.php

@@ -299,44 +299,44 @@ private function assertRequestCapability(string $method): void
 
             case 'tools/list':
             case 'tools/call':
-                if ($capabilities->tools === null) {
+                if (!$capabilities->tools) {
                     throw McpServerException::methodNotFound($method, 'Tools are not enabled on this server.');
                 }
                 break;
 
             case 'resources/list':
             case 'resources/templates/list':
             case 'resources/read':
-                if ($capabilities->resources === null) {
+                if (!$capabilities->resources) {
                     throw McpServerException::methodNotFound($method, 'Resources are not enabled on this server.');
                 }
                 break;
 
             case 'resources/subscribe':
             case 'resources/unsubscribe':
-                if ($capabilities->resources === null) {
+                if (!$capabilities->resources) {
                     throw McpServerException::methodNotFound($method, 'Resources are not enabled on this server.');
                 }
-                if (!$capabilities->resources['subscribe']) {
+                if (!$capabilities->resourcesSubscribe) {
                     throw McpServerException::methodNotFound($method, 'Resources subscription is not enabled on this server.');
                 }
                 break;
 
             case 'prompts/list':
             case 'prompts/get':
-                if ($capabilities->prompts === null) {
+                if (!$capabilities->prompts) {
                     throw McpServerException::methodNotFound($method, 'Prompts are not enabled on this server.');
                 }
                 break;
 
             case 'logging/setLevel':
-                if ($capabilities->logging === null) {
+                if (!$capabilities->logging) {
                     throw McpServerException::methodNotFound($method, 'Logging is not enabled on this server.');
                 }
                 break;
 
             case 'completion/complete':
-                if ($capabilities->completions === null) {
+                if (!$capabilities->completions) {
                     throw McpServerException::methodNotFound($method, 'Completions are not enabled on this server.');
                 }
                 break;
@@ -354,29 +354,29 @@ private function canSendNotification(string $method): bool
 
         switch ($method) {
             case 'notifications/message':
-                if ($capabilities->logging === null) {
+                if (!$capabilities->logging) {
                     $this->logger->warning('Logging is not enabled on this server. Notifications/message will not be sent.');
                     $valid = false;
                 }
                 break;
 
             case "notifications/resources/updated":
             case "notifications/resources/list_changed":
-                if ($capabilities->resources === null || !$capabilities->resources['listChanged']) {
+                if (!$capabilities->resources || !$capabilities->resourcesListChanged) {
                     $this->logger->warning('Resources list changed notifications are not enabled on this server. Notifications/resources/list_changed will not be sent.');
                     $valid = false;
                 }
                 break;
 
             case "notifications/tools/list_changed":
-                if ($capabilities->tools === null || !$capabilities->tools['listChanged']) {
+                if (!$capabilities->tools || !$capabilities->toolsListChanged) {
                     $this->logger->warning('Tools list changed notifications are not enabled on this server. Notifications/tools/list_changed will not be sent.');
                     $valid = false;
                 }
                 break;
 
             case "notifications/prompts/list_changed":
-                if ($capabilities->prompts === null || !$capabilities->prompts['listChanged']) {
+                if (!$capabilities->prompts || !$capabilities->promptsListChanged) {
                     $this->logger->warning('Prompts list changed notifications are not enabled on this server. Notifications/prompts/list_changed will not be sent.');
                     $valid = false;
                 }

tests/Unit/ConfigurationTest.php

@@ -90,7 +90,7 @@
 it('constructs configuration object with specific capabilities', function () {
     $customCaps = ServerCapabilities::make(
         resourcesSubscribe: true,
-        loggingEnabled: true,
+        logging: true,
     );
 
     $config = new Configuration(
@@ -104,5 +104,5 @@
 
     expect($config->capabilities)->toBe($customCaps);
     expect($config->capabilities->resourcesSubscribe)->toBeTrue();
-    expect($config->capabilities->loggingEnabled)->toBeTrue();
+    expect($config->capabilities->logging)->toBeTrue();
 });

tests/Unit/ProtocolTest.php

@@ -271,7 +271,7 @@ function expectSuccessResponse(mixed $response, mixed $expectedResult, string|in
     $request = createRequest('tools/list');
     $configuration = new Configuration(
         serverInfo: $this->configuration->serverInfo,
-        capabilities: ServerCapabilities::make(toolsEnabled: false),
+        capabilities: ServerCapabilities::make(tools: false),
         logger: $this->logger,
         loop: $this->configuration->loop,
         cache: $this->configuration->cache,
@@ -509,10 +509,10 @@ function expectSuccessResponse(mixed $response, mixed $expectedResult, string|in
 it('allows initialize and ping regardless of capabilities', function (string $method) {
     $request = createRequest($method);
     $capabilities = ServerCapabilities::make(
-        toolsEnabled: false,
-        resourcesEnabled: false,
-        promptsEnabled: false,
-        loggingEnabled: false,
+        tools: false,
+        resources: false,
+        prompts: false,
+        logging: false,
     );
     $configuration = new Configuration(
         serverInfo: $this->configuration->serverInfo,