NO_COLOR support

[jonathan-dev]

Goals

I want to be able to turn off colored output for the next command using the NO_COLOR convention.
If it is already possible to turn of the colored output I would like to know how since failed finding any documentation on that topic.

Non-Goals

No response

Background

a similar reqest has already been filed here in #54391

Proposal

Implement support for the NO_COLOR environment variable no color standard

[icyJoseph]

I see it is available here:

• https://github.com/vercel/next.js/blob/f1256388bb3d00ba328a5a90689f8763832a2bab/packages/next/src/lib/picocolors.ts#L21C1-L25C1

const enabled =
  env &&
  !env.NO_COLOR &&
  (env.FORCE_COLOR || (stdout?.isTTY && !env.CI && env.TERM !== 'dumb'))

If that's not enabled, then the formatter string, returns String, simply, to wrap the input string. 🤔

See here, https://stackblitz.com/edit/nextjs-fikshs4s?file=package.json:

[icyJoseph]

I've run out short on time today, but I think, this is where the code block/frame, you show is generated:

• next.js/packages/next/src/lib/typescript/diagnosticFormatter.ts

  Line 363 in f609222

  const { codeFrameColumns } = require('next/dist/compiled/babel/code-frame')

And it looks like Next.js hands over to their pre-compiled Babel version to format that code block... and it is that module that doesn't support NO_COLOR

But they do have this...:

process.argv||[],s=process.env;let i=!("NO_COLOR"in s

The mystery continues... I'll try to debug a bit more later on.

[icyJoseph]

The answer I needed was just a few lines further down... but I got distracted looking into the code-frame module:

• next.js/packages/next/src/lib/typescript/diagnosticFormatter.ts

  Line 391 in f609222

  { forceColor: true }

Modifying that directly, yields what you want:

I guess, I can make a PR where we make the forceColor, true, unless NO_COLOR is set by the user?

[jonathan-dev]

I guess that would make sense

[jonathan-dev]

Did you create the PR already?

[icyJoseph]

Hi, not yet. I plan to though. I just ran out of time before a vacation I had planned.