Skip to content

Conversation

@ValentaTomas
Copy link
Member

@ValentaTomas ValentaTomas commented Jan 22, 2026

Note

Introduces a way to signal EOF to non-PTY processes and hardens stdin handling.

  • Adds CloseStdin RPC to process.proto with generated clients/handlers in both internal and shared packages, and wires routing in processconnect files
  • Implements Handler.CloseStdin() and guards stdin with stdinMu; WriteStdin now synchronized and returns "stdin not enabled or closed" when unavailable
  • Exposes CloseStdin via service method in input.go that resolves a process and calls handler
  • Regenerates protobufs: new CloseStdinRequest/Response messages and updated method/type indexes

Written by Cursor Bugbot for commit ca4fd7f. This will update automatically on new commits. Configure here.

@ValentaTomas ValentaTomas changed the title Add stdin close to correctly handle command that wait for stdin EOF Add stdin close to envd correctly handle command that wait for stdin EOF Jan 22, 2026
@ValentaTomas ValentaTomas changed the title Add stdin close to envd correctly handle command that wait for stdin EOF Add stdin close to envd to correctly handle command that wait for stdin EOF Jan 22, 2026
Copy link
Contributor

@levb levb left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants