Skip to content

Conversation

Copy link

Copilot AI commented Jan 21, 2026

denoland/deno#31890 addresses crashes in Node compatibility when TTY streams are created with file descriptors beyond stdin/stdout/stderr. This PR adds a concise repository summary so consumers can quickly understand the scope and resolution.

  • Documentation update
    • Added a dedicated README section that captures the issue context, the op-level fix, and the JS TTYStream behavior.

Example excerpt:

## Upstream Issue Summary (denoland/deno#31890)

- Deno previously restricted TTY streams to file descriptors 0-2, which caused
  Node compatibility crashes when packages created TTY streams on other fds.
- The fix adds an op to open TTY resources from any valid terminal fd by safely
  duplicating the descriptor and validating it is a TTY.
- A JavaScript TTYStream wrapper now handles arbitrary fds, while ReadStream and
  WriteStream keep the 0-2 fast paths for stdin/stdout/stderr.
Original prompt

understand denoland#31890 and summerize


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: sigmaSd <22427111+sigmaSd@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix import issue in Deno environment Document denoland/deno#31890 TTY stream fix summary Jan 21, 2026
Copilot AI requested a review from sigmaSd January 21, 2026 17:30
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.

2 participants