Skip to content

Conversation

vorburger
Copy link

Motivation and Context

Fixes NPE seen in the wild as described in #605.

How Has This Been Tested?

As described in #605, unfortunately I'm unable to reproduce that NPE locally even myself, but I am consistently seeing this on a failing CI build. However I'm confident that this change improves the robustness of McpClientSession and will provide a much more clear and actionable error message when this occurs again next time.

Breaking Changes

No.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant