@adecore/agents 0.33.2 → 0.34.0

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.d.ts

@@ -5,6 +5,7 @@ export interface RecordDirectoryOptions<T> {
     schema: z.ZodType<T>;
     idOf(record: T): string;
     write?: (path: string, text: string) => Promise<void>;
+    remove?: (path: string) => Promise<void>;
     keep?(record: T): T | null;
 }
 export declare class RecordDirectory<T> {
@@ -12,9 +13,10 @@ export declare class RecordDirectory<T> {
     private readonly schema;
     private readonly idOf;
     private readonly writeFile;
+    private readonly removeFile;
     private readonly keep;
     private readonly records;
-    private readonly writes;
+    private readonly files;
     constructor(options: RecordDirectoryOptions<T>);
     load(): Promise<void>;
     get(id: string): T | undefined;

package/dist/record-directory.js

@@ -16,15 +16,20 @@ export class RecordDirectory {
     schema;
     idOf;
     writeFile;
+    removeFile;
     keep;
     records = new Map();
-    // One write at a time per record, so an older one never lands after a newer one.
-    writes = new KeyedSerializer();
+    /*
+     * One write or remove at a time per record, so an older write never lands after a newer one and
+     * no delete starts while another of the same file is pending, which Windows can fail with EFAULT.
+     */
+    files = new KeyedSerializer();
     constructor(options) {
         this.dir = options.dir;
         this.schema = options.schema;
         this.idOf = options.idOf;
         this.writeFile = options.write ?? writeAtomic;
+        this.removeFile = options.remove ?? removeFile;
         this.keep = options.keep ?? null;
     }
     /* Reads what an earlier run of the host wrote down. Call before anything can ask. */
@@ -59,7 +64,7 @@ export class RecordDirectory {
             }
             const record = this.keep === null ? parsed.data : this.keep(parsed.data);
             if (record === null) {
-                await removeFile(join(this.dir, name));
+                await this.removeFile(join(this.dir, name));
                 continue;
             }
             this.records.set(this.idOf(record), record);
@@ -77,28 +82,31 @@ 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);
         this.records.set(id, record);
-        return this.writes.run(id, async () => {
+        return this.files.run(id, async () => {
             if (this.records.get(id) !== record) {
                 return false;
             }
             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 removeFile(this.pathOf(id));
-                return false;
+                await this.removeFile(this.pathOf(id));
             }
-            return true;
+            return false;
         });
     }
-    /* Forgets a record here and now; its file follows. */
+    /* Forgets a record here and now; its file follows, after what is queued for it. */
     remove(id) {
         this.records.delete(id);
-        return removeFile(this.pathOf(id));
+        return this.files.run(id, () => this.removeFile(this.pathOf(id)));
     }
     async prune(drop) {
         for (const listed of this.all()) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adecore/agents",
-  "version": "0.33.2",
+  "version": "0.34.0",
   "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.2",
-    "@adecore/intelligent-ui": "0.33.2",
+    "@adecore/agent-contracts": "0.34.0",
+    "@adecore/intelligent-ui": "0.34.0",
     "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

@@ -14,6 +14,8 @@ export interface RecordDirectoryOptions<T> {
     schema: z.ZodType<T>;
     idOf(record: T): string;
     write?: (path: string, text: string) => Promise<void>;
+    // Stands in for `removeFile` whole, so it must take a missing file as removed: a write and a remove delete twice.
+    remove?: (path: string) => Promise<void>;
     /*
      * What a record read from disk is worth keeping as, for a store that drops what went stale while
      * the host was down. Null takes the file with it.
@@ -31,16 +33,21 @@ export class RecordDirectory<T> {
     private readonly schema: z.ZodType<T>;
     private readonly idOf: (record: T) => string;
     private readonly writeFile: (path: string, text: string) => Promise<void>;
+    private readonly removeFile: (path: string) => Promise<void>;
     private readonly keep: ((record: T) => T | null) | null;
     private readonly records = new Map<string, T>();
-    // One write at a time per record, so an older one never lands after a newer one.
-    private readonly writes = new KeyedSerializer();
+    /*
+     * One write or remove at a time per record, so an older write never lands after a newer one and
+     * no delete starts while another of the same file is pending, which Windows can fail with EFAULT.
+     */
+    private readonly files = new KeyedSerializer();
 
     constructor(options: RecordDirectoryOptions<T>) {
         this.dir = options.dir;
         this.schema = options.schema;
         this.idOf = options.idOf;
         this.writeFile = options.write ?? writeAtomic;
+        this.removeFile = options.remove ?? removeFile;
         this.keep = options.keep ?? null;
     }
 
@@ -74,7 +81,7 @@ export class RecordDirectory<T> {
             }
             const record = this.keep === null ? parsed.data : this.keep(parsed.data);
             if (record === null) {
-                await removeFile(join(this.dir, name));
+                await this.removeFile(join(this.dir, name));
                 continue;
             }
             this.records.set(this.idOf(record), record);
@@ -96,29 +103,32 @@ 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);
         this.records.set(id, record);
-        return this.writes.run(id, async () => {
+        return this.files.run(id, async () => {
             if (this.records.get(id) !== record) {
                 return false;
             }
             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 removeFile(this.pathOf(id));
-                return false;
+                await this.removeFile(this.pathOf(id));
             }
-            return true;
+            return false;
         });
     }
 
-    /* Forgets a record here and now; its file follows. */
+    /* Forgets a record here and now; its file follows, after what is queued for it. */
     remove(id: string): Promise<void> {
         this.records.delete(id);
-        return removeFile(this.pathOf(id));
+        return this.files.run(id, () => this.removeFile(this.pathOf(id)));
     }
 
     async prune(drop: (record: T) => boolean): Promise<void> {