@myagentroam/agent 0.9.84 → 0.9.85

package/dist/model/configuration.js

@@ -80,6 +80,11 @@ const rawModelSchema = z
         });
     if (value.protocol !== 'OPENAI_RESPONSES' && value.responsesPreviousResponseId)
         context.addIssue({ code: 'custom', message: 'Responses continuation requires Responses.' });
+    if (value.responsesPreviousResponseId && value.responsesTransport?.transport !== 'WEBSOCKET')
+        context.addIssue({
+            code: 'custom',
+            message: 'Responses continuation requires WebSocket transport.'
+        });
     if (value.protocol !== 'OPENAI_RESPONSES' && value.responsesTransport !== undefined)
         context.addIssue({ code: 'custom', message: 'Responses transport requires Responses.' });
     if (value.protocol !== 'OPENAI_RESPONSES' && value.responsesEncoding !== undefined)

package/dist/model/contracts.d.ts

@@ -1,10 +1,9 @@
 import type { MarAgentModelConfiguration, MarAgentReasoningEffort, ModelCredentialAcquireReason } from './configuration.js';
 export interface ModelContinuationCheckpoint {
-    readonly version: 1;
+    readonly version: 2;
     readonly protocol: 'OPENAI_RESPONSES';
     readonly configurationHash: string;
     readonly responseId: string;
-    readonly requestPropertiesHash: string;
     readonly inputPrefixLength: number;
     readonly inputPrefixHash: string;
 }

package/dist/model/openai-responses.d.ts

