feat(telegram): complete narrator integration with progress messages

- Wire narrator into handleChatMessage for engaging mode
- Send instant '🤔 Looking into that...' progress message
- Update progress message on each tool_call with narrated descriptions
- Delete progress message when answer arrives (clean chat)
- Add InteractionMode config tests (default, env, CLI override)
- Add /mode command test verifying interaction_mode documentation

Author: molty3000

cmd/odek/telegram.go

@@ -21,6 +21,7 @@ import (
 	"github.com/BackendStack21/kode/internal/llm"
 	"github.com/BackendStack21/kode/internal/loop"
 	"github.com/BackendStack21/kode/internal/render"
+	"github.com/BackendStack21/kode/internal/narrate"
 	"github.com/BackendStack21/kode/internal/session"
 	"github.com/BackendStack21/kode/internal/skills"
 	"github.com/BackendStack21/kode/internal/telegram"
@@ -987,11 +988,38 @@ func handleChatMessage(
 			}
 		}
 	}()
+	
+	// ── Engaging Mode: Immediate Progress ──────────────────────────
+	// Send an instant "working on it" message so the user sees feedback
+	// within milliseconds, not seconds. This message gets updated with
+	// narrator-style progress on each tool call, then deleted when the
+	// final answer arrives.
+	var narrator *narrate.Narrator
+	isEngaging := resolved.InteractionMode != "verbose"
+	if isEngaging {
+		narrator = narrate.New(true)
+	}
+	var progressMsgID int
+	if isEngaging {
+		msg, err := bot.SendMessage(chatID, "🤔 Looking into that...",
+			&telegram.SendOpts{ReplyToMessageID: messageID})
+		if err == nil {
+			progressMsgID = msg.ID
+		}
+	}
+	defer func() {
+		// Clean up the progress message when the task finishes.
+		// The final answer replaces it entirely — this avoids cluttering
+		// the chat with a stale "thinking" message.
+		if progressMsgID != 0 {
+			bot.DeleteMessage(chatID, progressMsgID)
+		}
+	}()
 
 	// ── Tool Tracing ───────────────────────────────────────────────
 	// Single editable message showing live tool execution progress.
-	// The message is created lazily — only when the first tool call
-	// fires, not before. This avoids the premature "🤔 Thinking…" spam.
+	// In verbose mode this shows raw tool names/results; in engaging
+	// mode it's replaced by the narrator progress message above.
 	var traceMsgID int
 	var traceMu sync.Mutex
 	traceLines := make([]string, 0, 8)
@@ -1064,8 +1092,16 @@ func handleChatMessage(
 		Tools:           agentTools,
 		Renderer:        rend,
 		ToolEventHandler: func(event string, name string, data string) {
-			// In engaging mode, skip the raw tool trace — narrator handles it.
+			// Engaging mode: update the progress message with narrated tool
+			// descriptions instead of raw traces. We skip tool_result events
+			// here — the next tool_call will overwrite the message anyway,
+			// keeping the chat scannable and avoiding flash edits.
 			if resolved.InteractionMode != "verbose" {
+				if progressMsgID != 0 && event == "tool_call" && narrator != nil {
+					if msg := narrator.ToolCallMessage(name, data); msg != "" {
+						bot.EditMessageText(chatID, progressMsgID, msg, nil)
+					}
+				}
 				return
 			}
 			traceMu.Lock()

cmd/odek/telegram_test.go

@@ -773,3 +773,43 @@ func newTestHandler(bot *telegram.Bot) *telegram.Handler {
 	h.Config = telegram.HandlerConfig{}
 	return h
 }
+
+// ── /mode command tests ──────────────────────────────────────────────
+
+func TestOnCommandMode_ShowsInteractionMode(t *testing.T) {
+	// /mode must document interaction_mode (engaging and verbose).
+	chatID := int64(99920)
+
+	bot := newTestBot(t)
+	h := newTestHandler(bot)
+
+	h.OnCommand = func(cid int64, mid int, cmdName string, argsStr string) (string, error) {
+		if cmdName == "mode" {
+			return "⚙️ *Agent Modes*\n\n" +
+				"Modes are set at startup via `odek.json` or CLI flags:\n" +
+				"• `interaction_mode: engaging` — emoji-rich narration (default)\n" +
+				"• `interaction_mode: verbose` — raw tool call output\n" +
+				"• `sandbox: true` — run in Docker isolation\n" +
+				"• `skills.verbose: true` — show skill learning details\n\n" +
+				"Restart the bot after changing config.", nil
+		}
+		return "", nil
+	}
+
+	result, err := h.OnCommand(chatID, 0, "mode", "")
+	if err != nil {
+		t.Fatalf("OnCommand /mode returned error: %v", err)
+	}
+
+	checks := []string{
+		"interaction_mode",
+		"engaging",
+		"verbose",
+		"Agent Modes",
+	}
+	for _, c := range checks {
+		if !strings.Contains(result, c) {
+			t.Errorf("expected /mode output to contain %q, got: %q", c, result)
+		}
+	}
+}

internal/config/loader_test.go

@@ -717,3 +717,34 @@ func TestLoadConfig_ClearsAPIKeyFromEnviron(t *testing.T) {
 		t.Errorf("OPENAI_API_KEY should be cleared after LoadConfig, got %q", v)
 	}
 }
+
+func TestLoadConfig_InteractionModeDefaults(t *testing.T) {
+	// When no interaction_mode is configured, the resolved config must
+	// default to "engaging".
+	cfg := LoadConfig(CLIFlags{})
+	if cfg.InteractionMode != "engaging" {
+		t.Errorf("InteractionMode = %q, want %q", cfg.InteractionMode, "engaging")
+	}
+}
+
+func TestLoadConfig_InteractionModeViaEnv(t *testing.T) {
+	// ODEK_INTERACTION_MODE should override the default.
+	os.Setenv("ODEK_INTERACTION_MODE", "verbose")
+	defer os.Unsetenv("ODEK_INTERACTION_MODE")
+
+	cfg := LoadConfig(CLIFlags{})
+	if cfg.InteractionMode != "verbose" {
+		t.Errorf("InteractionMode = %q, want %q", cfg.InteractionMode, "verbose")
+	}
+}
+
+func TestLoadConfig_InteractionModeViaCLI(t *testing.T) {
+	// CLI flag should take precedence over env.
+	os.Setenv("ODEK_INTERACTION_MODE", "engaging")
+	defer os.Unsetenv("ODEK_INTERACTION_MODE")
+
+	cfg := LoadConfig(CLIFlags{InteractionMode: "verbose"})
+	if cfg.InteractionMode != "verbose" {
+		t.Errorf("InteractionMode = %q, want %q", cfg.InteractionMode, "verbose")
+	}
+}