@myagentroam/agent 0.12.1 → 0.12.3

package/dist/extensions/compaction.d.ts

@@ -32,5 +32,4 @@ export interface DomainCompactOperations {
         output: ExtensionCompactResult | undefined;
     };
 }
-export interface DomainCompactFacts {
-}
+export type DomainCompactFacts = Record<never, never>;

package/dist/extensions/files.d.ts

@@ -60,8 +60,7 @@ export type ExtensionFileCompletion = {
     };
 }[keyof ExtensionFilesApi];
 export declare function bindCoreFiles(backend: ExtensionFilesBackend, run: <Value>(operation: (call: ExtensionOperationContext) => Promise<Value>) => Promise<Value>, hooks: Pick<ExtensionHooks<DomainFileOperations, DomainFileFacts>, 'invoke' | 'emit'>): ExtensionFilesApi;
-export interface DomainFileOperations extends ExtensionFileOperations {
-}
+export type DomainFileOperations = ExtensionFileOperations;
 export interface DomainFileFacts {
     'files.completed': ExtensionFileCompletion;
 }

package/dist/extensions/model-request.d.ts

@@ -61,5 +61,4 @@ export interface DomainModelRequestOperations {
         output: ExtensionModelRequest;
     };
 }
-export interface DomainModelRequestFacts {
-}
+export type DomainModelRequestFacts = Record<never, never>;

package/dist/extensions/prompt-composition.d.ts

@@ -30,8 +30,7 @@ export interface DomainPromptCompositionOperations {
         output: string | null;
     };
 }
-export interface DomainPromptCompositionFacts {
-}
+export type DomainPromptCompositionFacts = Record<never, never>;
 export interface ExtensionPromptSection {
     readonly id: string;
     readonly text: string;

package/dist/extensions/prompt-input.d.ts

@@ -26,5 +26,4 @@ export interface DomainPromptInputOperations {
         output: ExtensionPromptResult;
     };
 }
-export interface DomainPromptInputFacts {
-}
+export type DomainPromptInputFacts = Record<never, never>;

package/dist/extensions/resources.d.ts

@@ -3,11 +3,8 @@ export interface ExtensionResourceRegistration {
     readonly ready: Promise<void>;
     dispose(): Promise<void>;
 }
-export interface ExtensionResourceScope {
-    effect(acquire: () => ExtensionDisposer | Promise<ExtensionDisposer>): ExtensionResourceRegistration;
-}
 /** An instance's managed resources, not an OS sandbox or a process-wide resource owner. */
