codex - 💡(How to fix) Fix Expose callId in MCP tool call _meta for all servers [1 participants]

TL;DR

Add callId at the top level of _meta for all MCP servers to enable tracing, audit logging, and request deduplication.

Guidance

• Modify the build_mcp_tool_call_request_meta function to include call_id at the top level of _meta, similar to threadId.
• Verify the change by checking the _meta object in the MCP tool call request for the presence of callId.
• Test the change with custom MCP servers to ensure they receive the callId identifier.
• Consider adding tests to cover the new behavior, as mentioned in the issue.

Example

No code snippet is provided, as the issue does not contain sufficient code context.

Notes

The proposed change requires modifying the build_mcp_tool_call_request_meta function, which may have implications for existing code that relies on the current behavior.

Recommendation

Apply workaround: Modify the build_mcp_tool_call_request_meta function to include call_id at the top level of _meta, as this change is necessary to enable the desired use cases and is a targeted fix.