feat: add MCP retry policy to Agent Spec#200
Conversation
paul-cayet
requested review from
a team,
cesarebernardis,
dhilloulinoracle and
sonleoracle
oracle-contributor-agreement
Bot
added
the
OCA Verified
|
Internal regression succeeded 🍏: Build ID #503 |
|
|
||
| ``retry_policy`` optionally specifies a ``RetryPolicy`` for MCP toolbox | ||
| discovery/resolution and for the execution of tools generated by the toolbox. | ||
| This is distinct from retry policies on remote MCP transports, which apply to |
There was a problem hiding this comment.
I am not sure about what does this involve (same for the equivalent paragraph in MCPTool). What kind of failures does it apply to? Does it mean that, for example, service_error_retry_on_any_5xx and recoverable_statuses don't apply (in which case we should make clear that these attributes should not be part of it)?
I think we need to be more clear on what this applies to, as we say "it's not the same as this other thing", but I feel like we don't say what it is. Also, whenever there's a "Runtime implementations decide ..." we are giving up on cross-framework consistency at spec level. I think we need to say what it should apply to, and if runtimes don't support it, that's on the runtime adapter.
| runtime handling session management and data relay. | ||
|
|
||
| ``retry_policy`` optionally specifies a ``RetryPolicy`` for direct MCP tool | ||
| resolution and execution. This is distinct from retry policies on remote MCP |
There was a problem hiding this comment.
could explain more clearly what "resolution" and "execution" mean
| # .. end-##_OAuth_in_MCP_Tools | ||
|
|
||
| # .. start-##_MCP_Retry_Policy | ||
| mcp_client_for_retry = SSETransport( |
There was a problem hiding this comment.
should we have an example with both the retry on the transport and the retry on the MCP tool?
4 participants
Supporting Agent Spec PR for oracle/wayflow#170
Adds optional retry_policy support to MCPTool and MCPToolBox so clients can retry transient MCP tool resolution and execution failures.