@engine-room/after-effects-mcp 0.5.1 → 0.6.0

package/panel/client/framereader.js

@@ -1,248 +0,0 @@
-// framereader.js — wait for After Effects to finish writing a frame.
-//
-// `saveFrameToPng` is asynchronous: it returns before the bytes are on disk, so
-// the panel has to decide for itself when the file is finished. Getting that
-// decision wrong is issue #45.
-//
-// What it used to be: two `stat` calls 30ms apart reporting the same size. That
-// is not completion, it is a pause — and on a heavy comp (~88 layers, nested
-// precomps) the writer pauses routinely. The half-written file was read, shipped
-// and reported as a successful screenshot, and the agent got `truncated PNG:
-// chunk IDAT runs past the end of the file` for a render that was still
-// happening. Worse, the old passthrough path hashed those truncated bytes into
-// the stale-frame cache, so the next truncation at the same byte count came back
-// as "stale frame" — the wrong diagnosis, with the wrong remedy, for a bug that
-// was never about staleness.
-//
-// What it is now: a PNG is finished when it *is* a PNG — ends in a zero-length
-// IEND chunk with every chunk length from the signature adding up to exactly
-// that. A partial write cannot satisfy that test, so it can no longer be
-// delivered; it can only time out or be reported as corrupt.
-//
-// The two failures are kept apart on purpose, exactly as `BridgeTimeoutError`
-// and `BridgeUnreachableError` are kept apart on the server: "the render did not
-// finish in time" and "the file After Effects wrote is not a whole PNG" have
-// opposite remedies, and one sentence covering both would send half the readers
-// the wrong way.
-//
-// Node builtins only, and a CommonJS module, so `client/main.js` can require it
-// inside CEP's mixed context and `tests/unit/frame-integrity.mjs` can require
-// the same file under plain Node. There is no After Effects on a CI runner and
-// this is the code the bug lived in, so it must be exercisable without one.
-
-"use strict";
-
-var fs = require("fs");
-var pngCodec = require("./pngcodec.js");
-
-// How long a frame that has not arrived is still allowed to be on its way. A
-// cold render of a heavy 4K comp was measured taking well over 15s; the original
-// 5s silently failed screenshots that were merely still rendering.
-var FRAME_RENDER_BUDGET_MS = 120000;
-
-// How long a file may sit at exactly the same size, still not a whole PNG,
-// before the write is called abandoned rather than slow.
-//
-// Deliberately far short of the budget above. A corrupt frame has to fail fast
-// enough that one automatic re-render still fits inside the server's 300s
-// ceiling for this op — and a PNG write that has begun is a few hundred KB of
-// deflate output, not something that legitimately stalls for seconds.
-var FRAME_STALL_MS = 6000;
-
-var FRAME_POLL_MS = 40;
-
-function frameError(code, detail) {
-  var e = new Error(detail);
-  e.code = code;
-  e.detail = detail;
-  return e;
-}
-
-function unlinkQuietly(file) {
-  try { fs.unlinkSync(file); } catch (e) {}
-}
-
-/** True when the last 12 bytes are a zero-length IEND — the file ends where a PNG ends. */
-function endsWithIend(file, size) {
-  if (size < 12) return false;
-  var fd = null;
-  try {
-    fd = fs.openSync(file, "r");
-    var tail = Buffer.alloc(12);
-    fs.readSync(fd, tail, 0, 12, size - 12);
-    return tail.readUInt32BE(0) === 0 && tail.toString("ascii", 4, 8) === "IEND";
-  } catch (e) {
-    return false;
-  } finally {
-    if (fd !== null) { try { fs.closeSync(fd); } catch (e2) {} }
-  }
-}
-
-function readHead(file, n) {
-  var fd = null;
-  try {
-    fd = fs.openSync(file, "r");
-    var head = Buffer.alloc(n);
-    var got = fs.readSync(fd, head, 0, n, 0);
-    return head.subarray(0, got);
-  } catch (e) {
-    return null;
-  } finally {
-    if (fd !== null) { try { fs.closeSync(fd); } catch (e2) {} }
-  }
-}
-
-/**
- * Resolve with the bytes of a complete PNG, or reject saying which failure it
- * was. The temp file is removed on every path — a file left behind is one a
- * later read could find, which is how a corrupt frame would start coming back
- * for free.
- *
- * The tail probe in front of the full read is what keeps this cheap enough to
- * run every 40ms on a multi-megabyte file: IEND is the last chunk in the
- * format, so a file whose final twelve bytes are not one is definitively
- * unfinished and does not need reading at all. The whole chunk walk still runs
- * before any bytes are returned.
- *
- * Rejections:
- *   FRAME_INCOMPLETE  the file exists, stopped changing, and is not a PNG.
- *                     Re-rendering can fix it, and this fails within stallMs.
- *   RENDER_TIMEOUT    it never got there inside the budget. Re-rendering
- *                     immediately would only spend the budget a second time.
- */
-function waitForCompletePng(file, opts) {
-  var o = opts || {};
-  var budgetMs = o.budgetMs || FRAME_RENDER_BUDGET_MS;
-  var stallMs = o.stallMs || FRAME_STALL_MS;
-  var pollMs = o.pollMs || FRAME_POLL_MS;
-  var deadline = Date.now() + budgetMs;
-  return new Promise(function (resolve, reject) {
-    var lastSize = -1;
-    var lastGrewAt = Date.now();
-    var headChecked = false;
-    var reason = "After Effects has not written the file yet";
-
-    function fail(code, detail) {
-      unlinkQuietly(file);
-      reject(frameError(code, detail));
-    }
-
-    (function poll() {
-      var size = -1;
-      try {
-        if (fs.existsSync(file)) size = fs.statSync(file).size;
-      } catch (e) { size = -1; }
-
-      if (size >= 0) {
-        if (size !== lastSize) {
-          lastSize = size;
-          lastGrewAt = Date.now();
-        }
-        // Something that is not a PNG at all will not become one by gaining
-        // bytes. Say so now rather than at the end of a two-minute budget.
-        if (!headChecked && size >= 8) {
-          headChecked = true;
-          var head = readHead(file, 8);
-          if (head) {
-            var early = pngCodec.inspectPngStructure(head);
-            if (!early.complete && !early.growable) return fail("FRAME_INCOMPLETE", early.reason);
-          }
-        }
-        if (endsWithIend(file, size)) {
-          var buf = null;
-          try { buf = fs.readFileSync(file); } catch (e) { buf = null; }
-          if (buf) {
-            var st = pngCodec.inspectPngStructure(buf);
-            if (st.complete) {
-              unlinkQuietly(file);
-              return resolve(buf);
-            }
-            if (!st.growable) return fail("FRAME_INCOMPLETE", st.reason);
-            reason = st.reason;
-          }
-        } else if (size > 0) {
-          reason = "the file reached " + size + " bytes with no IEND chunk";
-        }
-        if (size > 0 && Date.now() - lastGrewAt > stallMs) {
-          return fail(
-            "FRAME_INCOMPLETE",
-            "After Effects wrote " + size + " bytes, stopped for " +
-              Math.round((Date.now() - lastGrewAt) / 1000) + "s, and " + reason
-          );
-        }
-      }
-
-      if (Date.now() > deadline) {
-        return fail(
-          "RENDER_TIMEOUT",
-          "nothing complete arrived within " + Math.round(budgetMs / 1000) + "s (" + reason + ")"
-        );
-      }
-      setTimeout(poll, pollMs);
-    })();
-  });
-}
-
-/**
- * Turn a one-sentence reader failure into the message an agent acts on.
- *
- * `attempts` is how many renders were spent, so the advice does not tell
- * somebody to retry something that has already been retried for them.
- *
- * Anything without one of the two codes is returned untouched: an ExtendScript
- * error travelling through here is not this module's to rewrite.
- */
-function dressFrameError(err, attempts) {
-  if (!err || (err.code !== "FRAME_INCOMPLETE" && err.code !== "RENDER_TIMEOUT")) return err;
-  var tried = attempts > 1 ? " Both the first attempt and one automatic re-render failed the same way." : "";
-  var lines;
-  if (err.code === "FRAME_INCOMPLETE") {
-    lines = [
-      "Corrupt frame: " + err.detail + ", so the file is not a whole PNG and nothing was sent." + tried,
-      "",
-      "This is NOT a timeout. The render stopped writing and what it left behind is",
-      "incomplete — a truncated PNG decodes to a wrong picture or to none at all, and",
-      "neither is a screenshot. It correlates with how heavy the comp is (issue #45).",
-      "",
-      "What to do next:",
-      "1. Retry at a higher `downsample` (6 or 8). A smaller frame is a smaller write,",
-      "   and completes where a large one did not.",
-      "2. On a heavy assembled comp, screenshot the shot precomps one at a time rather",
-      "   than the assembly.",
-      "3. If it repeats, verify the animation by reading keyframes (get_keyframes /",
-      "   get_layer_full) instead. That is exact; a picture is not.",
-      "",
-      "Do NOT disable layers to make the render succeed: this is a limit of the render",
-      "path, not a problem with the project.",
-    ];
-  } else {
-    lines = [
-      "Render timed out: " + err.detail + "." + tried,
-      "",
-      "This is NOT a corrupt file and NOT a lost bridge. After Effects was still working",
-      "on the frame when the panel gave up waiting, so the render is most likely still",
-      "running and nothing has gone wrong with the project.",
-      "",
-      "What to do next:",
-      "1. Wait a few seconds before doing anything else — a re-render started now would",
-      "   queue behind the one still going.",
-      "2. Then retry at a higher `downsample` (6 or 8). Fewer pixels is less render.",
-      "3. On a heavy assembled comp, screenshot the shot precomps one at a time.",
-      "4. If it repeats, read keyframes (get_keyframes / get_layer_full) instead of",
-      "   looking at it. That is exact and costs nothing to render.",
-    ];
-  }
-  var out = new Error(lines.join("\n"));
-  out.code = err.code;
-  out.detail = err.detail;
-  return out;
-}
-
-module.exports = {
-  waitForCompletePng: waitForCompletePng,
-  dressFrameError: dressFrameError,
-  frameError: frameError,
-  unlinkQuietly: unlinkQuietly,
-  FRAME_RENDER_BUDGET_MS: FRAME_RENDER_BUDGET_MS,
-  FRAME_STALL_MS: FRAME_STALL_MS,
-};

