html-minifier-next 1.1.5 → 1.2.1

package/package.json

@@ -15,22 +15,21 @@
   "description": "Highly configurable, well-tested, JavaScript-based HTML minifier.",
   "devDependencies": {
     "@commitlint/cli": "^19.8.1",
-    "@jest/globals": "^30.0.4",
+    "@jest/globals": "^30.0.5",
     "@rollup/plugin-commonjs": "^28.0.6",
     "@rollup/plugin-json": "^6.1.0",
     "@rollup/plugin-node-resolve": "^16.0.1",
     "@rollup/plugin-terser": "^0.4.4",
     "alpinejs": "^3.14.9",
     "commitlint-config-non-conventional": "^1.0.1",
-    "eslint": "^9.30.1",
+    "eslint": "^9.32.0",
     "husky": "^9.1.7",
     "is-ci": "^4.1.0",
-    "jest": "^30.0.4",
-    "jest-environment-jsdom": "^30.0.4",
-    "lint-staged": "^16.1.2",
-    "rollup": "^4.44.1",
+    "jest": "^30.0.5",
+    "lint-staged": "^16.1.5",
+    "rollup": "^4.45.1",
     "rollup-plugin-polyfill-node": "^0.13.0",
-    "vite": "^7.0.0"
+    "vite": "^7.0.5"
   },
   "exports": {
     ".": {
@@ -72,6 +71,11 @@
   "main": "./dist/htmlminifier.cjs",
   "module": "./src/htmlminifier.js",
   "name": "html-minifier-next",
+  "overrides": {
+    "glob": "^10.0.0",
+    "inflight": "npm:@nodelib/fs.stat@^3.0.0"
+  },
+  "overrides_comment": "@@ Remove when Jest fixes deprecated glob@7.2.3 and inflight dependencies",
   "repository": "https://github.com/j9t/html-minifier-next.git",
   "scripts": {
     "build": "rollup -c",
@@ -80,11 +84,9 @@
     "lint": "eslint .",
     "prepare": "husky",
     "serve": "npm run build && vite",
-    "test": "npm run test:node",
-    "test:node": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' jest --verbose",
-    "test:watch": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' jest verbose --watch",
-    "test:web": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' jest --verbose --environment=jsdom"
+    "test": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' jest",
+    "test:watch": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' jest --watch"
   },
   "type": "module",
-  "version": "1.1.5"
+  "version": "1.2.1"
 }

package/src/htmlminifier.js

@@ -52,28 +52,28 @@ function collapseWhitespace(str, options, trimLeft, trimRight, collapseAll) {
   }
 
   if (collapseAll) {
-    // strip non space whitespace then compress spaces to one
+    // Strip non-space whitespace then compress spaces to one
     str = collapseWhitespaceAll(str);
   }
 
   return lineBreakBefore + str + lineBreakAfter;
 }
 
-// non-empty tags that will maintain whitespace around them
-const inlineTags = new Set(['a', 'abbr', 'acronym', 'b', 'bdi', 'bdo', 'big', 'button', 'cite', 'code', 'del', 'dfn', 'em', 'font', 'i', 'ins', 'kbd', 'label', 'mark', 'math', 'nobr', 'object', 'q', 'rp', 'rt', 'rtc', 'ruby', 's', 'samp', 'select', 'small', 'span', 'strike', 'strong', 'sub', 'sup', 'svg', 'textarea', 'time', 'tt', 'u', 'var']);
-// non-empty tags that will maintain whitespace within them
-const inlineTextTags = new Set(['a', 'abbr', 'acronym', 'b', 'big', 'del', 'em', 'font', 'i', 'ins', 'kbd', 'mark', 'nobr', 'rp', 's', 'samp', 'small', 'span', 'strike', 'strong', 'sub', 'sup', 'time', 'tt', 'u', 'var']);
-// self-closing tags that will maintain whitespace around them
-const selfClosingInlineTags = new Set(['comment', 'img', 'input', 'wbr']);
+// Non-empty elements that will maintain whitespace around them
+const inlineElementsToKeepWhitespaceAround = ['a', 'abbr', 'acronym', 'b', 'bdi', 'bdo', 'big', 'button', 'cite', 'code', 'del', 'dfn', 'em', 'font', 'i', 'img', 'input', 'ins', 'kbd', 'label', 'mark', 'math', 'meter', 'nobr', 'object', 'output', 'progress', 'q', 'rp', 'rt', 'rtc', 'ruby', 's', 'samp', 'select', 'small', 'span', 'strike', 'strong', 'sub', 'sup', 'svg', 'textarea', 'time', 'tt', 'u', 'var', 'wbr'];
+// Non-empty elements that will maintain whitespace within them
+const inlineElementsToKeepWhitespaceWithin = new Set(['a', 'abbr', 'acronym', 'b', 'big', 'del', 'em', 'font', 'i', 'ins', 'kbd', 'mark', 'nobr', 'rp', 's', 'samp', 'small', 'span', 'strike', 'strong', 'sub', 'sup', 'time', 'tt', 'u', 'var']);
+// Elements that will always maintain whitespace around them
+const inlineElementsToKeepWhitespace = new Set(['comment', 'img', 'input', 'wbr']);
 
