postcss-discard-comments 8.0.1 → 8.0.3

package/LICENSE-MIT

@@ -1,4 +1,4 @@
-Copyright (c) Ben Briggs <beneb.info@gmail.com> (http://beneb.info)
+Copyright (c) Ben Briggs <beneb.info@gmail.com> (https://beneb.info)
 
 Permission is hereby granted, free of charge, to any person
 obtaining a copy of this software and associated documentation

package/README.md

@@ -108,7 +108,7 @@ See [CONTRIBUTORS.md](https://github.com/cssnano/cssnano/blob/master/CONTRIBUTOR
 
 ## License
 
-MIT © [Ben Briggs](http://beneb.info)
+MIT © [Ben Briggs](https://beneb.info)
 
 
 [postcss]: https://github.com/postcss/postcss

package/package.json

@@ -1,9 +1,19 @@
 {
   "name": "postcss-discard-comments",
-  "version": "8.0.1",
+  "version": "8.0.3",
   "description": "Discard comments in your CSS files with PostCSS.",
-  "main": "src/index.js",
-  "types": "types/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./types/index.d.ts",
+      "default": "./src/index.js"
+    },
+    "./src": "./src/index.js",
+    "./src/index": "./src/index.js",
+    "./src/index.js": "./src/index.js",
+    "./src/lib/*": "./src/lib/*.js",
+    "./src/lib/*.js": "./src/lib/*.js",
+    "./package.json": "./package.json"
+  },
   "files": [
     "src",
     "LICENSE-MIT",
@@ -20,11 +30,14 @@
   "author": {
     "name": "Ben Briggs",
     "email": "beneb.info@gmail.com",
-    "url": "http://beneb.info"
+    "url": "https://beneb.info"
+  },
+  "repository": {
+    "type": "git",
+    "url": "cssnano/cssnano"
   },
-  "repository": "cssnano/cssnano",
   "dependencies": {
-    "postcss-selector-parser": "^7.1.2"
+    "postcss-selector-parser": "^7.1.5"
   },
   "bugs": {
     "url": "https://github.com/cssnano/cssnano/issues"
@@ -33,11 +46,14 @@
     "node": "^22.11.0 || ^24.11.0 || >=26.0"
   },
   "devDependencies": {
-    "postcss": "^8.5.15",
+    "postcss": "^8.5.26",
     "postcss-scss": "^4.0.9",
     "postcss-simple-vars": "^7.0.1"
   },
   "peerDependencies": {
-    "postcss": "^8.5.15"
+    "postcss": "^8.5.26"
+  },
+  "scripts": {
+    "test": "node --test"
   }
 }

package/src/index.js

@@ -9,7 +9,6 @@ const selectorParser = require('postcss-selector-parser');
  *  @property {(s: string) => boolean=} remove
  */
 /**
- * @type {import('postcss').PluginCreator<Options>}
  * @param {Options} opts
  * @return {import('postcss').Plugin}
  */
@@ -65,7 +64,7 @@ function pluginCreator(opts = {}) {
       return replacerCache.get(key);
     }
 
-    if (source.indexOf('/*') === -1) {
+    if (!source.includes('/*')) {
       const normalized = space(source).join(' ');
 
       replacerCache.set(key, normalized);
@@ -112,7 +111,7 @@ function pluginCreator(opts = {}) {
     if (replacerCache.has(key)) {
       return replacerCache.get(key);
     }
-    if (source.indexOf('/*') === -1) {
+    if (!source.includes('/*')) {
       const normalized = space(source).join(' ');
 
       replacerCache.set(key, normalized);
@@ -147,93 +146,107 @@ function pluginCreator(opts = {}) {
     return result;
   }
 
-  return {
-    postcssPlugin: 'postcss-discard-comments',
+  /**
+   * @param {import('postcss').Declaration} node
+   * @param {(s: string) => string[]} space
+   */
+  function processDeclaration(node, space) {
+    if (node.raws.value && node.raws.value.raw) {
+      if (node.raws.value.value === node.value) {
+        node.value = replaceComments(node.raws.value.raw, space);
+      } else {
+        node.value = replaceComments(node.value, space);
+      }
 
-    OnceExit(css, { list }) {
-      css.walk((node) => {
-        if (node.type === 'comment' && remover.canRemove(node.text)) {
-          node.remove();
+      /** @type {null | {value: string, raw: string}} */ (node.raws.value) =
+        null;
+    }
 
-          return;
-        }
+    if (node.raws.important) {
+      node.raws.important = replaceComments(node.raws.important, space);
 
-        if (typeof node.raws.between === 'string') {
-          node.raws.between = replaceComments(node.raws.between, list.space);
-        }
+      const b = matchesComments(node.raws.important);
 
-        if (node.type === 'decl') {
-          if (node.raws.value && node.raws.value.raw) {
-            if (node.raws.value.value === node.value) {
-              node.value = replaceComments(node.raws.value.raw, list.space);
-            } else {
-              node.value = replaceComments(node.value, list.space);
-            }
-
-            /** @type {null | {value: string, raw: string}} */ (
-              node.raws.value
-            ) = null;
-          }
+      node.raws.important = b.length ? node.raws.important : '!important';
+    } else {
+      node.value = replaceComments(node.value, space);
+    }
+  }
 
-          if (node.raws.important) {
-            node.raws.important = replaceComments(
-              node.raws.important,
-              list.space
-            );
+  /**
+   * @param {import('postcss').Rule} node
+   * @param {(s: string) => string[]} space
+   */
+  function processRule(node, space) {
+    if (node.raws.selector && node.raws.selector.raw) {
+      node.raws.selector.raw = replaceCommentsInSelector(
+        node.raws.selector.raw,
+        space
+      );
+    } else if (node.selector && node.selector.includes('/*')) {
+      node.selector = replaceCommentsInSelector(node.selector, space);
+    }
+  }
 
-            const b = matchesComments(node.raws.important);
+  /**
+   * @param {import('postcss').AtRule} node
+   * @param {(s: string) => string[]} space
+   */
+  function processAtRule(node, space) {
+    if (node.raws.afterName) {
+      const commentsReplaced = replaceComments(node.raws.afterName, space);
+
+      if (!commentsReplaced.length) {
+        node.raws.afterName = commentsReplaced + ' ';
+      } else {
+        node.raws.afterName = ' ' + commentsReplaced + ' ';
+      }
+    }
 
-            node.raws.important = b.length ? node.raws.important : '!important';
-          } else {
-            node.value = replaceComments(node.value, list.space);
-          }
+    if (node.raws.params && node.raws.params.raw) {
+      node.raws.params.raw = replaceComments(node.raws.params.raw, space);
+    } else if (node.params && node.params.includes('/*')) {
+      node.params = replaceComments(node.params, space);
+    }
+  }
 
-          return;
-        }
+  /**
+   * @param {import('postcss').ChildNode} node
+   * @param {(s: string) => string[]} space
+   */
+  function processNode(node, space) {
+    if (node.type === 'comment' && remover.canRemove(node.text)) {
+      node.remove();
 
-        if (node.type === 'rule') {
-          if (node.raws.selector && node.raws.selector.raw) {
-            node.raws.selector.raw = replaceCommentsInSelector(
-              node.raws.selector.raw,
-              list.space
-            );
-          } else if (node.selector && node.selector.includes('/*')) {
-            node.selector = replaceCommentsInSelector(
-              node.selector,
-              list.space
-            );
-          }
+      return;
+    }
 
-          return;
-        }
+    if (typeof node.raws.between === 'string') {
+      node.raws.between = replaceComments(node.raws.between, space);
+    }
 
-        if (node.type === 'atrule') {
-          if (node.raws.afterName) {
-            const commentsReplaced = replaceComments(
-              node.raws.afterName,
-              list.space
-            );
-
-            if (!commentsReplaced.length) {
-              node.raws.afterName = commentsReplaced + ' ';
-            } else {
-              node.raws.afterName = ' ' + commentsReplaced + ' ';
-            }
-          }
+    if (node.type === 'decl') {
+      processDeclaration(node, space);
+    } else if (node.type === 'rule') {
+      processRule(node, space);
+    } else if (node.type === 'atrule') {
+      processAtRule(node, space);
+    }
+  }
 
-          if (node.raws.params && node.raws.params.raw) {
-            node.raws.params.raw = replaceComments(
-              node.raws.params.raw,
-              list.space
-            );
-          } else if (node.params && node.params.includes('/*')) {
-            node.params = replaceComments(node.params, list.space);
-          }
-        }
-      });
+  return {
+    postcssPlugin: 'postcss-discard-comments',
+    /**
+     * @param {import('postcss').Root} css
+     * @param {import('postcss').Helpers} helpers
+     */
+    OnceExit(css, { list }) {
+      css.walk((node) => processNode(node, list.space));
     },
   };
 }
 
 pluginCreator.postcss = true;
-module.exports = pluginCreator;
+module.exports = /** @type {import('postcss').PluginCreator<Options>}*/ (
+  pluginCreator
+);

package/src/lib/commentParser.js

@@ -8,6 +8,85 @@ const STATES = {
   IN_COMMENT: 3,
 };
 
+/**
+ * @typedef {object} ParserContext
+ * @property {string} input
+ * @property {[number, number, number][]} tokens
+ * @property {number} length
+ * @property {number} pos
+ * @property {number} state
+ * @property {number} tokenStart
+ * @property {number} commentStart
+ */
+
+/**
+ * @param {ParserContext} context
+ * @param {string} nextChar
+ * @return {boolean}
+ */
+function handleNormalState(context, nextChar) {
+  const { input, pos } = context;
+  const char = input[pos];
+
+  if (char === '/' && nextChar === '*') {
+    if (pos > context.tokenStart) {
+      context.tokens.push([0, context.tokenStart, pos]);
+    }
+    context.commentStart = pos;
+    context.state = STATES.IN_COMMENT;
+    context.pos += 2;
+    return true;
+  }
+
+  if (char === '"') {
+    context.state = STATES.IN_DOUBLE_QUOTE;
+  } else if (char === "'") {
+    context.state = STATES.IN_SINGLE_QUOTE;
+  }
+
+  return false;
+}
+
+/**
+ * @param {ParserContext} context
+ * @param {string} nextChar
+ * @param {string} quote
+ * @return {boolean}
+ */
+function handleQuoteState(context, nextChar, quote) {
+  const char = context.input[context.pos];
+
+  if (char === '\\' && nextChar) {
+    context.pos += 2;
+    return true;
+  }
+
+  if (char === quote) {
+    context.state = STATES.NORMAL;
+  }
+
+  return false;
+}
+
+/**
+ * @param {ParserContext} context
+ * @param {string} nextChar
+ * @return {boolean}
+ */
+function handleCommentState(context, nextChar) {
+  const { input, pos } = context;
+
+  if (input[pos] === '*' && nextChar === '/') {
+    context.tokens.push([1, context.commentStart + 2, pos]);
+    context.tokenStart = pos + 2;
+    context.state = STATES.NORMAL;
+    context.pos += 2;
+    return true;
+  }
+
+  return false;
+}
+
 /**
  * CSS Comment Parser with context awareness
  * Properly handles comments inside strings, URLs, and escaped characters
@@ -16,80 +95,55 @@ const STATES = {
  * @return {[number, number, number][]}
  */
 module.exports = function commentParser(input) {
-  /** @type {[number, number, number][]} */
-  const tokens = [];
-  const length = input.length;
-  let pos = 0;
+  /** @type {ParserContext} */
+  const context = {
+    input,
+    tokens: [],
+    length: input.length,
+    pos: 0,
+    state: STATES.NORMAL,
+    tokenStart: 0,
+    commentStart: 0,
+  };
 
-  let state = STATES.NORMAL;
-  let tokenStart = 0;
-  let commentStart = 0;
+  while (context.pos < context.length) {
+    const nextChar =
+      context.pos + 1 < context.length ? context.input[context.pos + 1] : '';
+    let handled;
 
-  while (pos < length) {
-    const char = input[pos];
-    const nextChar = pos + 1 < length ? input[pos + 1] : '';
-
-    switch (state) {
+    switch (context.state) {
       case STATES.NORMAL:
-        if (char === '/' && nextChar === '*') {
-          // Found comment start - add non-comment token if needed
-          if (pos > tokenStart) {
-            tokens.push([0, tokenStart, pos]);
-          }
-          commentStart = pos;
-          state = STATES.IN_COMMENT;
-          pos += 2; // Skip /*
-          continue;
-        } else if (char === '"') {
-          state = STATES.IN_DOUBLE_QUOTE;
-        } else if (char === "'") {
-          state = STATES.IN_SINGLE_QUOTE;
-        }
+        handled = handleNormalState(context, nextChar);
         break;
 
       case STATES.IN_SINGLE_QUOTE:
-        if (char === '\\' && nextChar) {
-          // Skip escaped character
-          pos += 2;
-          continue;
-        } else if (char === "'") {
-          state = STATES.NORMAL;
-        }
+        handled = handleQuoteState(context, nextChar, "'");
         break;
 
       case STATES.IN_DOUBLE_QUOTE:
-        if (char === '\\' && nextChar) {
-          // Skip escaped character
-          pos += 2;
-          continue;
-        } else if (char === '"') {
-          state = STATES.NORMAL;
-        }
+        handled = handleQuoteState(context, nextChar, '"');
         break;
 
       case STATES.IN_COMMENT:
-        if (char === '*' && nextChar === '/') {
-          // Found comment end
-          tokens.push([1, commentStart + 2, pos]);
-          tokenStart = pos + 2;
-          state = STATES.NORMAL;
-          pos += 2; // Skip */
-          continue;
-        }
+        handled = handleCommentState(context, nextChar);
         break;
     }
 
-    pos++;
+    if (handled) {
+      continue;
+    }
+
+    context.pos++;
   }
 
   // Handle remaining content
-  if (state === STATES.IN_COMMENT) {
+  if (context.state === STATES.IN_COMMENT) {
     // Unclosed comment - treat as comment to end
-    tokens.push([1, commentStart + 2, length]);
-  } else if (tokenStart < length) {
+    context.tokens.push([1, context.commentStart + 2, context.length]);
+  } else if (context.tokenStart < context.length) {
     // Add final non-comment token
-    tokens.push([0, tokenStart, length]);
+    context.tokens.push([0, context.tokenStart, context.length]);
   }
 
-  return tokens;
+  return context.tokens;
 };

package/src/lib/commentRemover.js

@@ -1,32 +1,35 @@
 'use strict';
 
-/** @param {import('../index.js').Options} options */
-function CommentRemover(options) {
-  this.options = options;
-}
-/**
- * @param {string} comment
- * @return {boolean | undefined}
- */
-CommentRemover.prototype.canRemove = function (comment) {
-  const remove = this.options.remove;
+class CommentRemover {
+  /** @param {import('../index.js').Options} options */
+  constructor(options) {
+    this.options = options;
+  }
+  /**
+   * @param {string} comment
+   * @return {boolean | undefined}
+   */
+  canRemove(comment) {
+    const remove = this.options.remove;
 
-  if (remove) {
-    return remove(comment);
-  } else {
-    const isImportant = comment.indexOf('!') === 0;
+    if (remove) {
+      return remove(comment);
+    } else {
+      const isImportant = comment.indexOf('!') === 0;
 
-    if (!isImportant) {
-      return true;
-    }
+      if (!isImportant) {
+        return true;
+      }
 
-    if (this.options.removeAll || this._hasFirst) {
-      return true;
-    } else if (this.options.removeAllButFirst && !this._hasFirst) {
-      this._hasFirst = true;
-      return false;
+      if (this.options.removeAll || this._hasFirst) {
+        return true;
+      } else if (this.options.removeAllButFirst && !this._hasFirst) {
+        this._hasFirst = true;
+        return false;
+      }
     }
+    return false;
   }
-};
+}
 
 module.exports = CommentRemover;

package/types/index.d.ts

@@ -1,20 +1,6 @@
-export = pluginCreator;
-/** @typedef {object} Options
- *  @property {boolean=} removeAll
- *  @property {boolean=} removeAllButFirst
- *  @property {(s: string) => boolean=} remove
- */
-/**
- * @type {import('postcss').PluginCreator<Options>}
- * @param {Options} opts
- * @return {import('postcss').Plugin}
- */
-declare function pluginCreator(opts?: Options): import("postcss").Plugin;
-declare namespace pluginCreator {
-    export { postcss, Options };
-}
-declare var postcss: true;
-type Options = {
+declare const _exports: import("postcss").PluginCreator<Options>;
+export = _exports;
+export type Options = {
     removeAll?: boolean | undefined;
     removeAllButFirst?: boolean | undefined;
     remove?: ((s: string) => boolean) | undefined;

package/types/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.js"],"names":[],"mappings":";AAKA;;;;GAIG;AACH;;;;GAIG;AACH,sCAHW,OAAO,GACN,OAAO,SAAS,EAAE,MAAM,CA8NnC;;;;;;gBArOc,OAAO,YAAC;wBACR,OAAO,YAAC;aACR,CAAA,CAAC,CAAC,EAAE,MAAM,KAAK,OAAO,aAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.js"],"names":[],"mappings":";;AAKI,YAAkB,OAAO,GACzB;IAAqB,SAAS,AAA9B,CACA,EADW,OAAO,YAAC,CACnB;IAAqB,iBAAiB,AAAtC,CACA,EADW,OAAO,YAAC,CACnB;IAAoC,MAAM,AAA1C,CACH,EADc,CAAA,CAAC,CAAC,EAAE,MAAM,KAAK,OAAO,aAAC,CACrC;CAAA,CAAA"}

package/types/lib/commentParser.d.ts

@@ -1,3 +1,19 @@
-declare function _exports(input: string): [number, number, number][];
-export = _exports;
+export = commentParser;
+/**
+ * CSS Comment Parser with context awareness
+ * Properly handles comments inside strings, URLs, and escaped characters
+ *
+ * @param {string} input
+ * @return {[number, number, number][]}
+ */
+declare function commentParser(input: string): [number, number, number][];
+export type ParserContext = {
+    input: string;
+    tokens: [number, number, number][];
+    length: number;
+    pos: number;
+    state: number;
+    tokenStart: number;
+    commentStart: number;
+};
 //# sourceMappingURL=commentParser.d.ts.map

package/types/lib/commentParser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"commentParser.d.ts","sourceRoot":"","sources":["../../src/lib/commentParser.js"],"names":[],"mappings":"AAiBiB,iCAHN,MAAM,GACL,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,CA+ErC"}
+{"version":3,"file":"commentParser.d.ts","sourceRoot":"","sources":["../../src/lib/commentParser.js"],"names":[],"mappings":";AAyFA;;;;;;GAMG;+BACqC,KAAK,EAHlC,MAGkC,GAFjC,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE;AAnFnC,YAAkB,aAAa,GAC/B;IAAmB,KAAK,EAAb,MAAM,CACjB;IAAuC,MAAM,EAAlC,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,CACrC;IAAmB,MAAM,EAAd,MAAM,CACjB;IAAmB,GAAG,EAAX,MAAM,CACjB;IAAmB,KAAK,EAAb,MAAM,CACjB;IAAmB,UAAU,EAAlB,MAAM,CACjB;IAAmB,YAAY,EAApB,MAAM,CACnB;CAAA,CAAA"}

package/types/lib/commentRemover.d.ts

@@ -1,15 +1,13 @@
 export = CommentRemover;
-/** @param {import('../index.js').Options} options */
-declare function CommentRemover(options: import("../index.js").Options): void;
 declare class CommentRemover {
-    /** @param {import('../index.js').Options} options */
-    constructor(options: import("../index.js").Options);
     options: import("../index.js").Options;
+    _hasFirst: boolean | undefined;
+    /** @param {import('../index.js').Options} options */
+    constructor(options: import('../index.js').Options);
     /**
      * @param {string} comment
      * @return {boolean | undefined}
      */
     canRemove(comment: string): boolean | undefined;
-    _hasFirst: boolean | undefined;
 }
 //# sourceMappingURL=commentRemover.d.ts.map

package/types/lib/commentRemover.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"commentRemover.d.ts","sourceRoot":"","sources":["../../src/lib/commentRemover.js"],"names":[],"mappings":";AAEA,qDAAqD;AACrD,yCADY,OAAO,aAAa,EAAE,OAAO,QAGxC;;IAHD,qDAAqD;IACrD,qBADY,OAAO,aAAa,EAAE,OAAO,EAGxC;IADC,uCAAsB;IAExB;;;OAGG;IACH,mBAHW,MAAM,GACL,OAAO,GAAG,SAAS,CAqB9B;IAJK,+BAAqB"}
+{"version":3,"file":"commentRemover.d.ts","sourceRoot":"","sources":["../../src/lib/commentRemover.js"],"names":[],"mappings":"SAkCiB,cAAc;AAhC/B,cAAM,cAAc;IAGX,OAAO;IAqBH,SAAS;IAvBpB,qDAAqD;IACrD,YAAY,OAAO,EADP,OAAO,aAAa,EAAE,OACf,EAElB;IACD;;;OAGG;IACH,SAAS,CAAC,OAAO,EAHN,MAGM,GAFL,OAAO,GAAG,SAAS,CAsB9B;CACF"}