cleanup, correctly resolve path

Author: jonathanlab

.gitignore

@@ -25,6 +25,7 @@ Thumbs.db
 # Temporary files
 *.tmp
 *.temp
+*.tgz
 
 AGENTS.md
 .claude/

forge.config.ts

@@ -16,10 +16,12 @@ const config: ForgeConfig = {
         {
           entry: "src/main/index.ts",
           config: "vite.main.config.mts",
+          target: "main",
         },
         {
           entry: "src/main/preload.ts",
           config: "vite.preload.config.mts",
+          target: "preload",
         },
       ],
       renderer: [

forge.env.d.ts

@@ -0,0 +1 @@
+/// <reference types="@electron-forge/plugin-vite/forge-vite-env" />

src/main/services/agent.ts

@@ -1,4 +1,8 @@
+import { execSync } from "node:child_process";
 import { randomUUID } from "node:crypto";
+import { existsSync } from "node:fs";
+import { homedir } from "node:os";
+import { join } from "node:path";
 import { Agent, PermissionMode } from "@posthog/agent";
 import { type BrowserWindow, type IpcMainInvokeEvent, ipcMain } from "electron";
 
@@ -36,6 +40,50 @@ function resolvePermissionMode(
   return (match as PermissionMode | undefined) ?? PermissionMode.ACCEPT_EDITS;
 }
 
+function findClaudeExecutable(): string | undefined {
+  // Common installation locations based on Claude Code docs
+  const commonPaths = [
+    join(homedir(), ".local", "bin", "claude"), // Native installer location
+    join(homedir(), ".claude", "local", "claude"), // Migrated local installation
+    join(homedir(), ".volta", "bin", "claude"), // Volta (Node version manager)
+    join(homedir(), ".nvm", "current", "bin", "claude"), // nvm
+    "/opt/homebrew/bin/claude", // Homebrew on Apple Silicon
+    "/usr/local/bin/claude", // Homebrew on Intel Mac / apt on Linux
+    "/usr/bin/claude", // System installation
+  ];
+
+  // Add npm global installation paths
+  try {
+    const npmPrefix = execSync("npm config get prefix", {
+      encoding: "utf-8",
+    }).trim();
+    if (npmPrefix) {
+      commonPaths.push(join(npmPrefix, "bin", "claude"));
+    }
+  } catch {
+    // npm not available or failed, continue
+  }
+
+  // Check common paths first
+  for (const path of commonPaths) {
+    if (existsSync(path)) {
+      return path;
+    }
+  }
+
+  // Fall back to using 'which' if available
+  try {
+    const path = execSync("which claude", { encoding: "utf-8" }).trim();
+    if (path && existsSync(path)) {
+      return path;
+    }
+  } catch {
+    // which command failed, continue
+  }
+
+  return undefined;
+}
+
 export function registerAgentIpc(
   taskControllers: Map<string, TaskController>,
   getMainWindow: () => BrowserWindow | null,
@@ -171,13 +219,21 @@ export function registerAgentIpc(
 
           const mcpOverrides = {};
 
+          const claudePath = findClaudeExecutable();
+          if (!claudePath) {
+            throw new Error(
+              "Claude Code executable not found in PATH. Please install Claude Code CLI.",
+            );
+          }
+
           await agent.runWorkflow(posthogTaskId, workflowId, {
             repositoryPath: repoPath,
             permissionMode: resolvedPermission,
             autoProgress: autoProgress ?? true,
             queryOverrides: {
               abortController,
               ...(model ? { model } : {}),
+              pathToClaudeCodeExecutable: claudePath,
               stderr: forwardClaudeStderr,
               env: envOverrides,
               mcpServers: mcpOverrides,

vite.main.config.mts

@@ -1,16 +1,41 @@
-import { defineConfig } from "vite";
+import { defineConfig, type Plugin } from "vite";
+
+/**
+ * Custom Vite plugin to fix circular __filename references in bundled ESM packages.
+ *
+ * When Vite bundles ESM packages like @posthog/agent that use import.meta.url,
+ * it transforms them into a complex polyfill that creates circular references:
+ * `const __filename2 = fileURLToPath(... pathToFileURL(__filename2) ...)`
+ *
+ * This plugin post-processes the bundle to replace the circular reference with
+ * a simple assignment to Node.js's global __filename variable.
+ */
+function fixFilenameCircularRef(): Plugin {
+  return {
+    name: "fix-filename-circular-ref",
+    enforce: "post",
+    generateBundle(_options, bundle) {
+      for (const fileName in bundle) {
+        const chunk = bundle[fileName];
+        if (chunk.type === "chunk") {
+          // Replace circular __filename references with direct __filename usage
+          chunk.code = chunk.code.replace(
+            /const __filename(\d+) = url\.fileURLToPath\(typeof document === "undefined" \? require\("url"\)\.pathToFileURL\(__filename\1\)\.href : [^;]+\);/g,
+            "const __filename$1 = __filename;",
+          );
+        }
+      }
+    },
+  };
+}
 
 export default defineConfig({
+  plugins: [fixFilenameCircularRef()],
   build: {
-    rollupOptions: {
-      onwarn(warning, warn) {
-        // Ignore unused import warnings from node_modules
-        if (warning.code === "UNUSED_EXTERNAL_IMPORT") return;
-        warn(warning);
-      },
+    target: "node18",
+    minify: false, // Disable minification to prevent variable name conflicts
+    commonjsOptions: {
+      transformMixedEsModules: true,
     },
   },
-  resolve: {
-    mainFields: ["module", "jsnext:main", "jsnext"],
-  },
 });

vite.preload.config.mts

@@ -1,7 +1,3 @@
 import { defineConfig } from "vite";
 
-export default defineConfig({
-  resolve: {
-    mainFields: ["module", "jsnext:main", "jsnext"],
-  },
-});
+export default defineConfig({});

vite.renderer.config.mts

@@ -7,15 +7,6 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 export default defineConfig({
   plugins: [react()],
-  base: "./",
-  build: {
-    outDir: "dist/renderer",
-    emptyOutDir: true,
-  },
-  server: {
-    port: 5173,
-    strictPort: true,
-  },
   resolve: {
     alias: {
       "@": path.resolve(__dirname, "./src"),