acp-sdk 0.12.0 → 0.14.0

data/sig/manual/acp/agent_connection.rbs

@@ -8,75 +8,23 @@ class ACP::AgentConnection
     def respond_to?: (Symbol) -> bool
   end
 
-  interface _LoadSession
+  interface _FullAgent
+    include _Agent
+    def authenticate: (ACP::Types::AuthenticateRequest) -> (ACP::Types::AuthenticateResponse | ACP::RequestError)
     def load_session: (ACP::Types::LoadSessionRequest) -> (ACP::Types::LoadSessionResponse | ACP::RequestError)
-  end
-
-  interface _ListSessions
     def list_sessions: (ACP::Types::ListSessionsRequest) -> (ACP::Types::ListSessionsResponse | ACP::RequestError)
-  end
-
-  interface _Authenticate
-    def authenticate: (ACP::Types::AuthenticateRequest) -> (ACP::Types::AuthenticateResponse | ACP::RequestError)
-  end
-
-  interface _ResumeSession
     def resume_session: (ACP::Types::ResumeSessionRequest) -> (ACP::Types::ResumeSessionResponse | ACP::RequestError)
-  end
-
-  interface _CloseSession
     def close_session: (ACP::Types::CloseSessionRequest) -> (ACP::Types::CloseSessionResponse | ACP::RequestError)
-  end
-
-  interface _DeleteSession
     def delete_session: (ACP::Types::DeleteSessionRequest) -> (ACP::Types::DeleteSessionResponse | ACP::RequestError)
-  end
-
-  interface _Logout
     def logout: (ACP::Types::LogoutRequest) -> (ACP::Types::LogoutResponse | ACP::RequestError)
-  end
-
-  interface _ForkSession
+    def change_session_mode: (ACP::Types::SetSessionModeRequest) -> (ACP::Types::SetSessionModeResponse | ACP::RequestError)
+    def change_session_config_option: (ACP::Types::SetSessionConfigOptionRequest) -> (ACP::Types::SetSessionConfigOptionResponse | ACP::RequestError)
     def fork_session: (ACP::Types::Unstable::ForkSessionRequest) -> (ACP::Types::Unstable::ForkSessionResponse | ACP::RequestError)
-  end
-
-  interface _ListProviders
     def list_providers: (ACP::Types::Unstable::ListProvidersRequest) -> (ACP::Types::Unstable::ListProvidersResponse | ACP::RequestError)
-  end
-
-  interface _SetProvider
     def set_provider: (ACP::Types::Unstable::SetProviderRequest) -> (ACP::Types::Unstable::SetProviderResponse | ACP::RequestError)
-  end
-
-  interface _DisableProvider
     def disable_provider: (ACP::Types::Unstable::DisableProviderRequest) -> (ACP::Types::Unstable::DisableProviderResponse | ACP::RequestError)
   end
 
-  interface _ChangeSessionMode
-    def change_session_mode: (ACP::Types::SetSessionModeRequest) -> (ACP::Types::SetSessionModeResponse | ACP::RequestError)
-  end
-
-  interface _ChangeSessionConfigOption
-    def change_session_config_option: (ACP::Types::SetSessionConfigOptionRequest) -> (ACP::Types::SetSessionConfigOptionResponse | ACP::RequestError)
-  end
-
-  interface _FullAgent
-    include _Agent
-    include _LoadSession
-    include _ListSessions
-    include _Authenticate
-    include _ResumeSession
-    include _CloseSession
-    include _DeleteSession
-    include _Logout
-    include _ForkSession
-    include _ListProviders
-    include _SetProvider
-    include _DisableProvider
-    include _ChangeSessionMode
-    include _ChangeSessionConfigOption
-  end
-
   interface _SessionCreated
     def session_created: (ACP::Types::NewSessionResponse) -> void
   end
@@ -85,26 +33,4 @@ class ACP::AgentConnection
     # Runs on the transport's reader thread, so it must return quickly.
     def mcp_message_notification: (ACP::Types::Unstable::MessageMcpNotification notification) -> void
   end
