openbot 0.4.2 → 0.4.3

package/dist/app/cli.js

@@ -16,7 +16,7 @@ function checkNodeVersion() {
     }
 }
 checkNodeVersion();
-program.name('openbot').description('OpenBot CLI').version('0.4.2');
+program.name('openbot').description('OpenBot CLI').version('0.4.3');
 program
     .command('start')
     .description('Start the OpenBot harness')

package/dist/plugins/delegation/index.js

@@ -70,6 +70,8 @@ export const delegationPlugin = {
                 channelId: context.state.channelId,
                 threadId: context.state.threadId,
                 publicBaseUrl: pluginContext.publicBaseUrl,
+                // Child events are re-yielded to the parent harness, which persists them once.
+                persistEvents: false,
                 onEvent: async (outEvent) => {
                     // Enrich events with parent metadata so the UI can track the hierarchy
                     const enrichedEvent = {

package/dist/plugins/storage/service.js

@@ -229,6 +229,12 @@ const readJsonFile = async (filePath, fallback) => {
         throw e;
     }
 };
+const writeJsonFileAtomically = async (filePath, data) => {
+    await fs.mkdir(path.dirname(filePath), { recursive: true });
+    const tmp = `${filePath}.tmp`;
+    await fs.writeFile(tmp, JSON.stringify(data, null, 2), 'utf-8');
+    await fs.rename(tmp, filePath);
+};
 const toVariablesRecord = (raw) => {
     if (!raw || typeof raw !== 'object') {
         return {};
@@ -408,8 +414,7 @@ export const storageService = {
             let displayName = name;
             let participants = [];
             try {
-                const stateContent = await fs.readFile(statePath, 'utf-8');
-                const parsed = JSON.parse(stateContent);
+                const parsed = await readJsonFile(statePath, {});
                 const fields = readChannelStateFileFields(parsed);
                 cwd = fields.cwd;
                 displayName = fields.name ?? name;
@@ -479,7 +484,7 @@ export const storageService = {
         await fs.mkdir(channelDir, { recursive: true });
         await fs.writeFile(specPath, spec?.trim() ||
             `# ${normalizedChannelId}\n\n`);
-        await fs.writeFile(statePath, JSON.stringify(finalState, null, 2));
+        await writeJsonFileAtomically(statePath, finalState);
     },
     deleteChannel: async ({ channelId }) => {
         const normalizedChannelId = channelId.trim();
@@ -540,8 +545,7 @@ export const storageService = {
         if (threadTitle?.trim()) {
             baseState.name = threadTitle.trim();
         }
-        await fs.mkdir(threadDir, { recursive: true });
-        await fs.writeFile(statePath, JSON.stringify(baseState, null, 2));
+        await writeJsonFileAtomically(statePath, baseState);
     },
     getThreads: async ({ channelId }) => {
         const threadsDir = resolvePath(resolveBaseDir() + '/' + DEFAULT_CHANNELS_DIR + '/' + channelId + '/threads');
@@ -560,8 +564,7 @@ export const storageService = {
             const threadStatePath = path.join(threadPath, 'state.json');
             let threadDisplayName = name;
             try {
-                const threadStateRaw = await fs.readFile(threadStatePath, 'utf-8');
-                const threadState = JSON.parse(threadStateRaw);
+                const threadState = await readJsonFile(threadStatePath, {});
                 const threadName = typeof threadState.name === 'string' ? threadState.name.trim() : '';
                 if (threadName) {
                     threadDisplayName = threadName;
@@ -597,8 +600,7 @@ export const storageService = {
         const statePath = `${threadDir}/state.json`;
         let state = {};
         try {
-            const stateContent = await fs.readFile(statePath, 'utf-8');
-            state = JSON.parse(stateContent);
+            state = await readJsonFile(statePath, {});
         }
         catch (error) {
             if (error?.code !== 'ENOENT') {
@@ -630,8 +632,7 @@ export const storageService = {
         }
         let state = {};
         try {
-            const stateContent = await fs.readFile(statePath, 'utf-8');
-            state = JSON.parse(stateContent);
+            state = await readJsonFile(statePath, {});
         }
         catch (error) {
             if (error?.code !== 'ENOENT') {
@@ -662,8 +663,7 @@ export const storageService = {
                 ...currentState,
                 ...patch,
             };
-            await fs.mkdir(channelDir, { recursive: true });
-            await fs.writeFile(statePath, JSON.stringify(newState, null, 2));
+            await writeJsonFileAtomically(statePath, newState);
         }
         catch (error) {
             console.error(`Failed to patch channel state for channel ${channelId}`, error);
@@ -680,8 +680,7 @@ export const storageService = {
                 ...currentState,
                 ...patch,
             };
-            await fs.mkdir(threadDir, { recursive: true });
-            await fs.writeFile(statePath, JSON.stringify(newState, null, 2));
+            await writeJsonFileAtomically(statePath, newState);
         }
         catch (error) {
             console.error(`Failed to patch thread state for channel ${channelId} thread ${threadId}`, error);

package/package.json

@@ -1,11 +1,16 @@
 {
   "name": "openbot",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "private": false,
   "type": "module",
   "engines": {
     "node": ">=20.12.0"
   },
+  "scripts": {
+    "dev": "tsx watch src/app/cli.ts start",
+    "build": "tsc && mkdir -p dist/assets && cp src/assets/icon.svg dist/assets/icon.svg",
+    "start": "node dist/app/cli.js start"
+  },
   "bin": {
     "openbot": "./dist/app/cli.js"
   },
@@ -30,10 +35,5 @@
     "@types/node": "^20.10.1",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3"
-  },
-  "scripts": {
-    "dev": "tsx watch src/app/cli.ts start",
-    "build": "tsc && mkdir -p dist/assets && cp src/assets/icon.svg dist/assets/icon.svg",
-    "start": "node dist/app/cli.js start"
   }
-}
+}

package/src/app/cli.ts

@@ -25,7 +25,7 @@ function checkNodeVersion() {
 
 checkNodeVersion();
 
-program.name('openbot').description('OpenBot CLI').version('0.4.2');
+program.name('openbot').description('OpenBot CLI').version('0.4.3');
 
 program
   .command('start')

package/src/plugins/delegation/index.ts

@@ -85,6 +85,8 @@ export const delegationPlugin: Plugin = {
         channelId: context.state.channelId,
         threadId: context.state.threadId,
         publicBaseUrl: pluginContext.publicBaseUrl,
+        // Child events are re-yielded to the parent harness, which persists them once.
+        persistEvents: false,
         onEvent: async (outEvent) => {
           // Enrich events with parent metadata so the UI can track the hierarchy
           const enrichedEvent = {

package/src/plugins/storage/service.ts

@@ -294,6 +294,13 @@ const readJsonFile = async <T>(filePath: string, fallback: T): Promise<T> => {
   }
 };
 
+const writeJsonFileAtomically = async (filePath: string, data: unknown): Promise<void> => {
+  await fs.mkdir(path.dirname(filePath), { recursive: true });
+  const tmp = `${filePath}.tmp`;
+  await fs.writeFile(tmp, JSON.stringify(data, null, 2), 'utf-8');
+  await fs.rename(tmp, filePath);
+};
+
 const toVariablesRecord = (raw: unknown): Record<string, string> => {
   if (!raw || typeof raw !== 'object') {
     return {};
@@ -502,8 +509,7 @@ export const storageService = {
         let participants: string[] = [];
 
         try {
-          const stateContent = await fs.readFile(statePath, 'utf-8');
-          const parsed = JSON.parse(stateContent);
+          const parsed = await readJsonFile(statePath, {});
           const fields = readChannelStateFileFields(parsed);
           cwd = fields.cwd;
           displayName = fields.name ?? name;
@@ -597,7 +603,7 @@ export const storageService = {
       spec?.trim() ||
       `# ${normalizedChannelId}\n\n`,
     );
-    await fs.writeFile(statePath, JSON.stringify(finalState, null, 2));
+    await writeJsonFileAtomically(statePath, finalState);
   },
   deleteChannel: async ({ channelId }: { channelId: string }): Promise<void> => {
     const normalizedChannelId = channelId.trim();
@@ -673,8 +679,7 @@ export const storageService = {
       baseState.name = threadTitle.trim();
     }
 
-    await fs.mkdir(threadDir, { recursive: true });
-    await fs.writeFile(statePath, JSON.stringify(baseState, null, 2));
+    await writeJsonFileAtomically(statePath, baseState);
   },
   getThreads: async ({ channelId }: { channelId: string }): Promise<Thread[]> => {
     const threadsDir = resolvePath(
@@ -698,8 +703,7 @@ export const storageService = {
         let threadDisplayName = name;
 
         try {
-          const threadStateRaw = await fs.readFile(threadStatePath, 'utf-8');
-          const threadState = JSON.parse(threadStateRaw) as Record<string, unknown>;
+          const threadState = await readJsonFile<Record<string, unknown>>(threadStatePath, {});
           const threadName =
             typeof threadState.name === 'string' ? threadState.name.trim() : '';
           if (threadName) {
@@ -748,8 +752,7 @@ export const storageService = {
 
     let state: unknown = {};
     try {
-      const stateContent = await fs.readFile(statePath, 'utf-8');
-      state = JSON.parse(stateContent);
+      state = await readJsonFile(statePath, {});
     } catch (error: unknown) {
       if ((error as NodeJS.ErrnoException)?.code !== 'ENOENT') {
         console.error(
@@ -787,8 +790,7 @@ export const storageService = {
 
     let state: unknown = {};
     try {
-      const stateContent = await fs.readFile(statePath, 'utf-8');
-      state = JSON.parse(stateContent);
+      state = await readJsonFile(statePath, {});
     } catch (error: unknown) {
       if ((error as NodeJS.ErrnoException)?.code !== 'ENOENT') {
         console.error(`Failed to read state file for channel ${channelId}`, error);
@@ -831,8 +833,7 @@ export const storageService = {
         ...(patch as Record<string, unknown>),
       };
 
-      await fs.mkdir(channelDir, { recursive: true });
-      await fs.writeFile(statePath, JSON.stringify(newState, null, 2));
+      await writeJsonFileAtomically(statePath, newState);
     } catch (error) {
       console.error(`Failed to patch channel state for channel ${channelId}`, error);
       throw error;
@@ -859,8 +860,7 @@ export const storageService = {
         ...(patch as Record<string, unknown>),
       };
 
-      await fs.mkdir(threadDir, { recursive: true });
-      await fs.writeFile(statePath, JSON.stringify(newState, null, 2));
+      await writeJsonFileAtomically(statePath, newState);
     } catch (error) {
       console.error(
         `Failed to patch thread state for channel ${channelId} thread ${threadId}`,