Restore compact fallback and fix Python RPC tests

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

Author: SteveSandersonMS

dotnet/src/Generated/Rpc.cs

@@ -4,7 +4,7 @@
 
 from copilot import CopilotClient
 from copilot.client import SubprocessConfig
-from copilot.generated.rpc import SessionAgentSelectParams
+from copilot.generated.rpc import AgentSelectRequest
 from copilot.session import PermissionHandler
 
 from .testharness import CLI_PATH, E2ETestContext
@@ -99,7 +99,7 @@ async def test_should_select_and_get_current_agent(self):
 
             # Select the agent
             select_result = await session.rpc.agent.select(
-                SessionAgentSelectParams(name="test-agent")
+                AgentSelectRequest(name="test-agent")
             )
             assert select_result.agent is not None
             assert select_result.agent.name == "test-agent"
@@ -135,7 +135,7 @@ async def test_should_deselect_current_agent(self):
             )
 
             # Select then deselect
-            await session.rpc.agent.select(SessionAgentSelectParams(name="test-agent"))
+            await session.rpc.agent.select(AgentSelectRequest(name="test-agent"))
             await session.rpc.agent.deselect()
 
             # Verify no agent is selected

go/rpc/generated_rpc.go

@@ -4,7 +4,7 @@
 
 from copilot import CopilotClient
 from copilot.client import SubprocessConfig
-from copilot.generated.rpc import PingParams
+from copilot.generated.rpc import PingRequest
 from copilot.session import PermissionHandler
 
 from .testharness import CLI_PATH, E2ETestContext
@@ -21,7 +21,7 @@ async def test_should_call_rpc_ping_with_typed_params(self):
         try:
             await client.start()
 
-            result = await client.rpc.ping(PingParams(message="typed rpc test"))
+            result = await client.rpc.ping(PingRequest(message="typed rpc test"))
             assert result.message == "pong: typed rpc test"
             assert isinstance(result.timestamp, (int, float))
 
@@ -150,7 +150,7 @@ async def test_get_and_set_session_mode(self):
     @pytest.mark.asyncio
     async def test_read_update_and_delete_plan(self):
         """Test reading, updating, and deleting plan"""
-        from copilot.generated.rpc import SessionPlanUpdateParams
+        from copilot.generated.rpc import PlanUpdateRequest
 
         client = CopilotClient(SubprocessConfig(cli_path=CLI_PATH, use_stdio=True))
 
@@ -167,7 +167,7 @@ async def test_read_update_and_delete_plan(self):
 
             # Create/update plan
             plan_content = "# Test Plan\n\n- Step 1\n- Step 2"
-            await session.rpc.plan.update(SessionPlanUpdateParams(content=plan_content))
+            await session.rpc.plan.update(PlanUpdateRequest(content=plan_content))
 
             # Verify plan exists and has correct content
             after_update = await session.rpc.plan.read()
@@ -191,8 +191,8 @@ async def test_read_update_and_delete_plan(self):
     async def test_create_list_and_read_workspace_files(self):
         """Test creating, listing, and reading workspace files"""
         from copilot.generated.rpc import (
-            SessionWorkspaceCreateFileParams,
-            SessionWorkspaceReadFileParams,
+            WorkspaceCreateFileRequest,
+            WorkspaceReadFileRequest,
         )
 
         client = CopilotClient(SubprocessConfig(cli_path=CLI_PATH, use_stdio=True))
@@ -210,7 +210,7 @@ async def test_create_list_and_read_workspace_files(self):
             # Create a file
             file_content = "Hello, workspace!"
             await session.rpc.workspace.create_file(
-                SessionWorkspaceCreateFileParams(content=file_content, path="test.txt")
+                WorkspaceCreateFileRequest(content=file_content, path="test.txt")
             )
 
             # List files
@@ -219,13 +219,13 @@ async def test_create_list_and_read_workspace_files(self):
 
             # Read file
             read_result = await session.rpc.workspace.read_file(
-                SessionWorkspaceReadFileParams(path="test.txt")
+                WorkspaceReadFileRequest(path="test.txt")
             )
             assert read_result.content == file_content
 
             # Create nested file
             await session.rpc.workspace.create_file(
-                SessionWorkspaceCreateFileParams(content="Nested content", path="subdir/nested.txt")
+                WorkspaceCreateFileRequest(content="Nested content", path="subdir/nested.txt")
             )
 
             after_nested = await session.rpc.workspace.list_files()

nodejs/src/generated/rpc.ts

@@ -6,14 +6,14 @@
 
 from copilot.generated.rpc import (
     FleetApi,
+    FleetStartRequest,
     ModeApi,
     ModeSetRequest,
     PlanApi,
     ServerModelsApi,
     ServerToolsApi,
-    SessionFleetStartParams,
     SessionMode,
-    ToolsListParams,
+    ToolsListRequest,
 )
 
 
