Repository navigation
Say when the downgrade list may be out of date, and check Steam updating a pinned game - #94
Merged
Merged
Conversation
…ing a pinned game The recipe index now records why the latest list wasn't used (offline, or for a newer ModSync). When the game needs a version the list can't reach, the Game version sheet and `modsync game status` say which copy was used and why. Tests cover the fallback with the network failing in several ways. A game newer than where the patches start no longer says "Update Skyrim in Steam first". `modsync game downgrade` with no terminal to answer stops with a message instead of a traceback. docs/advanced.md records what happens when Steam updates the game anyway, checked on a real install: a verify, a lost pin, and a pin queued while Steam was open. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…old ones
SKSE 2.0.20 names its scripts in CamelCase and 2.2.6 in lowercase, so
installing one over the other kept both on Linux. Wine may load either, and
Steam refuses to move the game ("Content file locked"). The installer now
matches names regardless of case, removes other spellings and keeps the
folder spelling already on disk. The game check counts scripts under two
spellings and offers to reinstall SKSE. The install summary no longer
lists files it replaced as removed.
Found by moving the real install to a second Steam library for #26.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Checked on a real install: a move marks the game "Update Required, Files Missing", ModSync finds it in the new library with its backup and offers "Keep this version", and pinning clears both flags. Also explains "Content file locked" when a move stops on names that differ only in case. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Proton#10270 (fix proposed in Proton#10276) adds DriverStore beside Wine's driverstore, and the proposed fix deletes DriverStore outright. Advise emptying DriverStore into driverstore, not the other way round, so Wine's drivers survive that fix. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…, bad lists - The SKSE installer never deletes through a link or outside the game folder, and cleans every spelling of a folder (Data/Scripts and Data/scripts), so nothing is left behind or reported missing. - Only scripts SKSE ships count as its leftovers: a mod's own duplicates aren't something reinstalling SKSE could fix. - A recipe index whose versions don't read is rejected while parsing, so the saved or bundled copy is used instead of the Game card failing. - An older or damaged index is "couldn't read", not "for a newer ModSync". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Works through the remaining items on #26.
Patch list unreachable
ModSync already fell back to the last list it downloaded, then to the one bundled with it. Now it also records why the latest list wasn't used: it couldn't be downloaded, or it's for a newer ModSync. Opening ModSync from Steam's Play skips the download on purpose, so that case isn't treated as a failure.
modsync game statusprints it too.Fixes found along the way
modsync game downgradewith no terminal to answer "Proceed?" crashed with a traceback. It now stops with a message pointing to--yes.ActiveMagicEffect.pexand 2.2.6 usesactivemagiceffect.pex. Installing one over the other kept both, because Linux treats them as different files. Under Wine either one may load, and Steam refused to move the game to another library ("Content file locked"). On the real install, 62 scripts were there twice.Review by GPT-6.1-Sol (2026-10-18)
It found five problems and reproduced each one. All are fixed, each with a test that fails without its fix:
datawas a link to another drive, the installer deleted old script spellings there. It now never deletes through a link, or anything outside the game folder.Data/ScriptsandData/scripts, the cleanup and the check looked in only one of them. Both now look in every spelling.Steam updating a pinned game, tested on a real install
docs/advanced.md has a new section, "When Steam updates the game anyway", which covers library moves and "Content file locked" too.
Closes #26.
🤖 Generated with Claude Code