Improve card drawing logic

[kuv2707]

Description

The drawCardFromDeck function in engine.js should handle the scenario when the deck is empty. If the deck is empty, the thrown cards (maintained in another array) should be moved back to the deck before drawing a card for the player.

Current Code Snippet

drawCardFromDeck(player) {
    //todo: Handle the case when the deck is empty and we have to move the thrown cards back to the deck
    this.players.find(p => p.id === player.id).cards.push(this.cardDeck.pop());
}

Expected Output

• The function should handle the case where this.cardDeck is empty.
• When the deck is empty, the thrown cards should be shuffled and moved back to the deck.
• The player should then be able to draw a card from the reshuffled deck.

Notes

• Ensure that the reshuffling of the thrown cards maintains the game's integrity.

[Abhishek-Punhani]

Can I work on this issue?

[ritwik-69]

can i be assigned this issue?

[aayush0325]

id love to work on this issue

[criticic]

Whoever is solving this, make sure to also handle the case where there is no player with the given id, as after converting to typescript in #33 we get this warning:
Object is possibly 'undefined'.ts(2532), due to find returnign undefined if a match is not found.