taskchef 7.14.2 → 7.15.0

package/src/dashboard/state.js

@@ -44,6 +44,7 @@ export function latestTurnPresentation(task) {
   const requestSummary = turn?.requestSummary
     ?? (turn ? "Request not recorded by this TaskChef version." : task.title);
   return {
+    turnRef: turn?.turnRef ?? turn?.turnId ?? task.turnRef ?? task.turnId ?? null,
     turnId: turn?.turnId ?? task.turnId ?? null,
     startedAt: turn?.startedAt ?? task.updatedAt ?? task.createdAt ?? null,
     requestSummary,
@@ -70,11 +71,25 @@ export function mergeProjectedTurns(task, preservedTurns = []) {
   if (!task.latestTurn) return preservedTurns;
   const turns = [...preservedTurns];
   const lastIndex = turns.length - 1;
-  if (lastIndex >= 0 && turns[lastIndex].turnId === task.latestTurn.turnId) {
+  const latestIdentity = task.latestTurn.turnRef ?? task.latestTurn.turnId;
+  const preservedIdentity = lastIndex >= 0
+    ? (turns[lastIndex].turnRef ?? turns[lastIndex].turnId)
+    : null;
+  const migratedFallbackIdentity = lastIndex >= 0
+    && preservedIdentity === null
+    && latestIdentity !== null
+    && turns[lastIndex].turnId == null
+    && task.latestTurn.turnId == null
+    && JSON.stringify({ ...turns[lastIndex], turnRef: null })
+      === JSON.stringify({ ...task.latestTurn, turnRef: null });
+  if (
+    lastIndex >= 0
+    && (preservedIdentity === latestIdentity || migratedFallbackIdentity)
+  ) {
     turns[lastIndex] = task.latestTurn;
   } else {
     if (
-      task.schemaVersion === 8
+      task.schemaVersion >= 8
       && lastIndex >= 0
       && turns[lastIndex].result === null
     ) {
@@ -144,7 +159,30 @@ export function nextDateFilterRefreshDelay(tasks, filter, now = Date.now()) {
 }
 
 export function taskSignature(task) {
-  return JSON.stringify([task.id, task.turnId ?? null, task.status ?? "unresolved"]);
+  return JSON.stringify([
+    task.id,
+    task.turnRef ?? task.turnId ?? null,
+    task.turnId ?? null,
+    task.status ?? "unresolved",
+  ]);
+}
+
+function signaturesDifferOnlyByMigratedFallback(previous, next) {
+  try {
+    const before = JSON.parse(previous);
+    const after = JSON.parse(next);
+    return Array.isArray(before)
+      && Array.isArray(after)
+      && before.length === after.length
+      && before[1] === null
+      && after[1] !== null
+      && before[2] === null
+      && after[2] === null
+      && before[3] !== "working"
+      && before.every((value, index) => index === 1 || value === after[index]);
+  } catch {
+    return false;
+  }
 }
 
 export function findCurrentTask(tasks, taskId) {
@@ -162,14 +200,15 @@ function notificationIdentity(task, event) {
   if (event === "created") {
     return JSON.stringify([task.id, null, event, task.createdAt ?? null]);
   }
-  return JSON.stringify([task.id, task.turnId ?? null, event]);
+  return JSON.stringify([task.id, task.turnRef ?? task.turnId ?? null, event]);
 }
 
 function eventTimestamp(task, event) {
   if (event === "created") return task.createdAt ?? task.updatedAt ?? null;
   if (
     task.lastResult?.status === task.status
-    && task.lastResult?.turnId === task.turnId
+    && (task.lastResult?.turnRef ?? task.lastResult?.turnId)
+      === (task.turnRef ?? task.turnId)
   ) {
     return task.lastResult.updatedAt;
   }
@@ -180,7 +219,8 @@ function eventSummary(task, event) {
   if (!["completed", "needs_input", "failed"].includes(event)) return null;
   if (
     task.lastResult?.status === task.status
-    && task.lastResult?.turnId === task.turnId
+    && (task.lastResult?.turnRef ?? task.lastResult?.turnId)
+      === (task.turnRef ?? task.turnId)
   ) {
     return task.lastResult.summary;
   }
@@ -195,6 +235,7 @@ export function notificationSnapshot(task, event = lifecycleEvent(task)) {
     title: task.title,
     status: created ? "working" : task.status,
     event,
+    turnRef: created ? null : task.turnRef ?? task.turnId ?? null,
     turnId: created ? null : task.turnId ?? null,
     timestamp: eventTimestamp(task, event),
     summary: eventSummary(task, event),
@@ -205,11 +246,16 @@ function resultNotificationSnapshot(task) {
   const result = task.lastResult;
   if (!result) return null;
   return Object.freeze({
-    id: notificationIdentity({ ...task, turnId: result.turnId }, result.status),
+    id: notificationIdentity({
+      ...task,
+      turnRef: result.turnRef ?? result.turnId,
+      turnId: result.turnId,
+    }, result.status),
     taskId: task.id,
     title: task.title,
     status: result.status,
     event: result.status,
+    turnRef: result.turnRef ?? result.turnId ?? null,
     turnId: result.turnId ?? null,
     timestamp: result.updatedAt,
     summary: result.summary,
@@ -259,12 +305,18 @@ export function reconcileNotifications(
     const candidates = [];
     if (!signatures.has(task.id)) {
       const resultNotification = resultNotificationSnapshot(task);
-      if (task.turnId || task.status !== "working" || resultNotification) {
+      if (task.turnRef || task.turnId || task.status !== "working" || resultNotification) {
         candidates.push(notificationSnapshot(task));
         if (resultNotification) candidates.push(resultNotification);
       }
       candidates.push(notificationSnapshot(task, "created"));
-    } else if (signatures.get(task.id) !== nextSignatures.get(task.id)) {
+    } else if (
+      signatures.get(task.id) !== nextSignatures.get(task.id)
+      && !signaturesDifferOnlyByMigratedFallback(
+        signatures.get(task.id),
+        nextSignatures.get(task.id),
+      )
+    ) {
       candidates.push(notificationSnapshot(task));
       const resultNotification = resultNotificationSnapshot(task);
       if (resultNotification) candidates.push(resultNotification);

package/src/dashboard.js

@@ -138,10 +138,13 @@ function assertDashboardTaskBounds(tasks, maximumTasks) {
     boundedText(task.instruction, 250_000, `${name} instruction`);
     boundedText(task.summary, 2_000, `${name} summary`);
     boundedText(task.threadId, 512, `${name} thread ID`);
+    boundedText(task.turnRef, 512, `${name} turn ref`);
     boundedText(task.turnId, 512, `${name} turn ID`);
     boundedText(task.lastResult?.summary, 2_000, `${name} last result summary`);
+    boundedText(task.lastResult?.turnRef, 512, `${name} last result turn ref`);
     boundedText(task.lastResult?.turnId, 512, `${name} last result turn ID`);
     boundedText(task.latestTurn?.requestSummary, 1_000, `${name} latest request summary`);
+    boundedText(task.latestTurn?.turnRef, 512, `${name} latest turn ref`);
     boundedText(task.latestTurn?.turnId, 512, `${name} latest turn ID`);
     const turns = task.turns ?? [];
     if (turns.length > 10_000) {
@@ -149,6 +152,7 @@ function assertDashboardTaskBounds(tasks, maximumTasks) {
     }
     for (const [turnIndex, turn] of turns.entries()) {
       boundedText(turn.requestSummary, 1_000, `${name} turn ${turnIndex + 1} request summary`);
+      boundedText(turn.turnRef, 512, `${name} turn ${turnIndex + 1} turn ref`);
       boundedText(turn.turnId, 512, `${name} turn ${turnIndex + 1} turn ID`);
       boundedText(turn.result?.summary, 2_000, `${name} turn ${turnIndex + 1} result summary`);
     }
@@ -158,6 +162,7 @@ function assertDashboardTaskBounds(tasks, maximumTasks) {
     }
     for (const [resultIndex, result] of results.entries()) {
       boundedText(result.summary, 2_000, `${name} result ${resultIndex + 1} summary`);
+      boundedText(result.turnRef, 512, `${name} result ${resultIndex + 1} turn ref`);
       boundedText(result.turnId, 512, `${name} result ${resultIndex + 1} turn ID`);
     }
     boundedText(task.project.name, 1_000, `${name} project name`);

package/src/delegation.js

@@ -70,17 +70,19 @@ function toolIdentifier(value) {
   return normalized.length > 0 ? normalized : null;
 }
 
-function attachCreationRecovery(error, taskId, resultReporting) {
+function attachCreationRecovery(error, taskId, turnRef, resultReporting) {
   const creationError = error instanceof Error ? error : new Error(String(error));
   try {
     Object.defineProperties(creationError, {
       taskChefTaskId: { value: taskId, enumerable: true },
+      taskChefTurnRef: { value: turnRef, enumerable: true },
       taskChefResultReporting: { value: resultReporting, enumerable: true },
     });
     return creationError;
   } catch {
     const wrapped = new Error(`Executor creation failed for recorded TaskChef task ${taskId}.`, { cause: creationError });
     wrapped.taskChefTaskId = taskId;
+    wrapped.taskChefTurnRef = turnRef;
     wrapped.taskChefResultReporting = resultReporting;
     return wrapped;
   }
@@ -238,16 +240,29 @@ export async function createAndRecordDelegation(input) {
   try {
     createResult = parseToolResult(await createThread({ prompt: prepared.instruction, title, target }), "create_thread result");
   } catch (error) {
+    const creationFailureTurnRef = randomUUID();
     let resultReporting = "unavailable";
     if (reportRecordedResult !== null) {
       try {
-        await reportRecordedResult({ taskId: prepared.id, threadId: null, turnId: null, status: "failed", summary: "Executor creation failed before the executor started." });
+        await reportRecordedResult({
+          taskId: prepared.id,
+          threadId: null,
+          turnRef: creationFailureTurnRef,
+          turnId: null,
+          status: "failed",
+          summary: "Executor creation failed before the executor started.",
+        });
         resultReporting = "recorded";
       } catch {
         resultReporting = "failed";
       }
     }
-    throw attachCreationRecovery(error, prepared.id, resultReporting);
+    throw attachCreationRecovery(
+      error,
+      prepared.id,
+      creationFailureTurnRef,
+      resultReporting,
+    );
   }
 
   const returnedThreadId = toolIdentifier(createResult.threadId);

package/src/mcp.js

@@ -22,7 +22,7 @@ const projectSchema = z.object({
 
 const taskSchema = z.object({
   schemaVersion: z.union([
-    z.literal(4), z.literal(5), z.literal(6), z.literal(7), z.literal(8),
+    z.literal(4), z.literal(5), z.literal(6), z.literal(7), z.literal(8), z.literal(9),
   ]),
   id: z.string(),
   project: projectSchema,
@@ -32,10 +32,12 @@ const taskSchema = z.object({
   createdAt: z.string(),
   status: z.enum(["working", "needs_input", "completed", "failed"]),
   summary: z.string().nullable(),
+  turnRef: z.string().nullable(),
   turnId: z.string().nullable(),
   updatedAt: z.string(),
   updatedBy: z.enum(["dispatcher", "mcp"]),
   turns: z.array(z.object({
+    turnRef: z.string().nullable(),
     turnId: z.string().nullable(),
     requestSummary: z.string().nullable(),
     startedAt: z.string(),
@@ -46,6 +48,7 @@ const taskSchema = z.object({
     }).nullable(),
   })),
   latestTurn: z.object({
+    turnRef: z.string().nullable(),
     turnId: z.string().nullable(),
     requestSummary: z.string().nullable(),
     startedAt: z.string(),
@@ -58,12 +61,14 @@ const taskSchema = z.object({
   results: z.array(z.object({
     status: z.enum(["needs_input", "completed", "failed"]),
     summary: z.string(),
+    turnRef: z.string().nullable(),
     turnId: z.string().nullable(),
     updatedAt: z.string(),
   })),
   lastResult: z.object({
     status: z.enum(["needs_input", "completed", "failed"]),
     summary: z.string(),
+    turnRef: z.string().nullable(),
     turnId: z.string().nullable(),
     updatedAt: z.string(),
   }).nullable(),
@@ -225,10 +230,11 @@ export function createTaskChefMcpServer({
     {
       title: "Report TaskChef state",
       description:
-        "Report this self-linked executor turn's lifecycle state. Use working before substantive work in a newly linked or follow-up turn, with summary omitted or null and a concise requestSummary. Preserve known repository context and delivered issues or pull requests as canonical GitHub URLs, including both child-repository and workspace pull requests when applicable. Before ending the same turn, report needs_input, completed, or failed with a concise semantic summary and requestSummary omitted. Exact retries are idempotent; stale or mismatched turns are rejected.",
+        "Report this self-linked executor turn's lifecycle state. Use one stable turnRef for working and its terminal report; pass the native Codex turn ID as both turnRef and turnId when available, otherwise pass a client-generated UUID turnRef and null turnId. Preserve known repository context and delivered links. Exact retries are idempotent; stale or mismatched turnRefs are rejected.",
       inputSchema: {
         taskId: z.string().min(1),
         threadId: z.string().min(1).nullable(),
+        turnRef: z.string().min(1).max(256).nullable().optional(),
         turnId: z.string().min(1).max(256).nullable(),
         status: z.enum(["working", "needs_input", "completed", "failed"]),
         summary: z.string().min(1).max(2_000).nullable().optional(),
@@ -256,6 +262,7 @@ export function createTaskChefMcpServer({
       inputSchema: {
         taskId: z.string().min(1),
         threadId: z.string().min(1).nullable(),
+        turnRef: z.string().min(1).max(256).nullable().optional(),
         turnId: z.string().min(1).max(256).nullable(),
         status: z.enum(["needs_input", "completed", "failed"]),
         summary: z.string().min(1).max(2_000),