Delete DLL backups on game update - #588

src/Data/GameHistory.cs:28

• After adding AssetPath to GameHistory, ensure you include a corresponding database migration or schema update to add the asset_path column to the GameHistory table. Without this, insert operations may fail at runtime.

    [Column("asset_path")]

src/Data/Game.cs:315

• [nitpick] The local function name ProcessGame_ProcessGameAsset duplicates the containing method's name and is quite verbose. Consider renaming it to something more concise like HandleGameAsset or ProcessAssetUpdate for clarity.

                    void ProcessGame_ProcessGameAsset(GameAsset gameAsset)

src/Data/Game.cs:312

• [nitpick] This method has grown significantly with nested local functions and branching. Consider extracting the asset-processing logic into private helper methods on the class to reduce complexity and improve readability.

                    var dllHistory = new List<GameHistory>();

src/Data/Game.cs:481

• [nitpick] There's a commented-out block for backupGameAsset logic that appears obsolete. Remove this commented code to keep the codebase clean.

                        }

src/Data/Game.cs:329

• Consider adding unit tests for ProcessGame to verify that the new history events (DLLDetected, DLLChangedExternally, DLLBackupRemoved) are correctly recorded when assets are added, changed, or backups removed.

                                dllHistory.Add(new GameHistory()