@sema-agent/core 1.286.1 → 1.288.0

package/dist/core/runner/runtask.js

@@ -220,6 +220,37 @@ export class Runner {
         if (this.deps.tiers) {
             this.deps = { ...this.deps, models: expandTiers(this.deps.models, this.deps.tiers) };
         }
+        if (this.deps.onError) {
+            const rawOnError = this.deps.onError;
+            const reportSinkFailure = (sinkErr, context) => {
+                try {
+                    emitTrace(this.deps.tracer, () => ({
+                        kind: "task.onerror_sink_failed",
+                        version: 1,
+                        sessionId: context.sessionId,
+                        phase: context.phase,
+                        reason: (sinkErr instanceof Error ? sinkErr.message : String(sinkErr)).slice(0, 256),
+                        ts: Date.now(),
+                    }));
+                }
+                catch {
+                }
+            };
+            this.deps = {
+                ...this.deps,
+                onError: (err, context) => {
+                    try {
+                        const ret = rawOnError(err, context);
+                        if (ret !== null && ret !== undefined && typeof ret.then === "function") {
+                            Promise.resolve(ret).then(undefined, (sinkErr) => reportSinkFailure(sinkErr, context));
+                        }
+                    }
+                    catch (sinkErr) {
+                        reportSinkFailure(sinkErr, context);
+                    }
+                },
+            };
+        }
         if (!this.deps.toolResultStore) {
             this.deps = { ...this.deps, toolResultStore: new RunnerSharedToolResultStore({ maxTotalChars: 64_000_000 }) };
         }
@@ -566,7 +597,10 @@ export class Runner {
             }
         });
         const upstreamTaskNotification = internals?.onTaskNotification;
