Skip to content

Conversation

himanshusinghs
Copy link
Contributor

Description

We're adding a few configuration options that should be in user's control and will be injected into the MCP server started by the VSCode extension.

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@himanshusinghs himanshusinghs requested a review from a team as a code owner August 21, 2025 14:30
@himanshusinghs himanshusinghs changed the title chore: adds config options relevant for MCP server chore: adds config options relevant for MCP server MCP-133 Aug 21, 2025
@himanshusinghs himanshusinghs added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Aug 21, 2025
Copy link
Contributor

@nirinchev nirinchev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may want to clarify that these settings apply to the MCP server exposed by the vscode extension. Currently, the descriptions are a little ambiguous as if I read

When set to true, only allows read, connect, and metadata operation types, disabling create/update/delete operations

without noticing that the setting is mdb.mcp.readOnly, I may assume this is a config option for the vscode extension as a whole.

@himanshusinghs himanshusinghs merged commit e8b3647 into feat/mcp-integration Aug 25, 2025
5 checks passed
@himanshusinghs himanshusinghs deleted the chore/MCP-133 branch August 25, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants