sync doesn't work

[inwenis]

const pino = require('pino')
const logger = pino({
  transport: {
    target: 'pino-pretty',
    options: {
      colorize: true,
      sync: true
    }
  }
}, pino.destination({sync:true}))

console.log('yyy');
logger.info('hi');
console.log('xxx');

Expected:

yyy
[15:02:27.684] INFO (8368): hi
xxx

Actual:

yyy
xxx
[15:02:27.684] INFO (8368): hi

It works fine without pretty-print:

const pino = require('pino')
const logger = pino({}, pino.destination({sync:true}))

console.log('yyy');
logger.info('hi');
console.log('xxx');

Output:

yyy
{"level":30,"time":1712668061532,"pid":6376,"hostname":"LAPTOP-GOU7DNK8","msg":"hi"}
xxx

package.json

{
  "name": "test11",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "pino": "^8.20.0",
    "pino-pretty": "^11.0.0"
  }
}

[mcollina]

A few notes:

1. you are using both a transport and a destination. You can't have a both (we should probably throw)
2. the transport logic is inherently asynchronous, as it ran off thread. If you want synchronous pretty printing, you should just use it as a stream.

Adding some docs on all the above would be good, as it seems it's a source of confusion.

[inwenis]

thank you for the answer, that explains it

[psi-4ward]

the transport logic is inherently asynchronous, as it ran off thread. If you want synchronous pretty printing, you should just use it as a stream.

Can one give a short example?

[jsumners]

https://getpino.io/#/docs/api?id=pinooptions-destination-gt-logger
https://getpino.io/#/docs/api?id=destination-number-string-object-destinationstream-sonicboomopts-writablestream

[hanoii]

Can one give a short example?

import pretty from 'pino-pretty'

// Initialize logger
const logger = pino(pretty({sync: true}))

[psi-4ward]

import pretty from 'pino-pretty'

// Initialize logger
const logger = pino(pretty({sync: true}))

Yes but the request was how to use it with other transports.

I've solved it by constructing my streams manually:

const pinoOptions: LoggerOptions = {
  // Set the global log level to the lowest level
  // We adjust the level per transport
  level: 'trace',
  hooks: {},
  serializers: {
    // Handle error properties as Error and serialize them correctly
    err: pino.stdSerializers.err,
    error: pino.stdSerializers.err,
    validationErrors: pino.stdSerializers.err,
  },
};

const streams: StreamEntry[] = [];

streams.push({
  level: "info",
  stream: PinoPretty({
    sync: true,
    colorize: true,
  })
});

streams.push({
  level: level,
  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
  stream: pino.transport({
    target: './app.log'
  })
});

streams.push({
  level: "debug",
  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
  stream: pino.transport({
    target: 'pino-opentelemetry-transport',
  })
});

export const baseLogger = pino(pinoOptions, pino.multistream(streams));