@animalabs/agent-framework 0.32.0 → 0.34.0

package/src/mcpl/checkpoint-manager.ts

@@ -17,6 +17,7 @@ import { ensureStateRegistered } from '../module-registry.js';
 import { boundedErrorMessage as thrownText } from '../error-message.js';
 
 import type { StateCheckpoint, JsonPatchOperation } from './types.js';
+import type { TraceEventInput } from '../types/trace.js';
 
 // ============================================================================
 // Constants
@@ -198,12 +199,12 @@ export function applyJsonPatch(doc: unknown, ops: JsonPatchOperation[]): unknown
 
 export class CheckpointManager {
   private store: JsStore;
-  private emitTraceFn: (event: { type: string; [key: string]: unknown }) => void;
+  private emitTraceFn: (event: TraceEventInput) => void;
   private trees = new Map<string, FeatureSetState>();
 
   constructor(
     store: JsStore,
-    emitTraceFn: (event: { type: string; [key: string]: unknown }) => void,
+    emitTraceFn: (event: TraceEventInput) => void,
   ) {
     this.store = store;
     this.emitTraceFn = emitTraceFn;

package/src/mcpl/feature-set-manager.ts

@@ -19,6 +19,7 @@ import {
   UNKNOWN_FEATURE_SET,
 } from './errors.js';
 import { isKnownCapabilityPath } from './capability-grant.js';
+import type { TraceEventInput } from '../types/trace.js';
 import { logValue } from './log-value.js';
 
 // ============================================================================
@@ -136,7 +137,7 @@ export class FeatureSetManager {
    *   turned off. Omit for scratch derivations (nothing is installed).
    */
   constructor(
-    private readonly emitTrace?: (event: { type: string; [key: string]: unknown }) => void,
+    private readonly emitTrace?: (event: TraceEventInput) => void,
   ) {}
 
   /**

package/src/mcpl/inbound-source.ts

@@ -175,7 +175,8 @@ export function conversationKey(source: InboundSource): string | undefined {
  * It expects JSON-shaped values, as the MCPL lanes deliver them (parsed from
  * JSON). Any other object is written by its own enumerable entries, unlike
  * JSON.stringify: a Date becomes `{}`, as the store's own write keeps it, but
- * a deferred write recovered from its JSON file brings it back as a string.
+ * a write an earlier release's deferral queue left in its JSON file, which
+ * the importer stores at boot, brings it back as a string.
  *
  * sourceBodyDigest's serializer, kept out of the package API: serialization
  * and framing both stay inside sourceBodyDigest, so consumers don't

package/src/mcpl/inference-router.ts

@@ -29,6 +29,7 @@ import { hostStoppedWhileHandling } from './errors.js';
 import type { HookOrchestrator } from './hook-orchestrator.js';
 import { logValue } from './log-value.js';
 import { streamBounds, wholeCallMs, withHostSetting, type InferenceBounds } from '../inference-bounds.js';
+import type { TraceEventInput } from '../types/trace.js';
 
 // ============================================================================
 // Responder interface
@@ -38,8 +39,10 @@ import { streamBounds, wholeCallMs, withHostSetting, type InferenceBounds } from
 interface Responder {
   respond(result: McplInferenceRequestResult): void;
   respondError(code: number, message: string, data?: unknown): void;
-  /** The JSON-RPC request ID, used for streaming chunk correlation. */
-  requestId?: string | number;
+  /** The JSON-RPC request ID, used for streaming chunk correlation: a
+   *  request sent with id null is answered whole, since its chunks couldn't
+   *  name it. */
+  requestId?: string | number | null;
   /** Sends a streamed chunk on the transport the request came on, as its
    *  answer is sent. */
   sendChunk?(params: InferenceChunkParams): void;
@@ -184,7 +187,7 @@ export class InferenceRouter {
   private hookOrchestrator: HookOrchestrator;
   private featureSetManager: FeatureSetManager;
   private routingPolicy: InferenceRoutingPolicy | null;
-  private emitTraceFn: (event: { type: string; [key: string]: unknown }) => void;
+  private emitTraceFn: (event: TraceEventInput) => void;
   /** The refusals for want of a model that have been logged: one line per
    *  server with no policy, and per server and feature set with one, so a
    *  server that keeps asking doesn't flood the operator's log. */
@@ -196,7 +199,7 @@ export class InferenceRouter {
     hookOrchestrator: HookOrchestrator,
     featureSetManager: FeatureSetManager,
     routingPolicy: InferenceRoutingPolicy | null,
-    emitTraceFn: (event: { type: string; [key: string]: unknown }) => void,
+    emitTraceFn: (event: TraceEventInput) => void,
     private readonly inferenceBounds: () => InferenceBounds | null = () => null,
   ) {
     this.membrane = membrane;