fix: MCP SSE Model tool mode issue #7360
Merged
Conversation
This file contains 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
edwinjosechittilappilly
requested review from
Copilot and
phact
and removed request for
Copilot
dosubot
bot
added
the
size:XS
There was a problem hiding this comment.
Pull Request Overview
This PR fixes an issue with the MCP SSE Model tool mode by removing the obsolete sse_url attribute from the MCPToolsComponent class.
- Removed the sse_url attribute from the MCPToolsComponent class.
- Code cleanup aimed at simplifying the component configuration.
Comments suppressed due to low confidence (1)
src/backend/base/langflow/components/tools/mcp_component.py:31
- The 'sse_url' attribute removal is not reflected in the default_keys list. Please update default_keys to remove the obsolete 'sse_url' entry.
default_keys: list[str] = ["code", "_type", "mode", "command", "sse_url", "tool_placeholder", "tool_mode", "tool"]
github-actions
bot
added
bug
phact
approved these changes
Mar 31, 2025
QDDCoder
pushed a commit
to QDDCoder/langflow
that referenced
this pull request
Apr 1, 2025
* commit '6c2ff92ccf083f038d9a97a749b4e0f30a60247a': (255 commits) docs: Update documentation and add TAVILY_API_KEY to environment variables (langflow-ai#7366) docs: mcp server component and integrations (langflow-ai#7286) docs: v1.3 (langflow-ai#7160) chore(deps-dev): bump vite from 5.4.14 to 5.4.16 in /src/frontend (langflow-ai#7365) fix: add secret key loading functionality to basic templates (langflow-ai#7363) docs: platform content (langflow-ai#7091) chore: bump langflow and langflow-base for 1.3.0 release (langflow-ai#7362) docs: ensure asset paths update on draft rebuilds (langflow-ai#7355) fix: Prevent API key banner from displaying when key is unavailable (langflow-ai#7356) fix: MCP SSE Model tool mode issue (langflow-ai#7360) feat: deprecate Parsing component and replace them in starter projects (langflow-ai#6697) fix: fixed the parser component (langflow-ai#7303) ref: split ast parsing / class exec loops (langflow-ai#7248) feat: Update AgentQL templates (langflow-ai#7165) feat: add a unified local vector store (langflow-ai#6995) fix: update currentflow to be fetched from Flow Store for icon to be rendered correctly (langflow-ai#7354) feat: expose serialization truncation constants in /config route (langflow-ai#7316) feat: lmprove MCP langflow port selection and error handling (langflow-ai#7327) fix: adds priority sorting for sidebar components and makes LanguageModel appear first (langflow-ai#7322) fix: make agent work with new tool mode backend updates (langflow-ai#7314) ... # Conflicts: # src/frontend/src/utils/styleUtils.ts 更新到V1.3.0
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 includes a few changes to the
MCPToolsComponentclass in thesrc/backend/base/langflow/components/tools/mcp_component.pyfile. The most significant change is the removal of thesse_urlattribute.Codebase simplification:
src/backend/base/langflow/components/tools/mcp_component.py: Removed thesse_urlattribute from theMCPToolsComponentclass.