chore(submodules): bump for EVO-2139 (Custom MCP test connection fix) - #164
Closed
marcelogorutuba wants to merge 1 commit into
Closed
chore(submodules): bump for EVO-2139 (Custom MCP test connection fix)#164marcelogorutuba wants to merge 1 commit into
marcelogorutuba wants to merge 1 commit into
Conversation
Bring in the fix for the Custom MCP "Test connection" button that was returning 404 for every compliant MCP server (bug: Go core did a raw GET /health against MCP targets; MCP spec doesn't define /health). Submodule bumps: - evo-ai-processor-community: new POST /custom-mcp-servers/test-connection endpoint using real MCP handshake via Google ADK MCPToolset. - evo-ai-core-service-community: Test() delegates to processor via DoPostJSON (mirrors discoverTools pattern, propagates X-Evo-Tenant-Id); ToResponse() normalizes Tools nil -> []; new TestResult.ToolsCount. - evo-ai-frontend-community: guard against server.tools=null in the toast (was crashing to generic error even on backend success); prefer test_result.tools_count for accurate count. Companion PRs (all -> develop): - processor: evolution-foundation/evo-ai-processor-community#36 - core-service: evolution-foundation/evo-ai-core-service-community#19 - frontend: evolution-foundation/evo-ai-frontend-community#260 Validated end-to-end locally via `docker compose build` + UI click: `https://mcp.deepwiki.com/mcp` -> toast verde "3 ferramentas descobertas". Linear: https://linear.app/evoai/issue/EVO-2139 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates three git submodule pointers (core service, frontend, and processor) to newer commits that contain the Custom MCP "Test connection" fix and related contract/tenant-header behavior, without making any direct code changes in this repository. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Before merging, consider rebasing this PR to point the submodules at the final merge commits on the main/develop branches rather than feature branch tips to avoid future SHA drift and confusion.
- Once the child PRs are merged, double-check that the submodule SHAs here include only the intended MCP fix and not additional in-progress feature work from those branches.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Before merging, consider rebasing this PR to point the submodules at the final merge commits on the main/develop branches rather than feature branch tips to avoid future SHA drift and confusion.
- Once the child PRs are merged, double-check that the submodule SHAs here include only the intended MCP fix and not additional in-progress feature work from those branches.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Member
Author
marcelogorutuba
deleted the
marcelosoares/evo-2139-bump-mcp-test-connection
branch
July 14, 2026 23:36
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
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.
Summary
Submodule bumps
Test plan
docker compose build evo-processor evo-core && docker compose up -d+ reload frontend dev server → botão Test comhttps://mcp.deepwiki.com/mcpretorna toast verde "3 ferramentas descobertas".test_result.success,test_result.error,test_result.tools_count(optional, novo).X-Evo-Tenant-Idset quando runtime context tem tenant, omitido caso contrário.Nota sobre re-bump
Os 3 pointers apontam para tips de branches feature (não develop). Se os PRs filhos forem merged como squash, os SHAs mudam e este PR precisa re-bump antes do merge — ou merge dos filhos primeiro e este PR bumpa pra develop tips depois.
Linear: https://linear.app/evoai/issue/EVO-2139
🤖 Generated with Claude Code
Summary by Sourcery
Chores: