This repository was archived by the owner on Aug 8, 2026. It is now read-only.
︎︎⠀ - #106
Merged
Merged
Conversation
Force reload now routes through the existing dl_load:gate/* confirmation
system (same pattern as ban/kick/disable) instead of running
unconditionally — it discards live engine state, so it belongs in the
dangerous-action class the gate system already protects.
- dl_load:load/force: checks datalib:engine {sandbox:1b} and defers to
gate/request when active; runs directly when gates are disabled
- add datalib:core/internal/load/force_apply — the actual re-init logic,
extracted so gate/run_action can call it post-confirmation
- main.mcfunction: update stale "no confirmation gate" header comment
and reload-warning message to reflect the gated force path
- initial load (main -> load/all) stays ungated on purpose, to avoid
stalling server startup on a confirmation prompt
Removed dead code: datalib:config/load_cfg.mcfunction defined
load.timeout_seconds / load.sandbox_enabled scores and referenced a
dl_load:load/confirm function that never existed. Nothing called it and
nothing read the scores it set — an abandoned earlier attempt at a load
gate, superseded by the dl_load:gate/* system above.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
No description provided.