Skip to content

feat: Support Progress Flow #407

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

feat: Support Progress Flow #407

wants to merge 4 commits into from
+360 −5

Conversation

134130
Copy link
Contributor

@134130 134130 commented Jul 17, 2025 •
edited
Loading

Motivation and Context

How Has This Been Tested?

  • WebFlux and SSE Integration tests

Breaking Changes

  • No, but there is deprecated APIs for McpServerFeatures.Async and McpServerFeatures.Sync

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@134130 134130 mentioned this pull request Jul 17, 2025
Closed
5 tasks
@134130
Copy link
Contributor Author

134130 commented Jul 17, 2025

@tzolov, I would appreciate it if you could provide a review.

@tzolov tzolov added this to the 0.11.0 milestone Jul 17, 2025
@tzolov tzolov self-assigned this Jul 17, 2025
tzolov pushed a commit that referenced this pull request Jul 17, 2025
@134130 @tzolov
feat: Add progress notification support for MCP operations (#407)
b04c30b 
Implement progress tracking for long-running operations with:
- New ProgressNotification schema and client/server support
- Progress consumer handlers in sync/async client builders
- Server exchange methods for sending progress updates
- Comprehensive integration tests
- Backwards compatibility maintained
- Add additional tests

Signed-off-by: Christian Tzolov <[email protected]>
@tzolov
Copy link
Contributor

tzolov commented Jul 17, 2025

Thanks you @134130 ,
I've added some additional tests

@tzolov
Copy link
Contributor

tzolov commented Jul 17, 2025

rebased, squashed, add additional tests and merged at b04c30b

@tzolov tzolov closed this Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tzolov tzolov

Labels
None yet
Projects
None yet
Milestone
0.11.0
Development

Successfully merging this pull request may close these issues.
2 participants
@134130 @tzolov