Skip to content

Comments

fix: restore MCP gateway tool name prefixing logic removed by agentic workflow#330

Merged
pethers merged 2 commits intocopilot/news-2026-02-19-v2-2f1ee22a2bc0c0c2from
copilot/sub-pr-326
Feb 19, 2026
Merged

fix: restore MCP gateway tool name prefixing logic removed by agentic workflow#330
pethers merged 2 commits intocopilot/news-2026-02-19-v2-2f1ee22a2bc0c0c2from
copilot/sub-pr-326

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

The agentic workflow that ran during news article generation (commit abdca05) silently deleted the gateway URL tool-name prefixing logic in scripts/mcp-client.js, replacing 3 lines with a no-op comment + const toolName = tool. This broke all agentic-workflow invocations through the MCP gateway proxy.

Root cause

The MCP gateway (host.docker.internal / /mcp/riksdag-regering) requires tool names prefixed as riksdag-regering--{tool} for routing. Without the prefix, the proxy cannot identify the target server and returns an error.

Fix

Restored the original detection + prefixing logic:

const isGateway = this.baseURL.includes('host.docker.internal') || this.baseURL.includes('/mcp/riksdag-regering');
const shouldPrefix = isGateway && !skipPrefix && !tool.includes('--');
const toolName = shouldPrefix ? `riksdag-regering--${tool}` : tool;

The existing fallback (retry without prefix on Internal error) was already in place and is unaffected.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add 42 news articles for 2026-02-19 fix: restore MCP gateway tool name prefixing logic removed by agentic workflow Feb 19, 2026
Copilot AI requested a review from pethers February 19, 2026 13:45
@pethers pethers marked this pull request as ready for review February 19, 2026 13:46
@pethers pethers merged commit 4004860 into copilot/news-2026-02-19-v2-2f1ee22a2bc0c0c2 Feb 19, 2026
@pethers pethers deleted the copilot/sub-pr-326 branch February 19, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants