feat: enums connection state and standardise tool discovery #26690
+141
−3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
agents-git-bot
bot
requested review from
a team,
elithrar,
irvinebroque,
rita3ko,
thomasgauvin and
threepointone
as code owners
github-actions
bot
added
the
product:agents
Contributor
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
agents-git-bot
bot
force-pushed
the
sync-docs-pr-672
branch
from
c0ca5d7 to
7f9565b
Compare
agents-git-bot
bot
changed the title
Contributor
agents-git-bot
bot
force-pushed
the
sync-docs-pr-672
branch
from
7f9565b to
eb32b10
Compare
agents-git-bot
bot
changed the title
agents-git-bot
bot
force-pushed
the
sync-docs-pr-672
branch
from
eb32b10 to
12ee0ff
Compare
Author
|
Added additional enhancements to the documentation:
The documentation now includes:
|
Synced from cloudflare/agents PR #672 - Document new MCPConnectionState enum with type-safe constants - Add new 'connected' state to connection state machine - Update state transition documentation - Document improved discovery error handling - Add changelog entry for breaking changes and new features Related PR: cloudflare/agents#672 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
agents-git-bot
bot
force-pushed
the
sync-docs-pr-672
branch
from
e9c93e1 to
b66c921
Compare
agents-git-bot
bot
changed the title
This sync updates the documentation to reflect changes from cloudflare/agents PR #672: - Documented new connection state machine with separate CONNECTED and READY states - ClientConnection.init() no longer automatically discovers capabilities - Added MCPConnectionState enum documentation with all possible states - Documented new CONNECTED state for transport connection without capabilities - Added detailed state transition flows for OAuth and non-OAuth connections - Updated error handling section to reflect fail-fast behavior with Promise.all - Added explanations for each connection state - Clarified the difference between CONNECTED and READY states Related PR: cloudflare/agents#672 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This sync updates the MCP client documentation to reflect changes from cloudflare/agents PR #672:
Breaking Changes
CONNECTEDandREADYstatesClientConnection.init()no longer automatically discovers capabilitiesNew Features
MCPConnectionStateenum documentation with all possible statesCONNECTEDstate for transport connection without capabilitiesImprovements
Promise.allCONNECTEDandREADYstatesRelated PR
cloudflare/agents#672
🤖 Generated with Claude Code