@adecore/agents 0.33.3 → 0.34.1

package/dist/chat/chat-process.d.ts

@@ -13,7 +13,7 @@ export interface ChatSpawnOptions {
     command: string[];
     cwd: string;
     env: Record<string, string>;
-    onExit(exitCode: number | null): void;
+    onExit(exitCode: number | null, signal?: string | null): void;
     onError?(error: Error): void;
 }
 export type SpawnChatProcess = (options: ChatSpawnOptions) => ChatProcess;
@@ -48,7 +48,7 @@ export declare class LineSplitter {
 export declare function readStreamLines(stream: ReadableStream<Uint8Array>, onLine: (line: string) => void, onOverflow: () => void, max?: number): Promise<void>;
 export interface ChatChildOptions extends Omit<ChatSpawnOptions, 'onExit'> {
     spawn?: SpawnChatProcess;
-    onExit(exitCode: number | null, stderr: string | null): void;
+    onExit(exitCode: number | null, stderr: string | null, signal?: string | null): void;
     onError?(error: Error): void;
 }
 export declare class ChatChild {

package/dist/chat/chat-process.js

@@ -26,12 +26,12 @@ export function spawnChatProcess(options) {
         throw new Error(`Could not start ${command}: no such executable`);
     }
     child.stdin.on('error', (error) => options.onError?.(error));
-    child.on('exit', (exitCode) => {
+    child.on('exit', (exitCode, signal) => {
         // The last lines may still be on their way through the pipe; they belong before the exit.
         const report = () => {
             clearTimeout(timer);
             child.stdout.off('end', report);
-            setImmediate(() => options.onExit(exitCode));
+            setImmediate(() => options.onExit(exitCode, signal));
         };
         const timer = setTimeout(report, STDOUT_DRAIN_MS);
         if (child.stdout.readableEnded) {
@@ -225,11 +225,11 @@ export class ChatChild {
             cwd: options.cwd,
             env: options.env,
             ...(options.onError ? { onError: options.onError } : {}),
-            onExit: (exitCode) => {
+            onExit: (exitCode, signal = null) => {
                 this.hasExited = true;
                 this.clearGrace();
                 settle();
-                void this.report(exitCode, options.onExit);
+                void this.report(exitCode, signal, options.onExit);
             }
         });
         this.tail = this.process.stderr ? new StreamTail(this.process.stderr) : null;
@@ -274,9 +274,9 @@ export class ChatChild {
             this.graceTimer = null;
         }
     }
-    async report(exitCode, onExit) {
+    async report(exitCode, signal, onExit) {
         if (exitCode === null || exitCode === 0 || this.tail === null) {
-            onExit(exitCode, null);
+            onExit(exitCode, null, signal);
             return;
         }
         let release = () => undefined;
@@ -288,6 +288,6 @@ export class ChatChild {
             })
         ]);
         clearTimeout(timer);
-        onExit(exitCode, this.tail.lines());
+        onExit(exitCode, this.tail.lines(), signal);
     }
 }

package/dist/record-directory.js

@@ -82,7 +82,8 @@ export class RecordDirectory {
     /*
      * Holds the record in memory at once, so the next question in the same tick already sees it, and
      * answers whether it is still the one this directory holds by the time the file is written. A
-     * record removed while its file was being written does not come back: the rename is undone.
+     * record removed while its file was being written does not come back: the rename is undone, unless
+     * a newer record took the id meanwhile, whose own write is queued behind this one and replaces the file.
      */
     write(record) {
         const id = this.idOf(record);
@@ -93,11 +94,13 @@ export class RecordDirectory {
             }
             await mkdir(this.dir, { recursive: true, mode: 0o700 });
             await this.writeFile(this.pathOf(id), JSON.stringify(record));
+            if (this.records.get(id) === record) {
+                return true;
+            }
             if (!this.records.has(id)) {
                 await this.removeFile(this.pathOf(id));
-                return false;
             }
-            return true;
+            return false;
         });
     }
     /* Forgets a record here and now; its file follows, after what is queued for it. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adecore/agents",
-  "version": "0.33.3",
+  "version": "0.34.1",
   "description": "Node-compatible agent chat hosts, providers, accounts, usage and durable task coordination.",
   "type": "module",
   "exports": {
@@ -81,8 +81,8 @@
     "test": "bun --conditions=source test"
   },
   "dependencies": {
-    "@adecore/agent-contracts": "0.33.3",
-    "@adecore/intelligent-ui": "0.33.3",
+    "@adecore/agent-contracts": "0.34.1",
+    "@adecore/intelligent-ui": "0.34.1",
     "zod": "^4.6.5"
   },
   "devDependencies": {

package/src/chat/chat-process.ts

@@ -24,7 +24,7 @@ export interface ChatSpawnOptions {
     command: string[];
     cwd: string;
     env: Record<string, string>;
-    onExit(exitCode: number | null): void;
+    onExit(exitCode: number | null, signal?: string | null): void;
     onError?(error: Error): void;
 }
 
@@ -56,12 +56,12 @@ export function spawnChatProcess(options: ChatSpawnOptions): ChatProcess {
         throw new Error(`Could not start ${command}: no such executable`);
     }
     child.stdin.on('error', (error) => options.onError?.(error));
-    child.on('exit', (exitCode) => {
+    child.on('exit', (exitCode, signal) => {
         // The last lines may still be on their way through the pipe; they belong before the exit.
         const report = (): void => {
             clearTimeout(timer);
             child.stdout.off('end', report);
-            setImmediate(() => options.onExit(exitCode));
+            setImmediate(() => options.onExit(exitCode, signal));
         };
         const timer = setTimeout(report, STDOUT_DRAIN_MS);
         if (child.stdout.readableEnded) {
@@ -259,7 +259,7 @@ export async function readStreamLines(
 export interface ChatChildOptions extends Omit<ChatSpawnOptions, 'onExit'> {
     spawn?: SpawnChatProcess;
     // `stderr` is the tail of what the CLI wrote there, for an exit with an error code only.
-    onExit(exitCode: number | null, stderr: string | null): void;
+    onExit(exitCode: number | null, stderr: string | null, signal?: string | null): void;
     onError?(error: Error): void;
 }
 
@@ -287,11 +287,11 @@ export class ChatChild {
             cwd: options.cwd,
             env: options.env,
             ...(options.onError ? { onError: options.onError } : {}),
-            onExit: (exitCode) => {
+            onExit: (exitCode, signal = null) => {
                 this.hasExited = true;
                 this.clearGrace();
                 settle();
-                void this.report(exitCode, options.onExit);
+                void this.report(exitCode, signal, options.onExit);
             }
         });
         this.tail = this.process.stderr ? new StreamTail(this.process.stderr) : null;
@@ -341,9 +341,9 @@ export class ChatChild {
         }
     }
 
-    private async report(exitCode: number | null, onExit: ChatChildOptions['onExit']): Promise<void> {
+    private async report(exitCode: number | null, signal: string | null, onExit: ChatChildOptions['onExit']): Promise<void> {
         if (exitCode === null || exitCode === 0 || this.tail === null) {
-            onExit(exitCode, null);
+            onExit(exitCode, null, signal);
             return;
         }
         let release: () => void = () => undefined;
@@ -355,6 +355,6 @@ export class ChatChild {
             })
         ]);
         clearTimeout(timer);
-        onExit(exitCode, this.tail.lines());
+        onExit(exitCode, this.tail.lines(), signal);
     }
 }

package/src/record-directory.ts

@@ -103,7 +103,8 @@ export class RecordDirectory<T> {
     /*
      * Holds the record in memory at once, so the next question in the same tick already sees it, and
      * answers whether it is still the one this directory holds by the time the file is written. A
-     * record removed while its file was being written does not come back: the rename is undone.
+     * record removed while its file was being written does not come back: the rename is undone, unless
+     * a newer record took the id meanwhile, whose own write is queued behind this one and replaces the file.
      */
     write(record: T): Promise<boolean> {
         const id = this.idOf(record);
@@ -114,11 +115,13 @@ export class RecordDirectory<T> {
             }
             await mkdir(this.dir, { recursive: true, mode: 0o700 });
             await this.writeFile(this.pathOf(id), JSON.stringify(record));
+            if (this.records.get(id) === record) {
+                return true;
+            }
             if (!this.records.has(id)) {
                 await this.removeFile(this.pathOf(id));
-                return false;
             }
-            return true;
+            return false;
         });
     }