-
Notifications
You must be signed in to change notification settings - Fork 9.1k
Description
Is your feature request related to a problem? Please describe.
This issue is paired with this request in the Inspector project:
And arises from a recent set of changes...
- Add periodic logging messages to Everything server. modelcontextprotocol/servers#847
- Add support for server logging messages #185
...that let log messages surface in the UI. For testing, the Everything example server was also amended to periodically send randomly leveled log messages. Now we want to add log level throttling.
Describe the solution you'd like
Right now, the setLevelRequest handler in the Everything server doesn't persist the level, it just sends a dummy debug log message indicating the new level was set.
We need to track the level, with a default set the same as the client, highest level letting all messages through.
In the setInterval handler that sends dummy log messages periodically, we should compare the level of the outgoing message and only send if appropriate.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Not doing this at all. Seems like we should though.
Additional context
Add any other context or screenshots about the feature request here.