node-poppler 8.0.1 → 8.0.3

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.1",
+	"version": "8.0.3",
 	"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,14 +1,15 @@
 "use strict";
 
 const { execFile, spawn, spawnSync } = require("node:child_process");
+const { normalize, resolve: pathResolve } = require("node:path");
+const { platform } = require("node:process");
 const { promisify } = require("node:util");
 const camelCase = require("camelcase");
 const { lt } = require("semver");
-const { normalize, resolve: pathResolve } = require("node:path");
 
 const execFileAsync = promisify(execFile);
 
-const errorMessages = {
+const ERROR_MSGS = {
 	0: "No Error",
 	1: "Error opening a PDF file",
 	2: "Error opening an output file",
@@ -19,8 +20,9 @@ const errorMessages = {
 };
 
 // Cache immutable regex as they are expensive to create and garbage collect
-const popplerVersionRegex = /(\d{1,2}\.\d{1,2}\.\d{1,2})/u;
-const pdfInfoFileSizesRegex = /(File\s+size:\s+)0(\s+)bytes/u;
+const POPPLER_VERSION_REG = /(\d{1,2}\.\d{1,2}\.\d{1,2})/u;
+const PDF_INFO_FILE_SIZES_REG = /(File\s+size:\s+)0(\s+)bytes/u;
+const PDF_INFO_PATH_REG = /(.+)pdfinfo/u;
 
 /**
  * @typedef {object} OptionDetails
@@ -50,10 +52,8 @@ function parseOptions(acceptedOptions, options, version) {
 	const args = [];
 	/** @type {string[]} */
 	const invalidArgs = [];
-	/**
-	 * Imperative loops are faster than functional loops.
-	 * @see {@link https://romgrk.com/posts/optimizing-javascript#3-avoid-arrayobject-methods || Optimizing JavaScript}
-	 */
+
+	// 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) {
@@ -63,7 +63,6 @@ function parseOptions(acceptedOptions, options, version) {
 			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) {
@@ -122,12 +121,10 @@ class Poppler {
 			/** @type {string|undefined} */
 			this.#popplerPath = binPath;
 		} else {
-			const { platform } = process;
-
 			const which = spawnSync(platform === "win32" ? "where" : "which", [
 				"pdfinfo",
 			]).stdout.toString();
-			const popplerPath = /(.+)pdfinfo/u.exec(which)?.[1];
+			const popplerPath = PDF_INFO_PATH_REG.exec(which)?.[1];
 
 			if (popplerPath) {
 				this.#popplerPath = popplerPath;
@@ -147,7 +144,7 @@ class Poppler {
 		/* istanbul ignore next: unable to test due to https://github.com/jestjs/jest/pull/14297 */
 		if (!this.#popplerPath) {
 			throw new Error(
-				`Unable to find ${process.platform} Poppler binaries, please pass the installation directory as a parameter to the Poppler instance.`
+				`Unable to find ${platform} Poppler binaries, please pass the installation directory as a parameter to the Poppler instance.`
 			);
 		}
 		this.#popplerPath = normalize(this.#popplerPath);
@@ -179,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;
 	}
 
 	/**
@@ -237,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;
 	}
 
 	/**
@@ -276,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 = POPPLER_VERSION_REG.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(ERROR_MSGS[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+							ERROR_MSGS[code] ||
+								`pdffonts ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -379,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 = POPPLER_VERSION_REG.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(ERROR_MSGS[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+							ERROR_MSGS[code] ||
+								`pdfimages ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -500,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 = POPPLER_VERSION_REG.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(
+							PDF_INFO_FILE_SIZES_REG,
+							`$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(ERROR_MSGS[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+							ERROR_MSGS[code] ||
+								`pdfinfo ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -624,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 = POPPLER_VERSION_REG.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;
 	}
 
 	/**
@@ -816,7 +782,7 @@ class Poppler {
 			);
 
 			// @ts-ignore: parseOptions checks if falsy
-			const versionInfo = popplerVersionRegex.exec(stderr)[1];
+			const versionInfo = POPPLER_VERSION_REG.exec(stderr)[1];
 
 			const args = parseOptions(acceptedOptions, options, versionInfo);
 
@@ -859,14 +825,14 @@ class Poppler {
 					if (stdOut !== "") {
 						resolve(stdOut.trim());
 					} else if (code === 0) {
-						resolve(errorMessages[code]);
+						resolve(ERROR_MSGS[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] ||
+								// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+								ERROR_MSGS[code] ||
 									`pdftocairo ${args.join(
 										" "
 									)} exited with code ${code}`
@@ -952,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 = POPPLER_VERSION_REG.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);
-		}
+		});
 	}
 
 	/**
@@ -1154,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 = POPPLER_VERSION_REG.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(ERROR_MSGS[code]);
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+							ERROR_MSGS[code] ||
+								`pdftoppm ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1388,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 = POPPLER_VERSION_REG.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(ERROR_MSGS[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+							ERROR_MSGS[code] ||
+								`pdftops ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1540,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 = POPPLER_VERSION_REG.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(ERROR_MSGS[code]);
+				} else if (stdErr !== "") {
+					reject(new Error(stdErr.trim()));
+				} else {
+					reject(
+						new Error(
+							// @ts-ignore: Second operand used if code is not in ERROR_MSGS
+							ERROR_MSGS[code] ||
+								`pdftotext ${args.join(
+									" "
+								)} exited with code ${code}`
+						)
+					);
+				}
 			});
-		} catch (err) {
-			return Promise.reject(err);
-		}
+		});
 	}
 
 	/**
@@ -1623,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 = POPPLER_VERSION_REG.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;
 	}
 }