Updating/Cleanup importing process

[matthew-pye]

Updating/Clean up importing process and various additions + fixes

This is a big PR so I will leave it as a draft request for now, in case any issues get found and changes can be made. I have attached a debug build of the changes that have been tested in a windows sandbox.

I would of packed it into a .pext but I'm unsure how to do it correctly as when I do it get sharpcompress errors like this #74)

RomM-ImportRefactor.zip

With this being a big pull request I want to say that no AI was used in the creation of any of this code the only resources used were documentation, blind testing to see what works and a bit of stack overflow

Changes

Rework

• Move import functionality to RomMImportController and moved processing data from server to RomMImport
• Replaced GameId with new id RomMId:SHA1
• Switch GetGames to ImportGames should make metadata stick and not be replaced [Feature] Metadata sync from romM #53
• Moved old GameId data to json this will allow for future expansion and changes. This means users don't have to reimport there library if any endpoint on the server side changes
• Swapped mappings to use RomM platforms instead of igdb platforms [Bug] Custom Platforms not able to be chosen in ROMM Integration #27 & Having an issue pulling roms from Romm For Browser Games #97

Additions

• Added SHA1 hash function for games that don't have a SHA1 hash (Wii U, Xbox 360, Switch, etc)
• Added migration for converting old GameId to new GameId
• Added support for GetMetadataDownloader so users can use the Download Metadata option in Playnite (Official Store) [Feature] Metadata sync from romM #53
• Added option to keep games deleted from server [Feature] Disable removal of games #61
• Added game collection data to categories [Feature] Adding Collections to PlayNite instead of whole Libraries #21
• Added manuals to the games metadata
• Added option to skip ROMs that are missing from RomM filesystem
• Added global exclude genre filter
• Added support for new client tokens with RomM server 4.8

Fixes

• Replace the endpoint used for single file ROMs to /api/romfiles & /api/roms/files (For 4.8) as sometimes in previous versions a single .nsp file with dlc and update folders would download a zip of all the files but the plugin has only requested the single .nsp causing the install to think it had only downloaded the single file should fix [Bug] Switch NSPs and XCIs Within Folders Downloaded Incorrectly #37
• Cleanup game revision merging and switching [Feature] Better handling of games with revisions #91
• Fixed Installer never checking to see if m3u is supported by the emulator
• Fixed 7z path not being saved correctly

New Settings Page

Other issues

I tried to tackle some of the other issue but may have already been fixed or I was unable to recreate

• I tried installing a single ROM @ 13.9GB and multi-file ROM @ 14.4GB both succeeded [Bug] Error when installing large games (timeout) #47
• I can get a 500 error when installing some unpacked Wii U ROMs but doesn't seem related to [Bug] Multifile directory trees break downloads #92
  [romm] 2026-03-13T15:35:31.978395884Z INFO: [RomM][rom][2026-03-13 15:35:31] User matt is downloading PAPER MARIO Color Splash
[romm] 2026-03-13T15:35:32.053132741Z INFO: [RomM][httptools_impl][2026-03-13 15:35:32] 192.168.1.99:0 - "GET /api/roms/635/content/PAPER%20MARIO%20Color%20Splash HTTP/1.0" 200
[romm] 2026-03-13T15:35:53.727227677Z 2026/03/13 15:35:53 [crit] 171#171: *144 open() "/romm/library/roms/Wii U/PAPER MARIO Color Splash/content/script/fld/map/G3/G3_LabOutside.bin.lz" failed (24: No file descriptors available), client: 192.168.1.99, server: localhost, request: "GET /api/roms/635/content/PAPER%20MARIO%20Color%20Splash HTTP/1.1", subrequest: "/library/roms/Wii U/PAPER MARIO Color Splash/content/script/fld/map/G3/G3_LabOutside.bin.lz", host: "192.168.1.100:1337"
[romm] 2026-03-13T15:35:53.727270878Z 2026/03/13 15:35:53 [error] 171#171: *144 mod_zip: a subrequest returned 500, aborting..., client: 192.168.1.99, server: localhost, request: "GET /api/roms/635/content/PAPER%20MARIO%20Color%20Splash HTTP/1.1", subrequest: "/library/roms/Wii U/PAPER MARIO Color Splash/content/script/fld/map/G3/G3_LabOutside.bin.lz", host: "192.168.1.100:1337"
[romm] 2026-03-13T15:35:53.727245067Z INFO: [RomM][nginx][2026-03-13 15:35:53] 192.168.1.99 | - | GET /api/roms/635/content/PAPER%20MARIO%20Color%20Splash 200 | 1010990977 | Unknown Unknown | 23.724

