postcss-normalize-string 9.0.2 → 9.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-normalize-string",
-  "version": "9.0.2",
+  "version": "9.0.4",
   "description": "Normalize wrapping quotes for CSS string literals.",
   "exports": {
     ".": {
@@ -33,7 +33,7 @@
     "url": "git+https://github.com/cssnano/cssnano.git"
   },
   "dependencies": {
-    "postcss-value-parser": "^4.2.0"
+    "@csstools/css-tokenizer": "^4.0.2"
   },
   "bugs": {
     "url": "https://github.com/cssnano/cssnano/issues"

package/src/index.js

@@ -1,233 +1,233 @@
-import valueParser from 'postcss-value-parser';
-
-/*
- * Constants (parser usage)
- */
+import { tokenize, TokenType } from '@csstools/css-tokenizer';
 
 const SINGLE_QUOTE = "'".charCodeAt(0);
 const DOUBLE_QUOTE = '"'.charCodeAt(0);
 const BACKSLASH = '\\'.charCodeAt(0);
 const NEWLINE = '\n'.charCodeAt(0);
-const SPACE = ' '.charCodeAt(0);
 const FEED = '\f'.charCodeAt(0);
-const TAB = '\t'.charCodeAt(0);
 const CR = '\r'.charCodeAt(0);
 
-const WORD_END = /[ \n\t\r\f'"\\]/g;
-
-/*
- * Constants (node type strings)
+/**
+ * Verify whether a string token has matching delimiters and balanced trailing escapes.
+ *
+ * @param {string} value
+ * @return {boolean}
  */
+function isClosedString(value) {
+  if (value.length < 2 || (value[0] !== "'" && value[0] !== '"')) {
+    return false;
+  }
 
-const C_STRING = 'string';
-const C_ESCAPED_SINGLE_QUOTE = 'escapedSingleQuote';
-const C_ESCAPED_DOUBLE_QUOTE = 'escapedDoubleQuote';
-const C_SINGLE_QUOTE = 'singleQuote';
-const C_DOUBLE_QUOTE = 'doubleQuote';
-const C_NEWLINE = 'newline';
-const C_SINGLE = 'single';
-
-/*
- * Literals
- */
+  let backslashes = 0;
+  for (
+    let index = value.length - 2;
+    index >= 0 && value[index] === '\\';
+    index--
+  ) {
+    backslashes++;
+  }
 
-const L_SINGLE_QUOTE = `'`;
-const L_DOUBLE_QUOTE = `"`;
-const L_NEWLINE = `\\\n`;
+  return value.at(-1) === value[0] && backslashes % 2 === 0;
+}
 
-/*
- * Parser nodes
+/**
+ * Return the length of an escaped line continuation, or 0 if not a newline escape.
+ *
+ * @param {string} inner
+ * @param {number} pos
+ * @param {number} len
+ * @return {number}
  */
+function getEscapedNewlineLength(inner, pos, len) {
+  const next = inner.charCodeAt(pos + 1);
+  if (next === NEWLINE || next === FEED) {
+    return 2;
+  }
+  if (next === CR) {
+    return pos + 2 < len && inner.charCodeAt(pos + 2) === NEWLINE ? 3 : 2;
+  }
+  return 0;
+}
 
-const T_ESCAPED_SINGLE_QUOTE = { type: C_ESCAPED_SINGLE_QUOTE, value: `\\'` };
-const T_ESCAPED_DOUBLE_QUOTE = { type: C_ESCAPED_DOUBLE_QUOTE, value: `\\"` };
-const T_SINGLE_QUOTE = { type: C_SINGLE_QUOTE, value: L_SINGLE_QUOTE };
-const T_DOUBLE_QUOTE = { type: C_DOUBLE_QUOTE, value: L_DOUBLE_QUOTE };
-const T_NEWLINE = { type: C_NEWLINE, value: L_NEWLINE };
-
-/** @typedef {typeof T_ESCAPED_SINGLE_QUOTE | typeof T_ESCAPED_DOUBLE_QUOTE | typeof T_SINGLE_QUOTE | typeof T_NEWLINE} StringAstNode */
 /**
- * @typedef {{nodes: StringAstNode[],
- *            types: {escapedSingleQuote: number, escapedDoubleQuote: number, singleQuote: number, doubleQuote: number},
- *            quotes: boolean}} StringAst
+ * Scan string contents to count quotes and detect escaped newlines or redundant escapes.
+ *
+ * @param {string} inner
+ * @param {string} currentQuote
+ * @return {{ singleCount: number, doubleCount: number, hasEscapedNewline: boolean, hasRedundantEscape: boolean }}
  */
+function scanQuotes(inner, currentQuote) {
+  const len = inner.length;
+  let singleCount = 0;
+  let doubleCount = 0;
+  let hasEscapedNewline = false;
+  let hasRedundantEscape = false;
+  let pos = 0;
+
+  while (pos < len) {
+    const code = inner.charCodeAt(pos);
+    if (code === BACKSLASH) {
+      if (pos + 1 < len) {
+        const newlineLen = getEscapedNewlineLength(inner, pos, len);
+        if (newlineLen > 0) {
+          hasEscapedNewline = true;
+          pos += newlineLen;
+          continue;
+        }
+        const next = inner.charCodeAt(pos + 1);
+        if (next === SINGLE_QUOTE) {
+          singleCount++;
+          if (currentQuote === '"') {
+            hasRedundantEscape = true;
+          }
+          pos += 2;
+          continue;
+        }
+        if (next === DOUBLE_QUOTE) {
+          doubleCount++;
+          if (currentQuote === "'") {
+            hasRedundantEscape = true;
+          }
+          pos += 2;
+          continue;
+        }
+        pos += 2;
+        continue;
+      }
+      pos++;
+      continue;
+    }
+    if (code === SINGLE_QUOTE) {
+      singleCount++;
+    } else if (code === DOUBLE_QUOTE) {
+      doubleCount++;
+    }
+    pos++;
+  }
+
+  return { singleCount, doubleCount, hasEscapedNewline, hasRedundantEscape };
+}
 
 /**
- * @param {StringAst} ast
+ * Determine the optimal wrapping quote delimiter.
+ *
+ * @param {number} singleCount
+ * @param {number} doubleCount
+ * @param {'single' | 'double'} preferredQuote
+ * @param {string} currentQuote
  * @return {string}
  */
-function stringify(ast) {
-  let str = '';
-  for (const { value } of ast.nodes) {
-    // Collapse multiple line strings automatically
-    if (value !== L_NEWLINE) {
-      str += value;
-    }
+function selectTargetQuote(
+  singleCount,
+  doubleCount,
+  preferredQuote,
+  currentQuote
+) {
+  if (singleCount > doubleCount) {
+    return '"';
+  }
+  if (doubleCount > singleCount) {
+    return "'";
   }
-  return str;
+  if (singleCount === 0 && doubleCount === 0) {
+    return preferredQuote === 'single' ? "'" : '"';
+  }
+  return currentQuote;
 }
 
 /**
- * @param {string} str
- * @return {StringAst}
+ * Reconstruct string with the target quote, collapsing newlines and updating escapes.
+ *
+ * @param {string} inner
+ * @param {string} targetQuote
+ * @return {string}
  */
-function parse(str) {
-  let code, next, value;
+function reconstructString(inner, targetQuote) {
+  const len = inner.length;
+  const targetCode = targetQuote.charCodeAt(0);
+  const otherQuote = targetQuote === "'" ? '"' : "'";
+  const otherCode = otherQuote.charCodeAt(0);
+  const escapedTargetQuote = '\\' + targetQuote;
+  const chunks = [targetQuote];
+  let cursor = 0;
   let pos = 0;
-  const len = str.length;
-
-  /** @type StringAst */
-  const ast = {
-    nodes: [],
-    types: {
-      escapedSingleQuote: 0,
-      escapedDoubleQuote: 0,
-      singleQuote: 0,
-      doubleQuote: 0,
-    },
-    quotes: false,
-  };
 
   while (pos < len) {
-    code = str.charCodeAt(pos);
-
-    switch (code) {
-      case SPACE:
-      case TAB:
-      case CR:
-      case FEED:
-        next = pos;
-
-        do {
-          next += 1;
-          code = str.charCodeAt(next);
-        } while (
-          code === SPACE ||
-          code === NEWLINE ||
-          code === TAB ||
-          code === CR ||
-          code === FEED
-        );
-
-        ast.nodes.push({
-          type: 'space',
-          value: str.slice(pos, next),
-        });
-        pos = next - 1;
-        break;
-      case SINGLE_QUOTE:
-        ast.nodes.push(T_SINGLE_QUOTE);
-        ast.types[C_SINGLE_QUOTE]++;
-        ast.quotes = true;
-        break;
-      case DOUBLE_QUOTE:
-        ast.nodes.push(T_DOUBLE_QUOTE);
-        ast.types[C_DOUBLE_QUOTE]++;
-        ast.quotes = true;
-        break;
-      case BACKSLASH:
-        next = pos + 1;
-
-        if (str.charCodeAt(next) === SINGLE_QUOTE) {
-          ast.nodes.push(T_ESCAPED_SINGLE_QUOTE);
-          ast.types[C_ESCAPED_SINGLE_QUOTE]++;
-          ast.quotes = true;
-          pos = next;
-          break;
-        } else if (str.charCodeAt(next) === DOUBLE_QUOTE) {
-          ast.nodes.push(T_ESCAPED_DOUBLE_QUOTE);
-          ast.types[C_ESCAPED_DOUBLE_QUOTE]++;
-          ast.quotes = true;
-          pos = next;
-          break;
-        } else if (str.charCodeAt(next) === NEWLINE) {
-          ast.nodes.push(T_NEWLINE);
-          pos = next;
-          break;
+    const code = inner.charCodeAt(pos);
+    if (code === BACKSLASH) {
+      if (pos + 1 < len) {
+        const newlineLen = getEscapedNewlineLength(inner, pos, len);
+        if (newlineLen > 0) {
+          if (pos > cursor) {
+            chunks.push(inner.slice(cursor, pos));
+          }
+          pos += newlineLen;
+          cursor = pos;
+          continue;
         }
-      /*
-       * We need to fall through here to handle the token as
-       * a whole word. The missing 'break' is intentional.
-       */
-      // oxlint-disable-next-line no-fallthrough
-      default:
-        WORD_END.lastIndex = pos + 1;
-        WORD_END.test(str);
-
-        if (WORD_END.lastIndex === 0) {
-          next = len - 1;
-        } else {
-          next = WORD_END.lastIndex - 2;
+        if (inner.charCodeAt(pos + 1) === otherCode) {
+          if (pos > cursor) {
+            chunks.push(inner.slice(cursor, pos));
+          }
+          chunks.push(otherQuote);
+          pos += 2;
+          cursor = pos;
+          continue;
         }
-
-        value = str.slice(pos, next + 1);
-
-        ast.nodes.push({
-          type: C_STRING,
-          value,
-        });
-
-        pos = next;
+        pos += 2;
+        continue;
+      }
+      pos++;
+      continue;
+    }
+    if (code === targetCode) {
+      if (pos > cursor) {
+        chunks.push(inner.slice(cursor, pos));
+      }
+      chunks.push(escapedTargetQuote);
+      pos++;
+      cursor = pos;
+      continue;
     }
     pos++;
   }
 
-  return ast;
+  if (cursor < len) {
+    chunks.push(inner.slice(cursor));
+  }
+  chunks.push(targetQuote);
+  return chunks.join('');
 }
 
 /**
- * @param {valueParser.StringNode} node
- * @param {StringAst} ast
- * @return {void}
+ * Normalizes quotes and line continuations in a single closed CSS string token.
+ *
+ * @param {string} raw
+ * @param {'single' | 'double'} preferredQuote
+ * @return {string}
  */
-function changeWrappingQuotes(node, ast) {
-  const { types } = ast;
-
-  if (types[C_SINGLE_QUOTE] || types[C_DOUBLE_QUOTE]) {
-    return;
-  }
-
-  if (
-    node.quote === L_SINGLE_QUOTE &&
-    types[C_ESCAPED_SINGLE_QUOTE] > 0 &&
-    !types[C_ESCAPED_DOUBLE_QUOTE]
-  ) {
-    node.quote = L_DOUBLE_QUOTE;
-  }
+function normalizeString(raw, preferredQuote) {
+  const currentQuote = raw[0];
+  const inner = raw.slice(1, -1);
+  const { singleCount, doubleCount, hasEscapedNewline, hasRedundantEscape } =
+    scanQuotes(inner, currentQuote);
+
+  const targetQuote = selectTargetQuote(
+    singleCount,
+    doubleCount,
+    preferredQuote,
+    currentQuote
+  );
 
   if (
-    node.quote === L_DOUBLE_QUOTE &&
-    types[C_ESCAPED_DOUBLE_QUOTE] > 0 &&
-    !types[C_ESCAPED_SINGLE_QUOTE]
+    targetQuote === currentQuote &&
+    !hasEscapedNewline &&
+    !hasRedundantEscape
   ) {
-    node.quote = L_SINGLE_QUOTE;
+    return raw;
   }
 
-  ast.nodes = changeChildQuotes(ast.nodes, node.quote);
-}
-/**
- * @param {StringAstNode[]} childNodes
- * @param {string} parentQuote
- * @return {StringAstNode[]}
- */
-function changeChildQuotes(childNodes, parentQuote) {
-  const updatedChildren = [];
-  for (const child of childNodes) {
-    if (
-      child.type === C_ESCAPED_DOUBLE_QUOTE &&
-      parentQuote === L_SINGLE_QUOTE
-    ) {
-      updatedChildren.push(T_DOUBLE_QUOTE);
-    } else if (
-      child.type === C_ESCAPED_SINGLE_QUOTE &&
-      parentQuote === L_DOUBLE_QUOTE
-    ) {
-      updatedChildren.push(T_SINGLE_QUOTE);
-    } else {
-      updatedChildren.push(child);
-    }
-  }
-  return updatedChildren;
+  return reconstructString(inner, targetQuote);
 }
 
 /**
@@ -236,29 +236,27 @@ function changeChildQuotes(childNodes, parentQuote) {
  * @return {string}
  */
 function normalize(value, preferredQuote) {
-  if (!value || !value.length) {
+  if (!value || (!value.includes("'") && !value.includes('"'))) {
     return value;
   }
 
-  return valueParser(value)
-    .walk((child) => {
-      if (child.type !== C_STRING) {
-        return;
-      }
-
-      const ast = parse(child.value);
-
-      if (ast.quotes) {
-        changeWrappingQuotes(child, ast);
-      } else if (preferredQuote === C_SINGLE) {
-        child.quote = L_SINGLE_QUOTE;
-      } else {
-        child.quote = L_DOUBLE_QUOTE;
-      }
-
-      child.value = stringify(ast);
-    })
-    .toString();
+  const chunks = [];
+  let cursor = 0;
+  let changed = false;
+  for (const [type, raw, start, end] of tokenize({ css: value })) {
+    if (type !== TokenType.String) continue;
+    if (!isClosedString(raw)) continue;
+    const normalized = normalizeString(raw, preferredQuote);
+    if (normalized !== raw) {
+      changed = true;
+    }
+    chunks.push(value.slice(cursor, start));
+    chunks.push(normalized);
+    cursor = end + 1;
+  }
+  if (!changed) return value;
+  chunks.push(value.slice(cursor));
+  return chunks.join('');
 }
 
 /**
@@ -277,6 +275,12 @@ function minify(original, cache, preferredQuote) {
   return newValue;
 }
 
+/** @param {import('postcss').Declaration} decl @param {string} value */
+function assignValue(decl, value) {
+  decl.value = value;
+  if (decl.raws.value?.raw) decl.raws.value = { raw: value, value };
+}
+
 /** @typedef {{preferredQuote?: 'double' | 'single'}} Options */
 /**
  * @param {Options} opts
@@ -306,7 +310,13 @@ function pluginCreator(opts) {
             node.selector = minify(node.selector, cache, preferredQuote);
             break;
           case 'decl':
-            node.value = minify(node.value, cache, preferredQuote);
+            {
+              const value =
+                node.raws.value?.value === node.value
+                  ? (node.raws.value.raw ?? node.value)
+                  : node.value;
+              assignValue(node, minify(value, cache, preferredQuote));
+            }
             break;
           case 'atrule':
             node.params = minify(node.params, cache, preferredQuote);

package/types/index.d.ts

@@ -1,30 +1,3 @@
-declare const T_ESCAPED_SINGLE_QUOTE: {
-    type: string;
-    value: string;
-};
-declare const T_ESCAPED_DOUBLE_QUOTE: {
-    type: string;
-    value: string;
-};
-declare const T_SINGLE_QUOTE: {
-    type: string;
-    value: string;
-};
-declare const T_NEWLINE: {
-    type: string;
-    value: string;
-};
-export type StringAstNode = typeof T_ESCAPED_SINGLE_QUOTE | typeof T_ESCAPED_DOUBLE_QUOTE | typeof T_SINGLE_QUOTE | typeof T_NEWLINE;
-export type StringAst = {
-    nodes: StringAstNode[];
-    types: {
-        escapedSingleQuote: number;
-        escapedDoubleQuote: number;
-        singleQuote: number;
-        doubleQuote: number;
-    };
-    quotes: boolean;
-};
 export type Options = {
     preferredQuote?: 'double' | 'single';
 };

package/types/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.js"],"names":[],"mappings":"AAyCA,QAAA,MAAM,sBAAsB;;;CAAiD,CAAC;AAC9E,QAAA,MAAM,sBAAsB;;;CAAiD,CAAC;AAC9E,QAAA,MAAM,cAAc;;;CAAkD,CAAC;AAEvE,QAAA,MAAM,SAAS;;;CAAwC,CAAC;AAEpD,YAAoH,aAAa,GAAvH,OAAO,sBAAsB,GAAG,OAAO,sBAAsB,GAAG,OAAO,cAAc,GAAG,OAAO,SAAS,CAAe;AAElI,YAE6B,SAAS,GAF5B;IAAC,KAAK,EAAE,aAAa,EAAE,CAAC;IACvB,KAAK,EAAE;QAAC,kBAAkB,EAAE,MAAM,CAAC;QAAC,kBAAkB,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAC,CAAC;IAC1G,MAAM,EAAE,OAAO,CAAA;CAAC,CAAW;AAoOrC,YAAkD,OAAO,GAA/C;IAAC,cAAc,CAAC,EAAE,QAAQ,GAAG,QAAQ,CAAA;CAAC,CAAS;AAA7D,gEAAgE;AAChE;;;GAGG;AACH,iBAAS,aAAa,CAAC,IAAI,EAHhB,OAGgB,GAFf,OAAO,SAAS,EAAE,MAAM,CAmCnC;kBAjCQ,aAAa;iBAkCX,IAAI;;AAEf,QAAA,MAAM,aAAa,sBAAgB,CAAC;AAEpC,OAAO,EAAE,aAAa,IAAI,OAAO,EAAE,aAAa,IAAI,gBAAgB,EAAE,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.js"],"names":[],"mappings":"AA2RI,YAAkD,OAAO,GAA/C;IAAC,cAAc,CAAC,EAAE,QAAQ,GAAG,QAAQ,CAAA;CAAC,CAAS;AAA7D,gEAAgE;AAChE;;;GAGG;AACH,iBAAS,aAAa,CAAC,IAAI,EAHhB,OAGgB,GAFf,OAAO,SAAS,EAAE,MAAM,CAyCnC;kBAvCQ,aAAa;iBAwCX,IAAI;;AAEf,QAAA,MAAM,aAAa,sBAAgB,CAAC;AAEpC,OAAO,EAAE,aAAa,IAAI,OAAO,EAAE,aAAa,IAAI,gBAAgB,EAAE,CAAC"}