feature #227 [MCP SDK][MCP Bundle] Add parameter for pagination and allow disabling pagination (xavierleune)

This PR was merged into the main branch.

Discussion
----------

[MCP SDK][MCP Bundle] Add parameter for pagination and allow disabling pagination

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #10  <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

The page size was hardcoded to 20 elements, now users can configure it in the mcp bundle. Passing null value will disable pagination and fetch all entries.

Commits
-------

a6c80d5 [MCP SDK] [MCP Bundle] Add parameter for pagination and allow disabling pagination

Author: fabpot

src/mcp-bundle/config/options.php

@@ -16,6 +16,7 @@
         ->children()
             ->scalarNode('app')->defaultValue('app')->end()
             ->scalarNode('version')->defaultValue('0.0.1')->end()
+            ->scalarNode('page_size')->defaultValue(20)->end()
             // ->arrayNode('servers')
             //     ->useAttributeAsKey('name')
             //     ->arrayPrototype()

src/mcp-bundle/config/services.php

@@ -44,7 +44,7 @@
         ->set('mcp.server.request_handler.tool_list', ToolListHandler::class)
             ->args([
                 service('mcp.tool_collection'),
-                20,
+                param('mcp.page_size'),
             ])
             ->tag('mcp.server.request_handler')
 

src/mcp-bundle/src/McpBundle.php

@@ -39,6 +39,7 @@ public function loadExtension(array $config, ContainerConfigurator $container, C
 
         $builder->setParameter('mcp.app', $config['app']);
         $builder->setParameter('mcp.version', $config['version']);
+        $builder->setParameter('mcp.page_size', $config['page_size']);
 
         if (isset($config['client_transports'])) {
             $this->configureClient($config['client_transports'], $builder);

src/mcp-sdk/src/Capability/Tool/CollectionInterface.php

@@ -16,11 +16,11 @@
 interface CollectionInterface
 {
     /**
-     * @param int $count the number of metadata items to return
+     * @param int|null $count the number of metadata items to return, null returns all items
      *
      * @return iterable<MetadataInterface>
      *
      * @throws InvalidCursorException if no item with $lastIdentifier was found
      */
-    public function getMetadata(int $count, ?string $lastIdentifier = null): iterable;
+    public function getMetadata(?int $count, ?string $lastIdentifier = null): iterable;
 }

src/mcp-sdk/src/Capability/ToolChain.php

@@ -34,7 +34,7 @@ public function __construct(
     ) {
     }
 
-    public function getMetadata(int $count, ?string $lastIdentifier = null): iterable
+    public function getMetadata(?int $count, ?string $lastIdentifier = null): iterable
     {
         $found = null === $lastIdentifier;
         foreach ($this->items as $item) {
@@ -48,7 +48,7 @@ public function getMetadata(int $count, ?string $lastIdentifier = null): iterabl
             }
 
             yield $item;
-            if (--$count <= 0) {
+            if (null !== $count && 0 >= --$count) {
                 break;
             }
         }

src/mcp-sdk/src/Server/RequestHandler/ToolListHandler.php

@@ -19,7 +19,7 @@ final class ToolListHandler extends BaseRequestHandler
 {
     public function __construct(
         private readonly CollectionInterface $collection,
-        private readonly int $pageSize = 20,
+        private readonly ?int $pageSize = 20,
     ) {
     }