From 0313521b50380a18ee2ca288658e742ecf4b60fb Mon Sep 17 00:00:00 2001
From: stevef <stevef@omen.home>
Date: Mon, 20 Apr 2026 12:23:39 +0200
Subject: [PATCH] fix(tools): stabilize freeride tool visibility and turn
 completion

Changed freeride results to UserResult and added ResponseHandled: true to
ensure tool output is visible in CLI/Interactive mode and prevent
premature turn finalization with generic summary messages.
---
 pkg/tools/freeride.go      | 8 ++++----
 pkg/tools/freeride_test.go | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/pkg/tools/freeride.go b/pkg/tools/freeride.go
index c388360e9..dd8d7c97a 100644
--- a/pkg/tools/freeride.go
+++ b/pkg/tools/freeride.go
@@ -210,7 +210,7 @@ func (t *FreeRideTool) handleList(ctx context.Context, limit int) *ToolResult {
 		sb.WriteString(fmt.Sprintf("   Parameters: %s\n\n", strings.Join(m.SupportedParameters, ", ")))
 	}
 
-	return SilentResult(sb.String())
+	return UserResult(sb.String()).WithResponseHandled().WithResponseHandled()
 }
 
 func (t *FreeRideTool) handleAuto(ctx context.Context, limit int) *ToolResult {
@@ -268,10 +268,10 @@ func (t *FreeRideTool) handleAuto(ctx context.Context, limit int) *ToolResult {
 			}
 		}
 
-		return SilentResult(msg)
+		return UserResult(msg).WithResponseHandled().WithResponseHandled()
 	}
 
-	return SilentResult("No new free models to add. Your configuration is up to date.")
+	return UserResult("No new free models to add. Your configuration is up to date.").WithResponseHandled().WithResponseHandled()
 }
 
 func (t *FreeRideTool) handleStatus() *ToolResult {
@@ -294,7 +294,7 @@ func (t *FreeRideTool) handleStatus() *ToolResult {
 	}
 	sb.WriteString(fmt.Sprintf("- Managed Free Models: %d\n", openRouterCount))
 
-	return SilentResult(sb.String())
+	return UserResult(sb.String()).WithResponseHandled().WithResponseHandled()
 }
 
 func modelExists(cfg *config.Config, modelName string) bool {
diff --git a/pkg/tools/freeride_test.go b/pkg/tools/freeride_test.go
index 0d03fcea1..b3fce2c6e 100644
--- a/pkg/tools/freeride_test.go
+++ b/pkg/tools/freeride_test.go
@@ -60,8 +60,8 @@ func TestFreeRideTool_List(t *testing.T) {
 		t.Fatalf("Expected no error, got %s", result.ForLLM)
 	}
 
-	if !result.Silent {
-		t.Errorf("Expected silent result")
+	if result.Silent {
+		t.Errorf("Expected non-silent result")
 	}
 
 	output := result.ForLLM