Fix summarize to new chat in local-agent mode (#2294)

e2e-tests/local_agent_summarize.spec.ts

+import { testSkipIfWindows } from "./helpers/test_helper";
+import { expect } from "@playwright/test";
+
+/**
+ * E2E test for summarization in local-agent mode
+ * Tests that summarize to new chat works correctly when using Agent v2
+ * Regression test for #2292
+ *
+ * This test directly triggers summarization by sending the "Summarize from chat-id=X"
+ * prompt, which is the same mechanism used by the "Summarize into new chat" button.
+ */
+testSkipIfWindows(
+  "local-agent - summarize to new chat works",
+  async ({ po }) => {
+    await po.setUpDyadPro({ localAgent: true });
+    await po.importApp("minimal");
+    await po.selectLocalAgentMode();
+
+    // First, send a message to create a chat with some content
+    // This simulates a chat with technical discussion
+    await po.sendPrompt("tc=local-agent/read-then-edit");
+
+    // Get the current chat URL to extract the chat ID
+    const url = po.page.url();
+    const chatIdMatch = url.match(/[?&]id=(\d+)/);
+    expect(chatIdMatch).toBeTruthy();
+    const originalChatId = chatIdMatch![1];
+
+    // Create a new chat by clicking the "New Chat" button
+    await po.clickNewChat();
+
+    // Now trigger summarization by sending the summarize prompt
+    // This is the same mechanism used by the "Summarize into new chat" button
+    await po.sendPrompt(`Summarize from chat-id=${originalChatId}`);
+
+    // Snapshot the messages in the new chat
+    // This verifies that the summarization actually ran and produced content
+    // (Before the fix, this would fail with "no technical discussion" error
+    // because the local agent handler wasn't receiving the formatted chat content)
+    await po.snapshotMessages();
+  },
+);

e2e-tests/snapshots/local_agent_summarize.spec.ts_local-agent---summarize-to-new-chat-works-1.aria.yml

+- paragraph: Summarize from chat-id=1
+- img
+- text: file1.txt
+- button "Edit":
+  - img
+- img
+- text: file1.txt
+- paragraph: More EOM
+- button:
+  - img
+- img
+- text: claude-opus-4-5
+- img
+- text: less than a minute ago
+- button "Request ID":
+  - img
+- button "Undo":
+  - img
+- button "Retry":
+  - img
\ No newline at end of file

src/ipc/handlers/chat_stream_handlers.ts

@@ -1020,9 +1020,16 @@ This conversation includes one or more image attachments. When the user uploads
 
           await handleLocalAgentStream(event, req, abortController, {
             placeholderMessageId: placeholderAssistantMessage.id,
+            // Note: this is using the read-only system prompt rather than the
+            // regular system prompt which gets overrides for special intents
+            // like summarize chat, security review, etc.
+            //
+            // This is OK because those intents should always happen in a new chat
+            // and new chats will default to non-ask modes.
             systemPrompt: readOnlySystemPrompt,
             dyadRequestId: dyadRequestId ?? "[no-request-id]",
             readOnly: true,
+            messageOverride: isSummarizeIntent ? chatMessages : undefined,
           });
           return;
         }
@@ -1038,6 +1045,7 @@ This conversation includes one or more image attachments. When the user uploads
             placeholderMessageId: placeholderAssistantMessage.id,
             systemPrompt,
             dyadRequestId: dyadRequestId ?? "[no-request-id]",
+            messageOverride: isSummarizeIntent ? chatMessages : undefined,
           });
           return;
         }

src/pro/main/ipc/handlers/local_agent/local_agent_handler.ts

@@ -108,6 +108,7 @@ export async function handleLocalAgentStream(
     systemPrompt,
     dyadRequestId,
     readOnly = false,
+    messageOverride,
   }: {
     placeholderMessageId: number;
     systemPrompt: string;
@@ -117,6 +118,11 @@ export async function handleLocalAgentStream(
      * State-modifying tools are disabled, and no commits/deploys are made.
      */
     readOnly?: boolean;
+    /**
+     * If provided, use these messages instead of fetching from the database.
+     * Used for summarization where messages need to be transformed.
+     */
+    messageOverride?: ModelMessage[];
   },
 ): Promise<void> {
   const settings = readSettings();
@@ -229,7 +235,10 @@ export async function handleLocalAgentStream(
     const allTools: ToolSet = { ...agentTools, ...mcpTools };
 
     // Prepare message history with graceful fallback
-    const messageHistory: ModelMessage[] = chat.messages
+    // Use messageOverride if provided (e.g., for summarization)
+    const messageHistory: ModelMessage[] = messageOverride
+      ? messageOverride
+      : chat.messages
           .filter((msg) => msg.content || msg.aiMessagesJson)
           .flatMap((msg) => parseAiMessagesJson(msg));