@openclaw/qqbot 2026.5.2 → 2026.5.3-beta.1

package/src/engine/utils/diagnostics.ts

@@ -1,109 +0,0 @@
-/**
- * Gateway startup diagnostics — extracted from utils/platform.ts.
- *
- * Depends on utils/platform.ts for detection functions, but no plugin-sdk.
- */
-
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-import { debugLog } from "./log.js";
-import {
-  getHomeDir,
-  getTempDir,
-  getQQBotDataDir,
-  getPlatform,
-  isWindows,
-  detectFfmpeg,
-  checkSilkWasmAvailable,
-} from "./platform.js";
-
-interface DiagnosticReport {
-  platform: string;
-  arch: string;
-  nodeVersion: string;
-  homeDir: string;
-  tempDir: string;
-  dataDir: string;
-  ffmpeg: string | null;
-  silkWasm: boolean;
-  warnings: string[];
-}
-
-/**
- * Run startup diagnostics and return an environment report.
- * Called during gateway startup to log environment details and warnings.
- */
-export async function runDiagnostics(): Promise<DiagnosticReport> {
-  const warnings: string[] = [];
-
-  const platform = `${process.platform} (${os.release()})`;
-  const arch = process.arch;
-  const nodeVersion = process.version;
-  const homeDir = getHomeDir();
-  const tempDir = getTempDir();
-  const dataDir = getQQBotDataDir();
-
-  const ffmpegPath = await detectFfmpeg();
-  if (!ffmpegPath) {
-    warnings.push(
-      isWindows()
-        ? "⚠️ ffmpeg is not installed. Audio/video conversion will be limited. Install it with choco install ffmpeg, scoop install ffmpeg, or from https://ffmpeg.org."
-        : getPlatform() === "darwin"
-          ? "⚠️ ffmpeg is not installed. Audio/video conversion will be limited. Install it with brew install ffmpeg."
-          : "⚠️ ffmpeg is not installed. Audio/video conversion will be limited. Install it with sudo apt install ffmpeg or sudo yum install ffmpeg.",
-    );
-  }
-
-  const silkWasm = await checkSilkWasmAvailable();
-  if (!silkWasm) {
-    warnings.push(
-      "⚠️ silk-wasm is unavailable. QQ voice send/receive will not work. Ensure Node.js >= 16 and WASM support are available.",
-    );
-  }
-
-  try {
-    const testFile = path.join(dataDir, ".write-test");
-    fs.writeFileSync(testFile, "test");
-    fs.unlinkSync(testFile);
-  } catch {
-    warnings.push(`⚠️ Data directory is not writable: ${dataDir}. Check filesystem permissions.`);
-  }
-
-  if (isWindows()) {
-    if (/[\u4e00-\u9fa5]/.test(homeDir) || homeDir.includes(" ")) {
-      warnings.push(
-        `⚠️ Home directory contains Chinese characters or spaces: ${homeDir}. Some tools may fail. Consider setting QQBOT_DATA_DIR to an ASCII-only path.`,
-      );
-    }
-  }
-
-  const report: DiagnosticReport = {
-    platform,
-    arch,
-    nodeVersion,
-    homeDir,
-    tempDir,
-    dataDir,
-    ffmpeg: ffmpegPath,
-    silkWasm,
-    warnings,
-  };
-
-  debugLog("=== QQBot Environment Diagnostics ===");
-  debugLog(`  Platform: ${platform} (${arch})`);
-  debugLog(`  Node: ${nodeVersion}`);
-  debugLog(`  Home: ${homeDir}`);
-  debugLog(`  Data dir: ${dataDir}`);
-  debugLog(`  ffmpeg: ${ffmpegPath ?? "not installed"}`);
-  debugLog(`  silk-wasm: ${silkWasm ? "available" : "unavailable"}`);
-  if (warnings.length > 0) {
-    debugLog("  --- Warnings ---");
-    for (const w of warnings) {
-      debugLog(`  ${w}`);
-    }
-  }
-  debugLog("======================");
-
-  return report;
-}

package/src/engine/utils/file-utils.test.ts

