html-minifier-next 7.5.2 → 7.5.3

package/README.md

@@ -1,6 +1,6 @@
 # HTML Minifier Next
 
-[![npm version](https://img.shields.io/npm/v/html-minifier-next.svg)](https://www.npmjs.com/package/html-minifier-next) [![Build status](https://github.com/j9t/html-minifier-next/workflows/Tests/badge.svg)](https://github.com/j9t/html-minifier-next/actions) [![Socket](https://badge.socket.dev/npm/package/html-minifier-next)](https://socket.dev/npm/package/html-minifier-next) [![GitHub Sponsors](https://badgen.net/static/Support/Open%20Source/cyan)](https://github.com/j9t/html-minifier-next?sponsor=1)
+[![npm version](https://img.shields.io/npm/v/html-minifier-next.svg)](https://www.npmjs.com/package/html-minifier-next) [![Build status](https://github.com/j9t/html-minifier-next/workflows/Tests/badge.svg)](https://github.com/j9t/html-minifier-next/actions) [![Socket](https://badge.socket.dev/npm/package/html-minifier-next)](https://socket.dev/npm/package/html-minifier-next) [![GitHub Sponsors](https://badgen.net/static/Support/Open%20Source/cyan)](https://github.com/sponsors/j9t)
 
 Your web page optimization precision tool: HTML Minifier Next (HMN) is a **highly effective, super-configurable, well-tested HTML minifier**, written in JavaScript, that also handles in-document CSS, JavaScript, and SVG minification.
 
@@ -40,13 +40,13 @@ Use `npx html-minifier-next --help` to check all available options:
 | `--output <file>`, `-o <file>` | Specify output file (reads from `--input` file argument or STDIN; outputs to STDOUT if not specified) | File to file: `npx html-minifier-next input.html -o output.html`<br>File to file (explicit): `npx html-minifier-next -i input.html -o output.html`<br>Pipe to file: `cat input.html \| npx html-minifier-next -o output.html`<br>File to STDOUT: `npx html-minifier-next input.html` |
 | `--file-ext <extensions>`, `-f <extensions>` | Specify file extension(s) to process (comma-separated, overrides config file setting); defaults to `html,htm,shtml,shtm`; use `*` for all files | `--file-ext=html,php`, `--file-ext='*'` |
 | `--preset <name>`, `-p <name>` | Use a preset configuration (conservative or comprehensive) | `--preset=conservative` |
-| `--config-file <file>`, `-c <file>` | Use a configuration file (defaults to `html-minifier-next.config.json` in the working directory, if present) | `--config-file=path/to/config.json` |
+| `--config-file <file>`, `-c <file>` | Use a configuration file (defaults to html-minifier-next.config.json in the working directory, if present) | `--config-file=path/to/config.json` |
 | `--verbose`, `-v` | Show detailed processing information (active options, file statistics) | `npx html-minifier-next --input-dir=src --output-dir=dist --verbose --collapse-whitespace` |
 | `--dry`, `-d` | Dry run: Process and report statistics without writing output | `npx html-minifier-next input.html --dry --collapse-whitespace` |
 
 ### Configuration file
 
-You can use a configuration file to specify options. If an `html-minifier-next.config.json` file is present in the current working directory—only there; subfolders (including `--input-dir`) and parent folders are not searched—the CLI picks it up automatically, with a note on STDERR confirming this. An explicit `--config-file` takes precedence. (The standalone `--zero` mode is deliberately config-free and ignores default config files.) The file can be either in JSON format or a JavaScript module that exports the configuration object:
+You can use a configuration file to specify options. If an html-minifier-next.config.json file is present in the current working directory—only there; subfolders (including `--input-dir`) and parent folders are not searched—the CLI picks it up automatically, with a note on STDERR confirming this. An explicit `--config-file` takes precedence. (The standalone `--zero` mode is deliberately config-free and ignores default config files.) The file can be either in JSON format or a JavaScript module that exports the configuration object:
 
 **JSON configuration example:**
 
@@ -645,7 +645,7 @@ Parameters:
 * `--save`: Saves the run as the baseline (e.g., on `main` before switching to a branch)
 * `--core`: Disables the external minifiers (CSS, JS, SVG, URLs) to isolate HMN’s own processing time
 * `--iterations=N`: Sets the number of timed iterations (default 5; the median is reported)
-* `--config=PATH`: Uses an alternative options file (default `html-minifier-next.config.json`)
+* `--config=PATH`: Uses an alternative options file (default html-minifier-next.config.json)
 
 To compare branches (A/B run), execute `npm run benchmark -- --save` on `main`, then `npm run benchmark` on the branch to see the deltas. Add `--core` on both ends when measuring changes to HMN’s own code rather than the bundled minifiers.
 

package/cli.js

@@ -535,7 +535,7 @@ program.helpOption('-h, --help', 'Display help for command');
   }
 
   /**
-   * Parse comma-separated ignore patterns into an array
+   * Parse comma-separated ignore patterns into an array.
    * @param {string} patterns - Comma-separated directory patterns (e.g., "libs,vendor")
    * @returns {string[]} Array of trimmed pattern strings with normalized separators
    */
@@ -548,8 +548,8 @@ program.helpOption('-h, --help', 'Display help for command');
   }
 
   /**
-   * Check if a directory should be ignored based on ignore patterns
-   * Supports matching by directory name or relative path
+   * Check if a directory should be ignored based on ignore patterns.
+   * Supports matching by directory name or relative path.
    * @param {string} dirPath - Absolute path to the directory
    * @param {string[]} ignorePatterns - Array of patterns to match against (with forward slashes)
    * @param {string} baseDir - Base directory for relative path calculation
@@ -791,9 +791,8 @@ program.helpOption('-h, --help', 'Display help for command');
       }
 
       // Prevent traversing into the output directory when it is inside the input directory
-      let inputReal;
       let outputReal;
-      inputReal = await fs.promises.realpath(inputDir).catch(() => undefined);
+      const inputReal = await fs.promises.realpath(inputDir).catch(() => undefined);
       try {
         outputReal = await fs.promises.realpath(outputDir);
       } catch {

package/dist/types/lib/urls.d.ts

@@ -7,7 +7,7 @@
  * - Otherwise: normalized absolute URL
  */
 /**
- * Create a URL minifier function for the given site context
+ * Create a URL minifier function for the given site context.
  * @param {string} site - The site base URL (used to compute relative URLs)
  * @returns {(url: string) => string} Minifier function that returns the shortest URL
  */

package/dist/types/lib/utils.d.ts

@@ -42,7 +42,7 @@ declare function isThenable(value: unknown): value is PromiseLike<any>;
 /** @param {string} value */
 declare function lowercase(value: string): string;
 /**
- * Asynchronously replace matches in a string
+ * Asynchronously replace matches in a string.
  * @param {string} str - Input string
  * @param {RegExp} regex - Regular expression with global flag
  * @param {Function} asyncFn - Async function to process each match

package/dist/types/presets.d.ts

@@ -36,7 +36,7 @@ export declare const presets: {
     };
 };
 /**
- * Get preset configuration by name
+ * Get preset configuration by name.
  * @param {string} name - Preset name (“conservative” or “comprehensive”)
  * @returns {object|null} Preset options object or null if not found
  */

package/package.json

@@ -10,17 +10,18 @@
     "entities": "^8.0.0",
     "lightningcss": "^1.33.0",
     "svgo": "^4.0.2",
-    "terser": "^5.49.0"
+    "terser": "^5.50.0"
   },
   "description": "Highly effective, super-configurable, well-tested web page minifier (enhanced successor to HTML Minifier)",
   "devDependencies": {
-    "@commitlint/cli": "^21.2.1",
+    "@commitlint/cli": "^21.2.2",
     "@eslint/js": "^10.0.1",
-    "@swc/core": "^1.15.46",
-    "@types/node": "^26.1.1",
-    "eslint": "^10.8.0",
+    "@swc/core": "^1.15.47",
+    "@types/node": "^26.2.0",
+    "eslint": "^10.8.1",
+    "globals": "^17.11.0",
     "typescript": "^7.0.2",
-    "vite": "^8.1.5"
+    "vite": "^8.2.1"
   },
   "engines": {
     "node": ">=22.13"
@@ -88,11 +89,11 @@
     "prepack": "npm run build",
     "prepare": "git config core.hooksPath .githooks || true",
     "serve": "vite",
-    "test": "npm run test:types && node --test tests/*.spec.js",
+    "test": "npm run test:types && node --test \"test/**/*.test.js\"",
     "test:types": "tsc --noEmit && tsc --project tsconfig.test.json",
-    "test:watch": "node --test --watch tests/*.spec.js"
+    "test:watch": "node --test --watch \"test/**/*.test.js\""
   },
   "type": "module",
   "types": "dist/types/htmlminifier.d.ts",
-  "version": "7.5.2"
+  "version": "7.5.3"
 }

