Add periodic stderr messages. #1423
Conversation
In everything.ts - add a 10 second interval for sending 'notifications/stderr' messages to the client This was created in order to test the display and clearing of stderr messages in the client. - see modelcontextprotocol/inspector#286
I thought about having an affordance to turn that off in the inspector, but it would mean nothing to any other server you try it with. The everything server is just a good way to see everything in action, so you can look at the code to see "oh, that's how that part works". There isn't anything else to do with it really. We could dial back the frequency to say every 30 seconds or something. It would be enough to reveal the area where error notifications are shown and the clear button that can be used. They don't need to pile up. I'll try that. Come to speak of it, the randomly-leveled log notifications are a little frequent as well. |
- subscription updates: 10 seconds - logging messages: 20 seconds - stderr messages: 30 seconds
|
Adjusted intervals for outgoing demo messages
|
Add periodic stderr messages and reduce frequency of generated notifcations.
Description
In everything.ts
Server Details
Motivation and Context
The everything server is meant to exercise all the MCP features, and is the goto server for testing features in the Inspector.
One of the features we don't exercise is the sending of
notifications/stderrmessages. They show up in the sidebar, and we had to add special notification handling for that message type in the past.When a new PR came in to the inspector to add a Clear button for clearing those messages, I had no obvious way to test it.
How Has This Been Tested?
Using the inspector:
Screen.Recording.2025-04-14.at.4.59.19.PM.mov
Breaking Changes
Types of changes
Checklist
Additional context