@@ -1,72 +0,0 @@
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-
-const adapterMocks = vi.hoisted(() => ({
-  fetchMedia: vi.fn(),
-}));
-
-vi.mock("../adapter/index.js", () => ({
-  getPlatformAdapter: () => ({
-    fetchMedia: (...args: unknown[]) => adapterMocks.fetchMedia(...args),
-  }),
-}));
-
-import { QQBOT_MEDIA_SSRF_POLICY, downloadFile } from "./file-utils.js";
-
-describe("qqbot file-utils downloadFile", () => {
-  let tempDir: string;
-
-  beforeEach(async () => {
-    adapterMocks.fetchMedia.mockReset();
-    tempDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), "qqbot-file-utils-"));
-  });
-
-  afterEach(async () => {
-    await fs.promises.rm(tempDir, { recursive: true, force: true });
-  });
-
-  it("downloads through the guarded media adapter with the qqbot SSRF policy", async () => {
-    adapterMocks.fetchMedia.mockResolvedValueOnce({
-      buffer: Buffer.from("image-bytes"),
-      contentType: "image/png",
-      fileName: "remote.png",
-    });
-
-    const savedPath = await downloadFile(
-      "https://media.qq.com/assets/photo.png",
-      tempDir,
-      "photo.png",
-    );
-
-    expect(savedPath).toBeTruthy();
-    expect(savedPath).toMatch(/photo_\d+_[0-9a-f]{6}\.png$/);
-    expect(await fs.promises.readFile(savedPath!, "utf8")).toBe("image-bytes");
-    expect(adapterMocks.fetchMedia).toHaveBeenCalledWith({
-      url: "https://media.qq.com/assets/photo.png",
-      filePathHint: "photo.png",
-      ssrfPolicy: QQBOT_MEDIA_SSRF_POLICY,
-    });
-    expect(QQBOT_MEDIA_SSRF_POLICY).toEqual({
-      hostnameAllowlist: [
-        "*.qpic.cn",
-        "*.qq.com",
-        "*.weiyun.com",
-        "*.qq.com.cn",
-        "*.ugcimg.cn",
-        "*.myqcloud.com",
-        "*.tencentcos.cn",
-        "*.tencentcos.com",
-      ],
-      allowRfc2544BenchmarkRange: true,
-    });
-  });
-
-  it("rejects non-HTTPS URLs before attempting a fetch", async () => {
-    const savedPath = await downloadFile("http://media.qq.com/assets/photo.png", tempDir);
-
-    expect(savedPath).toBeNull();
-    expect(adapterMocks.fetchMedia).not.toHaveBeenCalled();
-  });
-});

package/src/engine/utils/file-utils.ts

