prettier tweaked this

Author: fbartho

.prettierignore

@@ -0,0 +1,3 @@
+Example/ios/*
+Example/android/*
+node_modules/*

.vscode/settings.json

@@ -1,6 +1,3 @@
 {
-    "cSpell.words": [
-        "jrichardlai",
-        "taskrabbit"
-    ]
-}
+  "cSpell.words": ["jrichardlai", "taskrabbit"]
+}

package.json

@@ -6,7 +6,7 @@
   "types": "src/ParsedText.d.ts",
   "scripts": {
     "prettier": "prettier",
-    "prettier:all": "prettier --write ./*.json ./*.js ./**/*.js src/*.ts ",
+    "prettier:all": "prettier --write '**/*.json' '**/*.js' '**/*.ts' ",
     "lint": "eslint",
     "test": "jest"
   },

src/lib/TextExtraction.js

@@ -6,7 +6,7 @@ class TextExtraction {
    * @param {RegExp} patterns[].pattern - RegExp to be used for parsing
    */
   constructor(text, patterns) {
-    this.text     = text;
+    this.text = text;
     this.patterns = patterns || [];
   }
 
@@ -15,7 +15,7 @@ class TextExtraction {
    * @return {Object[]} - props for all the parts of the text
    */
   parse() {
-    let parsedTexts = [{children: this.text}];
+    let parsedTexts = [{ children: this.text }];
     this.patterns.forEach((pattern) => {
       let newParts = [];
 
@@ -27,27 +27,37 @@ class TextExtraction {
           return;
         }
 
-        let parts    = [];
+        let parts = [];
         let textLeft = parsedText.children;
         let indexOfMatchedString = 0;
 
-        while (textLeft) {
-          let matches = pattern.pattern.exec(textLeft);
-
-          if (!matches) { break; }
-
+        /** @type {RegExpExecArray} */
+        let matches;
+        // Global RegExps are stateful, this makes it start at 0 if reused
+        // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/exec
+        pattern.pattern.lastIndex = 0;
+        while (textLeft && (matches = pattern.pattern.exec(textLeft))) {
           let previousText = textLeft.substr(0, matches.index);
-          indexOfMatchedString += matches.index;
+          indexOfMatchedString = matches.index;
 
-          parts.push({children: previousText});
+          parts.push({ children: previousText });
 
-          parts.push(this.getMatchedPart(pattern, matches[0], matches, indexOfMatchedString));
+          parts.push(
+            this.getMatchedPart(
+              pattern,
+              matches[0],
+              matches,
+              indexOfMatchedString,
+            ),
+          );
 
           textLeft = textLeft.substr(matches.index + matches[0].length);
           indexOfMatchedString += matches[0].length;
+          // Global RegExps are stateful, this makes it operate on the "remainder" of the string
+          pattern.pattern.lastIndex = indexOfMatchedString;
         }
 
-        parts.push({children: textLeft});
+        parts.push({ children: textLeft });
 
         newParts.push(...parts);
       });
@@ -56,9 +66,9 @@ class TextExtraction {
     });
 
     // Remove _matched key.
-    parsedTexts.forEach((parsedText) => delete(parsedText._matched));
+    parsedTexts.forEach((parsedText) => delete parsedText._matched);
 
-    return parsedTexts.filter(t => !!t.children);
+    return parsedTexts.filter((t) => !!t.children);
   }
 
   // private
@@ -75,7 +85,9 @@ class TextExtraction {
     let props = {};
 
     Object.keys(matchedPattern).forEach((key) => {
-      if (key === 'pattern' || key === 'renderText') { return; }
+      if (key === 'pattern' || key === 'renderText') {
+        return;
+      }
 
       if (typeof matchedPattern[key] === 'function') {
         props[key] = () => matchedPattern[key](text, index);
@@ -85,7 +97,10 @@ class TextExtraction {
     });
 
     let children = text;
-    if (matchedPattern.renderText && typeof matchedPattern.renderText === 'function') {
+    if (
+      matchedPattern.renderText &&
+      typeof matchedPattern.renderText === 'function'
+    ) {
       children = matchedPattern.renderText(text, matches);
     }
 

test/TextExtraction.test.js

@@ -71,9 +71,7 @@ describe('TextExtraction', () => {
         'https://long-gtld.xn--vermgensberatung-pwb',
       ];
       const textExtraction = new TextExtraction(
-        `this is my website ${urls[0]} and this is also ${
-          urls[1]
-        } and why not this one also ${urls[2]}`,
+        `this is my website ${urls[0]} and this is also ${urls[1]} and why not this one also ${urls[2]}`,
         [
           {
             pattern: PATTERNS.url,