@sema-agent/core 1.280.0 → 1.282.0

package/dist/core/runner/runtask.js

@@ -300,6 +300,8 @@ export class Runner {
             for (const w of manualCompactRef.waiters.splice(0))
                 w(outcome);
         };
+        const runStartedAt = Date.now();
+        const taskIdRef = { current: spec.taskId, sessionId: spec.sessionId };
         const run = (async () => {
             const releaseLock = spec.sessionId
                 ? await this.acquireSessionLock(spec.sessionId)
@@ -316,7 +318,7 @@ export class Runner {
                     publishReady(h);
                 }, (s) => {
                     reapHandle = s;
-                }, manualCompactRef, resume, { ...(internals ?? {}), detachHub });
+                }, manualCompactRef, taskIdRef, resume, { ...(internals ?? {}), detachHub });
             }
             finally {
                 releaseLock?.();
@@ -346,15 +348,35 @@ export class Runner {
                 catch {
                 }
             }
+            if (resultValue !== undefined) {
+                try {
+                    this.deps.onError?.(err instanceof Error ? err : new Error(String(err)), { phase: "config", sessionId: resultValue.sessionId });
+                }
+                catch {
+                }
+                queue.close();
+                return;
+            }
             resultValue = {
-                taskId: spec.taskId ?? "unknown",
-                sessionId: spec.sessionId ?? "unknown",
+                taskId: taskIdRef.current ?? "unknown",
+                sessionId: taskIdRef.sessionId ?? "unknown",
                 status: "failed",
                 result: "",
                 errorMessage: err instanceof Error ? err.message : String(err),
                 errorCode: code,
                 stats: { turns: 0, tokens: 0, toolCalls: 0, cachedTokens: 0, costUsd: 0 },
             };
+            emitTrace(spec.tracer ?? this.deps.tracer, () => ({
+                kind: "task.end",
+                version: 1,
+                taskId: taskIdRef.current ?? "unknown",
+                status: "failed",
+                errorCode: code,
+                turns: 0,
+                tokens: 0,
+                durationMs: Date.now() - runStartedAt,
+                ts: Date.now(),
+            }));
             queue.push({ type: "done", result: resultValue });
             queue.close();
         });
@@ -460,7 +482,7 @@ export class Runner {
             })()),
         };
     }
