orion-ecs / keen-eye / 20818473799

65%

Build Type

push

github

Committed by tyevco

Commit Message

fix(mcp): Fix EntitySnapshot serialization and screenshot capture threading

Issue #843 - EntitySnapshot Serialization:
- Changed EntitySnapshot.Components to use JsonElement instead of object?
- Updated SerializeComponent() to return JsonElement via SerializeToElement()
- Added AOT suppression attributes for dynamic serialization
- state_query_entities and state_get_entity now work with component data

Issue #844 - Screenshot Capture Threading:
- Added InvokeOnRenderThreadAsync<T>() and InvokeOnRenderThread() to ILoopProvider
- Implemented render thread queue in SilkLoopProvider using ConcurrentQueue
- Updated CaptureControllerImpl to marshal GL operations to render thread
- capture_screenshot now works correctly from IPC/MCP requests

Also disabled state_get_component MCP tool due to framework serialization
limitations (see issue #853). Use state_get_entity as workaround.

Closes #843
Closes #844

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Coverage Stats

9156 of 12263 branches covered (74.66%)

Branch coverage included in aggregate %.

19 of 28 new or added lines in 8 files covered. (67.86%)

6 existing lines in 3 files now uncovered.

158441 of 179665 relevant lines covered (88.19%)

1.01 hits per line