test

Author: jonathanlab

apps/array/forge.config.ts

@@ -7,6 +7,14 @@ import { VitePlugin } from "@electron-forge/plugin-vite";
 import { PublisherGithub } from "@electron-forge/publisher-github";
 import type { ForgeConfig } from "@electron-forge/shared-types";
 
+// DEBUG: Log ARRAY_* env vars when forge config is loaded
+console.log("[forge.config.ts] ARRAY_* env vars:");
+for (const [key, value] of Object.entries(process.env)) {
+  if (key.startsWith("ARRAY_")) {
+    console.log(`  ${key}=${value}`);
+  }
+}
+
 const appleCodesignIdentity = process.env.APPLE_CODESIGN_IDENTITY;
 const appleTeamId = process.env.APPLE_TEAM_ID;
 const appleId = process.env.APPLE_ID;

apps/array/src/main/index.ts

@@ -10,6 +10,15 @@ import {
   type MenuItemConstructorOptions,
   shell,
 } from "electron";
+
+// DEBUG: Log all ARRAY_* env vars at startup
+console.log("[DEBUG] Main process env vars:");
+for (const [key, value] of Object.entries(process.env)) {
+  if (key.startsWith("ARRAY_")) {
+    console.log(`  ${key}=${value}`);
+  }
+}
+
 import "./lib/logger";
 import { ANALYTICS_EVENTS } from "../types/analytics.js";
 import {

apps/array/src/main/lib/shellManager.ts

@@ -103,12 +103,24 @@ class ShellManagerImpl {
     });
 
     ptyProcess.onData((data: string) => {
-      webContents.send(`shell:data:${sessionId}`, data);
+      try {
+        if (!webContents.isDestroyed()) {
+          webContents.send(`shell:data:${sessionId}`, data);
+        }
+      } catch {
+        // webContents was destroyed, ignore
+      }
     });
 
     ptyProcess.onExit(({ exitCode }) => {
       log.info(`Shell session ${sessionId} exited with code ${exitCode}`);
-      webContents.send(`shell:exit:${sessionId}`, { exitCode });
+      try {
+        if (!webContents.isDestroyed()) {
+          webContents.send(`shell:exit:${sessionId}`, { exitCode });
+        }
+      } catch {
+        // webContents was destroyed, ignore
+      }
       this.sessions.delete(sessionId);
       resolveExit({ exitCode });
     });

apps/array/src/main/services/fileWatcher.ts

