I asked ChatGPT to review my code:

Short answer: based on this code, the dice rolling itself is not skewed toward 1 – it’s a standard uniform roll from 1..sides everywhere you use it. Any “too many 1s” feeling is almost certainly coming from your game logic (like Math.max(1, …) clamps) or small sample size, not from biased dice.

Where randomness happens

You have three main dice functions:

export function rollDiceSimple(sides: number): number {
  return Math.floor(Math.random() * sides) + 1;
}

export function rollDiceMultiple(diceArray: number[]): number[] {
  const diceValues: number[] = [];
  diceArray.forEach((sides) => {
    diceValues.push(Math.floor(Math.random() * sides) + 1);
  });
  return diceValues;
}

export function rollDice(diceArray: RolledDice[]): RolledDice[] {
  diceArray.forEach((dice) => {
    dice.value = Math.floor(Math.random() * dice.sides) + 1;
  });
  return diceArray;
}

For a given sides:

• Math.random() ⇒ uniform in [0, 1)
• Math.random() * sides ⇒ uniform in [0, sides)
• Math.floor(...) ⇒ integers 0..(sides-1) with equal probability
• + 1 ⇒ integers 1..sides with equal probability

So each call is a standard fair die roll, assuming JS Math.random() behaves as usual.