Sweep: Refactor item ID to be UUIDS so to avoid collision

Here's the PR! https://github.com/Kuzat/Mr-Roller-the-Bot/pull/40.

⚡ Sweep Free Trial: I used GPT-4 to create this ticket. You have 3 GPT-4 tickets left. For more GPT-4 tickets, visit our payment portal.To get Sweep to recreate this ticket, leave a comment prefixed with "sweep:" or edit the issue.

Step 1: 🔍 Code Search

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/Kuzat/Mr-Roller-the-Bot/blob/05a0368dae04999901a8d77f6bf2ee44fff9df53/roller_bot/clients/backends/admin_commands_backend.py#L9-L167 https://github.com/Kuzat/Mr-Roller-the-Bot/blob/05a0368dae04999901a8d77f6bf2ee44fff9df53/roller_bot/views/buy_item_view.py#L18-L111 https://github.com/Kuzat/Mr-Roller-the-Bot/blob/05a0368dae04999901a8d77f6bf2ee44fff9df53/roller_bot/items/models/item.py#L1-L55 https://github.com/Kuzat/Mr-Roller-the-Bot/blob/05a0368dae04999901a8d77f6bf2ee44fff9df53/roller_bot/items/models/dice.py#L1-L103 https://github.com/Kuzat/Mr-Roller-the-Bot/blob/05a0368dae04999901a8d77f6bf2ee44fff9df53/roller_bot/models/user.py#L104-L153

I also found the following external resources that might be helpful:

Summaries of links found in the content:

Step 2: 🧐 Snippet Analysis

From looking through the relevant snippets, I decided to make the following modifications:

File Path	Proposed Changes
`roller_bot/items/models/item.py`	Change the `id` attribute to be a UUID and update all methods that use this attribute to handle UUIDs.
`roller_bot/items/models/dice.py`	Update all methods that use the `id` attribute to handle UUIDs.
`roller_bot/models/user.py`	Update the `has_item`, `get_item`, `get_bonus_values_for_item`, and `new_user` methods to handle UUIDs for item IDs.
`roller_bot/clients/backends/admin_commands_backend.py`	Update the `change_luck`, `add_item`, `remove_item`, `add_credit`, `remove_credit`, and `user_info` methods to handle UUIDs for item IDs.

Step 3: 📝 Planning

I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request:

Refactor item ID to use UUIDs to avoid collision sweep/refactor-item-id-to-uuids

Description

This PR refactors the item ID system in the bot to use UUIDs instead of integers. This change is made to avoid potential collisions and improve the overall robustness of the system. The following changes have been made:

Modified the Item class and its subclasses (Dice and others) to use UUIDs for item IDs.

Updated the User class and its methods that interact with item IDs to handle UUIDs.

Updated the AdminCommandsBackend class and other classes that handle item IDs to handle UUIDs.

Added a script to generate new UUIDs for all existing items in the database.

Summary of Changes

Modified Item class to use UUIDs for item IDs.

Updated Dice class to handle UUIDs for item IDs.

Updated User class methods (has_item, get_item, get_bonus_values_for_item, new_user) to handle UUIDs for item IDs.

Updated AdminCommandsBackend class methods (change_luck, add_item, remove_item, add_credit, remove_credit, user_info) to handle UUIDs for item IDs.

Added script to generate new UUIDs for all existing items in the database.

Step 4: ⌨️ Coding

I have finished coding the issue. I am now reviewing it for completeness.

Step 5: 🔁 Code Review

Success! 🚀

To recreate the pull request, leave a comment prefixed with "sweep:" or edit the issue. ^{Join Our Discord}

Kuzat / Mr-Roller-the-Bot