Merge pull request #39 from 343dev/bugfix/alpine

Add Alpine Linux support without additional packages

Author: 343dev

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [12.1.1] - 2025-01-02
+
+### Changed
+
+- Updated `@343dev/gifsicle` and `@343dev/guetzli` packages, which now allow Optimizt to work on Alpine Linux without installing additional packages. Previously, the `gcompat` package was required as the `guetzli` and `gifsicle` binaries were built for glibc-based systems, which is absent on Alpine Linux.
+
 ## [12.1.0] - 2025-12-24
 
 ### Added

cli.js

@@ -52,6 +52,6 @@ if (program.args.length === 0) {
 	});
 }
 
-process.on('unhandledRejection', error => {
+process.on('unhandledRejection', (error) => {
 	console.error(error);
 });

convert.js

@@ -54,45 +54,43 @@ export async function convert({ filePaths, config }) {
 	const cpuCount = os.cpus().length;
 	const tasksSimultaneousLimit = pLimit(cpuCount);
 
-	await Promise.all(
-		filePaths.reduce((accumulator, filePath) => {
-			if (shouldConvertToAvif) {
-				accumulator.push(
-					tasksSimultaneousLimit(
-						() => processFile({
-							filePath,
-							config: avifConfig || {},
-							progressBarContainer,
-							progressBar,
-							totalSize,
-							isForced,
-							format: 'AVIF',
-							processFunction: processAvif,
-						}),
-					),
-				);
-			}
-
-			if (shouldConvertToWebp) {
-				accumulator.push(
-					tasksSimultaneousLimit(
-						() => processFile({
-							filePath,
-							config: webpConfig || {},
-							progressBarContainer,
-							progressBar,
-							totalSize,
-							isForced,
-							format: 'WebP',
-							processFunction: processWebp,
-						}),
-					),
-				);
-			}
-
-			return accumulator;
-		}, []),
-	);
+	const tasks = [];
+	for (const filePath of filePaths) {
+		if (shouldConvertToAvif) {
+			tasks.push(
+				tasksSimultaneousLimit(
+					() => processFile({
+						filePath,
+						config: avifConfig || {},
+						progressBarContainer,
+						progressBar,
+						totalSize,
+						isForced,
+						format: 'AVIF',
+						processFunction: processAvif,
+					}),
+				),
+			);
+		}
+
+		if (shouldConvertToWebp) {
+			tasks.push(
+				tasksSimultaneousLimit(
+					() => processFile({
+						filePath,
+						config: webpConfig || {},
+						progressBarContainer,
+						progressBar,
+						totalSize,
+						isForced,
+						format: 'WebP',
+						processFunction: processWebp,
+					}),
+				),
+			);
+		}
+	}
+	await Promise.all(tasks);
 
 	progressBarContainer.update(); // Prevent logs lost. See: https://github.com/npkgz/cli-progress/issues/145#issuecomment-1859594159
 	progressBarContainer.stop();

eslint.config.js

@@ -4,6 +4,9 @@ export default [
 	...config,
 
 	{
-		ignores: ['.optimiztrc.cjs'],
+		ignores: [
+			'coverage/',
+			'.optimiztrc.cjs',
+		],
 	},
 ];

lib/find-config-file-path.js

@@ -31,7 +31,7 @@ export async function findConfigFilePath(providedConfigPath) {
 		const currentConfigPath = path.join(currentDirectoryPath, DEFAULT_CONFIG_FILENAME);
 
 		try {
-			const stat = await fs.promises.stat(currentConfigPath); // eslint-disable-line no-await-in-loop
+			const stat = await fs.promises.stat(currentConfigPath);
 
 			if (stat.isFile()) {
 				return currentConfigPath;

lib/log.js

@@ -49,7 +49,7 @@ export function log(title, { type, description } = {}) {
 
 export function logErrorAndExit(title) {
 	log(title, { type: LOG_TYPES.ERROR });
-	process.exit(1); // eslint-disable-line unicorn/no-process-exit
+	process.exit(1); // eslint-disable-line n/no-process-exit
 }
 
 export function logEmptyLine() {

lib/prepare-file-paths.js

@@ -20,7 +20,7 @@ export async function prepareFilePaths({
 	const inputPathsSet = new Set(inputPaths);
 
 	await Promise.all(
-		[...inputPathsSet].map(async currentPath => {
+		[...inputPathsSet].map(async (currentPath) => {
 			try {
 				const stat = await fs.promises.stat(currentPath);
 				const resolvedPath = path.resolve(currentPath);
@@ -30,11 +30,13 @@ export async function prepareFilePaths({
 				} else if (stat.isFile() && checkFileType(resolvedPath, extensions)) {
 					files.add(resolvedPath);
 				}
-			} catch {}
+			} catch {
+				// ¯\_(ツ)_/¯
+			}
 		}),
 	);
 
-	const crawler = new fdir() // eslint-disable-line new-cap
+	const crawler = new fdir()
 		.withFullPaths()
 		.withDirs()
 		.filter(currentPath => checkFileType(currentPath, extensions));
@@ -49,7 +51,7 @@ export async function prepareFilePaths({
 
 	const hasDirectories = directories.size > 0;
 
-	const result = [...files].map(filePath => {
+	const result = [...files].map((filePath) => {
 		let outputPath = filePath;
 
 		if (outputDirectoryPath) {

optimize.js

@@ -46,7 +46,7 @@ export async function optimize({ filePaths, config }) {
 	const guetzliTasksSimultaneousLimit = pLimit(1); // Guetzli uses a large amount of memory and a significant amount of CPU time. To reduce system load, we only allow one instance of guetzli to run at the same time.
 
 	await Promise.all(
-		filePaths.map(filePath => {
+		filePaths.map((filePath) => {
 			const extension = path.extname(filePath.input).toLowerCase();
 			const isJpeg = extension === '.jpg' || extension === '.jpeg';
 
@@ -231,15 +231,15 @@ function pipe({ command, commandOptions, inputBuffer }) {
 		process.stdin.end();
 
 		const stdoutChunks = [];
-		process.stdout.on('data', chunk => {
+		process.stdout.on('data', (chunk) => {
 			stdoutChunks.push(chunk);
 		});
 
-		process.on('error', error => {
+		process.on('error', (error) => {
 			reject(new Error(`Error processing image: ${error.message}`));
 		});
 
-		process.on('close', code => {
+		process.on('close', (code) => {
 			if (code !== 0) {
 				reject(new Error(`Image optimization process exited with code ${code}`));
 				return;