Skip to content

Commit f5613c6

Browse files
committed
refactor: remove unused validation and configuration helper methods
1 parent e5e49bf commit f5613c6

File tree

1 file changed

+0
-87
lines changed

1 file changed

+0
-87
lines changed

UnityMcpBridge/Editor/Windows/MCPForUnityEditorWindow.cs

Lines changed: 0 additions & 87 deletions
Original file line numberDiff line numberDiff line change
@@ -1099,51 +1099,6 @@ private void ToggleUnityBridge()
10991099
Repaint();
11001100
}
11011101

1102-
private static bool ValidateUvBinarySafe(string path)
1103-
{
1104-
try
1105-
{
1106-
if (!File.Exists(path)) return false;
1107-
var psi = new System.Diagnostics.ProcessStartInfo
1108-
{
1109-
FileName = path,
1110-
Arguments = "--version",
1111-
UseShellExecute = false,
1112-
RedirectStandardOutput = true,
1113-
RedirectStandardError = true,
1114-
CreateNoWindow = true
1115-
};
1116-
using var p = System.Diagnostics.Process.Start(psi);
1117-
if (p == null) return false;
1118-
if (!p.WaitForExit(3000)) { try { p.Kill(); } catch { } return false; }
1119-
if (p.ExitCode != 0) return false;
1120-
string output = p.StandardOutput.ReadToEnd().Trim();
1121-
return output.StartsWith("uv ");
1122-
}
1123-
catch { return false; }
1124-
}
1125-
1126-
private static bool ArgsEqual(string[] a, string[] b)
1127-
{
1128-
if (a == null || b == null) return a == b;
1129-
if (a.Length != b.Length) return false;
1130-
for (int i = 0; i < a.Length; i++)
1131-
{
1132-
if (!string.Equals(a[i], b[i], StringComparison.Ordinal)) return false;
1133-
}
1134-
return true;
1135-
}
1136-
1137-
private void ShowManualConfigurationInstructions(
1138-
string configPath,
1139-
McpClient mcpClient
1140-
)
1141-
{
1142-
mcpClient.SetStatus(McpStatus.Error, "Manual configuration required");
1143-
1144-
ShowManualInstructionsWindow(configPath, mcpClient);
1145-
}
1146-
11471102
// New method to show manual instructions without changing status
11481103
private void ShowManualInstructionsWindow(string configPath, McpClient mcpClient)
11491104
{
@@ -1231,42 +1186,6 @@ private string ConfigureMcpClient(McpClient mcpClient)
12311186
}
12321187
}
12331188

1234-
private void ShowCursorManualConfigurationInstructions(
1235-
string configPath,
1236-
McpClient mcpClient
1237-
)
1238-
{
1239-
mcpClient.SetStatus(McpStatus.Error, "Manual configuration required");
1240-
1241-
// Get the Python directory path using Package Manager API
1242-
string pythonDir = FindPackagePythonDirectory();
1243-
1244-
// Create the manual configuration message
1245-
string uvPath = FindUvPath();
1246-
if (uvPath == null)
1247-
{
1248-
UnityEngine.Debug.LogError("UV package manager not found. Cannot configure manual setup.");
1249-
return;
1250-
}
1251-
1252-
McpConfig jsonConfig = new()
1253-
{
1254-
mcpServers = new McpConfigServers
1255-
{
1256-
unityMCP = new McpConfigServer
1257-
{
1258-
command = uvPath,
1259-
args = new[] { "run", "--directory", pythonDir, "server.py" },
1260-
},
1261-
},
1262-
};
1263-
1264-
JsonSerializerSettings jsonSettings = new() { Formatting = Formatting.Indented };
1265-
string manualConfigJson = JsonConvert.SerializeObject(jsonConfig, jsonSettings);
1266-
1267-
ManualConfigEditorWindow.ShowWindow(configPath, manualConfigJson, mcpClient);
1268-
}
1269-
12701189
private void LoadValidationLevelSetting()
12711190
{
12721191
string savedLevel = EditorPrefs.GetString("MCPForUnity_ScriptValidationLevel", "standard");
@@ -1305,12 +1224,6 @@ private string GetValidationLevelDescription(int index)
13051224
};
13061225
}
13071226

1308-
public static string GetCurrentValidationLevel()
1309-
{
1310-
string savedLevel = EditorPrefs.GetString("MCPForUnity_ScriptValidationLevel", "standard");
1311-
return savedLevel;
1312-
}
1313-
13141227
private void CheckMcpConfiguration(McpClient mcpClient)
13151228
{
13161229
try

0 commit comments

Comments
 (0)