Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 19, 2026

Smoke test run 21112904973 failed on serena Go test with go: command not found. Root cause: serena MCP server was hardcoded to --context codex regardless of engine, causing wrong runtime environment.

Changes

Core fix:

  • pkg/workflow/mcp-config-serena.go: Added engineID parameter to renderSerenaMCPConfigWithOptions(), uses engine-specific context with fallback to "codex"
  • pkg/workflow/mcp_renderer.go: Added EngineID field to MCPRendererOptions, passes engine ID to serena renderer

Engine renderers:

  • Updated claude_mcp.go, codex_mcp.go, copilot_mcp.go to pass e.id when creating renderer

Tests:

  • Updated test calls to pass "codex" as engine ID parameter

Result

Workflows now use correct serena context:

  • Copilot workflows → --context copilot
  • Claude workflows → --context claude
  • Codex workflows → --context codex

Note: GitHub MCP test failure in same run is unrelated (Copilot agent not providing required tool parameters - requires Copilot CLI team investigation).

Original prompt

This section details on the original issue you should resolve

<issue_title>Smoke Test: Copilot - 21112904973</issue_title>
<issue_description># Smoke Test Results

Run ID: 21112904973
Timestamp: 2026-01-18T14:00:25.801Z
Status: ❌ FAIL

Test Results

Test Status
GitHub MCP ❌ Tool parameter errors, gh CLI not authenticated
Serena Go go: command not found
Playwright ✅ Successfully navigated to GitHub
File Writing ✅ Test file created
Bash Tool ✅ File verified

Passed: 3/5

View Run

AI generated by Smoke Copilot for #10562

Comments on the Issue (you are @copilot in this section)


✨ 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 and others added 2 commits January 19, 2026 23:13
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix parameter errors and command not found in Copilot smoke test Fix serena MCP context to use engine-specific values instead of hardcoded "codex" Jan 19, 2026
Copilot AI requested a review from mnkiefer January 19, 2026 23:24
@pelikhan pelikhan closed this Jan 20, 2026
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.

Smoke Test: Copilot - 21112904973

3 participants