Skip to content

fix: remove pino-pretty (breaks serverless/Vercel)#24

Merged
priosshrsth merged 2 commits into
mainfrom
fix/remove-pino-pretty
Jul 14, 2026
Merged

fix: remove pino-pretty (breaks serverless/Vercel)#24
priosshrsth merged 2 commits into
mainfrom
fix/remove-pino-pretty

Conversation

@priosshrsth

Copy link
Copy Markdown
Collaborator

Why

pino-pretty uses a worker-thread transport that fails in serverless runtimes (Vercel, Cloudflare). It was wired into the dev-only logger transport.

Changes

  • src/logger/index.ts — drop the transport: { target: "pino-pretty" } branch; emit plain JSON in all environments. Pipe through the pino-pretty CLI locally for pretty output.
  • package.json — remove pino-pretty from devDependencies, peerDependencies, and peerDependenciesMeta.
  • README.md — update logger docs.

Verification

  • bun run check
  • bun run build
  • bun run test ✅ (172 passing)

🤖 Generated with Claude Code

priosshrsth and others added 2 commits July 14, 2026 08:13
pino-pretty uses a worker-thread transport that fails in serverless
runtimes. Drop the dev-only transport config and emit plain JSON in all
environments; pipe through the pino-pretty CLI locally for pretty output.
Removes pino-pretty from dependencies, peerDependencies, and peerDependenciesMeta.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@priosshrsth
priosshrsth enabled auto-merge (squash) July 14, 2026 08:18
@priosshrsth
priosshrsth merged commit b270adf into main Jul 14, 2026
5 checks passed
@priosshrsth
priosshrsth deleted the fix/remove-pino-pretty branch July 14, 2026 08:18
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