[`PINO` BUG] wrong export of types in `pino` package

[x] I’ve read the docs of nestjs-pino

[x] I’ve read the docs of pino

[x] I couldn’t find the same open issue of nestjs-pino

What is the current behavior? When skipLibCheck is disabled in tsconfig.json compilation fails.

node_modules/nestjs-pino/params.d.ts:3:10 - error TS2614: Module '"pino"' has no exported member 'DestinationStream'. Did you mean to use 'import DestinationStream from "pino"' instead?

3 import { DestinationStream } from 'pino';
           ~~~~~~~~~~~~~~~~~

node_modules/nestjs-pino/params.d.ts:6:18 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino/pino"' has no exported member 'Logger'.

6     logger: pino.Logger;
                   ~~~~~~

node_modules/nestjs-pino/params.d.ts:13:25 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino-http/index"' has no exported member 'Options'.

13     pinoHttp?: pinoHttp.Options | DestinationStream | [pinoHttp.Options, DestinationStream];
                           ~~~~~~~

node_modules/nestjs-pino/params.d.ts:13:65 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino-http/index"' has no exported member 'Options'.

13     pinoHttp?: pinoHttp.Options | DestinationStream | [pinoHttp.Options, DestinationStream];
                                                                   ~~~~~~~

node_modules/nestjs-pino/PinoLogger.d.ts:3:38 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino/pino"' has no exported member 'BaseLogger'.

3 declare type PinoMethods = Pick<pino.BaseLogger, 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal'>;
                                       ~~~~~~~~~~

node_modules/nestjs-pino/PinoLogger.d.ts:11:32 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino/pino"' has no exported member 'Logger'.

11     static readonly root: pino.Logger;
                                  ~~~~~~

node_modules/nestjs-pino/PinoLogger.d.ts:29:24 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino/pino"' has no exported member 'Logger'.

29     get logger(): pino.Logger;
                          ~~~~~~

node_modules/nestjs-pino/PinoLogger.d.ts:30:25 - error TS2694: Namespace '"/nestjs-pino-bug/node_modules/pino/pino"' has no exported member 'Bindings'.

30     assign(fields: pino.Bindings): void;

What is the expected behavior? Library should work independently from skipLibCheck config flag.

Please provide minimal example repo. Without it this issue will be closed https://github.com/antspk/nestjs-pino-bug

Please mention other relevant information such as Node.js version and Operating System. node v16.13.0 macos 12.0.1

[iamolegga]

finally, all the types are fixed in v2.4.0. Big thanks to @jarcodallo . Unpinning this issue

[iamolegga]

OK, so pino-team had decided to move the types from @types/pino to the lib’s .d.ts file. Without backward compatibility

Subscribe to this issue and switch skipLibCheck until it’s fixed.

As this is not a bug of the current library I’m closing, but will pin it until it’s fixed.