+        const deliveredAtTurnOpen = new Set();
         const injectTaskNotification = (notification, opts) => {
+            if (deliveredAtTurnOpen.has(taskNotificationDedupKey(notification)))
+                return;
             if (!notificationLaneLive) {
                 if (notificationSessionId !== undefined)
                     this.pendingSessionNotifications.pend(notificationSessionId, notification);
@@ -599,6 +633,7 @@ export class Runner {
             const pendingIdle = this.pendingSessionNotifications.drain(prepared.sessionId);
             if (pendingIdle !== undefined) {
                 for (const payload of discloseDroppedPending(pendingIdle)) {
+                    deliveredAtTurnOpen.add(taskNotificationDedupKey(payload));
                     queue.push({ type: "task_notification", notification: payload });
                     void prepared.harness.nextTurn(renderTaskNotificationXml(payload)).catch(() => {
                         this.pendingSessionNotifications.pend(prepared.sessionId, payload);
@@ -669,12 +704,18 @@ export class Runner {
                 pricing = this.deps.pricing?.[m.id] ?? modelCostToPricing(m.cost);
                 cacheFamily = cacheFamilyOf(m);
             }
-            else if (this.deps.pricing?.[info.to]) {
-                pricing = this.deps.pricing[info.to];
+            else {
+                if (this.deps.pricing?.[info.to])
+                    pricing = this.deps.pricing[info.to];
+                cacheFamily = "input-excludes-cached";
             }
             degraded = info;
             emitTrace(tracer, () => ({ kind: "task.degraded", version: 1, taskId, from: info.from, to: info.to, reason: info.reason, atTurn: info.atTurn, ts: Date.now() }));
-            this.deps.onError?.(new Error(`degraded to a cheaper model: ${info.from} → ${info.to} (${info.reason}) at turn ${info.atTurn}`), { phase: "degraded", sessionId: prepared.sessionId });
+            try {
+                this.deps.onError?.(new Error(`degraded to a cheaper model: ${info.from} → ${info.to} (${info.reason}) at turn ${info.atTurn}`), { phase: "degraded", sessionId: prepared.sessionId });
+            }
+            catch {
+            }
         };
         const sliceWindowMicroUsd = spec.maxCostUsd !== undefined ? Math.round(spec.maxCostUsd * 1e6) : undefined;
         const remainingMicroUsd = prepared.resourceLedger
@@ -990,6 +1031,11 @@ export class Runner {
             if (event.entryId !== undefined && (m.role === "user" || m.role === "assistant" || m.role === "toolResult")) {
                 emitCommitted(event.entryId, m.role, m.role === "toolResult" ? m.toolCallId : undefined);
             }
+            if (degraded === undefined && m.role === "assistant") {
+                const deg = readDegradation(m);
+                if (deg)
+                    recordDegraded({ from: deg.from, to: deg.to, reason: deg.reason, atTurn: stats.turns + 1, ...(deg.chain && { chain: deg.chain }) });
+            }
             if (m.role === "assistant" && "usage" in m && m.usage) {
                 if (m.usageMissing)
                     turnUsageMissing = true;
@@ -1120,11 +1166,6 @@ export class Runner {
                 cadenceTurns = advanceCadenceClock(attachState, cadenceTurns, content, writeFamilyOf);
                 lastTurnHadToolCalls = Array.isArray(content) && content.some((c) => c?.type === "toolCall");
             }
-            if (degraded === undefined && m.role === "assistant") {
-                const deg = readDegradation(m);
-                if (deg)
-                    recordDegraded({ from: deg.from, to: deg.to, reason: deg.reason, atTurn: stats.turns + 1, ...(deg.chain && { chain: deg.chain }) });
-            }
             callStartAt = undefined;
             firstTokenAt = undefined;
             callOutputChars = 0;
@@ -1548,6 +1589,7 @@ export class Runner {
                                 ...(comp.postTriggerTokens !== undefined ? { tokensAfter: comp.postTriggerTokens } : {}),
                                 ...(comp.triggerTokens !== undefined ? { triggerTokensBefore: comp.triggerTokens } : {}),
                                 ...(comp.durationMs !== undefined ? { durationMs: comp.durationMs } : {}),
+                                ...(comp.phaseDurations !== undefined ? { phaseDurations: comp.phaseDurations } : {}),
                                 ...(comp.firstKeptEntryId !== undefined ? { preserved_segment: { firstKeptEntryId: comp.firstKeptEntryId } } : {}),
                                 ...(comp.attachedFiles !== undefined ? { attachedFiles: comp.attachedFiles } : {}),
                                 ...(comp.modelFallback ? { modelFallback: true } : {}),
@@ -1555,6 +1597,11 @@ export class Runner {
                                 ...(comp.clampedRatio !== undefined ? { clampedRatio: comp.clampedRatio } : {}),
                                 ...(comp.clampReason !== undefined ? { clampReason: comp.clampReason } : {}),
                             });
+                            if (comp.phaseDurations !== undefined && comp.durationMs !== undefined) {
+                                const pd = comp.phaseDurations;
+                                const pdDur = comp.durationMs;
+                                emitTrace(tracer, () => ({ kind: "compaction.phase_timings", version: 1, taskId, ...pd, durationMs: pdDur, ts: Date.now() }));
+                            }
                             prepared.cacheBreakDetector?.notifyCompaction();
                             if (attachState !== undefined) {
                                 attachState.postCompactPending = true;
@@ -1911,6 +1958,7 @@ export class Runner {
                         ...(comp.postTriggerTokens !== undefined ? { tokensAfter: comp.postTriggerTokens } : {}),
                         ...(comp.triggerTokens !== undefined ? { triggerTokensBefore: comp.triggerTokens } : {}),
                         ...(comp.durationMs !== undefined ? { durationMs: comp.durationMs } : {}),
+                        ...(comp.phaseDurations !== undefined ? { phaseDurations: comp.phaseDurations } : {}),
                         ...(comp.firstKeptEntryId !== undefined ? { preserved_segment: { firstKeptEntryId: comp.firstKeptEntryId } } : {}),
                         ...(comp.attachedFiles !== undefined ? { attachedFiles: comp.attachedFiles } : {}),
                         ...(comp.modelFallback ? { modelFallback: true } : {}),
@@ -1918,6 +1966,11 @@ export class Runner {
                         ...(comp.clampedRatio !== undefined ? { clampedRatio: comp.clampedRatio } : {}),
                         ...(comp.clampReason !== undefined ? { clampReason: comp.clampReason } : {}),
                     });
+                    if (comp.phaseDurations !== undefined && comp.durationMs !== undefined) {
+                        const pd = comp.phaseDurations;
+                        const pdDur = comp.durationMs;
+                        emitTrace(tracer, () => ({ kind: "compaction.phase_timings", version: 1, taskId, ...pd, durationMs: pdDur, ts: Date.now() }));
+                    }
                     prepared.cacheBreakDetector?.notifyCompaction();
                     if (attachState !== undefined) {
                         attachState.postCompactPending = true;
@@ -2137,6 +2190,7 @@ export class Runner {
                     ...(comp.postTriggerTokens !== undefined ? { tokensAfter: comp.postTriggerTokens } : {}),
                     ...(comp.triggerTokens !== undefined ? { triggerTokensBefore: comp.triggerTokens } : {}),
                     ...(comp.durationMs !== undefined ? { durationMs: comp.durationMs } : {}),
+                    ...(comp.phaseDurations !== undefined ? { phaseDurations: comp.phaseDurations } : {}),
                     ...(comp.firstKeptEntryId !== undefined ? { preserved_segment: { firstKeptEntryId: comp.firstKeptEntryId } } : {}),
                     ...(comp.attachedFiles !== undefined ? { attachedFiles: comp.attachedFiles } : {}),
                     ...(comp.modelFallback ? { modelFallback: true } : {}),
@@ -2144,6 +2198,11 @@ export class Runner {
                     ...(comp.clampedRatio !== undefined ? { clampedRatio: comp.clampedRatio } : {}),
                     ...(comp.clampReason !== undefined ? { clampReason: comp.clampReason } : {}),
                 });
+                if (comp.phaseDurations !== undefined && comp.durationMs !== undefined) {
+                    const pd = comp.phaseDurations;
+                    const pdDur = comp.durationMs;
+                    emitTrace(tracer, () => ({ kind: "compaction.phase_timings", version: 1, taskId, ...pd, durationMs: pdDur, ts: Date.now() }));
+                }
                 prepared.cacheBreakDetector?.notifyCompaction();
             }
             if (prepared.nestedStats.tasks > 0) {