fix: add default case for unknown DBNotificationTargetTypes

justandras · Julusian · web-flow · commit 793368cf9e74 · 2025-10-15T10:44:02.000+02:00
Co-authored-by: Julian Waller &lt;git@julusian.co.uk&gt;
diff --git a/packages/live-status-gateway/src/topics/helpers/notification/notificationTarget/toNotificationTarget.ts b/packages/live-status-gateway/src/topics/helpers/notification/notificationTarget/toNotificationTarget.ts
@@ -31,6 +31,9 @@ export function toNotificationTarget(dbTarget: DBNotificationTarget): Notificati
 			return toNotificationTargetPlaylist(dbTarget)
 		case DBNotificationTargetType.PIECEINSTANCE:
 			return toNotificationTargetPieceInstance(dbTarget)
+    default:
+      assertNever(dbTarget)
+      return undefined
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,9 @@ export function toNotificationTarget(dbTarget: DBNotificationTarget): Notificati`
`31`	`31`	`return toNotificationTargetPlaylist(dbTarget)`
`32`	`32`	`case DBNotificationTargetType.PIECEINSTANCE:`
`33`	`33`	`return toNotificationTargetPieceInstance(dbTarget)`
	`34`	`+ default:`
	`35`	`+ assertNever(dbTarget)`
	`36`	`+ return undefined`
`34`	`37`	`}`
`35`	`38`	`}`
`36`	`39`