Merged
Conversation
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
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.
This pull request introduces a major refactor to the MCP server management and its UI integration. The MCP server is now managed as a native Node.js instance within the Electron app, replacing the previous child process spawn approach. The UI receives a new, reactive MCP status indicator component, improving user feedback and error handling. Several improvements are made to project/environment switching logic for robustness.
MCP Server Management Refactor:
src/main/mcp-manager.tswith direct instantiation and management of the MCP server using the newcreateMcpServerAPI, enabling better integration, logging, and lifecycle management. [1] [2] [3]MCP Server API and Initialization:
createMcpServerandMcpServerInstanceinsrc/mcp-server/index.ts, supporting both HTTP and stdio modes, structured logging, and clean shutdown. Standalone script execution is supported for desktop usage. [1] [2]UI Integration and Status Indicator:
TheNavBar.vuewith a new reactiveNavBarMCP.vuecomponent, which displays real-time MCP status, errors, and allows navigation to relevant settings or logs. [1] [2] [3]Project/Environment Switcher Robustness:
NavBarProjectSwitcher.vueto prevent errors when project/environment info is missing, ensuring safer state updates and persistence. [1] [2] [3] [4]Minor Typing Improvements:
NavBarAlwaysOnTop.vuefor improved type checking.