@@ -210,14 +210,19 @@ class FileService {
 
   private emit(channel: string, data: unknown): void {
     try {
-      if (
-        this.mainWindow &&
-        !this.mainWindow.isDestroyed() &&
-        this.mainWindow.webContents &&
-        !this.mainWindow.webContents.isDestroyed()
-      ) {
-        this.mainWindow.webContents.send(channel, data);
-      }
+      const win = this.mainWindow;
+      if (!win) return;
+
+      // Check if window is destroyed before accessing webContents
+      if (typeof win.isDestroyed === "function" && win.isDestroyed()) return;
+
+      const wc = win.webContents;
+      if (!wc) return;
+
+      // Check if webContents is destroyed before sending
+      if (typeof wc.isDestroyed === "function" && wc.isDestroyed()) return;
+
+      wc.send(channel, data);
     } catch {
       // Window or webContents was destroyed, ignore
     }

apps/array/src/main/services/session-manager.ts

@@ -385,9 +385,13 @@ export class SessionManager {
     clientStreams: { readable: ReadableStream; writable: WritableStream },
   ): ClientSideConnection {
     const emitToRenderer = (payload: unknown) => {
-      const win = this.getMainWindow();
-      if (win && !win.isDestroyed()) {
-        win.webContents.send(channel, payload);
+      try {
+        const win = this.getMainWindow();
+        if (win && !win.isDestroyed() && !win.webContents.isDestroyed()) {
+          win.webContents.send(channel, payload);
+        }
+      } catch {
+        // Window or webContents was destroyed, ignore
       }
     };
 

apps/array/src/main/services/workspace/scriptRunner.ts

@@ -162,12 +162,20 @@ export class ScriptRunner {
   private emitTerminalCreated(
     info: WorkspaceTerminalInfo & { taskId: string },
   ): void {
-    const mainWindow = this.getMainWindow();
-    if (!mainWindow) {
-      log.warn("No main window available to emit terminal created event");
-      return;
+    try {
+      const mainWindow = this.getMainWindow();
+      if (
+        !mainWindow ||
+        mainWindow.isDestroyed() ||
+        mainWindow.webContents.isDestroyed()
+      ) {
+        log.warn("No main window available to emit terminal created event");
+        return;
+      }
+      mainWindow.webContents.send("workspace:terminal-created", info);
+    } catch {
+      // Window or webContents was destroyed, ignore
     }
-    mainWindow.webContents.send("workspace:terminal-created", info);
   }
 }
 

apps/array/src/main/services/workspace/workspaceService.ts

@@ -643,9 +643,17 @@ export class WorkspaceService {
   }
 
   private emitWorkspaceError(taskId: string, message: string): void {
-    const mainWindow = this.getMainWindow();
-    if (mainWindow) {
-      mainWindow.webContents.send("workspace:error", { taskId, message });
+    try {
+      const mainWindow = this.getMainWindow();
+      if (
+        mainWindow &&
+        !mainWindow.isDestroyed() &&
+        !mainWindow.webContents.isDestroyed()
+      ) {
+        mainWindow.webContents.send("workspace:error", { taskId, message });
+      }
+    } catch {
+      // Window or webContents was destroyed, ignore
     }
   }
 
@@ -654,13 +662,21 @@ export class WorkspaceService {
     title: string,
     message: string,
   ): void {
-    const mainWindow = this.getMainWindow();
-    if (mainWindow) {
-      mainWindow.webContents.send("workspace:warning", {
-        taskId,
-        title,
-        message,
-      });
+    try {
+      const mainWindow = this.getMainWindow();
+      if (
+        mainWindow &&
+        !mainWindow.isDestroyed() &&
+        !mainWindow.webContents.isDestroyed()
+      ) {
+        mainWindow.webContents.send("workspace:warning", {
+          taskId,
+          title,
+          message,
+        });
+      }
+    } catch {
+      // Window or webContents was destroyed, ignore
     }
   }
 }

scripts/start.sh

@@ -1,6 +1,9 @@
 #!/bin/bash
 set -e
 
+echo "DEBUG: ARRAY_WORKSPACE_NAME=$ARRAY_WORKSPACE_NAME"
+echo "DEBUG: ARRAY_WORKSPACE_PATH=$ARRAY_WORKSPACE_PATH"
+
 cd "$ARRAY_WORKSPACE_PATH"
 ARRAY_APP_IDENTIFIER="com.posthog.array.$ARRAY_WORKSPACE_NAME"
 WORKSPACE_DATA_DIR="$HOME/Library/Application Support/$ARRAY_APP_IDENTIFIER"
@@ -14,12 +17,15 @@ fi
 VITE_CACHE_DIR="$WORKSPACE_DATA_DIR/vite-cache"
 mkdir -p "$VITE_CACHE_DIR"
 
-# Maybe we should export these instead
-ARRAY_WORKSPACE_NAME="$ARRAY_WORKSPACE_NAME" \
-ARRAY_WORKSPACE_PATH="$ARRAY_WORKSPACE_PATH" \
-ARRAY_ROOT_PATH="$ARRAY_ROOT_PATH" \
-ARRAY_APP_IDENTIFIER="$ARRAY_APP_IDENTIFIER" \
-ARRAY_WORKSPACE_DATA_DIR="$WORKSPACE_DATA_DIR" \
-VITE_DEV_SERVER_PORT="$ARRAY_WORKSPACE_PORTS_START" \
-VITE_CACHE_DIR="$VITE_CACHE_DIR" \
+# Export all env vars so they're available to child processes
+export ARRAY_WORKSPACE_NAME
+export ARRAY_WORKSPACE_PATH
+export ARRAY_ROOT_PATH
+export ARRAY_APP_IDENTIFIER
+export ARRAY_WORKSPACE_DATA_DIR="$WORKSPACE_DATA_DIR"
+export VITE_DEV_SERVER_PORT="$ARRAY_WORKSPACE_PORTS_START"
+export VITE_CACHE_DIR
+
+echo "DEBUG: Exported ARRAY_WORKSPACE_NAME=$ARRAY_WORKSPACE_NAME"
+
 pnpm dev