fix: filter mappings by deviceid

packages/timeline-state-resolver/src/__tests__/conductor.spec.ts

@@ -58,10 +58,16 @@ describe('Conductor', () => {
 			deviceId: 'device1',
 			options: {},
 		}
-		const myLayerMapping: Mappings = {
+		const device0Mappings: Mappings = {
 			myLayer0: myLayerMapping0,
+		}
+		const device1Mappings: Mappings = {
 			myLayer1: myLayerMapping1,
 		}
+		const myLayerMapping: Mappings = {
+			...device0Mappings,
+			...device1Mappings,
+		}
 
 		const conductor = new Conductor({
 			multiThreadedResolver: false,
@@ -139,22 +145,22 @@ describe('Conductor', () => {
 					},
 					time: 10005,
 				}),
-				myLayerMapping // TODO - is this correct?
+				device0Mappings // TODO - is this correct?
 			)
 			expect(device0.handleState).toHaveBeenNthCalledWith(
 				2,
 				expect.objectContaining({
 					time: 11000,
 				}),
-				myLayerMapping // TODO - is this correct?
+				device0Mappings // TODO - is this correct?
 			)
 			expect(device0.handleState).toHaveBeenNthCalledWith(
 				3,
 				expect.objectContaining({
 					layers: {},
 					time: 12000,
 				}),
-				myLayerMapping // TODO - is this correct?
+				device0Mappings // TODO - is this correct?
 			)
 
 			// Ensure device1 has been fed sensible states
@@ -165,7 +171,7 @@ describe('Conductor', () => {
 				expect.objectContaining({
 					layers: {},
 				}),
-				myLayerMapping // TODO - is this correct?
+				device1Mappings // TODO - is this correct?
 			)
 			expect(device1.handleState).toHaveBeenNthCalledWith(
 				2,
@@ -180,14 +186,14 @@ describe('Conductor', () => {
 						}),
 					},
 				}),
-				myLayerMapping // TODO - is this correct?
+				device1Mappings // TODO - is this correct?
 			)
 			expect(device1.handleState).toHaveBeenNthCalledWith(
 				3,
 				expect.objectContaining({
 					layers: {},
 				}),
-				myLayerMapping // TODO - is this correct?
+				device1Mappings // TODO - is this correct?
 			)
 
 			// Remove the device

packages/timeline-state-resolver/src/conductor.ts

@@ -1087,8 +1087,13 @@ export class Conductor extends EventEmitter<ConductorEvents> {
 		deviceId: string,
 		time: number,
 		state: Timeline.TimelineState<TSRTimelineContent>,
-		mappings: Mappings
+		unfilteredMappings: Mappings
 	) {
+		// only take mappings that are for this deviceId
+		const mappings = Object.fromEntries(
+			Object.entries<Mapping<unknown>>(unfilteredMappings).filter(([_, mapping]) => mapping.deviceId === deviceId)
+		)
+
 		if (!this._deviceStates[deviceId]) this._deviceStates[deviceId] = []
 
 		// find all references to the datastore that are in this state