From c8b1d20e9446c9b2f9ab971773530795bc62ba40 Mon Sep 17 00:00:00 2001 From: Naiyuan Qing <145280634+NevilleQingNY@users.noreply.github.com> Date: Thu, 15 Jan 2026 20:43:15 +0800 Subject: [PATCH] fix: use correct multica session ID for permission request filtering After merging main branch, the permission request filtering in ChatView was not working because PermissionManager was sending ACP session ID instead of multica session ID. This caused AskUserQuestion UI to not display because the session ID comparison failed. Co-Authored-By: Claude Opus 4.5 --- src/main/permission/PermissionManager.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/main/permission/PermissionManager.ts b/src/main/permission/PermissionManager.ts index 0129296e21..2e0a824763 100644 --- a/src/main/permission/PermissionManager.ts +++ b/src/main/permission/PermissionManager.ts @@ -43,13 +43,15 @@ export class PermissionManager { ) const mainWindow = this.getMainWindow() + // Get the multica session ID (internal) from ACP session ID + const multicaSessionId = this.conductor.getMulticaSessionIdByAcp(params.sessionId) // Send permission request to renderer if (mainWindow && !mainWindow.isDestroyed()) { mainWindow.webContents.send(IPC_CHANNELS.PERMISSION_REQUEST, { requestId, sessionId: params.sessionId, - multicaSessionId: params.sessionId, // ACP session ID + multicaSessionId: multicaSessionId || params.sessionId, // Use multica session ID for filtering toolCall: { toolCallId: params.toolCall.toolCallId, title: params.toolCall.title,