-    async runLocked(spec, queue, setResult, onConsolidation, onSuggestions, onReady, onSuspend, manualCompactRef, resume, internals) {
+    async runLocked(spec, queue, setResult, onConsolidation, onSuggestions, onReady, onSuspend, manualCompactRef, taskIdRef, resume, internals) {
         const prepareResume = resume
             ? {
                 leafId: resume.cp.leafId,
@@ -569,6 +591,10 @@ export class Runner {
         const effectiveMaxTurns = resolveMaxTurns(spec.limits);
         const tracer = spec.tracer ?? this.deps.tracer;
         const taskId = spec.taskId ?? prepared.sessionId;
+        if (taskIdRef) {
+            taskIdRef.current = taskId;
+            taskIdRef.sessionId = prepared.sessionId;
+        }
         const taskStart = Date.now();
         void Promise.resolve(this.sessions.noteTaskRun?.(prepared.sessionId, taskId)).catch(() => { });
         const recordDegraded = (info, toModel) => {
@@ -783,6 +809,14 @@ export class Runner {
             }
         }
         let nudgeIdx = 0;
+        const buildFinalizeText = () => `<system-reminder>[deadline] the wall-clock budget is exhausted — this is the FINAL turn. ` +
+            `Write your best current work to the deliverable path NOW, even if incomplete — persist to disk FIRST; everything else comes after. ` +
+            `If a better version exists anywhere else (a draft, scratch or temp file), copy it to the deliverable path now. ` +
+            `Do NOT open new work, new investigations, or long commands; write out and stop.` +
+            (finalVerificationOn
+                ? ` After writing, if seconds permit, confirm the deliverable file actually EXISTS at the required path with your latest content.`
+                : "") +
+            `</system-reminder>`;
         const timeout = startTimeout(prepared.harness, prepared.abortController, spec.limits?.timeoutSec, prepared.suspendForResource !== undefined);
         const parentToolCallId = internals?.parentToolCallId;
         const ident = () => parentToolCallId !== undefined ? { eventId: uuidv7(), parentToolCallId, sourceTaskId: taskId } : { eventId: uuidv7() };
@@ -975,6 +1009,15 @@ export class Runner {
             }
             if (m.role === "assistant" && m.errorMessage === WALLTIME_CUTOFF_MESSAGE) {
                 callCutoffs += 1;
+                if (prepared.callCapRef)
+                    prepared.callCapRef.finalizeMode = true;
+                if (!finalizeInjected) {
+                    const finalizeText = buildFinalizeText();
+                    void prepared.harness.steer(finalizeText).catch(() => { });
+                    finalizeInjected = true;
+                    nudgeIdx = nudgeSchedule.length;
+                    queue.push({ type: "steering_injected", source: "finalize", preview: finalizeText.slice(0, 220), ...ident() });
+                }
             }
             if (m.role === "assistant") {
                 const rep = m.repetition;
@@ -1430,14 +1473,7 @@ export class Runner {
                         finalizeInjected = true;
                         prepared.callCapRef.finalizeMode = true;
                         nudgeIdx = nudgeSchedule.length;
-                        const finalizeText = `<system-reminder>[deadline] the wall-clock budget is exhausted — this is the FINAL turn. ` +
-                            `Write your best current work to the deliverable path NOW, even if incomplete — persist to disk FIRST; everything else comes after. ` +
-                            `If a better version exists anywhere else (a draft, scratch or temp file), copy it to the deliverable path now. ` +
-                            `Do NOT open new work, new investigations, or long commands; write out and stop.` +
-                            (finalVerificationOn
-                                ? ` After writing, if seconds permit, confirm the deliverable file actually EXISTS at the required path with your latest content.`
-                                : "") +
-                            `</system-reminder>`;
+                        const finalizeText = buildFinalizeText();
                         void prepared.harness.steer(finalizeText).catch(() => { });
                         boundarySteered = true;
                         queue.push({ type: "steering_injected", source: "finalize", preview: finalizeText.slice(0, 220), ...ident() });
@@ -1990,13 +2026,17 @@ export class Runner {
             }
             const capShrinks = prepared.callCapRef?.shrinks ?? 0;
             const toolClamps = prepared.callCapRef?.toolClamps ?? 0;
-            if (finalizeInjected || nudgesSent > 0 || capShrinks > 0 || callCutoffs > 0 || toolClamps > 0 || finalVerifyInjections > 0 || attachmentsInjected > 0 || repetitionCuts > 0 || repetitionSpared > 0) {
+            const toolCuts = prepared.callCapRef?.toolCuts ?? 0;
+            const pendingCutTools = prepared.callCapRef?.pendingCutTools ?? 0;
+            if (finalizeInjected || nudgesSent > 0 || capShrinks > 0 || callCutoffs > 0 || toolClamps > 0 || toolCuts > 0 || pendingCutTools > 0 || finalVerifyInjections > 0 || attachmentsInjected > 0 || repetitionCuts > 0 || repetitionSpared > 0) {
                 stats.mechanisms = {
                     ...(finalizeInjected ? { finalizeInjected: true } : {}),
                     ...(nudgesSent > 0 ? { nudgesSent } : {}),
                     ...(capShrinks > 0 ? { capShrinks } : {}),
                     ...(callCutoffs > 0 ? { callCutoffs } : {}),
                     ...(toolClamps > 0 ? { toolClamps } : {}),
+                    ...(toolCuts > 0 ? { toolCuts } : {}),
+                    ...(pendingCutTools > 0 ? { pendingCutTools } : {}),
                     ...(finalVerifyInjections > 0 ? { finalVerifyInjected: true } : {}),
                     ...(finalVerifyInjections > 0 ? { finalVerifyInjections } : {}),
                     ...(attachmentsInjected > 0 ? { attachmentsInjected } : {}),