fix: don't set Sisyfos labels to empty strings (SOFIE-4196) (#406)

Author: jstarpl

packages/timeline-state-resolver/src/integrations/sisyfos/__tests__/sisyfos.spec.ts

@@ -1744,6 +1744,45 @@ describe('Sisyfos', () => {
 				}),
 			])
 		})
+
+		it("doesn't emit commands for empty `label` string", async () => {
+			expect(
+				await compareStates(
+					{
+						channels: {
+							0: {
+								faderLevel: undefined,
+								label: 'asfdadf asdfadsf',
+								timelineObjIds: [],
+								pgmOn: undefined,
+								pstOn: undefined,
+								visible: undefined,
+								inputGain: undefined,
+								inputSelector: undefined,
+								muteOn: undefined,
+							},
+						},
+						resync: false,
+					},
+					{
+						channels: {
+							0: {
+								faderLevel: undefined,
+								label: '',
+								timelineObjIds: [],
+								pgmOn: undefined,
+								pstOn: undefined,
+								visible: undefined,
+								inputGain: undefined,
+								inputSelector: undefined,
+								muteOn: undefined,
+							},
+						},
+						resync: false,
+					}
+				)
+			).toHaveLength(0)
+		})
 	})
 })
 

packages/timeline-state-resolver/src/integrations/sisyfos/connection.ts

@@ -212,7 +212,7 @@ export class SisyfosApi extends EventEmitter<SisyfosApiEvents> {
 			pgmOn: typeof apiState.pgmOn === 'number' ? apiState.pgmOn === 1 : undefined,
 			voOn: typeof apiState.pgmOn === 'number' ? apiState.pgmOn === 2 : undefined,
 			pstOn: typeof apiState.pstOn === 'number' ? apiState.pstOn === 1 : undefined,
-			label: apiState.label,
+			label: apiState.label !== '' ? apiState.label : undefined,
 			faderLevel: apiState.faderLevel,
 			muteOn: apiState.muteOn,
 			inputGain: apiState.inputGain,