Address Copilot Code Review round 3

- Fix mock stub for session.ui.elicitation: return elicitation result shape
  ({:action :content}) instead of nested {:result {}}
- Fix changelog: update test count from 13 to 18

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: krukow

CHANGELOG.md

@@ -16,7 +16,7 @@ All notable changes to this project will be documented in this file. This change
 - **User input handler tests** — 2 tests for `userInput.request` server→client RPC
 - **System message transform tests** — 3 tests for `systemMessage.transform` callback invocation, error fallback, and passthrough
 - **Tool result normalization tests** — 3 tests for string, nil, and structured ToolResultObject results via v3 broadcast
-- **Session RPC wrapper tests** — 13 integration tests for all new RPC wrapper functions
+- **Session RPC wrapper tests** — 18 integration tests for all new RPC wrapper functions
 - **Mock server enhancements** — `send-rpc-request!` for testing server→client RPCs, response routing in server loop, 30+ new method stubs
 - Full `s/fdef` instrumentation for all 19 new public functions
 

src/github/copilot_sdk/session.clj

@@ -1268,7 +1268,7 @@
         conn (connection-io client)]
     (util/wire->clj
      (proto/send-request! conn "session.fleet.start"
-                          (merge {:session-id session-id} params)))))
+                          (assoc (merge {} params) :session-id session-id)))))
 
 ;; -- UI Elicitation ----------------------------------------------------------
 

test/github/copilot_sdk/mock_server.clj

@@ -348,7 +348,7 @@
                  "session.compaction.compact" {:success true}
                  "session.shell.exec" {:exitCode 0 :stdout "" :stderr ""}
                  "session.shell.kill" {:success true}
-                 "session.ui.elicitation" {:result {}}
+                 "session.ui.elicitation" {:action "accept" :content {}}
                  "mcp.config.list" {:servers []}
                  "mcp.config.add" {:success true}
                  "mcp.config.update" {:success true}