refactor(server): unify first-boot prompts into the ConfigurePrompts phase (#2477)
Stacked on #2475 (the `ConfigurePrompts` phase). Base will switch to `main` once #2475 merges. ## What Move the engine's own first-boot prompts — data directories, listeners, server name, expansion + map selection — out of `ServerConfiguration.Load` and into **`ServerConfiguration.ConfigurePrompts()`** (`[CallPriority(0)]`), so **all** first-boot prompting (engine and content) runs through the single `AssemblyHandler.Invoke("ConfigurePrompts")` phase. `Load` now only reads/creates the config file. ## Why it's safe - **Assembly loading uses `AssemblyDirectories` (default `./Assemblies`), not `DataDirectories`** — so assemblies load fine before the now-later data-dir prompt. This is the linchpin that makes the move possible. - **`UOClient.Load()`** (client-file discovery via `Core.FindDataFile`) needs `DataDirectories`, so it moved *with* the data-dir prompt into `ConfigurePrompts`. - **`Core.Expansion`** is now assigned in `ConfigurePrompts` (every non-mocked boot). Nothing between `LoadAssemblies` and that phase reads it — type initializers run lazily on first use, not during `LoadAssemblies`. - **`[CallPriority(0)]`** keeps the engine prompts (including map selection) ahead of content prompts such as the pathfinding pre-bake (priority 50), preserving "after map selection". - `Main.cs` already invokes the phase — **no startup-ordering edit** here. ## Tests `Server.Tests` **708/708**, `UOContent.Tests` **418/418**, build clean. Fixtures are unaffected: they call `Load(true)` (now just reads config) and set expansion/data dirs directly; `ConfigurePrompts` is gated on `m_Mocked`. ## ⚠️ Needs first-boot runtime verification `Main.cs` startup ordering is **not** covered by the fixture-based suite (the fixtures bypass `Main`). Please boot once with a fresh `modernuo.json` to confirm the first-boot prompt sequence (data dirs → … → expansion/maps → pathfinding pre-bake) and that `Core.Expansion` resolves correctly. Docs updated in `dev-docs/server-lifecycle.md`.
This commit is contained in:
parent
16bf3016fb
commit
eec37edd67
2 changed files with 35 additions and 17 deletions
|
|
@ -218,11 +218,13 @@ public static class ServerConfiguration
|
|||
Save();
|
||||
}
|
||||
|
||||
// If mock is enabled we skip the console readline.
|
||||
// Reads (or creates) the configuration file. The interactive first-boot prompts live in
|
||||
// ConfigurePrompts (run later via AssemblyHandler.Invoke("ConfigurePrompts")) so all
|
||||
// first-boot prompting — engine and content — shares one phase/wiring. mocked skips prompts
|
||||
// entirely (ConfigurePrompts is gated on m_Mocked and isn't invoked by the test fixtures).
|
||||
public static void Load(bool mocked = false)
|
||||
{
|
||||
m_Mocked = mocked;
|
||||
var updated = false;
|
||||
|
||||
if (File.Exists(m_FilePath))
|
||||
{
|
||||
|
|
@ -239,15 +241,27 @@ public static class ServerConfiguration
|
|||
}
|
||||
else
|
||||
{
|
||||
updated = true;
|
||||
_settings = new ServerSettings();
|
||||
}
|
||||
}
|
||||
|
||||
if (mocked)
|
||||
// First-boot interactive configuration, discovered + run by AssemblyHandler.Invoke(
|
||||
// "ConfigurePrompts") after assemblies load but before Serilog's first log line, so the
|
||||
// console prompts are not interleaved with the async console sink (see
|
||||
// dev-docs/server-lifecycle.md). CallPriority(0) so the engine's own prompts (data dirs,
|
||||
// listeners, server name, expansion + map selection) run before any content ConfigurePrompts
|
||||
// that build on them (e.g. map selection before a pathfinding pre-bake prompt). Also resolves
|
||||
// Core.Expansion on every non-mocked boot.
|
||||
[CallPriority(0)]
|
||||
public static void ConfigurePrompts()
|
||||
{
|
||||
if (m_Mocked)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
var updated = false;
|
||||
|
||||
if (_settings.DataDirectories.Count == 0)
|
||||
{
|
||||
updated = true;
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue