Dynamic Refresh of MCP servers and Tool Health Status#1507
Merged
crivetimihai merged 23 commits intomainfrom Nov 27, 2025
Merged
Dynamic Refresh of MCP servers and Tool Health Status#1507crivetimihai merged 23 commits intomainfrom
crivetimihai merged 23 commits intomainfrom
Conversation
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com>
a3d81b3 to
21771f7
Compare
Member
|
Freezes on doctest somewhere, working on it. |
The test was setting tool_service._event_subscribers but ToolService now delegates event publishing to _event_service. Updated to set _event_service._event_subscribers and force local mode. Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
crivetimihai
approved these changes
Nov 27, 2025
7 tasks
kcostell06
pushed a commit
to kcostell06/mcp-context-forge
that referenced
this pull request
Feb 24, 2026
* health check status update with events Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * event service Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * fix events notifications Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * use centralized event service Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * updated docstrings and doctests Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * added comments Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * linting Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * minor update Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * using centralized event service for prompts Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * event service updated Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * minor change Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * linting Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * updated event generator Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * resolved pylint issues Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * update test cases Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * update test case Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * fix tests Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * fix doctests Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * docstring coverage Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * web linting Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * minor flake8 fix Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> * Linting Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> * fix: update test to use EventService subscriber list The test was setting tool_service._event_subscribers but ToolService now delegates event publishing to _event_service. Updated to set _event_service._event_subscribers and force local mode. Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> --------- Signed-off-by: Keval Mahajan <mahajankeval23@gmail.com> Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🐛 Bug-fix PR
📌 Summary
Closes #464 and #1506
The health status of MCP servers and tools is automatically updated in the UI in real time, without requiring a manual page reload. This is achieved through a centralized event service that handles the publishing and subscription of events. These events are streamed to the UI as server-sent events, allowing the status of MCP servers and tools to be refreshed automatically based on the latest health checks.
🔁 Reproduction Steps
🐞 Root Cause
Previously, the UI relied on manual refresh to update the health status of MCP servers and tools. There was no mechanism to propagate real-time updates from the backend to the frontend, leading to stale health information.
💡 Fix Description
🧪 Verification
make lintmake testmake coverage📐 MCP Compliance (if relevant)
✅ Checklist
make black isort pre-commit)