Uh oh!
There was an error while loading. Please reload this page.
fall through if PRM endpoint is not implemented - #1562
Conversation
maxisbey
left a comment
There was a problem hiding this comment.
Thanks for the PR! Could you please add a unit test for this which would fail before the fix and pass with your changes
dogacancolak
commented
Nov 12, 2025
@maxisbey Thanks for the feedback! I added a unit test that closely mimics another test named |
dogacancolak
commented
Nov 13, 2025
Hi @maxisbey, seems like you've made some extensive changes that also cover this bug. I can fix the PR checks if you think this unit test will be useful. Otherwise we can close this PR. |
Motivation and Context
As raised in #1495, one of the changes in #1324 included raising an error if PRM endpoint request failed. This caused backwards compatibility issues with MCP servers that haven't yet added support for PRM
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context