fix: support remote MCP server migration in mcp-migrator

[kiloconnect]

Problem

The MCP migrator at packages/opencode/src/kilocode/mcp-migrator.ts only handled stdio/local MCP servers. When a user configured a remote streamable-http or sse MCP server in .kilocode/mcp_settings.json:

{
  "mcpServers": {
    "local-mcp": {
      "type": "streamable-http",
      "url": "http://localhost:4321/mcp"
    }
  }
}

It failed with: The "file" argument must be of type string. Received undefined

This happened because convertServer() always assumed local servers, building a command array from server.command — which is undefined for remote servers.

Solution

Updated mcp-migrator.ts to detect remote transport types (streamable-http, sse) and convert them to Config.McpRemote (type: "remote") instead of always producing Config.McpLocal.

Changes

packages/opencode/src/kilocode/mcp-migrator.ts:

• Added REMOTE_TYPES set and isRemote() helper to detect remote server configs
• Updated convertServer() to branch on remote vs local servers
• Remote servers map to { type: "remote", url, headers? } (matching Config.McpRemote)
• Made command optional in KilocodeMcpServer interface since remote servers don't have it
• Existing stdio/local server migration is unchanged

packages/opencode/test/kilocode/mcp-migrator.test.ts:

• Added 11 new tests covering:
  • streamable-http server conversion
  • sse server conversion
  • Remote servers with headers/auth
  • Disabled remote servers
  • Empty/missing headers handling
  • Mixed stdio + remote server migration
  • End-to-end migration via migrate() for all remote scenarios

Test Results

All 27 tests pass (20 existing + 7 new remote-specific tests via migrate() + 4 new convertServer unit tests):

 27 pass
 0 fail
 50 expect() calls

Built for Marius Wichtner by Kilo for Slack

[github-actions]

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

1. Open an issue describing the bug/feature (if one doesn't exist)
2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

[kiloconnect]

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File	Line	Issue
packages/opencode/src/kilocode/mcp-migrator.ts	54	convertServer() skips invalid servers (e.g., missing url/command) without recording them in MigrationResult.skipped

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File	Line	Issue

Files Reviewed (2 files)

• packages/opencode/src/kilocode/mcp-migrator.ts - 1 issue
• packages/opencode/test/kilocode/mcp-migrator.test.ts - 0 issues