🤖 fix: prevent IPC send to destroyed window

Add isDestroyed() checks before all mainWindow.webContents.send() calls
to prevent 'Object has been destroyed' errors when the window is closed
while background processes (like init stderr streams) are still emitting
events.

Fixes crash when closing window during workspace initialization.

Author: kylecarbs

src/node/services/ipcMain.ts

@@ -228,7 +228,7 @@ export class IpcMain {
     workspaceId: string,
     snapshot: WorkspaceActivitySnapshot | null
   ): void {
-    if (!this.mainWindow) {
+    if (!this.mainWindow || this.mainWindow.isDestroyed()) {
       return;
     }
     this.mainWindow.webContents.send(IPC_CHANNELS.WORKSPACE_ACTIVITY, {
@@ -508,15 +508,15 @@ export class IpcMain {
     });
 
     const chatUnsubscribe = session.onChatEvent((event) => {
-      if (!this.mainWindow) {
+      if (!this.mainWindow || this.mainWindow.isDestroyed()) {
         return;
       }
       const channel = getChatChannel(event.workspaceId);
       this.mainWindow.webContents.send(channel, event.message);
     });
 
     const metadataUnsubscribe = session.onMetadataEvent((event) => {
-      if (!this.mainWindow) {
+      if (!this.mainWindow || this.mainWindow.isDestroyed()) {
         return;
       }
       this.mainWindow.webContents.send(IPC_CHANNELS.WORKSPACE_METADATA, {
@@ -969,7 +969,7 @@ export class IpcMain {
           const session = this.sessions.get(workspaceId);
           if (session) {
             session.emitMetadata(updatedMetadata);
-          } else if (this.mainWindow) {
+          } else if (this.mainWindow && !this.mainWindow.isDestroyed()) {
             this.mainWindow.webContents.send(IPC_CHANNELS.WORKSPACE_METADATA, {
               workspaceId,
               metadata: updatedMetadata,
@@ -1379,7 +1379,9 @@ export class IpcMain {
             type: "delete",
             historySequences: deletedSequences,
           };
-          this.mainWindow.webContents.send(getChatChannel(workspaceId), deleteMessage);
+          if (this.mainWindow && !this.mainWindow.isDestroyed()) {
+            this.mainWindow.webContents.send(getChatChannel(workspaceId), deleteMessage);
+          }
         }
 
         return { success: true, data: undefined };
@@ -1417,7 +1419,7 @@ export class IpcMain {
           }
 
           // Send delete event to frontend for all old messages
-          if (deletedSequences.length > 0 && this.mainWindow) {
+          if (deletedSequences.length > 0 && this.mainWindow && !this.mainWindow.isDestroyed()) {
             const deleteMessage: DeleteMessage = {
               type: "delete",
               historySequences: deletedSequences,
@@ -1426,7 +1428,7 @@ export class IpcMain {
           }
 
           // Send the new summary message to frontend
-          if (this.mainWindow) {
+          if (this.mainWindow && !this.mainWindow.isDestroyed()) {
             this.mainWindow.webContents.send(getChatChannel(workspaceId), summaryMessage);
           }
 
@@ -1632,7 +1634,7 @@ export class IpcMain {
       const existingSession = this.sessions.get(workspaceId);
       if (existingSession) {
         existingSession.emitMetadata(null);
-      } else if (this.mainWindow) {
+      } else if (this.mainWindow && !this.mainWindow.isDestroyed()) {
         this.mainWindow.webContents.send(IPC_CHANNELS.WORKSPACE_METADATA, {
           workspaceId,
           metadata: null,
@@ -2091,7 +2093,7 @@ export class IpcMain {
         const chatChannel = getChatChannel(workspaceId);
 
         await session.replayHistory((event) => {
-          if (!this.mainWindow) {
+          if (!this.mainWindow || this.mainWindow.isDestroyed()) {
             return;
           }
           this.mainWindow.webContents.send(chatChannel, event.message);