From 653267a64f8cd3b270176ec90b37a34383bf5bf2 Mon Sep 17 00:00:00 2001
From: Tommaso Sciortino <sciortino@gmail.com>
Date: Wed, 20 Aug 2025 16:13:50 -0700
Subject: [PATCH] Remove unused attribute (#6661)

---
 packages/cli/src/ui/hooks/useToolScheduler.test.ts | 7 -------
 packages/core/src/tools/tools.test.ts              | 1 -
 packages/core/src/tools/tools.ts                   | 5 -----
 3 files changed, 13 deletions(-)

diff --git a/packages/cli/src/ui/hooks/useToolScheduler.test.ts b/packages/cli/src/ui/hooks/useToolScheduler.test.ts
index 8e060ee8..74c9fe32 100644
--- a/packages/cli/src/ui/hooks/useToolScheduler.test.ts
+++ b/packages/cli/src/ui/hooks/useToolScheduler.test.ts
@@ -194,7 +194,6 @@ describe('useReactToolScheduler in YOLO Mode', () => {
     (mockToolRequiresConfirmation.execute as Mock).mockResolvedValue({
       llmContent: expectedOutput,
       returnDisplay: 'YOLO Formatted tool output',
-      summary: 'YOLO summary',
     } as ToolResult);
 
     const { result } = renderSchedulerInYoloMode();
@@ -352,7 +351,6 @@ describe('useReactToolScheduler', () => {
     (mockTool.execute as Mock).mockResolvedValue({
       llmContent: 'Tool output',
       returnDisplay: 'Formatted tool output',
-      summary: 'Formatted summary',
     } as ToolResult);
     (mockTool.shouldConfirmExecute as Mock).mockResolvedValue(null);
 
@@ -517,7 +515,6 @@ describe('useReactToolScheduler', () => {
     (mockToolRequiresConfirmation.execute as Mock).mockResolvedValue({
       llmContent: expectedOutput,
       returnDisplay: 'Confirmed display',
-      summary: 'Confirmed summary',
     } as ToolResult);
 
     const { result } = renderScheduler();
@@ -684,7 +681,6 @@ describe('useReactToolScheduler', () => {
       resolveExecutePromise({
         llmContent: 'Final output',
         returnDisplay: 'Final display',
-        summary: 'Final summary',
       } as ToolResult);
     });
     await act(async () => {
@@ -718,7 +714,6 @@ describe('useReactToolScheduler', () => {
     tool1.execute.mockResolvedValue({
       llmContent: 'Output 1',
       returnDisplay: 'Display 1',
-      summary: 'Summary 1',
     } as ToolResult);
     tool1.shouldConfirmExecute.mockResolvedValue(null);
 
@@ -726,7 +721,6 @@ describe('useReactToolScheduler', () => {
     tool2.execute.mockResolvedValue({
       llmContent: 'Output 2',
       returnDisplay: 'Display 2',
-      summary: 'Summary 2',
     } as ToolResult);
     tool2.shouldConfirmExecute.mockResolvedValue(null);
 
@@ -809,7 +803,6 @@ describe('useReactToolScheduler', () => {
           resolve({
             llmContent: 'done',
             returnDisplay: 'done display',
-            summary: 'done summary',
           }),
         50,
       ),
diff --git a/packages/core/src/tools/tools.test.ts b/packages/core/src/tools/tools.test.ts
index a98d1121..f9828010 100644
--- a/packages/core/src/tools/tools.test.ts
+++ b/packages/core/src/tools/tools.test.ts
@@ -101,7 +101,6 @@ describe('DeclarativeTool', () => {
       const successResult: ToolResult = {
         llmContent: 'Success!',
         returnDisplay: 'Success!',
-        summary: 'Tool executed successfully',
       };
       const executeFn = vi.fn().mockResolvedValue(successResult);
       const invocation = new TestToolInvocation({}, executeFn);
diff --git a/packages/core/src/tools/tools.ts b/packages/core/src/tools/tools.ts
index 64a2e83a..58773d25 100644
--- a/packages/core/src/tools/tools.ts
+++ b/packages/core/src/tools/tools.ts
@@ -308,11 +308,6 @@ export abstract class BaseDeclarativeTool<
 export type AnyDeclarativeTool = DeclarativeTool<object, ToolResult>;
 
 export interface ToolResult {
-  /**
-   * A short, one-line summary of the tool's action and result.
-   * e.g., "Read 5 files", "Wrote 256 bytes to foo.txt"
-   */
-  summary?: string;
   /**
    * Content meant to be included in LLM history.
    * This should represent the factual outcome of the tool execution.