image-guard 5.2.4 → 5.2.5

package/CHANGELOG.md

@@ -4,6 +4,13 @@ Starting with version 5.2.0, all notable changes to Image Guard are documented i
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and the project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [5.2.5] - 2026-03-17
+
+### Changed
+
+* Added recommended ESLint rules to config (and fixed resulting warnings)
+* Promoted `no-unused-vars` and `eqeqeq` from warnings to errors in ESLint config
+
 ## [5.2.4] - 2026-03-04
 
 ### Fixed

package/bin/image-guard.test.js

@@ -305,7 +305,7 @@ describe('Image Guard', () => {
     copyFiles(testFolder, tempTestFolder)
 
     const originalCwd = process.cwd()
-    let stdout = ''
+    let stdout
     try {
       process.chdir(tempDir)
       stdout = execSync(`node "${imageGuardScript}" --quiet`, { encoding: 'utf8' })
@@ -333,7 +333,7 @@ describe('Image Guard', () => {
     })
 
     const originalCwd = process.cwd()
-    let stdout = ''
+    let stdout
     try {
       process.chdir(tempDir)
       stdout = execSync(`node "${imageGuardScript}" --dry --quiet`, { encoding: 'utf8' })
@@ -467,7 +467,7 @@ describe('Image Guard', () => {
     const filesBefore = fs.readdirSync(tempTestFolder).sort()
 
     const originalCwd = process.cwd()
-    let output = ''
+    let output
     try {
       process.chdir(tempDir)
       output = execSync(`node "${imageGuardScript}" --heic-to-avif --dry`, { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] })
@@ -520,7 +520,7 @@ describe('Image Guard', () => {
     copyFiles(testFolder, tempTestFolder)
 
     const originalCwd = process.cwd()
-    let output = ''
+    let output
     try {
       process.chdir(tempDir)
       output = execSync(`node "${imageGuardScript}"`, { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] })
@@ -538,7 +538,7 @@ describe('Image Guard', () => {
     copyFiles(testFolder, tempTestFolder)
 
     const originalCwd = process.cwd()
-    let output = ''
+    let output
     try {
       process.chdir(tempDir)
       output = execSync(`node "${imageGuardScript}" --keep-heic 2>&1`, { encoding: 'utf8' })

package/eslint.config.js

@@ -1,3 +1,5 @@
+import js from '@eslint/js';
+
 export default [
   {
     ignores: [
@@ -5,6 +7,7 @@ export default [
       'node_modules/**'
     ]
   },
+  js.configs.recommended,
   {
     languageOptions: {
       ecmaVersion: 'latest',
@@ -17,9 +20,8 @@ export default [
     },
     rules: {
       'no-console': 'off', // CLI tool—allow console
-      'no-undef': 'error', // Catch undefined vars
-      'no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
-      eqeqeq: ['warn', 'smart'],
+      'no-unused-vars': ['error', { argsIgnorePattern: '^_' }],
+      eqeqeq: ['error', 'smart'],
       curly: ['warn', 'multi-line']
     }
   }

package/package.json

@@ -13,6 +13,7 @@
   },
   "description": "Near-lossless image compressor that’s easy to use and easy to automate",
   "devDependencies": {
+    "@eslint/js": "^10.0.0",
     "eslint": "^10.0.0"
   },
   "funding": "https://github.com/j9t/image-guard?sponsor=1",
@@ -46,5 +47,5 @@
     "test": "node --test bin/image-guard.test.js"
   },
   "type": "module",
-  "version": "5.2.4"
+  "version": "5.2.5"
 }

package/src/index.js

@@ -179,7 +179,7 @@ export async function runImageGuard() {
   }
 
   const doConversion = argv['heic-to-avif']
-  let hadFailures = false
+  let hadFailures
   let totalFiles = 0
 
   if (argv.staged) {

package/src/utils.js

@@ -283,6 +283,6 @@ const size = async (file) => {
   return stats.size
 }
 
-const sizeReadable = (size) => `${(size / 1024).toFixed(2)} KB`
+const sizeReadable = (size) => `${(size / 1024).toFixed(2)} KB` // eslint-disable-line no-irregular-whitespace
 
 export const utils = { compression, conversion, sizeReadable }