From a526f0df065be6c86194c392eabb89c41c49af90 Mon Sep 17 00:00:00 2001 From: 35C4n0r Date: Tue, 9 Dec 2025 15:11:46 +0530 Subject: [PATCH] feat: temporarily remove report_task tool call logging --- lib/httpapi/server.go | 2 +- lib/msgfmt/message_box.go | 7 +------ lib/msgfmt/msgfmt.go | 9 ++++----- lib/msgfmt/msgfmt_test.go | 3 +-- 4 files changed, 7 insertions(+), 14 deletions(-) diff --git a/lib/httpapi/server.go b/lib/httpapi/server.go index 9d8a0412..18f2bf40 100644 --- a/lib/httpapi/server.go +++ b/lib/httpapi/server.go @@ -226,7 +226,7 @@ func NewServer(ctx context.Context, config ServerConfig) (*Server, error) { humaConfig.Info.Description = "HTTP API for Claude Code, Goose, and Aider.\n\nhttps://github.com/coder/agentapi" api := humachi.New(router, humaConfig) formatMessage := func(message string, userInput string) string { - return mf.FormatAgentMessage(config.AgentType, message, userInput, logger) + return mf.FormatAgentMessage(config.AgentType, message, userInput) } isAgentReadyForInitialPrompt := func(message string) bool { diff --git a/lib/msgfmt/message_box.go b/lib/msgfmt/message_box.go index 53d9925f..d7370c60 100644 --- a/lib/msgfmt/message_box.go +++ b/lib/msgfmt/message_box.go @@ -1,7 +1,6 @@ package msgfmt import ( - "log/slog" "strings" ) @@ -102,7 +101,7 @@ func removeAmpMessageBox(msg string) string { return formattedMsg } -func removeClaudeReportTaskToolCall(msg string, logger *slog.Logger) string { +func removeClaudeReportTaskToolCall(msg string) string { // Remove all tool calls that start with `● coder - coder_report_task (MCP)` till we encounter the next line starting with ● lines := strings.Split(msg, "\n") @@ -146,10 +145,6 @@ func removeClaudeReportTaskToolCall(msg string, logger *slog.Logger) string { idxPair := toolCallIdxs[i] start, end := idxPair[0], idxPair[1] - // Capture the tool call content before removing it - toolCallContent := strings.Join(lines[start:end], "\n") - logger.Info("Removing tool call", "content", toolCallContent) - lines = append(lines[:start], lines[end:]...) } diff --git a/lib/msgfmt/msgfmt.go b/lib/msgfmt/msgfmt.go index ebe8805d..975dc7f6 100644 --- a/lib/msgfmt/msgfmt.go +++ b/lib/msgfmt/msgfmt.go @@ -1,7 +1,6 @@ package msgfmt import ( - "log/slog" "strings" ) @@ -255,10 +254,10 @@ func formatGenericMessage(message string, userInput string, agentType AgentType) return message } -func formatClaudeMessage(message string, userInput string, logger *slog.Logger) string { +func formatClaudeMessage(message string, userInput string) string { message = RemoveUserInput(message, userInput, AgentTypeClaude) message = removeMessageBox(message) - message = removeClaudeReportTaskToolCall(message, logger) + message = removeClaudeReportTaskToolCall(message) message = trimEmptyLines(message) return message } @@ -284,10 +283,10 @@ func formatAmpMessage(message string, userInput string) string { return message } -func FormatAgentMessage(agentType AgentType, message string, userInput string, logger *slog.Logger) string { +func FormatAgentMessage(agentType AgentType, message string, userInput string) string { switch agentType { case AgentTypeClaude: - return formatClaudeMessage(message, userInput, logger) + return formatClaudeMessage(message, userInput) case AgentTypeGoose: return formatGenericMessage(message, userInput, agentType) case AgentTypeAider: diff --git a/lib/msgfmt/msgfmt_test.go b/lib/msgfmt/msgfmt_test.go index 92dc6b3e..780a3954 100644 --- a/lib/msgfmt/msgfmt_test.go +++ b/lib/msgfmt/msgfmt_test.go @@ -2,7 +2,6 @@ package msgfmt import ( "embed" - "log/slog" "path" "strings" "testing" @@ -234,7 +233,7 @@ func TestFormatAgentMessage(t *testing.T) { assert.NoError(t, err) expected, err := testdataDir.ReadFile(path.Join(dir, string(agentType), c.Name(), "expected.txt")) assert.NoError(t, err) - assert.Equal(t, string(expected), FormatAgentMessage(agentType, string(msg), string(userInput), slog.Default())) + assert.Equal(t, string(expected), FormatAgentMessage(agentType, string(msg), string(userInput))) }) } })