@plune-ai/cli 0.14.3 → 0.15.0

package/README.md

@@ -143,6 +143,11 @@ which reports as the run happens and needs no second step:
 reporter: [['list'], ['@plune-ai/playwright']],
 ```
 
+Either way a Playwright suite brings its screenshots: every image an attempt kept as a file — PNG,
+JPEG or WebP, up to 2 MB each, 20 per result — is uploaded to that result, a passed test's too, so a
+run in Plune shows each test with what it looked at. `plune run import` finds them where the report
+says they are, so run it where the report was written; the summary counts any it could not find.
+
 ### Several jobs, one run
 
 Two suites, or a sharded matrix, report into a single run when every job shares a key **and** knows

package/dist/cli.cjs

@@ -2241,7 +2241,7 @@ function createClient(opts) {
   const wait = opts.wait ?? sleep;
   const base = opts.apiUrl.replace(/\/+$/, "");
   const scrub = (text2) => opts.token === "" ? text2 : text2.split(opts.token).join("***");
-  async function send(method, path14, body) {
+  async function send(method, path14, payload) {
     let last = {
       kind: "unavailable",
       status: null,
@@ -2252,11 +2252,11 @@ function createClient(opts) {
       try {
         res = await doFetch(`${base}${path14}`, {
           method,
-          headers: { "content-type": "application/json", authorization: `Bearer ${opts.token}` },
+          headers: { "content-type": payload?.type ?? "application/json", authorization: `Bearer ${opts.token}` },
           // A DELETE carries none. Sending `"undefined"` as a body is what a naive
           // `JSON.stringify(body)` would do, and some proxies answer that with a 400 nobody can
           // explain.
-          ...body === void 0 ? {} : { body: JSON.stringify(body) }
+          ...payload === void 0 ? {} : { body: payload.body }
         });
       } catch (err) {
         last = { kind: "unavailable", status: null, detail: scrub(String(err)) };
@@ -2282,7 +2282,8 @@ function createClient(opts) {
     return { ok: false, ...last };
   }
   return {
-    post: (path14, body) => send("POST", path14, body),
+    post: (path14, body) => send("POST", path14, { body: JSON.stringify(body), type: "application/json" }),
+    upload: (path14, bytes, contentType) => send("POST", path14, { body: bytes, type: contentType }),
     del: (path14) => send("DELETE", path14)
   };
 }
@@ -2393,6 +2394,19 @@ var init_fallback = __esm({
 });
 
 // src/reporter-core/session.ts
+function nameOf(file) {
+  const own = file.name.split(/[\\/]/).pop()?.trim() || (0, import_node_path2.basename)(file.path);
+  const fileExt = (0, import_node_path2.extname)(file.path);
+  const ownExt = (0, import_node_path2.extname)(own);
+  const typed = ownExt !== "" && (ownExt.toLowerCase() === fileExt.toLowerCase() || TYPE_BY_EXTENSION[ownExt.toLowerCase()] !== void 0);
+  const ext = typed ? ownExt : fileExt;
+  let stem = typed ? own.slice(0, -ownExt.length) : own;
+  if (stem.length + ext.length > NAME_MAX) {
+    stem = stem.slice(0, Math.max(0, NAME_MAX - ext.length));
+    if (/[\uD800-\uDBFF]$/.test(stem)) stem = stem.slice(0, -1);
+  }
+  return `${stem}${ext}`;
+}
 function boundKey(key) {
   if (key.value.length <= KEY_MAX) return key;
   const digest = (0, import_node_crypto2.createHash)("sha256").update(key.value).digest("hex").slice(0, 16);
@@ -2458,7 +2472,8 @@ async function startRun(passed, declared) {
     offered: 0,
     created: 0,
     unoffered: 0,
-    deferred: 0
+    deferred: 0,
+    screenshots: { uploaded: 0, skipped: 0, failed: 0 }
   };
   const resolved = /* @__PURE__ */ new Map();
   let buffer = [];
@@ -2468,6 +2483,7 @@ async function startRun(passed, declared) {
   let offline = token === "";
   let done = false;
   const shortened = /* @__PURE__ */ new Set();
+  const skipsSaid = /* @__PURE__ */ new Set();
   function bound(keys) {
     return keys.map((key) => {
       const fit = boundKey(key);
@@ -2596,6 +2612,60 @@ async function startRun(passed, declared) {
       stats.created += out.body.results.filter((r) => r?.outcome === "created").length;
     }
   }
+  function skip(name, why) {
+    stats.screenshots.skipped += 1;
+    if (skipsSaid.has(why)) return;
+    skipsSaid.add(why);
+    log(`plune: screenshot "${name}" not uploaded \u2014 ${SKIPPED[why]}.`);
+  }
+  async function uploadOne(shot) {
+    let bytes;
+    try {
+      if ((await import_node_fs2.promises.stat(shot.path)).size > SCREENSHOT_BYTES) return skip(shot.name, "size");
+      bytes = await import_node_fs2.promises.readFile(shot.path);
+    } catch {
+      return skip(shot.name, "missing");
+    }
+    const out = await client.upload(
+      `/v1/results/${encodeURIComponent(shot.id)}/files?name=${encodeURIComponent(shot.name)}`,
+      bytes,
+      shot.type
+    );
+    if (out.ok) {
+      stats.screenshots.uploaded += 1;
+      return;
+    }
+    stats.screenshots.failed += 1;
+    if (stats.screenshots.failed === 1) {
+      log(`plune: could not upload a screenshot (${out.detail || out.kind}) \u2014 its result is reported without it.`);
+    }
+  }
+  async function upload(items, filesByKey) {
+    if (filesByKey.size === 0 || !Array.isArray(items)) return;
+    const shots = [];
+    for (const item of items) {
+      if (item?.status !== "accepted" || typeof item.id !== "string" || typeof item.resultKey !== "string") continue;
+      let room = SCREENSHOTS_PER_RESULT;
+      for (const file of filesByKey.get(item.resultKey) ?? []) {
+        const type = file.contentType?.split(";")[0]?.trim().toLowerCase() || TYPE_BY_EXTENSION[(0, import_node_path2.extname)(file.path).toLowerCase()];
+        if (!type?.startsWith("image/")) continue;
+        const name = nameOf(file);
+        if (file.name.startsWith("_")) skip(name, "hidden");
+        else if (!SCREENSHOT_TYPES.has(type)) skip(name, "type");
+        else if (room === 0) skip(name, "limit");
+        else {
+          room -= 1;
+          shots.push({ id: item.id, name, type, path: file.path });
+        }
+      }
+    }
+    let next = 0;
+    await Promise.all(
+      Array.from({ length: Math.min(UPLOADS_AT_ONCE, shots.length) }, async () => {
+        while (next < shots.length) await uploadOne(shots[next++]);
+      })
+    );
+  }
   async function flush() {
     if (buffer.length === 0) return;
     const pending = buffer;
@@ -2607,11 +2677,13 @@ async function startRun(passed, declared) {
     await resolveKeys(pending.flatMap((p) => p.keys));
     const submissions = [];
     const unanswered = [];
+    const filesByKey = /* @__PURE__ */ new Map();
     for (const result of pending) {
       const match = matchFor(result);
       if (match.kind === "found") {
-        const { keys: _keys, ...rest } = result;
+        const { keys: _keys, files, ...rest } = result;
         submissions.push({ ...rest, testCaseId: match.testCaseId });
+        if (files !== void 0) filesByKey.set(result.resultKey, files);
       } else if (match.kind === "unmatched") {
         stats.unresolved += 1;
         remember(result);
@@ -2629,14 +2701,16 @@ async function startRun(passed, declared) {
         defer(part);
         continue;
       }
-      const out = await client.post(`/v1/runs/${runId}/results`, {
-        results: part
-      });
+      const out = await client.post(
+        `/v1/runs/${runId}/results`,
+        { results: part }
+      );
       const counts = out.ok ? out.body?.counts : void 0;
       const told = COUNTED.filter((k) => typeof counts?.[k] === "number");
       if (told.length > 0) {
         for (const k of told) stats[k] += counts[k];
         progress();
+        await upload(out.ok ? out.body?.items : void 0, filesByKey);
         continue;
       }
       if (out.ok) {
@@ -2666,6 +2740,12 @@ async function startRun(passed, declared) {
     if (stats.unresolved > 0) parts.push(`${stats.unresolved} with no matching test case`);
     if (stats.offered > 0) parts.push(`${stats.offered} offered for review`);
     if (stats.created > 0) parts.push(`${stats.created} added as cases (trusted source)`);
+    const shots = stats.screenshots;
+    if (shots.uploaded + shots.skipped + shots.failed > 0) {
+      parts.push(
+        `${shots.uploaded} screenshot(s) uploaded` + (shots.skipped > 0 ? `, ${shots.skipped} skipped` : "") + (shots.failed > 0 ? `, ${shots.failed} failed to upload` : "")
+      );
+    }
     if (stats.deferred > 0) parts.push(`${stats.deferred} not delivered \u2014 written to ${fallbackPath}`);
     if (shortened.size > 0) parts.push(`${shortened.size} test name(s) shortened to fit a key`);
     log(`plune: ${parts.join(" \xB7 ")}`);
@@ -2717,12 +2797,14 @@ async function startRun(passed, declared) {
     }
   };
 }
-var import_node_crypto2, RESOLVE_CHUNK, BATCH_MAX, DISCOVER_MAX, BATCH_DEFAULT, EXPECTED_MAX, KEY_MAX, TITLE_MAX, COUNTED, BATCH_BYTES, ENVELOPE_BYTES;
+var import_node_crypto2, import_node_fs2, import_node_path2, RESOLVE_CHUNK, BATCH_MAX, DISCOVER_MAX, BATCH_DEFAULT, EXPECTED_MAX, KEY_MAX, TITLE_MAX, COUNTED, SCREENSHOT_TYPES, SCREENSHOT_BYTES, SCREENSHOTS_PER_RESULT, TYPE_BY_EXTENSION, UPLOADS_AT_ONCE, NAME_MAX, SKIPPED, BATCH_BYTES, ENVELOPE_BYTES;
 var init_session = __esm({
   "src/reporter-core/session.ts"() {
     "use strict";
     init_cjs_shims();
     import_node_crypto2 = require("crypto");
+    import_node_fs2 = require("fs");
+    import_node_path2 = require("path");
     init_api_url();
     init_credentials();
     init_client();
@@ -2736,6 +2818,24 @@ var init_session = __esm({
     KEY_MAX = 1024;
     TITLE_MAX = 300;
     COUNTED = ["accepted", "duplicate", "conflict", "rejected"];
+    SCREENSHOT_TYPES = /* @__PURE__ */ new Set(["image/png", "image/jpeg", "image/webp"]);
+    SCREENSHOT_BYTES = 2 * 1024 * 1024;
+    SCREENSHOTS_PER_RESULT = 20;
+    TYPE_BY_EXTENSION = {
+      ".png": "image/png",
+      ".jpg": "image/jpeg",
+      ".jpeg": "image/jpeg",
+      ".webp": "image/webp"
+    };
+    UPLOADS_AT_ONCE = 4;
+    NAME_MAX = 200;
+    SKIPPED = {
+      hidden: "its name starts with an underscore, which hides an attachment in Playwright's own reports too",
+      type: "Plune takes PNG, JPEG and WebP",
+      limit: `a result holds at most ${SCREENSHOTS_PER_RESULT}`,
+      missing: "the file cannot be read on this machine",
+      size: "Plune takes a file of at most 2 MB"
+    };
     BATCH_BYTES = 8 * 1024 * 1024;
     ENVELOPE_BYTES = Buffer.byteLength(JSON.stringify({ results: [] }));
   }
@@ -3191,15 +3291,18 @@ function failureOf(attempt, home = (0, import_node_os2.homedir)()) {
   return Object.keys(failure).length === 0 ? void 0 : failure;
 }
 function repoRootOf(dir, stopAt) {
-  for (let here = (0, import_node_path2.resolve)(dir); ; here = (0, import_node_path2.dirname)(here)) {
-    if ((0, import_node_fs2.existsSync)((0, import_node_path2.join)(here, ".git"))) return here;
-    if (here === stopAt || (0, import_node_path2.dirname)(here) === here) return void 0;
+  for (let here = (0, import_node_path3.resolve)(dir); ; here = (0, import_node_path3.dirname)(here)) {
+    if ((0, import_node_fs3.existsSync)((0, import_node_path3.join)(here, ".git"))) return here;
+    if (here === stopAt || (0, import_node_path3.dirname)(here) === here) return void 0;
   }
 }
 function errorContextOf(attempt, home = (0, import_node_os2.homedir)()) {
   const text2 = attempt.errors.map((e) => e.text).filter((t) => t !== "").join("\n\n");
   return cut(machineless(text2, attempt.repoRoot, home));
 }
+function filesOf(attempt) {
+  return attempt.attachments.filter((a) => a.path !== void 0);
+}
 function machineless(raw, repoRoot, home) {
   let text2 = (0, import_node_util.stripVTControlCharacters)(raw).replaceAll("\x1B", "");
   const root = repoRoot === void 0 ? void 0 : pathIn(repoRoot);
@@ -3279,14 +3382,14 @@ function fromRoot(place, root) {
 function webLink(href) {
   return href !== void 0 && /^https?:\/\//i.test(href) && URL.canParse(href) ? href : void 0;
 }
-var import_node_fs2, import_node_os2, import_node_path2, import_node_util, PATH_START, TEXT_LIMIT, HEADLINE_LIMIT, jsonBytes, ANY_HOME, fileNameOf, omitted, METADATA_TYPE, REPO_RELATIVE, FRAME, FRAME_MAX, firstLine, normal;
+var import_node_fs3, import_node_os2, import_node_path3, import_node_util, PATH_START, TEXT_LIMIT, HEADLINE_LIMIT, jsonBytes, ANY_HOME, fileNameOf, omitted, METADATA_TYPE, REPO_RELATIVE, FRAME, FRAME_MAX, firstLine, normal;
 var init_failure_detail = __esm({
   "src/reporter-core/failure-detail.ts"() {
     "use strict";
     init_cjs_shims();
-    import_node_fs2 = require("fs");
+    import_node_fs3 = require("fs");
     import_node_os2 = require("os");
-    import_node_path2 = require("path");
+    import_node_path3 = require("path");
     import_node_util = require("util");
     PATH_START = "(?<![\\w.~%/\\\\-])";
     TEXT_LIMIT = 524288;
@@ -3300,7 +3403,7 @@ var init_failure_detail = __esm({
     FRAME = /^\s+at (?:.*? \()?(.+?):(\d+):(\d+)\)?$/;
     FRAME_MAX = 8192;
     firstLine = (text2) => (0, import_node_util.stripVTControlCharacters)(text2).split("\n").map((line) => line.trim()).find((line) => line !== "");
-    normal = (path14) => import_node_path2.posix.normalize(path14.replace(/\\/g, "/")).replace(/^([a-z]):/, (_, drive) => `${drive.toUpperCase()}:`);
+    normal = (path14) => import_node_path3.posix.normalize(path14.replace(/\\/g, "/")).replace(/^([a-z]):/, (_, drive) => `${drive.toUpperCase()}:`);
   }
 });
 
@@ -3338,6 +3441,7 @@ function pendingFrom(spec, test, result, file, titles, ctx) {
   const attempt = attemptOf(result, file, ctx);
   const errorContext = errorContextOf(attempt);
   const failure = NOT_FAILED.has(attempt.status) ? void 0 : failureOf(attempt);
+  const files = filesOf(attempt);
   const keys = [
     ...test.id !== void 0 ? [{ kind: "playwright-id", value: test.id }] : [],
     { kind: "path-title", value: pathTitle }
@@ -3366,7 +3470,8 @@ function pendingFrom(spec, test, result, file, titles, ctx) {
       }
     } : {},
     ...errorContext !== "" ? { errorContext } : {},
-    ...failure !== void 0 ? { failure } : {}
+    ...failure !== void 0 ? { failure } : {},
+    ...files.length > 0 ? { files } : {}
   };
 }
 function walk(suite, describes, depth, ctx, out) {
@@ -3393,7 +3498,7 @@ function readPlaywrightJson(source, file) {
   const { config, suites } = parsed;
   const rootDir = typeof config?.rootDir === "string" ? config.rootDir : void 0;
   const buildHref = config?.metadata?.ci?.buildHref;
-  const repoRoot = rootDir !== void 0 && (0, import_node_fs3.existsSync)(rootDir) ? repoRootOf(rootDir) : void 0;
+  const repoRoot = rootDir !== void 0 && (0, import_node_fs4.existsSync)(rootDir) ? repoRootOf(rootDir) : void 0;
   const ctx = {
     ...rootDir !== void 0 ? { rootDir } : {},
     ...repoRoot !== void 0 ? { repoRoot } : {},
@@ -3407,12 +3512,12 @@ function readPlaywrightJson(source, file) {
 function looksLikePlaywrightJson(source) {
   return /^\s*\{/.test(source) && source.includes('"suites"');
 }
-var import_node_fs3, SOURCE2, TOKEN, NOT_FAILED, JsonReportError;
+var import_node_fs4, SOURCE2, TOKEN, NOT_FAILED, JsonReportError;
 var init_playwright_json = __esm({
   "src/importers/playwright-json.ts"() {
     "use strict";
     init_cjs_shims();
-    import_node_fs3 = require("fs");
+    import_node_fs4 = require("fs");
     init_failure_detail();
     init_result_key();
     SOURCE2 = "playwright";
@@ -3527,6 +3632,7 @@ async function handleRunImport(options) {
     created: stats.created,
     unoffered: stats.unoffered,
     deferred: stats.deferred,
+    screenshots: { ...stats.screenshots },
     unlocatable: results.filter((r) => r.specRef === void 0).length,
     offering: options.create === true
   };
@@ -3550,6 +3656,12 @@ function describe(result, apiUrl) {
   if (result.conflict > 0 || result.rejected > 0) {
     lines.push(`${result.conflict} conflicted and ${result.rejected} were refused.`);
   }
+  const shots = result.screenshots;
+  if (shots.uploaded + shots.skipped + shots.failed > 0) {
+    lines.push(
+      `${shots.uploaded} screenshot(s) uploaded to their results` + (shots.skipped > 0 ? `, ${shots.skipped} skipped` : "") + (shots.failed > 0 ? `, ${shots.failed} could not be uploaded` : "") + "."
+    );
+  }
   if (result.deferred > 0) {
     lines.push(
       `${result.deferred} could not be sent and are in the fallback file \u2014 "plune run report" retries them.`
@@ -4778,11 +4890,11 @@ module.exports = __toCommonJS(cli_exports);
 init_cjs_shims();
 init_api_url();
 var import_commander = require("commander");
-var import_node_fs4 = require("fs");
-var import_node_path3 = require("path");
+var import_node_fs5 = require("fs");
+var import_node_path4 = require("path");
 var import_node_url = require("url");
-var __dirname = (0, import_node_path3.dirname)((0, import_node_url.fileURLToPath)(importMetaUrl));
-var pkg = JSON.parse((0, import_node_fs4.readFileSync)((0, import_node_path3.join)(__dirname, "..", "package.json"), "utf-8"));
+var __dirname = (0, import_node_path4.dirname)((0, import_node_url.fileURLToPath)(importMetaUrl));
+var pkg = JSON.parse((0, import_node_fs5.readFileSync)((0, import_node_path4.join)(__dirname, "..", "package.json"), "utf-8"));
 function toFormat(s) {
   return s === "json" || s === "markdown" ? s : "console";
 }
@@ -4791,7 +4903,7 @@ function shouldColor(outputPath) {
 }
 function writeReport(text2, outputPath) {
   if (outputPath !== void 0) {
-    (0, import_node_fs4.writeFileSync)(outputPath, text2);
+    (0, import_node_fs5.writeFileSync)(outputPath, text2);
   } else {
     process.stdout.write(text2 + "\n");
   }
@@ -4857,7 +4969,7 @@ function createProgram() {
         Promise.resolve().then(() => (init_errors(), errors_exports)),
         Promise.resolve().then(() => (init_errors2(), errors_exports2))
       ]);
-      loadEnv2(configPath !== void 0 ? (0, import_node_path3.dirname)((0, import_node_path3.resolve)(configPath)) : process.cwd());
+      loadEnv2(configPath !== void 0 ? (0, import_node_path4.dirname)((0, import_node_path4.resolve)(configPath)) : process.cwd());
       try {
         const result = await handleRun2({
           dryRun: options.dryRun,
@@ -5022,7 +5134,7 @@ function createProgram() {
       Promise.resolve().then(() => (init_report(), report_exports)),
       Promise.resolve().then(() => (init_reporters(), reporters_exports))
     ]);
-    const cwd = globals.config !== void 0 ? (0, import_node_path3.dirname)((0, import_node_path3.resolve)(globals.config)) : process.cwd();
+    const cwd = globals.config !== void 0 ? (0, import_node_path4.dirname)((0, import_node_path4.resolve)(globals.config)) : process.cwd();
     loadEnv2(cwd);
     try {
       const result = handleReport2({ cwd });
@@ -5120,7 +5232,7 @@ function createProgram() {
     const globals = command.optsWithGlobals();
     const verbose = globals.verbose === true;
     const { handleSync: handleSync2, reportSyncFailure: reportSyncFailure2 } = await Promise.resolve().then(() => (init_sync(), sync_exports));
-    const cwd = globals.config !== void 0 ? (0, import_node_path3.dirname)((0, import_node_path3.resolve)(globals.config)) : process.cwd();
+    const cwd = globals.config !== void 0 ? (0, import_node_path4.dirname)((0, import_node_path4.resolve)(globals.config)) : process.cwd();
     try {
       const { id, url } = await handleSync2({
         cwd,
@@ -5238,7 +5350,7 @@ function isMainModule() {
   const entry = process.argv[1];
   if (entry === void 0) return false;
   try {
-    return (0, import_node_fs4.realpathSync)(entry) === (0, import_node_fs4.realpathSync)((0, import_node_url.fileURLToPath)(importMetaUrl));
+    return (0, import_node_fs5.realpathSync)(entry) === (0, import_node_fs5.realpathSync)((0, import_node_url.fileURLToPath)(importMetaUrl));
   } catch {
     return false;
   }

package/dist/reporter-core.cjs

@@ -37,6 +37,7 @@ __export(reporter_core_exports, {
   createClient: () => createClient,
   errorContextOf: () => errorContextOf,
   failureOf: () => failureOf,
+  filesOf: () => filesOf,
   readEnv: () => readEnv,
   repoRootOf: () => repoRootOf,
   resultKey: () => resultKey,
@@ -47,6 +48,8 @@ module.exports = __toCommonJS(reporter_core_exports);
 
 // src/reporter-core/session.ts
 var import_node_crypto = require("crypto");
+var import_node_fs2 = require("fs");
+var import_node_path2 = require("path");
 
 // src/cli/api-url.ts
 var DEFAULT_API_URL = "https://beta-api.plune.ai";
@@ -135,7 +138,7 @@ function createClient(opts) {
   const wait = opts.wait ?? sleep;
   const base = opts.apiUrl.replace(/\/+$/, "");
   const scrub = (text) => opts.token === "" ? text : text.split(opts.token).join("***");
-  async function send(method, path3, body) {
+  async function send(method, path3, payload) {
     let last = {
       kind: "unavailable",
       status: null,
@@ -146,11 +149,11 @@ function createClient(opts) {
       try {
         res = await doFetch(`${base}${path3}`, {
           method,
-          headers: { "content-type": "application/json", authorization: `Bearer ${opts.token}` },
+          headers: { "content-type": payload?.type ?? "application/json", authorization: `Bearer ${opts.token}` },
           // A DELETE carries none. Sending `"undefined"` as a body is what a naive
           // `JSON.stringify(body)` would do, and some proxies answer that with a 400 nobody can
           // explain.
-          ...body === void 0 ? {} : { body: JSON.stringify(body) }
+          ...payload === void 0 ? {} : { body: payload.body }
         });
       } catch (err) {
         last = { kind: "unavailable", status: null, detail: scrub(String(err)) };
@@ -176,7 +179,8 @@ function createClient(opts) {
     return { ok: false, ...last };
   }
   return {
-    post: (path3, body) => send("POST", path3, body),
+    post: (path3, body) => send("POST", path3, { body: JSON.stringify(body), type: "application/json" }),
+    upload: (path3, bytes, contentType) => send("POST", path3, { body: bytes, type: contentType }),
     del: (path3) => send("DELETE", path3)
   };
 }
@@ -269,6 +273,37 @@ var EXPECTED_MAX = 1e4;
 var KEY_MAX = 1024;
 var TITLE_MAX = 300;
 var COUNTED = ["accepted", "duplicate", "conflict", "rejected"];
+var SCREENSHOT_TYPES = /* @__PURE__ */ new Set(["image/png", "image/jpeg", "image/webp"]);
+var SCREENSHOT_BYTES = 2 * 1024 * 1024;
+var SCREENSHOTS_PER_RESULT = 20;
+var TYPE_BY_EXTENSION = {
+  ".png": "image/png",
+  ".jpg": "image/jpeg",
+  ".jpeg": "image/jpeg",
+  ".webp": "image/webp"
+};
+var UPLOADS_AT_ONCE = 4;
+var NAME_MAX = 200;
+function nameOf(file) {
+  const own = file.name.split(/[\\/]/).pop()?.trim() || (0, import_node_path2.basename)(file.path);
+  const fileExt = (0, import_node_path2.extname)(file.path);
+  const ownExt = (0, import_node_path2.extname)(own);
+  const typed = ownExt !== "" && (ownExt.toLowerCase() === fileExt.toLowerCase() || TYPE_BY_EXTENSION[ownExt.toLowerCase()] !== void 0);
+  const ext = typed ? ownExt : fileExt;
+  let stem = typed ? own.slice(0, -ownExt.length) : own;
+  if (stem.length + ext.length > NAME_MAX) {
+    stem = stem.slice(0, Math.max(0, NAME_MAX - ext.length));
+    if (/[\uD800-\uDBFF]$/.test(stem)) stem = stem.slice(0, -1);
+  }
+  return `${stem}${ext}`;
+}
+var SKIPPED = {
+  hidden: "its name starts with an underscore, which hides an attachment in Playwright's own reports too",
+  type: "Plune takes PNG, JPEG and WebP",
+  limit: `a result holds at most ${SCREENSHOTS_PER_RESULT}`,
+  missing: "the file cannot be read on this machine",
+  size: "Plune takes a file of at most 2 MB"
+};
 function boundKey(key) {
   if (key.value.length <= KEY_MAX) return key;
   const digest = (0, import_node_crypto.createHash)("sha256").update(key.value).digest("hex").slice(0, 16);
@@ -336,7 +371,8 @@ async function startRun(passed, declared) {
     offered: 0,
     created: 0,
     unoffered: 0,
-    deferred: 0
+    deferred: 0,
+    screenshots: { uploaded: 0, skipped: 0, failed: 0 }
   };
   const resolved = /* @__PURE__ */ new Map();
   let buffer = [];
@@ -346,6 +382,7 @@ async function startRun(passed, declared) {
   let offline = token === "";
   let done = false;
   const shortened = /* @__PURE__ */ new Set();
+  const skipsSaid = /* @__PURE__ */ new Set();
   function bound(keys) {
     return keys.map((key) => {
       const fit = boundKey(key);
@@ -474,6 +511,60 @@ async function startRun(passed, declared) {
       stats.created += out.body.results.filter((r) => r?.outcome === "created").length;
     }
   }
+  function skip(name, why) {
+    stats.screenshots.skipped += 1;
+    if (skipsSaid.has(why)) return;
+    skipsSaid.add(why);
+    log(`plune: screenshot "${name}" not uploaded \u2014 ${SKIPPED[why]}.`);
+  }
+  async function uploadOne(shot) {
+    let bytes;
+    try {
+      if ((await import_node_fs2.promises.stat(shot.path)).size > SCREENSHOT_BYTES) return skip(shot.name, "size");
+      bytes = await import_node_fs2.promises.readFile(shot.path);
+    } catch {
+      return skip(shot.name, "missing");
+    }
+    const out = await client.upload(
+      `/v1/results/${encodeURIComponent(shot.id)}/files?name=${encodeURIComponent(shot.name)}`,
+      bytes,
+      shot.type
+    );
+    if (out.ok) {
+      stats.screenshots.uploaded += 1;
+      return;
+    }
+    stats.screenshots.failed += 1;
+    if (stats.screenshots.failed === 1) {
+      log(`plune: could not upload a screenshot (${out.detail || out.kind}) \u2014 its result is reported without it.`);
+    }
+  }
+  async function upload(items, filesByKey) {
+    if (filesByKey.size === 0 || !Array.isArray(items)) return;
+    const shots = [];
+    for (const item of items) {
+      if (item?.status !== "accepted" || typeof item.id !== "string" || typeof item.resultKey !== "string") continue;
+      let room = SCREENSHOTS_PER_RESULT;
+      for (const file of filesByKey.get(item.resultKey) ?? []) {
+        const type = file.contentType?.split(";")[0]?.trim().toLowerCase() || TYPE_BY_EXTENSION[(0, import_node_path2.extname)(file.path).toLowerCase()];
+        if (!type?.startsWith("image/")) continue;
+        const name = nameOf(file);
+        if (file.name.startsWith("_")) skip(name, "hidden");
+        else if (!SCREENSHOT_TYPES.has(type)) skip(name, "type");
+        else if (room === 0) skip(name, "limit");
+        else {
+          room -= 1;
+          shots.push({ id: item.id, name, type, path: file.path });
+        }
+      }
+    }
+    let next = 0;
+    await Promise.all(
+      Array.from({ length: Math.min(UPLOADS_AT_ONCE, shots.length) }, async () => {
+        while (next < shots.length) await uploadOne(shots[next++]);
+      })
+    );
+  }
   async function flush() {
     if (buffer.length === 0) return;
     const pending = buffer;
@@ -485,11 +576,13 @@ async function startRun(passed, declared) {
     await resolveKeys(pending.flatMap((p) => p.keys));
     const submissions = [];
     const unanswered = [];
+    const filesByKey = /* @__PURE__ */ new Map();
     for (const result of pending) {
       const match = matchFor(result);
       if (match.kind === "found") {
-        const { keys: _keys, ...rest } = result;
+        const { keys: _keys, files, ...rest } = result;
         submissions.push({ ...rest, testCaseId: match.testCaseId });
+        if (files !== void 0) filesByKey.set(result.resultKey, files);
       } else if (match.kind === "unmatched") {
         stats.unresolved += 1;
         remember(result);
@@ -507,14 +600,16 @@ async function startRun(passed, declared) {
         defer(part);
         continue;
       }
-      const out = await client.post(`/v1/runs/${runId}/results`, {
-        results: part
-      });
+      const out = await client.post(
+        `/v1/runs/${runId}/results`,
+        { results: part }
+      );
       const counts = out.ok ? out.body?.counts : void 0;
       const told = COUNTED.filter((k) => typeof counts?.[k] === "number");
       if (told.length > 0) {
         for (const k of told) stats[k] += counts[k];
         progress();
+        await upload(out.ok ? out.body?.items : void 0, filesByKey);
         continue;
       }
       if (out.ok) {
@@ -544,6 +639,12 @@ async function startRun(passed, declared) {
     if (stats.unresolved > 0) parts.push(`${stats.unresolved} with no matching test case`);
     if (stats.offered > 0) parts.push(`${stats.offered} offered for review`);
     if (stats.created > 0) parts.push(`${stats.created} added as cases (trusted source)`);
+    const shots = stats.screenshots;
+    if (shots.uploaded + shots.skipped + shots.failed > 0) {
+      parts.push(
+        `${shots.uploaded} screenshot(s) uploaded` + (shots.skipped > 0 ? `, ${shots.skipped} skipped` : "") + (shots.failed > 0 ? `, ${shots.failed} failed to upload` : "")
+      );
+    }
     if (stats.deferred > 0) parts.push(`${stats.deferred} not delivered \u2014 written to ${fallbackPath}`);
     if (shortened.size > 0) parts.push(`${shortened.size} test name(s) shortened to fit a key`);
     log(`plune: ${parts.join(" \xB7 ")}`);
@@ -611,9 +712,9 @@ function resultKey(testId, retry) {
 }
 
 // src/reporter-core/failure-detail.ts
-var import_node_fs2 = require("fs");
+var import_node_fs3 = require("fs");
 var import_node_os2 = require("os");
-var import_node_path2 = require("path");
+var import_node_path3 = require("path");
 var import_node_util = require("util");
 function failureOf(attempt, home = (0, import_node_os2.homedir)()) {
   const error = chosen(attempt);
@@ -633,15 +734,18 @@ function failureOf(attempt, home = (0, import_node_os2.homedir)()) {
   return Object.keys(failure).length === 0 ? void 0 : failure;
 }
 function repoRootOf(dir, stopAt) {
-  for (let here = (0, import_node_path2.resolve)(dir); ; here = (0, import_node_path2.dirname)(here)) {
-    if ((0, import_node_fs2.existsSync)((0, import_node_path2.join)(here, ".git"))) return here;
-    if (here === stopAt || (0, import_node_path2.dirname)(here) === here) return void 0;
+  for (let here = (0, import_node_path3.resolve)(dir); ; here = (0, import_node_path3.dirname)(here)) {
+    if ((0, import_node_fs3.existsSync)((0, import_node_path3.join)(here, ".git"))) return here;
+    if (here === stopAt || (0, import_node_path3.dirname)(here) === here) return void 0;
   }
 }
 function errorContextOf(attempt, home = (0, import_node_os2.homedir)()) {
   const text = attempt.errors.map((e) => e.text).filter((t) => t !== "").join("\n\n");
   return cut(machineless(text, attempt.repoRoot, home));
 }
+function filesOf(attempt) {
+  return attempt.attachments.filter((a) => a.path !== void 0);
+}
 function machineless(raw, repoRoot, home) {
   let text = (0, import_node_util.stripVTControlCharacters)(raw).replaceAll("\x1B", "");
   const root = repoRoot === void 0 ? void 0 : pathIn(repoRoot);
@@ -720,7 +824,7 @@ function fileOf(raw) {
   }
   return /^\/[A-Za-z]:\//.test(path3) ? path3.slice(1) : path3;
 }
-var normal = (path3) => import_node_path2.posix.normalize(path3.replace(/\\/g, "/")).replace(/^([a-z]):/, (_, drive) => `${drive.toUpperCase()}:`);
+var normal = (path3) => import_node_path3.posix.normalize(path3.replace(/\\/g, "/")).replace(/^([a-z]):/, (_, drive) => `${drive.toUpperCase()}:`);
 function fromRoot(place, root) {
   if (place === void 0 || root === void 0 || !Number.isInteger(place.line) || place.line < 1) return void 0;
   const base = normal(root).replace(/\/+$/, "");
@@ -743,6 +847,7 @@ function webLink(href) {
   createClient,
   errorContextOf,
   failureOf,
+  filesOf,
   readEnv,
   repoRootOf,
   resultKey,

package/dist/reporter-core.d.cts

@@ -44,12 +44,18 @@ interface Execution {
     retry: number;
     worker?: string;
 }
-/** A link, not a file. Uploading artefacts is C3; this carries what someone else already hosts. */
+/** A link, not a file: this carries what someone else already hosts. A file the runner kept is `files`. */
 interface Attachment {
     name: string;
     url: string;
     contentType?: string;
 }
+/** A file the runner kept for an attempt, where it lies on this machine. */
+interface ResultFile {
+    name: string;
+    contentType?: string;
+    path: string;
+}
 /** One assertion's outcome, in the shape the platform already stores for its own runner. */
 interface AssertionRecord {
     type: string;
@@ -108,6 +114,12 @@ interface PendingResult {
     params?: Record<string, unknown>;
     attachments?: Attachment[];
     assertions?: AssertionRecord[];
+    /**
+     * The files the runner kept for this attempt, passed or not — built by `filesOf`. Never part of a
+     * batch: the paths are this machine's. Once the platform has accepted the result, the core uploads
+     * the screenshots among them to it (platform ADR 0040).
+     */
+    files?: ResultFile[];
 }
 /**
  * What a failed attempt says about itself on the run page (#790, platform ADR-0002): every field
@@ -125,7 +137,8 @@ interface FailureDetail {
         line: number;
         column?: number;
     };
-    /** The files the runner kept for the attempt, by name — the files themselves stay in the CI run. */
+    /** The files the runner kept for the attempt, by name. The screenshots among them are uploaded to the
+     * result besides (`files`); the rest stay in the CI run. */
     artifacts?: {
         name: string;
         contentType?: string;
@@ -178,7 +191,7 @@ interface FailedAttempt {
     repoRoot?: string;
 }
 /** What a run submission looks like once a key has resolved. Built by the core, never by a caller. */
-interface ResultSubmission extends Omit<PendingResult, 'keys' | 'testCaseId'> {
+interface ResultSubmission extends Omit<PendingResult, 'keys' | 'testCaseId' | 'files'> {
     testCaseId: string;
 }
 /** One entry of the "what this run intended to execute" list, from which the platform derives
@@ -233,6 +246,16 @@ interface RunStats extends SubmitCounts {
      * the platform could not be asked about. Apart from `rejected` and `unresolved`, which it did answer.
      */
     deferred: number;
+    /**
+     * The screenshots of the accepted results (platform ADR 0040): uploaded to their result; skipped
+     * because the platform would refuse them or the file is not here; refused or unanswered on the way.
+     * None of them costs a result — a result is reported whether its screenshots arrive or not.
+     */
+    screenshots: {
+        uploaded: number;
+        skipped: number;
+        failed: number;
+    };
 }
 /** Everything the core needs to talk to a deployment. */
 interface ReporterConfig {
@@ -415,6 +438,12 @@ declare function repoRootOf(dir: string, stopAt?: string): string | undefined;
  * searched for credentials: the platform does that before its own cut (ADR-0005).
  */
 declare function errorContextOf(attempt: FailedAttempt, home?: string): string;
+/**
+ * The files the runner kept for one attempt, whatever its status: a screenshot of a test that passed
+ * is how a person sees what it checked (plune-ai/plune#913). Which of them the platform takes, the
+ * session decides as it uploads — once, for both roads.
+ */
+declare function filesOf(attempt: FailedAttempt): ResultFile[];
 /**
  * A link only for an address a browser opens as a page — never `file:`, `javascript:` or half a URL.
  * As the platform checks it: `new URL` alone mends `https:/host` and `https:host` into https, and a
@@ -452,6 +481,11 @@ interface ClientOptions {
 }
 interface PlatformClient {
     post<T>(path: string, body: unknown): Promise<ClientOutcome<T>>;
+    /**
+     * A file's bytes as the body, under the file's own type (platform ADR 0040) — not JSON. Through the
+     * same retries as a POST, so a 429 on the platform's upload limit waits out its `Retry-After`.
+     */
+    upload<T>(path: string, bytes: Buffer, contentType: string): Promise<ClientOutcome<T>>;
     /**
      * A DELETE, through the same retries, the same classification and the same token scrubbing as a
      * POST. It is here rather than as a bare `fetch` in the one command that needs it because all
@@ -462,4 +496,4 @@ interface PlatformClient {
 }
 declare function createClient(opts: ClientOptions): PlatformClient;
 
-export { type AssertionRecord, type Attachment, type AttemptError, type ClientFailureKind, type ClientOptions, type ClientOutcome, DEFAULT_FALLBACK_PATH, type DeclaredStep, type DeferredBatch, type DeferredResult, type EnvSettings, type Execution, type ExpectedEntry, type ExternalKeyKind, type FailedAttempt, type FailureDetail, type KeyRef, type PendingResult, type PlatformClient, RESULT_KEY_MAX, type ReporterConfig, type ResultStatus, type ResultSubmission, type RunKind, type RunMeta, type RunRecord, type RunSession, type RunStats, type RunnerLocation, type SubmitCounts, UNSUPPORTED_VARS, appendBatch, createClient, errorContextOf, failureOf, readEnv, repoRootOf, resultKey, startRun, webLink };
+export { type AssertionRecord, type Attachment, type AttemptError, type ClientFailureKind, type ClientOptions, type ClientOutcome, DEFAULT_FALLBACK_PATH, type DeclaredStep, type DeferredBatch, type DeferredResult, type EnvSettings, type Execution, type ExpectedEntry, type ExternalKeyKind, type FailedAttempt, type FailureDetail, type KeyRef, type PendingResult, type PlatformClient, RESULT_KEY_MAX, type ReporterConfig, type ResultFile, type ResultStatus, type ResultSubmission, type RunKind, type RunMeta, type RunRecord, type RunSession, type RunStats, type RunnerLocation, type SubmitCounts, UNSUPPORTED_VARS, appendBatch, createClient, errorContextOf, failureOf, filesOf, readEnv, repoRootOf, resultKey, startRun, webLink };

package/dist/reporter-core.d.ts

@@ -44,12 +44,18 @@ interface Execution {
     retry: number;
     worker?: string;
 }
-/** A link, not a file. Uploading artefacts is C3; this carries what someone else already hosts. */
+/** A link, not a file: this carries what someone else already hosts. A file the runner kept is `files`. */
 interface Attachment {
     name: string;
     url: string;
     contentType?: string;
 }
+/** A file the runner kept for an attempt, where it lies on this machine. */
+interface ResultFile {
+    name: string;
+    contentType?: string;
+    path: string;
+}
 /** One assertion's outcome, in the shape the platform already stores for its own runner. */
 interface AssertionRecord {
     type: string;
@@ -108,6 +114,12 @@ interface PendingResult {
     params?: Record<string, unknown>;
     attachments?: Attachment[];
     assertions?: AssertionRecord[];
+    /**
+     * The files the runner kept for this attempt, passed or not — built by `filesOf`. Never part of a
+     * batch: the paths are this machine's. Once the platform has accepted the result, the core uploads
+     * the screenshots among them to it (platform ADR 0040).
+     */
+    files?: ResultFile[];
 }
 /**
  * What a failed attempt says about itself on the run page (#790, platform ADR-0002): every field
@@ -125,7 +137,8 @@ interface FailureDetail {
         line: number;
         column?: number;
     };
-    /** The files the runner kept for the attempt, by name — the files themselves stay in the CI run. */
+    /** The files the runner kept for the attempt, by name. The screenshots among them are uploaded to the
+     * result besides (`files`); the rest stay in the CI run. */
     artifacts?: {
         name: string;
         contentType?: string;
@@ -178,7 +191,7 @@ interface FailedAttempt {
     repoRoot?: string;
 }
 /** What a run submission looks like once a key has resolved. Built by the core, never by a caller. */
-interface ResultSubmission extends Omit<PendingResult, 'keys' | 'testCaseId'> {
+interface ResultSubmission extends Omit<PendingResult, 'keys' | 'testCaseId' | 'files'> {
     testCaseId: string;
 }
 /** One entry of the "what this run intended to execute" list, from which the platform derives
@@ -233,6 +246,16 @@ interface RunStats extends SubmitCounts {
      * the platform could not be asked about. Apart from `rejected` and `unresolved`, which it did answer.
      */
     deferred: number;
+    /**
+     * The screenshots of the accepted results (platform ADR 0040): uploaded to their result; skipped
+     * because the platform would refuse them or the file is not here; refused or unanswered on the way.
+     * None of them costs a result — a result is reported whether its screenshots arrive or not.
+     */
+    screenshots: {
+        uploaded: number;
+        skipped: number;
+        failed: number;
+    };
 }
 /** Everything the core needs to talk to a deployment. */
 interface ReporterConfig {
@@ -415,6 +438,12 @@ declare function repoRootOf(dir: string, stopAt?: string): string | undefined;
  * searched for credentials: the platform does that before its own cut (ADR-0005).
  */
 declare function errorContextOf(attempt: FailedAttempt, home?: string): string;
+/**
+ * The files the runner kept for one attempt, whatever its status: a screenshot of a test that passed
+ * is how a person sees what it checked (plune-ai/plune#913). Which of them the platform takes, the
+ * session decides as it uploads — once, for both roads.
+ */
+declare function filesOf(attempt: FailedAttempt): ResultFile[];
 /**
  * A link only for an address a browser opens as a page — never `file:`, `javascript:` or half a URL.
  * As the platform checks it: `new URL` alone mends `https:/host` and `https:host` into https, and a
@@ -452,6 +481,11 @@ interface ClientOptions {
 }
 interface PlatformClient {
     post<T>(path: string, body: unknown): Promise<ClientOutcome<T>>;
+    /**
+     * A file's bytes as the body, under the file's own type (platform ADR 0040) — not JSON. Through the
+     * same retries as a POST, so a 429 on the platform's upload limit waits out its `Retry-After`.
+     */
+    upload<T>(path: string, bytes: Buffer, contentType: string): Promise<ClientOutcome<T>>;
     /**
      * A DELETE, through the same retries, the same classification and the same token scrubbing as a
      * POST. It is here rather than as a bare `fetch` in the one command that needs it because all
@@ -462,4 +496,4 @@ interface PlatformClient {
 }
 declare function createClient(opts: ClientOptions): PlatformClient;
 
-export { type AssertionRecord, type Attachment, type AttemptError, type ClientFailureKind, type ClientOptions, type ClientOutcome, DEFAULT_FALLBACK_PATH, type DeclaredStep, type DeferredBatch, type DeferredResult, type EnvSettings, type Execution, type ExpectedEntry, type ExternalKeyKind, type FailedAttempt, type FailureDetail, type KeyRef, type PendingResult, type PlatformClient, RESULT_KEY_MAX, type ReporterConfig, type ResultStatus, type ResultSubmission, type RunKind, type RunMeta, type RunRecord, type RunSession, type RunStats, type RunnerLocation, type SubmitCounts, UNSUPPORTED_VARS, appendBatch, createClient, errorContextOf, failureOf, readEnv, repoRootOf, resultKey, startRun, webLink };
+export { type AssertionRecord, type Attachment, type AttemptError, type ClientFailureKind, type ClientOptions, type ClientOutcome, DEFAULT_FALLBACK_PATH, type DeclaredStep, type DeferredBatch, type DeferredResult, type EnvSettings, type Execution, type ExpectedEntry, type ExternalKeyKind, type FailedAttempt, type FailureDetail, type KeyRef, type PendingResult, type PlatformClient, RESULT_KEY_MAX, type ReporterConfig, type ResultFile, type ResultStatus, type ResultSubmission, type RunKind, type RunMeta, type RunRecord, type RunSession, type RunStats, type RunnerLocation, type SubmitCounts, UNSUPPORTED_VARS, appendBatch, createClient, errorContextOf, failureOf, filesOf, readEnv, repoRootOf, resultKey, startRun, webLink };

package/dist/reporter-core.js

@@ -1,5 +1,7 @@
 // src/reporter-core/session.ts
 import { createHash } from "crypto";
+import { promises as disk } from "fs";
+import { basename as basename2, extname } from "path";
 
 // src/cli/api-url.ts
 var DEFAULT_API_URL = "https://beta-api.plune.ai";
@@ -88,7 +90,7 @@ function createClient(opts) {
   const wait = opts.wait ?? sleep;
   const base = opts.apiUrl.replace(/\/+$/, "");
   const scrub = (text) => opts.token === "" ? text : text.split(opts.token).join("***");
-  async function send(method, path3, body) {
+  async function send(method, path3, payload) {
     let last = {
       kind: "unavailable",
       status: null,
@@ -99,11 +101,11 @@ function createClient(opts) {
       try {
         res = await doFetch(`${base}${path3}`, {
           method,
-          headers: { "content-type": "application/json", authorization: `Bearer ${opts.token}` },
+          headers: { "content-type": payload?.type ?? "application/json", authorization: `Bearer ${opts.token}` },
           // A DELETE carries none. Sending `"undefined"` as a body is what a naive
           // `JSON.stringify(body)` would do, and some proxies answer that with a 400 nobody can
           // explain.
-          ...body === void 0 ? {} : { body: JSON.stringify(body) }
+          ...payload === void 0 ? {} : { body: payload.body }
         });
       } catch (err) {
         last = { kind: "unavailable", status: null, detail: scrub(String(err)) };
@@ -129,7 +131,8 @@ function createClient(opts) {
     return { ok: false, ...last };
   }
   return {
-    post: (path3, body) => send("POST", path3, body),
+    post: (path3, body) => send("POST", path3, { body: JSON.stringify(body), type: "application/json" }),
+    upload: (path3, bytes, contentType) => send("POST", path3, { body: bytes, type: contentType }),
     del: (path3) => send("DELETE", path3)
   };
 }
@@ -222,6 +225,37 @@ var EXPECTED_MAX = 1e4;
 var KEY_MAX = 1024;
 var TITLE_MAX = 300;
 var COUNTED = ["accepted", "duplicate", "conflict", "rejected"];
+var SCREENSHOT_TYPES = /* @__PURE__ */ new Set(["image/png", "image/jpeg", "image/webp"]);
+var SCREENSHOT_BYTES = 2 * 1024 * 1024;
+var SCREENSHOTS_PER_RESULT = 20;
+var TYPE_BY_EXTENSION = {
+  ".png": "image/png",
+  ".jpg": "image/jpeg",
+  ".jpeg": "image/jpeg",
+  ".webp": "image/webp"
+};
+var UPLOADS_AT_ONCE = 4;
+var NAME_MAX = 200;
+function nameOf(file) {
+  const own = file.name.split(/[\\/]/).pop()?.trim() || basename2(file.path);
+  const fileExt = extname(file.path);
+  const ownExt = extname(own);
+  const typed = ownExt !== "" && (ownExt.toLowerCase() === fileExt.toLowerCase() || TYPE_BY_EXTENSION[ownExt.toLowerCase()] !== void 0);
+  const ext = typed ? ownExt : fileExt;
+  let stem = typed ? own.slice(0, -ownExt.length) : own;
+  if (stem.length + ext.length > NAME_MAX) {
+    stem = stem.slice(0, Math.max(0, NAME_MAX - ext.length));
+    if (/[\uD800-\uDBFF]$/.test(stem)) stem = stem.slice(0, -1);
+  }
+  return `${stem}${ext}`;
+}
+var SKIPPED = {
+  hidden: "its name starts with an underscore, which hides an attachment in Playwright's own reports too",
+  type: "Plune takes PNG, JPEG and WebP",
+  limit: `a result holds at most ${SCREENSHOTS_PER_RESULT}`,
+  missing: "the file cannot be read on this machine",
+  size: "Plune takes a file of at most 2 MB"
+};
 function boundKey(key) {
   if (key.value.length <= KEY_MAX) return key;
   const digest = createHash("sha256").update(key.value).digest("hex").slice(0, 16);
@@ -289,7 +323,8 @@ async function startRun(passed, declared) {
     offered: 0,
     created: 0,
     unoffered: 0,
-    deferred: 0
+    deferred: 0,
+    screenshots: { uploaded: 0, skipped: 0, failed: 0 }
   };
   const resolved = /* @__PURE__ */ new Map();
   let buffer = [];
@@ -299,6 +334,7 @@ async function startRun(passed, declared) {
   let offline = token === "";
   let done = false;
   const shortened = /* @__PURE__ */ new Set();
+  const skipsSaid = /* @__PURE__ */ new Set();
   function bound(keys) {
     return keys.map((key) => {
       const fit = boundKey(key);
@@ -427,6 +463,60 @@ async function startRun(passed, declared) {
       stats.created += out.body.results.filter((r) => r?.outcome === "created").length;
     }
   }
+  function skip(name, why) {
+    stats.screenshots.skipped += 1;
+    if (skipsSaid.has(why)) return;
+    skipsSaid.add(why);
+    log(`plune: screenshot "${name}" not uploaded \u2014 ${SKIPPED[why]}.`);
+  }
+  async function uploadOne(shot) {
+    let bytes;
+    try {
+      if ((await disk.stat(shot.path)).size > SCREENSHOT_BYTES) return skip(shot.name, "size");
+      bytes = await disk.readFile(shot.path);
+    } catch {
+      return skip(shot.name, "missing");
+    }
+    const out = await client.upload(
+      `/v1/results/${encodeURIComponent(shot.id)}/files?name=${encodeURIComponent(shot.name)}`,
+      bytes,
+      shot.type
+    );
+    if (out.ok) {
+      stats.screenshots.uploaded += 1;
+      return;
+    }
+    stats.screenshots.failed += 1;
+    if (stats.screenshots.failed === 1) {
+      log(`plune: could not upload a screenshot (${out.detail || out.kind}) \u2014 its result is reported without it.`);
+    }
+  }
+  async function upload(items, filesByKey) {
+    if (filesByKey.size === 0 || !Array.isArray(items)) return;
+    const shots = [];
+    for (const item of items) {
+      if (item?.status !== "accepted" || typeof item.id !== "string" || typeof item.resultKey !== "string") continue;
+      let room = SCREENSHOTS_PER_RESULT;
+      for (const file of filesByKey.get(item.resultKey) ?? []) {
+        const type = file.contentType?.split(";")[0]?.trim().toLowerCase() || TYPE_BY_EXTENSION[extname(file.path).toLowerCase()];
+        if (!type?.startsWith("image/")) continue;
+        const name = nameOf(file);
+        if (file.name.startsWith("_")) skip(name, "hidden");
+        else if (!SCREENSHOT_TYPES.has(type)) skip(name, "type");
+        else if (room === 0) skip(name, "limit");
+        else {
+          room -= 1;
+          shots.push({ id: item.id, name, type, path: file.path });
+        }
+      }
+    }
+    let next = 0;
+    await Promise.all(
+      Array.from({ length: Math.min(UPLOADS_AT_ONCE, shots.length) }, async () => {
+        while (next < shots.length) await uploadOne(shots[next++]);
+      })
+    );
+  }
   async function flush() {
     if (buffer.length === 0) return;
     const pending = buffer;
@@ -438,11 +528,13 @@ async function startRun(passed, declared) {
     await resolveKeys(pending.flatMap((p) => p.keys));
     const submissions = [];
     const unanswered = [];
+    const filesByKey = /* @__PURE__ */ new Map();
     for (const result of pending) {
       const match = matchFor(result);
       if (match.kind === "found") {
-        const { keys: _keys, ...rest } = result;
+        const { keys: _keys, files, ...rest } = result;
         submissions.push({ ...rest, testCaseId: match.testCaseId });
+        if (files !== void 0) filesByKey.set(result.resultKey, files);
       } else if (match.kind === "unmatched") {
         stats.unresolved += 1;
         remember(result);
@@ -460,14 +552,16 @@ async function startRun(passed, declared) {
         defer(part);
         continue;
       }
-      const out = await client.post(`/v1/runs/${runId}/results`, {
-        results: part
-      });
+      const out = await client.post(
+        `/v1/runs/${runId}/results`,
+        { results: part }
+      );
       const counts = out.ok ? out.body?.counts : void 0;
       const told = COUNTED.filter((k) => typeof counts?.[k] === "number");
       if (told.length > 0) {
         for (const k of told) stats[k] += counts[k];
         progress();
+        await upload(out.ok ? out.body?.items : void 0, filesByKey);
         continue;
       }
       if (out.ok) {
@@ -497,6 +591,12 @@ async function startRun(passed, declared) {
     if (stats.unresolved > 0) parts.push(`${stats.unresolved} with no matching test case`);
     if (stats.offered > 0) parts.push(`${stats.offered} offered for review`);
     if (stats.created > 0) parts.push(`${stats.created} added as cases (trusted source)`);
+    const shots = stats.screenshots;
+    if (shots.uploaded + shots.skipped + shots.failed > 0) {
+      parts.push(
+        `${shots.uploaded} screenshot(s) uploaded` + (shots.skipped > 0 ? `, ${shots.skipped} skipped` : "") + (shots.failed > 0 ? `, ${shots.failed} failed to upload` : "")
+      );
+    }
     if (stats.deferred > 0) parts.push(`${stats.deferred} not delivered \u2014 written to ${fallbackPath}`);
     if (shortened.size > 0) parts.push(`${shortened.size} test name(s) shortened to fit a key`);
     log(`plune: ${parts.join(" \xB7 ")}`);
@@ -595,6 +695,9 @@ function errorContextOf(attempt, home = homedir2()) {
   const text = attempt.errors.map((e) => e.text).filter((t) => t !== "").join("\n\n");
   return cut(machineless(text, attempt.repoRoot, home));
 }
+function filesOf(attempt) {
+  return attempt.attachments.filter((a) => a.path !== void 0);
+}
 function machineless(raw, repoRoot, home) {
   let text = stripVTControlCharacters(raw).replaceAll("\x1B", "");
   const root = repoRoot === void 0 ? void 0 : pathIn(repoRoot);
@@ -695,6 +798,7 @@ export {
   createClient,
   errorContextOf,
   failureOf,
+  filesOf,
   readEnv,
   repoRootOf,
   resultKey,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plune-ai/cli",
-  "version": "0.14.3",
+  "version": "0.15.0",
   "description": "AI-powered assertion testing for LLM apps — write assertions, run them against any provider, get a pass/fail report (local, CI, or diff).",
   "keywords": [
     "ai",