@crvy/rprtr 0.0.8 → 0.1.0

package/CHANGELOG.md

@@ -5,6 +5,61 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.1.0] - 2026-06-08
+
+### Added
+
+- **resolver:** Expose playwrightAnonymousVisualName
+- **resolver:** Add withResolvedVisualNames
+- **server:** Add isPathWithinRoots allowlist helper
+- **server:** Add allowlisted /file artifact route
+- **server:** Add /baseline route, consolidate baseline resolution
+- **server:** Wire artifactRoots and outputDir option
+- Add isCI helper
+- **report-utils:** Route absolute attachment paths to /file
+- **server:** Enrich declared-only images with /baseline urls
+- **reporter:** Add rewriteTestEndAttachments helper
+- **reporter:** Gate screenshot copying and artifacts on CI
+- **cli:** Add --output-dir for live native artifact serving
+
+### Changed
+
+- **resolver:** Compute anonymous title list once
+
+### Documentation
+
+- **spec:** CI-gated screenshot artifact handling design
+- **plans:** Naming fix, live-mode serving, CI-gated copying
+- **server:** Clarify outputDir is resolved against CWD
+
+### Fixed
+
+- Apply web socket payloads directly to the client tree
+- **reporter:** Use Playwright auto-name for unnamed screenshots
+- **server:** Realpath-guard /file route and set nosniff header
+- **server:** Guard /baseline decode, retry parsing, and read errors
+- **server:** Exclude test source dir from /file allowlist
+- **server:** Guard /baseline url parsing, decode testId, test nosniff
+- **server:** Decode /file URLs in approval path
+- **server:** Only enrich declared-only baselines for passed tests
+- **server:** Enrich baselines only when the snapshot file exists
+- **reporter:** Bound onEnd concurrency, fix retry attachments, stop CI queue growth
+
+### Styling
+
+- **client:** Center-align image comparison views
+
+### Testing
+
+- **report-state:** Assert no phantom entry for finalized unnamed screenshot
+- **report-state:** Make phantom-entry guard falsifiable
+- Document phantom fixture divergence and cover multi-occurrence rewrite
+- **server:** Cover raw-traversal and empty-roots cases for isPathWithinRoots
+## [0.0.9] - 2026-06-05
+
+### Fixed
+
+- Remove file watcher that interfered with live updates
 ## [0.0.8] - 2026-06-05
 
 ### Fixed

package/dist/{chunk-XU5VQ3FZ.js → chunk-35CZLYSW.js}

@@ -1,20 +1,21 @@
 import {
   ApproveRequestBodySchema,
+  IncomingWebSocketMessageSchema,
   LoadedReportDataSchema,
   OfflineReportSchema,
+  RunEndDataSchema,
   TestBeginDataSchema,
   TestEndDataSchema,
-  WebSocketMessageSchema,
   applyTestBeginEvent,
   applyTestEndEvent,
   createMutableReportState,
   finalizeRunEvent,
   resolveBaselineTargets,
   safeParse
-} from "./chunk-EJRLZZ22.js";
+} from "./chunk-43JLQN36.js";
 
 // src/server/app.ts
-import { dirname as dirname3, join as join4 } from "path";
+import { dirname as dirname3, join as join3 } from "path";
 import { fileURLToPath } from "url";
 import pLimit from "p-limit";
 
