feat: migrate backend logging from Winston to Pino with correlation IDs (#82) #91

Merged

egullickson merged 2 commits from issue-82-pino-migration into main 2026-02-04 02:24:33 +00:00

Conversation 0 Commits 2 Files Changed 6 +141 -256

egullickson commented 2026-02-04 02:04:44 +00:00

Owner

Copy Link

Summary

• Replace Winston with Pino using API-compatible wrapper (79 importing files require NO changes)
• Add LOG_LEVEL env var support with validation and fallback to 'info'
• Add correlation ID middleware (extracts X-Request-Id from Traefik or generates UUID)
• Configure PostgreSQL logging env vars (POSTGRES_LOG_STATEMENT, POSTGRES_LOG_MIN_DURATION)
• Configure Redis loglevel via command args

Fixes #82

Test Plan

• npm run type-check passes
• npm run lint passes (pre-existing warnings only)
• Pino JSON output confirmed in test logs
• API wrapper maintains backward compatibility (no changes needed to importing files)

Generated with Claude Code

## Summary - Replace Winston with Pino using API-compatible wrapper (79 importing files require NO changes) - Add LOG_LEVEL env var support with validation and fallback to 'info' - Add correlation ID middleware (extracts X-Request-Id from Traefik or generates UUID) - Configure PostgreSQL logging env vars (POSTGRES_LOG_STATEMENT, POSTGRES_LOG_MIN_DURATION) - Configure Redis loglevel via command args Fixes #82 ## Test Plan - [x] `npm run type-check` passes - [x] `npm run lint` passes (pre-existing warnings only) - [x] Pino JSON output confirmed in test logs - [x] API wrapper maintains backward compatibility (no changes needed to importing files) Generated with Claude Code

egullickson added 1 commit 2026-02-04 02:04:45 +00:00

feat: migrate backend logging from Winston to Pino with correlation IDs (refs #82) ... 2a34f8225e

- Replace Winston with Pino using API-compatible wrapper
- Add LOG_LEVEL env var support with validation and fallback
- Add correlation ID middleware (X-Request-Id from Traefik or UUID)
- Configure PostgreSQL logging env vars (POSTGRES_LOG_STATEMENT, POSTGRES_LOG_MIN_DURATION)
- Configure Redis loglevel via command args

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

egullickson added 1 commit 2026-02-04 02:11:47 +00:00

chore: align production verify loop with staging (refs #82) ... 6c4d8e47f9

Add Docker healthcheck loop to production verify-prod job matching
staging's 24 attempts x 10 seconds = 4 minutes max wait for backend
migrations.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

egullickson merged commit 58eec46f72 into main 2026-02-04 02:24:33 +00:00

egullickson deleted branch issue-82-pino-migration 2026-02-04 02:24:33 +00:00

egullickson referenced this issue from a commit 2026-02-04 02:24:35 +00:00

Merge pull request 'feat: migrate backend logging from Winston to Pino with correlation IDs (#82)' (#91) from issue-82-pino-migration into main

egullickson referenced this pull request 2026-02-05 03:01:25 +00:00

[Feature]: Implement Unified Debug Logging System with Centralized Log Aggregation #80

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

status

backlog

status/blocked

status

done

status

in-progress

status

ready

status

review

type

bug

type

chore

type

docs

type

feature

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

egullickson

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: egullickson/motovaultpro#91