@@ -45,8 +45,7 @@ declare class OpenAiResponsesTurnSession implements ModelTurnSession {
     private readonly adapter;
     private readonly state;
     private readonly cacheLease;
-    private readonly allowExternalContinuation;
-    constructor(adapter: OpenAiResponsesAdapter, state: ResponsesSessionState, cacheLease: boolean, allowExternalContinuation: boolean);
+    constructor(adapter: OpenAiResponsesAdapter, state: ResponsesSessionState, cacheLease: boolean);
     start(request: ModelRequest, signal: AbortSignal): AsyncIterable<ModelEvent>;
     close(): Promise<void>;
     resetContinuation(): void;

package/dist/model/openai-responses.js

@@ -35,14 +35,16 @@ export class OpenAiResponsesAdapter {
         if (this.#cacheLeased ||
             this.#cachedState.lastRequest !== undefined ||
             this.#cachedState.lastResponse !== undefined ||
+            !responsesContinuationEnabled(this.#configuration) ||
+            checkpoint.version !== 2 ||
             checkpoint.protocol !== 'OPENAI_RESPONSES' ||
-            checkpoint.configurationHash !== responsesConfigurationHash(this.#configuration))
+            checkpoint.configurationHash !== responsesContinuationIdentityHash(this.#configuration))
             return false;
         this.#cachedState.restoredContinuation = checkpoint;
         return true;
     }
     async *start(request, signal) {
-        const turn = new OpenAiResponsesTurnSession(this, {}, false, true);
+        const turn = new OpenAiResponsesTurnSession(this, {}, false);
         try {
             yield* turn.start(request, signal);
         }
@@ -59,7 +61,7 @@ export class OpenAiResponsesAdapter {
             this.#cachedState = {};
             this.#cacheLeased = true;
         }
-        return new OpenAiResponsesTurnSession(this, state, cacheLease, false);
+        return new OpenAiResponsesTurnSession(this, state, cacheLease);
     }
     async close() {
         if (this.#closed)
@@ -546,17 +548,15 @@ class OpenAiResponsesTurnSession {
     adapter;
     state;
     cacheLease;
-    allowExternalContinuation;
     #active = false;
     #closed = false;
     #turnState;
     #fullBody;
     #connectionDiagnostic;
-    constructor(adapter, state, cacheLease, allowExternalContinuation) {
+    constructor(adapter, state, cacheLease) {
         this.adapter = adapter;
         this.state = state;
         this.cacheLease = cacheLease;
-        this.allowExternalContinuation = allowExternalContinuation;
     }
     async *start(request, signal) {
         if (this.#closed)
@@ -573,7 +573,7 @@ class OpenAiResponsesTurnSession {
             this.#fullBody = prepared.fullBody;
             if (this.adapter.configuration.responsesEncoding === 'LITE' &&
                 this.adapter.configuration.responsesTransport?.transport === 'WEBSOCKET' &&
-                this.adapter.configuration.responsesPreviousResponseId &&
+                responsesContinuationEnabled(this.adapter.configuration) &&
                 this.state.lastRequest === undefined &&
                 request.allowTools !== false &&
                 request.toolChoice !== 'none' &&
@@ -626,7 +626,7 @@ class OpenAiResponsesTurnSession {
                     completed = event.reason === 'completed' || event.reason === 'tool_use';
                 yield event;
             }
-            if (this.adapter.configuration.responsesPreviousResponseId &&
+            if (responsesContinuationEnabled(this.adapter.configuration) &&
                 completed &&
                 responseId !== undefined &&
                 (outputItems.length > 0 || !sawUnrepresentedOutput)) {
@@ -656,17 +656,18 @@ class OpenAiResponsesTurnSession {
         clearResponsesContinuation(this.state);
     }
     continuationCheckpoint() {
+        if (!responsesContinuationEnabled(this.adapter.configuration))
+            return undefined;
         const request = this.state.lastRequest;
         const response = this.state.lastResponse;
         if (request === undefined || response === undefined)
             return this.state.restoredContinuation;
         const baseline = [...request.input, ...response.outputItems];
         return {
-            version: 1,
+            version: 2,
             protocol: 'OPENAI_RESPONSES',
-            configurationHash: responsesConfigurationHash(this.adapter.configuration),
+            configurationHash: responsesContinuationIdentityHash(this.adapter.configuration),
             responseId: response.responseId,
-            requestPropertiesHash: responsesRequestPropertiesHash(request),
             inputPrefixLength: baseline.length,
             inputPrefixHash: hashResponsesValue(baseline)
         };
@@ -677,7 +678,7 @@ class OpenAiResponsesTurnSession {
             this.state.socket.readyState !== WebSocket.OPEN)
             invalidateResponsesWebSocket(this.state, false);
         const fullBody = responsesRequestBody(this.adapter.configuration, request, request.messages, this.adapter.prefixIdentity);
-        if (this.adapter.configuration.responsesPreviousResponseId) {
+        if (responsesContinuationEnabled(this.adapter.configuration)) {
             const incremental = incrementalResponsesInput(this.state, fullBody);
             if (incremental !== undefined) {
                 const responseId = this.state.lastResponse.responseId;
@@ -692,20 +693,6 @@ class OpenAiResponsesTurnSession {
                     continuation: true
                 };
             }
-            if (this.allowExternalContinuation &&
-                request.continuation !== undefined &&
-                this.adapter.configuration.responsesTransport?.transport !== 'WEBSOCKET') {
-                const deltaBody = responsesRequestBody(this.adapter.configuration, request, request.continuation.deltaMessages);
-                return {
-                    fullBody,
-                    body: {
-                        ...fullBody,
-                        input: deltaBody.input,
-                        previous_response_id: request.continuation.previousResponseId
-                    },
-                    continuation: true
-                };
-            }
         }
         return {
             fullBody,
@@ -899,7 +886,6 @@ function incrementalResponsesInput(state, current) {
     if (previous === undefined || completion === undefined) {
         const restored = state.restoredContinuation;
         if (restored === undefined ||
-            restored.requestPropertiesHash !== responsesRequestPropertiesHash(current) ||
             current.input.length < restored.inputPrefixLength ||
             restored.inputPrefixHash !==
                 hashResponsesValue(current.input.slice(0, restored.inputPrefixLength)))
@@ -907,8 +893,6 @@ function incrementalResponsesInput(state, current) {
         state.lastResponse = { responseId: restored.responseId, outputItems: [] };
         return current.input.slice(restored.inputPrefixLength);
     }
-    if (!responsesRequestPropertiesMatch(previous, current))
-        return undefined;
     const baseline = [...previous.input, ...completion.outputItems];
     if (current.input.length < baseline.length)
         return undefined;
@@ -917,37 +901,24 @@ function incrementalResponsesInput(state, current) {
             return undefined;
     return current.input.slice(baseline.length);
 }
-function responsesRequestPropertiesMatch(previous, current) {
-    return RESPONSES_REQUEST_PROPERTY_KEYS.every((key) => isDeepStrictEqual(previous[key], current[key]));
-}
 function clearResponsesContinuation(state) {
     delete state.lastRequest;
     delete state.lastResponse;
     delete state.restoredContinuation;
 }
-const RESPONSES_REQUEST_PROPERTY_KEYS = [
-    'model',
-    'instructions',
-    'tools',
-    'tool_choice',
-    'parallel_tool_calls',
-    'reasoning',
-    'store',
-    'stream',
-    'include',
-    'prompt_cache_key',
-    'text',
-    'client_metadata'
-];
-function responsesRequestPropertiesHash(request) {
-    return hashResponsesValue(Object.fromEntries(RESPONSES_REQUEST_PROPERTY_KEYS.map((key) => [key, request[key]])));
-}
 function hashResponsesValue(value) {
     return createHash('sha256').update(JSON.stringify(value)).digest('hex');
 }
-function responsesConfigurationHash(configuration) {
-    const { apiKey: _apiKey, credentialProvider: _credentialProvider, imageGeneration: _imageGeneration, ...identity } = configuration;
-    return hashResponsesValue(identity);
+function responsesContinuationIdentityHash(configuration) {
+    return hashResponsesValue({
+        endpoint: new URL(responsesEndpoint(configuration.baseUrl)).toString(),
+        modelId: configuration.modelId,
+        responsesEncoding: configuration.responsesEncoding ?? 'STANDARD'
+    });
+}
+function responsesContinuationEnabled(configuration) {
+    return (configuration.responsesPreviousResponseId === true &&
+        configuration.responsesTransport?.transport === 'WEBSOCKET');
 }
 function responsesCredentialFingerprint(credential) {
     return hashResponsesValue({

package/dist/sdk/agent.js

@@ -1600,11 +1600,10 @@ function isModelContinuationCheckpoint(value) {
     if (value === null || typeof value !== 'object' || Array.isArray(value))
         return false;
     const checkpoint = value;
-    return (checkpoint.version === 1 &&
+    return (checkpoint.version === 2 &&
         checkpoint.protocol === 'OPENAI_RESPONSES' &&
         isBoundedString(checkpoint.configurationHash, 64) &&
         isBoundedString(checkpoint.responseId, 1_024) &&
-        isBoundedString(checkpoint.requestPropertiesHash, 64) &&
         Number.isSafeInteger(checkpoint.inputPrefixLength) &&
         checkpoint.inputPrefixLength >= 0 &&
         isBoundedString(checkpoint.inputPrefixHash, 64));

package/dist/tools/agent-message.js

@@ -33,7 +33,10 @@ function parseAgentMessageInput(value) {
         return invalid();
     const input = value;
     if (Object.keys(input).some((key) => !['agentId', 'message', 'delivery'].includes(key)) ||
-        (input.delivery !== undefined && input.delivery !== 'append' && input.delivery !== 'replace') ||
+        (input.delivery !== undefined &&
+            input.delivery !== null &&
+            input.delivery !== 'append' &&
+            input.delivery !== 'replace') ||
         typeof input.agentId !== 'string' ||
         input.agentId.trim().length === 0 ||
         typeof input.message !== 'string' ||
@@ -43,7 +46,9 @@ function parseAgentMessageInput(value) {
     return {
         agentId: input.agentId.trim(),
         message: input.message,
-        ...(input.delivery === undefined ? {} : { delivery: input.delivery })
+        ...(input.delivery === 'append' || input.delivery === 'replace'
+            ? { delivery: input.delivery }
+            : {})
     };
 }
 function invalid() {

package/dist/tools/agent-start.js

@@ -62,6 +62,7 @@ function parseAgentStartInput(value) {
             input.modelId !== null &&
             (typeof input.modelId !== 'string' || input.modelId.trim().length === 0)) ||
         (input.reasoningEffort !== undefined &&
+            input.reasoningEffort !== null &&
             (typeof input.reasoningEffort !== 'string' ||
                 !marAgentReasoningEffortSchema.safeParse(input.reasoningEffort.trim()).success)) ||
         (input.background !== undefined &&

package/dist/tools/agent-wait.js

@@ -46,11 +46,14 @@ function parseAgentWaitInput(value) {
     const input = value;
     if (Object.keys(input).some((key) => !['agentId', 'waitFor', 'waitMs'].includes(key)) ||
         (input.agentId !== undefined &&
+            input.agentId !== null &&
             (typeof input.agentId !== 'string' || input.agentId.trim().length === 0)) ||
         (input.waitFor !== undefined &&
+            input.waitFor !== null &&
             input.waitFor !== 'completion' &&
             input.waitFor !== 'message') ||
         (input.waitMs !== undefined &&
+            input.waitMs !== null &&
             (!Number.isInteger(input.waitMs) ||
                 input.waitMs < TOOL_EXECUTION_LIMITS.agentWaitMinWaitMs ||
                 input.waitMs > TOOL_EXECUTION_LIMITS.agentWaitMaxWaitMs)))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myagentroam/agent",
-  "version": "0.9.84",
+  "version": "0.9.85",
   "description": "Embeddable MAR coding agent SDK and CLI.",
   "license": "Apache-2.0",
   "type": "module",
@@ -42,6 +42,7 @@
     "build": "node ../scripts/clean-build-output.mjs dist && tsc -p tsconfig.json",
     "typecheck": "tsc -p tsconfig.json --noEmit",
     "test:unit": "vitest run --config vitest.config.ts test/unit",
+    "analyze:session": "node test/tools/session-analyzer.mjs",
     "benchmark:compaction": "node test/e2e/compaction-benchmark.mjs",
     "test:performance": "vitest run --config vitest.performance.config.ts",
     "test:memory": "node --expose-gc ./node_modules/vitest/vitest.mjs run --config vitest.memory.config.ts",