Skip to content

feat: trace pino transport target modules#571

Merged
jeffsee55 merged 1 commit intomainfrom
jeffsee55/pino-transport-trace
Mar 18, 2026
Merged

feat: trace pino transport target modules#571
jeffsee55 merged 1 commit intomainfrom
jeffsee55/pino-transport-trace

Conversation

@jeffsee55
Copy link
Contributor

@jeffsee55 jeffsee55 commented Mar 18, 2026

Pino's transport API accepts module specifiers as strings (e.g. pino.transport({ target: 'my-transport' })), which aren't detected by standard require/import analysis. This adds static analysis support for all three pino transport forms: single target, targets array, and pipeline array.

Pino's transport API accepts module specifiers as strings (e.g.
`pino.transport({ target: 'my-transport' })`), which aren't detected by
standard require/import analysis. This adds static analysis support for
all three pino transport forms: single `target`, `targets` array, and
`pipeline` array.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jeffsee55 jeffsee55 requested review from a team, icyJoseph, ijjk and styfle as code owners March 18, 2026 17:04
@jeffsee55 jeffsee55 merged commit 1402303 into main Mar 18, 2026
14 checks passed
@jeffsee55 jeffsee55 deleted the jeffsee55/pino-transport-trace branch March 18, 2026 17:21
@github-actions
Copy link

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

jeffsee55 added a commit that referenced this pull request Mar 20, 2026
The [previous pino support](#571) only
handled `pino.transport({ target: '...' })`. This adds support for two
additional patterns:
- `pino({ transport: { target: '...' } })` (constructor config)
- `fastify({ logger: { transport: { target: '...' } } })` (fastify
logger)

Also extracts shared transport config processing into a reusable helper.

<!--
PRs prefixed with `chore:` will skip creating a changelog entry and
release.
PRs prefixed with `fix:` will do a patch release.
PRs prefixed with `feat:` will do a minor release.
-->

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants