@storyteller-platform/audiobook 0.3.5 → 0.3.7

package/dist/ffmpeg.cjs

@@ -53,12 +53,18 @@ async function execCmdBuffer(command, args) {
   try {
     const { stdout } = await execFilePromise(command, args, {
       encoding: "buffer",
-      maxBuffer: 10 * 1024 * 1024,
+      // let's be very generous
+      maxBuffer: 50 * 1024 * 1024,
       cwd: process.cwd()
     });
     return stdout;
   } catch (error) {
     console.error(error);
+    if (error instanceof RangeError && error.message.includes("stdout maxBuffer length exceeded")) {
+      throw new Error(
+        "stdout maxBuffer length exceeded. This likely means that youre trying to process a very large file, and the ffmpeg process is running out of memory. Maybe check the image size of your cover art."
+      );
+    }
     if (error && typeof error === "object" && "stdout" in error) {
       console.warn((_a = error.stdout) == null ? void 0 : _a.toString());
       throw new Error((_b = error.stdout) == null ? void 0 : _b.toString());
@@ -90,7 +96,7 @@ function lookup(codecName) {
   }
 }
 async function getTrackMetadata(path) {
-  var _a, _b;
+  var _a, _b, _c, _d, _e, _f, _g, _h, _i, _j, _k, _l, _m, _n, _o, _p, _q, _r, _s, _t, _u, _v, _w, _x;
   const stdout = await execCmd(
     `ffprobe -i ${quotePath(path)} -v quiet -show_format -show_chapters -show_streams -output_format json`
   );
@@ -133,17 +139,17 @@ async function getTrackMetadata(path) {
     duration: parseFloat(format.duration),
     bitRate: format.bit_rate !== void 0 ? parseFloat(format.bit_rate) : format.bit_rate,
     tags: {
-      title: format.tags.title ?? format.tags.Title,
-      subtitle: format.tags.subtitle ?? format.tags.Subtitle,
-      date: format.tags.date ?? format.tags.Date,
-      album: format.tags.album ?? format.tags.Album,
-      albumArtist: format.tags.album_artist ?? format.tags.Album_Artist,
-      artist: format.tags.artist ?? format.tags.Artist,
-      performer: format.tags.performer ?? format.tags.Performer,
-      composer: format.tags.composer ?? format.tags.Composer,
-      comment: format.tags.comment ?? format.tags.Comment,
-      description: format.tags.description ?? format.tags.Description,
-      publisher: format.tags.publisher ?? format.tags.Publisher
+      title: ((_c = format.tags) == null ? void 0 : _c.title) ?? ((_d = format.tags) == null ? void 0 : _d.Title),
+      subtitle: ((_e = format.tags) == null ? void 0 : _e.subtitle) ?? ((_f = format.tags) == null ? void 0 : _f.Subtitle),
+      date: ((_g = format.tags) == null ? void 0 : _g.date) ?? ((_h = format.tags) == null ? void 0 : _h.Date),
+      album: ((_i = format.tags) == null ? void 0 : _i.album) ?? ((_j = format.tags) == null ? void 0 : _j.Album),
+      albumArtist: ((_k = format.tags) == null ? void 0 : _k.album_artist) ?? ((_l = format.tags) == null ? void 0 : _l.Album_Artist),
+      artist: ((_m = format.tags) == null ? void 0 : _m.artist) ?? ((_n = format.tags) == null ? void 0 : _n.Artist),
+      performer: ((_o = format.tags) == null ? void 0 : _o.performer) ?? ((_p = format.tags) == null ? void 0 : _p.Performer),
+      composer: ((_q = format.tags) == null ? void 0 : _q.composer) ?? ((_r = format.tags) == null ? void 0 : _r.Composer),
+      comment: ((_s = format.tags) == null ? void 0 : _s.comment) ?? ((_t = format.tags) == null ? void 0 : _t.Comment),
+      description: ((_u = format.tags) == null ? void 0 : _u.description) ?? ((_v = format.tags) == null ? void 0 : _v.Description),
+      publisher: ((_w = format.tags) == null ? void 0 : _w.publisher) ?? ((_x = format.tags) == null ? void 0 : _x.Publisher)
     },
     chapters: chapters.map(
       (chapter) => ({

package/dist/ffmpeg.js

@@ -27,12 +27,18 @@ async function execCmdBuffer(command, args) {
   try {
     const { stdout } = await execFilePromise(command, args, {
       encoding: "buffer",
-      maxBuffer: 10 * 1024 * 1024,
+      // let's be very generous
+      maxBuffer: 50 * 1024 * 1024,
       cwd: process.cwd()
     });
     return stdout;
   } catch (error) {
     console.error(error);
+    if (error instanceof RangeError && error.message.includes("stdout maxBuffer length exceeded")) {
+      throw new Error(
+        "stdout maxBuffer length exceeded. This likely means that youre trying to process a very large file, and the ffmpeg process is running out of memory. Maybe check the image size of your cover art."
+      );
+    }
     if (error && typeof error === "object" && "stdout" in error) {
       console.warn((_a = error.stdout) == null ? void 0 : _a.toString());
       throw new Error((_b = error.stdout) == null ? void 0 : _b.toString());
@@ -64,7 +70,7 @@ function lookup(codecName) {
   }
 }
 async function getTrackMetadata(path) {
-  var _a, _b;
+  var _a, _b, _c, _d, _e, _f, _g, _h, _i, _j, _k, _l, _m, _n, _o, _p, _q, _r, _s, _t, _u, _v, _w, _x;
   const stdout = await execCmd(
     `ffprobe -i ${quotePath(path)} -v quiet -show_format -show_chapters -show_streams -output_format json`
   );
@@ -107,17 +113,17 @@ async function getTrackMetadata(path) {
     duration: parseFloat(format.duration),
     bitRate: format.bit_rate !== void 0 ? parseFloat(format.bit_rate) : format.bit_rate,
     tags: {
-      title: format.tags.title ?? format.tags.Title,
-      subtitle: format.tags.subtitle ?? format.tags.Subtitle,
-      date: format.tags.date ?? format.tags.Date,
-      album: format.tags.album ?? format.tags.Album,
-      albumArtist: format.tags.album_artist ?? format.tags.Album_Artist,
-      artist: format.tags.artist ?? format.tags.Artist,
-      performer: format.tags.performer ?? format.tags.Performer,
-      composer: format.tags.composer ?? format.tags.Composer,
-      comment: format.tags.comment ?? format.tags.Comment,
-      description: format.tags.description ?? format.tags.Description,
-      publisher: format.tags.publisher ?? format.tags.Publisher
+      title: ((_c = format.tags) == null ? void 0 : _c.title) ?? ((_d = format.tags) == null ? void 0 : _d.Title),
+      subtitle: ((_e = format.tags) == null ? void 0 : _e.subtitle) ?? ((_f = format.tags) == null ? void 0 : _f.Subtitle),
+      date: ((_g = format.tags) == null ? void 0 : _g.date) ?? ((_h = format.tags) == null ? void 0 : _h.Date),
+      album: ((_i = format.tags) == null ? void 0 : _i.album) ?? ((_j = format.tags) == null ? void 0 : _j.Album),
+      albumArtist: ((_k = format.tags) == null ? void 0 : _k.album_artist) ?? ((_l = format.tags) == null ? void 0 : _l.Album_Artist),
+      artist: ((_m = format.tags) == null ? void 0 : _m.artist) ?? ((_n = format.tags) == null ? void 0 : _n.Artist),
+      performer: ((_o = format.tags) == null ? void 0 : _o.performer) ?? ((_p = format.tags) == null ? void 0 : _p.Performer),
+      composer: ((_q = format.tags) == null ? void 0 : _q.composer) ?? ((_r = format.tags) == null ? void 0 : _r.Composer),
+      comment: ((_s = format.tags) == null ? void 0 : _s.comment) ?? ((_t = format.tags) == null ? void 0 : _t.Comment),
+      description: ((_u = format.tags) == null ? void 0 : _u.description) ?? ((_v = format.tags) == null ? void 0 : _v.Description),
+      publisher: ((_w = format.tags) == null ? void 0 : _w.publisher) ?? ((_x = format.tags) == null ? void 0 : _x.Publisher)
     },
     chapters: chapters.map(
       (chapter) => ({

package/dist/index.cjs

@@ -1,9 +1,7 @@
 "use strict";
-var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
 var __getOwnPropNames = Object.getOwnPropertyNames;
-var __getProtoOf = Object.getPrototypeOf;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __knownSymbol = (name, symbol) => (symbol = Symbol[name]) ? symbol : Symbol.for("Symbol." + name);
 var __typeError = (msg) => {
@@ -21,14 +19,6 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
-  // If the importer is in node compatibility mode or this is not an ESM
-  // file that has been converted to a CommonJS file using a Babel-
-  // compatible transform (i.e. "__esModule" has not been set), then set
-  // "default" to the CommonJS "module.exports" for node compatibility.
-  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
-  mod
-));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var __using = (stack, value, async) => {
   if (value != null) {
@@ -92,30 +82,12 @@ var import_node_crypto = require("node:crypto");
 var import_node_fs = require("node:fs");
 var import_promises = require("node:fs/promises");
 var import_node_os = require("node:os");
-var util = __toESM(require("node:util"), 1);
+var import_promises2 = require("node:stream/promises");
 var import_mime_types = require("mime-types");
-var import_yauzl = __toESM(require("yauzl"), 1);
+var import_yauzl_promise = require("yauzl-promise");
 var import_yazl = require("yazl");
 var import_path = require("@storyteller-platform/path");
 var import_entry = require("./entry.cjs");
-function promisify(api) {
-  return function(arg, options) {
-    return new Promise(function(resolve, reject) {
-      api(arg, options, function(err, response) {
-        if (err) {
-          reject(err);
-          return;
-        }
-        resolve(response);
-      });
-    });
-  };
-}
-const unzipFromPath = promisify(
-  (arg, options, callback) => {
-    import_yauzl.default.open(arg, options, callback);
-  }
-);
 const COVER_IMAGE_FILE_EXTENSIONS = [
   ".jpeg",
   ".jpg",
@@ -162,54 +134,47 @@ class Audiobook {
   }
   async getEntries() {
     if (this.isZip) {
-      var _stack = [];
+      const [first] = this.inputs;
+      const extractPath = (0, import_path.join)(
+        (0, import_node_os.tmpdir)(),
+        `storyteller-platform-audiobook-${(0, import_node_crypto.randomUUID)()}`
+      );
+      this.extractPath = extractPath;
+      const entries = [];
       try {
-        const [first] = this.inputs;
-        const extractPath = (0, import_path.join)(
-          (0, import_node_os.tmpdir)(),
-          `storyteller-platform-audiobook-${(0, import_node_crypto.randomUUID)()}`
-        );
-        this.extractPath = extractPath;
-        await (0, import_promises.mkdir)(this.extractPath, { recursive: true });
-        const zipfile = await unzipFromPath(first, { lazyEntries: true });
-        const stack = __using(_stack, new DisposableStack());
-        stack.defer(() => {
-          zipfile.close();
-        });
-        const entries = [];
-        const { promise, resolve } = Promise.withResolvers();
-        zipfile.on("end", () => {
-          resolve();
-        });
-        const openReadStream = util.promisify(
-          zipfile.openReadStream.bind(zipfile)
-        );
-        zipfile.readEntry();
-        zipfile.on("entry", async (entry) => {
-          if (entry.fileName.endsWith("/")) {
-            zipfile.readEntry();
-          } else {
-            entries.push(entry.fileName);
-            const readStream = await openReadStream(entry);
-            readStream.on("end", function() {
-              zipfile.readEntry();
-            });
-            const writePath = (0, import_path.join)(extractPath, entry.fileName);
-            await (0, import_promises.mkdir)((0, import_path.dirname)(writePath), { recursive: true });
-            readStream.pipe((0, import_node_fs.createWriteStream)(writePath));
+        var _stack = [];
+        try {
+          const zipfile = await (0, import_yauzl_promise.open)(first);
+          const stack = __using(_stack, new AsyncDisposableStack(), true);
+          stack.defer(async () => {
+            await zipfile.close();
+          });
+          for await (const entry of zipfile) {
+            if (entry.filename.endsWith("/")) {
+            } else {
+              entries.push(entry.filename);
+              const readStream = await entry.openReadStream();
+              const writePath = (0, import_path.join)(extractPath, entry.filename);
+              await (0, import_promises.mkdir)((0, import_path.dirname)(writePath), { recursive: true });
+              const writeStream = (0, import_node_fs.createWriteStream)(writePath);
+              await (0, import_promises2.pipeline)(readStream, writeStream);
+            }
           }
-        });
-        await promise;
-        return entries.filter(
-          (entry) => AUDIO_FILE_EXTENSIONS.includes(
-            (0, import_path.extname)(entry)
-          )
-        ).map((entry) => new import_entry.AudiobookEntry((0, import_path.join)(extractPath, entry)));
-      } catch (_) {
-        var _error = _, _hasError = true;
-      } finally {
-        __callDispose(_stack, _error, _hasError);
+        } catch (_) {
+          var _error = _, _hasError = true;
+        } finally {
+          var _promise = __callDispose(_stack, _error, _hasError);
+          _promise && await _promise;
+        }
+      } catch (error) {
+        (0, import_node_fs.rmSync)(extractPath, { force: true, recursive: true });
+        throw error;
       }
+      return entries.filter(
+        (entry) => AUDIO_FILE_EXTENSIONS.includes(
+          (0, import_path.extname)(entry)
+        )
+      ).map((entry) => new import_entry.AudiobookEntry((0, import_path.join)(extractPath, entry)));
     } else {
       return this.inputs.map((input) => new import_entry.AudiobookEntry(input));
     }

package/dist/index.js

@@ -6,30 +6,12 @@ import { randomUUID } from "node:crypto";
 import { createWriteStream, rmSync } from "node:fs";
 import { cp, mkdir, readFile, readdir, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
-import * as util from "node:util";
+import { pipeline } from "node:stream/promises";
 import { lookup } from "mime-types";
-import yauzl from "yauzl";
+import { open } from "yauzl-promise";
 import { ZipFile } from "yazl";
 import { basename, dirname, extname, join } from "@storyteller-platform/path";
 import { AudiobookEntry } from "./entry.js";
-function promisify(api) {
-  return function(arg, options) {
-    return new Promise(function(resolve, reject) {
-      api(arg, options, function(err, response) {
-        if (err) {
-          reject(err);
-          return;
-        }
-        resolve(response);
-      });
-    });
-  };
-}
-const unzipFromPath = promisify(
-  (arg, options, callback) => {
-    yauzl.open(arg, options, callback);
-  }
-);
 const COVER_IMAGE_FILE_EXTENSIONS = [
   ".jpeg",
   ".jpg",
@@ -76,54 +58,47 @@ class Audiobook {
   }
   async getEntries() {
     if (this.isZip) {
-      var _stack = [];
+      const [first] = this.inputs;
+      const extractPath = join(
+        tmpdir(),
+        `storyteller-platform-audiobook-${randomUUID()}`
+      );
+      this.extractPath = extractPath;
+      const entries = [];
       try {
-        const [first] = this.inputs;
-        const extractPath = join(
-          tmpdir(),
-          `storyteller-platform-audiobook-${randomUUID()}`
-        );
-        this.extractPath = extractPath;
-        await mkdir(this.extractPath, { recursive: true });
-        const zipfile = await unzipFromPath(first, { lazyEntries: true });
-        const stack = __using(_stack, new DisposableStack());
-        stack.defer(() => {
-          zipfile.close();
-        });
-        const entries = [];
-        const { promise, resolve } = Promise.withResolvers();
-        zipfile.on("end", () => {
-          resolve();
-        });
-        const openReadStream = util.promisify(
-          zipfile.openReadStream.bind(zipfile)
-        );
-        zipfile.readEntry();
-        zipfile.on("entry", async (entry) => {
-          if (entry.fileName.endsWith("/")) {
-            zipfile.readEntry();
-          } else {
-            entries.push(entry.fileName);
-            const readStream = await openReadStream(entry);
-            readStream.on("end", function() {
-              zipfile.readEntry();
-            });
-            const writePath = join(extractPath, entry.fileName);
-            await mkdir(dirname(writePath), { recursive: true });
-            readStream.pipe(createWriteStream(writePath));
+        var _stack = [];
+        try {
+          const zipfile = await open(first);
+          const stack = __using(_stack, new AsyncDisposableStack(), true);
+          stack.defer(async () => {
+            await zipfile.close();
+          });
+          for await (const entry of zipfile) {
+            if (entry.filename.endsWith("/")) {
+            } else {
+              entries.push(entry.filename);
+              const readStream = await entry.openReadStream();
+              const writePath = join(extractPath, entry.filename);
+              await mkdir(dirname(writePath), { recursive: true });
+              const writeStream = createWriteStream(writePath);
+              await pipeline(readStream, writeStream);
+            }
           }
-        });
-        await promise;
-        return entries.filter(
-          (entry) => AUDIO_FILE_EXTENSIONS.includes(
-            extname(entry)
-          )
-        ).map((entry) => new AudiobookEntry(join(extractPath, entry)));
-      } catch (_) {
-        var _error = _, _hasError = true;
-      } finally {
-        __callDispose(_stack, _error, _hasError);
+        } catch (_) {
+          var _error = _, _hasError = true;
+        } finally {
+          var _promise = __callDispose(_stack, _error, _hasError);
+          _promise && await _promise;
+        }
+      } catch (error) {
+        rmSync(extractPath, { force: true, recursive: true });
+        throw error;
       }
+      return entries.filter(
+        (entry) => AUDIO_FILE_EXTENSIONS.includes(
+          extname(entry)
+        )
+      ).map((entry) => new AudiobookEntry(join(extractPath, entry)));
     } else {
       return this.inputs.map((input) => new AudiobookEntry(input));
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@storyteller-platform/audiobook",
-  "version": "0.3.5",
+  "version": "0.3.7",
   "type": "module",
   "license": "MIT",
   "module": "dist/index.js",
@@ -32,7 +32,9 @@
   "devDependencies": {
     "@storyteller-platform/tsup": "^0.1.0",
     "@tsconfig/strictest": "^2.0.5",
+    "@types/yauzl-promise": "^4",
     "@types/yazl": "^3",
+    "eslint": "^8.0.0",
     "markdown-toc": "^1.2.0",
     "remark-toc": "^9.0.0",
     "tsup": "^8.5.0",
@@ -43,14 +45,14 @@
     "typescript": "^5.9.2"
   },
   "dependencies": {
-    "@storyteller-platform/fs": "^0.1.3",
+    "@storyteller-platform/eslint": "0.1.0",
+    "@storyteller-platform/fs": "^0.1.4",
     "@storyteller-platform/path": "^0.1.1",
     "@types/mime-types": "^2",
     "@types/node": "^24.0.0",
-    "@types/yauzl": "^2.10.3",
     "mime-types": "^3.0.1",
     "music-metadata": "^11.9.0",
-    "yauzl": "^3.2.0",
+    "yauzl-promise": "^4.0.0",
     "yazl": "^3.3.1"
   },
   "publishConfig": {