@workbench-ai/workbench 0.0.77 → 0.0.79

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);
@@ -723,6 +723,7 @@ const API_REQUEST_MAX_ATTEMPTS = 3;
 const API_REQUEST_GZIP_THRESHOLD_BYTES = 1024 * 1024;
 const CLOUD_RUN_TIMEOUT_MS = 30 * 60 * 1000;
 const CLOUD_RUN_POLL_INTERVAL_MS = 3000;
+const LOGIN_WAIT_TIMEOUT_SECONDS = 120;
 async function handleLogin(parsed, io) {
     const provider = optionalPositional(parsed, 1);
     if (provider) {
@@ -739,7 +740,7 @@ async function handleLogin(parsed, io) {
     }
     if (parsed.flags["start-only"] === true && parsed.flags.wait === true) {
         throw new WorkbenchCodedError("usage", "workbench login accepts only one of --start-only or --wait.", {
-            remediation: "Run workbench login --start-only or workbench login --wait.",
+            remediation: `Run workbench login --start-only or workbench login --wait --timeout ${LOGIN_WAIT_TIMEOUT_SECONDS}.`,
             exitCode: 2,
         });
     }
@@ -748,7 +749,13 @@ async function handleLogin(parsed, io) {
     const timeoutSeconds = intFlag(parsed, "timeout");
     if (startOnly && timeoutSeconds !== undefined) {
         throw new WorkbenchCodedError("usage", "workbench login --timeout only applies with --wait.", {
-            remediation: "Run workbench login --start-only, then workbench login --wait.",
+            remediation: `Run workbench login --start-only, then workbench login --wait --timeout ${LOGIN_WAIT_TIMEOUT_SECONDS}.`,
+            exitCode: 2,
+        });
+    }
+    if (waitOnly && timeoutSeconds === undefined) {
+        throw new WorkbenchCodedError("usage", "workbench login --wait requires --timeout N.", {
+            remediation: `Run workbench login --wait --timeout ${LOGIN_WAIT_TIMEOUT_SECONDS}.`,
             exitCode: 2,
         });
     }
@@ -773,8 +780,8 @@ async function handleLogin(parsed, io) {
             verificationUriComplete: record.verification_uri_complete,
             userCode: record.user_code,
             expiresAt: record.expiresAt,
-            resume: "workbench login --wait",
-        }, parsed, io, () => `Open ${record.verification_uri_complete}\nCode: ${record.user_code}\nResume: workbench login --wait`);
+            resume: `workbench login --wait --timeout ${LOGIN_WAIT_TIMEOUT_SECONDS}`,
+        }, parsed, io, () => `Open ${record.verification_uri_complete}\nCode: ${record.user_code}\nResume: workbench login --wait --timeout ${LOGIN_WAIT_TIMEOUT_SECONDS}`);
     }
     await writePendingDeviceAuthorization(record);
     if (freshAuthorization && !parsed.flags.json) {
@@ -1047,8 +1054,7 @@ function formatFanOut(fanout) {
     if (fanout.linkedAgents.length === 0) {
         return "fanout: completed";
     }
-    const count = fanout.linkedAgents.length + (fanout.additionalAgents ?? 0);
-    return `fanout: linked ${count} ${count === 1 ? "agent" : "agents"}`;
+    return "fanout: completed";
 }
 async function latestInstallVersion(record) {
     const handle = normalizedOwnerSkillHandle(record.handle);
@@ -1727,7 +1733,7 @@ async function pollDeviceToken(baseUrl, authorization, timeoutSeconds) {
     }
     throw new WorkbenchCodedError("login_pending", "Device login is still waiting for browser authorization.", {
         retryable: true,
-        remediation: "Authorize the device in the browser, then run workbench login --wait.",
+        remediation: `Authorize the device in the browser, then run workbench login --wait --timeout ${LOGIN_WAIT_TIMEOUT_SECONDS}.`,
         subject: {
             retryAfterSeconds: Math.max(1, Math.ceil(intervalMs / 1000)),
             verificationUri: authorization.verification_uri,
@@ -1802,7 +1808,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 +1983,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);
 }
@@ -2748,11 +2757,6 @@ function snapshotHasWorkflowCase(snapshot) {
         /^\.workbench\/cases\/[^/]+\/case\.ya?ml$/u.test(file.path)) ?? [];
     return caseFiles.some((file) => file.kind === "text" && !/\n\s*smoke:\s*true(?:\s|$)/u.test(`\n${file.content}`));
 }
-function installHandleFromStatusRemote(remote) {
-    const publicationUrl = remote.publication.status === "published" ? remote.publication.installUrl : undefined;
-    const source = parseWorkbenchInstallSource(publicationUrl ?? remote.url);
-    return source ? `${source.owner}/${source.skill}` : publicationUrl ?? remote.url;
-}
 async function statusWithCausalNext(status, auth, core, machine) {
     if (!status.project.initialized) {
         return {
@@ -2803,19 +2807,59 @@ 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" &&
-        remote.publication.status === "published" &&
-        Boolean(remote.publication.installUrl));
-    if (publishedCloudRemote) {
-        return { ...status, next: `workbench install ${installHandleFromStatusRemote(publishedCloudRemote)}` };
-    }
     return {
         ...status,
         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]) {
@@ -3206,7 +3250,21 @@ async function evalSuccessNextCommand(core, runs) {
         return "edit .workbench/cases, then run workbench eval";
     }
     const snapshot = await createWorkbenchReadOnlyInspectionSnapshot(core);
-    return snapshotHasWorkflowCase(snapshot) ? "workbench publish" : "edit .workbench/cases, then run workbench eval";
+    if (!snapshotHasWorkflowCase(snapshot)) {
+        return "edit .workbench/cases, then run workbench eval";
+    }
+    const auth = await workbenchCliAuthStatus();
+    if (auth.workbenchCloud.status !== "authenticated") {
+        return "workbench login";
+    }
+    const status = await workbenchStatusSnapshot(core);
+    return statusHasPublishedCurrentCloudSource(status) ? null : "workbench publish";
+}
+function statusHasPublishedCurrentCloudSource(status) {
+    const currentVersionId = status.project.currentVersionId;
+    return Boolean(currentVersionId && status.remotes.some((remote) => remote.kind === "workbench-cloud" &&
+        remote.publication.status === "published" &&
+        remote.publication.versionId === currentVersionId));
 }
 function formatStatusSnapshot(status) {
     const lines = [
@@ -3315,7 +3373,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 +3386,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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@workbench-ai/workbench",
-  "version": "0.0.77",
+  "version": "0.0.79",
   "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.77",
-    "@workbench-ai/workbench-core": "0.0.77",
-    "@workbench-ai/workbench-contract": "0.0.77",
-    "@workbench-ai/workbench-protocol": "0.0.77"
+    "@workbench-ai/workbench-built-in-adapters": "0.0.79",
+    "@workbench-ai/workbench-core": "0.0.79",
+    "@workbench-ai/workbench-contract": "0.0.79",
+    "@workbench-ai/workbench-protocol": "0.0.79"
   },
   "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.77"
+    "@workbench-ai/workbench-ui": "0.0.79"
   },
   "scripts": {
     "build": "rm -rf dist && tsc -p tsconfig.json && chmod 755 dist/workbench.js && node ./scripts/build-dev-open-assets.mjs",