Testing

I have tested everything that I've written and changed all seem to be working, installing this version over a pervious one properly updates the gameId to the new Id and retains the users playtime data. It does require that the user to reselect the platform in there mappings.

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[gantoine]

@matthew-pye soon as this is ready set to ready for review and i'll have copilot take a look for ya

[matthew-pye]

Sure, I was ready but noticed a few bugs when I started using it on my real Playnite instance instead of just a sandbox instance, It should be ready now. Ill fix any issues that arise tomorrow though.

[Copilot]

Pull request overview

This PR refactors the RomM Playnite plugin’s import pipeline to use a new ImportGames flow (with local per-ROM JSON state keyed by RomMId:SHA1), expands settings/UI (platform sync, client token auth, new import options), and adds a RomM-backed metadata downloader.

Changes:

• Replaces the legacy GetGames import with ImportGames via RomMImportController/RomMImport and switches to RomMId:SHA1 identifiers with local JSON state.
• Updates settings UX and auth options (Basic Auth vs client token), plus new import/install toggles (keep deleted games, skip missing files, exclude genres, etc.).
• Adds GetMetadataDownloader support and extends RomM models to include additional IDs/manuals/hash info.

Reviewed changes

Copilot reviewed 20 out of 21 changed files in this pull request and generated 15 comments.

Show a summary per file

File	Description
Settings/SettingsView.xaml.cs	Adds settings UI event handlers (test connection, platform sync, mapping add).
Settings/SettingsView.xaml	Reworks the settings layout and adds new settings fields/options.
Settings/Settings.cs	Expands settings model (auth/token/profile info, platform list, new flags) and adds server connection test logic.
Settings/EmulatorMapping.cs	Refactors mapping model to use RomM platforms and new mapping UI fields.
RomM.csproj	Updates content packaging (adds profile.png, adjusts icon.png item metadata).
RomM.cs	Switches to ImportGames, adds metadata provider, revises local ROM data storage path, updates platform mapping usage.
IRomm.cs	Updates plugin interface to support new import/metadata flows (Id/Source/FetchRom).
Games/RomMMetadataProvider.cs	Adds RomM-backed metadata downloader implementation.
Games/RomMInstallController.cs	Updates install controller to use locally saved ROM install info (incl. revision selection output).
Games/RomMImportController.cs	Introduces import controller orchestrating platform fetch + per-mapping processing.
Games/RomMImport.cs	Implements the new import processing, local JSON persistence, SHA1 fallback, and migration logic.
Games/RomMGameInfo.cs	Adjusts install controller wiring to the new install-info model.
Downloads/VersionSelector.xaml.cs	Switches revision selector to operate on GameInstallInfo instead of sibling DTOs.
Downloads/VersionSelector.xaml	Updates bindings for the new revision selection model.
Models/RomM/RomMUser.cs	Adds model for /api/users/me response.
Models/RomM/RomMHeartbeat.cs	Adds model types for heartbeat/server version parsing.
Models/RomM/Rom/RomMRomUser.cs	Adds is_main_sibling field support.
Models/RomM/Rom/RomMRomLocal.cs	Adds local persisted ROM state model (base + sibling selection).
Models/RomM/Rom/RomMRom.cs	Extends ROM model (file IDs, extra IDs, sha1/manual fields, processed flag).
Models/RomM/Platform/RomMPlatform.cs	Adds equality overrides for platform objects.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.