Skip to content

[codex] Fix MCP hot-reload feedback#101

Merged
Cyberforker merged 1 commit into
Yapie0:mainfrom
Star-Star66:codex/fix-mcp-hot-reload-feedback
Jun 15, 2026
Merged

[codex] Fix MCP hot-reload feedback#101
Cyberforker merged 1 commit into
Yapie0:mainfrom
Star-Star66:codex/fix-mcp-hot-reload-feedback

Conversation

@Star-Star66

Copy link
Copy Markdown
Contributor

What changed

  • honor the MCP mutation API's requiresRestart result for both add and remove flows
  • only show the marketplace restart warning when an installed server is not live-bridged
  • update English and Chinese feedback so successful hot reload no longer claims a restart is required
  • add focused regression tests for restart and bridge-state decisions

Why

The dashboard previously ignored the remove response and always displayed restart guidance. Marketplace details also showed a stale restart warning even when the MCP server had already been bridged into the current session.

Validation

  • npm run verify
  • 338 test files passed
  • 3914 tests passed, 2 skipped
  • build, lint, and typecheck passed

@Star-Star66 Star-Star66 marked this pull request as ready for review June 14, 2026 11:25
@Cyberforker Cyberforker merged commit a2b8bb0 into Yapie0:main Jun 15, 2026
4 checks passed
@Cyberforker

Copy link
Copy Markdown
Collaborator

Review Summary

Reviewed & merged by Carbon Code

Tests: npm run verify - 3914/3914 passed, 2 skipped
Fix: honor MCP mutation API restart feedback and only show restart guidance when an installed server is not live-bridged.

Thanks @Star-Star66 for the focused dashboard hot-reload feedback fix and regression coverage.

Co-Authored-By: Carbon Code carboncode@code.ai6666.com

-- Carbon Code

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.

2 participants