@@ -1,225 +0,0 @@
-import crypto from "node:crypto";
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { getPlatformAdapter } from "../adapter/index.js";
-import type { SsrfPolicyConfig } from "../adapter/types.js";
-import { MediaFileType } from "../types.js";
-import { formatErrorMessage } from "./format.js";
-import { normalizeLowercaseStringOrEmpty, normalizeOptionalString } from "./string-normalize.js";
-
-/** Maximum file size accepted by the QQ Bot one-shot upload API (base64 direct). */
-export const MAX_UPLOAD_SIZE = 20 * 1024 * 1024;
-
-/** Absolute upper bound enforced on the chunked upload path (matches server policy). */
-const CHUNKED_UPLOAD_MAX_SIZE = 100 * 1024 * 1024;
-
-/** Threshold used to treat an upload as a large file (dispatch to chunked path). */
-export const LARGE_FILE_THRESHOLD = 5 * 1024 * 1024;
-
-/**
- * Per-{@link MediaFileType} upload metadata: the QQ Open Platform size
- * ceiling and the Chinese display name used in user-facing error messages.
- *
- * Keyed by the enum value so call sites read as
- * `MEDIA_FILE_TYPE_INFO[MediaFileType.IMAGE].maxSize`, and adding a new
- * type forces both fields to be supplied in a single place.
- */
-const MEDIA_FILE_TYPE_INFO: Record<MediaFileType, { maxSize: number; name: string }> = {
-  [MediaFileType.IMAGE]: { maxSize: 30 * 1024 * 1024, name: "图片" },
-  [MediaFileType.VIDEO]: { maxSize: 100 * 1024 * 1024, name: "视频" },
-  [MediaFileType.VOICE]: { maxSize: 20 * 1024 * 1024, name: "语音" },
-  [MediaFileType.FILE]: { maxSize: 100 * 1024 * 1024, name: "文件" },
-};
-
-/** Return the Chinese display name for a media file type code. Defaults to "文件". */
-export function getFileTypeName(fileType: number): string {
-  return MEDIA_FILE_TYPE_INFO[fileType as MediaFileType]?.name ?? "文件";
-}
-
-/** Return the upload ceiling for a given media file type. Defaults to 100MB. */
-export function getMaxUploadSize(fileType: number): number {
-  return MEDIA_FILE_TYPE_INFO[fileType as MediaFileType]?.maxSize ?? CHUNKED_UPLOAD_MAX_SIZE;
-}
-
-const QQBOT_MEDIA_HOSTNAME_ALLOWLIST = [
-  // QQ rich media
-  "*.qpic.cn",
-  "*.qq.com",
-  "*.weiyun.com",
-  "*.qq.com.cn",
-
-  // QQ Bot
-  "*.ugcimg.cn",
-
-  // Tencent Cloud COS
-  "*.myqcloud.com",
-  "*.tencentcos.cn",
-  "*.tencentcos.com",
-];
-
-export const QQBOT_MEDIA_SSRF_POLICY: SsrfPolicyConfig = {
-  hostnameAllowlist: QQBOT_MEDIA_HOSTNAME_ALLOWLIST,
-  allowRfc2544BenchmarkRange: true,
-};
-
-/** Result of local file-size validation. */
-interface FileSizeCheckResult {
-  ok: boolean;
-  size: number;
-  error?: string;
-}
-
-/** Validate that a file is within the allowed upload size. */
-export function checkFileSize(filePath: string, maxSize = MAX_UPLOAD_SIZE): FileSizeCheckResult {
-  try {
-    const stat = fs.statSync(filePath);
-    if (stat.size > maxSize) {
-      const sizeMB = (stat.size / (1024 * 1024)).toFixed(1);
-      const limitMB = (maxSize / (1024 * 1024)).toFixed(0);
-      return {
-        ok: false,
-        size: stat.size,
-        error: `File is too large (${sizeMB}MB); QQ Bot API limit is ${limitMB}MB`,
-      };
-    }
-    return { ok: true, size: stat.size };
-  } catch (err) {
-    return {
-      ok: false,
-      size: 0,
-      error: `Failed to read file metadata: ${formatErrorMessage(err)}`,
-    };
-  }
-}
-
-/** Read file contents asynchronously. */
-export async function readFileAsync(filePath: string): Promise<Buffer> {
-  return fs.promises.readFile(filePath);
-}
-
-/** Check file readability asynchronously. */
-export async function fileExistsAsync(filePath: string): Promise<boolean> {
-  try {
-    await fs.promises.access(filePath, fs.constants.R_OK);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/** Format a byte count into a human-readable size string. */
-export function formatFileSize(bytes: number): string {
-  if (bytes < 1024) {
-    return `${bytes}B`;
-  }
-  if (bytes < 1024 * 1024) {
-    return `${(bytes / 1024).toFixed(1)}KB`;
-  }
-  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
-}
-
-/** Infer a MIME type from the file extension. */
-export function getMimeType(filePath: string): string {
-  const ext = normalizeLowercaseStringOrEmpty(path.extname(filePath));
-  return MIME_TYPES[ext] ?? "application/octet-stream";
-}
-
-/** Canonical ext → MIME table. Single source of truth. */
-const MIME_TYPES: Record<string, string> = {
-  ".jpg": "image/jpeg",
-  ".jpeg": "image/jpeg",
-  ".png": "image/png",
-  ".gif": "image/gif",
-  ".webp": "image/webp",
-  ".bmp": "image/bmp",
-  ".mp4": "video/mp4",
-  ".mov": "video/quicktime",
-  ".avi": "video/x-msvideo",
-  ".mkv": "video/x-matroska",
-  ".webm": "video/webm",
-  ".pdf": "application/pdf",
-  ".doc": "application/msword",
-  ".docx": "application/vnd.openxmlformats-officedocument.wordprocessingml.document",
-  ".xls": "application/vnd.ms-excel",
-  ".xlsx": "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
-  ".zip": "application/zip",
-  ".tar": "application/x-tar",
-  ".gz": "application/gzip",
-  ".txt": "text/plain",
-};
-
-/** Extensions accepted as image uploads by the QQ Bot media pipeline. */
-const IMAGE_EXTENSIONS = new Set([".jpg", ".jpeg", ".png", ".gif", ".webp", ".bmp"]);
-
-/**
- * Return the image MIME type for a local file path, or `null` if the
- * extension is not in the supported image whitelist.
- *
- * Use this instead of `getMimeType` when the caller must enforce
- * "image formats only" as a business rule (e.g. constructing a
- * `data:image/...;base64,` URL).
- */
-export function getImageMimeType(filePath: string): string | null {
-  const ext = normalizeLowercaseStringOrEmpty(path.extname(filePath));
-  if (!IMAGE_EXTENSIONS.has(ext)) {
-    return null;
-  }
-  return MIME_TYPES[ext] ?? null;
-}
-
-/** Download a remote file into a local directory. */
-export async function downloadFile(
-  url: string,
-  destDir: string,
-  originalFilename?: string,
-): Promise<string | null> {
-  try {
-    let parsedUrl: URL;
-    try {
-      parsedUrl = new URL(url);
-    } catch {
-      return null;
-    }
-    if (parsedUrl.protocol !== "https:") {
-      return null;
-    }
-
-    if (!fs.existsSync(destDir)) {
-      fs.mkdirSync(destDir, { recursive: true });
-    }
-
-    const fetched = await getPlatformAdapter().fetchMedia({
-      url: parsedUrl.toString(),
-      filePathHint: originalFilename,
-      ssrfPolicy: QQBOT_MEDIA_SSRF_POLICY,
-    });
-
-    let filename = normalizeOptionalString(originalFilename) ?? "";
-    if (!filename) {
-      filename =
-        (normalizeOptionalString(fetched.fileName) ?? path.basename(parsedUrl.pathname)) ||
-        "download";
-    }
-
-    const ts = Date.now();
-    const ext = path.extname(filename);
-    const base = path.basename(filename, ext) || "file";
-    const rand = crypto.randomBytes(3).toString("hex");
-    const safeFilename = `${base}_${ts}_${rand}${ext}`;
-
-    const destPath = path.join(destDir, safeFilename);
-    await fs.promises.writeFile(destPath, fetched.buffer);
-    return destPath;
-  } catch (err) {
-    console.error(
-      `[qqbot:downloadFile] FAILED url=${url.slice(0, 120)} error=${err instanceof Error ? err.message : String(err)}`,
-    );
-    if (err instanceof Error && err.stack) {
-      console.error(`[qqbot:downloadFile] stack=${err.stack.split("\n").slice(0, 3).join(" | ")}`);
-    }
-    if (err instanceof Error && err.cause) {
-      console.error(`[qqbot:downloadFile] cause=${formatErrorMessage(err.cause)}`);
-    }
-    return null;
-  }
-}

package/src/engine/utils/format.test.ts

@@ -1,68 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { formatErrorMessage, formatDuration } from "./format.js";
-
-describe("engine/utils/format", () => {
-  describe("formatErrorMessage", () => {
-    it("extracts message from Error instances", () => {
-      expect(formatErrorMessage(new Error("boom"))).toBe("boom");
-    });
-
-    it("returns strings as-is", () => {
-      expect(formatErrorMessage("plain text")).toBe("plain text");
-    });
-
-    it("traverses the .cause chain", () => {
-      const inner = new Error("inner");
-      const outer = new Error("outer", { cause: inner });
-      expect(formatErrorMessage(outer)).toBe("outer | inner");
-    });
-
-    it("handles string cause", () => {
-      const err = new Error("outer", { cause: "string cause" });
-      expect(formatErrorMessage(err)).toBe("outer | string cause");
-    });
-
-    it("stringifies numbers", () => {
-      expect(formatErrorMessage(42)).toBe("42");
-    });
-
-    it("stringifies null", () => {
-      expect(formatErrorMessage(null)).toBe("null");
-    });
-
-    it("stringifies undefined", () => {
-      expect(formatErrorMessage(undefined)).toBe("undefined");
-    });
-
-    it("JSON-stringifies plain objects", () => {
-      expect(formatErrorMessage({ code: 500 })).toBe('{"code":500}');
-    });
-  });
-
-  describe("formatDuration", () => {
-    it("formats zero", () => {
-      expect(formatDuration(0)).toBe("0s");
-    });
-
-    it("formats sub-minute durations as seconds", () => {
-      expect(formatDuration(45_000)).toBe("45s");
-    });
-
-    it("formats exactly 60 seconds as 1m", () => {
-      expect(formatDuration(60_000)).toBe("1m");
-    });
-
-    it("formats mixed minutes and seconds", () => {
-      expect(formatDuration(90_000)).toBe("1m 30s");
-    });
-
-    it("formats exact minutes without trailing seconds", () => {
-      expect(formatDuration(300_000)).toBe("5m");
-    });
-
-    it("rounds sub-second values", () => {
-      expect(formatDuration(1_499)).toBe("1s");
-      expect(formatDuration(1_500)).toBe("2s");
-    });
-  });
-});

package/src/engine/utils/format.ts

@@ -1,70 +0,0 @@
-/**
- * General formatting and string utilities.
- * 通用格式化与字符串工具。
- *
- * Pure utility functions with zero external dependencies.
- * Replaces `openclaw/plugin-sdk/error-runtime` and `text-runtime`
- * helpers for use inside engine/.
- *
- * NOTE: The framework `formatErrorMessage` also applies `redactSensitiveText()`
- * for token masking. We intentionally omit that here — the framework's log
- * pipeline handles redaction at a higher level.
- */
-
-/**
- * Format any error object into a readable string.
- * 将任意错误对象格式化为可读字符串。
- *
- * Traverses the `.cause` chain for nested Error objects to include
- * the full error context (e.g. network errors wrapped inside HTTP errors).
- */
-export function formatErrorMessage(err: unknown): string {
-  if (err instanceof Error) {
-    let formatted = err.message || err.name || "Error";
-    let cause: unknown = err.cause;
-    const seen = new Set<unknown>([err]);
-    while (cause && !seen.has(cause)) {
-      seen.add(cause);
-      if (cause instanceof Error) {
-        if (cause.message) {
-          formatted += ` | ${cause.message}`;
-        }
-        cause = cause.cause;
-      } else if (typeof cause === "string") {
-        formatted += ` | ${cause}`;
-        break;
-      } else {
-        break;
-      }
-    }
-    return formatted;
-  }
-  if (typeof err === "string") {
-    return err;
-  }
-  if (
-    err === null ||
-    err === undefined ||
-    typeof err === "number" ||
-    typeof err === "boolean" ||
-    typeof err === "bigint"
-  ) {
-    return String(err);
-  }
-  try {
-    return JSON.stringify(err);
-  } catch {
-    return Object.prototype.toString.call(err);
-  }
-}
-
-/** Format a millisecond duration into a human-readable string (e.g. "5m 30s"). */
-export function formatDuration(durationMs: number): string {
-  const seconds = Math.round(durationMs / 1000);
-  if (seconds < 60) {
-    return `${seconds}s`;
-  }
-  const minutes = Math.floor(seconds / 60);
-  const remainSeconds = seconds % 60;
-  return remainSeconds > 0 ? `${minutes}m ${remainSeconds}s` : `${minutes}m`;
-}

package/src/engine/utils/image-size.test.ts

@@ -1,158 +0,0 @@
-import { Buffer } from "node:buffer";
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-const adapterMocks = vi.hoisted(() => ({
-  fetchMedia: vi.fn(),
-}));
-
-vi.mock("../adapter/index.js", () => ({
-  getPlatformAdapter: () => ({
-    fetchMedia: (...args: unknown[]) => adapterMocks.fetchMedia(...args),
-  }),
-}));
-
-import { getImageSizeFromUrl, parseImageSize } from "./image-size.js";
-
-/** Build a minimal valid PNG header with the given dimensions. */
-function buildPngHeader(width: number, height: number): Buffer {
-  const buf = Buffer.alloc(24);
-  // PNG signature
-  buf[0] = 0x89;
-  buf[1] = 0x50;
-  buf[2] = 0x4e;
-  buf[3] = 0x47;
-  buf[4] = 0x0d;
-  buf[5] = 0x0a;
-  buf[6] = 0x1a;
-  buf[7] = 0x0a;
-  // IHDR chunk length
-  buf.writeUInt32BE(13, 8);
-  // "IHDR"
-  buf.write("IHDR", 12, "ascii");
-  // Width and height
-  buf.writeUInt32BE(width, 16);
-  buf.writeUInt32BE(height, 20);
-  return buf;
-}
-
-describe("getImageSizeFromUrl", () => {
-  beforeEach(() => {
-    adapterMocks.fetchMedia.mockReset();
-  });
-
-  describe("fetchMedia options contract", () => {
-    it("passes maxBytes, maxRedirects, ssrfPolicy, and headers", async () => {
-      adapterMocks.fetchMedia.mockResolvedValueOnce({
-        buffer: buildPngHeader(800, 600),
-        contentType: "image/png",
-      });
-
-      await getImageSizeFromUrl("https://cdn.example.com/photo.png");
-
-      expect(adapterMocks.fetchMedia).toHaveBeenCalledOnce();
-      const opts = adapterMocks.fetchMedia.mock.calls[0][0];
-
-      expect(opts.url).toBe("https://cdn.example.com/photo.png");
-      expect(opts.maxBytes).toBe(65_536);
-      expect(opts.maxRedirects).toBe(0);
-      // Generic public-network-only policy: no hostname allowlist
-      expect(opts.ssrfPolicy).toEqual({});
-      expect(opts.requestInit.headers).toEqual({
-        Range: "bytes=0-65535",
-        "User-Agent": "QQBot-Image-Size-Detector/1.0",
-      });
-    });
-
-    it("threads caller abort signal through requestInit", async () => {
-      adapterMocks.fetchMedia.mockResolvedValueOnce({
-        buffer: buildPngHeader(100, 100),
-      });
-
-      await getImageSizeFromUrl("https://cdn.example.com/img.png", 3000);
-
-      const opts = adapterMocks.fetchMedia.mock.calls[0][0];
-      expect(opts.requestInit.signal).toBeInstanceOf(AbortSignal);
-    });
-  });
-
-  describe("SSRF blocking (adapter.fetchMedia rejects)", () => {
-    it("returns null when adapter.fetchMedia throws for loopback", async () => {
-      adapterMocks.fetchMedia.mockRejectedValueOnce(new Error("SSRF blocked: loopback address"));
-
-      const result = await getImageSizeFromUrl("https://127.0.0.1/img.png");
-
-      expect(result).toBeNull();
-    });
-
-    it("returns null when adapter.fetchMedia throws for IPv6 loopback", async () => {
-      adapterMocks.fetchMedia.mockRejectedValueOnce(new Error("SSRF blocked: loopback address"));
-
-      const result = await getImageSizeFromUrl("https://[::1]/img.png");
-
-      expect(result).toBeNull();
-    });
-
-    it("returns null when adapter.fetchMedia throws for link-local/metadata", async () => {
-      adapterMocks.fetchMedia.mockRejectedValueOnce(new Error("SSRF blocked: link-local address"));
-
-      const result = await getImageSizeFromUrl("https://169.254.169.254/latest/meta-data/");
-
-      expect(result).toBeNull();
-    });
-
-    it("returns null when adapter.fetchMedia throws for RFC1918 addresses", async () => {
-      adapterMocks.fetchMedia.mockRejectedValueOnce(new Error("SSRF blocked: private address"));
-
-      const result = await getImageSizeFromUrl("https://10.0.0.1/img.png");
-
-      expect(result).toBeNull();
-    });
-
-    it("returns null on http error from adapter.fetchMedia", async () => {
-      adapterMocks.fetchMedia.mockRejectedValueOnce(new Error("HTTP 403 Forbidden"));
-
-      const result = await getImageSizeFromUrl("https://cdn.example.com/forbidden.png");
-
-      expect(result).toBeNull();
-    });
-  });
-
-  describe("happy path", () => {
-    it("returns parsed dimensions for a valid PNG", async () => {
-      adapterMocks.fetchMedia.mockResolvedValueOnce({
-        buffer: buildPngHeader(1920, 1080),
-        contentType: "image/png",
-      });
-
-      const size = await getImageSizeFromUrl("https://cdn.example.com/banner.png");
-
-      expect(size).toEqual({ width: 1920, height: 1080 });
-    });
-
-    it("returns null when the buffer is not a recognized image format", async () => {
-      adapterMocks.fetchMedia.mockResolvedValueOnce({
-        buffer: Buffer.from("not an image"),
-        contentType: "text/html",
-      });
-
-      const size = await getImageSizeFromUrl("https://cdn.example.com/notimage.html");
-
-      expect(size).toBeNull();
-    });
-  });
-});
-
-describe("parseImageSize", () => {
-  it("parses PNG dimensions", () => {
-    const size = parseImageSize(buildPngHeader(640, 480));
-    expect(size).toEqual({ width: 640, height: 480 });
-  });
-
-  it("returns null for unrecognized data", () => {
-    expect(parseImageSize(Buffer.from("hello"))).toBeNull();
-  });
-
-  it("returns null for empty buffer", () => {
-    expect(parseImageSize(Buffer.alloc(0))).toBeNull();
-  });
-});