Fix task creation reload in dev (#17)

* fix(steps): guard against undefined summary crashing panel

StepEntry.summary is typed as required but AI-written steps.json may
omit it; truncate() called text.length unconditionally, throwing
TypeError: Cannot read properties of undefined (reading 'length').
Added ?? '' fallback matching existing pattern for detail/status.

* fix(steps): validate step entries and guard files_touched array check

Filter non-object entries in setStepsContent so null/primitive values
in steps.json never reach the render. Also tighten the files_touched
Show guard to Array.isArray to prevent iterating non-array values.

* fix(dev): preserve tasks on reload and exclude sibling worktrees from watch

- Expand preload allowlist check to match both single- and double-quoted
  channel names in preload.cjs (verifyPreloadAllowlist)
- Exclude sibling git worktree directories from Vite's file watcher to
  prevent spurious HMR reloads during multi-worktree dev sessions
- Persist task state immediately after createTask so tasks survive a
  dev reload; fire-and-forget since saveState handles errors internally

---------

Co-authored-by: Johannes Millan <johannes.millan@gmail.com>

Author: joshdoe

electron/main.ts

@@ -52,7 +52,9 @@ function verifyPreloadAllowlist(): void {
     const preloadPath = path.join(__dirname, '..', 'electron', 'preload.cjs');
     const preloadSrc = fs.readFileSync(preloadPath, 'utf8');
     const enumValues = new Set(Object.values(IPC));
-    const missing = [...enumValues].filter((v) => !preloadSrc.includes(`"${v}"`));
+    const hasChannel = (channel: string) =>
+      preloadSrc.includes(`'${channel}'`) || preloadSrc.includes(`"${channel}"`);
+    const missing = [...enumValues].filter((v) => !hasChannel(v));
     if (missing.length > 0) {
       console.warn(
         `[preload-sync] IPC channels missing from preload.cjs ALLOWED_CHANNELS: ${missing.join(', ')}`,

electron/vite.config.electron.ts

@@ -1,12 +1,22 @@
+import path from 'path';
 import { defineConfig } from 'vite';
 import solid from 'vite-plugin-solid';
 
+const rootDir = path.resolve(process.cwd());
+const parentDir = path.resolve(rootDir, '..');
+
 export default defineConfig({
   base: './',
   plugins: [solid()],
   clearScreen: false,
   server: {
     port: 1421,
     strictPort: true,
+    watch: {
+      ignored: (watchedPath) => {
+        const resolvedPath = path.resolve(watchedPath);
+        return resolvedPath.startsWith(parentDir) && !resolvedPath.startsWith(rootDir);
+      },
+    },
   },
 });

src/components/TaskStepsSection.tsx

@@ -284,7 +284,7 @@ export function TaskStepsSection(props: TaskStepsSectionProps) {
                               flex: '1',
                             }}
                           >
-                            {truncate(step.summary, 140)}
+                            {truncate(step.summary ?? '', 140)}
                           </span>
                         </div>
 
@@ -303,7 +303,11 @@ export function TaskStepsSection(props: TaskStepsSectionProps) {
                                 {truncate(step.detail ?? '', 280)}
                               </div>
                             </Show>
-                            <Show when={step.files_touched && step.files_touched.length > 0}>
+                            <Show
+                              when={
+                                Array.isArray(step.files_touched) && step.files_touched.length > 0
+                              }
+                            >
                               <div
                                 style={{
                                   display: 'flex',
@@ -355,7 +359,7 @@ export function TaskStepsSection(props: TaskStepsSectionProps) {
                         flex: '1',
                       }}
                     >
-                      {truncate(step().summary, 140)}
+                      {truncate(step().summary ?? '', 140)}
                     </span>
                     <Show when={step().timestamp}>
                       <span

src/store/tasks.ts

@@ -2,6 +2,7 @@ import { produce } from 'solid-js/store';
 import { invoke, Channel } from '../lib/ipc';
 import { IPC } from '../../electron/ipc/channels';
 import { store, setStore, cleanupPanelEntries } from './core';
+import { saveState } from './persistence';
 import { setTaskFocusedPanel } from './focus';
 import { getProject, getProjectPath, getProjectBranchPrefix, isProjectMissing } from './projects';
 import { setPendingShellCommand } from '../lib/bookmarks';
@@ -181,6 +182,7 @@ export async function createTask(opts: CreateTaskOptions): Promise<string> {
   };
 
   initTaskInStore(taskId, task, agent, projectId, agentDef);
+  saveState(); // fire-and-forget — errors handled internally
   return taskId;
 }
 
@@ -609,12 +611,10 @@ export function setPlanContent(
 }
 
 export function setStepsContent(taskId: string, steps: unknown[] | null): void {
-  setStore(
-    'tasks',
-    taskId,
-    'stepsContent',
-    steps && steps.length > 0 ? (steps as StepEntry[]) : undefined,
-  );
+  const valid = steps
+    ? (steps.filter((s) => s !== null && typeof s === 'object' && !Array.isArray(s)) as StepEntry[])
+    : [];
+  setStore('tasks', taskId, 'stepsContent', valid.length > 0 ? valid : undefined);
 }
 
 export function setTaskLastInputAt(taskId: string): void {