Problem
The Claude Code setup path runs mcp remove as a silent best-effort step and ignores the result before running mcp add.
Evidence
Editor/MCPCliInstaller.ClaudeCode.cs:39-40 runs the remove command and discards the returned bool.Editor/MCPCliInstaller.ClaudeCode.cs:43-44 immediately runs add, relying on the CLI/fallback path to handle any stale state.
Impact
If remove fails due to corrupted config or permissions, add may fail, duplicate, or fall back to direct .mcp.json editing while the CLI-managed state remains broken.
Suggested fix
Treat remove failures as actionable unless the CLI reports the server was already absent. Surface the remove error or include it in the fallback message.
Source report
Imported from audit report item: Editor/MCPCliInstaller.ClaudeCode.cs:39.
Problem
The Claude Code setup path runs
mcp removeas a silent best-effort step and ignores the result before runningmcp add.Evidence
Editor/MCPCliInstaller.ClaudeCode.cs:39-40runs the remove command and discards the returned bool.Editor/MCPCliInstaller.ClaudeCode.cs:43-44immediately runs add, relying on the CLI/fallback path to handle any stale state.Impact
If remove fails due to corrupted config or permissions, add may fail, duplicate, or fall back to direct
.mcp.jsonediting while the CLI-managed state remains broken.Suggested fix
Treat remove failures as actionable unless the CLI reports the server was already absent. Surface the remove error or include it in the fallback message.
Source report
Imported from audit report item:
Editor/MCPCliInstaller.ClaudeCode.cs:39.