Add native Windows tmux backend#85
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Thanks for your submission! It's still currently in draft so unsure if that was intentional, if so when you are finished with your work. Please go ahead and remove all the documentation changes you made just keep the adapter code and tests. We will work on adjusting the documentation if the PR is merged. Also, at this time we have updated how multiplexers are selected and managed per issue #87's request so ensure compatibility with the newest changes in main. |
Summary
WindowsTmuxMultiplexerfor native Windows hosts usingtmux-windows.-ccwd arguments to-c .while spawning tmux from the requested cwd.codexresolve inside panes.pipe-paneno-op tradeoff on Windows.Verification
PYTHONUTF8=1 uv run --all-extras --group dev pytest tests/test_tui_launch.py tests/test_multiplexer.py tests/test_backend_registry.py tests/test_portability_guard.py -q-> 65 passeduv run --group dev ruff check src/bmad_loop/adapters/tmux_windows_backend.py src/bmad_loop/adapters/multiplexer.py tests/test_multiplexer.py tests/test_backend_registry.py tests/test_portability_guard.py tests/test_tui_launch.py-> passgit diff --check-> passbmad-loop validate --project C:\DevProjects\zerg-> WindowsTmuxMultiplexer available, codex found, hooks registeredPYTHONUTF8=1 uv run --all-extras --group dev pytest --collect-only -q-> 1512 tests collectedNotes
pytest -qrun on the Windows workstation exceeded a 184s command timeout before producing buffered output, so this PR is draft pending CI/full-suite confirmation.pipe_paneis deliberately a no-op for this backend because live tmux-windows tests showedpipe-panecan terminate the tmux server when spawning Windows pipe commands.