@yipe/dice 0.2.2 → 0.2.4

package/dist/builder/index.d.cts

@@ -198,7 +198,6 @@ declare class ACBuilder extends RollBuilder {
     onHit(count: number, sides: number): AttackBuilder;
     onHit(count: number, die: RollBuilder, modifier: number): AttackBuilder;
     onHit(count: number, sides: number, modifier: number): AttackBuilder;
-    dc(_: number): never;
     get critThreshold(): number;
     critOn(threshold: number): ACBuilder;
     toExpression(): string;

package/dist/builder/index.d.ts

@@ -198,7 +198,6 @@ declare class ACBuilder extends RollBuilder {
     onHit(count: number, sides: number): AttackBuilder;
     onHit(count: number, die: RollBuilder, modifier: number): AttackBuilder;
     onHit(count: number, sides: number, modifier: number): AttackBuilder;
-    dc(_: number): never;
     get critThreshold(): number;
     critOn(threshold: number): ACBuilder;
     toExpression(): string;

package/dist/builder/index.js

@@ -2566,6 +2566,7 @@ var RollBuilder = class _RollBuilder {
   /** Apply one-pass reroll threshold (k): reroll faces 1..k once, must keep. */
   reroll(value) {
     if (isNaN(value)) throw new Error("Invalid NaN value for reroll");
+    if (value === this.lastConfig.reroll) return this;
     const newConfigs = this.getSubRollConfigs();
     newConfigs[newConfigs.length - 1].reroll = value;
     return new this.constructor(newConfigs);
@@ -2662,19 +2663,43 @@ var RollBuilder = class _RollBuilder {
     return new this.constructor(newConfigs);
   }
   toExpression() {
-    const diceConfigs = this.subRollConfigs.filter(
+    const originalDiceConfigs = this.subRollConfigs.filter(
       (config) => config.sides && config.sides > 0
     );
+    const configGroups = /* @__PURE__ */ new Map();
+    for (const config of originalDiceConfigs) {
+      const keyConfig = { ...config };
+      delete keyConfig.count;
+      delete keyConfig.modifier;
+      const key = JSON.stringify(keyConfig);
+      const existingGroup = configGroups.get(key);
+      if (existingGroup) {
+        existingGroup.totalCount += config.count;
+      } else {
+        configGroups.set(key, { config, totalCount: config.count });
+      }
+    }
+    const diceConfigs = Array.from(configGroups.values()).map(({ config, totalCount }) => ({
+      ...config,
+      count: totalCount
+    })).sort((a, b) => {
+      if (b.sides !== a.sides) return b.sides - a.sides;
+      const complexity = (c) => (c.reroll > 0 ? 1 : 0) + (c.minimum > 0 ? 1 : 0) + (c.explode > 0 ? 1 : 0) + (c.bestOf > 0 ? 1 : 0) + (c.keep ? 1 : 0);
+      return complexity(b) - complexity(a);
+    });
     const totalModifier = this.subRollConfigs.reduce(
       (sum, config) => sum + config.modifier,
       0
     );
     if (diceConfigs.length === 0) return totalModifier.toString();
     const rootConfig = this.getRootDieConfig();
+    const newRootConfig = rootConfig ? diceConfigs.find(
+      (c) => c.sides === rootConfig.sides && c.rollType === rootConfig.rollType && c.reroll === rootConfig.reroll && c.explode === rootConfig.explode && c.minimum === rootConfig.minimum && c.bestOf === rootConfig.bestOf && c.keep === rootConfig.keep
+    ) : void 0;
     const diceExpressions = diceConfigs.map(
       (config) => this.configToSingleExpressionWithoutModifier(
         config,
-        config === rootConfig
+        config === newRootConfig
       )
     );
     let result = "";
@@ -3275,11 +3300,6 @@ var ACBuilder = class _ACBuilder extends RollBuilder {
     const damageRoll = RollBuilder.fromArgs(...args);
     return new AttackBuilder(this, damageRoll);
   }
-  dc(_) {
-    throw new Error(
-      "Cannot use dc() on an AttackRollBuilder. Use ac() for attack rolls instead."
-    );
-  }
   get critThreshold() {
     return this.attackConfig.critThreshold;
   }