taskchef 7.21.1 → 7.22.0

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "taskchef",
-  "version": "7.21.1",
+  "version": "7.22.0",
   "description": "Dispatch work from a data-only workspace to visible Codex project tasks.",
   "author": {
     "name": "Favo Yang",

package/README.md

@@ -273,23 +273,25 @@ timeline, including clearly labeled interrupted turns.
 Task details offer infrequent administrative actions without cluttering task
 cards. Selecting **More task actions** (`…`) reveals the action list immediately
 beside the disclosure and changes it to a back/hide control. The list contains
-**Copy Task ID**, direct **Mark completed** and **Mark failed** actions for a
-`working` or `needs_input` task, and **Archive chat** whenever archival is
-eligible. The menu disclosure is the deliberate first step; choosing a terminal
-outcome submits it immediately without a second confirmation. There is no
+**Copy Task ID** and direct **Mark completed** and **Mark failed** actions.
+Working and needs-input tasks offer either terminal outcome; completed tasks
+offer **Mark failed**, and failed tasks offer **Mark completed**. The menu
+disclosure is the deliberate first step; choosing an outcome submits it
+immediately without a second confirmation. There is no
 free-form reason: the audit turn records a fixed summary, timestamp, dashboard
 provenance, optimistic preconditions, and a unique action ID while preserving
-every executor turn. Terminal tasks cannot be rewritten. Stale or concurrent
-changes are rejected and the dialog refreshes to the current task. A stalled
+every executor turn. Same-state terminal rewrites remain invalid. Stale or
+concurrent changes are rejected and the dialog refreshes to the current task. A stalled
 local request is aborted after a bounded wait so the dialog cannot remain
 permanently locked; retry keeps the same idempotency identity.
 
-The More actions list offers **Archive chat** for every linked task whose current
-TaskChef state is not `working`. After confirmation, TaskChef invokes only the
-Codex CLI at the canonical ChatGPT or Codex desktop app location under
-`/Applications`, to archive the exact thread UUID. The Codex chat leaves active chat lists while the TaskChef record
-and its activity timeline remain unchanged. If the bundled CLI is unavailable,
-the dashboard does not fall back to another `codex` executable from `PATH`.
+The archive implementation is retained behind a disabled capability gate, and
+the **Archive chat** control is hidden. The bundled `codex archive` command can
+reject a valid idle desktop-app thread even when the dashboard runs in the MCP
+host environment, while Codex's native app archive operation succeeds. TaskChef
+will keep the feature disabled until Codex exposes a reliable supported
+app-callable archive interface or guarantees CLI compatibility. The dormant
+server path also rejects requests before discovering or invoking the CLI.
 
 Task details also show whole-task and per-turn token usage when `ccusage` can
 map the linked Codex thread. A completed turn briefly shows “Tokens:

package/docs/spec.md

@@ -483,14 +483,19 @@ startup safely. Direct thread navigation
 MUST require a canonical Codex UUIDv7. Otherwise it MAY open the revalidated
 configured project. Project paths from task history MUST be matched against
 current configuration before use.
-The task-detail dashboard MAY offer manual terminal outcomes only for current
-`working` or `needs_input` tasks. It MUST permit exactly `completed` and
-`failed`, MUST NOT rewrite a terminal task, and MUST keep this infrequent
-administrative action out of list cards. A keyboard-accessible **More task
+The task-detail dashboard MAY offer manual outcomes from `working` or
+`needs_input` to either `completed` or `failed`, from `completed` to `failed`,
+and from `failed` to `completed`. It MUST reject same-state terminal
+transitions and MUST keep this infrequent administrative action out of list
+cards. A keyboard-accessible **More task
 actions** disclosure MUST reveal its action list immediately beside it and
 change from an ellipsis to an accessible back/hide control while expanded. The
-list MUST group **Copy Task ID**, **Mark completed**, and **Mark failed** for
-eligible tasks, plus **Archive chat** when archival is eligible. Opening the
+list MUST group **Copy Task ID** and each currently valid **Mark completed** or
+**Mark failed** action. **Archive chat** MUST remain hidden while the archive capability
+gate is disabled because the bundled CLI does not reliably archive desktop-app
+threads. The dormant server endpoint MUST reject requests before discovering or
+invoking the CLI. Re-enabling requires a reliable supported app-callable archive
+interface or guaranteed CLI compatibility. Opening the
 list is the deliberate disclosure step; choosing a terminal outcome submits it
 immediately without a second confirmation. Escape MUST close the idle list,
 pending controls MUST be

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "taskchef",
-  "version": "7.21.1",
+  "version": "7.22.0",
   "description": "A non-blocking interactive dispatcher for visible Codex tasks.",
   "license": "MIT",
   "author": "Favo Yang",

package/src/dashboard/app.js

@@ -506,7 +506,6 @@ function replaceCurrentTask(task) {
 function renderManualTransition(task) {
   const activeElement = document.activeElement;
   const focusWasInPanel = elements.manualTransitionPanel.contains?.(activeElement) ?? false;
-  const eligible = canManuallyTransitionTask(task);
   state.manualTransition = reconcileManualTransition(state.manualTransition, task);
   const pending = manualTransitionPending();
   const expanded = Boolean(state.manualTransition);
@@ -522,8 +521,8 @@ function renderManualTransition(task) {
   elements.manualTransitionPanel.hidden = state.manualTransition === null;
   elements.manualTransitionPanel.setAttribute("aria-busy", String(pending));
   elements.copyTaskId.disabled = pending || !task.id;
-  elements.markTaskCompleted.hidden = !eligible;
-  elements.markTaskFailed.hidden = !eligible;
+  elements.markTaskCompleted.hidden = !canManuallyTransitionTask(task, "completed");
+  elements.markTaskFailed.hidden = !canManuallyTransitionTask(task, "failed");
   elements.markTaskCompleted.disabled = pending;
   elements.markTaskFailed.disabled = pending;
   const archivePending = state.archivePendingThreadIds.has(task.threadId);
@@ -561,7 +560,11 @@ function renderManualTransition(task) {
 
 async function submitManualTransition(targetStatus, event) {
   const task = state.selectedTask;
-  if (!task || !canManuallyTransitionTask(task) || manualTransitionPending()) return;
+  if (
+    !task
+    || !canManuallyTransitionTask(task, targetStatus)
+    || manualTransitionPending()
+  ) return;
   const previous = state.manualTransition;
   const actionId = previous?.targetStatus === targetStatus && previous.actionId
     ? previous.actionId

package/src/dashboard/state.js

@@ -15,6 +15,12 @@ export const STATUS_FILTERS = [
 ];
 const CODEX_THREAD_ID_PATTERN = /^[0-9a-f]{8}-[0-9a-f]{4}-7[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
 
+// Disabled because the bundled `codex archive` command can reject valid idle
+// desktop-app threads while Codex's native archive operation succeeds. Keep
+// the implementation behind this gate until Codex exposes a reliable,
+// supported app-callable archive interface or guarantees CLI compatibility.
+export const CODEX_CHAT_ARCHIVE_ENABLED = false;
+
 const NOTIFICATION_TITLES = new Map([
   ["created", "Task created"],
   ["task_started", "Task started"],
@@ -41,14 +47,25 @@ export function taskStatusLabel(task) {
   return task.status === null ? "unresolved" : task.status.replaceAll("_", " ");
 }
 
-export function canArchiveTask(task) {
+export function isArchiveTaskEligible(task) {
   return task.status !== "working"
     && typeof task.threadId === "string"
     && CODEX_THREAD_ID_PATTERN.test(task.threadId);
 }
 
-export function canManuallyTransitionTask(task) {
-  return ["working", "needs_input"].includes(task.status);
+export function canArchiveTask(task) {
+  return CODEX_CHAT_ARCHIVE_ENABLED && isArchiveTaskEligible(task);
+}
+
+export function canManuallyTransitionTask(task, targetStatus = null) {
+  const targets = task.status === "completed"
+    ? ["failed"]
+    : task.status === "failed"
+      ? ["completed"]
+      : ["working", "needs_input"].includes(task.status)
+        ? ["completed", "failed"]
+        : [];
+  return targetStatus === null ? targets.length > 0 : targets.includes(targetStatus);
 }
 
 export function manualTransitionExpectedState(task) {

package/src/dashboard.js

@@ -23,6 +23,7 @@ import {
 } from "./workspace.js";
 import { DASHBOARD_SERVER_VERSION, TASKCHEF_VERSION } from "./version.js";
 import { taskGitHubProjection } from "./dashboard/github-links.js";
+import { CODEX_CHAT_ARCHIVE_ENABLED } from "./dashboard/state.js";
 import { createUsageTracker } from "./usage-tracker.js";
 
 const TASKS_FILE_NAME = "tasks.jsonl";
@@ -529,6 +530,7 @@ function publicMonitorError() {
 }
 
 export async function createDashboardServer({
+  archiveEnabled = CODEX_CHAT_ARCHIVE_ENABLED,
   archiveThread = archiveThreadInCodex,
   discoverArchiveCli = discoverBundledCodexCli,
   workspace,
@@ -759,6 +761,10 @@ export async function createDashboardServer({
 
     const archiveMatch = url.pathname.match(/^\/api\/tasks\/([a-zA-Z0-9._-]+)\/archive-codex$/);
     if (archiveMatch && method === "POST") {
+      if (!archiveEnabled) {
+        sendJson(response, 404, { message: "Chat archiving is not available." });
+        return;
+      }
       if (request.headers.origin !== allowedOrigin) {
         sendJson(response, 403, { message: "Dashboard origin validation failed." });
         return;

package/src/mcp.js

@@ -28,7 +28,7 @@ const turnProvenanceSchema = z.union([
   z.object({
     kind: z.literal("dashboard_manual"),
     actionId: z.string(),
-    fromStatus: z.enum(["working", "needs_input"]),
+    fromStatus: z.enum(["working", "needs_input", "completed", "failed"]),
     toStatus: z.enum(["completed", "failed"]),
     expectedTurnRef: z.string().nullable(),
     expectedThreadId: z.string().nullable(),

package/src/workspace.js

@@ -105,7 +105,9 @@ const MANUAL_TURN_PROVENANCE_FIELDS = new Set([
   "expectedUpdatedAt",
 ]);
 const INTERRUPTED_TURN_SUMMARY = "Turn interrupted before a terminal report.";
-const MANUAL_TRANSITION_STATUSES = new Set(["working", "needs_input"]);
+const MANUAL_TRANSITION_STATUSES = new Set([
+  "working", "needs_input", "completed", "failed",
+]);
 const MANUAL_TARGET_STATUSES = new Set(["completed", "failed"]);
 const MANUAL_TRANSITION_FIELDS = new Set([
   "actionId", "expected", "targetStatus",
@@ -1035,9 +1037,14 @@ async function validateDispatchShape(dispatch, name = "task") {
       if (provenance.expectedTurnRef !== (predecessor?.turnRef ?? null)) {
         throw new Error(`${turnName}.provenance.expectedTurnRef must match the prior turn`);
       }
+      if (!canManuallyTransition(provenance.fromStatus, provenance.toStatus)) {
+        throw new Error(`${turnName}.provenance describes an invalid manual transition`);
+      }
       const validPriorState = provenance.fromStatus === "needs_input"
         ? predecessor?.result?.status === "needs_input"
-        : predecessor === null || predecessor.result?.status === "interrupted";
+        : provenance.fromStatus === "working"
+          ? predecessor === null || predecessor.result?.status === "interrupted"
+          : predecessor?.result?.status === provenance.fromStatus;
       if (!validPriorState) {
         throw new Error(`${turnName}.provenance.fromStatus does not match the prior turn`);
       }
@@ -1048,9 +1055,9 @@ async function validateDispatchShape(dispatch, name = "task") {
       ) {
         throw new Error(`${turnName} must share its timestamp with the interrupted prior turn`);
       }
-      const expectedPriorTimestamp = provenance.fromStatus === "needs_input"
-        ? predecessor.result.updatedAt
-        : predecessor?.startedAt ?? null;
+      const expectedPriorTimestamp = provenance.fromStatus === "working"
+        ? predecessor?.startedAt ?? null
+        : predecessor.result.updatedAt;
       if (
         Date.parse(provenance.expectedUpdatedAt) < Date.parse(normalized.createdAt)
         || (
@@ -1666,6 +1673,13 @@ function manualTransitionRequestSummary(fromStatus, toStatus) {
   return `Manual dashboard transition from ${fromStatus} to ${toStatus}.`;
 }
 
+function canManuallyTransition(fromStatus, toStatus) {
+  if (!MANUAL_TRANSITION_STATUSES.has(fromStatus) || !MANUAL_TARGET_STATUSES.has(toStatus)) {
+    return false;
+  }
+  return ["working", "needs_input"].includes(fromStatus) || fromStatus !== toStatus;
+}
+
 function taskOperationError(code, message, task = null) {
   const error = new Error(message);
   error.code = code;
@@ -2009,7 +2023,7 @@ export async function manuallyTransitionTask(
     }
 
     const dispatch = dispatches[index];
-    if (!MANUAL_TRANSITION_STATUSES.has(dispatch.status)) {
+    if (!canManuallyTransition(dispatch.status, normalizedInput.targetStatus)) {
       throw taskOperationError(
         "invalid_transition",
         `task status cannot be changed manually from ${dispatch.status}: ${id}`,