-function collapseWhitespaceSmart(str, prevTag, nextTag, options) {
-  let trimLeft = prevTag && !selfClosingInlineTags.has(prevTag);
+function collapseWhitespaceSmart(str, prevTag, nextTag, options, inlineElements, inlineTextSet) {
+  let trimLeft = prevTag && !inlineElementsToKeepWhitespace.has(prevTag);
   if (trimLeft && !options.collapseInlineTagWhitespace) {
-    trimLeft = prevTag.charAt(0) === '/' ? !inlineTags.has(prevTag.slice(1)) : !inlineTextTags.has(prevTag);
+    trimLeft = prevTag.charAt(0) === '/' ? !inlineElements.has(prevTag.slice(1)) : !inlineTextSet.has(prevTag);
   }
-  let trimRight = nextTag && !selfClosingInlineTags.has(nextTag);
+  let trimRight = nextTag && !inlineElementsToKeepWhitespace.has(nextTag);
   if (trimRight && !options.collapseInlineTagWhitespace) {
-    trimRight = nextTag.charAt(0) === '/' ? !inlineTextTags.has(nextTag.slice(1)) : !inlineTags.has(nextTag);
+    trimRight = nextTag.charAt(0) === '/' ? !inlineTextSet.has(nextTag.slice(1)) : !inlineElements.has(nextTag);
   }
   return collapseWhitespace(str, options, trimLeft, trimRight, prevTag && nextTag);
 }
@@ -395,7 +395,7 @@ async function processScript(text, options, currentAttrs) {
 // Tag omission rules from https://html.spec.whatwg.org/multipage/syntax.html#optional-tags
 // with the following deviations:
 // - retain <body> if followed by <noscript>
-// - </rb>, </rt>, </rtc>, </rp> & </tfoot> follow https://www.w3.org/TR/html5/syntax.html#optional-tags
+// - </rb>, </rt>, </rtc>, </rp>, and </tfoot> follow https://www.w3.org/TR/html5/syntax.html#optional-tags
 // - retain all tags which are adjacent to non-standard HTML tags
 const optionalStartTags = new Set(['html', 'head', 'body', 'colgroup', 'tbody']);
 const optionalEndTags = new Set(['html', 'head', 'body', 'li', 'dt', 'dd', 'p', 'rb', 'rt', 'rtc', 'rp', 'optgroup', 'option', 'colgroup', 'caption', 'thead', 'tbody', 'tfoot', 'tr', 'td', 'th']);
@@ -605,7 +605,7 @@ function buildAttr(normalized, hasUnarySlash, options, isLast, uidAttr) {
       emittedAttrValue += ' ';
     }
   } else if (isLast && !hasUnarySlash && !/\/$/.test(attrValue)) {
-    // make sure trailing slash is not interpreted as HTML self-closing tag
+    // Make sure trailing slash is not interpreted as HTML self-closing tag
     emittedAttrValue = attrValue;
   } else {
     emittedAttrValue = attrValue + ' ';
@@ -866,10 +866,16 @@ async function minifyHTML(value, options, partialMarkup) {
   let uidIgnore;
   let uidAttr;
   let uidPattern;
-
-  // temporarily replace ignored chunks with comments,
-  // so that we don't have to worry what's there.
-  // for all we care there might be
+  // Create inline tags/text sets with custom elements
+  const customElementsInput = options.inlineCustomElements ?? [];
+  const customElementsArr = Array.isArray(customElementsInput) ? customElementsInput : Array.from(customElementsInput);
+  const normalizedCustomElements = customElementsArr.map(name => options.name(name));
+  const inlineTextSet = new Set([...inlineElementsToKeepWhitespaceWithin, ...normalizedCustomElements]);
+  const inlineElements = new Set([...inlineElementsToKeepWhitespaceAround, ...normalizedCustomElements]);
+
+  // Temporarily replace ignored chunks with comments,
+  // so that we don’t have to worry what’s there.
+  // For all we care there might be
   // completely-horribly-broken-alien-non-html-emoj-cthulhu-filled content
   value = value.replace(/<!-- htmlmin:ignore -->([\s\S]*?)<!-- htmlmin:ignore -->/g, function (match, group1) {
     if (!uidIgnore) {
@@ -990,20 +996,20 @@ async function minifyHTML(value, options, partialMarkup) {
     buffer.length = Math.max(0, index);
   }
 
-  // look for trailing whitespaces, bypass any inline tags
+  // Look for trailing whitespaces, bypass any inline tags
   function trimTrailingWhitespace(index, nextTag) {
     for (let endTag = null; index >= 0 && _canTrimWhitespace(endTag); index--) {
       const str = buffer[index];
       const match = str.match(/^<\/([\w:-]+)>$/);
       if (match) {
         endTag = match[1];
-      } else if (/>$/.test(str) || (buffer[index] = collapseWhitespaceSmart(str, null, nextTag, options))) {
+      } else if (/>$/.test(str) || (buffer[index] = collapseWhitespaceSmart(str, null, nextTag, options, inlineElements, inlineTextSet))) {
         break;
       }
     }
   }
 
-  // look for trailing whitespaces from previously processed text
+  // Look for trailing whitespaces from previously processed text
   // which may not be trimmed due to a following comment or an empty
   // element which has now been removed
   function squashTrailingWhitespace(nextTag) {
@@ -1034,7 +1040,7 @@ async function minifyHTML(value, options, partialMarkup) {
       tag = options.name(tag);
       currentTag = tag;
       charsPrevTag = tag;
-      if (!inlineTextTags.has(tag)) {
+      if (!inlineTextSet.has(tag)) {
         currentChars = '';
       }
       hasChars = false;
@@ -1052,7 +1058,7 @@ async function minifyHTML(value, options, partialMarkup) {
           removeStartTag();
         }
         optionalStartTag = '';
-        // end-tag-followed-by-start-tag omission rules
+        // End-tag-followed-by-start-tag omission rules
         if (htmlTag && canRemovePrecedingTag(optionalEndTag, tag)) {
           removeEndTag();
           // <colgroup> cannot be omitted if preceding </colgroup> is omitted
@@ -1062,7 +1068,7 @@ async function minifyHTML(value, options, partialMarkup) {
         optionalEndTag = '';
       }
 
-      // set whitespace flags for nested tags (eg. <code> within a <pre>)
+      // Set whitespace flags for nested tags (eg. <code> within a <pre>)
       if (options.collapseWhitespace) {
         if (!stackNoTrimWhitespace.length) {
           squashTrailingWhitespace(tag);
@@ -1098,7 +1104,7 @@ async function minifyHTML(value, options, partialMarkup) {
         buffer.push(' ');
         buffer.push.apply(buffer, parts);
       } else if (optional && optionalStartTags.has(tag)) {
-        // start tag must never be omitted if it has any attributes
+        // Start tag must never be omitted if it has any attributes
         optionalStartTag = tag;
       }
 
@@ -1115,7 +1121,7 @@ async function minifyHTML(value, options, partialMarkup) {
       }
       tag = options.name(tag);
 
-      // check if current tag is in a whitespace stack
+      // Check if current tag is in a whitespace stack
       if (options.collapseWhitespace) {
         if (stackNoTrimWhitespace.length) {
           if (tag === stackNoTrimWhitespace[stackNoTrimWhitespace.length - 1]) {
@@ -1153,7 +1159,7 @@ async function minifyHTML(value, options, partialMarkup) {
       }
 
       if (options.removeEmptyElements && isElementEmpty && canRemoveElement(tag, attrs)) {
-        // remove last "element" from buffer
+        // Remove last “element” from buffer
         removeStartTag();
         optionalStartTag = '';
         optionalEndTag = '';
@@ -1164,7 +1170,7 @@ async function minifyHTML(value, options, partialMarkup) {
           buffer.push('</' + tag + '>');
         }
         charsPrevTag = '/' + tag;
-        if (!inlineTags.has(tag)) {
+        if (!inlineElements.has(tag)) {
           currentChars = '';
         } else if (isElementEmpty) {
           currentChars += '|';
@@ -1203,12 +1209,12 @@ async function minifyHTML(value, options, partialMarkup) {
                 }
                 trimTrailingWhitespace(tagIndex - 1, 'br');
               }
-            } else if (inlineTextTags.has(prevTag.charAt(0) === '/' ? prevTag.slice(1) : prevTag)) {
+            } else if (inlineTextSet.has(prevTag.charAt(0) === '/' ? prevTag.slice(1) : prevTag)) {
               text = collapseWhitespace(text, options, /(?:^|\s)$/.test(currentChars));
             }
           }
           if (prevTag || nextTag) {
-            text = collapseWhitespaceSmart(text, prevTag, nextTag, options);
+            text = collapseWhitespaceSmart(text, prevTag, nextTag, options, inlineElements, inlineTextSet);
           } else {
             text = collapseWhitespace(text, options, true, true);
           }
@@ -1278,7 +1284,7 @@ async function minifyHTML(value, options, partialMarkup) {
         text = prefix + text + suffix;
       }
       if (options.removeOptionalTags && text) {
-        // preceding comments suppress tag omissions
+        // Preceding comments suppress tag omissions
         optionalStartTag = '';
         optionalEndTag = '';
       }
@@ -1382,4 +1388,4 @@ export const minify = async function (value, options) {
   return result;
 };
 
-export default { minify };
+export default { minify };

package/src/htmlparser.js

@@ -131,7 +131,7 @@ export class HTMLParser {
     let last, prevTag, nextTag;
     while (html) {
       last = html;
-      // Make sure we're not in a script or style element
+      // Make sure we’re not in a `script` or `style` element
       if (!lastTag || !special.has(lastTag)) {
         let textEnd = html.indexOf('<');
         if (textEnd === 0) {
@@ -207,7 +207,7 @@ export class HTMLParser {
           html = '';
         }
 
-        // next tag
+        // Next tag
         let nextTagMatch = parseStartTag(html);
         if (nextTagMatch) {
           nextTag = nextTagMatch.tagName;
@@ -320,9 +320,9 @@ export class HTMLParser {
 
       const attrs = match.attrs.map(function (args) {
         let name, value, customOpen, customClose, customAssign, quote;
-        const ncp = 7; // number of captured parts, scalar
+        const ncp = 7; // Number of captured parts, scalar
 
-        // hackish work around FF bug https://bugzilla.mozilla.org/show_bug.cgi?id=369778
+        // Hackish workaround for FF bug https://bugzilla.mozilla.org/show_bug.cgi?id=369778
         if (IS_REGEX_CAPTURING_BROKEN && args[0].indexOf('""') === -1) {
           if (args[3] === '') { delete args[3]; }
           if (args[4] === '') { delete args[4]; }
@@ -512,13 +512,13 @@ export const HTMLtoDOM = (html, doc) => {
     }
   }
 
-  // If we're working with a document, inject contents into
-  // the body element
+  // If we’re working with a document, inject contents into
+  // the `body` element
   let curParentNode = one.body;
 
   const parser = new HTMLParser(html, {
     start: function (tagName, attrs, unary) {
-      // If it's a pre-built element, then we can ignore
+      // If it’s a pre-built element, then we can ignore
       // its construction
       if (one[tagName]) {
         curParentNode = one[tagName];
@@ -552,7 +552,7 @@ export const HTMLtoDOM = (html, doc) => {
       curParentNode.appendChild(doc.createTextNode(text));
     },
     comment: function (/* text */) {
-      // create comment node
+      // Create comment node
     },
     ignore: function (/* text */) {
       // What to do here?
@@ -562,4 +562,4 @@ export const HTMLtoDOM = (html, doc) => {
   parser.parse();
 
   return doc;
-};
+};