feat(DIS-101): introduce pino structured logger, remove console.* #45

Merged

dev merged 2 commits from phase-1/pino-logger into main

2026-04-09 15:40:47 +00:00

dev commented

2026-04-09 15:09:53 +00:00

Owner

Summary

Add pino (dependency) and pino-pretty (devDependency) to package.json
Introduce src/runtime/logger.ts with rootLogger and childLogger(component) factory
Replace all console.* calls across src/ with structured pino child-loggers
pino-pretty transport active only when NODE_ENV !== production
Update tests/unit/workspace-root-resolve.test.ts to spy on pino logger instead of console.warn
Add tests/unit/logger.test.ts verifying childLogger("runner") sets component: "runner" in bindings

Test plan

npm run build compiles without TypeScript errors
npm test � all 39 tests pass (8 test files)
No console.* remaining in src/
pino-pretty not used in production build

## Summary - Add `pino` (dependency) and `pino-pretty` (devDependency) to package.json - Introduce `src/runtime/logger.ts` with `rootLogger` and `childLogger(component)` factory - Replace all `console.*` calls across `src/` with structured pino child-loggers - `pino-pretty` transport active only when `NODE_ENV !== production` - Update `tests/unit/workspace-root-resolve.test.ts` to spy on pino logger instead of `console.warn` - Add `tests/unit/logger.test.ts` verifying `childLogger("runner")` sets `component: "runner"` in bindings ## Test plan - [ ] `npm run build` compiles without TypeScript errors - [ ] `npm test` � all 39 tests pass (8 test files) - [ ] No `console.*` remaining in `src/` - [ ] `pino-pretty` not used in production build

dev added 1 commit 2026-04-09 15:09:53 +00:00

feat(DIS-101): introduce pino structured logger, remove console.*

CI / build-and-test (ubuntu-latest) (pull_request) Has been cancelled

Details

CI / build-and-test (windows-latest) (pull_request) Has been cancelled

Details

CI / lint (pull_request) Has been cancelled

Details

1cafaabe2a

Replace all console.* calls in src/ with pino child-loggers.
Add rootLogger + childLogger(component) in src/runtime/logger.ts.
pino-pretty used as dev transport only.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

dev added 1 commit 2026-04-09 15:39:50 +00:00

merge main into phase-1/pino-logger

CI / build-and-test (ubuntu-latest) (pull_request) Has been cancelled

Details

CI / build-and-test (windows-latest) (pull_request) Has been cancelled

Details

CI / lint (pull_request) Has been cancelled

Details

e5502a51b2