-export declare class ExtensionResourceScope implements ExtensionResourceScope {
+export declare class ExtensionResourceScope {
     #private;
     readonly instanceId: `${string}-${string}-${string}-${string}-${string}`;
     static get initializing(): boolean;
@@ -15,6 +12,7 @@ export declare class ExtensionResourceScope implements ExtensionResourceScope {
     get state(): "STAGED" | "ACTIVE" | "RETIRING" | "DISPOSED";
     get signal(): AbortSignal;
     activate(): void;
+    effect(acquire: () => ExtensionDisposer | Promise<ExtensionDisposer>): ExtensionResourceRegistration;
     run<T>(operation: (signal: AbortSignal) => Promise<T>, parent?: AbortSignal): Promise<T>;
     after(durationMs: number, callback: (signal: AbortSignal) => void | Promise<void>, onError?: (error: unknown) => void): ExtensionResourceRegistration;
     every(durationMs: number, callback: (signal: AbortSignal) => void | Promise<void>, onError?: (error: unknown) => void): ExtensionResourceRegistration;

package/dist/host/local-exec-tool.d.ts

@@ -1,5 +1,5 @@
 import { type LocalProcessExecutor } from './local-process-executor.js';
-import type { ExecOutputSnapshot, ExecResult } from '../tools/contracts.js';
+import type { ExecOutputSnapshot, ExecResult, ExecutionSource } from '../tools/contracts.js';
 import { type ToolWaitInterrupt } from '../tools/wait.js';
 import { WorkspacePathResolver } from '../tools/shared/path-resolver.js';
 import { type ExtensionProcessInput, type ExtensionProcessHandle } from '../extensions/processes.js';
@@ -13,7 +13,7 @@ export declare class ExecTool {
     private readonly shell?;
     private readonly processExecutor;
     constructor(paths: WorkspacePathResolver, maxOutputBytes?: number, maxProcesses?: number, environment?: NodeJS.ProcessEnv, shell?: string | undefined, processExecutor?: LocalProcessExecutor);
-    execute(arguments_: unknown, signal?: AbortSignal, owner?: NonNullable<import('./local-process-executor.js').LocalProcessSpawnRequest['owner']>, waitInterrupt?: ToolWaitInterrupt, onOutput?: (output: ExecOutputSnapshot) => Promise<void>): Promise<ExecResult>;
+    execute(arguments_: unknown, signal?: AbortSignal, owner?: NonNullable<import('./local-process-executor.js').LocalProcessSpawnRequest['owner']>, waitInterrupt?: ToolWaitInterrupt, onOutput?: (output: ExecOutputSnapshot) => Promise<void>, onExecutionSource?: (source: ExecutionSource) => Promise<void> | void): Promise<ExecResult>;
     spawn(input: ExtensionProcessInput, call: ExtensionOperationContext): Promise<ExtensionProcessHandle>;
     listSessionResources(sessionId: string): {
         processId: string;

package/dist/host/local-exec-tool.js

@@ -6,6 +6,14 @@ import { TOOL_EXECUTION_LIMITS } from '../tools/execution-limits.js';
 import { waitForToolCondition, waitMetadata } from '../tools/wait.js';
 import { validateCoreProcessInput } from '../extensions/processes.js';
 import { parseExecInput } from '../tools/local/exec.js';
+function processSource(process) {
+    return {
+        kind: 'SHELL',
+        text: process.command.slice(0, 10_000),
+        cwd: process.cwd.slice(0, 10_000),
+        truncated: process.command.length > 10_000 || process.cwd.length > 10_000
+    };
+}
 export class ExecTool {
     paths;
     maxOutputBytes;
@@ -25,7 +33,7 @@ export class ExecTool {
         this.shell = shell;
         this.processExecutor = processExecutor;
     }
-    async execute(arguments_, signal, owner, waitInterrupt, onOutput) {
+    async execute(arguments_, signal, owner, waitInterrupt, onOutput, onExecutionSource) {
         if (signal?.aborted)
             throw executionAbortReason(signal);
         const input = parseExecInput(arguments_);
@@ -36,6 +44,10 @@ export class ExecTool {
             if (completed) {
                 if (owner !== undefined && completed.ownerSessionId !== owner.sessionId)
                     throw new AgentError('MAR_AGENT_PROCESS_NOT_FOUND', 'Managed process was not found.');
+                if (onExecutionSource)
+                    await onExecutionSource(processSource(completed));
+                if (signal?.aborted)
+                    throw executionAbortReason(signal);
                 if (input.action === 'write')
                     throw new AgentError('MAR_AGENT_TOOL_INPUT_INVALID', 'Completed process cannot accept input.');
                 if (input.action === 'cancel')
@@ -52,6 +64,10 @@ export class ExecTool {
                 process.native ||
                 (owner !== undefined && process.ownerSessionId !== owner.sessionId))
                 throw new AgentError('MAR_AGENT_PROCESS_NOT_FOUND', 'Managed process was not found.');
+            if (onExecutionSource)
+                await onExecutionSource(processSource(process));
+            if (signal?.aborted)
+                throw executionAbortReason(signal);
             let outcome;
             if (input.action === 'poll' && input.yieldMs)
                 outcome = await waitForProcessCompletion(input.processId, process, Math.min(Math.max(input.yieldMs, 0), TOOL_EXECUTION_LIMITS.execMaxPollYieldMs), signal, waitInterrupt, onOutput);
@@ -427,12 +443,7 @@ export class ExecTool {
                 stdoutDroppedBytes: stdout.droppedBytes,
                 stderrDroppedBytes: stderr.droppedBytes,
                 displayOutput: displayOutput(process),
-                executionSource: {
-                    kind: 'SHELL',
-                    text: process.command.slice(0, 10_000),
-                    cwd: process.cwd.slice(0, 10_000),
-                    truncated: process.command.length > 10_000 || process.cwd.length > 10_000
-                }
+                executionSource: processSource(process)
             }
         });
         while (this.#completed.size > TOOL_EXECUTION_LIMITS.execMaxCompletedResults)
@@ -496,12 +507,7 @@ export class ExecTool {
             stdoutDroppedBytes: stdout.droppedBytes,
             stderrDroppedBytes: stderr.droppedBytes,
             displayOutput: displayOutput(process),
-            executionSource: {
-                kind: 'SHELL',
-                text: process.command.slice(0, 10_000),
-                cwd: process.cwd.slice(0, 10_000),
-                truncated: process.command.length > 10_000 || process.cwd.length > 10_000
-            }
+            executionSource: processSource(process)
         };
     }
     async dispose() {

package/dist/host/local-tool-registry.js

@@ -98,7 +98,7 @@ export class LocalToolRegistry {
                                     ? {}
                                     : { ownerInstanceId: context.operation.ownerInstanceId })
                             })
-                    }, context.waitInterrupt, context.onOutput);
+                    }, context.waitInterrupt, context.onOutput, context.onExecutionSource);
                     return formatExecResult(result);
                 }
             ])

