dda seems to merge stderr with stdout

[rdesgroppes]

As of version 0.29.0, messages printed to /dev/stderr seem to be redirected to /dev/stdout:

$ dda inv command-that-prints-to-both-streams 2>/dev/null
message_printed_to_stderr
message_printed_to_stdout

If I pretend I'm in CI, the behavior changes as follows:

$ CI=true dda inv command-that-prints-to-both-streams 2>/dev/null

message_printed_to_stdout

(with an extra newline)

I would expect the following output, whether in CI or not:

$ dda inv command-that-prints-to-both-streams 2>/dev/null
message_printed_to_stdout

$ CI=true dda inv command-that-prints-to-both-streams 2>/dev/null
message_printed_to_stdout

[ofek]

Can you check if this is fixed in the latest release? https://github.com/DataDog/datadog-agent-dev/releases/tag/v0.33.1

[rdesgroppes]

@ofek, here you go:

$ dda --version
dda, version 0.33.1

Bare invocation looks fine:

$ dda inv command-that-prints-to-both-streams
STDOUT
STDERR

Pretending we're "in CI" prepends a blank line and a progress message:

$ CI=true dda inv command-that-prints-to-both-streams
Synchronizing dependencies

STDOUT
STDERR

Redirecting stdout swallows stderr and adds a progress message:

$ dda inv command-that-prints-to-both-streams >/dev/null
Synchronizing dependencies

Redirecting stdout "in CI" is effective but prepends a progress message:

$ CI=true dda inv command-that-prints-to-both-streams >/dev/null
Synchronizing dependencies
STDERR

Redirecting stderr is ineffective but hides the progress message:

$ dda inv command-that-prints-to-both-streams 2>/dev/null
STDOUT
STDERR

Redirecting stderr "in CI" is effective but prepends a blank line:

$ CI=true dda inv command-that-prints-to-both-streams 2>/dev/null

STDOUT