[Draft] Integration test branch for current fixes#2751
Closed
mindless2831 wants to merge 20 commits into
Closed
Conversation
added 20 commits
May 17, 2026 00:49
Applies the focused Nautilus GameInput/keybind compatibility work from JackNytely's PR SubnauticaNitrox#2663. This restores compatibility with newer Nautilus builds that extend Subnautica's GameInput system, while excluding unrelated launcher/server changes from the original PR. Credit: based on JackNytely's work in SubnauticaNitrox#2663.
Removes readonly from UwePrefab so spawning code can mutate Probability without throwing a MissingMethodException for UwePrefab.set_Probability. This fixes repeated server-side sync exceptions during multiplayer world/entity spawning.
Member
|
Closing for now, we'll test the changes individually 😄 |
Contributor
Author
|
Sorry, this wasn't supposed to be reviewed or anything, it was just for me
to use and test stuff lol. That is why I madenit a draft. Was there a
better, more preferable way to do this?
…On Wed, May 20, 2026, 8:50 AM dartasen ***@***.***> wrote:
*dartasen* left a comment (SubnauticaNitrox/Nitrox#2751)
<#2751 (comment)>
Closing for now, we'll test the changes individually 😄
—
Reply to this email directly, view it on GitHub
<#2751 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACP6WRTQBW77C5PJCR3NF5T43WZ3LAVCNFSM6AAAAACZCGUAMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DIOJZGA3DQOBYHA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
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.
Purpose
This is a temporary integration/testing branch that combines the current active fixes from my fork into one build for anyone who wants to test them together before they are individually merged.
This PR is not intended to replace the focused PRs. The focused PRs remain the proper review/merge targets. This branch is only meant to make it easier for users and maintainers to test a practical build that includes the related fixes together.
Included fixes
#2749 - Fix Release launcher dependency loading from
libFixes Release launcher dependency loading so the packaged Release layout resolves assemblies consistently from the launcher-local dependency path.
This addresses Release-only issues such as:
#2740 - Restore Nautilus GameInput compatibility
Restores compatibility between Nitrox and Nautilus GameInput/keybind handling.
This addresses Nautilus-related issues such as:
#2747 - Fix launcher server Stop command
Fixes the launcher Stop button path for external servers.
This addresses cases where the launcher sent the stop/quit command but the external server did not reliably receive or process it, causing the launcher to remain in a closing/stopping state.
#2750 - Fix UwePrefab probability setter
Fixes the
UwePrefabprobability mutation issue during world/entity spawning.This addresses server-side world-sync crashes where batch/entity spawning attempted to mutate
ProbabilityonUwePrefab, causing missing-method/runtime failures during multiplayer sync or additional cell/batch loading.#2752 - Fix batteryless tool battery duplication
Fixes the exploit where battery-powered tools could generate a new fully charged battery after being dropped and picked back up with no battery installed.
This also preserves correct behavior for:
Why this integration branch exists
Several of these fixes are easier to test together in a real modded Release setup than in isolation.
For example:
This branch gives testers one branch to pull while the individual PRs continue through normal focused review.
Suggested testing
Recommended coverage:
Notes
This branch is a convenience/testing branch only.
Once the individual PRs are merged into
master, this integration branch can be deleted or rebased as needed.