roll-parser 3.3.0 → 3.3.1

package/src/evaluator/modifiers/flags.ts

@@ -68,11 +68,27 @@ export function stripFlags(
   return modifiers.filter((modifier) => !excluded.includes(modifier));
 }
 
-/** Returns `modifiers` with `excluded` removed and `added` appended, in order. */
+/**
+ * Returns `modifiers` with `excluded` removed and `added` appended, in order.
+ *
+ * ! Always a fresh array. `mergeMetaRolls` clones a die as
+ * ! `{ ...die, modifiers: rewriteFlags(...) }`, so returning the input when it
+ * ! already matches would leave clone and original sharing mutated flags.
+ */
 export function rewriteFlags(
   modifiers: readonly DieModifier[],
   excluded: readonly DieModifier[],
   ...added: DieModifier[]
 ): DieModifier[] {
-  return [...stripFlags(modifiers, excluded), ...added];
+  // One array, not `stripFlags`'s filter plus a spread — this runs per die.
+  const rewritten: DieModifier[] = [];
+
+  for (const modifier of modifiers) {
+    if (!excluded.includes(modifier)) rewritten.push(modifier);
+  }
+  for (const flag of added) {
+    rewritten.push(flag);
+  }
+
+  return rewritten;
 }

package/src/render.ts

@@ -342,8 +342,9 @@ function renderGroup(
   plain: boolean,
 ): string {
   const { keptIndices } = part;
+  const kept = keptIndices == null ? null : new Set(keptIndices);
   const subRolls = part.parts.map((sub, index) => {
-    if (keptIndices == null || keptIndices.includes(index)) {
+    if (kept == null || kept.has(index)) {
       return renderPart(sub, marks, plain);
     }
     const inner = renderPart(sub, marks, true);

package/src/rng/seeded.ts

@@ -246,6 +246,10 @@ export class SeededRNG implements RNG {
    */
   private toSeedString(seed?: string | number): string {
     // ! The library's one permitted `Math.random()` site — both draws stay here.
+    // ! Decimal double-to-string is a JIT fast path and other radixes are not, so
+    // ! every shorter encoding measured slower despite the shorter `hashSeed` loop.
+    // ! Only truncating each draw to a uint32 beats it, at 64 draw bits against the
+    // ! ~100 documented above. A faster auto-seed must skip the string, not shorten it.
     if (seed == null) return `${Date.now()}-${Math.random()}-${Math.random()}`;
     return String(seed);
   }

package/src/version.ts

@@ -1,2 +1,2 @@
 // Generated by `bun run generate:version` from package.json — do not edit.
-export const version = '3.3.0';
+export const version = '3.3.1';