No more "MatchAll" (#613)

* Move matchPatterns

* No More Matchall

* 4.5.7

* Format

package.json

@@ -1,6 +1,6 @@
 {
   "name": "randsum",
-  "version": "4.5.6",
+  "version": "4.5.7",
   "private": false,
   "author": "Alex Jarvis",
   "icon": "https://raw.githubusercontent.com/RANDSUM/randsum-ts/main/icon.webp",

src/roll/parseRollArguments/guards.ts

@@ -1,4 +1,4 @@
-import { completeRollPattern, coreNotationPattern } from '~constants'
+import { completeRollPattern, coreNotationPattern } from '~matchPattern'
 import { DiceNotation, DicePoolOptions, Modifiers, RollArgument } from '~types'
 
 export const isDiceNotation = (argument: unknown): argument is DiceNotation => {
@@ -7,11 +7,16 @@ export const isDiceNotation = (argument: unknown): argument is DiceNotation => {
 
   const cleanArg = argument.replace(/\s/g, '')
 
-  const matches = [...cleanArg.matchAll(completeRollPattern)].map(
-    (arr) => arr[0]
-  )
+  const matches = []
 
-  const remaining = matches.reduce((acc, curr) => {
+  let parsed: RegExpExecArray | null
+  while ((parsed = completeRollPattern.exec(cleanArg))) {
+    if (parsed && parsed.groups) {
+      matches.push(Object.values(parsed.groups))
+    }
+  }
+
+  const remaining = matches.flat().reduce((acc, curr) => {
     return acc.replace(curr, '')
   }, cleanArg)
 

src/roll/parseRollArguments/parseNotation.ts

@@ -1,4 +1,4 @@
-import { completeRollPattern } from '~constants'
+import { completeRollPattern } from '~matchPattern'
 import { DiceNotation, DicePoolOptions } from '~types'
 import {
   isCoreNotationMatch,
@@ -8,10 +8,15 @@ import {
   parseModifiers
 } from './parseModifiers'
 
-const findMatches = (notations: string): Match[] =>
-  [...notations.matchAll(completeRollPattern)].map(
-    ({ groups: match }) => match as Match
-  )
+const findMatches = (notations: string): Match[] => {
+  const matches = []
+  let parsed: RegExpExecArray | null
+  while ((parsed = completeRollPattern.exec(notations))) {
+    matches.push(parsed.groups as Match)
+  }
+
+  return matches
+}
 
 const parseNotation = (
   notationString: DiceNotation

tsconfig.json

@@ -3,7 +3,7 @@
     "paths": {
       "~roll": ["./src/roll/index.ts"],
       "~Die": ["./src/Die/index.ts"],
-      "~constants": ["./src/constants.ts"],
+      "~matchPattern": ["./src/matchPattern.ts"],
       "~types": ["./src/types.ts"]
     },
     "lib": ["ESNext"],