modelcontextprotocol/php-sdk
Watch
1
0
Fork
You've already forked php-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[Server] Remove redundant Builder.setServerCapabilities method (#107)

Browse source
This commit is contained in:
Adebayo120 2025-10-12 13:51:21 +01:00 • committed by GitHub
commit 4486776e7d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 6 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

2
examples/stdio-explicit-registration/server.php
View file

@ -28,7 +28,7 @@ $server = Server::builder()
->addResource([SimpleHandlers::class, 'getAppVersion'], 'app://version', 'application_version', mimeType: 'text/plain') ->addResource([SimpleHandlers::class, 'getAppVersion'], 'app://version', 'application_version', mimeType: 'text/plain')
->addPrompt([SimpleHandlers::class, 'greetingPrompt'], 'personalized_greeting') ->addPrompt([SimpleHandlers::class, 'greetingPrompt'], 'personalized_greeting')
->addResourceTemplate([SimpleHandlers::class, 'getItemDetails'], 'item://{itemId}/details', 'get_item_details', mimeType: 'application/json') ->addResourceTemplate([SimpleHandlers::class, 'getItemDetails'], 'item://{itemId}/details', 'get_item_details', mimeType: 'application/json')
->setServerCapabilities(new ServerCapabilities( ->setCapabilities(new ServerCapabilities(
tools: true, tools: true,
toolsListChanged: false, toolsListChanged: false,
resources: true, resources: true,

18
src/Server/Builder.php
View file

@ -75,8 +75,6 @@ final class Builder
private ?string $instructions = null; private ?string $instructions = null;
private ?ServerCapabilities $explicitCapabilities = null;
/** /**
* @var array<int, MethodHandlerInterface> * @var array<int, MethodHandlerInterface>
*/ */
@ -177,9 +175,9 @@ final class Builder
/** /**
* Explicitly set server capabilities. If set, this overrides automatic detection. * Explicitly set server capabilities. If set, this overrides automatic detection.
*/ */
public function setCapabilities(ServerCapabilities $capabilities): self public function setCapabilities(ServerCapabilities $serverCapabilities): self
{ {
$this->explicitCapabilities = $capabilities; $this->serverCapabilities = $serverCapabilities;
return $this; return $this;
} }
@ -266,13 +264,6 @@ final class Builder
return $this; return $this;
} }
public function setServerCapabilities(ServerCapabilities $serverCapabilities): self
{
$this->serverCapabilities = $serverCapabilities;
return $this;
}
/** /**
* Manually registers a tool handler. * Manually registers a tool handler.
* *
@ -370,7 +361,7 @@ final class Builder
$sessionStore = $this->sessionStore ?? new InMemorySessionStore($sessionTtl); $sessionStore = $this->sessionStore ?? new InMemorySessionStore($sessionTtl);
$messageFactory = MessageFactory::make(); $messageFactory = MessageFactory::make();
$capabilities = $this->explicitCapabilities ?? $registry->getCapabilities(); $capabilities = $registry->getCapabilities();
$configuration = new Configuration($this->serverInfo, $capabilities, $this->paginationLimit, $this->instructions); $configuration = new Configuration($this->serverInfo, $capabilities, $this->paginationLimit, $this->instructions);
$referenceHandler = new ReferenceHandler($container); $referenceHandler = new ReferenceHandler($container);
@ -597,7 +588,8 @@ final class Builder
} }
if (\is_array($handler)) { if (\is_array($handler)) {
return \sprintf('%s::%s', return \sprintf(
'%s::%s',
\is_object($handler[0]) ? $handler[0]::class : $handler[0], \is_object($handler[0]) ? $handler[0]::class : $handler[0],
$handler[1], $handler[1],
); );