Skip to content

Commit 81c5ea8

Browse files
author
Claude
committed
feat(setup): refine Unity MCP Asset Store compliance implementation
- Clean up menu items for production (hide dev/test items behind compile flag) - Integrate MCP client configuration into setup wizard as new step - Complete end-to-end setup experience with 6-step wizard - Enhanced client detection and auto-configuration - Production-ready menu structure with essential items only - Users are left 100% ready to use MCP after completing setup
1 parent 2fca7fc commit 81c5ea8

File tree

15 files changed

+746
-30
lines changed

15 files changed

+746
-30
lines changed

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Dependencies.meta

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Dependencies/DependencyManagerTests.cs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,10 @@ public static class DependencyManagerTests
1212
{
1313
/// <summary>
1414
/// Test basic dependency detection functionality
15+
/// Only available in development builds
1516
/// </summary>
16-
[UnityEditor.MenuItem("Window/MCP for Unity/Run Dependency Tests", priority = 100)]
17+
#if UNITY_EDITOR && MCP_DEVELOPMENT_MODE
18+
[UnityEditor.MenuItem("Window/MCP for Unity/Development/Run Dependency Tests", priority = 100)]
1719
public static void RunBasicTests()
1820
{
1921
Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Running Dependency Manager Tests...");
@@ -62,8 +64,9 @@ public static void RunBasicTests()
6264

6365
/// <summary>
6466
/// Test setup wizard functionality
67+
/// Only available in development builds
6568
/// </summary>
66-
[UnityEditor.MenuItem("Window/MCP for Unity/Test Setup Wizard", priority = 101)]
69+
[UnityEditor.MenuItem("Window/MCP for Unity/Development/Test Setup Wizard", priority = 101)]
6770
public static void TestSetupWizard()
6871
{
6972
Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Testing Setup Wizard...");
@@ -82,8 +85,9 @@ public static void TestSetupWizard()
8285

8386
/// <summary>
8487
/// Reset setup state for testing
88+
/// Only available in development builds
8589
/// </summary>
86-
[UnityEditor.MenuItem("Window/MCP for Unity/Reset Setup State (Test)", priority = 102)]
90+
[UnityEditor.MenuItem("Window/MCP for Unity/Development/Reset Setup State (Test)", priority = 102)]
8791
public static void ResetSetupStateForTesting()
8892
{
8993
Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Resetting setup state for testing...");
@@ -98,5 +102,6 @@ public static void ResetSetupStateForTesting()
98102
Debug.LogError($"<b><color=#FF6B6B>MCP-FOR-UNITY</color></b>: Setup state reset failed: {ex.Message}");
99103
}
100104
}
105+
#endif
101106
}
102107
}

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Dependencies/Validators.meta

Lines changed: 8 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Helpers/PortManager.cs.meta

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Installation/Downloaders.meta

Lines changed: 8 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Installation/Installers.meta

Lines changed: 8 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ava-worktrees/feature/ava-asset-store-compliance/UnityMcpBridge/Editor/Setup/SetupWizard.cs

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -221,29 +221,18 @@ public static void ResetSetupState()
221221
}
222222

223223
/// <summary>
224-
/// Force show setup wizard (for manual invocation)
224+
/// Show setup wizard (for manual invocation)
225225
/// </summary>
226226
[MenuItem("Window/MCP for Unity/Setup Wizard", priority = 1)]
227227
public static void ShowSetupWizardManual()
228228
{
229229
ShowSetupWizard();
230230
}
231231

232-
/// <summary>
233-
/// Reset setup and show wizard again
234-
/// </summary>
235-
[MenuItem("Window/MCP for Unity/Reset Setup", priority = 2)]
236-
public static void ResetAndShowSetup()
237-
{
238-
ResetSetupState();
239-
_hasCheckedThisSession = false;
240-
ShowSetupWizard();
241-
}
242-
243232
/// <summary>
244233
/// Check dependencies and show status
245234
/// </summary>
246-
[MenuItem("Window/MCP for Unity/Check Dependencies", priority = 3)]
235+
[MenuItem("Window/MCP for Unity/Check Dependencies", priority = 2)]
247236
public static void CheckDependencies()
248237
{
249238
var result = DependencyManager.CheckAllDependencies();
@@ -274,5 +263,18 @@ public static void CheckDependencies()
274263
);
275264
}
276265
}
266+
267+
#if UNITY_EDITOR && MCP_DEVELOPMENT_MODE
268+
/// <summary>
269+
/// Reset setup and show wizard again (development only)
270+
/// </summary>
271+
[MenuItem("Window/MCP for Unity/Development/Reset Setup", priority = 10)]
272+
public static void ResetAndShowSetup()
273+
{
274+
ResetSetupState();
275+
_hasCheckedThisSession = false;
276+
ShowSetupWizard();
277+
}
278+
#endif
277279
}
278280
}

0 commit comments

Comments
 (0)