publish: fix ModConfig.json regex inclusion duplicate check

[Nenkai]

Summary by CodeRabbit

• Bug Fixes
  • Fixed an issue in the mod publishing dialog where ModConfig.json could be added multiple times to the included files list. The dialog now properly detects existing entries using case-insensitive comparison, preventing duplicate configurations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Fixed duplicate-detection logic in PublishModDialogViewModel constructor to compare StringWrapper values instead of wrapper objects. Changed from direct Contains check to iterating wrapper entries and checking their Value property with case-insensitive matching.

Changes

Cohort / File(s)	Summary
Dialog ViewModel Fix source/Reloaded.Mod.Launcher.Lib/Models/ViewModel/Dialog/PublishModDialogViewModel.cs	Modified IncludeRegexes initialization to prevent duplicate "ModConfig.json" entries by checking wrapper values with case-insensitive comparison instead of comparing StringWrapper objects directly.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A wrapper once tripped on its own,
Comparing wrong types, now plainly shown!
With .Value checked case-free and right,
"ModConfig" dupes vanish from sight.
The logic now hops without a care! 🎉

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Sewer56]

This is just a casing change.

But in the screenshot the casing is the same.
Anything I'm missing?

[Sewer56]

Merging in any case, as names on Windows FileSystem are case insensitive, but nonetheless; kinda curious to know.

[Nenkai]

Root issue is that List<T>.Contains calls Equals & StringWrapper doesn't implement it so it was always going to fail.
Casing was just incase but let me know if it should be changed.

[Sewer56]

Ah, checks out. I didn't catch that was a stringwrapper from the web UI