lsd-pi 1.3.2 → 1.3.7

package/dist/resources/extensions/browser-tools/tools/state-persistence.js

@@ -1,180 +1,117 @@
 import { Type } from "@sinclair/typebox";
 /**
- * State persistence tools — save/restore cookies, localStorage, sessionStorage.
+ * State persistence — save/restore cookies, localStorage, sessionStorage.
  */
 const STATE_DIR = ".gsd/browser-state";
 export function registerStatePersistenceTools(pi, deps) {
-    // -------------------------------------------------------------------------
-    // browser_save_state
-    // -------------------------------------------------------------------------
     pi.registerTool({
-        name: "browser_save_state",
-        label: "Browser Save State",
-        description: "Save cookies, localStorage, and sessionStorage to disk so authenticated sessions survive browser restarts. " +
-            "State files are written to .gsd/browser-state/ and should be gitignored (may contain auth tokens). " +
-            "Never displays secret values in output.",
+        name: "browser_state",
+        label: "Browser State",
+        description: "Save or restore browser state (cookies, localStorage, sessionStorage) to persist sessions across browser restarts. " +
+            "State files written to .gsd/browser-state/ (should be gitignored).",
         parameters: Type.Object({
-            name: Type.Optional(Type.String({ description: "Name for the state file (default: 'default'). Used as the filename stem." })),
+            action: Type.Union([
+                Type.Literal("save"),
+                Type.Literal("restore"),
+            ], { description: "'save' — persist current state, 'restore' — load previously saved state" }),
+            name: Type.Optional(Type.String({ description: "State file name (default: 'default'). Used as filename stem." })),
         }),
         async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
             try {
-                const { context: ctx, page: p } = await deps.ensureBrowser();
-                const name = deps.sanitizeArtifactName(params.name ?? "default", "default");
-                const { mkdir, writeFile } = await import("node:fs/promises");
-                const path = await import("node:path");
-                const stateDir = path.resolve(process.cwd(), STATE_DIR);
-                await mkdir(stateDir, { recursive: true });
-                // 1. Playwright storageState: cookies + localStorage
-                const storageState = await ctx.storageState();
-                // 2. sessionStorage: must be extracted per-origin via page.evaluate
-                const sessionStorageData = {};
-                try {
-                    const origin = new URL(p.url()).origin;
-                    const ssData = await p.evaluate(() => {
-                        const data = {};
-                        for (let i = 0; i < sessionStorage.length; i++) {
-                            const key = sessionStorage.key(i);
-                            if (key)
-                                data[key] = sessionStorage.getItem(key) ?? "";
-                        }
-                        return data;
-                    });
-                    if (Object.keys(ssData).length > 0) {
-                        sessionStorageData[origin] = ssData;
-                    }
+                if (params.action === "save") {
+                    return await saveState(deps, params.name ?? "default");
                 }
-                catch {
-                    // Page may not have a valid origin (about:blank, etc.)
+                else {
+                    return await restoreState(deps, params.name ?? "default");
                 }
-                const combined = {
-                    storageState,
-                    sessionStorage: sessionStorageData,
-                    savedAt: new Date().toISOString(),
-                    url: p.url(),
-                };
-                const filePath = path.join(stateDir, `${name}.json`);
-                await writeFile(filePath, JSON.stringify(combined, null, 2));
-                // Ensure .gitignore covers the state dir
-                const gitignorePath = path.resolve(process.cwd(), STATE_DIR, ".gitignore");
-                await writeFile(gitignorePath, "*\n!.gitignore\n").catch(() => { });
-                const cookieCount = storageState.cookies?.length ?? 0;
-                const localStorageOrigins = storageState.origins?.length ?? 0;
-                const sessionStorageOrigins = Object.keys(sessionStorageData).length;
-                return {
-                    content: [{
-                            type: "text",
-                            text: `State saved: ${filePath}\nCookies: ${cookieCount}\nlocalStorage origins: ${localStorageOrigins}\nsessionStorage origins: ${sessionStorageOrigins}`,
-                        }],
-                    details: {
-                        path: filePath,
-                        cookieCount,
-                        localStorageOrigins,
-                        sessionStorageOrigins,
-                    },
-                };
             }
             catch (err) {
                 return {
-                    content: [{ type: "text", text: `Save state failed: ${err.message}` }],
+                    content: [{ type: "text", text: `State '${params.action}' failed: ${err.message}` }],
                     details: { error: err.message },
                     isError: true,
                 };
             }
         },
     });
-    // -------------------------------------------------------------------------
-    // browser_restore_state
-    // -------------------------------------------------------------------------
-    pi.registerTool({
-        name: "browser_restore_state",
-        label: "Browser Restore State",
-        description: "Restore cookies, localStorage, and sessionStorage from a previously saved state file. " +
-            "Injects cookies via context.addCookies() and storage via page.evaluate(). " +
-            "For full fidelity, restore before navigating to the target site.",
-        parameters: Type.Object({
-            name: Type.Optional(Type.String({ description: "Name of the state file to restore (default: 'default')." })),
-        }),
-        async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
-            try {
-                const { context: ctx, page: p } = await deps.ensureBrowser();
-                const name = deps.sanitizeArtifactName(params.name ?? "default", "default");
-                const { readFile } = await import("node:fs/promises");
-                const path = await import("node:path");
-                const filePath = path.join(process.cwd(), STATE_DIR, `${name}.json`);
-                let raw;
-                try {
-                    raw = await readFile(filePath, "utf-8");
-                }
-                catch {
-                    return {
-                        content: [{ type: "text", text: `State file not found: ${filePath}` }],
-                        details: { error: "file_not_found", path: filePath },
-                        isError: true,
-                    };
-                }
-                const combined = JSON.parse(raw);
-                const storageState = combined.storageState;
-                const sessionStorageData = combined.sessionStorage ?? {};
-                // 1. Restore cookies
-                let cookieCount = 0;
-                if (storageState?.cookies?.length) {
-                    await ctx.addCookies(storageState.cookies);
-                    cookieCount = storageState.cookies.length;
+    async function saveState(deps, name) {
+        const { context: ctx, page: p } = await deps.ensureBrowser();
+        name = deps.sanitizeArtifactName(name, "default");
+        const { mkdir, writeFile } = await import("node:fs/promises");
+        const path = await import("node:path");
+        const stateDir = path.resolve(process.cwd(), STATE_DIR);
+        await mkdir(stateDir, { recursive: true });
+        const storageState = await ctx.storageState();
+        const sessionStorageData = {};
+        try {
+            const origin = new URL(p.url()).origin;
+            const ssData = await p.evaluate(() => {
+                const data = {};
+                for (let i = 0; i < sessionStorage.length; i++) {
+                    const key = sessionStorage.key(i);
+                    if (key)
+                        data[key] = sessionStorage.getItem(key) ?? "";
                 }
-                // 2. Restore localStorage via page.evaluate
-                let localStorageOrigins = 0;
-                if (storageState?.origins?.length) {
-                    for (const origin of storageState.origins) {
-                        try {
-                            await p.evaluate((items) => {
-                                for (const { name, value } of items) {
-                                    localStorage.setItem(name, value);
-                                }
-                            }, origin.localStorage ?? []);
-                            localStorageOrigins++;
-                        }
-                        catch {
-                            // Origin mismatch — localStorage can only be set on matching origin
-                        }
-                    }
-                }
-                // 3. Restore sessionStorage via page.evaluate
-                let sessionStorageOrigins = 0;
-                for (const [_origin, data] of Object.entries(sessionStorageData)) {
-                    try {
-                        await p.evaluate((items) => {
-                            for (const [key, value] of Object.entries(items)) {
-                                sessionStorage.setItem(key, value);
-                            }
-                        }, data);
-                        sessionStorageOrigins++;
-                    }
-                    catch {
-                        // Origin mismatch
-                    }
+                return data;
+            });
+            if (Object.keys(ssData).length > 0)
+                sessionStorageData[origin] = ssData;
+        }
+        catch { /* Page may not have a valid origin */ }
+        const combined = { storageState, sessionStorage: sessionStorageData, savedAt: new Date().toISOString(), url: p.url() };
+        const filePath = path.join(stateDir, `${name}.json`);
+        await writeFile(filePath, JSON.stringify(combined, null, 2));
+        const gitignorePath = path.resolve(process.cwd(), STATE_DIR, ".gitignore");
+        await writeFile(gitignorePath, "*\n!.gitignore\n").catch(() => { });
+        return {
+            content: [{ type: "text", text: `State saved: ${filePath}\nCookies: ${storageState.cookies?.length ?? 0}\nlocalStorage origins: ${storageState.origins?.length ?? 0}\nsessionStorage origins: ${Object.keys(sessionStorageData).length}` }],
+            details: { path: filePath, cookieCount: storageState.cookies?.length ?? 0, localStorageOrigins: storageState.origins?.length ?? 0, sessionStorageOrigins: Object.keys(sessionStorageData).length },
+        };
+    }
+    async function restoreState(deps, name) {
+        const { context: ctx, page: p } = await deps.ensureBrowser();
+        name = deps.sanitizeArtifactName(name, "default");
+        const { readFile } = await import("node:fs/promises");
+        const path = await import("node:path");
+        const filePath = path.join(process.cwd(), STATE_DIR, `${name}.json`);
+        let raw;
+        try {
+            raw = await readFile(filePath, "utf-8");
+        }
+        catch {
+            return { content: [{ type: "text", text: `State file not found: ${filePath}` }], details: { error: "file_not_found", path: filePath }, isError: true };
+        }
+        const combined = JSON.parse(raw);
+        const storageState = combined.storageState;
+        const sessionStorageData = combined.sessionStorage ?? {};
+        let cookieCount = 0;
+        if (storageState?.cookies?.length) {
+            await ctx.addCookies(storageState.cookies);
+            cookieCount = storageState.cookies.length;
+        }
+        let localStorageOrigins = 0;
+        if (storageState?.origins?.length) {
+            for (const origin of storageState.origins) {
+                try {
+                    await p.evaluate((items) => { for (const { name, value } of items)
+                        localStorage.setItem(name, value); }, origin.localStorage ?? []);
+                    localStorageOrigins++;
                 }
-                return {
-                    content: [{
-                            type: "text",
-                            text: `State restored from: ${filePath}\nCookies: ${cookieCount}\nlocalStorage origins: ${localStorageOrigins}\nsessionStorage origins: ${sessionStorageOrigins}\nSaved at: ${combined.savedAt ?? "unknown"}`,
-                        }],
-                    details: {
-                        path: filePath,
-                        cookieCount,
-                        localStorageOrigins,
-                        sessionStorageOrigins,
-                        savedAt: combined.savedAt,
-                        savedUrl: combined.url,
-                    },
-                };
+                catch { /* Origin mismatch */ }
             }
-            catch (err) {
-                return {
-                    content: [{ type: "text", text: `Restore state failed: ${err.message}` }],
-                    details: { error: err.message },
-                    isError: true,
-                };
+        }
+        let sessionStorageOrigins = 0;
+        for (const [_origin, data] of Object.entries(sessionStorageData)) {
+            try {
+                await p.evaluate((items) => { for (const [key, value] of Object.entries(items))
+                    sessionStorage.setItem(key, value); }, data);
+                sessionStorageOrigins++;
             }
-        },
-    });
+            catch { /* Origin mismatch */ }
+        }
+        return {
+            content: [{ type: "text", text: `State restored from: ${filePath}\nCookies: ${cookieCount}\nlocalStorage origins: ${localStorageOrigins}\nsessionStorage origins: ${sessionStorageOrigins}` }],
+            details: { path: filePath, cookieCount, localStorageOrigins, sessionStorageOrigins, savedAt: combined.savedAt, savedUrl: combined.url },
+        };
+    }
 }

package/dist/resources/extensions/browser-tools/utils.js

@@ -463,7 +463,7 @@ export function formatVersionedRef(version, key) {
     return `@v${version}:${key}`;
 }
 export function staleRefGuidance(refDisplay, reason) {
-    return `Ref ${refDisplay} could not be resolved (${reason}). The ref is likely stale after DOM/navigation changes. Call browser_snapshot_refs again to refresh refs.`;
+    return `Ref ${refDisplay} could not be resolved (${reason}). The ref is likely stale after DOM/navigation changes. Call browser_ref with action='snapshot' to refresh refs.`;
 }
 // ---------------------------------------------------------------------------
 // Compact state summary formatting

package/dist/resources/extensions/cache-timer/index.js

@@ -18,6 +18,7 @@ const IS_MEMORY_MAINTENANCE_WORKER = process.env.LSD_MEMORY_EXTRACT === "1" || p
 const IS_CACHE_TIMER_FORCED_OFF = process.env.LSD_DISABLE_CACHE_TIMER === "1" || process.env.GSD_DISABLE_CACHE_TIMER === "1";
 // ANSI color codes for timer display
 const ANSI_RESET = "\x1b[0m";
+const ANSI_GREEN = "\x1b[32m";
 const ANSI_YELLOW = "\x1b[33m";
 const ANSI_RED = "\x1b[31m";
 function getSettingsPath() {
@@ -63,8 +64,8 @@ function formatElapsed(ms) {
         // 5–10 minutes: yellow
         return `${ANSI_YELLOW}⏱ ${time}${ANSI_RESET}`;
     }
-    // Under 5 minutes: plain (inherits footer dim styling)
-    return `⏱ ${time}`;
+    // Under 5 minutes: green
+    return `${ANSI_GREEN}⏱ ${time}${ANSI_RESET}`;
 }
 export default function cacheTimerExtension(pi) {
     if (IS_MEMORY_MAINTENANCE_WORKER || IS_CACHE_TIMER_FORCED_OFF) {

package/dist/resources/extensions/slash-commands/extension-manifest.json

@@ -2,11 +2,11 @@
 	"id": "slash-commands",
 	"name": "Slash Commands",
 	"version": "1.0.0",
-	"description": "Bundled slash commands for context inspection, planning, and lazy tool search",
+	"description": "Bundled slash commands for context inspection, planning, fast mode, and lazy tool search",
 	"tier": "bundled",
 	"requires": { "platform": ">=2.29.0" },
 	"provides": {
-		"commands": ["audit", "clear", "context", "plan", "execute", "cancel-plan", "tools"],
+		"commands": ["audit", "clear", "context", "fast", "plan", "execute", "cancel-plan", "tools"],
 		"tools": ["tool_search", "tool_enable"],
 		"flags": ["plan"]
 	}

package/dist/resources/extensions/slash-commands/fast.js

@@ -0,0 +1,73 @@
+import { getAgentDir, SettingsManager } from "@gsd/pi-coding-agent";
+function parseFastCommandAction(args) {
+    const normalized = args.trim().toLowerCase();
+    if (!normalized)
+        return "toggle";
+    if (normalized === "on")
+        return "on";
+    if (normalized === "off")
+        return "off";
+    if (normalized === "status")
+        return "status";
+    return "invalid";
+}
+function supportsFastMode(model) {
+    if (!model)
+        return false;
+    if (model.api === "openai-codex-responses")
+        return true;
+    if (model.api !== "openai-responses")
+        return false;
+    if (model.provider !== "openai")
+        return false;
+    return model.capabilities?.supportsServiceTier === true;
+}
+function getModelLabel(model) {
+    if (!model)
+        return "no active model";
+    return `${model.provider}/${model.id}`;
+}
+function getSettingsManager() {
+    return SettingsManager.create(process.cwd(), getAgentDir());
+}
+export const __testing = {
+    parseFastCommandAction,
+    supportsFastMode,
+};
+export default function fastCommand(pi) {
+    pi.registerCommand("fast", {
+        description: "Toggle fast mode for OpenAI/Codex models (service_tier=priority)",
+        getArgumentCompletions(prefix) {
+            const options = [
+                { value: "on", label: "on", description: "Enable fast mode" },
+                { value: "off", label: "off", description: "Disable fast mode" },
+                { value: "status", label: "status", description: "Show current fast-mode status" },
+            ];
+            const query = prefix.trim().toLowerCase();
+            return options.filter((item) => item.value.startsWith(query));
+        },
+        async handler(args, ctx) {
+            const settings = getSettingsManager();
+            const current = settings.getFastMode();
+            const action = parseFastCommandAction(args);
+            const model = ctx.model;
+            const supported = supportsFastMode(model);
+            const modelLabel = getModelLabel(model);
+            if (action === "invalid") {
+                ctx.ui.notify("Usage: /fast [on|off|status]", "warning");
+                return;
+            }
+            if (action === "status") {
+                ctx.ui.notify(`Fast mode: ${current ? "ON" : "OFF"} · model ${modelLabel} is ${supported ? "supported" : "unsupported"}`, "info");
+                return;
+            }
+            const next = action === "toggle" ? !current : action === "on";
+            settings.setFastMode(next);
+            if (!supported) {
+                ctx.ui.notify(`Fast mode: ${next ? "ON" : "OFF"} (saved). Current model ${modelLabel} does not support fast mode.`, "warning");
+                return;
+            }
+            ctx.ui.notify(`Fast mode: ${next ? "ON" : "OFF"} (saved). ${next ? "Requests will include service_tier=priority." : "Requests will omit service_tier."}`, "info");
+        },
+    });
+}

package/dist/resources/extensions/slash-commands/index.js

@@ -1,6 +1,7 @@
 import auditCommand from "./audit.js";
 import clearCommand from "./clear.js";
 import contextCommand from "./context.js";
+import fastCommand from "./fast.js";
 import initCommand from "./init.js";
 import planCommand from "./plan.js";
 import toolSearchExtension from "./tools.js";
@@ -8,6 +9,7 @@ export default function slashCommands(pi) {
     auditCommand(pi);
     clearCommand(pi);
     contextCommand(pi);
+    fastCommand(pi);
     initCommand(pi);
     planCommand(pi);
     toolSearchExtension(pi);

package/dist/resources/extensions/slash-commands/plan.js

@@ -28,9 +28,8 @@ const BLOCKED_TOOLS = new Set([
     "async_bash",
     "bg_shell",
     "browser_navigate",
-    "browser_go_back",
-    "browser_go_forward",
-    "browser_reload",
+    "browser_pages",
+    "browser_frames",
     "browser_click",
     "browser_drag",
     "browser_type",
@@ -41,18 +40,13 @@ const BLOCKED_TOOLS = new Set([
     "browser_select_option",
     "browser_set_checked",
     "browser_set_viewport",
-    "browser_click_ref",
-    "browser_hover_ref",
-    "browser_fill_ref",
+    "browser_ref",
     "browser_act",
     "browser_batch",
     "browser_fill_form",
-    "browser_mock_route",
-    "browser_block_urls",
-    "browser_clear_routes",
+    "browser_network",
     "browser_emulate_device",
-    "browser_save_state",
-    "browser_restore_state",
+    "browser_state",
     "browser_generate_test",
     "browser_verify",
     "write",
@@ -70,6 +64,7 @@ const REVISE_LABEL = "Revise plan";
 const CANCEL_LABEL = "Cancel";
 const DEFAULT_PLAN_REVIEW_AGENT = "generic";
 const DEFAULT_PLAN_CODING_AGENT = "worker";
+const MIN_PLAN_CONFIDENCE = 8;
 const INITIAL_STATE = {
     active: false,
     task: "",
@@ -366,6 +361,9 @@ function buildPlanModeSystemPrompt() {
         "You are currently in plan mode.",
         "Investigate, clarify scope, and produce a persisted execution plan before making source changes.",
         "If requirements are ambiguous or constraints are missing, ask concise clarifying questions before drafting or saving a plan.",
+        `Before writing or updating a plan artifact, make sure your confidence is at least ${MIN_PLAN_CONFIDENCE}/10. If confidence is lower, investigate more or ask clarifying questions first.`,
+        "Include an explicit confidence line in every saved plan, for example: \"Confidence: 8/10\" or higher.",
+        "When adjusting an existing saved plan, prefer the edit tool for targeted changes. Rewrite the whole file only when the structure changes substantially or an exact edit is impractical.",
         "Do not modify source files or run side-effect commands while plan mode is active.",
         "Persist plan artifacts under .lsd/plan/.",
     ];
@@ -400,7 +398,7 @@ function buildApprovalActionInstructions() {
     const newSessionLabel = buildNewSessionOptionLabel();
     return [
         "Ask for plan approval now via exactly one ask_user_questions tool call.",
-        `Question 1 (single-select) id \"${PLAN_APPROVAL_ACTION_QUESTION_ID}\": ask what to do next with the plan.`,
+        `Question 1 (single-select) id \"${PLAN_APPROVAL_ACTION_QUESTION_ID}\": ask what to do next with the plan. In the question text, tell the user that if they choose \"${REVISE_LABEL}\" they should type exact requested changes in the dialog notes field before submitting.`,
         `Question 1 options: ${APPROVE_LABEL}, ${REVIEW_LABEL}, ${REVISE_LABEL}. Put "${APPROVE_LABEL}" first with a "(Recommended)" suffix in the description, not in the label.`,
         `Do not include \"${CANCEL_LABEL}\" as an explicit option — if the user wants to cancel they should choose \"None of the above\" and type \"${CANCEL_LABEL}\" in the note.`,
         `Question 2 (single-select) id \"${PLAN_APPROVAL_PERMISSION_QUESTION_ID}\": ask which execution mode to use.`,
@@ -454,6 +452,23 @@ function buildReviewSteeringMessage(planPath, planMarkdown) {
     details.push("After the subagent responds, summarize its feedback for the user, present the current plan again, and then ask for approval again.", buildApprovalActionInstructions());
     return details.join("\n\n");
 }
+function buildRevisionSteeringMessage(planPath, requestedChanges) {
+    const details = [
+        `The user selected \"${REVISE_LABEL}\" for ${planPath}.`,
+        "Revise the existing saved plan instead of drafting a fresh replacement unless a full rewrite is genuinely necessary.",
+        "Prefer the edit tool for targeted adjustments to the current plan artifact. Use write only if the structure changes substantially or an exact edit is impractical.",
+        `Before saving the revised plan, make sure confidence is at least ${MIN_PLAN_CONFIDENCE}/10. If lower, investigate more or ask clarifying questions first.`,
+        "Keep an explicit confidence line in the plan, for example: \"Confidence: 8/10\" or higher.",
+    ];
+    if (requestedChanges) {
+        details.push(`User-requested changes: ${requestedChanges}`);
+    }
+    else {
+        details.push("No concrete revision note was provided yet. Ask one concise clarifying question about what should change before editing the plan.");
+    }
+    details.push("After revising the saved plan artifact, ask for approval again.");
+    return details.join("\n\n");
+}
 function approvalSelectionToExecutionMode(selected) {
     if (!selected)
         return undefined;
@@ -482,6 +497,12 @@ function getAnswerValues(answer) {
     }
     return values;
 }
+function getAnswerNote(answer) {
+    if (typeof answer?.notes !== "string")
+        return undefined;
+    const note = answer.notes.trim();
+    return note.length > 0 ? note : undefined;
+}
 function selectionRequestsCancel(selected) {
     return selected.some((value) => {
         if (typeof value !== "string")
@@ -506,6 +527,8 @@ export const __testing = {
     buildApprovalSteeringMessage,
     buildPlanPreviewMessage,
     buildReviewSteeringMessage,
+    buildRevisionSteeringMessage,
+    buildPlanModeSystemPrompt,
     buildAutoSuggestPlanModeSystemPrompt,
     readAutoSuggestPlanModeSetting,
     PLAN_SUGGEST_QUESTION_ID,
@@ -696,9 +719,11 @@ export default function planCommand(pi) {
             return;
         }
         if (actionSelection.includes(REVISE_LABEL)) {
+            const requestedChanges = getAnswerNote(actionAnswer);
             enablePlanMode(pi, ctx.model ? { provider: ctx.model.provider, id: ctx.model.id } : undefined, {
                 approvalStatus: "revising",
             });
+            pi.sendUserMessage(buildRevisionSteeringMessage(state.latestPlanPath ?? "the latest plan", requestedChanges), { deliverAs: "steer" });
         }
     });
     pi.registerCommand("plan", {

package/dist/resources/extensions/subagent/background-job-manager.js

@@ -38,6 +38,19 @@ export class BackgroundJobManager {
     adoptRunning(agentName, task, cwd, abortController, resultPromise, metadata) {
         return this.attachJob(agentName, task, cwd, abortController, resultPromise, metadata);
     }
+    /**
+     * Adopt an in-process subagent handle into background tracking.
+     */
+    adoptHandle(agentName, task, cwd, handle, resultPromise, metadata) {
+        const abortController = new AbortController();
+        const onAbort = () => handle.abort();
+        abortController.signal.addEventListener("abort", onAbort, { once: true });
+        const wrapped = resultPromise.finally(() => {
+            abortController.signal.removeEventListener("abort", onAbort);
+            handle.dispose();
+        });
+        return this.attachJob(agentName, task, cwd, abortController, wrapped, metadata);
+    }
     /**
      * Cancel a running job.
      */