release: v0.13.0 (#39)

Author: ThomasK33

README.md

@@ -14,7 +14,7 @@ Learn more about the protocol itself at <https://agentclientprotocol.com>.
 <!-- `$ printf 'go get github.com/coder/acp-go-sdk@v%s\n' "$(cat schema/version)"` as bash -->
 
 ```bash
-go get github.com/coder/acp-go-sdk@v0.12.2
+go get github.com/coder/acp-go-sdk@v0.13.0
 ```
 
 ## Get Started

agent_gen.go

@@ -36,7 +36,8 @@ func WriteDispatchJen(outDir string, schema *load.Schema, meta *load.Meta) error
 			caseBody = append(caseBody, jUnmarshalValidate(mi.Notif)...)
 			// Special-case: session/cancel should also cancel any in-flight prompt ctx for the session.
 			if mi.Method == "session/cancel" {
-				caseBody = append(caseBody,
+				caseBody = append(
+					caseBody,
 					// cancel active prompt context if present
 					Id("a").Dot("mu").Dot("Lock").Call(),
 					If(List(Id("cn"), Id("ok")).Op(":=").Id("a").Dot("sessionCancels").Index(Id("string").Call(Id("p").Dot("SessionId"))), Id("ok")).Block(
@@ -63,7 +64,8 @@ func WriteDispatchJen(outDir string, schema *load.Schema, meta *load.Meta) error
 			}
 			if mi.Method == "session/prompt" {
 				// Derive a cancellable context per session prompt.
-				caseBody = append(caseBody,
+				caseBody = append(
+					caseBody,
 					Var().Id("reqCtx").Qual("context", "Context"), Var().Id("cancel").Qual("context", "CancelFunc"),
 					List(Id("reqCtx"), Id("cancel")).Op("=").Qual("context", "WithCancel").Call(Id("ctx")),
 					Id("a").Dot("mu").Dot("Lock").Call(),
@@ -72,7 +74,8 @@ func WriteDispatchJen(outDir string, schema *load.Schema, meta *load.Meta) error
 					Id("a").Dot("mu").Dot("Unlock").Call(),
 				)
 				// Call agent.Prompt(reqCtx, p)
-				caseBody = append(caseBody,
+				caseBody = append(
+					caseBody,
 					List(Id("resp"), Id("err")).Op(":=").Id(recv).Dot(methodName).Call(Id("reqCtx"), Id("p")),
 					// cleanup entry after return
 					Id("a").Dot("mu").Dot("Lock").Call(),
@@ -93,7 +96,8 @@ func WriteDispatchJen(outDir string, schema *load.Schema, meta *load.Meta) error
 	}
 	switchCases = append(switchCases, Default().Block(Return(Nil(), Id("NewMethodNotFound").Call(Id("method")))))
 	fAgent.Func().Params(Id("a").Op("*").Id("AgentSideConnection")).Id("handle").Params(
-		Id("ctx").Qual("context", "Context"), Id("method").String(), Id("params").Qual("encoding/json", "RawMessage")).
+		Id("ctx").Qual("context", "Context"), Id("method").String(), Id("params").Qual("encoding/json", "RawMessage"),
+	).
 		Params(Any(), Op("*").Id("RequestError")).
 		Block(Switch(Id("method")).Block(switchCases...))
 
@@ -201,7 +205,8 @@ func WriteDispatchJen(outDir string, schema *load.Schema, meta *load.Meta) error
 	}
 	cCases = append(cCases, Default().Block(Return(Nil(), Id("NewMethodNotFound").Call(Id("method")))))
 	fClient.Func().Params(Id("c").Op("*").Id("ClientSideConnection")).Id("handle").Params(
-		Id("ctx").Qual("context", "Context"), Id("method").String(), Id("params").Qual("encoding/json", "RawMessage")).
+		Id("ctx").Qual("context", "Context"), Id("method").String(), Id("params").Qual("encoding/json", "RawMessage"),
+	).
 		Params(Any(), Op("*").Id("RequestError")).
 		Block(Switch(Id("method")).Block(cCases...))
 

client_gen.go

@@ -8,6 +8,7 @@
     "document_did_save": "document/didSave",
     "initialize": "initialize",
     "logout": "logout",
+    "mcp_message": "mcp/message",
     "nes_accept": "nes/accept",
     "nes_close": "nes/close",
     "nes_reject": "nes/reject",
@@ -33,6 +34,9 @@
     "elicitation_create": "elicitation/create",
     "fs_read_text_file": "fs/read_text_file",
     "fs_write_text_file": "fs/write_text_file",
+    "mcp_connect": "mcp/connect",
+    "mcp_disconnect": "mcp/disconnect",
+    "mcp_message": "mcp/message",
     "session_request_permission": "session/request_permission",
     "session_update": "session/update",
     "terminal_create": "terminal/create",

cmd/generate/internal/emit/dispatch.go

@@ -77,7 +77,7 @@
                   "title": "ExtNotification"
                 }
               ],
-              "description": "All possible notifications that an agent can send to a client.\n\nThis enum is used internally for routing RPC notifications. You typically won't need\nto use this directly - use the notification methods on the [`Client`] trait instead.\n\nNotifications do not expect a response."
+              "description": "All possible notifications that an agent can send to a client.\n\nThis enum is used internally for routing RPC notifications. You typically won't need\nto use this directly.\n\nNotifications do not expect a response."
             },
             {
               "type": "null"
@@ -183,7 +183,7 @@
                   "title": "ExtMethodRequest"
                 }
               ],
