fix: make transports.options and transports.level optional (#1831)

* fix: make transports.options and transports.level optional According to https://github.com/pinojs/pino/blob/HEAD/docs/transports.md#v7-transports, the options and level are non-required properties for a transport * adding test for optional level/options
pinojs · Nov 15, 2023 · 40bdff5 · 40bdff5
1 parent 177c786
commit 40bdff5
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/pino.d.ts b/pino.d.ts
@@ -244,8 +244,8 @@ declare namespace pino {
 
     interface TransportTargetOptions<TransportOptions = Record<string, any>> {
         target: string
-        options: TransportOptions
-        level: LevelWithSilentOrString
+        options?: TransportOptions
+        level?: LevelWithSilentOrString
     }
 
     interface TransportBaseOptions<TransportOptions = Record<string, any>> {

diff --git a/test/types/pino-transport.test-d.ts b/test/types/pino-transport.test-d.ts
@@ -74,6 +74,23 @@ expectType<pino.Logger>(pino({
     },
 }))
 
+const transportsWithoutOptions = pino.transport({
+    targets: [
+        { target: '#pino/pretty' },
+        { target: '#pino/file' }
+    ], levels: { foo: 35 }
+})
+pino(transports)
+
+expectType<pino.Logger>(pino({
+    transport: {
+        targets: [
+            { target: '#pino/pretty' },
+            { target: '#pino/file' }
+        ], levels: { foo: 35 }
+    },
+}))
+
 const pipelineTransport = pino.transport({
     pipeline: [{
         target: './my-transform.js'