Skip to content

Conversation

@KyleTryon
Copy link
Contributor

@KyleTryon KyleTryon commented Dec 29, 2025

This pull request refactors the MCP configuration utility to provide more consolidated and informative feedback when configuring multiple editors. Instead of logging individual success messages for each editor, the code now collects the results and outputs a single, combined summary indicating which configuration files were created or updated. The test suite is also updated to reflect these changes.

Key changes:

Refactoring configuration functions:

  • The addCursorMcpConfig, addVsCodeMcpConfig, addClaudeCodeMcpConfig, and addOpenCodeMcpConfig functions now return a result object indicating the filename and whether the file was created or updated, instead of logging directly or returning void. [1] [2] [3] [4] [5] [6] [7] [8]

Consolidated logging in project-scoped configuration:

  • The main offerProjectScopedMcpConfig function now aggregates the results from all editor configuration functions and logs a single, consolidated success message summarizing which files were created or updated, along with a unified note about reloading or restarting editors. [1] [2] [3]

Test updates:

  • Tests are updated to expect the new consolidated log messages, checking for strings containing the relevant filenames and the new summary message, instead of the previous per-editor messages. [1] [2] [3] [4] [5] [6] [7] [8]- Updated addCursorMcpConfig, addVsCodeMcpConfig, addClaudeCodeMcpConfig, and addOpenCodeMcpConfig functions to return an object indicating the filename and action (created/updated) instead of logging directly.
  • Consolidated success messages in offerProjectScopedMcpConfig to provide a summary of created and updated files.
  • Improved logging for better clarity and consistency across MCP configuration processes.
image

- Updated `addCursorMcpConfig`, `addVsCodeMcpConfig`, `addClaudeCodeMcpConfig`, and `addOpenCodeMcpConfig` functions to return an object indicating the filename and action (created/updated) instead of logging directly.
- Consolidated success messages in `offerProjectScopedMcpConfig` to provide a summary of created and updated files.
- Improved logging for better clarity and consistency across MCP configuration processes.
@KyleTryon KyleTryon requested a review from Lms24 December 29, 2025 18:36
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