test message

Author: Patel230

cmd/iterate/repl.go

@@ -131,40 +131,19 @@ func replHooks() iteragent.AgentHooks {
 	}
 }
 
-// ctrlCExitCh is closed when the user presses Ctrl+C while idle (no active request).
-// The REPL loop listens on this channel to exit cleanly.
-var ctrlCExitCh = make(chan struct{}, 1)
-
 // initREPL loads config, applies theme, sets up signal handling and runtime state.
 func setupSigintHandler() {
 	sigCh := make(chan os.Signal, 1)
 	signal.Notify(sigCh, syscall.SIGINT)
 	go func() {
-		var lastIdle time.Time
 		for range sigCh {
 			if sess.RequestCancel != nil {
-				// Active request — cancel it.
 				sess.RequestCancel()
+				// Snapshot colors under read lock — applyTheme writes these from the main goroutine.
 				colorMu.RLock()
 				y, r := colorYellow, colorReset
 				colorMu.RUnlock()
 				fmt.Printf("\r\033[K%s[cancelled]%s\n", y, r)
-				lastIdle = time.Time{} // reset idle timer after cancel
-			} else {
-				// Idle — double-Ctrl+C to exit.
-				now := time.Now()
-				if !lastIdle.IsZero() && now.Sub(lastIdle) < 2*time.Second {
-					select {
-					case ctrlCExitCh <- struct{}{}:
-					default:
-					}
-				} else {
-					lastIdle = now
-					colorMu.RLock()
-					d, r := colorDim, colorReset
-					colorMu.RUnlock()
-					fmt.Printf("\r\033[K%s(press Ctrl+C again to exit)%s\n", d, r)
-				}
 			}
 		}
 	}()
@@ -242,20 +221,8 @@ func runREPL(ctx context.Context, p iteragent.Provider, repoPath string, thinkin
 	}
 
 	for {
-		// Check for double-Ctrl+C exit signal before blocking on ReadInput.
-		select {
-		case <-ctrlCExitCh:
-			return
-		default:
-		}
-
 		line, ok := selector.ReadInput()
 		if !ok {
-			// Check if this was a double-Ctrl+C exit.
-			select {
-			case <-ctrlCExitCh:
-			default:
-			}
 			break
 		}
 		if line == "" {

internal/commands/newfeatures_test.go

@@ -306,6 +306,25 @@ func TestCmdTrim_KeepsLastTurns(t *testing.T) {
 	}
 }
 
+// TestCmdTrim_NoUserMessageInTail covers the out-of-bounds case where every
+// message in the kept tail has a non-user role (pathological but must not panic).
+func TestCmdTrim_NoUserMessageInTail(t *testing.T) {
+	ag := &iteragent.Agent{}
+	// 10 assistant-only messages — no user messages anywhere.
+	for i := 0; i < 10; i++ {
+		ag.Messages = append(ag.Messages, iteragent.Message{Role: "assistant", Content: "a"})
+	}
+	ctx := Context{
+		Parts: []string{"/trim", "2"},
+		Agent: ag,
+	}
+	// Must not panic.
+	result := cmdTrim(ctx)
+	if !result.Handled {
+		t.Error("expected handled")
+	}
+}
+
 func TestCmdTrim_AlreadySmall(t *testing.T) {
 	ag := &iteragent.Agent{}
 	ag.Messages = []iteragent.Message{

internal/commands/session.go

@@ -474,9 +474,14 @@ func cmdTrim(ctx Context) Result {
 
 	// Align to a user-message boundary so we always start on a user turn.
 	startIdx := len(msgs) - keep
-	for startIdx > 0 && msgs[startIdx].Role != "user" {
+	for startIdx > 0 && startIdx < len(msgs) && msgs[startIdx].Role != "user" {
 		startIdx++
 	}
+	if startIdx >= len(msgs) {
+		// Pathological case: no user message found in the tail — keep everything.
+		fmt.Printf("Context already at minimum (%d messages).\n", len(msgs))
+		return Result{Handled: true}
+	}
 
 	before := len(msgs)
 	ctx.Agent.Messages = msgs[startIdx:]