Como mencioné, el problema está en tener un loop con 54
vueltas, pero no en todas las vueltas añades un número a tu array, dependiendo de si el número ya se había generado o no.
La solución pasa por generar números que no existan en el array.
const contains = (n, arr) => arr.includes(n);
const generator = (from, to, arr) => {
const n = Math.floor(Math.random() * to) + from;
return contains(n, arr)
? generator(from, to, arr)
: n;
}
#contains
es una función recursiva que genera un número aleatorio. Si este número existe en el array, generará otro; si no existe, lo devuelve.
Des del método principal, sólo hace falta llamar al generador y nos aseguramos que el número que devuelve no existe en el array.
const cartas = [];
for (let i = 0; i < 54; ++i) {
const n = generator(1, 54, cartas);
cartas.push(n);
}
console.table(cartas);
Espero que sirva.