-
-  # What an agent sees of ACP::AgentConnection::Client, so tests can pass a fake.
-  interface _Client
-    def update: (String session_id, ACP::Types::SessionUpdate::t update) -> void
-    def available_commands: (String session_id, Array[ACP::Types::AvailableCommand] commands) -> void
-    def request_permission: (ACP::Types::RequestPermissionRequest) -> (ACP::Types::RequestPermissionResponse | ACP::RequestError)
-    def cancel_requests: () -> void
-    def cancelled?: () -> bool
-    def read_text_file: (ACP::Types::ReadTextFileRequest) -> (ACP::Types::ReadTextFileResponse | ACP::RequestError)
-    def write_text_file: (ACP::Types::WriteTextFileRequest) -> (ACP::Types::WriteTextFileResponse | ACP::RequestError)
-    def create_terminal: (ACP::Types::CreateTerminalRequest) -> (ACP::Types::CreateTerminalResponse | ACP::RequestError)
-    def terminal_output: (ACP::Types::TerminalOutputRequest) -> (ACP::Types::TerminalOutputResponse | ACP::RequestError)
-    def wait_for_terminal_exit: (ACP::Types::WaitForTerminalExitRequest) -> (ACP::Types::WaitForTerminalExitResponse | ACP::RequestError)
-    def kill_terminal: (ACP::Types::KillTerminalRequest) -> (ACP::Types::KillTerminalResponse | ACP::RequestError)
-    def release_terminal: (ACP::Types::ReleaseTerminalRequest) -> (ACP::Types::ReleaseTerminalResponse | ACP::RequestError)
-    def create_elicitation: (ACP::Types::CreateElicitationRequest request) -> (ACP::Types::CreateElicitationResponse | ACP::RequestError)
-    def complete_elicitation: (ACP::Types::CompleteElicitationNotification notification) -> void
-    def mcp_message: (ACP::Types::Unstable::MessageMcpRequest request) -> (ACP::Types::Unstable::MessageMcpResponse::t | ACP::RequestError)
-    def ext_request: (String method, ?untyped params) -> (Hash[String, untyped] | ACP::RequestError)
-    def ext_notify: (String method, ?untyped params) -> void
-    def capabilities: () -> ACP::Types::ClientCapabilities?
-  end
 end

data/sig/manual/acp/client_connection.rbs

@@ -8,11 +8,31 @@ class ACP::ClientConnection
     # Runs on the transport's reader thread, so it must return quickly.
     def call: (ACP::Types::SessionNotification) -> void
   end
-end
 
-class ACP::ClientConnection::Streams
-  interface _Sender
-    def request: (String method, ?untyped params) -> (Hash[String, untyped] | ACP::RequestError)
-    def cancellation: () -> ACP::Transport::Cancellation?
+  # Each handler object needs only the methods of the capabilities initialize
+  # advertises; ClientConnection probes for the rest.
+  interface _FileSystem
+    def read_text_file: (ACP::Types::ReadTextFileRequest) -> (ACP::Types::ReadTextFileResponse | ACP::RequestError)
+    def write_text_file: (ACP::Types::WriteTextFileRequest) -> (ACP::Types::WriteTextFileResponse | ACP::RequestError)
+    def respond_to?: (Symbol) -> bool
+  end
+
+  # Each method runs on its own thread and may block for as long as the
+  # command runs.
+  interface _Terminal
+    def create_terminal: (ACP::Types::CreateTerminalRequest) -> (ACP::Types::CreateTerminalResponse | ACP::RequestError)
+    def terminal_output: (ACP::Types::TerminalOutputRequest) -> (ACP::Types::TerminalOutputResponse | ACP::RequestError)
+    def wait_for_terminal_exit: (ACP::Types::WaitForTerminalExitRequest) -> (ACP::Types::WaitForTerminalExitResponse | ACP::RequestError)
+    def kill_terminal: (ACP::Types::KillTerminalRequest) -> (ACP::Types::KillTerminalResponse | ACP::RequestError)
+    def release_terminal: (ACP::Types::ReleaseTerminalRequest) -> (ACP::Types::ReleaseTerminalResponse | ACP::RequestError)
+    def respond_to?: (Symbol) -> bool
+  end
+
+  interface _Elicitation
+    def create_elicitation: (ACP::Types::CreateElicitationRequest) -> (ACP::Types::CreateElicitationResponse | ACP::RequestError)
+    # Optional, for url mode. Runs on the transport's reader thread, so it
+    # must return quickly.
+    def complete_elicitation: (ACP::Types::CompleteElicitationNotification) -> void
+    def respond_to?: (Symbol) -> bool
   end
 end