@@ -33,7 +33,7 @@ async def test_default_timeout_not_forwarded(self):
         client.request = AsyncMock(return_value={"started": True})
         api = FleetApi(client, "sess-1")
 
-        await api.start(SessionFleetStartParams(prompt="go"))
+        await api.start(FleetStartRequest(prompt="go"))
 
         client.request.assert_called_once()
         _, kwargs = client.request.call_args
@@ -45,7 +45,7 @@ async def test_custom_timeout_forwarded(self):
         client.request = AsyncMock(return_value={"started": True})
         api = FleetApi(client, "sess-1")
 
-        await api.start(SessionFleetStartParams(prompt="go"), timeout=600.0)
+        await api.start(FleetStartRequest(prompt="go"), timeout=600.0)
 
         _, kwargs = client.request.call_args
         assert kwargs["timeout"] == 600.0
@@ -93,7 +93,7 @@ async def test_timeout_on_server_params_method(self):
         client.request = AsyncMock(return_value={"tools": []})
         api = ServerToolsApi(client)
 
-        await api.list(ToolsListParams(), timeout=60.0)
+        await api.list(ToolsListRequest(), timeout=60.0)
 
         _, kwargs = client.request.call_args
         assert kwargs["timeout"] == 60.0
@@ -104,7 +104,7 @@ async def test_default_timeout_on_server_params_method(self):
         client.request = AsyncMock(return_value={"tools": []})
         api = ServerToolsApi(client)
 
-        await api.list(ToolsListParams())
+        await api.list(ToolsListRequest())
 
         _, kwargs = client.request.call_args
         assert "timeout" not in kwargs

python/copilot/generated/rpc.py

@@ -8,3 +8,58 @@ conversations:
         content: What is 2+2?
       - role: assistant
         content: 2+2 = 4
+      - role: user
+        content: ${compaction_prompt}
+      - role: assistant
+        content: >-
+          <overview>
+
+          The user asked a simple arithmetic question: "What is 2+2?" This was answered directly (4). No technical work,
+          coding tasks, or file modifications were performed. This appears to have been a minimal test interaction with
+          no substantive goals or technical requirements.
+
+          </overview>
+
+
+          <history>
+
+          1. The user asked "What is 2+2?"
+             - Provided the answer: 4
+             - No further actions were taken or requested
+          </history>
+
+
+          <work_done>
+
+          No files were created, modified, or deleted. No technical tasks were performed. The conversation consisted
+          solely of answering a basic arithmetic question.
+
+
+          Current state: No active work in progress.
+
+          </work_done>
+
+
+          <technical_details>
+
+          No technical concepts, decisions, or issues were encountered in this conversation. This was a straightforward
+          arithmetic question with no technical context.
+
+          </technical_details>
+
+
+          <important_files>
+
+          No files were involved in this conversation.
+
+          </important_files>
+
+
+          <next_steps>
+
+          No pending work or tasks. The user's question was fully addressed. Awaiting new requests or instructions.
+
+          </next_steps>
+
+
+          <checkpoint_title>Simple arithmetic question answered</checkpoint_title>

python/e2e/test_agent_and_compact_rpc.py

@@ -8,3 +8,62 @@ conversations:
         content: What is 2+2?
       - role: assistant
         content: 2 + 2 = 4
+      - role: user
+        content: ${compaction_prompt}
+      - role: assistant
+        content: >-
+          <overview>
+
+          The user asked a simple arithmetic question ("What is 2+2?") which I answered correctly (4). No coding work,
+          file modifications, or technical implementation was requested or performed. This appears to be a minimal test
+          interaction before the conversation history is compacted.
+
+          </overview>
+
+
+          <history>
+
+          1. The user asked "What is 2+2?"
+             - I provided the arithmetic answer: 4
+             - No follow-up questions or additional requests were made
+
+          2. The user requested a detailed summary for conversation compaction
+             - Currently preparing this checkpoint summary
+          </history>
+
+
+          <work_done>
+
+          No files were created, modified, or deleted. No code changes were made. No tasks were assigned or completed
+          beyond answering a basic arithmetic question.
+
+
+          Current state: No active work in progress. The conversation consisted only of a single question and answer
+          exchange.
+
+          </work_done>
+
+
+          <technical_details>
+
+          No technical work was performed. No issues were encountered. No architectural decisions were made. No code was
+          explored or modified.
+
+          </technical_details>
+
+
+          <important_files>
+
+          No files were accessed or are relevant to this conversation.
+
+          </important_files>
+
+
+          <next_steps>
+
+          No pending work. No tasks were assigned. The user may continue with new requests after the history compaction.
+
+          </next_steps>
+
+
+          <checkpoint_title>Answered arithmetic question</checkpoint_title>