feat(mcp): add FastMCP search and CodeMode on top of skills#7401
Closed
MagnusS0 wants to merge 3 commits into
Closed
feat(mcp): add FastMCP search and CodeMode on top of skills#7401MagnusS0 wants to merge 3 commits into
MagnusS0 wants to merge 3 commits into
Conversation
Integrates FastMCP BM25SearchTransform + CodeMode discovery onto the upstream feature/mcp-skills branch, which adds the skills infrastructure. Key changes: - REMOVE: CategoryIndex model + 5 admin tools (available_categories, available_tools, activate_tools, deactivate_tools, activate_category) - ADD: utils/code_mode.py with BM25SearchTransform and CodeMode transforms - ADD: _apply_discovery_transforms() — CodeMode or BM25 search depending on enable_code_mode / enable_tool_discovery settings - ADD: Code mode settings fields (enable_code_mode, code_mode_max_*, code_mode_search_max_results, code_mode_search_output_format) - ADD: include_tags / exclude_tags fields for tag-based visibility - ADD: Provider interface integration (dynamic provider tags on tools) - ADD: _get_default_openbb_app() lazy loader - KEEP: All skills infrastructure (SkillsDirectoryProvider, vendor providers, install_skill tool, SKILL.md files) - UPDATE: fastmcp pin to >=3.1.0,<4 (public release) - UPDATE: mcp.enable/disable to use keys= instead of names= - UPDATE: Tests — remove CategoryIndex patches, add transform/code_mode tests Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
|
@MagnusS0, can you please reopen this pointing to |
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
2 participants
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.
OpenBB MCP - FastMCP search and CodeMode on top of skills
This PR integrates the transform-based search discovery and Code Mode work from #7385 onto the skills-based FastMCP v3 branch from #7380.
Target/base for this PR is
feature/mcp-skills, so this branch should be read as:openbb-mcp-server: Refactor for FastMCP V3 #7380openbb-mcp-server: Refactor for FastMCP V3 #7380 with the transform-based search discovery and Code Mode flow from refactor(mcp_server): migrate to FastMCP v3 and add OpenBB Code Mode #7385Concretely, this merges the two lines of work into a single branch:
openbb-mcp-server: Refactor for FastMCP V3 #7380: FastMCP v3 server refactor plus bundled/vendor skills supportWhat changed
feature/mcp-skillsbranch.list_categoriessearchget_schemacall_toolin standalone discovery modeexecutefor sandboxed chained tool calls viapython-montyopenbb-mcp-server: Refactor for FastMCP V3 #7380:keys=enable/disable semantics.enable_code_modecode_mode_*include_tagsexclude_tagsfastmcp>=3.1.0,<4.Checklist
feature/mcp-skillsopenbb-mcp-server: Refactor for FastMCP V3 #7380 and refactor(mcp_server): migrate to FastMCP v3 and add OpenBB Code Mode #7385