Simplify tool schema transformation for OpenAI strict mode

OpenAI's Structured Outputs requires all properties to be in the
'required' array, even optional ones (which use nullable types).

The simplified transformation:
- Only ensures all properties are listed in required array
- Recursively processes nested objects and array items
- Doesn't modify type definitions (already correct in our tools)
- Much simpler than the original complex transformation

Author: daniel-lxs

src/api/providers/openai-native.ts

@@ -190,6 +190,38 @@ export class OpenAiNativeHandler extends BaseProvider implements SingleCompletio
 		reasoningEffort: ReasoningEffortExtended | undefined,
 		metadata?: ApiHandlerCreateMessageMetadata,
 	): any {
+		// Ensure all properties are in the required array for OpenAI's strict mode
+		// This recursively processes nested objects and array items
+		const ensureAllRequired = (schema: any): any => {
+			if (!schema || typeof schema !== "object" || schema.type !== "object") {
+				return schema
+			}
+
+			const result = { ...schema }
+
+			if (result.properties) {
+				const allKeys = Object.keys(result.properties)
+				result.required = allKeys
+
+				// Recursively process nested objects
+				const newProps = { ...result.properties }
+				for (const key of allKeys) {
+					const prop = newProps[key]
+					if (prop.type === "object") {
+						newProps[key] = ensureAllRequired(prop)
+					} else if (prop.type === "array" && prop.items?.type === "object") {
+						newProps[key] = {
+							...prop,
+							items: ensureAllRequired(prop.items),
+						}
+					}
+				}
+				result.properties = newProps
+			}
+
+			return result
+		}
+
 		// Build a request body for the OpenAI Responses API.
 		// Ensure we explicitly pass max_output_tokens based on Roo's reserved model response calculation
 		// so requests do not default to very large limits (e.g., 120k).
@@ -266,7 +298,7 @@ export class OpenAiNativeHandler extends BaseProvider implements SingleCompletio
 						type: "function",
 						name: tool.function.name,
 						description: tool.function.description,
-						parameters: tool.function.parameters,
+						parameters: ensureAllRequired(tool.function.parameters),
 						strict: true,
 					})),
 			}),