Consider adding stream metadata types to pino.d.ts

[segevfiner]

e.g.

const metadata = Symbol.for("pino.metadata");

export interface DestinationStreamMetadata {
  [metadata]?: boolean;
  lastLevel?: number;
  lastTime?: string;
  lastMsg?: string;
  lastObj?: object;
  lastLogger?: pino.Logger;
}

Though using discriminated union type narrowing might make for a better type:

const metadata = Symbol.for("pino.metadata");

interface DestinationStreamHasMetadata {
  [metadata]: true;
  lastLevel: number;
  lastTime: string;
  lastMsg: string;
  lastObj: object;
  lastLogger: pino.Logger;
}

export type DestinationStreamMetadata = { [metadata]?: boolean } | DestinationStreamHasMetadata;

[mcollina]

Note that I hope to remove all of those in a upcoming major. Those were a hack to support the old pretty print option.

[segevfiner]

The builtin multistream also uses this. What API would they be replaced with?

[mcollina]

Ah. I forgot about that one. I guess they are here to stay for one more cycle then. Would you like to send a PR to add the types?

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.