package/panel/client/mogrt.js

@@ -1,328 +0,0 @@
-// mogrt.js — replace the still thumbnail inside an exported .mogrt.
-//
-// A .mogrt is a zip holding project.aegraphic, definition.json, thumb.mp4 and
-// thumb.png. After Effects has no scriptable poster time, and the export
-// ignores comp.time, so thumb.png is whatever AE picked — in practice black
-// (issue #23). The frame the user actually wants can be rendered by
-// saveFrameToPng; getting it into the archive is what this file does.
-//
-// Why the panel rather than the MCP server: the panel is already the layer that
-// post-processes files After Effects has just written (see pngcodec.js), and it
-// is on the same machine as AE by construction rather than by convention. It
-// also means export_mogrt stays an ordinary forwarded op with no new branch in
-// the server.
-//
-// Node builtins only, and CommonJS, for the same reasons as pngcodec.js: it has
-// to load inside CEP's mixed context and be requireable by a unit test, because
-// there is no After Effects on a CI runner and this is real archive surgery.
-//
-// The zip work is deliberately narrow. Entries other than the one being
-// replaced are copied across as their original compressed bytes — never
-// re-compressed — so the only entry this code can possibly corrupt is the one
-// it means to rewrite.
-
-"use strict";
-
-var zlib = require("zlib");
-var pngCodec = require("./pngcodec.js");
-
-var THUMB_ENTRY = "thumb.png";
-
-var LOCAL_SIG = 0x04034b50;
-var CENTRAL_SIG = 0x02014b50;
-var EOCD_SIG = 0x06054b50;
-var ZIP64_EOCD_LOCATOR_SIG = 0x07064b50;
-var EOCD_MIN_SIZE = 22;
-var ZIP64_SENTINEL = 0xffffffff;
-
-// ---------- zip reading ----------
-
-function findEocd(buf) {
-  // The EOCD is last, but a zip comment can follow it — scan back over the
-  // largest comment the format allows plus the record itself.
-  var minStart = Math.max(0, buf.length - (0xffff + EOCD_MIN_SIZE));
-  for (var i = buf.length - EOCD_MIN_SIZE; i >= minStart; i--) {
-    if (buf.readUInt32LE(i) !== EOCD_SIG) continue;
-    var commentLength = buf.readUInt16LE(i + 20);
-    if (i + EOCD_MIN_SIZE + commentLength === buf.length) return i;
-  }
-  throw new Error("not a zip: no end-of-central-directory record");
-}
-
-/**
- * Parse the central directory. It, not the local headers, is the authoritative
- * record of sizes — a local header is allowed to carry zeros and defer to a
- * data descriptor after the entry, which is exactly the case that silently
- * truncates a naive rewriter.
- */
-function readEntries(buf) {
-  var eocd = findEocd(buf);
-
-  // Zip64 changes the offsets these fields hold. A .mogrt is a handful of MB,
-  // so rather than implement it we refuse: corrupting somebody's template is a
-  // much worse outcome than declining to touch an archive this was not built for.
-  if (eocd >= 20 && buf.readUInt32LE(eocd - 20) === ZIP64_EOCD_LOCATOR_SIG) {
-    throw new Error("zip64 archives are not supported");
-  }
-  var count = buf.readUInt16LE(eocd + 10);
-  var cdSize = buf.readUInt32LE(eocd + 12);
-  var cdOffset = buf.readUInt32LE(eocd + 16);
-  if (cdOffset === ZIP64_SENTINEL || cdSize === ZIP64_SENTINEL || count === 0xffff) {
-    throw new Error("zip64 archives are not supported");
-  }
-  if (cdOffset + cdSize > buf.length) throw new Error("corrupt zip: central directory runs past the end of the file");
-
-  var entries = [];
-  var pos = cdOffset;
-  for (var n = 0; n < count; n++) {
-    if (buf.readUInt32LE(pos) !== CENTRAL_SIG) throw new Error("corrupt zip: bad central directory signature at entry " + n);
-    var nameLength = buf.readUInt16LE(pos + 28);
-    var extraLength = buf.readUInt16LE(pos + 30);
-    var commentLength = buf.readUInt16LE(pos + 32);
-    var localOffset = buf.readUInt32LE(pos + 42);
-    if (localOffset === ZIP64_SENTINEL) throw new Error("zip64 archives are not supported");
-
-    var entry = {
-      versionMadeBy: buf.readUInt16LE(pos + 4),
-      versionNeeded: buf.readUInt16LE(pos + 6),
-      flags: buf.readUInt16LE(pos + 8),
-      method: buf.readUInt16LE(pos + 10),
-      modTime: buf.readUInt16LE(pos + 12),
-      modDate: buf.readUInt16LE(pos + 14),
-      crc32: buf.readUInt32LE(pos + 16),
-      compressedSize: buf.readUInt32LE(pos + 20),
-      uncompressedSize: buf.readUInt32LE(pos + 24),
-      internalAttrs: buf.readUInt16LE(pos + 36),
-      externalAttrs: buf.readUInt32LE(pos + 38),
-      name: buf.toString("utf8", pos + 46, pos + 46 + nameLength),
-      localOffset: localOffset,
-    };
-    if (entry.compressedSize === ZIP64_SENTINEL || entry.uncompressedSize === ZIP64_SENTINEL) {
-      throw new Error("zip64 archives are not supported");
-    }
-
-    // The compressed bytes start after the *local* header, whose name and extra
-    // lengths can differ from the central directory's.
-    if (buf.readUInt32LE(localOffset) !== LOCAL_SIG) throw new Error("corrupt zip: bad local header for " + entry.name);
-    var dataStart = localOffset + 30 + buf.readUInt16LE(localOffset + 26) + buf.readUInt16LE(localOffset + 28);
-    var dataEnd = dataStart + entry.compressedSize;
-    if (dataEnd > buf.length) throw new Error("corrupt zip: " + entry.name + " runs past the end of the file");
-    entry.data = buf.subarray(dataStart, dataEnd);
-
-    entries.push(entry);
-    pos += 46 + nameLength + extraLength + commentLength;
-  }
-  return entries;
-}
-
-function inflateEntry(entry) {
-  if (entry.method === 0) return Buffer.from(entry.data);
-  if (entry.method === 8) return zlib.inflateRawSync(entry.data);
-  throw new Error("unsupported zip compression method " + entry.method + " for " + entry.name);
-}
-
-// ---------- zip writing ----------
-
-/**
- * Rebuild an archive from parsed entries. Extra fields and comments are
- * dropped: in a .mogrt they hold timestamps, and carrying them would mean
- * keeping the local and central copies consistent for no gain. Data descriptors
- * are folded away — every size is known here, so the flag bit that says
- * "look after the entry for them" is cleared.
- */
-function writeZip(entries) {
-  var parts = [];
-  var central = [];
-  var offset = 0;
-
-  for (var i = 0; i < entries.length; i++) {
-    var e = entries[i];
-    var nameBuf = Buffer.from(e.name, "utf8");
-    var flags = e.flags & ~0x0008; // no data descriptor
-
-    var local = Buffer.alloc(30);
-    local.writeUInt32LE(LOCAL_SIG, 0);
-    local.writeUInt16LE(e.versionNeeded, 4);
-    local.writeUInt16LE(flags, 6);
-    local.writeUInt16LE(e.method, 8);
-    local.writeUInt16LE(e.modTime, 10);
-    local.writeUInt16LE(e.modDate, 12);
-    local.writeUInt32LE(e.crc32, 14);
-    local.writeUInt32LE(e.data.length, 18);
-    local.writeUInt32LE(e.uncompressedSize, 22);
-    local.writeUInt16LE(nameBuf.length, 26);
-    local.writeUInt16LE(0, 28);
-
-    parts.push(local, nameBuf, e.data);
-
-    var cd = Buffer.alloc(46);
-    cd.writeUInt32LE(CENTRAL_SIG, 0);
-    cd.writeUInt16LE(e.versionMadeBy, 4);
-    cd.writeUInt16LE(e.versionNeeded, 6);
-    cd.writeUInt16LE(flags, 8);
-    cd.writeUInt16LE(e.method, 10);
-    cd.writeUInt16LE(e.modTime, 12);
-    cd.writeUInt16LE(e.modDate, 14);
-    cd.writeUInt32LE(e.crc32, 16);
-    cd.writeUInt32LE(e.data.length, 20);
-    cd.writeUInt32LE(e.uncompressedSize, 24);
-    cd.writeUInt16LE(nameBuf.length, 28);
-    cd.writeUInt16LE(0, 30); // extra
-    cd.writeUInt16LE(0, 32); // comment
-    cd.writeUInt16LE(0, 34); // disk number
-    cd.writeUInt16LE(e.internalAttrs, 36);
-    cd.writeUInt32LE(e.externalAttrs, 38);
-    cd.writeUInt32LE(offset, 42);
-    central.push(cd, nameBuf);
-
-    offset += local.length + nameBuf.length + e.data.length;
-  }
-
-  var cdBuf = Buffer.concat(central);
-  var eocd = Buffer.alloc(EOCD_MIN_SIZE);
-  eocd.writeUInt32LE(EOCD_SIG, 0);
-  eocd.writeUInt16LE(0, 4);
-  eocd.writeUInt16LE(0, 6);
-  eocd.writeUInt16LE(entries.length, 8);
-  eocd.writeUInt16LE(entries.length, 10);
-  eocd.writeUInt32LE(cdBuf.length, 12);
-  eocd.writeUInt32LE(offset, 16);
-  eocd.writeUInt16LE(0, 20);
-
-  return Buffer.concat(parts.concat([cdBuf, eocd]));
-}
-
-// ---------- resampling ----------
-
-/**
- * Box-filter downscale, then centre the result inside `outW`x`outH` without
- * changing its aspect ratio. Padding is fully transparent where the image has
- * an alpha channel and black where it does not.
- *
- * A box filter rather than bilinear because this is always a reduction, often
- * by 3x or more, and averaging every source pixel that lands in a destination
- * pixel is both the correct answer for that and cheaper than sampling.
- */
-function resampleFit(src, srcW, srcH, channels, outW, outH) {
-  var scale = Math.min(outW / srcW, outH / srcH);
-  var drawW = Math.max(1, Math.min(outW, Math.round(srcW * scale)));
-  var drawH = Math.max(1, Math.min(outH, Math.round(srcH * scale)));
-  var offsetX = Math.floor((outW - drawW) / 2);
-  var offsetY = Math.floor((outH - drawH) / 2);
-
-  var out = Buffer.alloc(outW * outH * channels, 0);
-  var acc = new Float64Array(channels);
-
-  for (var dy = 0; dy < drawH; dy++) {
-    var sy0 = Math.floor((dy * srcH) / drawH);
-    var sy1 = Math.max(sy0 + 1, Math.floor(((dy + 1) * srcH) / drawH));
-    if (sy1 > srcH) sy1 = srcH;
-    for (var dx = 0; dx < drawW; dx++) {
-      var sx0 = Math.floor((dx * srcW) / drawW);
-      var sx1 = Math.max(sx0 + 1, Math.floor(((dx + 1) * srcW) / drawW));
-      if (sx1 > srcW) sx1 = srcW;
-
-      for (var c = 0; c < channels; c++) acc[c] = 0;
-      var n = 0;
-      for (var sy = sy0; sy < sy1; sy++) {
-        var rowBase = sy * srcW * channels;
-        for (var sx = sx0; sx < sx1; sx++) {
-          var base = rowBase + sx * channels;
-          for (var ci = 0; ci < channels; ci++) acc[ci] += src[base + ci];
-          n++;
-        }
-      }
-      var dst = ((dy + offsetY) * outW + (dx + offsetX)) * channels;
-      for (var co = 0; co < channels; co++) out[dst + co] = Math.round(acc[co] / n);
-    }
-  }
-  return { data: out, drawWidth: drawW, drawHeight: drawH, offsetX: offsetX, offsetY: offsetY };
-}
-
-// ---------- the operation ----------
-
-/**
- * Replace thumb.png inside a .mogrt with `posterPng`, resized to the exact
- * dimensions of the thumbnail AE already wrote.
- *
- * Matching AE's dimensions rather than hardcoding 640x360 is the point: that is
- * what a 16:9 comp produced here, but nothing documents the rule for other
- * shapes, and reading it costs one IHDR.
- *
- * Returns a description of what changed. Throws if the archive has no
- * thumb.png, or is not a zip this code is prepared to rewrite — the caller
- * treats that as "the export succeeded, the thumbnail did not", never as a
- * failed export.
- */
-function patchThumbnail(mogrtBuf, posterPng) {
-  var entries = readEntries(mogrtBuf);
-
-  var target = null;
-  for (var i = 0; i < entries.length; i++) {
-    if (entries[i].name === THUMB_ENTRY) { target = entries[i]; break; }
-  }
-  if (!target) {
-    var names = [];
-    for (var j = 0; j < entries.length; j++) names.push(entries[j].name);
-    throw new Error("no " + THUMB_ENTRY + " in the template (entries: " + names.join(", ") + ")");
-  }
-
-  var existing = pngCodec.decodePng8(inflateEntry(target));
-  var poster = pngCodec.decodePng8(posterPng);
-
-  // Encode at the thumbnail's own colour type so an archive whose thumbnail has
-  // no alpha does not gain one, and vice versa.
-  var channels = existing.channels;
-  var source = poster.channels === channels
-    ? poster.pixels
-    : convertChannels(poster.pixels, poster.channels, channels, poster.width * poster.height);
-
-  var fitted = resampleFit(source, poster.width, poster.height, channels, existing.width, existing.height);
-  var encoded = pngCodec.encodePng8(existing.width, existing.height, existing.colorType, fitted.data);
-
-  target.data = zlib.deflateRawSync(encoded, { level: 6 });
-  target.method = 8;
-  target.crc32 = pngCodec.crc32(encoded);
-  target.uncompressedSize = encoded.length;
-
-  return {
-    buffer: writeZip(entries),
-    width: existing.width,
-    height: existing.height,
-    sourceWidth: poster.width,
-    sourceHeight: poster.height,
-    // Present when the poster's aspect ratio did not match the thumbnail's, so
-    // a caller can say why there are bars rather than leaving them a surprise.
-    letterboxed: fitted.drawWidth !== existing.width || fitted.drawHeight !== existing.height,
-    bytes: encoded.length,
-  };
-}
-
-/** Grow or shrink a pixel buffer between grey/greyA/RGB/RGBA. */
-function convertChannels(src, from, to, pixelCount) {
-  var out = Buffer.alloc(pixelCount * to);
-  for (var p = 0; p < pixelCount; p++) {
-    var s = p * from;
-    var d = p * to;
-    var r, g, b, a;
-    if (from === 1) { r = g = b = src[s]; a = 255; }
-    else if (from === 2) { r = g = b = src[s]; a = src[s + 1]; }
-    else if (from === 3) { r = src[s]; g = src[s + 1]; b = src[s + 2]; a = 255; }
-    else { r = src[s]; g = src[s + 1]; b = src[s + 2]; a = src[s + 3]; }
-
-    if (to === 1) { out[d] = Math.round((r * 299 + g * 587 + b * 114) / 1000); }
-    else if (to === 2) { out[d] = Math.round((r * 299 + g * 587 + b * 114) / 1000); out[d + 1] = a; }
-    else if (to === 3) { out[d] = r; out[d + 1] = g; out[d + 2] = b; }
-    else { out[d] = r; out[d + 1] = g; out[d + 2] = b; out[d + 3] = a; }
-  }
-  return out;
-}
-
-module.exports = {
-  patchThumbnail: patchThumbnail,
-  // Exported for the unit test, which needs to build and inspect archives.
-  readEntries: readEntries,
-  writeZip: writeZip,
-  inflateEntry: inflateEntry,
-  resampleFit: resampleFit,
-};