Skip to content

fix: respect no color for inspected values#133

Merged
kvz merged 1 commit into
mainfrom
sev-no-color-inspect
May 28, 2026
Merged

fix: respect no color for inspected values#133
kvz merged 1 commit into
mainfrom
sev-no-color-inspect

Conversation

@kvz
Copy link
Copy Markdown
Member

@kvz kvz commented May 28, 2026

Why

Running the full SevLogger.playground.ts as an npm consumer from /tmp/foo worked, but showed one polish bug: NO_COLOR=1 disabled logger colors while inspected object values still emitted ANSI escape codes.

What changed

  • Route inspected non-string log values through the logger's color policy.
  • Disable inspect colors for file output and NO_COLOR=1.
  • Add a regression test for inspected object formatting under NO_COLOR=1.
  • Version @transloadit/sev-logger to 1.0.3 with a changelog entry.

Verification

  • corepack yarn install --immutable
  • corepack yarn dedupe --check
  • corepack yarn check
  • corepack yarn npm audit --recursive --all
  • git diff --check
  • /tmp/foo npm-consumer playground run against @transloadit/sev-logger@1.0.2 before the fix

@kvz kvz merged commit 1b01daa into main May 28, 2026
2 checks passed
@kvz kvz deleted the sev-no-color-inspect branch May 28, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant