Skip to content

fixed race condition causing lslib to attempt installation twice in quick succession#21929

Merged
IDCs merged 2 commits intomasterfrom
fix/app-98
Mar 30, 2026
Merged

fixed race condition causing lslib to attempt installation twice in quick succession#21929
IDCs merged 2 commits intomasterfrom
fix/app-98

Conversation

@IDCs
Copy link
Copy Markdown
Contributor

@IDCs IDCs commented Mar 27, 2026

Two concurrent installers were fighting over the extracted files, potentially generating a corrupt copy in staging folder.

also consolidated how we build the fileList for the installer, and added instruction sanity check to block copy instructions from being used with directories

Because it's a race condition, it's extremely intermittent and there's no reliable way to reproduce this.

Only way you can test this is to leave a breakpoint in the lslib installer and make sure it only runs once when downloading through the lslib requirement dialog which pops up whenever lslib is missing from the user's mod setup.

fixes https://linear.app/nexus-mods/issue/APP-98/bg3-lslib-installer-fails-on-folder-entries

also consolidated how we build the fileList for the installer, and added
instruction sanity check to block copy instructions from being used with
directories

fixes https://linear.app/nexus-mods/issue/APP-98/bg3-lslib-installer-fails-on-folder-entries
@IDCs IDCs self-assigned this Mar 27, 2026
@IDCs IDCs requested a review from a team as a code owner March 27, 2026 09:03
@IDCs IDCs changed the title fixed race condition causing lslib to attempt installation twice fixed race condition causing lslib to attempt installation twice in quick succession Mar 27, 2026
@IDCs IDCs linked an issue Mar 27, 2026 that may be closed by this pull request
@IDCs IDCs merged commit b530f5f into master Mar 30, 2026
5 checks passed
@IDCs IDCs deleted the fix/app-98 branch March 30, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LsLib Divine Tool cant be downloaded

2 participants