@optima-chat/browser-cli 0.15.0 → 0.15.1

package/dist/index.js

@@ -436,9 +436,13 @@ function clearSessionId(onlyIfEquals) {
 }
 
 // src/formatter.ts
+var URL_STALE_WARNING = "\u26A0 URL/title may be stale: the live page probe failed, values above are cached. Verify with a screenshot before concluding a navigation failed.";
 function formatPageState(state) {
   const lines = [];
   lines.push(`Page: ${state.title} (${state.url})`);
+  if (state.url_stale) {
+    lines.push(URL_STALE_WARNING);
+  }
   lines.push("");
   lines.push("DOM:");
   lines.push(state.dom);
@@ -478,6 +482,9 @@ function formatTabs(tabList) {
   for (const tab of tabList.tabs) {
     lines.push(`  [${tab.tab_id}] ${tab.title} (${tab.url})`);
   }
+  if (tabList.url_stale) {
+    lines.push(URL_STALE_WARNING);
+  }
   return lines.join("\n");
 }
 function formatSearch(result) {
@@ -764,8 +771,9 @@ function registerTabCommands(program2) {
 }
 
 // src/commands/script.ts
-import { mkdirSync as mkdirSync2, readFileSync as readFileSync3, writeFileSync as writeFileSync3 } from "fs";
-import { basename, dirname, resolve } from "path";
+import { existsSync as existsSync3, mkdirSync as mkdirSync2, readFileSync as readFileSync3, statSync, writeFileSync as writeFileSync3 } from "fs";
+import { tmpdir } from "os";
+import { basename, dirname, join as join3, resolve } from "path";
 var POLL_INTERVAL_MS = 2e3;
 var POLL_TIMEOUT_MS = 18e4;
 var REQUIRED_ARG_FLAG = {
@@ -778,16 +786,50 @@ function formatRequiredArgs(s) {
   const flags = s.required_args.map((a) => REQUIRED_ARG_FLAG[a] ?? `--param ${a}=<value>`).join(" ");
   return `  requires: ${flags}`;
 }
+var ArtifactLostError = class extends Error {
+};
+var errMsg = (err) => err instanceof Error ? err.message : String(err);
+function resolveArtifactTarget(out, name) {
+  if (!out) return resolve(`./${name}`);
+  const wantsDir = /[\\/]$/.test(out) || existsSync3(out) && statSync(out).isDirectory();
+  return wantsDir ? resolve(out, name) : resolve(out);
+}
 async function deliverArtifact(task, out) {
   const output = task.output;
   if (!output) return null;
-  if (output.path) return output.path;
+  if (output.path) return { path: output.path, size: output.size };
   if (!output.artifact_path) return null;
-  const target = resolve(out ?? `./${basename(output.filename ?? "artifact")}`);
-  const content = await api.fetchArtifact(output.artifact_path);
+  const name = basename(output.filename ?? "") || "artifact";
+  const target = resolveArtifactTarget(out, name);
   mkdirSync2(dirname(target), { recursive: true });
-  writeFileSync3(target, content);
-  return target;
+  const content = await api.fetchArtifact(output.artifact_path);
+  try {
+    writeFileSync3(target, content);
+  } catch (err) {
+    const fallback = join3(tmpdir(), name);
+    try {
+      writeFileSync3(fallback, content);
+    } catch (err2) {
+      throw new ArtifactLostError(
+        `Could not write ${target} (${errMsg(err)}) nor ${fallback} (${errMsg(err2)}); the artifact was already fetched and is no longer on the server`
+      );
+    }
+    console.error(`Could not write ${target} (${errMsg(err)}); saved to ${fallback} instead`);
+    return { path: fallback, size: content.length };
+  }
+  return { path: target, size: content.length };
+}
+function artifactUrl(artifactPath) {
+  return `${BROWSER_API_URL}${artifactPath}`;
+}
+function hasStagedArtifact(task) {
+  const output = task.output;
+  return task.status === "completed" && !!output?.artifact_path;
+}
+function downloadHint(task) {
+  const filename = task.output?.filename;
+  const dest = filename ? ` ./${basename(filename)}` : "";
+  return `Download with: browser-cli script-task ${task.id} --download${dest}`;
 }
 function formatScriptList(scripts) {
   if (scripts.length === 0) return "No scripts registered.";
@@ -810,7 +852,7 @@ function formatScriptTask(task) {
     lines.push("\u2713 Script completed");
     if (task.output) {
       for (const [k, v] of Object.entries(task.output)) {
-        lines.push(`${k}: ${v}`);
+        lines.push(k === "artifact_path" ? `${k}: ${artifactUrl(String(v))}` : `${k}: ${v}`);
       }
     }
   } else if (task.status === "needs_fallback") {
@@ -921,12 +963,15 @@ function registerScriptCommands(program2) {
       if (result.status === "completed") {
         try {
           const saved = await deliverArtifact(result, opts.out);
-          if (saved) console.log(`Saved -> ${saved}`);
+          if (saved) console.log(`Saved -> ${saved.path}`);
         } catch (err) {
           const staged = result.output?.artifact_path;
           console.error(`${formatError(err)}
 Script completed; only saving the file failed.`);
-          if (staged) console.error(`The artifact is still at ${staged}.`);
+          if (staged && !(err instanceof ArtifactLostError)) {
+            console.error(`The artifact is still at ${artifactUrl(staged)}.
+${downloadHint(result)}`);
+          }
           process.exit(1);
         }
       }
@@ -935,12 +980,26 @@ Script completed; only saving the file failed.`);
       process.exit(1);
     }
   });
-  program2.command("script-task").argument("<task-id>", "Task ID").description("Check script task status").action(async (taskId) => {
+  program2.command("script-task").argument("<task-id>", "Task ID").description("Check script task status").option("--download [path]", "Save the task's exported file (default: ./<filename>)").action(async (taskId, opts) => {
     try {
       const task = await api.getScriptTask(taskId);
-      console.log(formatScriptTask(task));
+      if (opts.download === void 0) {
+        console.log(formatScriptTask(task));
+        if (hasStagedArtifact(task)) console.log(downloadHint(task));
+        return;
+      }
+      if (task.status !== "completed") {
+        throw new Error(`Task ${taskId} is ${task.status} \u2014 nothing to download yet`);
+      }
+      const saved = await deliverArtifact(task, typeof opts.download === "string" ? opts.download : void 0);
+      if (!saved) throw new Error(`Task ${taskId} produced no file`);
+      console.log(`Saved -> ${saved.path}${saved.size === void 0 ? "" : ` (${saved.size} bytes)`}`);
     } catch (err) {
-      console.error(formatError(err));
+      const gone = err.errorCode === "artifact_gone";
+      console.error(
+        gone ? `${formatError(err)}
+Each artifact can be fetched once; if runscript printed "Saved ->", the file is already local.` : formatError(err)
+      );
       process.exit(1);
     }
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@optima-chat/browser-cli",
-  "version": "0.15.0",
+  "version": "0.15.1",
   "type": "module",
   "main": "./dist/index.js",
   "bin": {