Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: CoplayDev/unity-mcp
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v6.2.4
Choose a base ref
...
head repository: CoplayDev/unity-mcp
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v6.2.5
Choose a head ref
  • 4 commits
  • 26 files changed
  • 2 contributors

Commits on Oct 23, 2025

  1. Fix port discovery protocol mismatch (#341)

    - Update _try_probe_unity_mcp to recognize Unity bridge welcome message
    - Unity bridge sends 'WELCOME UNITY-MCP' instead of JSON pong response
    - Maintains backward compatibility with JSON pong format
    - Fixes MCP server connection to Unity Editor
    dsarno authored Oct 23, 2025
    Configuration menu
    Copy the full SHA
    15bf793 View commit details
    Browse the repository at this point in the history

Commits on Oct 24, 2025

  1. Harden MCP tool parameter handling + add material workflow tests (TDD) (

    #343)
    
    * Add TDD tests for MCP material management issues
    
    - MCPMaterialTests.cs: Tests for material creation, assignment, and data reading
    - MCPParameterHandlingTests.cs: Tests for JSON parameter parsing issues
    - SphereMaterialWorkflowTests.cs: Tests for complete sphere material workflow
    
    These tests document the current issues with:
    - JSON parameter parsing in manage_asset and manage_gameobject tools
    - Material creation with properties
    - Material assignment to GameObjects
    - Material component data reading
    
    All tests currently fail (Red phase of TDD) and serve as specifications
    for what needs to be fixed in the MCP system.
    
    * Refine TDD tests to focus on actual MCP tool parameter parsing issues
    
    - Removed redundant tests that verify working functionality (GameObjectSerializer, Unity APIs)
    - Kept focused tests that document the real issue: MCP tool parameter validation
    - Tests now clearly identify the root cause: JSON string parsing in MCP tools
    - Tests specify exactly what needs to be fixed: parameter type flexibility
    
    The issue is NOT in Unity APIs or serialization (which work fine),
    but in MCP tool parameter validation being too strict.
    
    * Fix port discovery protocol mismatch
    
    - Update _try_probe_unity_mcp to recognize Unity bridge welcome message
    - Unity bridge sends 'WELCOME UNITY-MCP' instead of JSON pong response
    - Maintains backward compatibility with JSON pong format
    - Fixes MCP server connection to Unity Editor
    
    * Resolve merge: unify manage_gameobject param coercion and schema widening
    
    * Tests: add MaterialParameterToolTests; merge port probe fix; widen tool schemas for JSON-string params
    
    * refactor: extract JSON coercion helper; docs + exception narrowing\nfix: fail fast on bad component_properties JSON\ntest: unify setup, avoid test-to-test calls\nchore: use relative MCP package path
    
    * chore(tests): track MCPToolParameterTests.cs.meta to keep GUID stable
    
    * test: decouple MaterialParameterToolTests with helpers (no inter-test calls)
    dsarno authored Oct 24, 2025
    Configuration menu
    Copy the full SHA
    a0287af View commit details
    Browse the repository at this point in the history
  2. Material tools: support direct shader property keys + add EditMode co…

    …verage (#344)
    
    * Add TDD tests for MCP material management issues
    
    - MCPMaterialTests.cs: Tests for material creation, assignment, and data reading
    - MCPParameterHandlingTests.cs: Tests for JSON parameter parsing issues
    - SphereMaterialWorkflowTests.cs: Tests for complete sphere material workflow
    
    These tests document the current issues with:
    - JSON parameter parsing in manage_asset and manage_gameobject tools
    - Material creation with properties
    - Material assignment to GameObjects
    - Material component data reading
    
    All tests currently fail (Red phase of TDD) and serve as specifications
    for what needs to be fixed in the MCP system.
    
    * Refine TDD tests to focus on actual MCP tool parameter parsing issues
    
    - Removed redundant tests that verify working functionality (GameObjectSerializer, Unity APIs)
    - Kept focused tests that document the real issue: MCP tool parameter validation
    - Tests now clearly identify the root cause: JSON string parsing in MCP tools
    - Tests specify exactly what needs to be fixed: parameter type flexibility
    
    The issue is NOT in Unity APIs or serialization (which work fine),
    but in MCP tool parameter validation being too strict.
    
    * Fix port discovery protocol mismatch
    
    - Update _try_probe_unity_mcp to recognize Unity bridge welcome message
    - Unity bridge sends 'WELCOME UNITY-MCP' instead of JSON pong response
    - Maintains backward compatibility with JSON pong format
    - Fixes MCP server connection to Unity Editor
    
    * Resolve merge: unify manage_gameobject param coercion and schema widening
    
    * Tests: add MaterialParameterToolTests; merge port probe fix; widen tool schemas for JSON-string params
    
    * feat(material): support direct shader property keys and texture paths; add EditMode tests
    
    * chore(tests): track required .meta files; remove ephemeral Assets/Editor test helper
    
    * fix(manage_gameobject): validate parsed component_properties is a dict; return clear error
    dsarno authored Oct 24, 2025
    Configuration menu
    Copy the full SHA
    075d68d View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    4e6577c View commit details
    Browse the repository at this point in the history
Loading