fix(remark-lesetid): estimate text one time instead of each text node

luxass · Feb 3, 2024 · b8e009e · b8e009e
1 parent 0178bd5
commit b8e009e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 11 deletions.
diff --git a/.changeset/shy-coins-wait.md b/.changeset/shy-coins-wait.md
@@ -0,0 +1,5 @@
+---
+"remark-lesetid": patch
+---
+
+only estimate text one time
diff --git a/packages/remark-lesetid/src/index.ts b/packages/remark-lesetid/src/index.ts
@@ -20,21 +20,21 @@ const remarkLesetid: Plugin<Options[], Root> = (options) => {
   }
 
   const { dataKey } = options;
-
   return (tree, file) => {
+    let textOnPage = "";
     visit(tree, "text", (node) => {
-      const textOnPage = node.value;
-      const estimation = estimate(textOnPage);
+      textOnPage += node.value;
+    });
 
-      // if matter is defined in data, the frontmatter
-      // parser is vfile-matter.
-      if ("matter" in file.data && file.data.matter) {
-        file.data.matter[dataKey] ||= estimation;
-        return;
-      }
+    const estimation = estimate(textOnPage);
+    // if matter is defined in data, the frontmatter
+    // parser is vfile-matter.
+    if ("matter" in file.data && file.data.matter) {
+      file.data.matter[dataKey] ||= estimation;
+      return;
+    }
 
-      file.data[dataKey] ||= estimation;
-    });
+    file.data[dataKey] ||= estimation;
   };
 };