@openclaw/diffs 2026.5.7-beta.1 → 2026.5.9-beta.1

package/dist/index.js

@@ -10,12 +10,15 @@ import { normalizeLowercaseStringOrEmpty, normalizeOptionalString } from "opencl
 import crypto from "node:crypto";
 import fs from "node:fs/promises";
 import { fileURLToPath } from "node:url";
+import { root, writeExternalFileWithinRoot } from "openclaw/plugin-sdk/security-runtime";
+import { stringEnum } from "openclaw/plugin-sdk/channel-actions";
 import { formatErrorMessage } from "openclaw/plugin-sdk/error-runtime";
 import { Type } from "typebox";
 import { constants } from "node:fs";
 import { chromium } from "playwright-core";
 import { RegisteredCustomThemes, ResolvedThemes, ResolvingThemes, parsePatchFiles, resolveLanguage } from "@pierre/diffs";
 import { preloadFileDiff, preloadMultiFileDiff } from "@pierre/diffs/ssr";
+import { readJsonFileWithFallback } from "openclaw/plugin-sdk/json-store";
 //#region extensions/diffs/src/types.ts
 const DIFF_LAYOUTS = ["unified", "split"];
 const DIFF_MODES = [
@@ -597,8 +600,9 @@ var DiffArtifactStore = class {
 			htmlPath,
 			...params.context ? { context: params.context } : {}
 		};
-		await fs.mkdir(artifactDir, { recursive: true });
-		await fs.writeFile(htmlPath, params.html, "utf8");
+		const root = await this.artifactRoot();
+		await root.mkdir(id);
+		await root.write(path.posix.join(id, "viewer.html"), params.html);
 		await this.writeMeta(meta);
 		this.scheduleCleanup();
 		return meta;
@@ -617,7 +621,7 @@ var DiffArtifactStore = class {
 		const meta = await this.readMeta(id);
 		if (!meta) throw new Error(`Diff artifact not found: ${id}`);
 		const htmlPath = this.normalizeStoredPath(meta.htmlPath, "htmlPath");
-		return await fs.readFile(htmlPath, "utf8");
+		return await (await this.artifactRoot()).readText(this.relativeStoredPath(htmlPath));
 	}
 	async updateFilePath(id, filePath) {
 		const meta = await this.readMeta(id);
@@ -654,7 +658,7 @@ var DiffArtifactStore = class {
 			filePath: this.normalizeStoredPath(filePath, "filePath"),
 			...params.context ? { context: params.context } : {}
 		};
-		await fs.mkdir(artifactDir, { recursive: true });
+		await (await this.artifactRoot()).mkdir(id);
 		await this.writeStandaloneMeta(meta);
 		this.scheduleCleanup();
 		return {
@@ -671,10 +675,9 @@ var DiffArtifactStore = class {
 		this.maybeCleanupExpired();
 	}
 	async cleanupExpired() {
-		await this.ensureRoot();
-		const entries = await fs.readdir(this.rootDir, { withFileTypes: true }).catch(() => []);
+		const entries = await (await this.artifactRoot()).list("", { withFileTypes: true }).catch(() => []);
 		const now = Date.now();
-		await Promise.all(entries.filter((entry) => entry.isDirectory()).map(async (entry) => {
+		await Promise.all(entries.filter((entry) => entry.isDirectory).map(async (entry) => {
 			const id = entry.name;
 			const meta = await this.readMeta(id);
 			if (meta) {
@@ -686,15 +689,16 @@ var DiffArtifactStore = class {
 				if (isExpired(standaloneMeta)) await this.deleteArtifact(id);
 				return;
 			}
-			const artifactPath = this.artifactDir(id);
-			const stat = await fs.stat(artifactPath).catch(() => null);
-			if (!stat) return;
-			if (now - stat.mtimeMs > SWEEP_FALLBACK_AGE_MS) await this.deleteArtifact(id);
+			if (now - entry.mtimeMs > SWEEP_FALLBACK_AGE_MS) await this.deleteArtifact(id);
 		}));
 	}
 	async ensureRoot() {
 		await fs.mkdir(this.rootDir, { recursive: true });
 	}
+	async artifactRoot() {
+		await this.ensureRoot();
+		return await root(this.rootDir);
+	}
 	maybeCleanupExpired() {
 		const now = Date.now();
 		if (this.cleanupInFlight || now < this.nextCleanupAt) return;
@@ -740,15 +744,12 @@ var DiffArtifactStore = class {
 			return null;
 		}
 	}
-	metaFilePath(id, fileName) {
-		return path.join(this.artifactDir(id), fileName);
-	}
 	async writeJsonMeta(id, fileName, data) {
-		await fs.writeFile(this.metaFilePath(id, fileName), JSON.stringify(data, null, 2), "utf8");
+		await (await this.artifactRoot()).writeJson(path.posix.join(id, fileName), data, { space: 2 });
 	}
 	async readJsonMeta(id, fileName, context) {
 		try {
-			const raw = await fs.readFile(this.metaFilePath(id, fileName), "utf8");
+			const raw = await (await this.artifactRoot()).readText(path.posix.join(id, fileName));
 			return JSON.parse(raw);
 		} catch (error) {
 			if (isFileNotFound(error)) return null;
@@ -772,6 +773,9 @@ var DiffArtifactStore = class {
 		this.assertWithinRoot(candidate, label);
 		return candidate;
 	}
+	relativeStoredPath(storedPath) {
+		return path.relative(this.rootDir, this.normalizeStoredPath(storedPath, "path")).split(path.sep).join(path.posix.sep);
+	}
 	assertWithinRoot(candidate, label = "path") {
 		const relative = path.relative(this.rootDir, candidate);
 		if (relative === "" || !relative.startsWith(`..${path.sep}`) && relative !== ".." && !path.isAbsolute(relative)) return;
@@ -790,7 +794,8 @@ function isExpired(meta) {
 	return Date.now() >= expiresAt;
 }
 function isFileNotFound(error) {
-	return error instanceof Error && "code" in error && error.code === "ENOENT";
+	const code = error instanceof Error && "code" in error ? error.code : void 0;
+	return code === "ENOENT" || code === "not-found";
 }
 function normalizeArtifactContext(value) {
 	if (!value || typeof value !== "object" || Array.isArray(value)) return;
@@ -819,7 +824,6 @@ var PlaywrightDiffScreenshotter = class {
 		this.browserIdleMs = params.browserIdleMs ?? DEFAULT_BROWSER_IDLE_MS;
 	}
 	async screenshotHtml(params) {
-		await fs.mkdir(path.dirname(params.outputPath), { recursive: true });
 		const lease = await acquireSharedBrowser({
 			config: this.config,
 			idleMs: this.browserIdleMs
@@ -918,16 +922,22 @@ var PlaywrightDiffScreenshotter = class {
 					const estimatedPixels = pdfWidth * pdfHeight;
 					const estimatedPages = Math.ceil(pdfHeight / PDF_REFERENCE_PAGE_HEIGHT_PX);
 					if (estimatedPixels > params.image.maxPixels || estimatedPages > MAX_PDF_PAGES) throw new Error(IMAGE_SIZE_LIMIT_ERROR);
-					await page.pdf({
-						path: params.outputPath,
-						width: `${pdfWidth}px`,
-						height: `${pdfHeight}px`,
-						printBackground: true,
-						margin: {
-							top: "0",
-							right: "0",
-							bottom: "0",
-							left: "0"
+					const pageForPdf = page;
+					await writeExternalArtifactFile({
+						outputPath: params.outputPath,
+						write: async (tempPath) => {
+							await pageForPdf.pdf({
+								path: tempPath,
+								width: `${pdfWidth}px`,
+								height: `${pdfHeight}px`,
+								printBackground: true,
+								margin: {
+									top: "0",
+									right: "0",
+									bottom: "0",
+									left: "0"
+								}
+							});
 						}
 					});
 					return params.outputPath;
@@ -956,15 +966,21 @@ var PlaywrightDiffScreenshotter = class {
 					}
 					throw new Error(IMAGE_SIZE_LIMIT_ERROR);
 				}
-				await page.screenshot({
-					path: params.outputPath,
-					type: "png",
-					scale: "device",
-					clip: {
-						x,
-						y,
-						width: cssWidth,
-						height: cssHeight
+				const pageForScreenshot = page;
+				await writeExternalArtifactFile({
+					outputPath: params.outputPath,
+					write: async (tempPath) => {
+						await pageForScreenshot.screenshot({
+							path: tempPath,
+							type: "png",
+							scale: "device",
+							clip: {
+								x,
+								y,
+								width: cssWidth,
+								height: cssHeight
+							}
+						});
 					}
 				});
 				return params.outputPath;
@@ -980,6 +996,15 @@ var PlaywrightDiffScreenshotter = class {
 		}
 	}
 };
+async function writeExternalArtifactFile(params) {
+	const rootDir = path.dirname(params.outputPath);
+	await fs.mkdir(rootDir, { recursive: true });
+	await writeExternalFileWithinRoot({
+		rootDir,
+		path: path.basename(params.outputPath),
+		write: params.write
+	});
+}
 function injectBaseHref(html) {
 	if (html.includes("<base ")) return html;
 	return html.replace("<head>", `<head><base href="${LOCAL_VIEWER_BASE_HREF}" />`);
@@ -1169,13 +1194,8 @@ async function isExecutable(candidate) {
 //#endregion
 //#region extensions/diffs/src/language-hints.ts
 const PASSTHROUGH_LANGUAGE_HINTS = new Set(["ansi", "text"]);
-function normalizeOptionalString$1(value) {
-	if (typeof value !== "string") return;
-	const trimmed = value.trim();
-	return trimmed ? trimmed : void 0;
-}
 async function normalizeSupportedLanguageHint(value) {
-	const normalized = normalizeOptionalString$1(value);
+	const normalized = normalizeOptionalString(value);
 	if (!normalized) return;
 	if (PASSTHROUGH_LANGUAGE_HINTS.has(normalized)) return normalized;
 	try {
@@ -1246,9 +1266,9 @@ function createThemeLoader(themeName, themeSpecifier) {
 	let cachedTheme;
 	return async () => {
 		if (cachedTheme) return cachedTheme;
-		const themePath = themeRequire.resolve(themeSpecifier);
+		const { value: theme } = await readJsonFileWithFallback(themeRequire.resolve(themeSpecifier), {});
 		cachedTheme = {
-			...JSON.parse(await fs.readFile(themePath, "utf8")),
+			...theme,
 			name: themeName
 		};
 		return cachedTheme;
@@ -1677,14 +1697,6 @@ const MAX_PATCH_BYTES = 2 * 1024 * 1024;
 const MAX_TITLE_BYTES = 1024;
 const MAX_PATH_BYTES = 2048;
 const MAX_LANG_BYTES = 128;
-function stringEnum(values, description, options = {}) {
-	return Type.Unsafe({
-		type: "string",
-		enum: [...values],
-		description,
-		...options
-	});
-}
 const DiffsToolSchema = Type.Object({
 	before: Type.Optional(Type.String({ description: "Original text content." })),
 	after: Type.Optional(Type.String({ description: "Updated text content." })),
@@ -1704,11 +1716,11 @@ const DiffsToolSchema = Type.Object({
 		description: "Optional title for the rendered diff.",
 		maxLength: MAX_TITLE_BYTES
 	})),
-	mode: Type.Optional(stringEnum(DIFF_MODES, "Output mode: view, file, image (deprecated alias for file), or both. Default: both.")),
-	theme: Type.Optional(stringEnum(DIFF_THEMES, "Viewer theme. Default: dark.")),
-	layout: Type.Optional(stringEnum(DIFF_LAYOUTS, "Diff layout. Default: unified.")),
-	fileQuality: Type.Optional(stringEnum(DIFF_IMAGE_QUALITY_PRESETS, "File quality preset: standard, hq, or print.")),
-	fileFormat: Type.Optional(stringEnum(DIFF_OUTPUT_FORMATS, "Rendered file format: png or pdf.")),
+	mode: Type.Optional(stringEnum(DIFF_MODES, { description: "Output mode: view, file, image (deprecated alias for file), or both. Default: both." })),
+	theme: Type.Optional(stringEnum(DIFF_THEMES, { description: "Viewer theme. Default: dark." })),
+	layout: Type.Optional(stringEnum(DIFF_LAYOUTS, { description: "Diff layout. Default: unified." })),
+	fileQuality: Type.Optional(stringEnum(DIFF_IMAGE_QUALITY_PRESETS, { description: "File quality preset: standard, hq, or print." })),
+	fileFormat: Type.Optional(stringEnum(DIFF_OUTPUT_FORMATS, { description: "Rendered file format: png or pdf." })),
 	fileScale: Type.Optional(Type.Number({
 		description: "Optional rendered-file device scale factor override (1-4).",
 		minimum: 1,
@@ -1720,9 +1732,15 @@ const DiffsToolSchema = Type.Object({
 		maximum: 2400
 	})),
 	/** @deprecated Use fileQuality. */
-	imageQuality: Type.Optional(stringEnum(DIFF_IMAGE_QUALITY_PRESETS, "Deprecated alias for fileQuality.", { deprecated: true })),
+	imageQuality: Type.Optional(stringEnum(DIFF_IMAGE_QUALITY_PRESETS, {
+		description: "Deprecated alias for fileQuality.",
+		deprecated: true
+	})),
 	/** @deprecated Use fileFormat. */
-	imageFormat: Type.Optional(stringEnum(DIFF_OUTPUT_FORMATS, "Deprecated alias for fileFormat.", { deprecated: true })),
+	imageFormat: Type.Optional(stringEnum(DIFF_OUTPUT_FORMATS, {
+		description: "Deprecated alias for fileFormat.",
+		deprecated: true
+	})),
 	/** @deprecated Use fileScale. */
 	imageScale: Type.Optional(Type.Number({
 		description: "Deprecated alias for fileScale.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openclaw/diffs",
-  "version": "2026.5.7-beta.1",
+  "version": "2026.5.9-beta.1",
   "description": "OpenClaw diff viewer plugin",
   "repository": {
     "type": "git",
@@ -14,7 +14,7 @@
     "@pierre/diffs": "1.1.20",
     "@pierre/theme": "0.0.29",
     "playwright-core": "1.59.1",
-    "typebox": "1.1.37"
+    "typebox": "1.1.38"
   },
   "devDependencies": {
     "@openclaw/plugin-sdk": "workspace:*"
@@ -30,10 +30,10 @@
       "minHostVersion": ">=2026.4.30"
     },
     "compat": {
-      "pluginApi": ">=2026.5.7-beta.1"
+      "pluginApi": ">=2026.5.9-beta.1"
     },
     "build": {
-      "openclawVersion": "2026.5.7-beta.1",
+      "openclawVersion": "2026.5.9-beta.1",
       "staticAssets": [
         {
           "source": "./assets/viewer-runtime.js",
@@ -56,7 +56,7 @@
     "skills/**"
   ],
   "peerDependencies": {
-    "openclaw": ">=2026.5.7-beta.1"
+    "openclaw": ">=2026.5.9-beta.1"
   },
   "peerDependenciesMeta": {
     "openclaw": {