From fd80780581a2beb5a63cf5db8864ceb65006b252 Mon Sep 17 00:00:00 2001
From: Codex <codex@local>
Date: Sun, 24 May 2026 03:10:30 +0000
Subject: [PATCH] Trim command tool summaries

---
 internal/telegram/bot.go         | 21 ++++-----------------
 internal/telegram/render_test.go |  2 +-
 2 files changed, 5 insertions(+), 18 deletions(-)

diff --git a/internal/telegram/bot.go b/internal/telegram/bot.go
index 173fd33..0dbbea7 100644
--- a/internal/telegram/bot.go
+++ b/internal/telegram/bot.go
@@ -23,7 +23,6 @@ import (
 const (
 	telegramDownloadLimit              = 20 * 1024 * 1024
 	resumeThreadPageSize               = 8
-	commandSummaryLimit                = 120
 	telegramPhotoDirectiveStart        = "<!-- telegram-photo "
 	telegramThreadRenameDirectiveStart = "<!-- codex-thread-rename "
 	telegramThreadCWDDirectiveStart    = "<!-- codex-thread-cwd "
@@ -1194,7 +1193,7 @@ func parseCodexThreadItem(raw json.RawMessage) (codexThreadItemView, error) {
 func renderCodexItemStarted(item codexThreadItemView) string {
 	switch item.Type {
 	case "commandExecution":
-		return SummaryDetailsRawHTMLLimited(joinNonEmpty("Tool call: command started", commandSummaryLine(item.Command)), commandStartedDetailsHTML(item), TelegramHTMLMessageLimit)
+		return SummaryDetailsRawHTMLLimited("Tool call: command started", commandStartedDetailsHTML(item), TelegramHTMLMessageLimit)
 	case "fileChange":
 		return "Tool call: file change started"
 	case "mcpToolCall":
@@ -1221,7 +1220,7 @@ func renderCodexItemCompleted(item codexThreadItemView) string {
 		if item.ExitCode != nil {
 			status = fmt.Sprintf("Exit code: %d", *item.ExitCode)
 		}
-		return SummaryDetailsRawHTMLLimited(joinNonEmpty("Tool call: command finished", commandSummaryLine(item.Command), status), renderCodexItemDetailsHTML(item), TelegramHTMLMessageLimit)
+		return SummaryDetailsRawHTMLLimited(joinNonEmpty("Tool call: command finished", status), renderCodexItemDetailsHTML(item), TelegramHTMLMessageLimit)
 	case "fileChange":
 		return joinNonEmpty("Tool call: file change finished", fmt.Sprintf("Changed files: %d", len(item.Changes)), "Status: "+item.Status)
 	case "mcpToolCall":
@@ -1247,21 +1246,9 @@ func renderCodexItemCompleted(item codexThreadItemView) string {
 	}
 }
 
-func commandSummaryLine(command string) string {
-	command = strings.TrimSpace(command)
-	if command == "" {
-		return ""
-	}
-	runes := []rune(command)
-	if len(runes) <= commandSummaryLimit {
-		return "Command: " + command
-	}
-	return "Command: " + string(runes[:commandSummaryLimit]) + "..."
-}
-
 func commandStartedDetailsHTML(item codexThreadItemView) string {
 	var parts []string
-	if command := strings.TrimSpace(item.Command); command != "" && len([]rune(command)) > commandSummaryLimit {
+	if command := strings.TrimSpace(item.Command); command != "" {
 		parts = append(parts, "<b>Command</b>", CodeBlockHTML("bash", command))
 	}
 	if cwd := strings.TrimSpace(item.CWD); cwd != "" {
@@ -1296,7 +1283,7 @@ func renderCodexItemDetailsHTML(item codexThreadItemView) string {
 	switch item.Type {
 	case "commandExecution":
 		appendField("CWD", item.CWD)
-		if command := strings.TrimSpace(item.Command); command != "" && len([]rune(command)) > commandSummaryLimit {
+		if command := strings.TrimSpace(item.Command); command != "" {
 			parts = append(parts, "<b>Command</b>", CodeBlockHTML("bash", command))
 		}
 		appendInt("Exit code", item.ExitCode)
diff --git a/internal/telegram/render_test.go b/internal/telegram/render_test.go
index 4987eb9..661ecb7 100644
--- a/internal/telegram/render_test.go
+++ b/internal/telegram/render_test.go
@@ -156,7 +156,7 @@ func TestRenderCodexCommandExecutionItem(t *testing.T) {
 		ExitCode:         &exitCode,
 	}
 	text := renderCodexItemCompleted(item)
-	for _, want := range []string{"Tool call: command finished", "Command: go test ./...", "Exit code: 0", "<pre><code class=\"language-text\">line 1\nline 2</code></pre>"} {
+	for _, want := range []string{"Tool call: command finished", "<b>Command</b>", "<pre><code class=\"language-bash\">go test ./...</code></pre>", "Exit code: 0", "<pre><code class=\"language-text\">line 1\nline 2</code></pre>"} {
 		if !strings.Contains(text, want) {
 			t.Fatalf("rendered command item missing %q in %q", want, text)
 		}