@workbench-ai/workbench 0.0.76 → 0.0.78

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAgEA,MAAM,WAAW,KAAK;IACpB,MAAM,EAAE,MAAM,CAAC,cAAc,CAAC;IAC9B,MAAM,EAAE,MAAM,CAAC,cAAc,CAAC;CAC/B;AAuTD,wBAAsB,MAAM,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,EAAE,EAAE,GAAE,KAGzD,GAAG,OAAO,CAAC,MAAM,CAAC,CAoMlB"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAiEA,MAAM,WAAW,KAAK;IACpB,MAAM,EAAE,MAAM,CAAC,cAAc,CAAC;IAC9B,MAAM,EAAE,MAAM,CAAC,cAAc,CAAC;CAC/B;AAuTD,wBAAsB,MAAM,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,EAAE,EAAE,GAAE,KAGzD,GAAG,OAAO,CAAC,MAAM,CAAC,CAoMlB"}

package/dist/index.js

@@ -404,7 +404,7 @@ export async function runCli(argv, io = {
         if (command === "diff") {
             const range = optionalPositional(parsed, 1) ?? await defaultDiffRange(core);
             const diffs = await diffWorkbenchVersions(range, core);
-            return output(diffs, parsed, io, () => diffs.map((entry) => `${entry.status}\t${entry.path}`).join("\n") || "No diff.");
+            return output(diffs, parsed, io, () => formatDiff(diffs));
         }
         if (command === "show") {
             return await handleShow(parsed, io);
@@ -1047,8 +1047,7 @@ function formatFanOut(fanout) {
     if (fanout.linkedAgents.length === 0) {
         return "fanout: completed";
     }
-    const suffix = fanout.additionalAgents ? ` and ${fanout.additionalAgents} more` : "";
-    return `fanned out to: ${fanout.linkedAgents.join(", ")}${suffix}`;
+    return "fanout: completed";
 }
 async function latestInstallVersion(record) {
     const handle = normalizedOwnerSkillHandle(record.handle);
@@ -1413,7 +1412,7 @@ function cloudExecutionNextCommand(runs, successCommand) {
     if (!first) {
         return "workbench log --runs";
     }
-    if (first.status === "running" || first.status === "failed" || first.status === "canceled") {
+    if (first.status === "queued" || first.status === "running" || first.status === "failed" || first.status === "canceled") {
         return `workbench show ${displayRef(first.id)}`;
     }
     return successCommand;
@@ -1802,7 +1801,7 @@ async function apiRequest(apiPath, options = {}, baseUrlOverride) {
         : selectWorkbenchBaseUrl({ configBaseUrl: config.baseUrl });
     const token = await workbenchCloudToken({ baseUrl });
     const method = options.method ?? "GET";
-    const canRetry = method === "GET";
+    const canRetry = isIdempotentApiRequestMethod(method);
     const requestBody = encodeJsonRequestBody(options.body);
     let lastError = null;
     for (let attempt = 1; attempt <= API_REQUEST_MAX_ATTEMPTS; attempt += 1) {
@@ -1977,6 +1976,9 @@ function isTransientFetchError(error) {
 function isTransientApiRequestError(error) {
     return error instanceof WorkbenchApiRequestError && (error.status === 429 || error.status >= 500);
 }
+function isIdempotentApiRequestMethod(method) {
+    return method === "GET" || method === "PUT" || method === "DELETE";
+}
 function errorMessage(error) {
     return error instanceof Error ? error.message : String(error);
 }
@@ -2764,7 +2766,7 @@ async function statusWithCausalNext(status, auth, core, machine) {
     const lastRun = snapshot?.runs
         .slice()
         .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
-    if ((lastRun?.status === "running" || lastRun?.status === "failed" || lastRun?.status === "canceled") && lastRun.id) {
+    if ((lastRun?.status === "queued" || lastRun?.status === "running" || lastRun?.status === "failed" || lastRun?.status === "canceled") && lastRun.id) {
         return { ...status, next: `workbench show ${displayRef(lastRun.id)}` };
     }
     const failedRemote = status.remotes.find((remote) => remote.sync.status === "error");
@@ -2803,6 +2805,15 @@ async function statusWithCausalNext(status, auth, core, machine) {
         currentVersionId !== undefined &&
         remote.publication.versionId !== currentVersionId);
     if (canPublish && stalePublishedCloudRemote) {
+        const publishedVersionId = stalePublishedCloudRemote.publication.versionId;
+        if (snapshot && publishedVersionId && currentVersionId) {
+            if (versionHasAncestor(snapshot, currentVersionId, publishedVersionId)) {
+                return { ...status, next: "workbench publish" };
+            }
+            if (versionHasAncestor(snapshot, publishedVersionId, currentVersionId)) {
+                return { ...status, next: `workbench switch ${displayRef(publishedVersionId)}` };
+            }
+        }
         return { ...status, next: "workbench publish" };
     }
     const publishedCloudRemote = status.remotes.find((remote) => remote.kind === "workbench-cloud" &&
@@ -2816,6 +2827,43 @@ async function statusWithCausalNext(status, auth, core, machine) {
         next: null,
     };
 }
+function versionHasAncestor(snapshot, versionId, ancestorId) {
+    if (versionId === ancestorId) {
+        return true;
+    }
+    const parentsByChild = new Map();
+    for (const edge of snapshot.lineage) {
+        const parents = parentsByChild.get(edge.childId) ?? [];
+        parents.push(edge.parentId);
+        parentsByChild.set(edge.childId, parents);
+    }
+    for (const version of snapshot.versions) {
+        if (version.parentIds.length === 0) {
+            continue;
+        }
+        const parents = parentsByChild.get(version.id) ?? [];
+        for (const parentId of version.parentIds) {
+            if (!parents.includes(parentId)) {
+                parents.push(parentId);
+            }
+        }
+        parentsByChild.set(version.id, parents);
+    }
+    const seen = new Set();
+    const stack = [...(parentsByChild.get(versionId) ?? [])];
+    while (stack.length > 0) {
+        const next = stack.pop();
+        if (next === ancestorId) {
+            return true;
+        }
+        if (seen.has(next)) {
+            continue;
+        }
+        seen.add(next);
+        stack.push(...(parentsByChild.get(next) ?? []));
+    }
+    return false;
+}
 function displayRef(id) {
     const version = /^v_([0-9a-f]{8,})$/iu.exec(id);
     if (version?.[1]) {
@@ -3315,7 +3363,8 @@ function formatJob(job) {
 }
 function formatComparison(comparison) {
     const lines = ["version\tskill\tagent\tstatus\tscore\tcost\tlatency\trun"];
-    for (const cell of comparison.cells) {
+    const evidenceCells = comparison.cells.filter((cell) => cell.runId || cell.status);
+    for (const cell of evidenceCells) {
         lines.push([
             displayRef(cell.versionId),
             cell.skillName,
@@ -3327,7 +3376,73 @@ function formatComparison(comparison) {
             cell.runId ? displayRef(cell.runId) : "n/a",
         ].join("\t"));
     }
-    return lines.join("\n");
+    return lines.length === 1 ? "No comparable runs." : lines.join("\n");
+}
+function formatDiff(entries) {
+    if (entries.length === 0) {
+        return "No diff.";
+    }
+    return entries.map(formatDiffEntry).join("\n");
+}
+function formatDiffEntry(entry) {
+    const before = entry.before ?? "";
+    const after = entry.after ?? "";
+    if (entry.status === "modified" || entry.status === "added" || entry.status === "removed") {
+        return [
+            `diff --workbench ${entry.path}`,
+            `--- ${entry.status === "added" ? "/dev/null" : `a/${entry.path}`}`,
+            `+++ ${entry.status === "removed" ? "/dev/null" : `b/${entry.path}`}`,
+            ...unifiedLineDiff(before, after),
+        ].join("\n");
+    }
+    return `${entry.status}\t${entry.path}`;
+}
+function unifiedLineDiff(before, after) {
+    const beforeLines = splitDiffLines(before);
+    const afterLines = splitDiffLines(after);
+    const table = longestCommonSubsequenceTable(beforeLines, afterLines);
+    const lines = [];
+    let left = 0;
+    let right = 0;
+    while (left < beforeLines.length && right < afterLines.length) {
+        if (beforeLines[left] === afterLines[right]) {
+            lines.push(` ${beforeLines[left]}`);
+            left += 1;
+            right += 1;
+        }
+        else if (table[left + 1][right] >= table[left][right + 1]) {
+            lines.push(`-${beforeLines[left]}`);
+            left += 1;
+        }
+        else {
+            lines.push(`+${afterLines[right]}`);
+            right += 1;
+        }
+    }
+    while (left < beforeLines.length) {
+        lines.push(`-${beforeLines[left]}`);
+        left += 1;
+    }
+    while (right < afterLines.length) {
+        lines.push(`+${afterLines[right]}`);
+        right += 1;
+    }
+    return lines.length > 0 ? lines : [" "];
+}
+function splitDiffLines(value) {
+    const withoutFinalNewline = value.endsWith("\n") ? value.slice(0, -1) : value;
+    return withoutFinalNewline ? withoutFinalNewline.split(/\r?\n/u) : [];
+}
+function longestCommonSubsequenceTable(left, right) {
+    const table = Array.from({ length: left.length + 1 }, () => Array.from({ length: right.length + 1 }, () => 0));
+    for (let i = left.length - 1; i >= 0; i -= 1) {
+        for (let j = right.length - 1; j >= 0; j -= 1) {
+            table[i][j] = left[i] === right[j]
+                ? table[i + 1][j + 1] + 1
+                : Math.max(table[i + 1][j], table[i][j + 1]);
+        }
+    }
+    return table;
 }
 function shortObjectId(id) {
     return id.length > 8 ? id.slice(0, 8) : id;

package/dist/workbench.js

@@ -1,4 +1,12 @@
 #!/usr/bin/env node
 import { runCli } from "./index.js";
+for (const stream of [process.stdout, process.stderr]) {
+    stream.on("error", (error) => {
+        if (error.code === "EPIPE") {
+            process.exit(0);
+        }
+        throw error;
+    });
+}
 const exitCode = await runCli(process.argv.slice(2));
 process.exitCode = exitCode;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@workbench-ai/workbench",
-  "version": "0.0.76",
+  "version": "0.0.78",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/workbench-ai/workbench.git",
@@ -22,10 +22,10 @@
   "dependencies": {
     "skills": "1.5.11",
     "yaml": "^2.8.2",
-    "@workbench-ai/workbench-built-in-adapters": "0.0.76",
-    "@workbench-ai/workbench-core": "0.0.76",
-    "@workbench-ai/workbench-contract": "0.0.76",
-    "@workbench-ai/workbench-protocol": "0.0.76"
+    "@workbench-ai/workbench-built-in-adapters": "0.0.78",
+    "@workbench-ai/workbench-protocol": "0.0.78",
+    "@workbench-ai/workbench-contract": "0.0.78",
+    "@workbench-ai/workbench-core": "0.0.78"
   },
   "devDependencies": {
     "@tailwindcss/postcss": "^4.2.2",
@@ -36,7 +36,7 @@
     "react-dom": "^19.2.0",
     "typescript": "^5.9.2",
     "vitest": "^3.2.4",
-    "@workbench-ai/workbench-ui": "0.0.76"
+    "@workbench-ai/workbench-ui": "0.0.78"
   },
   "scripts": {
     "build": "rm -rf dist && tsc -p tsconfig.json && chmod 755 dist/workbench.js && node ./scripts/build-dev-open-assets.mjs",