MCP server config uses unrecognized 'mcpServers' key in altimate-code.json #619

Activity

1. VJ-yadav

   added a commit that references this issue on Apr 3, 2026

   fix: accept mcpServers as alias for mcp in config

   ...

   This issue will close once commit c292be1 is merged into the 'main' branch.

   c292be1
2. anandgupta42 commented on Apr 5, 2026

   

   anandgupta42

   on Apr 5, 2026

   Contributor

   More actions

   Status: Fixed by #639 (merged April 4, 2026)

   The compatibility gap you reported — AI tools writing `mcpServers` instead of `mcp` in `altimate-code.json` — has been fully resolved.

   What was shipped

   #639 added `normalizeMcpConfig()` in `packages/opencode/src/config/config.ts:1372-1421` which does:

   1. Top-level key aliasing: `mcpServers` → `mcp` (with `log.warn` when migrated, `log.debug` when dropped due to conflict)
   2. Per-server shape translation: converts Claude Code / Copilot format to altimate-code's schema:
      • `{command: "npx", args: [...], env: {...}}` → `{type: "local", command: [...], environment: {...}}`
      • String `command` + `args` array → merged into `command` array
      • `env` → `environment`
      • Remote servers with `url` → `{type: "remote", url, headers}`
   3. Conflict handling: always deletes `mcpServers` from the parsed config before Zod schema validation, regardless of whether `mcp` is also present

   Adversarial test coverage

   Comprehensive tests at `packages/opencode/test/config/config.test.ts:346-490` covering:

   • Normal case: `mcpServers` only → aliased to `mcp`
   • Array `command` format → preserved as-is
   • Conflict: both `mcp` and `mcpServers` → `mcp` wins, `mcpServers` dropped silently
   • Remote servers without `type` field → auto-inferred as `"remote"`
   • Typed entries with external field names → normalized
   • Empty `mcpServers` object → loads without error
   • Malformed entries → rejected cleanly

   Verifying the fix

   # Drop an altimate-code.json with Claude Code's mcpServers format:
   {
     \"mcpServers\": {
       \"gitlab\": {
         \"command\": \"npx\",
         \"args\": [\"-y\", \"@modelcontextprotocol/server-gitlab\"],
         \"env\": { \"GITLAB_PERSONAL_ACCESS_TOKEN\": \"your-token\" }
       }
     }
   }

   altimate-code will auto-normalize it on load and log a warning instead of crashing. No config rewrite needed — the transformation happens in-memory before schema validation.

   Closing this issue as resolved. Thanks @VJ-yadav for reporting it!
3. anandgupta42

   closed this as completedon Apr 5, 2026