Skip to content

Conversation

@xierenyuan
Copy link
Contributor

Some MCP operations may timeout when execution takes too long. In such cases, we want to be able to customize the timeout duration to ensure MCP can execute successfully.
image

@changeset-bot
Copy link

changeset-bot bot commented Jul 31, 2025

🦋 Changeset detected

Latest commit: 89830ab

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@openai/agents-core Patch
@openai/agents-openai Patch
@openai/agents-realtime Patch
@openai/agents Patch
@openai/agents-extensions Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@xierenyuan xierenyuan changed the title feat: add timeout parameter to callTool method feat: add timeout parameter to mcp callTool Jul 31, 2025
@seratch seratch added enhancement New feature or request package:agents-core labels Jul 31, 2025
Copy link
Member

@seratch seratch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding this parameter looks great to me, but can you check a few minor things to adjust?

@xierenyuan
Copy link
Contributor Author

xierenyuan commented Aug 1, 2025

Okay, I'm done adjusting

@seratch
Copy link
Member

seratch commented Aug 1, 2025

Cannot update this protected ref.

All look good to me, but is your branch protected from updating on your side (this is my first time to see this, so I am actually unsure what it is)? We're unable to rebase your PR toward the latest main branch. It blocks us to merge it in a straight-forward way.

@xierenyuan
Copy link
Contributor Author

  • Cannot update this protected ref.

Wait a minute. I'll see if I have any settings.

@xierenyuan
Copy link
Contributor Author

I've checked, I haven't set up any other configurations, and I'm trying to add you as a maintainer to see if I can merge

@xierenyuan
Copy link
Contributor Author

image I suspect it's because I mentioned this pr in the private repository I'll re-initiate it and see if it's mergeable.

@xierenyuan xierenyuan closed this Aug 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants