* fix: remove broken root rename assertion from prefab test
LoadAssetAtPath returns the asset filename as .name for prefab roots,
not the internally renamed root object name. Remove the rename + assert
that was causing CombinesWithOtherModifications to fail. The test still
verifies that componentProperties works alongside position changes.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: remove yield frames causing StdioBridge reconnect test flakiness
The 5-frame yield between client2 handshake and ping created a window
where the MCP Python server could reconnect and close our test client
as stale. Stale-client cleanup runs synchronously in HandleClientAsync
before the read loop, so no yield is needed after reading the handshake.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: ensure focus nudge fires and logs correctly during test polling
- Add file handler to root logger so __name__-based loggers (focus_nudge,
run_tests) write to the log file instead of only stderr
- Add fire-and-forget nudge check on non-wait_timeout get_test_job calls
so stalls are detected regardless of client polling style
- Add 13 unit tests for should_nudge logic, backoff reset, and gating
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: address PR #804 review feedback
- Store asyncio.create_task reference in module-level set to prevent GC (CodeRabbit)
- Add test for _get_frontmost_app() returning None (CodeRabbit nitpick)
- Add comment explaining why prefab root rename is not tested (Sourcery)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* docs: add Claude Code to MCP client examples in README
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
dsarno
1f6d242472