fix: Detect tty for prqlc prompt

[max-sixty]

Closes #3228

[max-sixty]

FYI @aj-bagwell — not sure if this is a helpful @ or not — feel free to ignore if not.

The approach in https://github.com/PRQL/prql/pull/2885/files#diff-5042ecb23fbc2e8750e6452e008e9e96007bc06be4dff012653aedeaede02865R410 didn't seem to detect a tty correctly (or at least how I'm thinking about a TTY, where someone is typing into stdin, rather than piping text in).

More details in #3228. I spent 20 minutes looking through dependencies — it seems like clio is calling the same code, though the code changes slightly by versions, so I'm likely missing something.

We're v happy to stick with this for a bit while upgrades percolate through the ecosystem.

[eitsupi]

Thanks for working on this!
Could we add a test for this?

[max-sixty]

Thanks for working on this! Could we add a test for this?

For sure we should have one — but it should be in our tests already! e.g.

prql/crates/prqlc/tests/test.rs

Line 65 in 9743446

let mut cmd = StdinCommand::new(get_cargo_bin("prqlc"), "from tracks");

I think possibly it isn't picking it up because it's not a terminal, whereas the error was happening while we were in a terminal but not a tty? (I'm not tight on these concepts...)

[eitsupi]

Thank you for pointing that out. It seems difficult to test this....