fix: uv_tty_init error on Windows

[sapphi-red]

This PR fixes the following error happening on Windows for non-tty terminals.

TTY initialization failed: uv_tty_init returned EBADF (bad file descriptor)

The error was happening this line.

clack/packages/core/src/prompts/prompt.ts

Line 136 in d6d9ce7

const sink = new WriteStream(0);

It seems calling new tty.WriteStream with non-tty file descriptor throws the error above.

I changed tty.WriteStream with stream.Writable and set terminal: true option for readline.createInterface so that readline will treat stream.Writable as a terminal.

I first went with replacing new WriteStream(0) with isatty(0) ? new WriteStream(0) : fs.createWriteStream(null, { fd: 0 }). It worked locally, but that didn't pass the tests.

refs #192

[changeset-bot]

🦋 Changeset detected

Latest commit: 17a011c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@clack/core	Patch
@clack/prompts	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Zamiell]

@sapphi-red Thanks so much! I am trying out clack for the first time on Windows, using the built-in Git Bash for Windows that comes automatically when you install Git, and I was confronted with this error.

Here's hoping we can get this merged soon with a new release!

[natemoo-re]

Thanks so much for the fix!