@iconify/tools 3.0.5 → 4.0.0-beta.1

package/lib/svg/index.d.ts

@@ -1,3 +1,4 @@
+import * as cheerio from 'cheerio';
 import { IconifyIcon } from '@iconify/types';
 export { IconifyIcon } from '@iconify/types';
 import { IconifyIconCustomisations } from '@iconify/utils/lib/customisations/defaults';
@@ -14,7 +15,7 @@ interface ViewBox {
  * SVG class, used to manipulate icon content.
  */
 declare class SVG {
-    $svg: cheerio.Root;
+    $svg: cheerio.CheerioAPI;
     viewBox: ViewBox;
     /**
      * Constructor

package/lib/svg/index.mjs

@@ -1,4 +1,4 @@
-import cheerio from 'cheerio';
+import * as cheerio from 'cheerio';
 import { iconToSVG, trimSVG } from '@iconify/utils';
 
 class SVG {

package/lib/svg/parse-style.cjs

@@ -9,156 +9,147 @@ require('../css/parser/error.cjs');
 require('../css/parser/strings.cjs');
 require('../css/parser/text.cjs');
 
-function parseItem(item, callback, done) {
-  const tagName = item.tagName;
-  const $element = item.$element;
-  function parseStyleItem(done2) {
-    const content = $element.text();
-    if (typeof content !== "string") {
-      $element.remove();
-      return done2();
-    }
-    const tokens = css_parser_tokens.getTokens(content);
-    if (!(tokens instanceof Array)) {
-      throw new Error("Error parsing style");
-    }
-    let changed2 = false;
-    const selectorStart = [];
-    let newTokens = [];
-    const parsedTokens = () => {
-      if (changed2) {
-        const tree = css_parser_tree.tokensTree(
-          newTokens.filter((token) => token !== null)
-        );
-        if (!tree.length) {
-          $element.remove();
-        } else {
-          const newContent = css_parser_export.tokensToString(tree);
-          item.$element.text("\n" + newContent);
-        }
+function assertNotOldCode(value) {
+  if (value instanceof Promise) {
+    throw new Error("parseSVGStyle does not support async callbacks");
+  }
+}
+function parseSVGStyle(svg, callback) {
+  svg_parse.parseSVG(svg, (item) => {
+    const tagName = item.tagName;
+    const $element = item.$element;
+    function parseStyleItem() {
+      const content = $element.text();
+      if (typeof content !== "string") {
+        $element.remove();
+        return;
       }
-      done2();
-    };
-    const nextToken = () => {
-      const token = tokens.shift();
-      if (token === void 0) {
-        return parsedTokens();
+      const tokens = css_parser_tokens.getTokens(content);
+      if (!(tokens instanceof Array)) {
+        throw new Error("Error parsing style");
       }
-      switch (token.type) {
-        case "selector":
-          selectorStart.push(newTokens.length);
-          newTokens.push(token);
-          return nextToken();
-        case "close":
-          selectorStart.pop();
-          newTokens.push(token);
-          return nextToken();
-        case "at-rule": {
-          selectorStart.push(newTokens.length);
-          const prop = token.rule;
-          const value = token.value;
-          const isAnimation = prop === "keyframes" || prop.slice(0, 1) === "-" && prop.split("-").pop() === "keyframes";
-          const childTokens = [];
-          const animationRules = /* @__PURE__ */ Object.create(null);
-          let depth = 1;
-          let index = 0;
-          let isFrom = false;
-          while (depth > 0) {
-            const childToken = tokens[index];
-            index++;
-            if (!childToken) {
-              throw new Error("Something went wrong parsing CSS");
-            }
-            childTokens.push(childToken);
-            switch (childToken.type) {
-              case "close": {
-                depth--;
-                isFrom = false;
-                break;
+      let changed2 = false;
+      const selectorStart = [];
+      let newTokens = [];
+      while (tokens.length) {
+        const token = tokens.shift();
+        if (!token) {
+          break;
+        }
+        switch (token.type) {
+          case "selector":
+            selectorStart.push(newTokens.length);
+            newTokens.push(token);
+            break;
+          case "close":
+            selectorStart.pop();
+            newTokens.push(token);
+            break;
+          case "at-rule": {
+            selectorStart.push(newTokens.length);
+            const prop = token.rule;
+            const value = token.value;
+            const isAnimation = prop === "keyframes" || prop.slice(0, 1) === "-" && prop.split("-").pop() === "keyframes";
+            const childTokens = [];
+            const animationRules = /* @__PURE__ */ Object.create(null);
+            let depth = 1;
+            let index = 0;
+            let isFrom = false;
+            while (depth > 0) {
+              const childToken = tokens[index];
+              index++;
+              if (!childToken) {
+                throw new Error(
+                  "Something went wrong parsing CSS"
+                );
               }
-              case "selector": {
-                depth++;
-                if (isAnimation) {
-                  const rule = childToken.code;
-                  if (rule === "from" || rule === "0%") {
-                    isFrom = true;
+              childTokens.push(childToken);
+              switch (childToken.type) {
+                case "close": {
+                  depth--;
+                  isFrom = false;
+                  break;
+                }
+                case "selector": {
+                  depth++;
+                  if (isAnimation) {
+                    const rule = childToken.code;
+                    if (rule === "from" || rule === "0%") {
+                      isFrom = true;
+                    }
                   }
+                  break;
                 }
-                break;
-              }
-              case "at-rule": {
-                depth++;
-                if (isAnimation) {
-                  throw new Error(
-                    "Nested at-rule in keyframes ???"
-                  );
+                case "at-rule": {
+                  depth++;
+                  if (isAnimation) {
+                    throw new Error(
+                      "Nested at-rule in keyframes ???"
+                    );
+                  }
+                  break;
                 }
-                break;
-              }
-              case "rule": {
-                if (isAnimation && isFrom) {
-                  animationRules[childToken.prop] = childToken.value;
+                case "rule": {
+                  if (isAnimation && isFrom) {
+                    animationRules[childToken.prop] = childToken.value;
+                  }
+                  break;
                 }
-                break;
               }
             }
-          }
-          const skipCount = childTokens.length;
-          callback(
-            isAnimation ? {
-              type: "keyframes",
-              prop,
-              value,
-              token,
-              childTokens,
-              from: animationRules,
-              prevTokens: newTokens,
-              nextTokens: tokens.slice(0)
-            } : {
-              type: "at-rule",
-              prop,
-              value,
-              token,
-              childTokens,
-              prevTokens: newTokens,
-              nextTokens: tokens.slice(0)
-            },
-            (result) => {
-              if (result !== void 0) {
-                if (isAnimation) {
-                  if (result !== value) {
-                    changed2 = true;
-                    token.value = result;
-                  }
-                  newTokens.push(token);
-                  for (let i = 0; i < skipCount; i++) {
-                    tokens.shift();
-                  }
-                  newTokens = newTokens.concat(childTokens);
-                } else {
-                  if (result !== value) {
-                    throw new Error(
-                      "Changing value for at-rule is not supported"
-                    );
-                  }
-                  newTokens.push(token);
+            const skipCount = childTokens.length;
+            const result = callback(
+              isAnimation ? {
+                type: "keyframes",
+                prop,
+                value,
+                token,
+                childTokens,
+                from: animationRules,
+                prevTokens: newTokens,
+                nextTokens: tokens.slice(0)
+              } : {
+                type: "at-rule",
+                prop,
+                value,
+                token,
+                childTokens,
+                prevTokens: newTokens,
+                nextTokens: tokens.slice(0)
+              }
+            );
+            if (result !== void 0) {
+              assertNotOldCode(result);
+              if (isAnimation) {
+                if (result !== value) {
+                  changed2 = true;
+                  token.value = result;
                 }
-              } else {
-                changed2 = true;
+                newTokens.push(token);
                 for (let i = 0; i < skipCount; i++) {
                   tokens.shift();
                 }
+                newTokens = newTokens.concat(childTokens);
+              } else {
+                if (result !== value) {
+                  throw new Error(
+                    "Changing value for at-rule is not supported"
+                  );
+                }
+                newTokens.push(token);
+              }
+            } else {
+              changed2 = true;
+              for (let i = 0; i < skipCount; i++) {
+                tokens.shift();
               }
-              nextToken();
             }
-          );
-          return;
-        }
-        case "rule": {
-          const value = token.value;
-          const selectorTokens = selectorStart.map((index) => newTokens[index]).filter((item2) => item2 !== null);
-          callback(
-            {
+            break;
+          }
+          case "rule": {
+            const value = token.value;
+            const selectorTokens = selectorStart.map((index) => newTokens[index]).filter((item2) => item2 !== null);
+            const result = callback({
               type: "global",
               prop: token.prop,
               value,
@@ -179,122 +170,74 @@ function parseItem(item, callback, done) {
               ),
               prevTokens: newTokens,
               nextTokens: tokens.slice(0)
-            },
-            (result) => {
-              if (result !== void 0) {
-                if (result !== value) {
-                  changed2 = true;
-                  token.value = result;
-                }
-                newTokens.push(token);
-              } else {
+            });
+            if (result !== void 0) {
+              assertNotOldCode(result);
+              if (result !== value) {
                 changed2 = true;
+                token.value = result;
               }
-              nextToken();
+              newTokens.push(token);
+            } else {
+              changed2 = true;
             }
-          );
-          return;
+            break;
+          }
         }
       }
-    };
-    nextToken();
-  }
-  if (tagName === "style") {
-    return parseStyleItem(done);
-  }
-  const attribs = item.element.attribs;
-  if (attribs.style === void 0) {
-    return done();
-  }
-  const parsedStyle = css_parse.parseInlineStyle(attribs.style);
-  if (parsedStyle === null) {
-    $element.removeAttr("style");
-    return done();
-  }
-  const propsQueue = Object.keys(parsedStyle);
-  let changed = false;
-  const parsedProps = () => {
-    if (changed) {
-      const newStyle = Object.keys(parsedStyle).map((key) => key + ":" + parsedStyle[key] + ";").join("");
-      if (!newStyle.length) {
-        $element.removeAttr("style");
-      } else {
-        $element.attr("style", newStyle);
+      if (changed2) {
+        const tree = css_parser_tree.tokensTree(
+          newTokens.filter((token) => token !== null)
+        );
+        if (!tree.length) {
+          $element.remove();
+        } else {
+          const newContent = css_parser_export.tokensToString(tree);
+          item.$element.text("\n" + newContent);
+        }
       }
     }
-    done();
-  };
-  const nextProp = () => {
-    const prop = propsQueue.shift();
-    if (prop === void 0) {
-      return parsedProps();
+    if (tagName === "style") {
+      parseStyleItem();
+      return;
+    }
+    const attribs = item.element.attribs;
+    if (attribs.style === void 0) {
+      return;
+    }
+    const parsedStyle = css_parse.parseInlineStyle(attribs.style);
+    if (parsedStyle === null) {
+      $element.removeAttr("style");
+      return;
     }
-    const value = parsedStyle[prop];
-    callback(
-      {
+    let changed = false;
+    for (const prop in parsedStyle) {
+      const value = parsedStyle[prop];
+      const result = callback({
         type: "inline",
         prop,
         value,
         item
-      },
-      (result) => {
-        if (result !== value) {
-          changed = true;
-          if (result === void 0) {
-            delete parsedStyle[prop];
-          } else {
-            parsedStyle[prop] = result;
-          }
+      });
+      assertNotOldCode(result);
+      if (result !== value) {
+        changed = true;
+        if (result === void 0) {
+          delete parsedStyle[prop];
+        } else {
+          parsedStyle[prop] = result;
         }
-        nextProp();
-      }
-    );
-  };
-  nextProp();
-}
-async function parseSVGStyle(svg, callback) {
-  return svg_parse.parseSVG(svg, (item) => {
-    return new Promise((fulfill, reject) => {
-      try {
-        parseItem(
-          item,
-          (styleItem, done) => {
-            try {
-              const result = callback(styleItem);
-              if (result instanceof Promise) {
-                result.then(done).catch(reject);
-              } else {
-                done(result);
-              }
-            } catch (err) {
-              reject(err);
-            }
-          },
-          fulfill
-        );
-      } catch (err) {
-        reject(err);
       }
-    });
-  });
-}
-function parseSVGStyleSync(svg, callback) {
-  let isSync = true;
-  svg_parse.parseSVGSync(svg, (item) => {
-    parseItem(
-      item,
-      (styleItem, done) => {
-        done(callback(styleItem));
-      },
-      () => {
-        if (!isSync) {
-          throw new Error("parseSVGStyleSync callback was async");
-        }
+    }
+    if (changed) {
+      const newStyle = Object.keys(parsedStyle).map((key) => key + ":" + parsedStyle[key] + ";").join("");
+      if (!newStyle.length) {
+        $element.removeAttr("style");
+      } else {
+        $element.attr("style", newStyle);
       }
-    );
+    }
   });
-  isSync = false;
 }
 
 exports.parseSVGStyle = parseSVGStyle;
-exports.parseSVGStyleSync = parseSVGStyleSync;

package/lib/svg/parse-style.d.ts

@@ -1,6 +1,7 @@
 import { SVG } from './index.js';
 import { CSSRuleToken, CSSToken, CSSAtRuleToken } from '../css/parser/types.js';
 import { ParseSVGCallbackItem } from './parse.js';
+import 'cheerio';
 import '@iconify/types';
 import '@iconify/utils/lib/customisations/defaults';
 import '../misc/cheerio.js';
@@ -45,19 +46,13 @@ type ParseSVGStyleCallbackResult = string | undefined;
 /**
  * Callback function
  */
-type ParseSVGStyleCallback = (item: ParseSVGStyleCallbackItem) => ParseSVGStyleCallbackResult | Promise<ParseSVGStyleCallbackResult>;
-type ParseSVGStyleCallbackSync = (item: ParseSVGStyleCallbackItem) => ParseSVGStyleCallbackResult;
+type ParseSVGStyleCallback = (item: ParseSVGStyleCallbackItem) => ParseSVGStyleCallbackResult;
 /**
  * Parse styles in SVG
  *
  * This function finds CSS in SVG, parses it, calls callback for each rule.
  * Callback should return new value (string) or undefined to remove rule.
- * Callback can be asynchronous.
  */
-declare function parseSVGStyle(svg: SVG, callback: ParseSVGStyleCallback): Promise<void>;
-/**
- * Synchronous version
- */
-declare function parseSVGStyleSync(svg: SVG, callback: ParseSVGStyleCallbackSync): void;
+declare function parseSVGStyle(svg: SVG, callback: ParseSVGStyleCallback): void;
 
-export { ParseSVGStyleCallback, ParseSVGStyleCallbackItem, ParseSVGStyleCallbackResult, ParseSVGStyleCallbackSync, parseSVGStyle, parseSVGStyleSync };
+export { ParseSVGStyleCallback, ParseSVGStyleCallbackItem, ParseSVGStyleCallbackResult, parseSVGStyle };