@workbench-ai/workbench 0.0.86 → 0.0.88

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAkEA,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,CAqMlB"}
+{"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,CA2MlB"}

package/dist/index.js

@@ -7,7 +7,6 @@ import { gzipSync } from "node:zlib";
 import { addWorkbenchRemote, addWorkbenchAgent, compareWorkbench, createWorkbenchVersionRuntimeSnapshot, createWorkbenchInspectionSnapshot, createWorkbenchAdapterAuthBundle, createWorkbenchReadOnlyInspectionSnapshot, diffWorkbenchVersions, evalWorkbenchSkill, improveWorkbenchSkill, initWorkbenchSkill, listWorkbenchAgents, listWorkbenchVersions, localWorkbenchAdapterAuthStore, parseWorkbenchAdapterAuthTarget, prepareWorkbenchCloudEvalRequest, prepareWorkbenchCloudImproveRequest, publishWorkbenchVersion, removeWorkbenchAgent, showWorkbenchRef, switchWorkbenchVersion, syncWorkbenchRemote, workbenchJobEvidenceForSnapshot, workbenchStatusSnapshot, WorkbenchCodedError, WorkbenchUserError, } from "@workbench-ai/workbench-core";
 import { normalizeWorkbenchSkillName } from "@workbench-ai/workbench-contract";
 import { emitError, emitResult } from "./output.js";
-import { fanOutSkill, manualFanOutCommand } from "./fanout.js";
 import { installedInventoryToJson, installSnapshotToStore, normalizeInstallSnapshotPath, readInstalledSkillsInventory, } from "./install-targets.js";
 import { startWorkbenchOpenServer } from "./open-server.js";
 const require = createRequire(import.meta.url);
@@ -449,14 +448,21 @@ export async function runCli(argv, io = {
                     `next: workbench install ${preview.installHandle}`,
                 ].join("\n"));
             }
-            const remote = await ensurePublishRemote(parsed);
-            const result = await publishWorkbenchVersion({
-                ...core,
-                version: optionalPositional(parsed, 1),
-                remote,
-                dryRun: parsed.flags["dry-run"] === true,
-                visibility: parsePublishVisibilityFlags(parsed),
-            });
+            let remote;
+            let result;
+            try {
+                remote = await ensurePublishRemote(parsed);
+                result = await publishWorkbenchVersion({
+                    ...core,
+                    version: optionalPositional(parsed, 1),
+                    remote,
+                    dryRun: parsed.flags["dry-run"] === true,
+                    visibility: parsePublishVisibilityFlags(parsed),
+                });
+            }
+            catch (error) {
+                throw await publishErrorWithCliContext(error, parsed, remote);
+            }
             return emitResult("workbench.cli.publish.v1", {
                 remote: result.remote,
                 version: versionSummary(result.version),
@@ -907,10 +913,6 @@ async function handleInstall(parsed, io) {
             baseUrl: workbenchSource.baseUrl,
         },
     });
-    const fanout = parsed.flags["dry-run"] === true
-        ? skippedFanOut(result.directoryName, result.destination, result.result === "unchanged" ? "unchanged" : "dry-run")
-        : await fanOutSkill(result.directoryName, { skillDir: result.destination });
-    const next = installNextCommand(fanout);
     return emitResult("workbench.cli.install.v1", {
         source: sourceSummary,
         result: result.result,
@@ -921,14 +923,9 @@ async function handleInstall(parsed, io) {
         filesCopied: result.filesCopied,
         contentHash: result.contentHash,
         provenancePath: result.provenancePath,
-        fanout: fanOutToJson(fanout),
-        next: next,
+        next: null,
         ...(parsed.flags["dry-run"] === true ? { dryRun: true } : {}),
-    }, parsed, io, () => [
-        formatInstallOutcome(result, parsed.flags["dry-run"] === true),
-        formatFanOut(fanout),
-        ...(next ? [`next: ${next}`] : []),
-    ].join("\n"));
+    }, parsed, io, () => formatInstallOutcome(result, parsed.flags["dry-run"] === true));
 }
 async function handleCloudEval(parsed, io) {
     const started = await startCloudExecution("eval", parsed, io);
@@ -1008,19 +1005,6 @@ async function handleCloudImprove(parsed, io) {
         ...(next ? [`next: ${next}`] : []),
     ].filter(Boolean).join("\n"));
 }