data/sig/manual/acp/transport/stdio.rbs

@@ -1,8 +1,4 @@
 class ACP::Transport::Stdio
-  interface _ToH
-    def to_h: () -> Hash[String, untyped]
-  end
-
   interface _Reader
     def each_line: () { (String) -> void } -> untyped
   end
@@ -13,7 +9,7 @@ class ACP::Transport::Stdio
   end
 
   interface _Handler
-    def call: (untyped params) -> (_ToH | ACP::RequestError | ACP::Transport::Reply)
+    def call: (untyped params) -> (ACP::Transport::_Response | ACP::RequestError | ACP::Transport::Reply)
   end
 
   interface _NotificationHandler

data/sig/manual/acp/transport.rbs

@@ -5,22 +5,6 @@ module ACP::Transport
     def error: (String message) -> void
   end
 
-  interface _Peer
-    def request: (String method, ?untyped params) -> (Hash[String, untyped] | ACP::RequestError)
-    def notify: (String method, ?untyped params) -> void
-    def cancellation: () -> ACP::Transport::Cancellation?
-    def cancel_requests: () -> void
-  end
-
-  interface _Transport
-    include _Peer
-
-    def start: (
-      requests: Hash[String, ACP::Transport::Stdio::_Handler],
-      notifications: Hash[String, ACP::Transport::Stdio::_NotificationHandler]
-    ) -> Thread
-  end
-
   interface _Parser
     def from_h: (Hash[String, untyped]) -> untyped
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: acp-sdk
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.14.0
 platform: ruby
 authors:
 - Jake Bottrall
@@ -34,10 +34,12 @@ files:
 - README.md
 - lib/acp/agent_connection.rb
 - lib/acp/agent_connection/client.rb
-- lib/acp/agent_connection/optional_method.rb
-- lib/acp/agent_connection/optional_method/spec.rb
+- lib/acp/agent_connection/optional_route.rb
 - lib/acp/agent_process.rb
 - lib/acp/client_connection.rb
+- lib/acp/client_connection/elicitations.rb
+- lib/acp/client_connection/inbound.rb
+- lib/acp/client_connection/inbound/route.rb
 - lib/acp/client_connection/streams.rb
 - lib/acp/extensions.rb
 - lib/acp/protocol.rb
@@ -392,10 +394,12 @@ files:
 - lib/acp/version.rb
 - sig/generated/acp/agent_connection.rbs
 - sig/generated/acp/agent_connection/client.rbs
-- sig/generated/acp/agent_connection/optional_method.rbs
-- sig/generated/acp/agent_connection/optional_method/spec.rbs
+- sig/generated/acp/agent_connection/optional_route.rbs
 - sig/generated/acp/agent_process.rbs
 - sig/generated/acp/client_connection.rbs
+- sig/generated/acp/client_connection/elicitations.rbs
+- sig/generated/acp/client_connection/inbound.rbs
+- sig/generated/acp/client_connection/inbound/route.rbs
 - sig/generated/acp/client_connection/streams.rbs
 - sig/generated/acp/extensions.rbs
 - sig/generated/acp/protocol.rbs

data/lib/acp/agent_connection/optional_method/spec.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-class ACP::AgentConnection::OptionalMethod::Spec
-  # @dynamic rpc_method
-  attr_reader :rpc_method #: String
-  # @dynamic agent_method
-  attr_reader :agent_method #: Symbol
-  # @dynamic path
-  attr_reader :path #: Array[Symbol]
-
-  # @rbs rpc_method: String
-  # @rbs agent_method: Symbol
-  # @rbs path: Array[Symbol]
-  # @rbs return: void
-  def initialize(rpc_method:, agent_method:, path:)
-    @rpc_method = rpc_method
-    @agent_method = agent_method
-    @path = path.freeze
-    freeze
-  end
-
-  # Readers missing along the chain count as unadvertised, so the stable
-  # capabilities answer false for the unstable-only paths (fork, providers).
-  #
-  # @rbs initialize_response: ACP::Types::InitializeResponse
-  # @rbs return: bool
-  def advertised?(initialize_response)
-    # reduce's U pins to the InitializeResponse argument, but the chain walks
-    # through unrelated capability types.
-    value = path.reduce(initialize_response) { |node, name| follow(node, name) } #: untyped
-    value ? true : false
-  end
-
-  private
-
-  # @rbs node: untyped
-  # @rbs name: Symbol
-  # @rbs return: untyped
-  def follow(node, name)
-    node.respond_to?(name) ? node.public_send(name) : nil
-  end
-end

