node-poppler 8.0.2 → 8.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "node-poppler",
-	"version": "8.0.2",
+	"version": "8.0.3",
 	"description": "Asynchronous Node.js wrapper for the Poppler PDF rendering library",
 	"keywords": [
 		"async",

package/src/index.js

@@ -2,13 +2,14 @@
 
 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 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
@@ -119,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;
@@ -144,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);
@@ -271,7 +271,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);
 
@@ -304,14 +304,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] ||
 								`pdffonts ${args.join(
 									" "
 								)} exited with code ${code}`
@@ -370,7 +370,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);
 
@@ -407,14 +407,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] ||
 								`pdfimages ${args.join(
 									" "
 								)} exited with code ${code}`
@@ -487,7 +487,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);
 
@@ -529,7 +529,7 @@ class Poppler {
 				if (stdOut !== "") {
 					if (fileSize) {
 						stdOut = stdOut.replace(
-							pdfInfoFileSizesRegex,
+							PDF_INFO_FILE_SIZES_REG,
 							`$1${fileSize}$2bytes`
 						);
 					}
@@ -551,14 +551,14 @@ class Poppler {
 						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] ||
 								`pdfinfo ${args.join(
 									" "
 								)} exited with code ${code}`
@@ -600,7 +600,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);
 		args.push(file, outputPattern);
@@ -782,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);
 
@@ -825,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}`
@@ -924,7 +924,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);
 
@@ -1119,7 +1119,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);
 
@@ -1147,12 +1147,12 @@ class Poppler {
 				if (stdErr !== "") {
 					reject(new Error(stdErr.trim()));
 				} else if (code === 0) {
-					resolve(errorMessages[code]);
+					resolve(ERROR_MSGS[code]);
 				} 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] ||
 								`pdftoppm ${args.join(
 									" "
 								)} exited with code ${code}`
@@ -1349,7 +1349,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);
 
@@ -1382,14 +1382,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] ||
 								`pdftops ${args.join(
 									" "
 								)} exited with code ${code}`
@@ -1494,7 +1494,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);
 
@@ -1527,14 +1527,14 @@ class Poppler {
 				if (stdOut !== "") {
 					resolve(options.maintainLayout ? stdOut : 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] ||
 								`pdftotext ${args.join(
 									" "
 								)} exited with code ${code}`
@@ -1568,7 +1568,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);
 		args.push(...files, outputFile);