Skip to content

馃悰 Stop config file watcher on client shutdown - #12

Merged
JulianVennen merged 1 commit into
aternosorg:masterfrom
Zoriot:shutdown-non-daemon-threads
Aug 13, 2026
Merged

馃悰 Stop config file watcher on client shutdown#12
JulianVennen merged 1 commit into
aternosorg:masterfrom
Zoriot:shutdown-non-daemon-threads

Conversation

@Zoriot

@Zoriot Zoriot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

NightConfig's default FileWatcher keeps a non-daemon executor thread alive after the Minecraft client has finished shutting down. This prevents the JVM from terminating normally and eventually triggers Minecraft's client shutdown watchdog.

Add a common shutdown hook that stops the default FileWatcher and invoke it in all stop lifecycle events.

This ensures the configuration watcher is terminated before the client exits and prevents the shutdown watchdog crash.

NightConfig's default FileWatcher keeps a non-daemon executor thread alive
after the Minecraft client has finished shutting down. This prevents the JVM
from terminating normally and eventually triggers Minecraft's client shutdown
watchdog.

Add a common shutdown hook that stops the default FileWatcher and invoke it in all stop lifecycle events.

This ensures the configuration watcher is terminated before the client exits
and prevents the shutdown watchdog crash.
@CLAassistant

CLAassistant commented Aug 12, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@JulianVennen

Copy link
Copy Markdown
Member

Weird, this doesn't seem to be an issue in the dev environment for some reason, but I can reproduce it in prism.

@JulianVennen
JulianVennen merged commit 2a1cd2b into aternosorg:master Aug 13, 2026
2 checks passed
@Zoriot
Zoriot deleted the shutdown-non-daemon-threads branch August 13, 2026 23:02
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.

3 participants