-function skippedFanOut(name, destination, reason) {
-    return {
-        status: "skipped",
-        command: manualFanOutCommand(destination, name),
-        linkedAgents: [],
-        reason,
-    };
-}
-function installNextCommand(fanout) {
-    return fanout.status === "failed" || (fanout.status === "skipped" && fanout.reason !== "dry-run" && fanout.reason !== "unchanged")
-        ? fanout.command
-        : null;
-}
 function formatInstallOutcome(result, dryRun) {
     if (dryRun) {
         if (result.result === "unchanged") {
@@ -1048,34 +1032,6 @@ function formatInstallOutcome(result, dryRun) {
 function formatFileCount(count) {
     return `${count} ${count === 1 ? "file" : "files"}`;
 }
-function fanOutToJson(fanout) {
-    return {
-        status: fanout.status,
-        command: fanout.command,
-        linkedAgents: fanout.linkedAgents,
-        ...(fanout.additionalAgents ? { additionalAgents: fanout.additionalAgents } : {}),
-        ...(fanout.reason ? { reason: fanout.reason } : {}),
-        ...(fanout.exitCode !== undefined ? { exitCode: fanout.exitCode } : {}),
-    };
-}
-function formatFanOut(fanout) {
-    if (fanout.status === "skipped") {
-        if (fanout.reason === "dry-run") {
-            return "fanout: planned";
-        }
-        if (fanout.reason === "unchanged") {
-            return "fanout: unchanged";
-        }
-        return `fanout skipped: ${fanout.reason ?? "not available"}`;
-    }
-    if (fanout.status === "failed") {
-        return `fanout failed: ${fanout.reason ?? "unknown failure"}`;
-    }
-    if (fanout.linkedAgents.length === 0) {
-        return "fanout: completed";
-    }
-    return "fanout: completed";
-}
 async function latestInstallVersion(record) {
     const handle = normalizedOwnerSkillHandle(record.handle);
     if (!handle) {
@@ -2689,6 +2645,33 @@ function installHandleFromCloudRemote(remote) {
     }
     return `${source.owner}/${source.skill}`;
 }
+async function publishErrorWithCliContext(error, parsed, remoteName) {
+    if (!(error instanceof WorkbenchCodedError) || error.code !== "auth_required") {
+        return error;
+    }
+    if (error.message.startsWith("workbench publish")) {
+        return error;
+    }
+    return new WorkbenchCodedError("auth_required", "workbench publish requires Workbench Cloud auth.", {
+        remediation: await publishAuthRemediation(parsed, remoteName, error.remediation),
+        ...(error.subject ? { subject: error.subject } : {}),
+        exitCode: error.exitCode,
+    });
+}
+async function publishAuthRemediation(parsed, remoteName, fallback) {
+    const root = path.resolve(dirFlag(parsed) ?? process.cwd());
+    try {
+        const snapshot = await createWorkbenchReadOnlyInspectionSnapshot({ dir: root });
+        const remote = remoteName
+            ? snapshot.remotes.find((entry) => entry.name === remoteName)
+            : cloudRemoteLinkTargetFromRemotes(snapshot.remotes).existing;
+        const source = remote ? parseWorkbenchInstallSource(remote.url) : undefined;
+        return workbenchLoginRemediation(source?.baseUrl);
+    }
+    catch {
+        return fallback ?? "Run workbench login.";
+    }
+}
 function parseOwnerSkillHandle(input) {
     const handle = normalizedOwnerSkillHandle(input);
     if (!handle) {

package/package.json

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

package/dist/fanout.d.ts

@@ -1,13 +0,0 @@
-export interface FanOutResult {
-    status: "completed" | "skipped" | "failed";
-    command: string;
-    linkedAgents: string[];
-    additionalAgents?: number;
-    reason?: string;
-    exitCode?: number | null;
-}
-export declare function fanOutSkill(name: string, options: {
-    skillDir: string;
-}): Promise<FanOutResult>;
-export declare function manualFanOutCommand(skillDir: string, name: string): string;
-//# sourceMappingURL=fanout.d.ts.map

package/dist/fanout.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"fanout.d.ts","sourceRoot":"","sources":["../src/fanout.ts"],"names":[],"mappings":"AASA,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,WAAW,GAAG,SAAS,GAAG,QAAQ,CAAC;IAC3C,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC1B;AAED,wBAAsB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE;IAAE,QAAQ,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,YAAY,CAAC,CAoEpG;AAED,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAO1E"}

package/dist/fanout.js

@@ -1,223 +0,0 @@
-import { spawn } from "node:child_process";
-import { promises as fs } from "node:fs";
-import { createRequire } from "node:module";
-import os from "node:os";
-import path from "node:path";
-const require = createRequire(import.meta.url);
-const FANOUT_TIMEOUT_MS = 30_000;
-export async function fanOutSkill(name, options) {
-    const resolved = await resolveSkillsBin().catch((error) => ({
-        binPath: null,
-        reason: error instanceof Error ? error.message : "Unable to resolve skills package.",
-    }));
-    const command = manualFanOutCommand(options.skillDir, name);
-    if (!resolved.binPath) {
-        return {
-            status: "skipped",
-            command,
-            linkedAgents: [],
-            reason: resolved.reason,
-        };
-    }
-    const stagingRoot = await fs.mkdtemp(path.join(os.tmpdir(), "workbench-skills-fanout-"));
-    const stagedSkillDir = path.join(stagingRoot, name);
-    try {
-        await fs.cp(options.skillDir, stagedSkillDir, {
-            recursive: true,
-            force: true,
-            dereference: true,
-        });
-        const args = [
-            resolved.binPath,
-            "add",
-            stagedSkillDir,
-            "--global",
-            "--yes",
-        ];
-        const child = await runNode(args, minimalChildEnv(process.env), FANOUT_TIMEOUT_MS);
-        const { linkedAgents, additionalAgents } = parseFanOutAgents(`${child.stdout}\n${child.stderr}`);
-        if (child.timedOut) {
-            return {
-                status: "failed",
-                command,
-                linkedAgents,
-                ...(additionalAgents > 0 ? { additionalAgents } : {}),
-                reason: `skills fan-out timed out after ${FANOUT_TIMEOUT_MS}ms.`,
-                exitCode: child.exitCode,
-            };
-        }
-        if (child.exitCode !== 0) {
-            return {
-                status: "failed",
-                command,
-                linkedAgents,
-                ...(additionalAgents > 0 ? { additionalAgents } : {}),
-                reason: conciseFailureReason(`${child.stdout}\n${child.stderr}`) ?? `skills fan-out exited ${child.exitCode}.`,
-                exitCode: child.exitCode,
-            };
-        }
-        return {
-            status: "completed",
-            command,
-            linkedAgents,
-            ...(additionalAgents > 0 ? { additionalAgents } : {}),
-            exitCode: child.exitCode,
-        };
-    }
-    catch (error) {
-        return {
-            status: "failed",
-            command,
-            linkedAgents: [],
-            reason: error instanceof Error ? error.message : "skills fan-out failed.",
-        };
-    }
-    finally {
-        await fs.rm(stagingRoot, { recursive: true, force: true });
-    }
-}
-export function manualFanOutCommand(skillDir, name) {
-    const quotedName = quoteShellArg(name);
-    return [
-        "tmpdir=$(mktemp -d)",
-        `cp -R ${quoteShellArg(skillDir)} "$tmpdir"/${quotedName}`,
-        `skills add "$tmpdir"/${quotedName} --global --yes`,
-    ].join(" && ");
-}
-function quoteShellArg(value) {
-    return /^[A-Za-z0-9_./:=@+-]+$/u.test(value)
-        ? value
-        : `'${value.replace(/'/gu, "'\\''")}'`;
-}
-async function resolveSkillsBin() {
-    const packageJsonPath = require.resolve("skills/package.json");
-    const packageRoot = path.dirname(packageJsonPath);
-    const packageJson = JSON.parse(await fs.readFile(packageJsonPath, "utf8"));
-    const bin = typeof packageJson.bin === "string" ? packageJson.bin : packageJson.bin?.skills;
-    if (!bin) {
-        throw new Error("The skills package does not declare a skills bin.");
-    }
-    return {
-        binPath: path.resolve(packageRoot, bin),
-    };
-}
-function runNode(args, env, timeoutMs) {
-    return new Promise((resolve) => {
-        const child = spawn(process.execPath, args, {
-            env,
-            stdio: ["ignore", "pipe", "pipe"],
-            windowsHide: true,
-        });
-        let stdout = "";
-        let stderr = "";
-        let timedOut = false;
-        const timer = setTimeout(() => {
-            timedOut = true;
-            child.kill("SIGTERM");
-        }, timeoutMs);
-        child.stdout?.setEncoding("utf8");
-        child.stderr?.setEncoding("utf8");
-        child.stdout?.on("data", (chunk) => {
-            stdout += chunk;
-        });
-        child.stderr?.on("data", (chunk) => {
-            stderr += chunk;
-        });
-        child.on("error", (error) => {
-            clearTimeout(timer);
-            resolve({
-                exitCode: 1,
-                stdout,
-                stderr: stderr || error.message,
-                timedOut,
-            });
-        });
-        child.on("close", (exitCode) => {
-            clearTimeout(timer);
-            resolve({
-                exitCode,
-                stdout,
-                stderr,
-                timedOut,
-            });
-        });
-    });
-}
-// The skills CLI changes behavior when it detects it is running inside an
-// agent (via @vercel/detect-agent reading agent-specific environment
-// variables). Fan-out must always run in machine mode, so the child gets a
-// minimal allowlisted environment instead of a hand-maintained blacklist of
-// every agent's marker variables.
-function minimalChildEnv(env) {
-    const allowed = [
-        "PATH",
-        "HOME",
-        "USER",
-        "SHELL",
-        "TMPDIR",
-        "TMP",
-        "TEMP",
-        "LANG",
-        "NODE_EXTRA_CA_CERTS",
-        "HTTP_PROXY",
-        "HTTPS_PROXY",
-        "NO_PROXY",
-        "http_proxy",
-        "https_proxy",
-        "no_proxy",
-        "SYSTEMROOT",
-        "SystemRoot",
-        "COMSPEC",
-        "APPDATA",
-        "LOCALAPPDATA",
-        "USERPROFILE",
-    ];
-    const next = {};
-    for (const key of Object.keys(env)) {
-        if (allowed.includes(key) || key.startsWith("XDG_") || key.startsWith("LC_")) {
-            next[key] = env[key];
-        }
-    }
-    return next;
-}
-function parseFanOutAgents(output) {
-    const cleaned = stripAnsi(output);
-    const linked = new Set();
-    let additionalAgents = 0;
-    for (const line of cleaned.split(/\r?\n/u)) {
-        const normalized = line.replace(/[\u2502\u25c7\u25cf\u25a0\u2713\u2717]/gu, " ").trim();
-        const match = /^(?:universal|symlinked|copied|copy\s*\u2192|symlink\s*\u2192):\s*(.+)$/u.exec(normalized);
-        if (!match) {
-            continue;
-        }
-        for (const rawAgent of match[1].split(",").map((entry) => entry.trim())) {
-            const moreMatch = /\+(\d+) more$/u.exec(rawAgent);
-            if (moreMatch) {
-                additionalAgents += Number(moreMatch[1]);
-            }
-            const agent = rawAgent.replace(/\s*\+\d+ more$/u, "").trim();
-            if (agent) {
-                linked.add(agent);
-            }
-        }
-    }
-    return {
-        linkedAgents: [...linked].sort((left, right) => left.localeCompare(right)),
-        additionalAgents,
-    };
-}
-function conciseFailureReason(output) {
-    const lines = stripAnsi(output)
-        .split(/\r?\n/u)
-        .map((line) => line.replace(/[\u2500-\u257f\u25a0\u25cf\u25c7\u2713\u2717]/gu, " ").trim())
-        .filter((line) => line.length > 0 &&
-        !/^skills\s*$/iu.test(line) &&
-        !/^Tip:/u.test(line) &&
-        !/^(Source|Local path|Found \d+ skill|Available skills|Installed \d+ skill|Done!|Review skills)/u.test(line) &&
-        !/^- /u.test(line));
-    const reason = lines.at(-1);
-    return reason ? (reason.length > 200 ? `${reason.slice(0, 197)}...` : reason) : null;
-}
-function stripAnsi(value) {
-    return value.replace(/\x1B\[[0-?]*[ -/]*[@-~]/gu, "");
-}