data/lib/acp/agent_connection/optional_method.rb

@@ -1,40 +0,0 @@
-# frozen_string_literal: true
-
-module ACP::AgentConnection::OptionalMethod
-  SPECS = [
-    Spec.new(rpc_method: 'authenticate', agent_method: :authenticate, path: %i[auth_methods any?]),
-    Spec.new(rpc_method: 'session/load', agent_method: :load_session, path: %i[agent_capabilities load_session]),
-    Spec.new(
-      rpc_method: 'session/list',
-      agent_method: :list_sessions,
-      path: %i[agent_capabilities session_capabilities list]
-    ),
-    Spec.new(
-      rpc_method: 'session/resume',
-      agent_method: :resume_session,
-      path: %i[agent_capabilities session_capabilities resume]
-    ),
-    Spec.new(
-      rpc_method: 'session/close',
-      agent_method: :close_session,
-      path: %i[agent_capabilities session_capabilities close]
-    ),
-    Spec.new(
-      rpc_method: 'session/delete',
-      agent_method: :delete_session,
-      path: %i[agent_capabilities session_capabilities delete]
-    ),
-    Spec.new(rpc_method: 'logout', agent_method: :logout, path: %i[agent_capabilities auth logout])
-  ].freeze #: Array[Spec]
-
-  UNSTABLE_SPECS = [
-    Spec.new(
-      rpc_method: 'session/fork',
-      agent_method: :fork_session,
-      path: %i[agent_capabilities session_capabilities fork]
-    ),
-    Spec.new(rpc_method: 'providers/list', agent_method: :list_providers, path: %i[agent_capabilities providers]),
-    Spec.new(rpc_method: 'providers/set', agent_method: :set_provider, path: %i[agent_capabilities providers]),
-    Spec.new(rpc_method: 'providers/disable', agent_method: :disable_provider, path: %i[agent_capabilities providers])
-  ].freeze #: Array[Spec]
-end

data/sig/generated/acp/agent_connection/optional_method/spec.rbs

@@ -1,32 +0,0 @@
-# Generated from lib/acp/agent_connection/optional_method/spec.rb with RBS::Inline
-
-class ACP::AgentConnection::OptionalMethod::Spec
-  # @dynamic rpc_method
-  attr_reader rpc_method: String
-
-  # @dynamic agent_method
-  attr_reader agent_method: Symbol
-
-  # @dynamic path
-  attr_reader path: Array[Symbol]
-
-  # @rbs rpc_method: String
-  # @rbs agent_method: Symbol
-  # @rbs path: Array[Symbol]
-  # @rbs return: void
-  def initialize: (rpc_method: String, agent_method: Symbol, path: Array[Symbol]) -> void
-
-  # Readers missing along the chain count as unadvertised, so the stable
-  # capabilities answer false for the unstable-only paths (fork, providers).
-  #
-  # @rbs initialize_response: ACP::Types::InitializeResponse
-  # @rbs return: bool
-  def advertised?: (ACP::Types::InitializeResponse initialize_response) -> bool
-
-  private
-
-  # @rbs node: untyped
-  # @rbs name: Symbol
-  # @rbs return: untyped
-  def follow: (untyped node, Symbol name) -> untyped
-end

data/sig/generated/acp/agent_connection/optional_method.rbs

@@ -1,7 +0,0 @@
-# Generated from lib/acp/agent_connection/optional_method.rb with RBS::Inline
-
-module ACP::AgentConnection::OptionalMethod
-  SPECS: Array[Spec]
-
-  UNSTABLE_SPECS: Array[Spec]
-end