taskchef 7.14.2 → 7.15.1

package/src/dashboard/app.js

@@ -61,13 +61,19 @@ const elements = {
 };
 let notificationDescriptionSerial = 0;
 
-function referenceLink(link, { compact = false } = {}) {
+function referenceLink(link, { compact = false, typePrefix = false } = {}) {
   const anchor = document.createElement("a");
   anchor.className = compact ? "github-link github-link-compact" : "github-link";
   anchor.href = link.url;
   anchor.target = "_blank";
   anchor.rel = "noopener noreferrer";
-  anchor.textContent = link.label ?? link.text;
+  const label = link.label ?? link.text;
+  const visibleLabel = typePrefix && link.type === "pull"
+    ? `PR ${label}`
+    : typePrefix && link.type === "issue"
+      ? `Issue ${label}`
+      : label;
+  anchor.textContent = visibleLabel;
   const githubKind = link.type === "issue"
     ? ", GitHub issue"
     : link.type === "pull"
@@ -75,7 +81,7 @@ function referenceLink(link, { compact = false } = {}) {
       : link.provider === "github" || link.owner ? " on GitHub" : "";
   anchor.setAttribute(
     "aria-label",
-    `${link.label ?? link.text}${githubKind} (opens in a new tab)`,
+    `${visibleLabel}${githubKind} (opens in a new tab)`,
   );
   anchor.addEventListener("click", (event) => event.stopPropagation());
   return anchor;
@@ -87,7 +93,7 @@ function appendLinkedText(container, text, task) {
     taskRepository: task.relatedGitHubRepository,
   }).map((segment) => {
     if (segment.kind === "text") return document.createTextNode(segment.text);
-    if (segment.kind === "link") return referenceLink(segment);
+    if (segment.kind === "link") return referenceLink(segment, { typePrefix: true });
     const ambiguous = document.createElement("span");
     ambiguous.className = "github-reference-ambiguous";
     ambiguous.textContent = segment.text;
@@ -100,7 +106,12 @@ function appendLinkedText(container, text, task) {
 }
 
 function relatedGitHubLinks(task, { compact = false } = {}) {
-  const links = task.relatedGitHubLinks ?? [];
+  const seen = new Set();
+  const links = (task.relatedGitHubLinks ?? []).filter((link) => {
+    if (link.type === "repository" || seen.has(link.label)) return false;
+    seen.add(link.label);
+    return true;
+  });
   const container = document.createElement("nav");
   container.className = `github-links${compact ? " github-links-compact" : ""}`;
   container.setAttribute("aria-label", `Related GitHub links for ${task.title}`);
@@ -118,6 +129,26 @@ function relatedGitHubLinks(task, { compact = false } = {}) {
   return container;
 }
 
+function renderProject(container, task) {
+  const children = [document.createTextNode(task.project.name)];
+  const repository = task.relatedGitHubRepository;
+  if (repository) {
+    const [owner, repositoryName] = repository.split("/");
+    children.push(
+      document.createTextNode(" "),
+      referenceLink({
+        label: repository,
+        owner,
+        provider: "github",
+        repository: repositoryName,
+        type: "repository",
+        url: `https://github.com/${repository}`,
+      }),
+    );
+  }
+  container.replaceChildren(...children);
+}
+
 function timestampControl(value, { accessibleName, key, prefix = "" }) {
   if (!parsedTimestamp(value)) {
     const missing = document.createElement("span");
@@ -259,7 +290,7 @@ function notificationAnnouncement(notification) {
     notificationTitle(notification),
     notification.title,
     notification.summary,
-    notification.turnId ? `Turn ${notification.turnId}` : null,
+    notification.turnRef ? `Turn ref ${notification.turnRef}` : null,
     formatRelativeTime(notification.timestamp),
   ].filter(Boolean).join(". ");
 }
@@ -302,7 +333,7 @@ function turnTimeline(task) {
     const turnStatus = presentation.status;
     status.className = `status status-${turnStatus}`;
     status.textContent = turnStatus.replaceAll("_", " ");
-    const turnKey = turn.turnId ?? `no-turn:${index}`;
+    const turnKey = turn.turnRef ?? turn.turnId ?? `no-turn:${index}`;
     const timestamp = timestampControl(presentation.updatedAt, {
       accessibleName: `Turn updated time for ${turnStatus.replaceAll("_", " ")}`,
       key: `detail:${task.id}:turn:${turnKey}`,
@@ -324,9 +355,7 @@ function turnTimeline(task) {
     appendLinkedText(result, presentation.summary, task);
     const turnMetadata = document.createElement("p");
     turnMetadata.className = "result-history-turn";
-    turnMetadata.textContent = turn.turnId
-      ? `Turn ${turn.turnId}`
-      : "No turn ID (creation failure)";
+    turnMetadata.textContent = `Turn ref ${turn.turnRef ?? "not recorded"}; Codex turn ${turn.turnId ?? "unavailable"}`;
     item.append(header, requestLabel, request, resultLabel, result, turnMetadata);
     return item;
   });
@@ -342,7 +371,7 @@ function renderDialog(task) {
     results: task.results ?? preservedResults ?? [],
   };
   state.selectedTask = detailedTask;
-  elements.dialogProject.textContent = task.project.name;
+  renderProject(elements.dialogProject, task);
   elements.dialogTitle.textContent = task.title;
   elements.dialogRelatedLinks.replaceChildren(...relatedGitHubLinks(detailedTask).children);
   elements.dialogRelatedLinks.setAttribute(
@@ -357,9 +386,11 @@ function renderDialog(task) {
   elements.copyThreadId.disabled = !task.threadId;
   elements.dialogMetadata.replaceChildren(
     ...detailRow("Current status", taskStatusLabel(task)),
-    ...detailRow("Current turn ID", task.turnId),
+    ...detailRow("Current turn ref", task.turnRef),
+    ...detailRow("Current Codex turn ID", task.turnId),
     ...detailRow("Last result status", task.lastResult?.status?.replaceAll("_", " ")),
-    ...detailRow("Last result turn ID", task.lastResult?.turnId),
+    ...detailRow("Last result turn ref", task.lastResult?.turnRef),
+    ...detailRow("Last result Codex turn ID", task.lastResult?.turnId),
     ...detailRow("Last result updated", timestampControl(task.lastResult?.updatedAt, {
       accessibleName: `Last result updated time for ${task.title}`,
       key: `detail:${task.id}:last-result-updated`,
@@ -421,7 +452,7 @@ function taskCard(task) {
   heading.append(title, badge);
   const project = document.createElement("p");
   project.className = "task-project";
-  project.textContent = task.project.name;
+  renderProject(project, task);
   const summary = document.createElement("p");
   summary.className = "task-summary";
   const latest = latestTurnPresentation(task);

package/src/dashboard/github-links.js

@@ -323,7 +323,8 @@ export function taskGitHubProjection(task) {
       taskRepository,
     })) {
       if (segment.kind !== "link") continue;
-      const key = segment.url;
+      if (segment.type === "repository") continue;
+      const key = relatedLinkLabel(segment);
       if (seen.has(key)) continue;
       if (links.length === MAX_RELATED_GITHUB_LINKS) {
         truncated = true;

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),