Use direct calls instead of public_send in fetch_all_pages

`fetch_all_pages` now returns the page results, and each method extracts its items with `flat_map` a direct call, rather than `public_send` on a passed-in method name.

Author: baxter2

lib/mcp/client.rb

@@ -146,7 +146,7 @@ def list_tools(cursor: nil)
     #   end
     def tools
       # TODO: consider renaming to `list_all_tools`.
-      fetch_all_pages(:tools) { |cursor| list_tools(cursor: cursor) }
+      fetch_all_pages { |cursor| list_tools(cursor: cursor) }.flat_map(&:tools)
     end
 
     # Returns a single page of resources from the server.
@@ -175,7 +175,7 @@ def list_resources(cursor: nil)
     # @return [Array<Hash>] An array of available resources.
     def resources
       # TODO: consider renaming to `list_all_resources`.
-      fetch_all_pages(:resources) { |cursor| list_resources(cursor: cursor) }
+      fetch_all_pages { |cursor| list_resources(cursor: cursor) }.flat_map(&:resources)
     end
 
     # Returns a single page of resource templates from the server.
@@ -204,7 +204,7 @@ def list_resource_templates(cursor: nil)
     # @return [Array<Hash>] An array of available resource templates.
     def resource_templates
       # TODO: consider renaming to `list_all_resource_templates`.
-      fetch_all_pages(:resource_templates) { |cursor| list_resource_templates(cursor: cursor) }
+      fetch_all_pages { |cursor| list_resource_templates(cursor: cursor) }.flat_map(&:resource_templates)
     end
 
     # Returns a single page of prompts from the server.
@@ -233,7 +233,7 @@ def list_prompts(cursor: nil)
     # @return [Array<Hash>] An array of available prompts.
     def prompts
       # TODO: consider renaming to `list_all_prompts`.
-      fetch_all_pages(:prompts) { |cursor| list_prompts(cursor: cursor) }
+      fetch_all_pages { |cursor| list_prompts(cursor: cursor) }.flat_map(&:prompts)
     end
 
     # Calls a tool via the transport layer and returns the full response from the server.
@@ -324,25 +324,25 @@ def ping
 
     private
 
-    # Follows `next_cursor` across every page of a list endpoint and returns all
-    # items concatenated. The `seen` set guards against a server that repeats or
+    # Walks every page of a list endpoint, following `next_cursor`, and returns
+    # the page results. The `seen` set guards against a server that repeats or
     # cycles cursors, so the loop always terminates.
-    def fetch_all_pages(items_method)
-      items = []
+    def fetch_all_pages
+      pages = []
       seen = Set.new
       cursor = nil
 
       loop do
         page = yield(cursor)
-        items.concat(page.public_send(items_method))
+        pages << page
         next_cursor = page.next_cursor
         break if next_cursor.nil? || seen.include?(next_cursor)
 
         seen << next_cursor
         cursor = next_cursor
       end
 
-      items
+      pages
     end
 
     def request(method:, params: nil)