Skip runtime detection for containerized MCP servers

[Copilot]

Refactor runtime detection to skip containerized MCP servers

Since all MCP servers now run in containers, the runtime manager should not scan them for runtime requirements.

Plan

• Understand the issue
  • All MCP servers (github, playwright, serena, etc.) now run in Docker containers
  • Serena uses uvx command but gets auto-containerized via getWellKnownContainer()
  • Runtime detection still scans MCP Command fields and adds unnecessary runtime setup steps
  • Need to skip MCP servers that are containerized
• Modify detectFromMCPConfigs in runtime_detection.go
  • Check if MCP server has Container field set or Type is "docker"
  • Skip runtime detection for containerized MCP servers
  • Keep runtime detection only for non-containerized custom MCP servers
• Remove special Serena language detection
  • Modified detectSerenaLanguages() to return immediately with log message
  • Modified GenerateSerenaLanguageServiceSteps() to return empty slice
  • Language services are now inside the container, not on the host
• Update related tests
  • Updated tests that expected runtime detection from Serena
  • Updated MCP config tests to match new containerized format
  • Verified tests for custom MCP servers still work correctly
• Recompile workflow lock files
  • Ran make recompile to regenerate all workflow lock files
  • Updated 16 workflow lock files with new compiled output
• Merge main branch
  • Merged origin/main into feature branch
  • Resolved conflicts in workflow lock files
  • Recompiled to ensure consistency
• Run make agent-finish before final commit

Original prompt

Since all mcps are now running in tbe containers, the runtime manager does not need to scan them to determine which actions is needed to setup.

For example, serena runs from a docker container so it does not need uvx, and setup-uv anymore.

---

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

[pelikhan]

@copilot recompile

[Copilot]

@copilot recompile

Recompiled workflow files in bf49900. Updated 16 workflow lock files to reflect the removal of unnecessary runtime setup steps for containerized MCP servers.

[pelikhan]

@copilot merge main

[Copilot]

@copilot merge main

Merged main branch in e028cf5. All workflow lock files were automatically merged and recompiled successfully with no conflicts.

[github-actions]

🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

[github-actions]

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

[github-actions]

💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️

[github-actions]

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

[github-actions]

Smoke Test Results - Claude Engine (Run #20891059098)

Last 2 Merged PRs:

• Limit secrets passed to agentic engine execution steps #9589: Limit secrets passed to agentic engine execution steps
• Remove Bearer prefix from safe-inputs authorization headers #9594: Remove Bearer prefix from safe-inputs authorization headers

Test Results:

• ✅ GitHub MCP: Successfully fetched recent PRs
• ✅ Serena Go: Tool initialized successfully
• ✅ Playwright: Navigated to github.com (page title contains "GitHub")
• ❌ Safe Input gh Tool: Command not found (safeinputs-gh not available)
• ✅ File Writing: Created test file successfully
• ✅ Bash Tool: Read file back successfully

Status: PARTIAL PASS (5/6 tests passed)

AI generated by Smoke Claude