Repository navigation
Add isolated Windows TEST1714 startup support - #19
Draft
valzargaming wants to merge 5 commits into
Draft
valzargaming wants to merge 5 commits into
valzargaming wants to merge 5 commits into
Conversation
This branch has not been deployed
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
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.
The stable bot bootstrap assumes Linux process commands and ambient production configuration, preventing an isolated Windows TEST1714 setup. Add an explicit portable profile checked before Composer/environment loading, separate configuration and state paths, shell-free owned PHP lifecycle/logging, loopback HTTP, and early rejection of unsupported native game lifecycle commands. Linux without the profile retains its existing entry path.
This is stable-branch work based on
603f0a0f3be754bc728f49764cdc5503bcf2fdfd; it does not migrate V5 or alter PR15/18. The example is disabled and has no destination IDs. Operator-provisioned credentials and reviewed channel isolation are required before activation.Validation reported by the implementation owner at
a4936fb693cd3fe8b02385c0b9175eeeb984d297: seven PHP syntax checks and the temporary-storage/FakeProcess suite pass. Tests cover exclusive state locking, stop markers/replay, argv construction, profile/path/port rejection and production GameServer lifecycle rejection before game/Discord dependencies. Windows/Linux PHP8.3/8.4 offline CI is included. Independent bootstrap/default-property isolation review and exact-head hosted CI are pending.Draft: no real bot startup, Discord connection, gateway/command registration, TEST1714 file IPC or process binding has been verified. No Debian service, game, credential/configuration, installation or live Discord action was performed.
docs/windows-test1714.mdrecords the activation boundary.Changelog: Add an isolated Windows startup profile and owned lifecycle tools for a separately configured TEST1714 bot.