feat(session): add non-interactive mode flag to distinguish from tools approval

- Add `NonInteractive` field to Session struct to explicitly mark sessions running without user interaction
- Update runtime loop to use `NonInteractive` instead of `ToolsApproved` for auto-stopping logic
- Set non-interactive mode in A2A adapter, evaluation framework, and MCP server
- This separates the concept of approved tools from non-interactive execution context

Author: tdabasinskas

pkg/a2a/adapter.go

@@ -49,6 +49,7 @@ func runDockerAgent(ctx agent.InvocationContext, t *team.Team, agentName string,
 			session.WithMaxConsecutiveToolCalls(a.MaxConsecutiveToolCalls()),
 			session.WithMaxOldToolCallTokens(a.MaxOldToolCallTokens()),
 			session.WithToolsApproved(true),
+			session.WithNonInteractive(true),
 		)
 
 		// Create runtime

pkg/evaluation/save.go

@@ -59,6 +59,7 @@ func SessionFromEvents(events []map[string]any, title string, questions []string
 	sess := session.New(
 		session.WithTitle(title),
 		session.WithToolsApproved(true),
+		session.WithNonInteractive(true),
 	)
 
 	// Add user questions as initial messages.

pkg/mcp/server.go

@@ -164,6 +164,7 @@ func CreateToolHandler(t *team.Team, agentName string) func(context.Context, *mc
 			session.WithMaxOldToolCallTokens(ag.MaxOldToolCallTokens()),
 			session.WithUserMessage(input.Message),
 			session.WithToolsApproved(true),
+			session.WithNonInteractive(true),
 		)
 
 		rt, err := runtime.New(t,

pkg/runtime/loop.go

@@ -177,7 +177,7 @@ func (r *LocalRuntime) RunStream(ctx context.Context, sess *session.Session) <-c
 
 				// In non-interactive mode (e.g. MCP server), auto-stop instead of
 				// blocking forever waiting for user input.
-				if sess.ToolsApproved {
+				if sess.NonInteractive {
 					slog.Debug("Auto-stopping after max iterations (non-interactive)", "agent", a.Name())
 
 					assistantMessage := chat.Message{

pkg/session/session.go

@@ -76,6 +76,12 @@ type Session struct {
 	// ToolsApproved is a flag to indicate if the tools have been approved
 	ToolsApproved bool `json:"tools_approved"`
 
+	// NonInteractive indicates the session is running in a non-interactive context
+	// (e.g. MCP server, A2A adapter, evaluation framework) where there is no user
+	// to provide input. This is distinct from ToolsApproved which can also be set
+	// in interactive TUI sessions when a user approves all tools.
+	NonInteractive bool `json:"non_interactive,omitempty"`
+
 	// HideToolResults is a flag to indicate if tool results should be hidden
 	HideToolResults bool `json:"hide_tool_results"`
 
@@ -476,6 +482,12 @@ func WithToolsApproved(toolsApproved bool) Opt {
 	}
 }
 
+func WithNonInteractive(nonInteractive bool) Opt {
+	return func(s *Session) {
+		s.NonInteractive = nonInteractive
+	}
+}
+
 func WithHideToolResults(hideToolResults bool) Opt {
 	return func(s *Session) {
 		s.HideToolResults = hideToolResults