Skip to content

Conversation

@Shourya742
Copy link
Contributor

@Shourya742 Shourya742 commented Dec 11, 2025

This PR adds a bidirectional proc-macro protocol so the server can ask the client for data while expanding a macro . The approach is still synchronous and single-request-at-a-time. The server may emit sub-requests during expansion, the client replies immediately, and the server then finishes with the final response. All messages use the same request id, and stdin/stdout remain strictly ordered. Legacy protocols are untouched

@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch 4 times, most recently from 0a41ccb to bb61e2e Compare December 11, 2025 17:12
@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch from bb61e2e to 376bca9 Compare December 15, 2025 14:00
@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch from 339c074 to 57fdf52 Compare December 18, 2025 08:36
@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch from a01d06d to 19e816d Compare December 18, 2025 09:17
@Shourya742 Shourya742 marked this pull request as ready for review December 18, 2025 09:34
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 18, 2025
Copy link
Contributor

@ChayimFriedman2 ChayimFriedman2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall good code, left some comments.

@Veykril would you like to review this too?

@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch from 92ae11e to 336f025 Compare December 23, 2025 02:29
@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch from df4a87f to 9b03499 Compare December 23, 2025 13:01
@Shourya742 Shourya742 requested a review from Veykril December 23, 2025 15:44
@Shourya742 Shourya742 force-pushed the 2025-11-27-bidirectional-protocol branch from e04b4bb to 41dade9 Compare December 23, 2025 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants