node-poppler 8.0.0 → 8.0.2

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2019-2025 Frazer Smith
+Copyright (c) 2019-present Frazer Smith
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "node-poppler",
-	"version": "8.0.0",
+	"version": "8.0.2",
 	"description": "Asynchronous Node.js wrapper for the Poppler PDF rendering library",
 	"keywords": [
 		"async",
@@ -51,6 +51,6 @@
 	},
 	"dependencies": {
 		"camelcase": "^6.3.0",
-		"semver": "^7.6.3"
+		"semver": "^7.7.2"
 	}
 }

package/src/index.js

@@ -1,10 +1,10 @@
 "use strict";
 
 const { execFile, spawn, spawnSync } = require("node:child_process");
+const { normalize, resolve: pathResolve } = require("node:path");
 const { promisify } = require("node:util");
 const camelCase = require("camelcase");
 const { lt } = require("semver");
-const { normalize, resolve: pathResolve } = require("node:path");
 
 const execFileAsync = promisify(execFile);
 
@@ -50,15 +50,17 @@ function parseOptions(acceptedOptions, options, version) {
 	const args = [];
 	/** @type {string[]} */
 	const invalidArgs = [];
-	const keys = Object.keys(options);
-	const keysLength = keys.length;
-	for (let i = 0; i < keysLength; i += 1) {
-		const key = keys[i];
+
+	// Imperative loops are faster than functional loops, see https://romgrk.com/posts/optimizing-javascript
+	const entries = Object.entries(options);
+	const entriesLength = entries.length;
+	for (let i = 0; i < entriesLength; i += 1) {
+		// Destructuring adds overhead, so use index access
+		const key = entries[i][0];
 		if (Object.hasOwn(acceptedOptions, key)) {
-			const option = options[key];
+			const option = entries[i][1];
 			const acceptedOption = acceptedOptions[key];
 
-			// eslint-disable-next-line valid-typeof -- `type` is a string
 			if (acceptedOption.type === typeof option) {
 				// Skip boolean options if false
 				if (acceptedOption.type !== "boolean" || option) {
@@ -174,18 +176,14 @@ class Poppler {
 			replace: { arg: "-replace", type: "boolean" },
 		};
 
-		try {
-			const args = parseOptions(acceptedOptions, options);
-			args.push(file, fileToAttach, outputFile);
+		const args = parseOptions(acceptedOptions, options);
+		args.push(file, fileToAttach, outputFile);
 
-			const { stdout } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfattach"),
-				args
-			);
-			return Promise.resolve(stdout);
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		const { stdout } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfattach"),
+			args
+		);
+		return stdout;
 	}
 
 	/**
@@ -232,18 +230,14 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const args = parseOptions(acceptedOptions, options);
-			args.push(file);
+		const args = parseOptions(acceptedOptions, options);
+		args.push(file);
 
-			const { stdout } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfdetach"),
-				args
-			);
-			return Promise.resolve(stdout);
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		const { stdout } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfdetach"),
+			args
+		);
+		return stdout;
 	}
 
 	/**
@@ -271,65 +265,61 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdffonts"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdffonts"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			return new Promise((resolve, reject) => {
-				args.push(Buffer.isBuffer(file) ? "-" : file);
+		return new Promise((resolve, reject) => {
+			args.push(Buffer.isBuffer(file) ? "-" : file);
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdffonts"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdffonts"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdOut = "";
-				let stdErr = "";
+			let stdOut = "";
+			let stdErr = "";
 
-				child.stdout.on("data", (data) => {
-					stdOut += data;
-				});
+			child.stdout.on("data", (data) => {
+				stdOut += data;
+			});
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				child.on("close", (code) => {
-					/* istanbul ignore else */
-					if (stdOut !== "") {
-						resolve(stdOut.trim());
-					} else if (code === 0) {
-						resolve(errorMessages[code]);
-					} else if (stdErr !== "") {
-						reject(new Error(stdErr.trim()));
-					} else {
-						reject(
-							new Error(
-								// @ts-ignore: Second operand used if code is not in errorMessages
-								errorMessages[code] ||
-									`pdffonts ${args.join(
-										" "
-									)} exited with code ${code}`
-							)
-						);
-					}
-				});
+			child.on("close", (code) => {
+				/* istanbul ignore else */
+				if (stdOut !== "") {
+					resolve(stdOut.trim());
+				} else if (code === 0) {
+					resolve(errorMessages[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in errorMessages
+							errorMessages[code] ||
+								`pdffonts ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -374,69 +364,65 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfimages"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfimages"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			return new Promise((resolve, reject) => {
-				args.push(Buffer.isBuffer(file) ? "-" : file);
+		return new Promise((resolve, reject) => {
+			args.push(Buffer.isBuffer(file) ? "-" : file);
 
-				if (outputPrefix) {
-					args.push(outputPrefix);
-				}
+			if (outputPrefix) {
+				args.push(outputPrefix);
+			}
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdfimages"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdfimages"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdOut = "";
-				let stdErr = "";
+			let stdOut = "";
+			let stdErr = "";
 
-				child.stdout.on("data", (data) => {
-					stdOut += data;
-				});
+			child.stdout.on("data", (data) => {
+				stdOut += data;
+			});
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				child.on("close", (code) => {
-					/* istanbul ignore else */
-					if (stdOut !== "") {
-						resolve(stdOut.trim());
-					} else if (code === 0) {
-						resolve(errorMessages[code]);
-					} else if (stdErr !== "") {
-						reject(new Error(stdErr.trim()));
-					} else {
-						reject(
-							new Error(
-								// @ts-ignore: Second operand used if code is not in errorMessages
-								errorMessages[code] ||
-									`pdfimages ${args.join(
-										" "
-									)} exited with code ${code}`
-							)
-						);
-					}
-				});
+			child.on("close", (code) => {
+				/* istanbul ignore else */
+				if (stdOut !== "") {
+					resolve(stdOut.trim());
+				} else if (code === 0) {
+					resolve(errorMessages[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in errorMessages
+							errorMessages[code] ||
+								`pdfimages ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -495,103 +481,92 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfinfo"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfinfo"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			/**
-			 * Poppler does not set the "File size" metadata value if passed
-			 * a Buffer via stdin, so need to retrieve it from the Buffer.
-			 */
-			/** @type {number} */
-			let fileSize;
+		// Fetch file size if stdin input is a Buffer, as Poppler omits it
+		/** @type {number} */
+		let fileSize;
 
-			return new Promise((resolve, reject) => {
-				if (Buffer.isBuffer(file)) {
-					args.push("-");
-					fileSize = file.length;
-				} else {
-					args.push(file);
-				}
+		return new Promise((resolve, reject) => {
+			if (Buffer.isBuffer(file)) {
+				args.push("-");
+				fileSize = file.length;
+			} else {
+				args.push(file);
+			}
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdfinfo"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdfinfo"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdOut = "";
-				let stdErr = "";
+			let stdOut = "";
+			let stdErr = "";
 
-				child.stdout.on("data", (data) => {
-					stdOut += data;
-				});
+			child.stdout.on("data", (data) => {
+				stdOut += data;
+			});
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				child.on("close", (code) => {
-					/* istanbul ignore else */
-					if (stdOut !== "") {
-						if (fileSize) {
-							stdOut = stdOut.replace(
-								pdfInfoFileSizesRegex,
-								`$1${fileSize}$2bytes`
-							);
-						}
+			child.on("close", (code) => {
+				/* istanbul ignore else */
+				if (stdOut !== "") {
+					if (fileSize) {
+						stdOut = stdOut.replace(
+							pdfInfoFileSizesRegex,
+							`$1${fileSize}$2bytes`
+						);
+					}
 
-						/**
-						 * Convert output to JSON.
-						 * @see {@link https://github.com/Fdawgs/node-poppler/issues/248#issuecomment-845948080 | Node-Poppler Issue #248}
-						 */
-						if (options.printAsJson === true) {
-							const info = {};
-							const stdOutLines = stdOut.split("\n");
-							const stdOutLinesLength = stdOutLines.length;
-							for (let i = 0; i < stdOutLinesLength; i += 1) {
-								const line = stdOutLines[i];
-								const lines = line.split(": ");
-								if (lines.length > 1) {
-									// @ts-ignore: creating dynamic object keys
-									info[camelCase(lines[0])] = lines[1].trim();
-								}
+					if (options.printAsJson === true) {
+						const info = {};
+						const stdOutLines = stdOut.split("\n");
+						const stdOutLinesLength = stdOutLines.length;
+						for (let i = 0; i < stdOutLinesLength; i += 1) {
+							const line = stdOutLines[i];
+							const lines = line.split(": ");
+							if (lines.length > 1) {
+								// @ts-ignore: creating dynamic object keys
+								info[camelCase(lines[0])] = lines[1].trim();
 							}
-							resolve(info);
-						} else {
-							resolve(stdOut.trim());
 						}
-					} else if (code === 0) {
-						resolve(errorMessages[code]);
-					} else if (stdErr !== "") {
-						reject(new Error(stdErr.trim()));
+						resolve(info);
 					} else {
-						reject(
-							new Error(
-								// @ts-ignore: Second operand used if code is not in errorMessages
-								errorMessages[code] ||
-									`pdfinfo ${args.join(
-										" "
-									)} exited with code ${code}`
-							)
-						);
+						resolve(stdOut.trim());
 					}
-				});
+				} else if (code === 0) {
+					resolve(errorMessages[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in errorMessages
+							errorMessages[code] ||
+								`pdfinfo ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -619,26 +594,22 @@ class Poppler {
 			printVersionInfo: { arg: "-v", type: "boolean" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfseparate"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfseparate"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
-			args.push(file, outputPattern);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
+		args.push(file, outputPattern);
 
-			const { stdout } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfseparate"),
-				args
-			);
-			return Promise.resolve(stdout);
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		const { stdout } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfseparate"),
+			args
+		);
+		return stdout;
 	}
 
 	/**
@@ -947,60 +918,53 @@ class Poppler {
 			zoom: { arg: "-zoom", type: "number" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdftohtml"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdftohtml"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			return new Promise((resolve, reject) => {
-				args.push(Buffer.isBuffer(file) ? "-" : file);
+		return new Promise((resolve, reject) => {
+			args.push(Buffer.isBuffer(file) ? "-" : file);
 
-				if (outputFile) {
-					args.push(outputFile);
-				}
+			if (outputFile) {
+				args.push(outputFile);
+			}
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdftohtml"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdftohtml"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdOut = "";
-				let stdErr = "";
+			let stdOut = "";
+			let stdErr = "";
 
-				child.stdout.on("data", (data) => {
-					stdOut += data;
-				});
+			child.stdout.on("data", (data) => {
+				stdOut += data;
+			});
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				/**
-				 * PdfToHtml does not return an exit code so check output to see if it was successful.
-				 * @see {@link https://gitlab.freedesktop.org/poppler/poppler/-/blob/master/utils/pdftohtml.1 | Poppler pdftohtml man}
-				 */
-				child.on("close", () => {
-					if (stdOut !== "") {
-						resolve(stdOut.trim());
-					} else {
-						reject(new Error(stdErr ? stdErr.trim() : undefined));
-					}
-				});
+			// PdfToHtml has no exit code; check output for success
+			child.on("close", () => {
+				if (stdOut !== "") {
+					resolve(stdOut.trim());
+				} else {
+					reject(new Error(stdErr ? stdErr.trim() : undefined));
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1149,58 +1113,54 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdftoppm"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdftoppm"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			return new Promise((resolve, reject) => {
-				args.push(Buffer.isBuffer(file) ? "-" : file, outputPath);
+		return new Promise((resolve, reject) => {
+			args.push(Buffer.isBuffer(file) ? "-" : file, outputPath);
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdftoppm"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdftoppm"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdErr = "";
+			let stdErr = "";
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				child.on("close", (code) => {
-					/* istanbul ignore else */
-					if (stdErr !== "") {
-						reject(new Error(stdErr.trim()));
-					} else if (code === 0) {
-						resolve(errorMessages[code]);
-					} else {
-						reject(
-							new Error(
-								// @ts-ignore: Second operand used if code is not in errorMessages
-								errorMessages[code] ||
-									`pdftoppm ${args.join(
-										" "
-									)} exited with code ${code}`
-							)
-						);
-					}
-				});
+			child.on("close", (code) => {
+				/* istanbul ignore else */
+				if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else if (code === 0) {
+					resolve(errorMessages[code]);
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in errorMessages
+							errorMessages[code] ||
+								`pdftoppm ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1383,68 +1343,61 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdftops"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdftops"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			return new Promise((resolve, reject) => {
-				args.push(
-					Buffer.isBuffer(file) ? "-" : file,
-					outputFile || "-"
-				);
+		return new Promise((resolve, reject) => {
+			args.push(Buffer.isBuffer(file) ? "-" : file, outputFile || "-");
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdftops"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdftops"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdOut = "";
-				let stdErr = "";
+			let stdOut = "";
+			let stdErr = "";
 
-				child.stdout.on("data", (data) => {
-					stdOut += data;
-				});
+			child.stdout.on("data", (data) => {
+				stdOut += data;
+			});
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				child.on("close", (code) => {
-					/* istanbul ignore else */
-					if (stdOut !== "") {
-						resolve(stdOut.trim());
-					} else if (code === 0) {
-						resolve(errorMessages[code]);
-					} else if (stdErr !== "") {
-						reject(new Error(stdErr.trim()));
-					} else {
-						reject(
-							new Error(
-								// @ts-ignore: Second operand used if code is not in errorMessages
-								errorMessages[code] ||
-									`pdftops ${args.join(
-										" "
-									)} exited with code ${code}`
-							)
-						);
-					}
-				});
+			child.on("close", (code) => {
+				/* istanbul ignore else */
+				if (stdOut !== "") {
+					resolve(stdOut.trim());
+				} else if (code === 0) {
+					resolve(errorMessages[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in errorMessages
+							errorMessages[code] ||
+								`pdftops ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1535,70 +1488,61 @@ class Poppler {
 			userPassword: { arg: "-upw", type: "string" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdftotext"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdftotext"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
 
-			return new Promise((resolve, reject) => {
-				args.push(
-					Buffer.isBuffer(file) ? "-" : file,
-					outputFile || "-"
-				);
+		return new Promise((resolve, reject) => {
+			args.push(Buffer.isBuffer(file) ? "-" : file, outputFile || "-");
 
-				const child = spawn(
-					pathResolve(this.#popplerPath, "pdftotext"),
-					args
-				);
+			const child = spawn(
+				pathResolve(this.#popplerPath, "pdftotext"),
+				args
+			);
 
-				if (Buffer.isBuffer(file)) {
-					child.stdin.write(file);
-					child.stdin.end();
-				}
+			if (Buffer.isBuffer(file)) {
+				child.stdin.write(file);
+				child.stdin.end();
+			}
 
-				let stdOut = "";
-				let stdErr = "";
+			let stdOut = "";
+			let stdErr = "";
 
-				child.stdout.on("data", (data) => {
-					stdOut += data;
-				});
+			child.stdout.on("data", (data) => {
+				stdOut += data;
+			});
 
-				child.stderr.on("data", (data) => {
-					stdErr += data;
-				});
+			child.stderr.on("data", (data) => {
+				stdErr += data;
+			});
 
-				child.on("close", (code) => {
-					/* istanbul ignore else */
-					if (stdOut !== "") {
-						resolve(
-							options.maintainLayout ? stdOut : stdOut.trim()
-						);
-					} else if (code === 0) {
-						resolve(errorMessages[code]);
-					} else if (stdErr !== "") {
-						reject(new Error(stdErr.trim()));
-					} else {
-						reject(
-							new Error(
-								// @ts-ignore: Second operand used if code is not in errorMessages
-								errorMessages[code] ||
-									`pdftotext ${args.join(
-										" "
-									)} exited with code ${code}`
-							)
-						);
-					}
-				});
+			child.on("close", (code) => {
+				/* istanbul ignore else */
+				if (stdOut !== "") {
+					resolve(options.maintainLayout ? stdOut : stdOut.trim());
+				} else if (code === 0) {
+					resolve(errorMessages[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in errorMessages
+							errorMessages[code] ||
+								`pdftotext ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1618,26 +1562,22 @@ class Poppler {
 			printVersionInfo: { arg: "-v", type: "boolean" },
 		};
 
-		try {
-			const { stderr } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfunite"),
-				["-v"]
-			);
+		const { stderr } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfunite"),
+			["-v"]
+		);
 
-			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+		// @ts-ignore: parseOptions checks if falsy
+		const versionInfo = popplerVersionRegex.exec(stderr)[1];
 
-			const args = parseOptions(acceptedOptions, options, versionInfo);
-			args.push(...files, outputFile);
+		const args = parseOptions(acceptedOptions, options, versionInfo);
+		args.push(...files, outputFile);
 
-			const { stdout } = await execFileAsync(
-				pathResolve(this.#popplerPath, "pdfunite"),
-				args
-			);
-			return Promise.resolve(stdout);
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		const { stdout } = await execFileAsync(
+			pathResolve(this.#popplerPath, "pdfunite"),
+			args
+		);
+		return stdout;
 	}
 }