Editor: use macConfigPath on macOS for MCP client config writes (Claude Desktop, etc.). Fallback to linuxConfigPath only if mac path missing.

dsarno · dsarno · commit 99d7a120c903 · 2025-08-25T15:26:06.000-07:00
diff --git a/UnityMcpBridge/Editor/Windows/MCPForUnityEditorWindow.cs b/UnityMcpBridge/Editor/Windows/MCPForUnityEditorWindow.cs
@@ -1413,7 +1413,14 @@ private string ConfigureMcpClient(McpClient mcpClient)
                 }
                 else if (
                     RuntimeInformation.IsOSPlatform(OSPlatform.OSX)
-                    || RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+                )
+                {
+                    configPath = string.IsNullOrEmpty(mcpClient.macConfigPath)
+                        ? mcpClient.linuxConfigPath
+                        : mcpClient.macConfigPath;
+                }
+                else if (
+                    RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
                 )
                 {
                     configPath = mcpClient.linuxConfigPath;
@@ -1455,7 +1462,14 @@ private string ConfigureMcpClient(McpClient mcpClient)
                 }
                 else if (
                     RuntimeInformation.IsOSPlatform(OSPlatform.OSX)
-                    || RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+                )
+                {
+                    configPath = string.IsNullOrEmpty(mcpClient.macConfigPath)
+                        ? mcpClient.linuxConfigPath
+                        : mcpClient.macConfigPath;
+                }
+                else if (
+                    RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
                 )
                 {
                     configPath = mcpClient.linuxConfigPath;
@@ -1567,7 +1581,14 @@ private void CheckMcpConfiguration(McpClient mcpClient)
                 }
                 else if (
                     RuntimeInformation.IsOSPlatform(OSPlatform.OSX)
-                    || RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+                )
+                {
+                    configPath = string.IsNullOrEmpty(mcpClient.macConfigPath)
+                        ? mcpClient.linuxConfigPath
+                        : mcpClient.macConfigPath;
+                }
+                else if (
+                    RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
                 )
                 {
                     configPath = mcpClient.linuxConfigPath;

Original file line number	Diff line number	Diff line change
`@@ -1413,7 +1413,14 @@ private string ConfigureMcpClient(McpClient mcpClient)`
`1413`	`1413`	`}`
`1414`	`1414`	`else if (`
`1415`	`1415`	`RuntimeInformation.IsOSPlatform(OSPlatform.OSX)`
`1416`		`- \|\| RuntimeInformation.IsOSPlatform(OSPlatform.Linux)`
	`1416`	`+ )`
	`1417`	`+ {`
	`1418`	`+ configPath = string.IsNullOrEmpty(mcpClient.macConfigPath)`
	`1419`	`+ ? mcpClient.linuxConfigPath`
	`1420`	`+ : mcpClient.macConfigPath;`
	`1421`	`+ }`
	`1422`	`+ else if (`
	`1423`	`+ RuntimeInformation.IsOSPlatform(OSPlatform.Linux)`
`1417`	`1424`	`)`
`1418`	`1425`	`{`
`1419`	`1426`	`configPath = mcpClient.linuxConfigPath;`
`@@ -1455,7 +1462,14 @@ private string ConfigureMcpClient(McpClient mcpClient)`
`1455`	`1462`	`}`
`1456`	`1463`	`else if (`
`1457`	`1464`	`RuntimeInformation.IsOSPlatform(OSPlatform.OSX)`
`1458`		`- \|\| RuntimeInformation.IsOSPlatform(OSPlatform.Linux)`
	`1465`	`+ )`
	`1466`	`+ {`
	`1467`	`+ configPath = string.IsNullOrEmpty(mcpClient.macConfigPath)`
	`1468`	`+ ? mcpClient.linuxConfigPath`
	`1469`	`+ : mcpClient.macConfigPath;`
	`1470`	`+ }`
	`1471`	`+ else if (`
	`1472`	`+ RuntimeInformation.IsOSPlatform(OSPlatform.Linux)`
`1459`	`1473`	`)`
`1460`	`1474`	`{`
`1461`	`1475`	`configPath = mcpClient.linuxConfigPath;`
`@@ -1567,7 +1581,14 @@ private void CheckMcpConfiguration(McpClient mcpClient)`
`1567`	`1581`	`}`
`1568`	`1582`	`else if (`
`1569`	`1583`	`RuntimeInformation.IsOSPlatform(OSPlatform.OSX)`
`1570`		`- \|\| RuntimeInformation.IsOSPlatform(OSPlatform.Linux)`
	`1584`	`+ )`
	`1585`	`+ {`
	`1586`	`+ configPath = string.IsNullOrEmpty(mcpClient.macConfigPath)`
	`1587`	`+ ? mcpClient.linuxConfigPath`
	`1588`	`+ : mcpClient.macConfigPath;`
	`1589`	`+ }`
	`1590`	`+ else if (`
	`1591`	`+ RuntimeInformation.IsOSPlatform(OSPlatform.Linux)`
`1571`	`1592`	`)`
`1572`	`1593`	`{`
`1573`	`1594`	`configPath = mcpClient.linuxConfigPath;`