From 7aae5ef316b69725161c96f767ac9f69962efe18 Mon Sep 17 00:00:00 2001
From: Michael Neale <michael.neale@gmail.com>
Date: Sat, 18 Oct 2025 01:29:04 +1100
Subject: [PATCH] fix: correct schema for openai tools (#5229)

Co-authored-by: Jack Amadeo <jackamadeo@squareup.com>
---
 .../src/agents/recipe_tools/dynamic_task_tools.rs    | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/crates/goose/src/agents/recipe_tools/dynamic_task_tools.rs b/crates/goose/src/agents/recipe_tools/dynamic_task_tools.rs
index 721ccd45..91acfd05 100644
--- a/crates/goose/src/agents/recipe_tools/dynamic_task_tools.rs
+++ b/crates/goose/src/agents/recipe_tools/dynamic_task_tools.rs
@@ -47,6 +47,8 @@ impl From<ExecutionModeParam> for ExecutionMode {
     }
 }
 
+type JsonObject = serde_json::Map<String, Value>;
+
 /// Parameters for a single task
 #[derive(Debug, Serialize, Deserialize, JsonSchema)]
 pub struct TaskParameter {
@@ -63,19 +65,19 @@ pub struct TaskParameter {
     pub description: Option<String>,
 
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub extensions: Option<Vec<Value>>,
+    pub extensions: Option<Vec<JsonObject>>,
 
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub settings: Option<Value>,
+    pub settings: Option<JsonObject>,
 
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub parameters: Option<Vec<Value>>,
+    pub parameters: Option<Vec<JsonObject>>,
 
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub response: Option<Value>,
+    pub response: Option<JsonObject>,
 
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub retry: Option<Value>,
+    pub retry: Option<JsonObject>,
 
     #[serde(skip_serializing_if = "Option::is_none")]
     pub context: Option<Vec<String>>,