Skip to content

[MCP Bundle] Add configurable tool discovery options #808

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 23, 2025
Merged

[MCP Bundle] Add configurable tool discovery options #808

merged 1 commit into from
Oct 23, 2025

Conversation

@OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Oct 23, 2025

Q A
Bug fix? no
New feature? yes
Docs? no
Issues Fix #801
License MIT

Summary

  • Add support for configurable scan_dirs and exclude_dirs in tool discovery
  • Allows users to specify multiple directories to scan for MCP tools
  • Allows users to exclude specific directories from discovery

Configuration Example

# config/packages/mcp.yaml
mcp:
    discovery:
        scan_dirs: ['src', 'lib']
        exclude_dirs: ['src/DataFixtures']

@carsonbot carsonbot added Status: Needs Review Feature New feature MCP Bundle Issues & PRs about the MCP SDK integration bundle labels Oct 23, 2025
@OskarStark OskarStark mentioned this pull request Oct 23, 2025
Closed
chr-hertel
chr-hertel approved these changes Oct 23, 2025
View reviewed changes
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

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

Thanks @OskarStark 🫶

@chr-hertel chr-hertel force-pushed the mcp-tool-discovery-options branch from da8fc02 to b1c6e93 Compare October 23, 2025 17:19
@chr-hertel
Copy link
Member

chr-hertel commented Oct 23, 2025

Thank you @OskarStark.

@chr-hertel chr-hertel merged commit 7a631c6 into symfony:main Oct 23, 2025
14 checks passed
@OskarStark OskarStark deleted the mcp-tool-discovery-options branch October 23, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chr-hertel chr-hertel chr-hertel approved these changes

@Nyholm Nyholm Awaiting requested review from Nyholm Nyholm is a code owner

Assignees

No one assigned

Labels

Feature New feature MCP Bundle Issues & PRs about the MCP SDK integration bundle Status: Reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[MCP Bundle] Add options for tool discovery

3 participants

@OskarStark @chr-hertel @carsonbot