Skip to content

Fix contradictory MCP documentation in agentic workflows#244

Merged
pethers merged 5 commits intomainfrom
copilot/update-evening-analysis
Feb 16, 2026
Merged

Fix contradictory MCP documentation in agentic workflows#244
pethers merged 5 commits intomainfrom
copilot/update-evening-analysis

Conversation

Copy link
Contributor

Copilot AI commented Feb 16, 2026

Documentation contradicted itself: "DO NOT" section prohibited MCP_SERVER_URL and scripts, while "Sandbox Networking" section required them. Agents would be confused about correct approach.

Root Cause

Two distinct MCP contexts were conflated:

  • Prompt tool calls: Use simple function names (get_calendar_events())
  • Script execution: Requires proxy config (export MCP_SERVER_URL="http://host.docker.internal:80/mcp/riksdag-regering")

Changes

Scoped "DO NOT" warnings to prompt context only

  • Renamed section: "DO NOT Try to Call MCP Manually From Prompts"
  • Added explicit note: "Scripts ARE used by agentic workflows"
  • Cross-referenced Sandbox Networking section

Example of corrected pattern:

# ✅ For scripts executed via bash
export MCP_SERVER_URL="http://host.docker.internal:80/mcp/riksdag-regering"
node scripts/generate-news-enhanced.js
// ✅ For MCP tool calls in prompts
get_calendar_events({ from: "2026-02-16", tom: "2026-02-16" })
search_voteringar({ rm: "2025/26", limit: 50 })

Files updated:

  • .github/workflows/news-article-generator.md
  • .github/workflows/news-evening-analysis.md
  • .github/workflows/news-realtime-monitor.md

Both patterns are valid in their respective contexts. Documentation now reflects this.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update evening analysis articles based on new data Fix agentic workflow MCP documentation - remove manual approach instructions Feb 16, 2026
Copilot AI requested a review from pethers February 16, 2026 18:14
Copilot AI changed the title Fix agentic workflow MCP documentation - remove manual approach instructions Fix agentic workflow MCP tool syntax - use simple function names Feb 16, 2026
Copilot AI changed the title Fix agentic workflow MCP tool syntax - use simple function names Correct repository memories - agentic workflow MCP syntax is simple names only Feb 16, 2026
@github-actions github-actions bot added documentation Documentation updates workflow GitHub Actions workflows ci-cd CI/CD pipeline changes news News articles and content generation agentic-workflow Agentic workflow changes size-l Large change (250-1000 lines) labels Feb 16, 2026
@github-actions
Copy link
Contributor

🔍 Lighthouse Performance Audit

Category Score Status
Performance 85/100 🟡
Accessibility 95/100 🟢
Best Practices 90/100 🟢
SEO 95/100 🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot AI changed the title Correct repository memories - agentic workflow MCP syntax is simple names only Restore sandbox proxy instructions for Node.js script execution Feb 16, 2026
Copilot AI changed the title Restore sandbox proxy instructions for Node.js script execution Fix contradictory MCP documentation in agentic workflows Feb 16, 2026
@pethers pethers marked this pull request as ready for review February 16, 2026 19:00
@pethers pethers merged commit cc70625 into main Feb 16, 2026
1 check passed
@pethers pethers deleted the copilot/update-evening-analysis branch February 16, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agentic-workflow Agentic workflow changes ci-cd CI/CD pipeline changes documentation Documentation updates news News articles and content generation size-l Large change (250-1000 lines) workflow GitHub Actions workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants