Review

Verified against Node source and the enabled parallel/test-console-tty-colors.js.

Correctness — message matches Node exactly

Node's Console constructor at lib/internal/console/constructor.js:134 calls:

validateOneOf(colorMode, 'colorMode', ['auto', true, false]);

validateOneOf at lib/internal/validators.js:200-209 builds the reason string as:

const allowed = ArrayPrototypeJoin(
  ArrayPrototypeMap(oneOf, (v) =>
    (typeof v === 'string' ? `'${v}'` : String(v))),
  ', ');
const reason = 'must be one of: ' + allowed;

For ['auto', true, false]:

• 'auto' (string) → "'auto'"
• true (boolean) → "true"
• false (boolean) → "false"
• joined: "'auto', true, false"
• reason: "must be one of: 'auto', true, false"

PR hardcodes the same string — character-for-character match with what validateOneOf would have generated.

Message composition

Deno's ERR_INVALID_ARG_VALUE at ext/node/polyfills/internal/errors.ts:784-793 builds:

`The ${type} '${name}' ${reason}. Received ${inspected}`

With type='argument' (no dot in 'colorMode'), name='colorMode', reason="must be one of: 'auto', true, false", inspected=inspect(value), the result is:

The argument 'colorMode' must be one of: 'auto', true, false. Received <inspected>

Test expectation at test-console-tty-colors.js:70:

message: `The argument 'colorMode' must be one of: 'auto', true, false. Received ${received}`

Exact match. ✅

Test trace

The test throws-with-matching-message for [0, 'true', null, {}, [], () => {}]:

All trigger the PR's branch and produce the expected message.

Second test branch (not touched by PR, works already)

L76-93 tests that passing both colorMode and inspectOptions.colors throws ERR_INCOMPATIBLE_OPTION_PAIR. The polyfill at ext/node/polyfills/internal/console/constructor.mjs:167-175 already implements this — no change needed.

Nit — could have used validateOneOf directly

Node uses validateOneOf(colorMode, 'colorMode', ['auto', true, false]) rather than an inline typeof check. If Deno's validateOneOf exists in ext:deno_node/internal/validators.mjs (it does — used elsewhere in this codebase), swapping to it would be a truer Node port:

validateOneOf(colorMode, "colorMode", ["auto", true, false]);

Advantages:

1. Single call vs. typeof + literal, more readable.
2. Automatically keeps the reason string in sync if the accepted set ever changes.
3. Uses hideStackFrames so the stack trace doesn't include the validator frame (matching Node).

Not required for this PR — the string is stable, the behavior matches. Keep the existing form if you prefer, but the cleaner form is right there.

Safety / performance

• Single string allocation on the throw path. No change in happy path.
• ERR_INVALID_ARG_VALUE is a NodeTypeError subclass with code: 'ERR_INVALID_ARG_VALUE' — matches Node.
• No side effects.

Config

• Added parallel/test-console-tty-colors.js as a plain {} entry, alphabetically ordered between -console-table and -console-with-frozen-intrinsics. Clean.

LGTM.