address comments

Amp-Thread-ID: https://ampcode.com/threads/T-019c548c-70ef-76f5-b64d-3dca51658b9f
Co-authored-by: Amp <amp@ampcode.com>

Author: baxen

crates/goose/src/agents/agent.rs

@@ -13,9 +13,6 @@ use super::final_output_tool::FinalOutputTool;
 use super::platform_tools;
 use super::tool_execution::{ToolCallResult, CHAT_MODE_TOOL_SKIPPED_RESPONSE, DECLINED_RESPONSE};
 use crate::action_required_manager::ActionRequiredManager;
-use crate::agents::code_summary_tool::{
-    create_summarize_tool, handle_summarize_tool, SUMMARIZE_TOOL_NAME,
-};
 use crate::agents::extension::{ExtensionConfig, ExtensionResult, ToolInfo};
 use crate::agents::extension_manager::{get_parameter_names, ExtensionManager};
 use crate::agents::extension_manager_extension::MANAGE_EXTENSIONS_TOOL_NAME_COMPLETE;
@@ -511,34 +508,7 @@ impl Agent {
         }
 
         debug!("WAITING_TOOL_START: {}", tool_call.name);
-        let result: ToolCallResult = if tool_call.name == SUMMARIZE_TOOL_NAME {
-            let provider = match self.provider().await {
-                Ok(p) => p,
-                Err(e) => {
-                    return (
-                        request_id,
-                        Err(ErrorData::new(
-                            ErrorCode::INTERNAL_ERROR,
-                            format!("Failed to get provider: {}", e),
-                            None,
-                        )),
-                    );
-                }
-            };
-
-            let arguments = tool_call
-                .arguments
-                .clone()
-                .map(Value::Object)
-                .unwrap_or(Value::Object(serde_json::Map::new()));
-
-            handle_summarize_tool(
-                provider,
-                session.id.clone(),
-                arguments,
-                session.working_dir.clone(),
-            )
-        } else if self.is_frontend_tool(&tool_call.name).await {
+        let result: ToolCallResult = if self.is_frontend_tool(&tool_call.name).await {
             // For frontend tools, return an error indicating we need frontend execution
             ToolCallResult::from(Err(ErrorData::new(
                 ErrorCode::INTERNAL_ERROR,
@@ -790,7 +760,7 @@ impl Agent {
                 prefixed_tools.push(final_output_tool.tool());
             }
 
-            prefixed_tools.push(create_summarize_tool());
+
         }
 
         prefixed_tools