Skip to content

refactor: Decompose ipc_server.rs into module directory#139

Merged
2witstudios merged 1 commit intomainfrom
pu/research-ipc-server
Mar 10, 2026
Merged

refactor: Decompose ipc_server.rs into module directory#139
2witstudios merged 1 commit intomainfrom
pu/research-ipc-server

Conversation

@2witstudios
Copy link
Owner

Summary

  • Extract streaming protocol handlers (attach, grid, status) from ipc_server.rs into ipc_server/streams.rs, isolating the highest-churn code from server lifecycle and dispatch logic
  • Move all 9 tests into ipc_server/tests.rs
  • Server lifecycle, dispatch, and wire helpers remain in ipc_server/mod.rs
  • Zero logic changes — purely structural extraction following the existing engine/ module pattern

Test plan

  • cargo clippy -p pu-engine — clean, no warnings
  • cargo test -p pu-engine --lib ipc_server — all 9 tests pass

🤖 Generated with Claude Code

Extract streaming protocol handlers (attach, grid, status) into
ipc_server/streams.rs and tests into ipc_server/tests.rs, isolating
the highest-churn code from server lifecycle and dispatch logic.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@2witstudios 2witstudios merged commit 124d5ea into main Mar 10, 2026
4 checks passed
@2witstudios 2witstudios deleted the pu/research-ipc-server branch March 10, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant