Skip to content

feat(hyt): add Hytale config.json - #158

Merged
dgibbs64 merged 1 commit into
mainfrom
feat/hyt-config
Oct 4, 2026
Merged

dgibbs64 merged 1 commit into
mainfrom
feat/hyt-config

Conversation

@dgibbs64

@dgibbs64 dgibbs64 commented Oct 4, 2026

Copy link
Copy Markdown
Member

Description

Adds hyt/config.json for Hytale (hytserver, GameServerManagers/LinuxGSM#4928).

The LinuxGSM details-check (hyt) CI job downloads this file and currently gets a 404.

LinuxGSM doesn't copy this file on install, because the Hytale server generates Server/config.json on first start (the same approach as hz, unt and vints). This sample is used by CI and as a reference. It's the unmodified file generated by the server (config Version 4, Hytale server 0.6.8).

Testing

Copied from a fresh server's generated Server/config.json on Ubuntu 26.04 (Docker). It contains no credentials. The login is stored separately in auth.enc.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The configuration is valid JSON, contains no credentials, and fulfills the stated LinuxGSM integration requirement.

Review effort: Balanced
Findings: None

What changed in this PR

Adds the generated Hytale server configuration sample required by LinuxGSM CI and reference workflows.

Changes:

  • Adds Hytale config version 4 with default server, storage, authentication, and module settings.
  • Uses an empty password and external encrypted credential storage.
File Description
hyt/​config.json Adds the valid default Hytale server configuration sample.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@dgibbs64
dgibbs64 merged commit 9e11cf1 into main Oct 4, 2026
3 checks passed
@dgibbs64
dgibbs64 deleted the feat/hyt-config branch October 4, 2026 14:14
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.

2 participants