Addressing feedback

Author: Ginja

examples/sse_test_client.rb renamed to examples/streaming_http_test_client.rb

@@ -137,7 +137,7 @@ def call(message:, delay: 0)
 end
 
 # Print usage instructions
-puts "=== MCP SSE Test Server ==="
+puts "=== MCP Streaming HTTP Test Server ==="
 puts ""
 puts "Starting server on http://localhost:9393"
 puts ""

examples/sse_test_server.rb renamed to examples/streaming_http_test_server.rb

@@ -8,11 +8,6 @@ class Server
     module Transports
       class StdioTransport < Transport
         STATUS_INTERRUPTED = Signal.list["INT"] + 128
-        class << self
-          def register
-            super("stdio", self)
-          end
-        end
 
         def initialize(server)
           @server = server

lib/mcp/server/transports/stdio_transport.rb

@@ -8,12 +8,6 @@ module MCP
   class Server
     module Transports
       class StreamableHTTPTransport < Transport
-        class << self
-          def register
-            super("http", self)
-          end
-        end
-
         def initialize(server)
           super
           # { session_id => { stream: stream_object }

lib/mcp/server/transports/streamable_http_transport.rb

@@ -2,19 +2,6 @@
 
 module MCP
   class Transport
-    class << self
-      def register(name, transport_class)
-        @transports ||= {}
-        @transports[name] = transport_class
-      end
-
-      def get(name)
-        raise "Transport #{name} not found" unless @transports
-
-        @transports.fetch(name) { raise "Transport #{name} not found" }
-      end
-    end
-
     # Initialize the transport with the server instance
     def initialize(server)
       @server = server

lib/mcp/transport.rb

@@ -126,11 +126,5 @@ def send_notification(method, params = nil)
       assert_equal Methods::NOTIFICATIONS_PROMPTS_LIST_CHANGED, notifications[1][:method]
       assert_equal Methods::NOTIFICATIONS_RESOURCES_LIST_CHANGED, notifications[2][:method]
     end
-
-    test "notification method constants are defined correctly" do
-      assert_equal "notifications/tools/list_changed", Methods::NOTIFICATIONS_TOOLS_LIST_CHANGED
-      assert_equal "notifications/prompts/list_changed", Methods::NOTIFICATIONS_PROMPTS_LIST_CHANGED
-      assert_equal "notifications/resources/list_changed", Methods::NOTIFICATIONS_RESOURCES_LIST_CHANGED
-    end
   end
 end

test/mcp/server_notification_test.rb

@@ -28,22 +28,6 @@ def close
         resources: [],
       )
       @transport = TestTransport.new(@server)
-      # Clear transports registry before each test
-      Transport.instance_variable_set(:@transports, nil)
-    end
-
-    teardown do
-      # Clear transports registry after each test
-      Transport.instance_variable_set(:@transports, nil)
-    end
-
-    test "registers transport class" do
-      Transport.register("test", TestTransport)
-      assert_equal TestTransport, Transport.get("test")
-    end
-
-    test "raises error for unknown transport" do
-      assert_raises(RuntimeError, "Transport unknown not found") { Transport.get("unknown") }
     end
 
     test "initializes with server instance" do