@@ -157,7 +158,10 @@ async function respondWithFile(filePath, contentType) {
   try {
     const file = await readFile(filePath);
     const resolvedContentType = contentType ?? inferContentType(filePath);
-    const headers = resolvedContentType === void 0 ? void 0 : { "Content-Type": resolvedContentType };
+    const headers = { "X-Content-Type-Options": "nosniff" };
+    if (resolvedContentType !== void 0) {
+      headers["Content-Type"] = resolvedContentType;
+    }
     return new Response(file, { headers });
   } catch (error) {
     if (isFileNotFound(error)) {
@@ -167,166 +171,212 @@ async function respondWithFile(filePath, contentType) {
   }
 }
 
+// src/server/handlers.ts
+import { existsSync as existsSync2 } from "fs";
+
+// src/server/artifact-routes.ts
+import { existsSync } from "fs";
+import { realpath } from "fs/promises";
+import { dirname as dirname2, resolve as resolve2 } from "path";
+
 // src/server/utils.ts
+import { isAbsolute, relative, resolve, sep } from "path";
+var LIVE_UPDATES_WEBSOCKET_PATH = "/";
 function broadcastToBrowsers(wsClients, msg) {
   const payload = JSON.stringify(msg);
   wsClients.forEach((ws) => {
     ws.send(payload);
   });
 }
+function isWebSocketUpgradeRequest(req) {
+  return new URL(req.url).pathname === LIVE_UPDATES_WEBSOCKET_PATH && req.headers.get("upgrade")?.toLowerCase() === "websocket";
+}
+function isPathWithinRoots(target, roots) {
+  const resolvedTarget = resolve(target);
+  return roots.some((root) => {
+    const rel = relative(resolve(root), resolvedTarget);
+    return rel === "" || !rel.startsWith(`..${sep}`) && rel !== ".." && !isAbsolute(rel);
+  });
+}
+
+// src/server/artifact-routes.ts
+async function realpathOrNull(path) {
+  try {
+    return await realpath(path);
+  } catch {
+    return null;
+  }
+}
+async function handleFile(ctx, req) {
+  const notFound = () => new Response("Not Found", { status: 404 });
+  let decodedPath;
+  try {
+    decodedPath = resolve2(decodeURIComponent(new URL(req.url).pathname.slice("/file/".length)));
+  } catch {
+    return notFound();
+  }
+  const realTarget = await realpathOrNull(decodedPath);
+  if (realTarget === null) {
+    return notFound();
+  }
+  const realRoots = (await Promise.all((ctx.artifactRoots ?? []).map((root) => realpathOrNull(resolve2(root))))).filter(
+    (root) => root !== null
+  );
+  if (!isPathWithinRoots(realTarget, realRoots)) {
+    return notFound();
+  }
+  try {
+    const file = await respondWithFile(realTarget);
+    return file ?? notFound();
+  } catch {
+    return notFound();
+  }
+}
+function reporterTitlePath(test) {
+  const testFile = test.location?.file;
+  return ["", test.browser, testFile ?? "", ...test.titlePath, test.title];
+}
+function resolveBaselineSnapshotPath(routing, test, retry, imageName) {
+  const testFile = test.location?.file;
+  const declaration = test.results?.[retry]?.visualDeclarations?.find((candidate) => candidate.visualName === imageName);
+  if (routing === void 0 || testFile === void 0 || declaration === void 0) {
+    return null;
+  }
+  const targets = resolveBaselineTargets({
+    testFile,
+    reporterTitlePath: reporterTitlePath(test),
+    declarations: [declaration],
+    config: {
+      configDir: routing.configDir,
+      testDir: routing.playwrightTestDir ?? dirname2(testFile),
+      snapshotDir: routing.playwrightSnapshotDir ?? dirname2(testFile),
+      projectName: test.browser,
+      snapshotSuffix: process.platform,
+      snapshotPathTemplate: routing.playwrightSnapshotPathTemplate,
+      toHaveScreenshotPathTemplate: routing.playwrightToHaveScreenshotPathTemplate
+    },
+    snapshotPathExists: existsSync
+  });
+  return targets.length === 1 ? targets[0]?.snapshotPath ?? null : null;
+}
+async function handleBaseline(ctx, req) {
+  const notFound = () => new Response("Not Found", { status: 404 });
+  let testId;
+  let retry;
+  let visualName;
+  try {
+    const segments = new URL(req.url).pathname.slice("/baseline/".length).split("/");
+    const [rawTestId, retryRaw, ...visualNameParts] = segments;
+    if (rawTestId === void 0 || retryRaw === void 0 || !/^\d+$/.test(retryRaw) || visualNameParts.length === 0) {
+      return notFound();
+    }
+    testId = decodeURIComponent(rawTestId);
+    retry = Number(retryRaw);
+    visualName = decodeURIComponent(visualNameParts.join("/"));
+  } catch {
+    return notFound();
+  }
+  const test = ctx.reportData.tests[testId];
+  if (test === void 0) {
+    return notFound();
+  }
+  const snapshotPath = resolveBaselineSnapshotPath(ctx.approvalRouting, test, retry, visualName);
+  if (snapshotPath === null) {
+    return notFound();
+  }
+  try {
+    const file = await respondWithFile(snapshotPath);
+    return file ?? notFound();
+  } catch {
+    return notFound();
+  }
+}
+function handleArtifactRoute(ctx, req, pathname) {
+  if (pathname.startsWith("/file/")) {
+    return handleFile(ctx, req);
+  }
+  if (pathname.startsWith("/baseline/")) {
+    return handleBaseline(ctx, req);
+  }
+  return null;
+}
 
 // src/server/handlers.ts
 function handleTestBegin(ctx, data) {
   const test = applyTestBeginEvent(ctx, data);
   ctx.reportData.isRunning = true;
   console.log(`  \u25B6 [${test.browser ?? "?"}] ${test.title}`);
-  broadcastToBrowsers(ctx.wsClients, { type: "test-begin", data });
+  const message = { type: "test-begin", data: test };
+  broadcastToBrowsers(ctx.wsClients, message);
+}
+function enrichDeclaredBaselines(ctx, test) {
+  const retry = (test.results?.length ?? 0) - 1;
+  const images = test.results?.[retry]?.images;
+  if (retry < 0 || images === void 0) {
+    return;
+  }
+  for (const [visualName, image] of Object.entries(images)) {
+    if (image === void 0 || image.source !== "declared-only") {
+      continue;
+    }
+    const snapshotPath = resolveBaselineSnapshotPath(ctx.approvalRouting, test, retry, visualName);
+    if (snapshotPath === null || !existsSync2(snapshotPath)) {
+      continue;
+    }
+    image.expect = `/baseline/${encodeURIComponent(test.id)}/${retry}/${encodeURIComponent(visualName)}`;
+    image.source = "baseline-only";
+  }
 }
 function handleTestEnd(ctx, data) {
   const result = applyTestEndEvent(ctx, data);
-  if (result !== null) {
-    const { test, diffCount } = result;
-    const icon = data.status === "passed" ? "\u2713" : data.status === "skipped" ? "\u2013" : "\u2717";
-    const dur = data.duration === null || data.duration === void 0 ? "" : ` (${data.duration}ms)`;
-    const diffNote = diffCount > 0 ? ` [${diffCount} diff(s)]` : "";
-    const errNote = data.error !== null && data.error !== void 0 ? `
-    Error: ${data.error}` : "";
-    console.log(`  ${icon} [${test.browser}] ${test.title}${dur}${diffNote}${errNote}`);
+  if (result === null) {
+    console.error("[Server] test-end for unknown test id:", data.id);
+    return;
   }
-  broadcastToBrowsers(ctx.wsClients, { type: "test-update", data });
+  const { test, diffCount } = result;
+  if (data.status === "passed") {
+    enrichDeclaredBaselines(ctx, test);
+  }
+  const icon = data.status === "passed" ? "\u2713" : data.status === "skipped" ? "\u2013" : "\u2717";
+  const dur = data.duration === null || data.duration === void 0 ? "" : ` (${data.duration}ms)`;
+  const diffNote = diffCount > 0 ? ` [${diffCount} diff(s)]` : "";
+  const errNote = data.error !== null && data.error !== void 0 ? `
+    Error: ${data.error}` : "";
+  console.log(`  ${icon} [${test.browser}] ${test.title}${dur}${diffNote}${errNote}`);
+  const message = { type: "test-update", data: test };
+  broadcastToBrowsers(ctx.wsClients, message);
 }
 async function handleRunEnd(ctx, data) {
+  const removedTestIds = Object.keys(ctx.reportData.tests).filter((id) => !ctx.currentRunIds.has(id));
   const { passed, failed, pending } = finalizeRunEvent(ctx);
   await ctx.saveReport();
   console.log(`
 Run complete \u2014 ${passed} passed, ${failed} failed, ${pending} skipped`);
-  broadcastToBrowsers(ctx.wsClients, { type: "run-end", data });
+  const message = {
+    type: "run-end",
+    data: { status: data.status, removedTestIds }
+  };
+  broadcastToBrowsers(ctx.wsClients, message);
 }
 function handleApprove() {
   console.log("[Server] Received approve message via WebSocket (handled via HTTP API)");
 }
 function handleSync(ctx) {
   console.log("[Server] Received sync message");
-  broadcastToBrowsers(ctx.wsClients, { type: "sync", data: ctx.reportData });
-}
-
-// src/server/report-watch.ts
-import { readdir as readdir2, stat as stat2 } from "fs/promises";
-import { join as join2 } from "path";
-var OFFLINE_REPORT_FILE_PATTERN2 = /^crvy-rprtr(?:-\d+)?\.json$/;
-function createDebouncedRefresh(reload, delayMs = 50) {
-  let timer = null;
-  return () => {
-    if (timer !== null) {
-      clearTimeout(timer);
-    }
-    timer = setTimeout(() => {
-      timer = null;
-      void reload();
-    }, delayMs);
-  };
-}
-function isFileNotFound2(error) {
-  return typeof error === "object" && error !== null && "code" in error && error.code === "ENOENT";
-}
-async function describeFile(filePath, label) {
-  try {
-    const fileStats = await stat2(filePath);
-    return `${label}:${fileStats.size}:${fileStats.mtimeMs}`;
-  } catch (error) {
-    if (isFileNotFound2(error)) {
-      return null;
+  const message = {
+    type: "sync",
+    data: {
+      tests: ctx.reportData.tests,
+      isUpdateMode: ctx.reportData.isUpdateMode
     }
-    throw error;
-  }
-}
-async function createOfflineFingerprint(offlineReportDir) {
-  if (!await isDirectory(offlineReportDir)) {
-    return "offline:missing";
-  }
-  try {
-    const entries = await readdir2(offlineReportDir, { withFileTypes: true });
-    const relevantEntries = entries.filter(
-      (entry) => entry.isFile() && (entry.name === "report.json" || OFFLINE_REPORT_FILE_PATTERN2.test(entry.name))
-    ).sort((left, right) => left.name.localeCompare(right.name));
-    const parts = await Promise.all(
-      relevantEntries.map((entry) => describeFile(join2(offlineReportDir, entry.name), entry.name))
-    );
-    return `offline:${parts.filter((part) => part !== null).join("|")}`;
-  } catch (error) {
-    const errorMsg = error instanceof Error ? error.message : String(error);
-    console.warn(`[Server] Unable to scan ${offlineReportDir}: ${errorMsg}`);
-    return "offline:error";
-  }
-}
-async function createScreenshotFingerprint(screenshotDir, relativePath = "") {
-  const directoryPath = relativePath === "" ? screenshotDir : join2(screenshotDir, relativePath);
-  if (!await isDirectory(directoryPath)) {
-    return relativePath === "" ? ["screenshots:missing"] : [];
-  }
-  try {
-    const entries = await readdir2(directoryPath, { withFileTypes: true });
-    const parts = await Promise.all(
-      entries.sort((left, right) => left.name.localeCompare(right.name)).map((entry) => {
-        const entryRelativePath = relativePath === "" ? entry.name : join2(relativePath, entry.name);
-        if (entry.isDirectory()) {
-          return createScreenshotFingerprint(screenshotDir, entryRelativePath);
-        }
-        return describeFile(join2(screenshotDir, entryRelativePath), entryRelativePath);
-      })
-    );
-    return parts.flat().filter((part) => part !== null);
-  } catch (error) {
-    if (isFileNotFound2(error)) {
-      return relativePath === "" ? ["screenshots:missing"] : [];
-    }
-    const errorMsg = error instanceof Error ? error.message : String(error);
-    console.warn(`[Server] Unable to scan ${directoryPath}: ${errorMsg}`);
-    return relativePath === "" ? ["screenshots:error"] : [];
-  }
-}
-async function createArtifactsFingerprint(options) {
-  const [offlineFingerprint, screenshotFingerprint] = await Promise.all([
-    createOfflineFingerprint(options.offlineReportDir),
-    createScreenshotFingerprint(options.screenshotDir)
-  ]);
-  return `${offlineFingerprint}::${screenshotFingerprint.join("|")}`;
-}
-async function watchReportArtifacts(options) {
-  let fingerprint = await createArtifactsFingerprint(options);
-  let isPolling = false;
-  const interval = setInterval(() => {
-    if (isPolling) {
-      return;
-    }
-    isPolling = true;
-    void createArtifactsFingerprint(options).then((nextFingerprint) => {
-      if (nextFingerprint === fingerprint) {
-        return;
-      }
-      fingerprint = nextFingerprint;
-      options.scheduleRefresh();
-    }).catch((error) => {
-      const errorMsg = error instanceof Error ? error.message : String(error);
-      console.warn(`[Server] Artifact refresh poll failed: ${errorMsg}`);
-    }).finally(() => {
-      isPolling = false;
-    });
-  }, 100);
-  return () => {
-    clearInterval(interval);
   };
+  broadcastToBrowsers(ctx.wsClients, message);
 }
 
 // src/server/routes.ts
-import { existsSync } from "fs";
-import { dirname as dirname2, join as join3 } from "path";
-var LIVE_UPDATES_WEBSOCKET_PATH = "/";
-function isWebSocketUpgradeRequest(req) {
-  return new URL(req.url).pathname === LIVE_UPDATES_WEBSOCKET_PATH && req.headers.get("upgrade")?.toLowerCase() === "websocket";
-}
+import { join as join2 } from "path";
 async function handleRoot(ctx) {
-  const html = await respondWithFile(join3(ctx.staticDir, "index.html"), "text/html");
+  const html = await respondWithFile(join2(ctx.staticDir, "index.html"), "text/html");
   return html ?? new Response("Not Found", { status: 404 });
 }
 async function handleAppCss() {
@@ -345,34 +395,13 @@ function handleApiReport(ctx) {
 }
 var APPROVAL_TARGET_ERROR = "Could not resolve approval target";
 function actualPathFromUrl(ctx, actualUrl) {
-  return actualUrl.startsWith("/screenshots/") ? join3(ctx.reportData.screenshotDir, actualUrl.slice("/screenshots/".length)) : actualUrl;
-}
-function reporterTitlePath(test) {
-  const testFile = test.location?.file;
-  return ["", test.browser, testFile ?? "", ...test.titlePath, test.title];
-}
-function resolveApprovalTarget(ctx, test, retry, imageName) {
-  const testFile = test.location?.file;
-  const declaration = test.results?.[retry]?.visualDeclarations?.find((candidate) => candidate.visualName === imageName);
-  if (ctx.approvalRouting === void 0 || testFile === void 0 || declaration === void 0) {
-    return null;
+  if (actualUrl.startsWith("/screenshots/")) {
+    return join2(ctx.reportData.screenshotDir, actualUrl.slice("/screenshots/".length));
   }
-  const targets = resolveBaselineTargets({
-    testFile,
-    reporterTitlePath: reporterTitlePath(test),
-    declarations: [declaration],
-    config: {
-      configDir: ctx.approvalRouting.configDir,
-      testDir: ctx.approvalRouting.playwrightTestDir ?? dirname2(testFile),
-      snapshotDir: ctx.approvalRouting.playwrightSnapshotDir ?? dirname2(testFile),
-      projectName: test.browser,
-      snapshotSuffix: process.platform,
-      snapshotPathTemplate: ctx.approvalRouting.playwrightSnapshotPathTemplate,
-      toHaveScreenshotPathTemplate: ctx.approvalRouting.playwrightToHaveScreenshotPathTemplate
-    },
-    snapshotPathExists: existsSync
-  });
-  return targets.length === 1 ? targets[0]?.snapshotPath ?? null : null;
+  if (actualUrl.startsWith("/file/")) {
+    return decodeURIComponent(actualUrl.slice("/file/".length));
+  }
+  return actualUrl;
 }
 async function handleApiApprove(ctx, req) {
   try {
@@ -391,7 +420,7 @@ async function handleApiApprove(ctx, req) {
     if (actualUrl === void 0) {
       return Response.json({ success: false, error: "Actual image not found" }, { status: 409 });
     }
-    const snapshotPath = resolveApprovalTarget(ctx, test, retry, image);
+    const snapshotPath = resolveBaselineSnapshotPath(ctx.approvalRouting, test, retry, image);
     if (snapshotPath === null) {
       return Response.json({ success: false, error: APPROVAL_TARGET_ERROR }, { status: 409 });
     }
@@ -426,7 +455,7 @@ function createBulkApprovalUpdates(ctx) {
       if (actualUrl === void 0) {
         return [Promise.resolve({ kind: "unresolved" })];
       }
-      const snapshotPath = resolveApprovalTarget(ctx, test, lastRetry, imageName);
+      const snapshotPath = resolveBaselineSnapshotPath(ctx.approvalRouting, test, lastRetry, imageName);
       if (snapshotPath === null) {
         return [Promise.resolve({ kind: "unresolved" })];
       }
@@ -489,7 +518,7 @@ async function handleScreenshots(ctx, req) {
 }
 async function handleDist(ctx, req) {
   const path = new URL(req.url).pathname.slice("/dist/".length);
-  const filePath = join3(ctx.staticDir, path);
+  const filePath = join2(ctx.staticDir, path);
   const contentType = filePath.endsWith(".css") ? "text/css" : filePath.endsWith(".js") ? "application/javascript" : filePath.endsWith(".svelte") ? "text/plain" : "application/octet-stream";
   const file = await respondWithFile(filePath, contentType);
   return file ?? new Response("Not Found", { status: 404 });
@@ -517,6 +546,10 @@ function handleHttpRequest(ctx, req) {
   if (pathname.startsWith("/api/images/")) {
     return handleApiImages(req);
   }
+  const artifactResponse = handleArtifactRoute(ctx, req, pathname);
+  if (artifactResponse !== null) {
+    return artifactResponse;
+  }
   if (pathname.startsWith("/screenshots/")) {
     return handleScreenshots(ctx, req);
   }
@@ -537,12 +570,13 @@ function createReportData(options) {
     screenshotDir: options.screenshotDir ?? "./screenshots"
   };
 }
-function createHandlerContext(reportData, wsClients, currentRunIds, saveReport) {
+function createHandlerContext(reportData, wsClients, currentRunIds, saveReport, approvalRouting) {
   return {
     reportData,
     wsClients,
     currentRunIds,
-    saveReport
+    saveReport,
+    approvalRouting
   };
 }
 async function loadReport(reportPath, reportData) {
@@ -592,10 +626,6 @@ async function loadOfflineReports(offlineReportDir, reportData) {
     screenshotsBaseUrl: "/screenshots/"
   });
 }
-function resetReloadableReportData(reportData) {
-  reportData.tests = {};
-  reportData.isUpdateMode = false;
-}
 async function handleParsedWebSocketMessage(ctx, msg) {
   switch (msg.type) {
     case "test-begin": {
@@ -616,9 +646,15 @@ async function handleParsedWebSocketMessage(ctx, msg) {
       handleTestEnd(ctx, parsed);
       break;
     }
-    case "run-end":
-      await handleRunEnd(ctx, msg.data);
+    case "run-end": {
+      const parsed = safeParse(RunEndDataSchema, msg.data);
+      if (parsed === null) {
+        console.error("Invalid run-end message data", msg.data);
+        break;
+      }
+      await handleRunEnd(ctx, parsed);
       break;
+    }
     case "approve":
       handleApprove();
       break;
@@ -631,7 +667,7 @@ function createWebSocketMessageHandler(getHandlerContext) {
   return async function handleWebSocketMessage(message) {
     try {
       const parsed = JSON.parse(message);
-      const wsMessage = safeParse(WebSocketMessageSchema, parsed);
+      const wsMessage = safeParse(IncomingWebSocketMessageSchema, parsed);
       if (wsMessage === null) {
         console.error("Invalid WebSocket message: missing or invalid type", parsed);
         return;
@@ -647,16 +683,16 @@ async function resolveStaticDir(staticDir) {
   const currentDir = dirname3(fileURLToPath(import.meta.url));
   const candidates = staticDir === void 0 ? [
     currentDir,
-    join4(currentDir, "dist"),
-    join4(currentDir, "..", "dist"),
-    join4(currentDir, "..", "..", "dist"),
-    join4(currentDir, ".."),
-    join4(currentDir, "..", "..")
-  ] : [staticDir, join4(staticDir, "dist")];
+    join3(currentDir, "dist"),
+    join3(currentDir, "..", "dist"),
+    join3(currentDir, "..", "..", "dist"),
+    join3(currentDir, ".."),
+    join3(currentDir, "..", "..")
+  ] : [staticDir, join3(staticDir, "dist")];
   const resolvedCandidates = await Promise.all(
     candidates.map(async (candidate) => ({
       candidate,
-      exists: await fileExists(join4(candidate, "index.html"))
+      exists: await fileExists(join3(candidate, "index.html"))
     }))
   );
   const resolved = resolvedCandidates.find(({ exists }) => exists);
@@ -667,15 +703,19 @@ async function resolveStaticDir(staticDir) {
 }
 async function resolveReportPath(reportPath) {
   if (await isDirectory(reportPath)) {
-    return { reportFile: join4(reportPath, "report.json"), offlineReportDir: reportPath };
+    return { reportFile: join3(reportPath, "report.json"), offlineReportDir: reportPath };
   }
   return { reportFile: reportPath, offlineReportDir: dirname3(reportPath) };
 }
 function createRoutesContext(reportData, staticDir, saveReport, options) {
+  const artifactRoots = [reportData.screenshotDir, options.outputDir, options.playwrightSnapshotDir].filter(
+    (root) => root !== void 0 && root !== ""
+  );
   return {
     reportData,
     staticDir,
     saveReport,
+    artifactRoots,
     approvalRouting: {
       configDir: options.configDir ?? process.cwd(),
       playwrightTestDir: options.playwrightTestDir,
@@ -697,25 +737,16 @@ async function createServerApp(options = {}) {
     await writeJsonFile(reportFile, reportData);
   }
   const routesContext = createRoutesContext(reportData, staticDir, saveReport, options);
-  const getHandlerContext = () => createHandlerContext(reportData, wsClients, currentRunIds, saveReport);
+  const getHandlerContext = () => createHandlerContext(reportData, wsClients, currentRunIds, saveReport, routesContext.approvalRouting);
   const handleRequest = (req) => handleHttpRequest(routesContext, req);
   const handleWebSocketMessage = createWebSocketMessageHandler(getHandlerContext);
-  const reloadFromDisk = async () => {
-    resetReloadableReportData(reportData);
-    await loadReport(reportFile, reportData);
-    await loadOfflineReports(offlineReportDir, reportData);
-    broadcastToBrowsers(wsClients, { type: "sync", data: reportData });
-  };
-  await reloadFromDisk();
-  const close = await watchReportArtifacts({
-    offlineReportDir,
-    screenshotDir: reportData.screenshotDir,
-    scheduleRefresh: createDebouncedRefresh(reloadFromDisk)
-  });
+  await loadReport(reportFile, reportData);
+  await loadOfflineReports(offlineReportDir, reportData);
   return {
     port,
     wsClients,
-    close,
+    close: () => {
+    },
     handleRequest,
     handleWebSocketMessage
   };
@@ -896,7 +927,7 @@ function attachWebSocketServer(server, app) {
   });
 }
 async function listen(server, port) {
-  await new Promise((resolve, reject) => {
+  await new Promise((resolve3, reject) => {
     const onError = (error) => {
       server.off("error", onError);
       reject(error);
@@ -904,7 +935,7 @@ async function listen(server, port) {
     server.on("error", onError);
     server.listen(port, () => {
       server.off("error", onError);
-      resolve();
+      resolve3();
     });
   });
 }