-              "description": "All possible requests that an agent can send to a client.\n\nThis enum is used internally for routing RPC requests. You typically won't need\nto use this directly - instead, use the methods on the [`Client`] trait.\n\nThis enum encompasses all method calls from agent to client."
+              "description": "All possible requests that an agent can send to a client.\n\nThis enum is used internally for routing RPC requests. You typically won't need\nto use this directly.\n\nThis enum encompasses all method calls from agent to client."
             },
             {
               "type": "null"
@@ -600,7 +600,7 @@
                   "title": "ExtNotification"
                 }
               ],
-              "description": "All possible notifications that a client can send to an agent.\n\nThis enum is used internally for routing RPC notifications. You typically won't need\nto use this directly - use the notification methods on the [`Agent`] trait instead.\n\nNotifications do not expect a response."
+              "description": "All possible notifications that a client can send to an agent.\n\nThis enum is used internally for routing RPC notifications. You typically won't need\nto use this directly.\n\nNotifications do not expect a response."
             },
             {
               "type": "null"
@@ -724,7 +724,7 @@
                   "title": "ExtMethodRequest"
                 }
               ],
-              "description": "All possible requests that a client can send to an agent.\n\nThis enum is used internally for routing RPC requests. You typically won't need\nto use this directly - instead, use the methods on the [`Agent`] trait.\n\nThis enum encompasses all method calls from client to agent."
+              "description": "All possible requests that a client can send to an agent.\n\nThis enum is used internally for routing RPC requests. You typically won't need\nto use this directly.\n\nThis enum encompasses all method calls from client to agent."
             },
             {
               "type": "null"
@@ -2213,7 +2213,7 @@
           "type": "string"
         }
       ],
-      "description": "JSON RPC Request Id\n\nAn identifier established by the Client that MUST contain a String, Number, or NULL value if included. If it is not included it is assumed to be a notification. The value SHOULD normally not be Null [1] and Numbers SHOULD NOT contain fractional parts [2]\n\nThe Server MUST reply with the same value in the Response object if included. This member is used to correlate the context between the two objects.\n\n[1] The use of Null as a value for the id member in a Request object is discouraged, because this specification uses a value of Null for Responses with an unknown id. Also, because JSON-RPC 1.0 uses an id value of Null for Notifications this could cause confusion in handling.\n\n[2] Fractional parts may be problematic, since many decimal fractions cannot be represented exactly as binary fractions."
+      "description": "JSON RPC Request Id\n\nAn identifier established by the Client that MUST contain a String, Number, or NULL value if included. If it is not included it is assumed to be a notification. The value SHOULD normally not be Null \\[1\\] and Numbers SHOULD NOT contain fractional parts \\[2\\]\n\nThe Server MUST reply with the same value in the Response object if included. This member is used to correlate the context between the two objects.\n\n\\[1\\] The use of Null as a value for the id member in a Request object is discouraged, because this specification uses a value of Null for Responses with an unknown id. Also, because JSON-RPC 1.0 uses an id value of Null for Notifications this could cause confusion in handling.\n\n\\[2\\] Fractional parts may be problematic, since many decimal fractions cannot be represented exactly as binary fractions."
     },
     "RequestPermissionOutcome": {
       "description": "The outcome of a permission request.",