chore(deps): Upgrade Pino to v10 - #2584
Conversation
Bumps pino 9.7.0 -> 10.3.1 in @cedarjs/api and @cedarjs/api-server (devDependencies), and pino-abstract-transport 1.2.0 -> 3.0.0 in @cedarjs/api-server (devDependencies). The only breaking change in either major bump is dropping Node.js 18 support, which Cedar already doesn't support (engines >=24). No source changes needed - both packages use standard, unaffected Pino APIs.
👷 Deploy request for cedarjs pending review.Visit the deploys page to approve it
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
Included review availability: Your plan provides up to 2 included reviews per hour; 0 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughChangesPino dependency upgrade
Merge Risk: ⚪ Minimal · up to This localized logging dependency upgrade changes no application source behavior and has passed the relevant builds and tests; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.) Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThe PR upgrades the API logging stack to Pino 10.3.1 and pino-abstract-transport 3.0.0 while retaining Cedar’s existing logging behavior.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains. Important Files Changed
Reviews (3): Last reviewed commit: "fix(mailer-core): Handle Pino v10 Logger..." | Re-trigger Greptile |
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run-many -t build --output-style=stream |
❌ Failed | 1m 50s | View ↗ |
💡 Dealing with memory or CPU issues? See memory and CPU details with the resource usage add-on ↗.
☁️ Nx Cloud last updated this comment at 2026-09-01 08:12:03 UTC
…nsole Pino v10's Logger type is no longer compatible with Console. When no Pino logger is provided to Mailer, it falls back to console. Work around the type mismatch by casting console to Logger.
4a5e4d7 to
13b5e34
Compare

Bumps
pino9.7.0 → 10.3.1 in@cedarjs/api(dependencies) and@cedarjs/api-server(devDependencies), andpino-abstract-transport1.2.0 → 3.0.0 in@cedarjs/api-server(devDependencies).Renovate flags major version bumps for manual dashboard approval, so this one had been sitting unapplied. Following up on #2583 (Express 5), this is the next most important major bump: Pino backs the logger used by every generated Cedar app's api-side server.
The only breaking change in either major version is dropping Node.js 18 support (Pino v10 release notes, pino-abstract-transport v3 release notes), which Cedar already doesn't support —
packages/api'senginesfield requires>=24.No source changes were needed.
packages/api/src/logger/index.tsandpackages/api-server's test suite both use standard, unaffected Pino APIs (pino(options, stream),logger.info/warn/error/debug, and the abstract transport'sbuild()helper).pinoisn't a peer dependency of any Cedar package, so no action is needed in Cedar apps.