Node doesn't reset tty on early/aborted exit

Version

v16.13.1, v17.2.0 tested

Platform

Linux lux0 5.11.22-100.fc32.x86_64 #1 SMP Wed May 19 18:58:25 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Subsystem

No response

What steps will reproduce the bug?

Create this script file in test.js:

#!/usr/bin/env node
for (let i = 0; i < 100000; i++) {
console.log("i:", i)
}

In a shell window, run:

test.js |less

Hit "q" to quit less (after only one page of output), which also kills test.js while it still has undelivered data in the stdout buffer.

Your tty will be left in raw mode (no echo, other command characters disabled). Must do a "stty sane" to restore usability of the console.

How often does it reproduce? Is there a required condition?

Happens every time. Have tested multiple versions of node on Linux and MacOS.

What is the expected behavior?

Should restore my tty settings no matter how ugly the exit conditions.

What do you see instead?

Tty is left in raw mode with no echo. Can't see my typed commands.

Additional information

Is there a standard way of dealing with this in case a script is killed before it can write its output? Or is this a bug in node?

Bắt đầu bằng cách tái hiện báo cáo với script test.js và test.js | less, sau đó kiểm tra cách Node.js xử lý trạng thái terminal khi tiến trình thoát do bị hủy hoặc thoát sớm. Bổ sung kiểm thử hồi quy cho việc thoát khỏi less trong khi đầu ra vẫn còn được đệm, và xác minh rằng tty trở về trạng thái bình thường mà không cần stty sane.