fix(cli): Redirect messages to stderr

[magic-akari]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: d4865bf

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

This PR includes changesets to release 1 package

Name	Type
@swc/cli	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

[kdy1]

Can you rebase?

[vercel]

@magic-akari is attempting to deploy a commit to the SWC Project Team on Vercel.

A member of the Team first needs to authorize it.

[rathpc]

Is there any context about why this change was made?

[kdy1]

Error messages should go to the stderr. It’s correct default, so we fixed it.

[Reflejo]

That makes sense, although this PR also changed a non-error message, aka:

-            console.log(message, duration.toFixed(2));
+            stderr.write(message);

which in turned broke nx https://github.com/nrwl/nx/blob/master/packages/js/src/utils/swc/compile-swc.ts#L95 - Pretty nasty the way they check for success (instead of exit code) but it broke nonetheless

[rathpc]

That makes sense, although this PR also changed a non-error message, aka:

-            console.log(message, duration.toFixed(2));
+            stderr.write(message);

which in turned broke nx https://github.com/nrwl/nx/blob/master/packages/js/src/utils/swc/compile-swc.ts#L95 - Pretty nasty the way they check for success (instead of exit code) but it broke nonetheless

This was the reason I ended up here actually. Attempted updating swc in an NX project and learned pretty quick it did not work as expected despite a successful build.

[kdy1]

•        console.log(message, duration.toFixed(2));
  

This is a log, so it should go to stderr