fix(voip): guard mute sync by callUUID in performSetMutedCallAction

Prevent a stale/other CallKit session from flipping mute on the active
JS call by mirroring the UUID check pattern used in didToggleHoldCallAction.
Also adds iOS-specific tests for the mute handler.

Author: diegolmello

app/lib/services/voip/MediaCallEvents.ios.test.ts

@@ -0,0 +1,128 @@
+/**
+ * @jest-environment node
+ *
+ * iOS-only mute tests: requires isIOS = true so performSetMutedCallAction listener is registered.
+ */
+import { DeviceEventEmitter } from 'react-native';
+import RNCallKeep from 'react-native-callkeep';
+
+import { DEEP_LINKING } from '../../../actions/actionsTypes';
+import type { VoipPayload } from '../../../definitions/Voip';
+import NativeVoipModule from '../../native/NativeVoip';
+import { getInitialMediaCallEvents, setupMediaCallEvents } from './MediaCallEvents';
+import { useCallStore } from './useCallStore';
+
+const mockDispatch = jest.fn();
+const mockSetNativeAcceptedCallId = jest.fn();
+const mockAddEventListener = jest.fn();
+const mockRNCallKeepClearInitialEvents = jest.fn();
+const mockSetCurrentCallActive = jest.fn();
+
+jest.mock('../../methods/helpers', () => ({
+	...jest.requireActual('../../methods/helpers'),
+	isIOS: true
+}));
+
+jest.mock('../../store', () => ({
+	__esModule: true,
+	default: {
+		dispatch: (...args: unknown[]) => mockDispatch(...args)
+	}
+}));
+
+jest.mock('./useCallStore', () => ({
+	useCallStore: {
+		getState: jest.fn()
+	}
+}));
+
+jest.mock('../../native/NativeVoip', () => ({
+	__esModule: true,
+	default: {
+		clearInitialEvents: jest.fn(),
+		getInitialEvents: jest.fn(() => null)
+	}
+}));
+
+jest.mock('react-native-callkeep', () => ({
+	__esModule: true,
+	default: {
+		addEventListener: (...args: unknown[]) => mockAddEventListener(...args),
+		clearInitialEvents: (...args: unknown[]) => mockRNCallKeepClearInitialEvents(...args),
+		setCurrentCallActive: (...args: unknown[]) => mockSetCurrentCallActive(...args),
+		getInitialEvents: jest.fn(() => Promise.resolve([]))
+	}
+}));
+
+jest.mock('./MediaSessionInstance', () => ({
+	mediaSessionInstance: {
+		endCall: jest.fn()
+	}
+}));
+
+jest.mock('../restApi', () => ({
+	registerPushToken: jest.fn(() => Promise.resolve())
+}));
+
+const activeCallBase = {
+	call: {} as object,
+	callId: 'uuid-1',
+	nativeAcceptedCallId: null as string | null
+};
+
+function getMuteHandler(): (payload: { muted: boolean; callUUID: string }) => void {
+	const call = mockAddEventListener.mock.calls.find(([name]) => name === 'performSetMutedCallAction');
+	if (!call) {
+		throw new Error('performSetMutedCallAction listener not registered');
+	}
+	return call[1] as (payload: { muted: boolean; callUUID: string }) => void;
+}
+
+describe('setupMediaCallEvents — performSetMutedCallAction (iOS)', () => {
+	const toggleMute = jest.fn();
+	const getState = useCallStore.getState as jest.Mock;
+
+	beforeEach(() => {
+		jest.clearAllMocks();
+		toggleMute.mockClear();
+		mockAddEventListener.mockImplementation(() => ({ remove: jest.fn() }));
+		getState.mockReturnValue({ ...activeCallBase, isMuted: false, toggleMute });
+	});
+
+	it('registers performSetMutedCallAction via RNCallKeep.addEventListener', () => {
+		setupMediaCallEvents();
+		expect(mockAddEventListener).toHaveBeenCalledWith('performSetMutedCallAction', expect.any(Function));
+	});
+
+	it('calls toggleMute when muted state differs from OS and UUIDs match', () => {
+		setupMediaCallEvents();
+		getMuteHandler()({ muted: true, callUUID: 'uuid-1' });
+		expect(toggleMute).toHaveBeenCalledTimes(1);
+	});
+
+	it('does not call toggleMute when muted state already matches OS even if UUIDs match', () => {
+		getState.mockReturnValue({ ...activeCallBase, isMuted: true, toggleMute });
+		setupMediaCallEvents();
+		getMuteHandler()({ muted: true, callUUID: 'uuid-1' });
+		expect(toggleMute).not.toHaveBeenCalled();
+	});
+
+	it('drops event when callUUID does not match active call id', () => {
+		setupMediaCallEvents();
+		getMuteHandler()({ muted: true, callUUID: 'uuid-2' });
+		expect(toggleMute).not.toHaveBeenCalled();
+	});
+
+	it('drops event when there is no active call object even if UUIDs match', () => {
+		getState.mockReturnValue({
+			call: null,
+			callId: 'uuid-1',
+			nativeAcceptedCallId: null,
+			isMuted: false,
+			toggleMute
+		});
+		setupMediaCallEvents();
+		getMuteHandler()({ muted: true, callUUID: 'uuid-1' });
+		expect(toggleMute).not.toHaveBeenCalled();
+	});
+});

app/lib/services/voip/MediaCallEvents.test.ts

@@ -87,6 +87,14 @@ const activeCallBase = {
 	nativeAcceptedCallId: null as string | null
 };
 
+function getMuteHandler(): (payload: { muted: boolean; callUUID: string }) => void {
+	const call = mockAddEventListener.mock.calls.find(([name]) => name === 'performSetMutedCallAction');
+	if (!call) {
+		throw new Error('performSetMutedCallAction listener not registered');
+	}
+	return call[1] as (payload: { muted: boolean; callUUID: string }) => void;
+}
+
 describe('MediaCallEvents cross-server accept (slice 3)', () => {
 	const getState = useCallStore.getState as jest.Mock;
 

app/lib/services/voip/MediaCallEvents.ts

@@ -90,8 +90,16 @@ export const setupMediaCallEvents = (): (() => void) => {
 		);
 
 		subscriptions.push(
-			RNCallKeep.addEventListener('performSetMutedCallAction', ({ muted, callUUID: _callUUID }) => {
-				const { toggleMute, isMuted } = useCallStore.getState();
+			RNCallKeep.addEventListener('performSetMutedCallAction', ({ muted, callUUID }) => {
+				const { call, callId, nativeAcceptedCallId, toggleMute, isMuted } = useCallStore.getState();
+				const eventUuid = callUUID.toLowerCase();
+				const activeUuid = (callId ?? nativeAcceptedCallId ?? '').toLowerCase();
+
+				// No active media call or event is for another CallKit/Telecom session — drop stale closure state
+				if (!call || !activeUuid || eventUuid !== activeUuid) {
+					return;
+				}
+
 				// Sync mute state if it doesn't match what the OS is reporting
 				if (muted !== isMuted) {
 					toggleMute();