package/dist/runtime/context/gc-runtime.js

@@ -166,6 +166,3 @@ async function matchingCheckpointWasPersisted(store, sessionId, draft, records,
     });
     return matches ? { records: updatedRecords, projection } : false;
 }
-function isRecord(value) {
-    return value !== null && typeof value === 'object' && !Array.isArray(value);
-}

package/dist/runtime/model-progress.js

@@ -53,12 +53,16 @@ export async function* observeModelProgress(start, parent) {
     }
     finally {
         controller.abort();
-        try {
-            await iterator.return?.();
+        if (failed) {
+            try {
+                await iterator.return?.();
+            }
+            catch {
+                // Keep the generation failure rather than replacing it with a cleanup failure.
+            }
         }
-        catch (error) {
-            if (!failed)
-                throw error;
+        else {
+            await iterator.return?.();
         }
     }
 }

package/dist/sandbox/linux/bubblewrap-launch.js

@@ -107,7 +107,7 @@ export async function launchBubblewrap(spec) {
         if (childExited || childError || child.exitCode !== null || child.signalCode !== null)
             throw unavailable();
     }
-    catch (error) {
+    catch {
         failed = true;
         failure = spec.signal?.aborted ? spec.signal.reason : unavailable();
     }

package/dist/sdk/session-execution.js

@@ -80,7 +80,7 @@ export class SessionExecution {
         this.snapshot = snapshot;
         this.compactRequests = new CompactRequests(executionId, () => this.compact.assertNotReentrant());
         this.#bindings = { compactRequests: this.compactRequests };
-        const owner = this;
+        const bindings = () => this.#bindings;
         this.context = Object.freeze({
             executionId,
             modelId: snapshot.modelId,
@@ -88,19 +88,19 @@ export class SessionExecution {
             models: snapshot.models,
             instruction: snapshot.instruction,
             get compactRequests() {
-                return owner.#bindings.compactRequests;
+                return bindings().compactRequests;
             },
             get steer() {
-                return owner.#bindings.steer;
+                return bindings().steer;
             },
             get requestMessages() {
-                return owner.#bindings.requestMessages;
+                return bindings().requestMessages;
             },
             get appendedMessages() {
-                return owner.#bindings.appendedMessages;
+                return bindings().appendedMessages;
             },
             get toolSources() {
-                return owner.#bindings.toolSources;
+                return bindings().toolSources;
             }
         });
         this.modelResources = createExecutionModelResources(activeAdapters);
@@ -134,13 +134,16 @@ export class SessionExecution {
             throw error;
         }
         finally {
-            try {
-                await turn.close();
+            if (failed) {
+                try {
+                    await turn.close();
+                }
+                catch {
+                    // A cleanup failure cannot replace the generation failure.
+                }
             }
-            catch (error) {
-                // A cleanup failure cannot replace the generation failure.
-                if (!failed)
-                    throw error;
+            else {
+                await turn.close();
             }
         }
     }
@@ -403,10 +406,10 @@ export class SessionExecution {
         return this.resultPromise;
     }
     handle() {
-        const owner = this;
+        const persistedEvents = () => this.persistedTerminal;
         return {
             get persistedEvents() {
-                return owner.persistedTerminal;
+                return persistedEvents();
             },
             executionId: this.executionId,
             turnId: this.turnId,
@@ -415,7 +418,7 @@ export class SessionExecution {
                 this.events.discardIfUnobserved();
                 return this.resultPromise;
             },
-            cancel: () => owner.cancel(new AgentError('MAR_AGENT_CANCELLED', 'Execution was cancelled.'))
+            cancel: () => this.cancel(new AgentError('MAR_AGENT_CANCELLED', 'Execution was cancelled.'))
         };
     }
     cancel(reason) {

package/dist/sdk/session-maintenance.js

@@ -72,12 +72,16 @@ export class SessionMaintenanceCoordinator {
                 throw error;
             }
             finally {
-                try {
-                    await maintenance.release();
+                if (failed) {
+                    try {
+                        await maintenance.release();
+                    }
+                    catch {
+                        // Preserve the failure from clearing the Session.
+                    }
                 }
-                catch (error) {
-                    if (!failed)
-                        throw error;
+                else {
+                    await maintenance.release();
                 }
             }
         });
@@ -150,11 +154,17 @@ export class SessionMaintenanceCoordinator {
             }
             finally {
                 try {
-                    await maintenance.release();
-                }
-                catch (error) {
-                    if (!failed)
-                        throw error;
+                    if (failed) {
+                        try {
+                            await maintenance.release();
+                        }
+                        catch {
+                            // Preserve the failure from deleting the Sessions.
+                        }
+                    }
+                    else {
+                        await maintenance.release();
+                    }
                 }
                 finally {
                     for (const sessionId of ids) {

package/dist/sdk/session.d.ts

@@ -1,4 +1,3 @@
 import type { SessionServices } from './session-composition.js';
-import type { AgentSession } from './types.js';
 /** SDK Session orchestration owns warm resources; Runtime owns only the model/tool loop. */
-export declare function createSessionOpener(services: SessionServices): (id: string, workspace?: string) => Promise<AgentSession>;
+export declare function createSessionOpener(services: SessionServices): (id: string, workspace?: string) => Promise<import("./types.js").AgentSession>;

package/dist/session/catalog.js

@@ -225,12 +225,16 @@ export function createSessionCatalog(store, borrowed) {
                 throw error;
             }
             finally {
-                try {
-                    await maintenance.release();
+                if (failed) {
+                    try {
+                        await maintenance.release();
+                    }
+                    catch {
+                        // Preserve the failure from deleting the Sessions.
+                    }
                 }
-                catch (error) {
-                    if (!failed)
-                        throw error;
+                else {
+                    await maintenance.release();
                 }
             }
         },

package/dist/session/context-gc-checkpoint.js

@@ -1165,6 +1165,3 @@ function isNonnegativeInteger(value) {
 function isNonnegativeFinite(value) {
     return typeof value === 'number' && Number.isFinite(value) && value >= 0;
 }
-function isValidBoundary(value) {
-    return Number.isSafeInteger(value) && value > 0;
-}

package/dist/session/context-gc-protection.js

@@ -42,7 +42,7 @@ export function buildContextGcProtection(records, modelId, contextWindowTokens)
                 if (attempt?.matching) {
                     if (payload.state === 'SUCCEEDED')
                         successful.push(attempt.start);
-                    for (const _identity of attempt.native)
+                    for (let i = 0; i < attempt.native.size; i++)
                         thinking.push(attempt.start);
                 }
                 attempts.delete(key);
@@ -60,7 +60,7 @@ export function buildContextGcProtection(records, modelId, contextWindowTokens)
     }
     for (const attempt of attempts.values())
         if (attempt.matching)
-            for (const _identity of attempt.native)
+            for (let i = 0; i < attempt.native.size; i++)
                 thinking.push(attempt.start);
     successful.sort((a, b) => a - b);
     thinking.sort((a, b) => a - b);

package/dist/session/context-gc-replacements.js

@@ -353,9 +353,6 @@ function stableJson(value) {
 function isRecord(value) {
     return value !== null && typeof value === 'object' && !Array.isArray(value);
 }
-function isNonemptyString(value) {
-    return typeof value === 'string' && value.length > 0;
-}
 function isNonnegativeInteger(value) {
     return Number.isSafeInteger(value) && Number(value) >= 0;
 }

package/dist/session/store/jsonl/images.js

@@ -19,7 +19,7 @@ export function generatedImageFileNames(records) {
                 artifact.kind !== 'image' ||
                 typeof artifact.fileName !== 'string' ||
                 basename(artifact.fileName) !== artifact.fileName ||
-                !/^[a-zA-Z0-9][a-zA-Z0-9._-]{0,127}\.png$/u.test(artifact.fileName))
+                !/^[a-zA-Z0-9][a-zA-Z0-9._-]{0,127}\.(?:png|gif)$/u.test(artifact.fileName))
                 continue;
             names.add(artifact.fileName);
         }

package/dist/session/store/jsonl/jsonl-store.js

@@ -2146,6 +2146,7 @@ export class JsonlSessionStore {
             return;
         const releases = [];
         let failure;
+        let failed = false;
         try {
             for (const member of [...transaction.members].sort((a, b) => a.id.localeCompare(b.id))) {
                 if (await sessionDirectoryExists(join(root, member.id)))
@@ -2171,22 +2172,22 @@ export class JsonlSessionStore {
             await this.#refreshIndex();
         }
         catch (error) {
+            failed = true;
             failure = error;
-            throw error;
         }
-        finally {
-            let releaseFailure;
-            for (const release of releases.reverse()) {
-                try {
-                    await release();
-                }
-                catch (error) {
-                    releaseFailure ??= error;
-                }
+        let releaseFailure;
+        for (const release of releases.reverse()) {
+            try {
+                await release();
+            }
+            catch (error) {
+                releaseFailure ??= error;
             }
-            if (failure === undefined && releaseFailure !== undefined)
-                throw releaseFailure;
         }
+        if (failure === undefined && releaseFailure !== undefined)
+            throw releaseFailure;
+        if (failed)
+            throw failure;
     }
     async delete(sessionId, maintenance) {
         this.#assertOpen();

package/dist/session/store/jsonl/records.js

@@ -48,7 +48,6 @@ export async function* readRecordsFromHead(path, encoding = 1) {
                 pending.push(data.subarray(start));
         }
         if (pending.length > 0) {
-            let record;
             const tail = Buffer.concat(pending).toString('utf8');
             let value;
             try {
@@ -57,7 +56,7 @@ export async function* readRecordsFromHead(path, encoding = 1) {
             catch {
                 return; // Readers ignore only an interrupted final line.
             }
-            record = decodeJsonlValue(value, encoding);
+            const record = decodeJsonlValue(value, encoding);
             yield record;
         }
     }

package/dist/session/store/types.d.ts

@@ -22,6 +22,7 @@ export interface SessionInputImageMetadata {
     readonly urlExpiresAt?: number;
 }
 export interface RestoredSessionInputImage extends ModelImage {
+    readonly mimeType: ModelImage['mimeType'];
 }
 export interface SessionInputImageData extends SessionInputImageMetadata {
     readonly dataBase64: string;

package/dist/tools/code-mode/cells.js

@@ -298,6 +298,9 @@ export class CodeCells {
         const cell = this.#cells.get(input.cellId);
         if (!cell || cell.sessionId !== context.sessionId)
             throw new AgentError('MAR_AGENT_CODE_CELL_NOT_FOUND', 'Code Exec cell is unavailable in this Session.');
+        if (context.onExecutionSource)
+            await context.onExecutionSource({ ...cell.source });
+        context.signal.throwIfAborted();
         if (input.action === 'cancel' && cell.status === 'running') {
             cell.controller.abort(new AgentError('MAR_AGENT_CODE_CANCELLED', 'Code Exec cell was terminated.'));
             await new Promise((resolve, reject) => {

package/dist/tools/contracts.d.ts

@@ -33,6 +33,8 @@ export interface ToolExecutionContext {
     waitInterrupt?: import('./wait.js').ToolWaitInterrupt | undefined;
     /** Live-only process output; does not consume the eventual tool result. */
     onOutput?: (output: ExecOutputSnapshot) => Promise<void>;
+    /** Publish owner-confirmed source metadata before waiting, without consuming output. */
+    onExecutionSource?: (source: ExecutionSource) => Promise<void> | void;
     sessionId: string;
     executionId: string;
     /** Current model identity for neutral Host-owned tool policies. */
@@ -88,7 +90,7 @@ export interface McpToolCallContext extends Pick<ToolExecutionContext, 'sessionI
 export interface AgentToolImageArtifact {
     kind: 'image';
     name: string;
-    mimeType: 'image/png';
+    mimeType: 'image/png' | 'image/gif';
     byteSize: number;
     sha256: string;
     fileName: string;

package/dist/tools/file-change-diff.js

@@ -119,8 +119,10 @@ function compare(a, b) {
                     : k - 1;
                 const previousX = previous.get(previousK) ?? 0;
                 const previousY = previousX - previousK;
-                while (x > previousX && y > previousY)
-                    (reversed.push({ kind: ' ', text: left[--x] }), y--);
+                while (x > previousX && y > previousY) {
+                    reversed.push({ kind: ' ', text: left[--x] });
+                    y--;
+                }
                 if (d === 0)
                     break;
                 if (x === previousX)

package/dist/tools/input-diagnostics/source-preview.js

@@ -56,7 +56,10 @@ function displayWidth(text) {
 function displayCharacter(character) {
     if (character === '\t')
         return '    ';
-    if (/[\u0000-\u001f\u007f-\u009f\u2028\u2029\p{Bidi_Control}]/u.test(character)) {
+    const code = character.codePointAt(0);
+    if (code <= 0x1f ||
+        (code >= 0x7f && code <= 0x9f) ||
+        /[\u2028\u2029\p{Bidi_Control}]/u.test(character)) {
         const escaped = JSON.stringify(character).slice(1, -1);
         return escaped === character ? `\\u{${character.codePointAt(0).toString(16)}}` : escaped;
     }

package/dist/tools/local/apply-patch-parser.js

@@ -171,7 +171,9 @@ function commonPrefixLength(actual, expected) {
 }
 function characterCount(value) {
     let count = 0;
-    for (const _character of value)
-        count++;
+    for (let index = 0; index < value.length; index++, count++) {
+        if (value.codePointAt(index) > 0xffff)
+            index++;
+    }
     return count;
 }

package/dist/tools/output-projection.js

@@ -439,11 +439,12 @@ export function structuredToolArtifacts(value) {
         value.length !== 1 ||
         value.some((artifact) => !artifact ||
             artifact.kind !== 'image' ||
-            artifact.mimeType !== 'image/png' ||
+            (artifact.mimeType !== 'image/png' && artifact.mimeType !== 'image/gif') ||
             typeof artifact.name !== 'string' ||
             artifact.name.length === 0 ||
             typeof artifact.fileName !== 'string' ||
-            !/^[a-zA-Z0-9][a-zA-Z0-9._-]{0,127}\.png$/u.test(artifact.fileName) ||
+            !/^[a-zA-Z0-9][a-zA-Z0-9._-]{0,127}\.(?:png|gif)$/u.test(artifact.fileName) ||
+            !artifact.fileName.endsWith(artifact.mimeType === 'image/gif' ? '.gif' : '.png') ||
             !isNonnegativeInteger(artifact.byteSize) ||
             typeof artifact.sha256 !== 'string' ||
             !/^[a-f0-9]{64}$/u.test(artifact.sha256)))

package/dist/tools/runtime.js

@@ -53,6 +53,13 @@ export async function executeCapturedTool(kind, call, context) {
         ...((name === 'exec' || name === 'exec_wait' || name === 'code_exec_wait') &&
             callId !== undefined
             ? {
+                onExecutionSource: (source) => context.emit({
+                    type: 'tool.updated',
+                    toolName: name,
+                    callId,
+                    ...(context.parentCallId === undefined ? {} : { parentCallId: context.parentCallId }),
+                    data: { executionSource: source }
+                }),
                 onOutput: (output) => context.emit({
                     type: 'tool.output',
                     toolName: name,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myagentroam/agent",
-  "version": "0.12.1",
+  "version": "0.12.3",
   "description": "Embeddable MAR coding agent SDK and CLI.",
   "license": "Apache-2.0",
   "type": "module",