Conversation
amitksingh1490
commented
Mar 9, 2026
- feat(task): add task tool for agent delegation with session resumption
- feat(task): capitalize task tool name and prevent self-delegation in task tool
- style(tests): reformat function call arguments to comply with line width limits
| "required": [ | ||
| "agent_id", | ||
| "session_id", | ||
| "tasks" | ||
| ], |
There was a problem hiding this comment.
Critical schema bug: session_id is incorrectly listed in the required array but should be optional. The TaskInput struct defines session_id as Option<String> with #[serde(skip_serializing_if = "Option::is_none")], indicating it's optional. However, OpenAI's strict mode will reject any task tool calls that don't include this field.
Impact: All task tool invocations without session_id will fail with validation errors from OpenAI.
Fix: Remove "session_id" from the required array:
"required": [
"agent_id",
"tasks"
]| "required": [ | |
| "agent_id", | |
| "session_id", | |
| "tasks" | |
| ], | |
| "required": [ | |
| "agent_id", | |
| "tasks" | |
| ], |
Spotted by Graphite
Is this helpful? React 👍 or 👎 to let us know.
There was a problem hiding this comment.
OpenAI strict mode: required field exists with all property keys
216182d to
347c295
Compare
|
Action required: PR inactive for 5 days. |