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: v8.3.0
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: v8.4.0
Choose a head ref
  • 7 commits
  • 56 files changed
  • 5 contributors

Commits on Dec 22, 2025

  1. Fix on Issue #465 (#477)

    * Update ScreenshotUtility.cs
    
    Fix issue in issue#465
    
    * Update ScreenshotUtility.cs
    Scriptwonder authored Dec 22, 2025
    Configuration menu
    Copy the full SHA
    90758f1 View commit details
    Browse the repository at this point in the history

Commits on Dec 23, 2025

  1. Test/478 matrix4x4 serialization crash (#481)

    * Fix #478: Add Matrix4x4Converter to prevent Cinemachine serialization crash
    
    The `get_components` action crashes Unity when serializing Cinemachine
    camera components because Newtonsoft.Json accesses computed Matrix4x4
    properties (lossyScale, rotation) that call ValidTRS() on non-TRS matrices.
    
    This fix adds a safe Matrix4x4Converter that only accesses raw matrix
    elements (m00-m33), avoiding the dangerous computed properties entirely.
    
    Changes:
    - Add Matrix4x4Converter to UnityTypeConverters.cs
    - Register converter in GameObjectSerializer serializer settings
    
    Tested with Cinemachine 3.1.5 on Unity 6 - get_components now returns
    full component data without crashing.
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
    
    * Add unit tests for Matrix4x4Converter
    
    Tests cover:
    - Identity matrix serialization/deserialization
    - Translation matrix round-trip
    - Degenerate matrix (determinant=0) - key regression test
    - Non-TRS matrix (projection) - validates ValidTRS() is never called
    - Null handling
    - Ensures dangerous properties are not in output
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
    
    * Address code review feedback
    
    - Fix null handling consistency: return zero matrix instead of identity
      (consistent with missing field defaults of 0f)
    - Improve degenerate matrix test to verify:
      - JSON only contains raw mXY properties
      - Values roundtrip correctly
    - Rename test to reflect expanded coverage
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
    
    * Move tests to TestProject per review feedback
    
    Moved Matrix4x4ConverterTests from MCPForUnity/Editor/Tests/ to
    TestProjects/UnityMCPTests/Assets/Tests/EditMode/Helpers/ as requested.
    
    Also added MCPForUnity.Runtime reference to the test asmdef since
    the converter lives in the Runtime assembly.
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
    
    * Fix Matrix4x4 deserialization guard + UI Toolkit USS warning
    
    ---------
    
    Co-authored-by: Alexander Mangel <cygnusfear@gmail.com>
    Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
    3 people authored Dec 23, 2025
    Configuration menu
    Copy the full SHA
    91b6f4d View commit details
    Browse the repository at this point in the history

Commits on Dec 24, 2025

  1. Fix/ci cleanup (#484)

    * CI: tighten NL/T reporting and fail job on missing/failed tests
    
    * CI: include test titles in summary checklist
    dsarno authored Dec 24, 2025
    Configuration menu
    Copy the full SHA
    8e3bd1d View commit details
    Browse the repository at this point in the history

Commits on Dec 26, 2025

  1. Fix test teardown to avoid dropping MCP bridge (#487)

    * Fix test teardown to avoid dropping MCP bridge
    
    CodexConfigHelperTests was calling MCPServiceLocator.Reset() in TearDown, which disposes the active bridge/transport during MCP-driven test runs. Replace with restoring only the mutated service (IPlatformService).
    
    * Avoid leaking PlatformService in CodexConfigHelperTests
    
    Capture the original IPlatformService before this fixture runs and restore it in TearDown. This preserves the MCP connection safety fix (no MCPServiceLocator.Reset()) while avoiding global state leakage to subsequent tests.
    dsarno authored Dec 26, 2025
    Configuration menu
    Copy the full SHA
    523e81b View commit details
    Browse the repository at this point in the history

Commits on Dec 29, 2025

  1. feature/Add new manage_scriptable_object tool (#489)

    * Fix test teardown to avoid dropping MCP bridge
    
    CodexConfigHelperTests was calling MCPServiceLocator.Reset() in TearDown, which disposes the active bridge/transport during MCP-driven test runs. Replace with restoring only the mutated service (IPlatformService).
    
    * Avoid leaking PlatformService in CodexConfigHelperTests
    
    Capture the original IPlatformService before this fixture runs and restore it in TearDown. This preserves the MCP connection safety fix (no MCPServiceLocator.Reset()) while avoiding global state leakage to subsequent tests.
    
    * Fix SO MCP tooling: validate folder roots, normalize paths, expand tests; remove vestigial SO tools
    
    * Remove UnityMCPTests stress artifacts and ignore Assets/Temp
    
    * Ignore UnityMCPTests Assets/Temp only
    
    * Clarify array_resize fallback logic comments
    
    * Refactor: simplify action set and reuse slash sanitization
    
    * Enhance: preserve GUID on overwrite & support Vector/Color types in ScriptableObject tools
    
    * Fix: ensure asset name matches filename to suppress Unity warnings
    
    * Fix: resolve Unity warnings by ensuring asset name match and removing redundant import
    
    * Refactor: Validate assetName, strict object parsing for vectors, remove broken SO logic from ManageAsset
    
    * Hardening: reject Windows drive paths; clarify supported asset types
    
    * Delete FixscriptableobjecPlan.md
    
    * docs: Add manage_scriptable_object tool description to README
    dsarno authored Dec 29, 2025
    Configuration menu
    Copy the full SHA
    28f60b4 View commit details
    Browse the repository at this point in the history
  2. Payload-safe paging for hierarchy/components + safer asset search + d…

    …ocs (#490)
    
    * Fix test teardown to avoid dropping MCP bridge
    
    CodexConfigHelperTests was calling MCPServiceLocator.Reset() in TearDown, which disposes the active bridge/transport during MCP-driven test runs. Replace with restoring only the mutated service (IPlatformService).
    
    * Avoid leaking PlatformService in CodexConfigHelperTests
    
    Capture the original IPlatformService before this fixture runs and restore it in TearDown. This preserves the MCP connection safety fix (no MCPServiceLocator.Reset()) while avoiding global state leakage to subsequent tests.
    
    * Fix SO MCP tooling: validate folder roots, normalize paths, expand tests; remove vestigial SO tools
    
    * Remove UnityMCPTests stress artifacts and ignore Assets/Temp
    
    * Ignore UnityMCPTests Assets/Temp only
    
    * Clarify array_resize fallback logic comments
    
    * Refactor: simplify action set and reuse slash sanitization
    
    * Enhance: preserve GUID on overwrite & support Vector/Color types in ScriptableObject tools
    
    * Fix: ensure asset name matches filename to suppress Unity warnings
    
    * Fix: resolve Unity warnings by ensuring asset name match and removing redundant import
    
    * Refactor: Validate assetName, strict object parsing for vectors, remove broken SO logic from ManageAsset
    
    * Hardening: reject Windows drive paths; clarify supported asset types
    
    * Delete FixscriptableobjecPlan.md
    
    * Paginate get_hierarchy and get_components to prevent large payload crashes
    
    * dev: add uvx dev-mode refresh + safer HTTP stop; fix server typing eval
    
    * Payload-safe paging defaults + docs; harden asset search; stabilize Codex tests
    
    * chore: align uvx args + coercion helpers; tighten safety guidance
    
    * chore: minor cleanup + stabilize EditMode SO tests
    dsarno authored Dec 29, 2025
    Configuration menu
    Copy the full SHA
    35165e1 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    eea02d1 View commit details
    Browse the repository at this point in the history
Loading