package/src/htmlminifier.js

@@ -910,23 +910,23 @@ async function createSortFns(value, options, uidIgnore, uidAttr, ignoredMarkupCh
     // Memoize `sortClassNames` results—class lists often repeat in templates
     const classNameCache = new LRU(500);
 
-    options.sortClassNames = function (/** @type {string} */ value) {
+    options.sortClassNames = function (/** @type {string} */ classNames) {
       // Fast path: Single class (no spaces) needs no sorting
-      if (value.indexOf(' ') === -1) {
-        return value;
+      if (classNames.indexOf(' ') === -1) {
+        return classNames;
       }
 
       // Check cache first
-      const cached = /** @type {string | undefined} */ (classNameCache.get(value));
+      const cached = /** @type {string | undefined} */ (classNameCache.get(classNames));
       if (cached !== undefined) {
         return cached;
       }
 
       // Expand UID tokens back to original content before sorting
       // Fast path: Skip if no HTML comments (UID markers) present
-      let expandedValue = value;
-      if (uidReplacePattern && value.indexOf('<!--') !== -1) {
-        expandedValue = value.replace(uidReplacePattern, function (/** @type {string} */ _match, /** @type {string} */ index) {
+      let expandedValue = classNames;
+      if (uidReplacePattern && classNames.indexOf('<!--') !== -1) {
+        expandedValue = classNames.replace(uidReplacePattern, function (/** @type {string} */ _match, /** @type {string} */ index) {
           return ignoredMarkupChunks[+index] || '';
         });
         // Reset `lastIndex` for pattern reuse
@@ -939,7 +939,7 @@ async function createSortFns(value, options, uidIgnore, uidAttr, ignoredMarkupCh
       const result = sorted.join(' ');
 
       // Cache the result
-      classNameCache.set(value, result);
+      classNameCache.set(classNames, result);
       return result;
     };
   }
@@ -1148,11 +1148,11 @@ async function minifyHTML(value, options, partialMarkup) {
 
   // Look for trailing whitespaces, bypass any inline tags
   function trimTrailingWhitespace(/** @type {number} */ index, /** @type {string} */ nextTag) {
-    for (let endTag = ''; index >= 0 && canTrimWhitespace(endTag, emptyAttrs); index--) {
+    for (let prevTag = ''; index >= 0 && canTrimWhitespace(prevTag, emptyAttrs); index--) {
       const str = buffer[index] ?? '';
       const match = str.match(/^<\/([\w:-]+)>$/);
       if (match) {
-        endTag = match[1] ?? '';
+        prevTag = match[1] ?? '';
       } else if (/>$/.test(str) || (buffer[index] = collapseWhitespaceSmart(str, '', nextTag, emptyAttrs, emptyAttrs, options, inlineElements, inlineTextSet))) {
         break;
       }
@@ -1398,13 +1398,13 @@ async function minifyHTML(value, options, partialMarkup) {
                   // Collapse if both sides are element/closing tags or HTML comments, and neither is inline
                   if ((currentTagMatch || currentIsHtmlComment || currentClosingTagMatch) &&
                       (prevTagMatch || prevIsHtmlComment || prevClosingTagMatch)) {
-                    const currentTag = currentTagMatch ? resolveName(currentTagMatch[1] ?? '')
+                    const currentTagName = currentTagMatch ? resolveName(currentTagMatch[1] ?? '')
                       : currentClosingTagMatch ? resolveName(currentClosingTagMatch[1] ?? '') : null;
-                    const prevTag = prevTagMatch ? resolveName(prevTagMatch[1] ?? '')
+                    const prevTagName = prevTagMatch ? resolveName(prevTagMatch[1] ?? '')
                       : prevClosingTagMatch ? resolveName(prevClosingTagMatch[1] ?? '') : null;
 
                     // Don’t collapse between inline elements (HTML comments count as non-inline)
-                    if (!inlineElements.has(currentTag ?? '') && !inlineElements.has(prevTag ?? '')) {
+                    if (!inlineElements.has(currentTagName ?? '') && !inlineElements.has(prevTagName ?? '')) {
                       // Collapse whitespace respecting context rules
                       let collapsedText = prevText;
 

package/src/htmlparser.js

@@ -154,7 +154,7 @@ function buildAttrRegex(handler) {
 
 /** @param {HTMLParserHandler} handler */
 function getAttrRegexForHandler(handler) {
-  let cached = attrRegexCache.get(handler);
+  const cached = attrRegexCache.get(handler);
   if (cached) return cached;
   const compiled = buildAttrRegex(handler);
   attrRegexCache.set(handler, compiled);
@@ -166,7 +166,7 @@ const attrRegexStickyCache = new WeakMap();
 
 /** @param {HTMLParserHandler} handler */
 function getAttrRegexStickyForHandler(handler) {
-  let cached = attrRegexStickyCache.get(handler);
+  const cached = attrRegexStickyCache.get(handler);
   if (cached) return cached;
   const nonSticky = getAttrRegexForHandler(handler);
   // Derive sticky version: Remove `^` anchor, add `y` flag
@@ -657,12 +657,12 @@ export class HTMLParser {
 
     // `needle` must already be lowercase
     function findTagInCurrentTable(/** @type {string} */ needle) {
-      let pos;
-      for (pos = stack.length - 1; pos >= 0; pos--) {
-        const entry = stack[pos];
+      let stackIndex;
+      for (stackIndex = stack.length - 1; stackIndex >= 0; stackIndex--) {
+        const entry = stack[stackIndex];
         const currentTag = entry?.lowerTag;
         if (currentTag === needle) {
-          return pos;
+          return stackIndex;
         }
         // Stop searching if hitting a table boundary
         if (currentTag === 'table') {
@@ -672,25 +672,25 @@ export class HTMLParser {
       return -1;
     }
 
-    function parseEndTagAt(/** @type {number} */ pos) {
-      // Close all open elements up to `pos` (mirrors `parseEndTag`’s core branch);
+    function parseEndTagAt(/** @type {number} */ stackIndex) {
+      // Close all open elements up to `stackIndex` (mirrors `parseEndTag`’s core branch);
       // `end` handlers are synchronous—invoked without awaiting, as in `parseEndTag`
-      for (let i = stack.length - 1; i >= pos; i--) {
+      for (let i = stack.length - 1; i >= stackIndex; i--) {
         const entry = /** @type {NonNullable<(typeof stack)[number]>} */ (stack[i]);
         if (handler.end) {
           handler.end(entry.tag, entry.attrs, true);
         }
       }
-      stack.length = pos;
-      lastTag = pos ? (stack[pos - 1]?.tag ?? '') : '';
-      lastTagLower = pos ? (stack[pos - 1]?.lowerTag ?? '') : '';
+      stack.length = stackIndex;
+      lastTag = stackIndex ? (stack[stackIndex - 1]?.tag ?? '') : '';
+      lastTagLower = stackIndex ? (stack[stackIndex - 1]?.lowerTag ?? '') : '';
     }
 
     function closeIfFoundInCurrentTable(/** @type {string} */ tagName) {
-      const pos = findTagInCurrentTable(tagName);
-      if (pos >= 0) {
+      const stackIndex = findTagInCurrentTable(tagName);
+      if (stackIndex >= 0) {
         // Close at the specific index to avoid re-searching
-        parseEndTagAt(pos);
+        parseEndTagAt(stackIndex);
         return true;
       }
       return false;
@@ -812,38 +812,38 @@ export class HTMLParser {
 
     // `needle` must already be lowercase
     function findTag(/** @type {string} */ needle) {
-      let pos;
-      for (pos = stack.length - 1; pos >= 0; pos--) {
-        if (stack[pos]?.lowerTag === needle) {
+      let stackIndex;
+      for (stackIndex = stack.length - 1; stackIndex >= 0; stackIndex--) {
+        if (stack[stackIndex]?.lowerTag === needle) {
           break;
         }
       }
-      return pos;
+      return stackIndex;
     }
 
     async function parseEndTag(/** @type {string} */ tag, /** @type {string} */ tagName) {
-      let pos;
+      let stackIndex;
       const lowerTagName = tagName ? tagName.toLowerCase() : '';
 
       // Find the closest opened tag of the same type
       if (tagName) {
-        pos = findTag(lowerTagName);
+        stackIndex = findTag(lowerTagName);
       } else { // If no tag name is provided, clean shop
-        pos = 0;
+        stackIndex = 0;
       }
 
-      if (pos >= 0) {
+      if (stackIndex >= 0) {
         // Close all the open elements, up the stack
-        for (let i = stack.length - 1; i >= pos; i--) {
+        for (let i = stack.length - 1; i >= stackIndex; i--) {
           if (handler.end) {
-            handler.end(stack[i]?.tag, stack[i]?.attrs, i > pos || !tag);
+            handler.end(stack[i]?.tag, stack[i]?.attrs, i > stackIndex || !tag);
           }
         }
 
         // Remove the open elements from the stack
-        stack.length = pos;
-        lastTag = pos ? (stack[pos - 1]?.tag ?? '') : '';
-        lastTagLower = pos ? (stack[pos - 1]?.lowerTag ?? '') : '';
+        stack.length = stackIndex;
+        lastTag = stackIndex ? (stack[stackIndex - 1]?.tag ?? '') : '';
+        lastTagLower = stackIndex ? (stack[stackIndex - 1]?.lowerTag ?? '') : '';
       } else if (handler.partialMarkup && tagName) {
         // In partial markup mode, preserve stray end tags
         if (handler.end) {

package/src/lib/attributes.js

@@ -487,7 +487,7 @@ function cleanAttributeValue(tag, attrName, attrValue, options, attrs, minifyHTM
       }
       // Sync path (`minifyCSS` disabled—identity function)
       if (cssResult && /^(?:[a-z-]+:[;\s]*)+$/i.test(cssResult)) return '';
-      return cssResult != null ? cssResult : attrValue;
+      return cssResult ?? attrValue;
     }
     return attrValue;
   }
@@ -589,7 +589,7 @@ function cleanAttributeValue(tag, attrName, attrValue, options, attrs, minifyHTM
         return originalAttrValue;
       });
     }
-    return cssResult != null ? cssResult : attrValue;
+    return cssResult ?? attrValue;
   }
 
   if (tag === 'iframe' && attrName === 'srcdoc') {
@@ -635,7 +635,7 @@ function chooseAttributeQuote(attrValue, options) {
  */
 function normalizeAttr(attr, attrs, tag, options, minifyHTML) {
   const attrName = options.name(attr.name);
-  let attrValue = attr.value;
+  const attrValue = attr.value;
 
   // Entity decoding requires a lazy import—async only when `&` is present
   if (options.decodeEntities && attrValue && attrValue.indexOf('&') !== -1) {

package/src/lib/urls.js

@@ -12,7 +12,7 @@ const REJECTED_SCHEMES = new Set(['data:', 'javascript:', 'mailto:']);
 const DIRECTORY_INDEXES = ['index.html', 'index.htm'];
 
 /**
- * Get the directory portion of a pathname (up to and including the last `/`)
+ * Get the directory portion of a pathname (up to and including the last `/`).
  * @param {string} pathname
  * @returns {string}
  */
@@ -22,7 +22,7 @@ function getDirectory(pathname) {
 }
 
 /**
- * Compute a path-relative URL from a base directory to a target path
+ * Compute a path-relative URL from a base directory to a target path.
  * @param {string} baseDir - Base directory path (must end with `/`)
  * @param {string} targetPath - Target pathname
  * @returns {string}
@@ -63,7 +63,7 @@ function relativize(baseDir, targetPath) {
 }
 
 /**
- * Remove directory index from the end of a pathname
+ * Remove directory index from the end of a pathname.
  * @param {string} pathname
  * @returns {string}
  */
@@ -77,7 +77,7 @@ function removeDirectoryIndex(pathname) {
 }
 
 /**
- * Create a URL minifier function for the given site context
+ * Create a URL minifier function for the given site context.
  * @param {string} site - The site base URL (used to compute relative URLs)
  * @returns {(url: string) => string} Minifier function that returns the shortest URL
  */

package/src/lib/utils.js

@@ -109,7 +109,7 @@ function lowercase(value) {
 // Replace async helper
 
 /**
- * Asynchronously replace matches in a string
+ * Asynchronously replace matches in a string.
  * @param {string} str - Input string
  * @param {RegExp} regex - Regular expression with global flag
  * @param {Function} asyncFn - Async function to process each match

package/src/presets.js

@@ -38,7 +38,7 @@ export const presets = {
 };
 
 /**
- * Get preset configuration by name
+ * Get preset configuration by name.
  * @param {string} name - Preset name (“conservative” or “comprehensive”)
  * @returns {object|null} Preset options object or null if not found
  */