fix: update session ID handling in MCP apps configuration

Author: mattdholloway

python/copilot/client.py

@@ -2012,7 +2012,7 @@ def _register_inline(raw_response: Any) -> None:
             session._workspace_path = response.get("workspacePath")
             capabilities = response.get("capabilities")
             session._set_capabilities(capabilities)
-            _warn_if_mcp_apps_dropped(enable_mcp_apps, capabilities, actual_session_id)
+            _warn_if_mcp_apps_dropped(enable_mcp_apps, capabilities, registered_session_id)
         except BaseException as exc:
             if registered_session_id is not None:
                 with self._sessions_lock:

rust/src/types.rs

@@ -3959,7 +3959,7 @@ mod tests {
         assert!(cfg.enable_mcp_apps);
 
         let (wire, _runtime) = cfg
-            .into_wire(SessionId::from("enable-mcp-apps"))
+            .into_wire(Some(SessionId::from("enable-mcp-apps")))
             .expect("enable_mcp_apps config has no duplicate handlers");
         assert!(wire.request_mcp_apps);