WIP

.run/Agent 1.run.xml

@@ -6,7 +6,7 @@
     <option name="PASS_PARENT_ENVS" value="1" />
     <envs>
       <env name="AGENT_KEY_FILE" value="./key" />
-      <env name="ALLOWED_ADDITIONAL_PORTS" value="25575,27000,27001" />
+      <env name="ALLOWED_RCON_PORTS" value="25575,27000,27001" />
       <env name="ALLOWED_SERVER_PORTS" value="25565,26000,26001" />
       <env name="CONTROLLER_HOST" value="localhost" />
       <env name="JAVA_SEARCH_PATH" value="~/.jdks" />
@@ -24,7 +24,7 @@
     <option name="PROJECT_ARGUMENTS_TRACKING" value="1" />
     <option name="PROJECT_WORKING_DIRECTORY_TRACKING" value="0" />
     <option name="PROJECT_KIND" value="DotNetCore" />
-    <option name="PROJECT_TFM" value="net10.0" />
+    <option name="PROJECT_TFM" value="net9.0" />
     <method v="2">
       <option name="Build" />
     </method>

.run/Agent 2.run.xml

@@ -6,7 +6,7 @@
     <option name="PASS_PARENT_ENVS" value="1" />
     <envs>
       <env name="AGENT_KEY_FILE" value="./key" />
-      <env name="ALLOWED_ADDITIONAL_PORTS" value="27002-27006" />
+      <env name="ALLOWED_RCON_PORTS" value="27002-27006" />
       <env name="ALLOWED_SERVER_PORTS" value="26002-26006" />
       <env name="CONTROLLER_HOST" value="localhost" />
       <env name="JAVA_SEARCH_PATH" value="~/.jdks" />
@@ -24,7 +24,7 @@
     <option name="PROJECT_ARGUMENTS_TRACKING" value="1" />
     <option name="PROJECT_WORKING_DIRECTORY_TRACKING" value="0" />
     <option name="PROJECT_KIND" value="DotNetCore" />
-    <option name="PROJECT_TFM" value="net10.0" />
+    <option name="PROJECT_TFM" value="net9.0" />
     <method v="2">
       <option name="Build" />
     </method>

.run/Agent 3.run.xml

@@ -6,7 +6,7 @@
     <option name="PASS_PARENT_ENVS" value="1" />
     <envs>
       <env name="AGENT_KEY_FILE" value="./key" />
-      <env name="ALLOWED_ADDITIONAL_PORTS" value="27007" />
+      <env name="ALLOWED_RCON_PORTS" value="27007" />
       <env name="ALLOWED_SERVER_PORTS" value="26007" />
       <env name="CONTROLLER_HOST" value="localhost" />
       <env name="JAVA_SEARCH_PATH" value="~/.jdks" />
@@ -24,7 +24,7 @@
     <option name="PROJECT_ARGUMENTS_TRACKING" value="1" />
     <option name="PROJECT_WORKING_DIRECTORY_TRACKING" value="0" />
     <option name="PROJECT_KIND" value="DotNetCore" />
-    <option name="PROJECT_TFM" value="net10.0" />
+    <option name="PROJECT_TFM" value="net9.0" />
     <method v="2">
       <option name="Build" />
     </method>

Agent/Phantom.Agent.Minecraft.Tests/Java/JavaPropertiesStreamTests.cs

@@ -1,9 +1,9 @@
 using System.Collections.Immutable;
 using NUnit.Framework;
-using Phantom.Agent.Services.Java;
+using Phantom.Agent.Minecraft.Java;
 using Phantom.Utils.Collections;
 
-namespace Phantom.Agent.Services.Tests.Java;
+namespace Phantom.Agent.Minecraft.Tests.Java;
 
 [TestFixture]
 public sealed class JavaPropertiesStreamTests {

Agent/Phantom.Agent.Minecraft.Tests/Phantom.Agent.Minecraft.Tests.csproj

@@ -17,7 +17,7 @@
   </ItemGroup>
   
   <ItemGroup>
-    <ProjectReference Include="..\Phantom.Agent.Services\Phantom.Agent.Services.csproj" />
+    <ProjectReference Include="..\Phantom.Agent.Minecraft\Phantom.Agent.Minecraft.csproj" />
   </ItemGroup>
 
 </Project>

Agent/Phantom.Agent.Minecraft/Command/MinecraftCommand.cs

@@ -0,0 +1,15 @@
+namespace Phantom.Agent.Minecraft.Command;
+
+public static class MinecraftCommand {
+	public const string SaveOn = "save-on";
+	public const string SaveOff = "save-off";
+	public const string Stop = "stop";
+	
+	public static string Say(string message) {
+		return "say " + message;
+	}
+	
+	public static string SaveAll(bool flush) {
+		return flush ? "save-all flush" : "save-all";
+	}
+}

Agent/Phantom.Agent.Minecraft/Instance/InstanceProcess.cs

@@ -2,9 +2,9 @@
 using Phantom.Utils.Processes;
 using Phantom.Utils.Tasks;
 
-namespace Phantom.Agent.Services.Instances.State;
+namespace Phantom.Agent.Minecraft.Instance;
 
-sealed class InstanceProcess : IDisposable {
+public sealed class InstanceProcess : IDisposable {
 	public InstanceProperties InstanceProperties { get; }
 	
 	private readonly RingBuffer<string> outputBuffer = new (100);
@@ -27,15 +27,6 @@ sealed class InstanceProcess : IDisposable {
 		await process.StandardInput.WriteLineAsync(command.AsMemory(), cancellationToken);
 	}
 	
-	public async Task<bool> TrySendCommand(string command, TimeSpan timeout, CancellationToken cancellationToken) {
-		try {
-			await SendCommand(command, cancellationToken).WaitAsync(timeout, cancellationToken);
-			return true;
-		} catch (TimeoutException) {
-			return false;
-		}
-	}
-	
 	public void AddOutputListener(EventHandler<string> listener, uint maxLinesToReadFromHistory = uint.MaxValue) {
 		OutputEvent += listener;
 		

Agent/Phantom.Agent.Minecraft/Instance/InstanceProperties.cs

@@ -0,0 +1,6 @@
+namespace Phantom.Agent.Minecraft.Instance;
+
+public sealed record InstanceProperties(
+	Guid InstanceGuid,
+	string InstanceFolder
+);

Agent/Phantom.Agent.Minecraft/Java/JavaPropertiesFileEditor.cs

@@ -1,4 +1,4 @@
-namespace Phantom.Agent.Services.Java;
+namespace Phantom.Agent.Minecraft.Java;
 
 sealed class JavaPropertiesFileEditor {
 	private readonly Dictionary<string, string> overriddenProperties = new ();

Agent/Phantom.Agent.Minecraft/Java/JavaPropertiesStream.cs

@@ -4,7 +4,7 @@ using System.Runtime.CompilerServices;
 using System.Text;
 using Phantom.Utils.Collections;
 
-namespace Phantom.Agent.Services.Java;
+namespace Phantom.Agent.Minecraft.Java;
 
 static class JavaPropertiesStream {
 	internal static readonly Encoding Encoding = Encoding.GetEncoding("ISO-8859-1");
@@ -30,7 +30,7 @@ static class JavaPropertiesStream {
 		}
 		
 		public Reader(string path) {
-			this.reader = new StreamReader(path, Encoding, detectEncodingFromByteOrderMarks: true, CreateFileStreamOptions(FileMode.Open, FileAccess.Read));
+			this.reader = new StreamReader(path, Encoding, detectEncodingFromByteOrderMarks: false, CreateFileStreamOptions(FileMode.Open, FileAccess.Read));
 		}
 		
 		public async IAsyncEnumerable<KeyValuePair<string, string>> ReadProperties([EnumeratorCancellation] CancellationToken cancellationToken) {

Agent/Phantom.Agent.Minecraft/Java/JavaRuntimeDiscovery.cs

@@ -6,7 +6,7 @@ using Phantom.Utils.IO;
 using Phantom.Utils.Logging;
 using Serilog;
 
-namespace Phantom.Agent.Services.Java;
+namespace Phantom.Agent.Minecraft.Java;
 
 public sealed class JavaRuntimeDiscovery {
 	private static readonly ILogger Logger = PhantomLogger.Create(nameof(JavaRuntimeDiscovery));

Agent/Phantom.Agent.Minecraft/Java/JavaRuntimeExecutable.cs

@@ -0,0 +1,5 @@
+using Phantom.Common.Data.Java;
+
+namespace Phantom.Agent.Minecraft.Java;
+
+public sealed record JavaRuntimeExecutable(string ExecutablePath, JavaRuntime Runtime);

Agent/Phantom.Agent.Minecraft/Java/JavaRuntimeRepository.cs

@@ -3,7 +3,7 @@ using System.Diagnostics.CodeAnalysis;
 using Phantom.Common.Data.Java;
 using Phantom.Utils.Cryptography;
 
-namespace Phantom.Agent.Services.Java;
+namespace Phantom.Agent.Minecraft.Java;
 
 public sealed class JavaRuntimeRepository {
 	private readonly ImmutableDictionary<Guid, JavaRuntimeExecutable> runtimesByGuid;
@@ -25,7 +25,7 @@ public sealed class JavaRuntimeRepository {
 		                     .ToImmutableArray();
 	}
 	
-	internal bool TryGetByGuid(Guid guid, [MaybeNullWhen(false)] out JavaRuntimeExecutable runtime) {
+	public bool TryGetByGuid(Guid guid, [MaybeNullWhen(false)] out JavaRuntimeExecutable runtime) {
 		return runtimesByGuid.TryGetValue(guid, out runtime);
 	}
 	

Agent/Phantom.Agent.Minecraft/Java/JvmArgumentBuilder.cs

@@ -0,0 +1,25 @@
+using System.Collections.ObjectModel;
+
+namespace Phantom.Agent.Minecraft.Java;
+
+sealed class JvmArgumentBuilder(JvmProperties basicProperties) {
+	private readonly List<string> customArguments = [];
+	
+	public void Add(string argument) {
+		customArguments.Add(argument);
+	}
+	
+	public void AddProperty(string key, string value) {
+		customArguments.Add("-D" + key + "=\"" + value + "\""); // TODO test quoting?
+	}
+	
+	public void Build(Collection<string> target) {
+		foreach (var property in customArguments) {
+			target.Add(property);
+		}
+		
+		// In case of duplicate JVM arguments, typically the last one wins.
+		target.Add("-Xms" + basicProperties.InitialHeapMegabytes + "M");
+		target.Add("-Xmx" + basicProperties.MaximumHeapMegabytes + "M");
+	}
+}

Agent/Phantom.Agent.Minecraft/Java/JvmProperties.cs

@@ -0,0 +1,6 @@
+namespace Phantom.Agent.Minecraft.Java;
+
+public sealed record JvmProperties(
+	uint InitialHeapMegabytes,
+	uint MaximumHeapMegabytes
+);

Agent/Phantom.Agent.Minecraft/Launcher/InstanceLauncher.cs

@@ -0,0 +1,115 @@
+using System.Collections.Immutable;
+using Phantom.Agent.Minecraft.Instance;
+using Phantom.Agent.Minecraft.Java;
+using Phantom.Agent.Minecraft.Server;
+using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Instance.Launch;
+using Phantom.Common.Data.Minecraft;
+using Phantom.Utils.Processes;
+using Serilog;
+
+namespace Phantom.Agent.Minecraft.Launcher;
+
+public sealed class InstanceLauncher(
+	FileDownloadManager downloadManager,
+	IInstancePathResolver pathResolver,
+	IInstanceValueResolver valueResolver,
+	InstanceProperties instanceProperties,
+	InstanceLaunchRecipe launchRecipe
+) {
+	public async Task<LaunchResult> Launch(ILogger logger, Action<IInstanceStatus?> reportStatus, CancellationToken cancellationToken) {
+		string? executablePath = launchRecipe.Executable.Resolve(pathResolver);
+		if (executablePath == null) {
+			logger.Error("Could not resolve server executable: {Path}", launchRecipe.Executable);
+			return new LaunchResult.CouldNotFindServerExecutable();
+		}
+		
+		var stepVisitor = new StepExecutor(downloadManager, pathResolver, reportStatus, cancellationToken);
+		var steps = launchRecipe.Preparation;
+		
+		for (int stepIndex = 0; stepIndex < steps.Length; stepIndex++) {
+			var step = steps[stepIndex];
+			try {
+				await step.Run(stepVisitor);
+			} catch (Exception e) {
+				logger.Error(e, "Failed preparation step {StepIndex} out of {StepCount}: {Step}", stepIndex, steps.Length, step);
+				return new LaunchResult.CouldNotPrepareServerInstance();
+			}
+		}
+		
+		var processConfigurator = new ProcessConfigurator {
+			FileName = executablePath,
+			WorkingDirectory = instanceProperties.InstanceFolder,
+			RedirectInput = true,
+			UseShellExecute = false,
+		};
+		
+		var processArguments = processConfigurator.ArgumentList;
+		
+		foreach (var value in launchRecipe.Arguments) {
+			if (value.Resolve(valueResolver) is {} resolved) {
+				processArguments.Add(resolved);
+			}
+			else {
+				logger.Error("Could not resolve server executable argument: {Value}", value);
+				return new LaunchResult.CouldNotPrepareServerInstance();
+			}
+		}
+		
+		var process = processConfigurator.CreateProcess();
+		var instanceProcess = new InstanceProcess(instanceProperties, process);
+		
+		try {
+			process.Start();
+		} catch (Exception launchException) {
+			logger.Error(launchException, "Caught exception launching the server process.");
+			
+			try {
+				process.Kill();
+			} catch (Exception killException) {
+				logger.Error(killException, "Caught exception trying to kill the server process after a failed launch.");
+			}
+			
+			return new LaunchResult.CouldNotStartServerExecutable();
+		}
+		
+		return new LaunchResult.Success(instanceProcess);
+	}
+	
+	private sealed class StepExecutor(FileDownloadManager downloadManager, IInstancePathResolver pathResolver, Action<IInstanceStatus?> reportStatus, CancellationToken cancellationToken) : IInstanceLaunchStepExecutor {
+		public async Task DownloadFile(FileDownloadInfo downloadInfo, IInstancePath path) {
+			string? filePath = path.Resolve(pathResolver);
+			if (filePath == null) {
+				throw new FileNotFoundException("Could not resolve path");
+			}
+			
+			byte? lastDownloadProgress = null;
+			
+			void OnDownloadProgress(object? sender, DownloadProgressEventArgs args) {
+				byte? progress = args.TotalBytes is not {} totalBytes ? null : (byte) Math.Min(args.DownloadedBytes * 100 / totalBytes, val2: 100);
+				
+				if (lastDownloadProgress != progress) {
+					lastDownloadProgress = progress;
+					reportStatus(InstanceStatus.Downloading(progress));
+				}
+			}
+			
+			if (await downloadManager.DownloadAndGetPath(downloadInfo, filePath, OnDownloadProgress, cancellationToken) == null) {
+				throw new FileNotFoundException("Could not download file");
+			}
+			
+			reportStatus(null);
+		}
+		
+		public async Task EditPropertiesFile(InstancePath.Local path, string comment, ImmutableDictionary<string, string> newValues) {
+			string? filePath = path.Resolve(pathResolver);
+			if (filePath == null) {
+				throw new FileNotFoundException("Could not resolve path");
+			}
+			
+			var editor = new JavaPropertiesFileEditor();
+			editor.SetAll(newValues);
+			await editor.EditOrCreate(filePath, comment, cancellationToken);
+		}
+	}
+}

Agent/Phantom.Agent.Minecraft/Launcher/LaunchResult.cs

@@ -0,0 +1,15 @@
+using Phantom.Agent.Minecraft.Instance;
+
+namespace Phantom.Agent.Minecraft.Launcher;
+
+public abstract record LaunchResult {
+	private LaunchResult() {}
+	
+	public sealed record Success(InstanceProcess Process) : LaunchResult;
+	
+	public sealed record CouldNotPrepareServerInstance : LaunchResult;
+	
+	public sealed record CouldNotFindServerExecutable : LaunchResult;
+	
+	public sealed record CouldNotStartServerExecutable : LaunchResult;
+}

Agent/Phantom.Agent.Minecraft/Launcher/ServerJarInfo.cs

@@ -0,0 +1,7 @@
+using System.Collections.Immutable;
+
+namespace Phantom.Agent.Minecraft.Launcher;
+
+sealed record ServerJarInfo(string FilePath, ImmutableArray<string> ExtraArgs) {
+	public ServerJarInfo(string filePath) : this(filePath, ImmutableArray<string>.Empty) {}
+}

Agent/Phantom.Agent.Minecraft/Phantom.Agent.Minecraft.csproj

@@ -0,0 +1,18 @@
+<Project Sdk="Microsoft.NET.Sdk">
+  
+  <PropertyGroup>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <Nullable>enable</Nullable>
+  </PropertyGroup>
+  
+  <ItemGroup>
+    <InternalsVisibleTo Include="Phantom.Agent.Minecraft.Tests" />
+  </ItemGroup>
+  
+  <ItemGroup>
+    <ProjectReference Include="..\..\Common\Phantom.Common.Data\Phantom.Common.Data.csproj" />
+    <ProjectReference Include="..\..\Utils\Phantom.Utils\Phantom.Utils.csproj" />
+    <ProjectReference Include="..\..\Utils\Phantom.Utils.Logging\Phantom.Utils.Logging.csproj" />
+  </ItemGroup>
+
+</Project>

Agent/Phantom.Agent.Minecraft/Server/DownloadProgressEventArgs.cs

@@ -1,6 +1,6 @@
-namespace Phantom.Agent.Services.Downloads;
+namespace Phantom.Agent.Minecraft.Server;
 
-sealed class DownloadProgressEventArgs : EventArgs {
+public sealed class DownloadProgressEventArgs : EventArgs {
 	public ulong DownloadedBytes { get; }
 	public ulong? TotalBytes { get; }
 	

Agent/Phantom.Agent.Minecraft/Server/FileDownloadListener.cs

@@ -1,3 +1,3 @@
-namespace Phantom.Agent.Services.Downloads;
+namespace Phantom.Agent.Minecraft.Server;
 
 sealed record FileDownloadListener(EventHandler<DownloadProgressEventArgs> DownloadProgressEventHandler, CancellationToken CancellationToken);

Agent/Phantom.Agent.Minecraft/Server/FileDownloadManager.cs

@@ -1,16 +1,16 @@
-using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Utils.IO;
 using Phantom.Utils.Logging;
 using Serilog;
 
-namespace Phantom.Agent.Services.Downloads;
+namespace Phantom.Agent.Minecraft.Server;
 
-sealed class FileDownloadManager {
+public sealed class FileDownloadManager {
 	private static readonly ILogger Logger = PhantomLogger.Create<FileDownloadManager>();
 	
 	private readonly Dictionary<string, FileDownloader> runningDownloadersByPath = new ();
 	
-	public async Task<string?> DownloadAndGetPath(FileDownloadInfo fileDownloadInfo, string filePath, EventHandler<DownloadProgressEventArgs> progressEventHandler, CancellationToken cancellationToken) {
+	internal async Task<string?> DownloadAndGetPath(FileDownloadInfo fileDownloadInfo, string filePath, EventHandler<DownloadProgressEventArgs> progressEventHandler, CancellationToken cancellationToken) {
 		var fileInfo = new FileInfo(filePath);
 		if (fileInfo.Exists) {
 			return filePath;

Agent/Phantom.Agent.Minecraft/Server/FileDownloader.cs

@@ -1,12 +1,13 @@
 using System.Security.Cryptography;
-using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Utils.Cryptography;
 using Phantom.Utils.IO;
 using Phantom.Utils.Logging;
+using Phantom.Utils.Net;
 using Phantom.Utils.Runtime;
 using Serilog;
 
-namespace Phantom.Agent.Services.Downloads;
+namespace Phantom.Agent.Minecraft.Server;
 
 sealed class FileDownloader {
 	private static readonly ILogger Logger = PhantomLogger.Create<FileDownloader>();
@@ -102,7 +103,7 @@ sealed class FileDownloader {
 	}
 	
 	private static async Task DownloadFile(string filePath, FileDownloadInfo fileDownloadInfo, DownloadProgressCallback progressCallback, CancellationToken cancellationToken) {
-		string downloadUrl = fileDownloadInfo.Url;
+		string downloadUrl = fileDownloadInfo.DownloadUrl;
 		
 		DownloadResult result;
 		try {
@@ -110,7 +111,7 @@ sealed class FileDownloader {
 			var response = await httpClient.GetAsync(downloadUrl, HttpCompletionOption.ResponseHeadersRead, cancellationToken);
 			response.EnsureSuccessStatusCode();
 			
-			FileSize? fileSize = response.Content.Headers.ContentLength is {} length and >= 0 ? new FileSize((ulong) length) : null;
+			FileSize? fileSize = response.Headers.ContentLength;
 			ulong? fileSizeBytes = fileSize?.Bytes;
 			
 			Logger.Information("Downloading {Url} ({Size})...", downloadUrl, FormatFileSize(fileSize));

Agent/Phantom.Agent.Minecraft/Server/ServerStatusProtocol.cs

@@ -5,9 +5,9 @@ using System.Net.Sockets;
 using System.Text;
 using Phantom.Common.Data.Instance;
 
-namespace Phantom.Agent.Services.Games;
+namespace Phantom.Agent.Minecraft.Server;
 
-static class MinecraftServerStatusProtocol {
+public static class ServerStatusProtocol {
 	public static async Task<InstancePlayerCounts> GetPlayerCounts(ushort serverPort, CancellationToken cancellationToken) {
 		using var tcpClient = new TcpClient();
 		await tcpClient.ConnectAsync(IPAddress.Loopback, serverPort, cancellationToken);

Agent/Phantom.Agent.Services/AgentRegistrationHandler.cs

@@ -79,7 +79,6 @@ public sealed class AgentRegistrationHandler {
 			configureInstanceMessage.Info,
 			configureInstanceMessage.LaunchRecipe,
 			configureInstanceMessage.LaunchNow,
-			configureInstanceMessage.StopRecipe,
 			AlwaysReportStatus: true
 		);
 	}

Agent/Phantom.Agent.Services/AgentServices.cs

@@ -1,7 +1,7 @@
 using Akka.Actor;
+using Phantom.Agent.Minecraft.Java;
 using Phantom.Agent.Services.Backups;
 using Phantom.Agent.Services.Instances;
-using Phantom.Agent.Services.Java;
 using Phantom.Agent.Services.Rpc;
 using Phantom.Common.Data.Agent;
 using Phantom.Utils.Actor;

Agent/Phantom.Agent.Services/Backups/BackupArchiver.cs

@@ -1,7 +1,7 @@
 using System.Collections.Immutable;
 using System.Diagnostics.CodeAnalysis;
 using System.Formats.Tar;
-using Phantom.Agent.Services.Instances;
+using Phantom.Agent.Minecraft.Instance;
 using Phantom.Common.Data.Backups;
 using Phantom.Utils.IO;
 using Phantom.Utils.Logging;

Agent/Phantom.Agent.Services/Backups/BackupManager.cs

@@ -1,4 +1,4 @@
-using Phantom.Agent.Services.Instances.State;
+using Phantom.Agent.Minecraft.Instance;
 using Phantom.Common.Data.Backups;
 using Phantom.Utils.Logging;
 using Serilog;

Agent/Phantom.Agent.Services/Backups/BackupServerCommandDispatcher.cs

@@ -1,7 +1,7 @@
 using System.Collections.Immutable;
 using System.Text.RegularExpressions;
-using Phantom.Agent.Services.Games;
+using Phantom.Agent.Minecraft.Command;
-using Phantom.Agent.Services.Instances.State;
+using Phantom.Agent.Minecraft.Instance;
 using Phantom.Utils.Tasks;
 using Serilog;
 

Agent/Phantom.Agent.Services/Games/MinecraftCommand.cs

@@ -1,10 +0,0 @@
-namespace Phantom.Agent.Services.Games;
-
-static class MinecraftCommand {
-	public const string SaveOn = "save-on";
-	public const string SaveOff = "save-off";
-	
-	public static string SaveAll(bool flush) {
-		return flush ? "save-all flush" : "save-all";
-	}
-}

Agent/Phantom.Agent.Services/Instances/InstanceActor.cs

@@ -1,11 +1,10 @@
-using Phantom.Agent.Services.Backups;
+using Phantom.Agent.Minecraft.Launcher;
-using Phantom.Agent.Services.Instances.Launch;
+using Phantom.Agent.Services.Backups;
 using Phantom.Agent.Services.Instances.State;
 using Phantom.Agent.Services.Rpc;
-using Phantom.Common.Data.Agent.Instance;
-using Phantom.Common.Data.Agent.Instance.Stop;
 using Phantom.Common.Data.Backups;
 using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Messages.Agent.ToController;
 using Phantom.Utils.Actor;
@@ -85,7 +84,7 @@ sealed class InstanceActor : ReceiveActor<InstanceActor.ICommand> {
 	
 	public sealed record LaunchInstanceCommand(InstanceInfo Info, InstanceLauncher Launcher, InstanceTicketManager.Ticket Ticket, bool IsRestarting) : ICommand;
 	
-	public sealed record StopInstanceCommand(InstanceStopRecipe StopRecipe) : ICommand;
+	public sealed record StopInstanceCommand(MinecraftStopStrategy StopStrategy) : ICommand;
 	
 	public sealed record SendCommandToInstanceCommand(string Command) : ICommand, ICanReply<SendCommandToInstanceResult>;
 	
@@ -93,7 +92,7 @@ sealed class InstanceActor : ReceiveActor<InstanceActor.ICommand> {
 	
 	public sealed record HandleProcessEndedCommand(IInstanceStatus Status) : ICommand, IJumpAhead;
 	
-	public sealed record ShutdownCommand(InstanceStopRecipe StopRecipe) : ICommand;
+	public sealed record ShutdownCommand : ICommand;
 	
 	private void ReportInstanceStatus(ReportInstanceStatusCommand command) {
 		ReportCurrentStatus();
@@ -125,7 +124,7 @@ sealed class InstanceActor : ReceiveActor<InstanceActor.ICommand> {
 		IInstanceStatus oldStatus = currentStatus;
 		SetAndReportStatus(InstanceStatus.Stopping);
 		
-		if (await InstanceStopProcedure.Run(context, command.StopRecipe, runningState, SetAndReportStatus, shutdownCancellationToken)) {
+		if (await InstanceStopProcedure.Run(context, command.StopStrategy, runningState, SetAndReportStatus, shutdownCancellationToken)) {
 			instanceTicketManager.Release(runningState.Ticket);
 			TransitionState(null);
 		}
@@ -167,7 +166,7 @@ sealed class InstanceActor : ReceiveActor<InstanceActor.ICommand> {
 	}
 	
 	private async Task Shutdown(ShutdownCommand command) {
-		await StopInstance(new StopInstanceCommand(command.StopRecipe));
+		await StopInstance(new StopInstanceCommand(MinecraftStopStrategy.Instant));
 		await actorCancellationTokenSource.CancelAsync();
 		
 		await Task.WhenAll(

Agent/Phantom.Agent.Services/Instances/InstanceManagerActor.cs

@@ -1,13 +1,13 @@
-using Phantom.Agent.Services.Backups;
+using Phantom.Agent.Minecraft.Instance;
-using Phantom.Agent.Services.Downloads;
+using Phantom.Agent.Minecraft.Java;
-using Phantom.Agent.Services.Instances.Launch;
+using Phantom.Agent.Minecraft.Launcher;
-using Phantom.Agent.Services.Java;
+using Phantom.Agent.Minecraft.Server;
+using Phantom.Agent.Services.Backups;
 using Phantom.Agent.Services.Rpc;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Agent.Instance;
-using Phantom.Common.Data.Agent.Instance.Launch;
-using Phantom.Common.Data.Agent.Instance.Stop;
 using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Instance.Launch;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Utils.Actor;
 using Phantom.Utils.IO;
@@ -52,15 +52,15 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 		ReceiveAsync<ShutdownCommand>(Shutdown);
 	}
 	
-	private sealed record Instance(ActorRef<InstanceActor.ICommand> Actor, InstanceInfo Info, InstanceProperties Properties, InstanceLaunchRecipe? LaunchRecipe, InstanceStopRecipe StopRecipe);
+	private sealed record Instance(ActorRef<InstanceActor.ICommand> Actor, InstanceInfo Info, InstanceProperties Properties, InstanceLaunchRecipe? LaunchRecipe);
 	
 	public interface ICommand;
 	
-	public sealed record ConfigureInstanceCommand(Guid InstanceGuid, InstanceInfo InstanceInfo, InstanceLaunchRecipe? LaunchRecipe, bool LaunchNow, InstanceStopRecipe StopRecipe, bool AlwaysReportStatus) : ICommand, ICanReply<Result<ConfigureInstanceResult, InstanceActionFailure>>;
+	public sealed record ConfigureInstanceCommand(Guid InstanceGuid, InstanceInfo Info, InstanceLaunchRecipe? LaunchRecipe, bool LaunchNow, bool AlwaysReportStatus) : ICommand, ICanReply<Result<ConfigureInstanceResult, InstanceActionFailure>>;
 	
 	public sealed record LaunchInstanceCommand(Guid InstanceGuid) : ICommand, ICanReply<Result<LaunchInstanceResult, InstanceActionFailure>>;
 	
-	public sealed record StopInstanceCommand(Guid InstanceGuid, InstanceStopRecipe StopRecipe) : ICommand, ICanReply<Result<StopInstanceResult, InstanceActionFailure>>;
+	public sealed record StopInstanceCommand(Guid InstanceGuid, MinecraftStopStrategy StopStrategy) : ICommand, ICanReply<Result<StopInstanceResult, InstanceActionFailure>>;
 	
 	public sealed record SendCommandToInstanceCommand(Guid InstanceGuid, string Command) : ICommand, ICanReply<Result<SendCommandToInstanceResult, InstanceActionFailure>>;
 	
@@ -68,15 +68,13 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 	
 	private Result<ConfigureInstanceResult, InstanceActionFailure> ConfigureInstance(ConfigureInstanceCommand command) {
 		var instanceGuid = command.InstanceGuid;
-		var instanceInfo = command.InstanceInfo;
+		var instanceInfo = command.Info;
 		var launchRecipe = command.LaunchRecipe;
-		var stopRecipe = command.StopRecipe;
 		
 		if (instances.TryGetValue(instanceGuid, out var instance)) {
 			instances[instanceGuid] = instance with {
 				Info = instanceInfo,
 				LaunchRecipe = launchRecipe,
-				StopRecipe = stopRecipe,
 			};
 			
 			Logger.Information("Reconfigured instance \"{Name}\" (GUID {Guid}).", instanceInfo.InstanceName, instanceGuid);
@@ -87,20 +85,20 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 		}
 		else {
 			var instanceLoggerName = PhantomLogger.ShortenGuid(instanceGuid) + "/" + Interlocked.Increment(ref instanceLoggerSequenceId);
-			var instanceFolder = Path.Combine(agentFolders.InstancesFolderPath, instanceGuid.ToString());
+			var instanceProperties = new InstanceProperties(instanceGuid, Path.Combine(agentFolders.InstancesFolderPath, instanceGuid.ToString()));
-			var instanceProperties = new InstanceProperties(instanceGuid, instanceFolder);
 			var instanceInit = new InstanceActor.Init(agentState, instanceGuid, instanceLoggerName, instanceServices, instanceTicketManager, shutdownCancellationToken);
-			instances[instanceGuid] = instance = new Instance(Context.ActorOf(InstanceActor.Factory(instanceInit), "Instance-" + instanceGuid), instanceInfo, instanceProperties, launchRecipe, stopRecipe);
+			instances[instanceGuid] = instance = new Instance(Context.ActorOf(InstanceActor.Factory(instanceInit), "Instance-" + instanceGuid), instanceInfo, instanceProperties, launchRecipe);
 			
 			Logger.Information("Created instance \"{Name}\" (GUID {Guid}).", instanceInfo.InstanceName, instanceGuid);
 			
 			instance.Actor.Tell(new InstanceActor.ReportInstanceStatusCommand());
 		}
 		
+		string instanceFolder = instance.Properties.InstanceFolder;
 		try {
-			Directories.Create(instance.Properties.InstanceFolder, Chmod.URWX_GRX);
+			Directories.Create(instanceFolder, Chmod.URWX_GRX);
 		} catch (Exception e) {
-			Logger.Error(e, "Could not create instance folder: {Path}", instance.Properties.InstanceFolder);
+			Logger.Error(e, "Could not create instance folder: {Path}", instanceFolder);
 			return ConfigureInstanceResult.CouldNotCreateInstanceFolder;
 		}
 		
@@ -161,7 +159,7 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 			}
 		}
 		
-		instanceInfo.Actor.Tell(new InstanceActor.StopInstanceCommand(command.StopRecipe));
+		instanceInfo.Actor.Tell(new InstanceActor.StopInstanceCommand(command.StopStrategy));
 		return StopInstanceResult.StopInitiated;
 	}
 	
@@ -183,7 +181,7 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 		
 		await shutdownCancellationTokenSource.CancelAsync();
 		
-		await Task.WhenAll(instances.Values.Select(static instance => instance.Actor.Stop(new InstanceActor.ShutdownCommand(instance.StopRecipe))));
+		await Task.WhenAll(instances.Values.Select(static instance => instance.Actor.Stop(new InstanceActor.ShutdownCommand())));
 		instances.Clear();
 		
 		shutdownCancellationTokenSource.Dispose();

Agent/Phantom.Agent.Services/Instances/InstancePathResolver.cs

@@ -0,0 +1,30 @@
+using System.Collections.Immutable;
+using Phantom.Agent.Minecraft.Instance;
+using Phantom.Agent.Minecraft.Java;
+using Phantom.Common.Data.Instance;
+
+namespace Phantom.Agent.Services.Instances;
+
+sealed class InstancePathResolver(AgentFolders agentFolders, JavaRuntimeRepository javaRuntimeRepository, InstanceProperties instanceProperties) : IInstancePathResolver {
+	public string Global(ImmutableArray<string> path) {
+		return CombinePath(agentFolders.ServerExecutableFolderPath, path);
+	}
+	
+	public string Local(ImmutableArray<string> path) {
+		return CombinePath(instanceProperties.InstanceFolder, path);
+	}
+	
+	public string? Runtime(Guid guid) {
+		if (javaRuntimeRepository.TryGetByGuid(guid, out var runtime)) {
+			return runtime.ExecutablePath;
+		}
+		else {
+			return null;
+		}
+	}
+	
+	private string CombinePath(string basePath, ImmutableArray<string> additionalParts) {
+		// TODO validation
+		return Path.Combine([basePath, ..additionalParts]);
+	}
+}

Agent/Phantom.Agent.Services/Instances/InstanceProperties.cs

@@ -1,6 +0,0 @@
-namespace Phantom.Agent.Services.Instances;
-
-sealed record InstanceProperties(
-	Guid InstanceGuid,
-	string InstanceFolder
-);

Agent/Phantom.Agent.Services/Instances/InstanceServices.cs

@@ -1,6 +1,6 @@
-using Phantom.Agent.Services.Backups;
+using Phantom.Agent.Minecraft.Java;
-using Phantom.Agent.Services.Downloads;
+using Phantom.Agent.Minecraft.Server;
-using Phantom.Agent.Services.Java;
+using Phantom.Agent.Services.Backups;
 using Phantom.Agent.Services.Rpc;
 
 namespace Phantom.Agent.Services.Instances;

Agent/Phantom.Agent.Services/Instances/InstanceTicketManager.cs

@@ -2,7 +2,7 @@
 using Phantom.Agent.Services.Rpc;
 using Phantom.Common.Data;
 using Phantom.Common.Data.Agent;
-using Phantom.Common.Data.Agent.Instance;
+using Phantom.Common.Data.Instance;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Messages.Agent.ToController;
 using Phantom.Utils.Logging;

Agent/Phantom.Agent.Services/Instances/InstanceValueResolver.cs

@@ -1,9 +1,13 @@
-using Phantom.Common.Data.Agent.Instance;
+using Phantom.Common.Data.Instance;
 
-namespace Phantom.Agent.Services.Instances.Launch;
+namespace Phantom.Agent.Services.Instances;
 
 sealed class InstanceValueResolver(IInstancePathResolver pathResolver) : IInstanceValueResolver {
 	public string? Path(IInstancePath value) {
 		return value.Resolve(pathResolver);
 	}
+	
+	public string? Variable(InstanceVariable value) {
+		return null;
+	}
 }

Agent/Phantom.Agent.Services/Instances/Launch/InstanceLaunchResult.cs

@@ -1,15 +0,0 @@
-using Phantom.Agent.Services.Instances.State;
-
-namespace Phantom.Agent.Services.Instances.Launch;
-
-abstract record InstanceLaunchResult {
-	private InstanceLaunchResult() {}
-	
-	public sealed record Success(InstanceProcess Process) : InstanceLaunchResult;
-	
-	public sealed record CouldNotPrepareServerInstance : InstanceLaunchResult;
-	
-	public sealed record CouldNotFindServerExecutable : InstanceLaunchResult;
-	
-	public sealed record CouldNotStartServerExecutable : InstanceLaunchResult;
-}

Agent/Phantom.Agent.Services/Instances/Launch/InstanceLauncher.cs

@@ -1,145 +0,0 @@
-using System.Collections.Immutable;
-using Phantom.Agent.Services.Downloads;
-using Phantom.Agent.Services.Instances.State;
-using Phantom.Agent.Services.Java;
-using Phantom.Common.Data.Agent;
-using Phantom.Common.Data.Agent.Instance;
-using Phantom.Common.Data.Agent.Instance.Launch;
-using Phantom.Common.Data.Instance;
-using Phantom.Utils.Processes;
-using Serilog;
-
-namespace Phantom.Agent.Services.Instances.Launch;
-
-sealed class InstanceLauncher(
-	FileDownloadManager downloadManager,
-	IInstancePathResolver pathResolver,
-	IInstanceValueResolver valueResolver,
-	InstanceProperties instanceProperties,
-	InstanceLaunchRecipe launchRecipe
-) {
-	public IInstanceValueResolver ValueResolver => valueResolver;
-	
-	public async Task<InstanceLaunchResult> Launch(ILogger logger, Action<IInstanceStatus?> reportStatus, CancellationToken cancellationToken) {
-		string? executablePath = launchRecipe.Executable.Resolve(pathResolver);
-		if (executablePath == null) {
-			logger.Error("Could not resolve server executable path: {Path}", launchRecipe.Executable);
-			return new InstanceLaunchResult.CouldNotFindServerExecutable();
-		}
-		
-		var stepExecutor = new StepExecutor(logger, downloadManager, pathResolver, reportStatus, cancellationToken);
-		if (!await RunPreparationSteps(logger, stepExecutor)) {
-			return new InstanceLaunchResult.CouldNotPrepareServerInstance();
-		}
-		
-		var processConfigurator = new ProcessConfigurator {
-			FileName = executablePath,
-			WorkingDirectory = instanceProperties.InstanceFolder,
-			RedirectInput = true,
-			UseShellExecute = false,
-		};
-		
-		var processArguments = processConfigurator.ArgumentList;
-		
-		foreach (IInstanceValue value in launchRecipe.Arguments) {
-			if (value.Resolve(valueResolver) is {} resolved) {
-				processArguments.Add(resolved);
-			}
-			else {
-				logger.Error("Could not resolve server executable argument: {Value}", value);
-				return new InstanceLaunchResult.CouldNotPrepareServerInstance();
-			}
-		}
-		
-		var process = processConfigurator.CreateProcess();
-		var instanceProcess = new InstanceProcess(instanceProperties, process);
-		
-		try {
-			process.Start();
-		} catch (Exception launchException) {
-			logger.Error(launchException, "Caught exception launching the server process.");
-			
-			try {
-				process.Kill();
-			} catch (Exception killException) {
-				logger.Error(killException, "Caught exception trying to kill the server process after a failed launch.");
-			}
-			
-			return new InstanceLaunchResult.CouldNotStartServerExecutable();
-		}
-		
-		return new InstanceLaunchResult.Success(instanceProcess);
-	}
-	
-	private async Task<bool> RunPreparationSteps(ILogger logger, StepExecutor stepExecutor) {
-		var steps = launchRecipe.Preparation;
-		
-		for (int stepIndex = 0; stepIndex < steps.Length; stepIndex++) {
-			var step = steps[stepIndex];
-			try {
-				if (await step.Run(stepExecutor)) {
-					continue;
-				}
-			} catch (OperationCanceledException) {
-				throw;
-			} catch (Exception e) {
-				logger.Error(e, "Failed preparation step {StepIndex} out of {StepCount}: {StepName}", stepIndex, steps.Length, step.GetType().Name);
-			}
-			
-			return false;
-		}
-		
-		return true;
-	}
-	
-	private sealed class StepExecutor(ILogger logger, FileDownloadManager downloadManager, IInstancePathResolver pathResolver, Action<IInstanceStatus?> reportStatus, CancellationToken cancellationToken) : IInstanceLaunchStepExecutor<bool> {
-		public async Task<bool> DownloadFile(FileDownloadInfo downloadInfo, IInstancePath path) {
-			string? filePath = path.Resolve(pathResolver);
-			if (filePath == null) {
-				logger.Error("Could not resolve download file path: {Path}", path);
-				return false;
-			}
-			
-			byte? lastDownloadProgress = null;
-			
-			void OnDownloadProgress(object? sender, DownloadProgressEventArgs args) {
-				byte? progress = args.TotalBytes is not {} totalBytes ? null : (byte) Math.Min(args.DownloadedBytes * 100 / totalBytes, val2: 100);
-				
-				if (lastDownloadProgress != progress) {
-					lastDownloadProgress = progress;
-					reportStatus(InstanceStatus.Downloading(progress));
-				}
-			}
-			
-			reportStatus(InstanceStatus.Downloading(null));
-			
-			if (await downloadManager.DownloadAndGetPath(downloadInfo, filePath, OnDownloadProgress, cancellationToken) == null) {
-				logger.Error("Could not download file: {Url}", downloadInfo.Url);
-				return false;
-			}
-			
-			reportStatus(null);
-			return true;
-		}
-		
-		public async Task<bool> EditPropertiesFile(InstancePath.Local path, string comment, ImmutableDictionary<string, string> newValues) {
-			string? filePath = path.Resolve(pathResolver);
-			if (filePath == null) {
-				logger.Error("Could not resolve properties file path: {Path}", path);
-				return false;
-			}
-			
-			var editor = new JavaPropertiesFileEditor();
-			editor.SetAll(newValues);
-			
-			try {
-				await editor.EditOrCreate(filePath, comment, cancellationToken);
-			} catch (Exception e) {
-				logger.Error(e, "Could not edit properties file: {Path}", filePath);
-				return false;
-			}
-			
-			return true;
-		}
-	}
-}

Agent/Phantom.Agent.Services/Instances/Launch/InstancePathResolver.cs

@@ -1,40 +0,0 @@
-using System.Buffers;
-using System.Collections.Immutable;
-using Phantom.Agent.Services.Java;
-using Phantom.Common.Data.Agent.Instance;
-
-namespace Phantom.Agent.Services.Instances.Launch;
-
-sealed class InstancePathResolver(AgentFolders agentFolders, JavaRuntimeRepository javaRuntimeRepository, InstanceProperties instanceProperties) : IInstancePathResolver {
-	public string? Global(ImmutableArray<string> segments) {
-		return ValidateAndCombinePath(agentFolders.ServerExecutableFolderPath, segments);
-	}
-	
-	public string? Local(ImmutableArray<string> segments) {
-		return ValidateAndCombinePath(instanceProperties.InstanceFolder, segments);
-	}
-	
-	public string? Runtime(Guid guid) {
-		return javaRuntimeRepository.TryGetByGuid(guid, out var runtime) ? runtime.ExecutablePath : null;
-	}
-	
-	private static readonly SearchValues<char> InvalidPathPartChars = SearchValues.Create([
-		Path.DirectorySeparatorChar,
-		Path.AltDirectorySeparatorChar,
-		..Path.GetInvalidPathChars(),
-	]);
-	
-	private string? ValidateAndCombinePath(string basePath, ImmutableArray<string> pathSegments) {
-		string path = basePath;
-		
-		foreach (string segment in pathSegments) {
-			if (segment == "." || segment == ".." || segment.ContainsAny(InvalidPathPartChars) || Path.IsPathRooted(segment)) {
-				return null;
-			}
-			
-			path = Path.Combine(path, segment);
-		}
-		
-		return Path.GetFullPath(path);
-	}
-}

Agent/Phantom.Agent.Services/Instances/State/InstanceLaunchProcedure.cs

@@ -1,9 +1,10 @@
-using Phantom.Agent.Services.Instances.State;
+using Phantom.Agent.Minecraft.Instance;
+using Phantom.Agent.Minecraft.Launcher;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Agent.Instance;
 using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Instance.Launch;
 
-namespace Phantom.Agent.Services.Instances.Launch;
+namespace Phantom.Agent.Services.Instances.State;
 
 static class InstanceLaunchProcedure {
 	public static async Task<InstanceRunningState?> Run(InstanceContext context, InstanceInfo info, InstanceLauncher launcher, InstanceTicketManager ticketManager, InstanceTicketManager.Ticket ticket, Action<IInstanceStatus?> reportStatus, CancellationToken cancellationToken) {
@@ -43,18 +44,18 @@ static class InstanceLaunchProcedure {
 		cancellationToken.ThrowIfCancellationRequested();
 		
 		switch (await launcher.Launch(context.Logger, reportStatus, cancellationToken)) {
-			case InstanceLaunchResult.Success launchSuccess:
+			case LaunchResult.Success launchSuccess:
 				return launchSuccess.Process;
 			
-			case InstanceLaunchResult.CouldNotPrepareServerInstance:
+			case LaunchResult.CouldNotPrepareServerInstance:
 				context.Logger.Error("Session failed to launch, could not prepare server instance.");
 				return InstanceLaunchFailReason.CouldNotPrepareServerInstance;
 			
-			case InstanceLaunchResult.CouldNotFindServerExecutable:
+			case LaunchResult.CouldNotFindServerExecutable:
 				context.Logger.Error("Session failed to launch, could not find server executable.");
 				return InstanceLaunchFailReason.CouldNotFindServerExecutable;
 			
-			case InstanceLaunchResult.CouldNotStartServerExecutable:
+			case LaunchResult.CouldNotStartServerExecutable:
 				context.Logger.Error("Session failed to launch, could not start server executable.");
 				return InstanceLaunchFailReason.CouldNotStartServerExecutable;
 			

Agent/Phantom.Agent.Services/Instances/State/InstancePlayerCountTracker.cs

@@ -1,5 +1,6 @@
 using System.Net.Sockets;
-using Phantom.Agent.Services.Games;
+using Phantom.Agent.Minecraft.Instance;
+using Phantom.Agent.Minecraft.Server;
 using Phantom.Agent.Services.Rpc;
 using Phantom.Common.Data.Instance;
 using Phantom.Common.Messages.Agent.ToController;
@@ -59,8 +60,8 @@ sealed class InstancePlayerCountTracker : CancellableBackgroundTask {
 	
 	private async Task<InstancePlayerCounts?> TryGetPlayerCounts() {
 		try {
-			return await MinecraftServerStatusProtocol.GetPlayerCounts(serverPort, CancellationToken);
+			return await ServerStatusProtocol.GetPlayerCounts(serverPort, CancellationToken);
-		} catch (MinecraftServerStatusProtocol.ProtocolException e) {
+		} catch (ServerStatusProtocol.ProtocolException e) {
 			Logger.Error("{Message}", e.Message);
 			return null;
 		} catch (SocketException e) {

Agent/Phantom.Agent.Services/Instances/State/InstanceRunningState.cs

@@ -1,8 +1,9 @@
-using Phantom.Agent.Services.Backups;
+using Phantom.Agent.Minecraft.Instance;
-using Phantom.Agent.Services.Instances.Launch;
+using Phantom.Agent.Minecraft.Launcher;
-using Phantom.Common.Data.Agent.Instance;
+using Phantom.Agent.Services.Backups;
 using Phantom.Common.Data.Backups;
 using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Instance.Launch;
 using Phantom.Common.Data.Replies;
 
 namespace Phantom.Agent.Services.Instances.State;
@@ -11,7 +12,6 @@ sealed class InstanceRunningState : IDisposable {
 	public InstanceTicketManager.Ticket Ticket { get; }
 	public InstanceProcess Process { get; }
 	
-	internal IInstanceValueResolver ValueResolver => launcher.ValueResolver;
 	internal bool IsStopping { get; set; }
 	
 	private readonly InstanceContext context;

Agent/Phantom.Agent.Services/Instances/State/InstanceStopProcedure.cs

@@ -1,50 +1,38 @@
 using System.Diagnostics;
-using Phantom.Common.Data.Agent.Instance;
+using Phantom.Agent.Minecraft.Command;
-using Phantom.Common.Data.Agent.Instance.Stop;
+using Phantom.Agent.Minecraft.Instance;
 using Phantom.Common.Data.Instance;
-using Serilog;
+using Phantom.Common.Data.Minecraft;
 
 namespace Phantom.Agent.Services.Instances.State;
 
 static class InstanceStopProcedure {
-	public static async Task<bool> Run(InstanceContext context, InstanceStopRecipe stopRecipe, InstanceRunningState runningState, Action<IInstanceStatus> reportStatus, CancellationToken cancellationToken) {
+	private static readonly ushort[] Stops = [60, 30, 10, 5, 4, 3, 2, 1, 0];
-		var logger = context.Logger;
-		
-		var stopCommand = stopRecipe.StopCommand.Resolve(runningState.ValueResolver);
-		if (stopCommand == null) {
-			logger.Error("Could not resolve stop command");
-			return false;
-		}
 	
+	public static async Task<bool> Run(InstanceContext context, MinecraftStopStrategy stopStrategy, InstanceRunningState runningState, Action<IInstanceStatus> reportStatus, CancellationToken cancellationToken) {
+		var process = runningState.Process;
 		runningState.IsStopping = true;
 		
-		bool continueStopping = false;
+		var seconds = stopStrategy.Seconds;
+		if (seconds > 0) {
 			try {
-			var stepExecutor = new StepExecutor(logger, runningState.ValueResolver, runningState.Process, cancellationToken);
+				await CountDownWithAnnouncements(context, process, seconds, cancellationToken);
-			continueStopping = await RunPreparationSteps(context, stopRecipe, stepExecutor);
+			} catch (OperationCanceledException) {
-		} finally {
-			if (!continueStopping) {
 				runningState.IsStopping = false;
-			}
-		}
-		
-		if (!continueStopping) {
 				return false;
 			}
+		}
 		
 		try {
 			// Too late to cancel the stop procedure now.
 			runningState.OnStopInitiated();
 			
-			if (!runningState.Process.HasEnded) {
+			if (!process.HasEnded) {
-				logger.Information("Sending stop command...");
+				context.Logger.Information("Session stopping now.");
-				await TrySendStopCommand(context, runningState.Process, stopCommand);
+				await DoStop(context, process);
-				
-				logger.Information("Waiting for session to end...");
-				await WaitForSessionToEnd(context, runningState.Process);
 			}
 		} finally {
-			logger.Information("Session stopped.");
+			context.Logger.Information("Session stopped.");
 			reportStatus(InstanceStatus.NotRunning);
 			context.ReportEvent(InstanceEvent.Stopped);
 		}
@@ -52,51 +40,39 @@ static class InstanceStopProcedure {
 		return true;
 	}
 	
-	private static async Task<bool> RunPreparationSteps(InstanceContext context, InstanceStopRecipe stopRecipe, StepExecutor executor) {
+	private static async Task CountDownWithAnnouncements(InstanceContext context, InstanceProcess process, ushort seconds, CancellationToken cancellationToken) {
-		var steps = stopRecipe.Preparation;
+		context.Logger.Information("Session stopping in {Seconds} seconds.", seconds);
 		
-		for (int stepIndex = 0; stepIndex < steps.Length; stepIndex++) {
+		foreach (var stop in Stops) {
-			var step = steps[stepIndex];
+			// TODO change to event-based cancellation
-			try {
+			if (process.HasEnded) {
-				if (await step.Run(executor)) {
+				return;
-					continue;
-				}
-			} catch (OperationCanceledException) {
-				throw;
-			} catch (Exception e) {
-				context.Logger.Error(e, "Failed preparation step {StepIndex} out of {StepCount}: {StepName}", stepIndex, steps.Length, step.GetType().Name);
 			}
 			
-			return false;
+			if (seconds > stop) {
+				await process.SendCommand(GetCountDownAnnouncementCommand(seconds), cancellationToken);
+				await Task.Delay(TimeSpan.FromSeconds(seconds - stop), cancellationToken);
+				seconds = stop;
 			}
-		
-		return true;
-	}
-	
-	private sealed class StepExecutor(ILogger logger, IInstanceValueResolver valueResolver, InstanceProcess process, CancellationToken cancellationToken) : IInstanceStopStepExecutor<bool> {
-		public async Task<bool> Wait(TimeSpan duration) {
-			await Task.Delay(duration, cancellationToken);
-			return true;
-		}
-		
-		public async Task<bool> SendToStandardInput(IInstanceValue line) {
-			string? command = line.Resolve(valueResolver);
-			if (command == null) {
-				logger.Error("Could not resolve standard input line: {Value}", line);
-				return false;
-			}
-			
-			// If the process can't process standard input, wait a bit but don't block or fail the whole stop procedure.
-			await process.TrySendCommand(command, TimeSpan.FromMilliseconds(500), cancellationToken);
-			
-			return true;
 		}
 	}
 	
-	private static async Task TrySendStopCommand(InstanceContext context, InstanceProcess process, string command) {
+	private static string GetCountDownAnnouncementCommand(ushort seconds) {
+		return MinecraftCommand.Say("Server shutting down in " + seconds + (seconds == 1 ? " second." : " seconds."));
+	}
+	
+	private static async Task DoStop(InstanceContext context, InstanceProcess process) {
+		context.Logger.Information("Sending stop command...");
+		await TrySendStopCommand(context, process);
+		
+		context.Logger.Information("Waiting for session to end...");
+		await WaitForSessionToEnd(context, process);
+	}
+	
+	private static async Task TrySendStopCommand(InstanceContext context, InstanceProcess process) {
 		using var timeout = new CancellationTokenSource(TimeSpan.FromSeconds(5));
 		try {
-			await process.SendCommand(command, timeout.Token);
+			await process.SendCommand(MinecraftCommand.Stop, timeout.Token);
 		} catch (OperationCanceledException) {
 			// Ignore.
 		} catch (ObjectDisposedException e) when (e.ObjectName == typeof(Process).FullName && process.HasEnded) {

Agent/Phantom.Agent.Services/Java/JavaRuntimeExecutable.cs

@@ -1,5 +0,0 @@
-using Phantom.Common.Data.Java;
-
-namespace Phantom.Agent.Services.Java;
-
-sealed record JavaRuntimeExecutable(string ExecutablePath, JavaRuntime Runtime);

Agent/Phantom.Agent.Services/Phantom.Agent.Services.csproj

@@ -5,12 +5,9 @@
     <Nullable>enable</Nullable>
   </PropertyGroup>
   
-  <ItemGroup>
-    <InternalsVisibleTo Include="Phantom.Agent.Services.Tests" />
-  </ItemGroup>
-  
   <ItemGroup>
     <ProjectReference Include="..\..\Common\Phantom.Common.Messages.Agent\Phantom.Common.Messages.Agent.csproj" />
+    <ProjectReference Include="..\Phantom.Agent.Minecraft\Phantom.Agent.Minecraft.csproj" />
   </ItemGroup>
 
 </Project>

Agent/Phantom.Agent.Services/Rpc/ControllerMessageHandlerActor.cs

@@ -26,7 +26,7 @@ public sealed class ControllerMessageHandlerActor : ReceiveActor<IMessageToAgent
 	}
 	
 	private async Task<Result<ConfigureInstanceResult, InstanceActionFailure>> HandleConfigureInstance(ConfigureInstanceMessage message) {
-		return await agent.InstanceManager.Request(new InstanceManagerActor.ConfigureInstanceCommand(message.InstanceGuid, message.Info, message.LaunchRecipe, message.LaunchNow, message.StopRecipe, AlwaysReportStatus: false));
+		return await agent.InstanceManager.Request(new InstanceManagerActor.ConfigureInstanceCommand(message.InstanceGuid, message.Info, message.LaunchRecipe, message.LaunchNow, AlwaysReportStatus: false));
 	}
 	
 	private async Task<Result<LaunchInstanceResult, InstanceActionFailure>> HandleLaunchInstance(LaunchInstanceMessage message) {
@@ -34,7 +34,7 @@ public sealed class ControllerMessageHandlerActor : ReceiveActor<IMessageToAgent
 	}
 	
 	private async Task<Result<StopInstanceResult, InstanceActionFailure>> HandleStopInstance(StopInstanceMessage message) {
-		return await agent.InstanceManager.Request(new InstanceManagerActor.StopInstanceCommand(message.InstanceGuid, message.StopRecipe));
+		return await agent.InstanceManager.Request(new InstanceManagerActor.StopInstanceCommand(message.InstanceGuid, message.StopStrategy));
 	}
 	
 	private async Task<Result<SendCommandToInstanceResult, InstanceActionFailure>> HandleSendCommandToInstance(SendCommandToInstanceMessage message) {

Agent/Phantom.Agent/Phantom.Agent.csproj

@@ -12,6 +12,7 @@
   
   <ItemGroup>
     <ProjectReference Include="..\..\Utils\Phantom.Utils\Phantom.Utils.csproj" />
+    <ProjectReference Include="..\Phantom.Agent.Minecraft\Phantom.Agent.Minecraft.csproj" />
     <ProjectReference Include="..\Phantom.Agent.Services\Phantom.Agent.Services.csproj" />
   </ItemGroup>
 

Agent/Phantom.Agent/Program.cs

@@ -1,7 +1,7 @@
 using System.Reflection;
 using Phantom.Agent;
+using Phantom.Agent.Minecraft.Java;
 using Phantom.Agent.Services;
-using Phantom.Agent.Services.Java;
 using Phantom.Agent.Services.Rpc;
 using Phantom.Common.Data.Agent;
 using Phantom.Common.Messages.Agent;

Agent/Phantom.Agent/Variables.cs

@@ -1,4 +1,4 @@
-using Phantom.Agent.Services.Java;
+using Phantom.Agent.Minecraft.Java;
 using Phantom.Common.Data;
 using Phantom.Utils.Logging;
 using Phantom.Utils.Runtime;
@@ -43,7 +43,7 @@ sealed record Variables(
 		try {
 			return LoadOrThrow();
 		} catch (Exception e) {
-			PhantomLogger.Root.Fatal("{Error}", e.Message);
+			PhantomLogger.Root.Fatal("{}", e.Message);
 			throw StopProcedureException.Instance;
 		}
 	}

Common/Phantom.Common.Data.Agent/Instance/IInstancePathResolver.cs

@@ -1,9 +0,0 @@
-using System.Collections.Immutable;
-
-namespace Phantom.Common.Data.Agent.Instance;
-
-public interface IInstancePathResolver {
-	string? Global(ImmutableArray<string> segments);
-	string? Local(ImmutableArray<string> segments);
-	string? Runtime(Guid guid);
-}

Common/Phantom.Common.Data.Agent/Instance/IInstanceValueResolver.cs

@@ -1,5 +0,0 @@
-namespace Phantom.Common.Data.Agent.Instance;
-
-public interface IInstanceValueResolver {
-	string? Path(IInstancePath value);
-}

Common/Phantom.Common.Data.Agent/Instance/Launch/IInstanceLaunchStepExecutor.cs

@@ -1,8 +0,0 @@
-using System.Collections.Immutable;
-
-namespace Phantom.Common.Data.Agent.Instance.Launch;
-
-public interface IInstanceLaunchStepExecutor<TResult> {
-	Task<TResult> DownloadFile(FileDownloadInfo downloadInfo, IInstancePath path);
-	Task<TResult> EditPropertiesFile(InstancePath.Local path, string comment, ImmutableDictionary<string, string> newValues);
-}

Common/Phantom.Common.Data.Agent/Instance/Stop/IInstanceLaunchStepExecutor.cs

@@ -1,6 +0,0 @@
-namespace Phantom.Common.Data.Agent.Instance.Stop;
-
-public interface IInstanceStopStepExecutor<TResult> {
-	Task<TResult> Wait(TimeSpan duration);
-	Task<TResult> SendToStandardInput(IInstanceValue line);
-}

Common/Phantom.Common.Data.Agent/Instance/Stop/IInstanceStopStep.cs

@@ -1,30 +0,0 @@
-using MemoryPack;
-
-namespace Phantom.Common.Data.Agent.Instance.Stop;
-
-[MemoryPackable]
-[MemoryPackUnion(tag: 0, typeof(InstanceStopStep.Wait))]
-[MemoryPackUnion(tag: 1, typeof(InstanceStopStep.SendToStandardInput))]
-public partial interface IInstanceStopStep {
-	Task<TResult> Run<TResult>(IInstanceStopStepExecutor<TResult> executor);
-}
-
-public static partial class InstanceStopStep {
-	[MemoryPackable(GenerateType.VersionTolerant)]
-	public sealed partial record Wait(
-		[property: MemoryPackOrder(0)] TimeSpan Duration
-	) : IInstanceStopStep {
-		public Task<TResult> Run<TResult>(IInstanceStopStepExecutor<TResult> executor) {
-			return executor.Wait(Duration);
-		}
-	}
-	
-	[MemoryPackable(GenerateType.VersionTolerant)]
-	public sealed partial record SendToStandardInput(
-		[property: MemoryPackOrder(0)] IInstanceValue Line
-	) : IInstanceStopStep {
-		public Task<TResult> Run<TResult>(IInstanceStopStepExecutor<TResult> executor) {
-			return executor.SendToStandardInput(Line);
-		}
-	}
-}

Common/Phantom.Common.Data.Agent/Instance/Stop/InstanceStopRecipe.cs

@@ -1,10 +0,0 @@
-using System.Collections.Immutable;
-using MemoryPack;
-
-namespace Phantom.Common.Data.Agent.Instance.Stop;
-
-[MemoryPackable(GenerateType.VersionTolerant)]
-public sealed partial record InstanceStopRecipe(
-	[property: MemoryPackOrder(0)] ImmutableArray<IInstanceStopStep> Preparation,
-	[property: MemoryPackOrder(1)] IInstanceValue StopCommand
-);

Common/Phantom.Common.Data.Agent/Phantom.Common.Data.Agent.csproj

@@ -1,16 +0,0 @@
-<Project Sdk="Microsoft.NET.Sdk">
-  
-  <PropertyGroup>
-    <ImplicitUsings>enable</ImplicitUsings>
-    <Nullable>enable</Nullable>
-  </PropertyGroup>
-  
-  <ItemGroup>
-    <PackageReference Include="MemoryPack" />
-  </ItemGroup>
-  
-  <ItemGroup>
-    <ProjectReference Include="..\Phantom.Common.Data\Phantom.Common.Data.csproj" />
-  </ItemGroup>
-
-</Project>

Common/Phantom.Common.Data.Web/Agent/Agent.cs

@@ -1,5 +1,6 @@
 using System.Collections.Immutable;
 using MemoryPack;
+using Phantom.Common.Data.Agent;
 
 namespace Phantom.Common.Data.Web.Agent;
 

Common/Phantom.Common.Data.Web/Agent/AgentRuntimeInfo.cs

@@ -1,4 +1,5 @@
 using MemoryPack;
+using Phantom.Common.Data.Agent;
 
 namespace Phantom.Common.Data.Web.Agent;
 
@@ -9,4 +10,8 @@ public sealed partial record AgentRuntimeInfo(
 	[property: MemoryPackOrder(2)] RamAllocationUnits? MaxMemory = null,
 	[property: MemoryPackOrder(3)] AllowedPorts? AllowedServerPorts = null,
 	[property: MemoryPackOrder(4)] AllowedPorts? AllowedAdditionalPorts = null
-);
+) {
+	public static AgentRuntimeInfo From(AgentInfo agentInfo) {
+		return new AgentRuntimeInfo(new AgentVersionInfo(agentInfo.ProtocolVersion, agentInfo.BuildVersion), agentInfo.MaxInstances, agentInfo.MaxMemory, agentInfo.AllowedServerPorts, agentInfo.AllowedAdditionalPorts);
+	}
+}

Common/Phantom.Common.Data.Web/Instance/CreateOrUpdateInstanceResult.cs

@@ -5,7 +5,7 @@ public enum CreateOrUpdateInstanceResult : byte {
 	Success,
 	InstanceNameMustNotBeEmpty,
 	InstanceMemoryMustNotBeZero,
-	MinecraftVersionNotFound,
+	MinecraftVersionDownloadInfoNotFound,
 	AgentNotFound,
 }
 
@@ -15,7 +15,7 @@ public static class CreateOrUpdateInstanceResultExtensions {
 			CreateOrUpdateInstanceResult.Success                              => "Success.",
 			CreateOrUpdateInstanceResult.InstanceNameMustNotBeEmpty           => "Instance name must not be empty.",
 			CreateOrUpdateInstanceResult.InstanceMemoryMustNotBeZero          => "Memory must not be 0 MB.",
-			CreateOrUpdateInstanceResult.MinecraftVersionNotFound    => "Minecraft version not found.",
+			CreateOrUpdateInstanceResult.MinecraftVersionDownloadInfoNotFound => "Could not find download information for the selected Minecraft version.",
 			CreateOrUpdateInstanceResult.AgentNotFound                        => "Agent not found.",
 			_                                                                 => "Unknown error.",
 		};

Common/Phantom.Common.Data.Web/Minecraft/JvmArgumentsHelper.cs

@@ -1,6 +1,6 @@
 using System.Collections.Immutable;
 
-namespace Phantom.Common.Data.Web.Java;
+namespace Phantom.Common.Data.Web.Minecraft;
 
 public static class JvmArgumentsHelper {
 	public static ImmutableArray<string> Split(string arguments) {

Common/Phantom.Common.Data/Agent/AgentStats.cs

@@ -1,6 +1,6 @@
 using MemoryPack;
 
-namespace Phantom.Common.Data.Web.Agent;
+namespace Phantom.Common.Data.Agent;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record AgentStats(

Common/Phantom.Common.Data/Instance/IInstanceEvent.cs

@@ -1,5 +1,6 @@
 using MemoryPack;
 using Phantom.Common.Data.Backups;
+using Phantom.Common.Data.Instance.Launch;
 
 namespace Phantom.Common.Data.Instance;
 

Common/Phantom.Common.Data/Instance/IInstancePath.cs

@@ -1,7 +1,7 @@
 using System.Collections.Immutable;
 using MemoryPack;
 
-namespace Phantom.Common.Data.Agent.Instance;
+namespace Phantom.Common.Data.Instance;
 
 [MemoryPackable]
 [MemoryPackUnion(tag: 0, typeof(InstancePath.Global))]
@@ -14,27 +14,19 @@ public partial interface IInstancePath {
 public static partial class InstancePath {
 	[MemoryPackable(GenerateType.VersionTolerant)]
 	public sealed partial record Global(
-		[property: MemoryPackOrder(0)] ImmutableArray<string> Segments
+		[property: MemoryPackOrder(0)] ImmutableArray<string> Path
 	) : IInstancePath {
 		public string? Resolve(IInstancePathResolver resolver) {
-			return resolver.Global(Segments);
+			return resolver.Global(Path);
-		}
-		
-		public override string ToString() {
-			return "Global[" + string.Join(separator: '/', Segments) + "]";
 		}
 	}
 	
 	[MemoryPackable(GenerateType.VersionTolerant)]
 	public sealed partial record Local(
-		[property: MemoryPackOrder(0)] ImmutableArray<string> Segments
+		[property: MemoryPackOrder(0)] ImmutableArray<string> Path
 	) : IInstancePath {
 		public string? Resolve(IInstancePathResolver resolver) {
-			return resolver.Local(Segments);
+			return resolver.Local(Path);
-		}
-		
-		public override string ToString() {
-			return "Local[" + string.Join(separator: '/', Segments) + "]";
 		}
 	}
 	
@@ -45,9 +37,5 @@ public static partial class InstancePath {
 		public string? Resolve(IInstancePathResolver resolver) {
 			return resolver.Runtime(Guid);
 		}
-		
-		public override string ToString() {
-			return "Runtime[" + Guid + "]";
-		}
 	}
 }

Common/Phantom.Common.Data/Instance/IInstancePathResolver.cs

@@ -0,0 +1,9 @@
+using System.Collections.Immutable;
+
+namespace Phantom.Common.Data.Instance;
+
+public interface IInstancePathResolver {
+	string? Global(ImmutableArray<string> path);
+	string? Local(ImmutableArray<string> path);
+	string? Runtime(Guid guid);
+}

Common/Phantom.Common.Data/Instance/IInstanceStatus.cs

@@ -1,4 +1,5 @@
 using MemoryPack;
+using Phantom.Common.Data.Instance.Launch;
 
 namespace Phantom.Common.Data.Instance;
 

Common/Phantom.Common.Data/Instance/IInstanceValue.cs

@@ -2,12 +2,13 @@
 using System.Text;
 using MemoryPack;
 
-namespace Phantom.Common.Data.Agent.Instance;
+namespace Phantom.Common.Data.Instance;
 
 [MemoryPackable]
 [MemoryPackUnion(tag: 0, typeof(InstanceValues.Concatenation))]
 [MemoryPackUnion(tag: 1, typeof(InstanceValues.Text))]
 [MemoryPackUnion(tag: 2, typeof(InstanceValues.Path))]
+[MemoryPackUnion(tag: 3, typeof(InstanceValues.Variable))]
 public partial interface IInstanceValue {
 	string? Resolve(IInstanceValueResolver resolver);
 }
@@ -29,10 +30,6 @@ public static partial class InstanceValues {
 			
 			return result.ToString();
 		}
-		
-		public override string ToString() {
-			return "Concatenation[" + string.Join(",", Values) + "]";
-		}
 	}
 	
 	[MemoryPackable]
@@ -40,10 +37,6 @@ public static partial class InstanceValues {
 		public string Resolve(IInstanceValueResolver resolver) {
 			return Value;
 		}
-		
-		public override string ToString() {
-			return "Text[" + Value + "]";
-		}
 	}
 	
 	[MemoryPackable]
@@ -51,9 +44,14 @@ public static partial class InstanceValues {
 		public string? Resolve(IInstanceValueResolver resolver) {
 			return resolver.Path(Value);
 		}
+	}
 	
-		public override string ToString() {
+	[MemoryPackable]
-			return "Path[" + Value + "]";
+	public sealed partial record Variable(InstanceVariable Value) : IInstanceValue {
+		public string? Resolve(IInstanceValueResolver resolver) {
+			return resolver.Variable(Value);
 		}
 	}
 }
+
+public enum InstanceVariable {}

Common/Phantom.Common.Data/Instance/IInstanceValueResolver.cs

@@ -0,0 +1,6 @@
+namespace Phantom.Common.Data.Instance;
+
+public interface IInstanceValueResolver {
+	string? Path(IInstancePath value);
+	string? Variable(InstanceVariable value);
+}

Common/Phantom.Common.Data/Instance/InstanceConfiguration.cs

@@ -1,8 +1,8 @@
 using System.Collections.Immutable;
 using MemoryPack;
-using Phantom.Common.Data.Web.Minecraft;
+using Phantom.Common.Data.Minecraft;
 
-namespace Phantom.Common.Data.Web.Instance;
+namespace Phantom.Common.Data.Instance;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record InstanceConfiguration(
@@ -15,4 +15,7 @@ public sealed partial record InstanceConfiguration(
 	[property: MemoryPackOrder(6)] RamAllocationUnits MemoryAllocation,
 	[property: MemoryPackOrder(7)] Guid JavaRuntimeGuid,
 	[property: MemoryPackOrder(8)] ImmutableArray<string> JvmArguments
-);
+) {
+	[MemoryPackIgnore]
+	public InstanceInfo AsInfo => new (InstanceName, ServerPort, [RconPort], MemoryAllocation);
+}

Common/Phantom.Common.Data/Instance/InstanceInfo.cs

@@ -1,7 +1,7 @@
 using System.Collections.Immutable;
 using MemoryPack;
 
-namespace Phantom.Common.Data.Agent.Instance;
+namespace Phantom.Common.Data.Instance;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record InstanceInfo(

Common/Phantom.Common.Data/Instance/Launch/IInstanceLaunchStep.cs

@@ -1,13 +1,14 @@
 using System.Collections.Immutable;
 using MemoryPack;
+using Phantom.Common.Data.Minecraft;
 
-namespace Phantom.Common.Data.Agent.Instance.Launch;
+namespace Phantom.Common.Data.Instance.Launch;
 
 [MemoryPackable]
 [MemoryPackUnion(tag: 0, typeof(InstanceLaunchStep.DownloadFile))]
 [MemoryPackUnion(tag: 1, typeof(InstanceLaunchStep.EditPropertiesFile))]
 public partial interface IInstanceLaunchStep {
-	Task<TResult> Run<TResult>(IInstanceLaunchStepExecutor<TResult> executor);
+	Task Run(IInstanceLaunchStepExecutor executor);
 }
 
 public static partial class InstanceLaunchStep {
@@ -16,7 +17,7 @@ public static partial class InstanceLaunchStep {
 		[property: MemoryPackOrder(0)] FileDownloadInfo DownloadInfo,
 		[property: MemoryPackOrder(1)] IInstancePath Path
 	) : IInstanceLaunchStep {
-		public Task<TResult> Run<TResult>(IInstanceLaunchStepExecutor<TResult> executor) {
+		public Task Run(IInstanceLaunchStepExecutor executor) {
 			return executor.DownloadFile(DownloadInfo, Path);
 		}
 	}
@@ -27,7 +28,7 @@ public static partial class InstanceLaunchStep {
 		[property: MemoryPackOrder(1)] string Comment,
 		[property: MemoryPackOrder(2)] ImmutableDictionary<string, string> NewValues
 	) : IInstanceLaunchStep {
-		public Task<TResult> Run<TResult>(IInstanceLaunchStepExecutor<TResult> executor) {
+		public Task Run(IInstanceLaunchStepExecutor executor) {
 			return executor.EditPropertiesFile(Path, Comment, NewValues);
 		}
 	}

Common/Phantom.Common.Data/Instance/Launch/IInstanceLaunchStepExecutor.cs

@@ -0,0 +1,9 @@
+using System.Collections.Immutable;
+using Phantom.Common.Data.Minecraft;
+
+namespace Phantom.Common.Data.Instance.Launch;
+
+public interface IInstanceLaunchStepExecutor {
+	Task DownloadFile(FileDownloadInfo downloadInfo, IInstancePath path);
+	Task EditPropertiesFile(InstancePath.Local path, string comment, ImmutableDictionary<string, string> newValues);
+}

Common/Phantom.Common.Data/Instance/Launch/InstanceLaunchFailReason.cs

@@ -1,4 +1,4 @@
-namespace Phantom.Common.Data.Instance;
+namespace Phantom.Common.Data.Instance.Launch;
 
 public enum InstanceLaunchFailReason : byte {
 	UnknownError                  = 0,

Common/Phantom.Common.Data/Instance/Launch/InstanceLaunchRecipe.cs

@@ -1,7 +1,7 @@
 using System.Collections.Immutable;
 using MemoryPack;
 
-namespace Phantom.Common.Data.Agent.Instance.Launch;
+namespace Phantom.Common.Data.Instance.Launch;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record InstanceLaunchRecipe(

Common/Phantom.Common.Data/Minecraft/FileDownloadInfo.cs

@@ -1,12 +1,12 @@
 using MemoryPack;
 using Phantom.Utils.Cryptography;
 
-namespace Phantom.Common.Data.Agent;
+namespace Phantom.Common.Data.Minecraft;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial class FileDownloadInfo {
 	[MemoryPackOrder(0)]
-	public string Url { get; }
+	public string DownloadUrl { get; }
 	
 	[MemoryPackOrder(1)]
 	[MemoryPackInclude]
@@ -15,11 +15,11 @@ public sealed partial class FileDownloadInfo {
 	[MemoryPackIgnore]
 	public Sha1String? Hash => hash == null ? null : Sha1String.FromString(hash);
 	
-	public FileDownloadInfo(string url, Sha1String? hash = null) : this(url, hash?.ToString()) {}
+	public FileDownloadInfo(string downloadUrl, Sha1String? hash = null) : this(downloadUrl, hash?.ToString()) {}
 	
 	[MemoryPackConstructor]
-	private FileDownloadInfo(string url, string? hash) {
+	private FileDownloadInfo(string downloadUrl, string? hash) {
-		this.Url = url;
+		this.DownloadUrl = downloadUrl;
 		this.hash = hash;
 	}
 }

Common/Phantom.Common.Data/Minecraft/MinecraftServerKind.cs

@@ -1,4 +1,4 @@
-namespace Phantom.Common.Data.Web.Minecraft;
+namespace Phantom.Common.Data.Minecraft;
 
 public enum MinecraftServerKind : ushort {
 	Vanilla = 1,

Common/Phantom.Common.Data/Minecraft/MinecraftStopStrategy.cs

@@ -0,0 +1,10 @@
+using MemoryPack;
+
+namespace Phantom.Common.Data.Minecraft;
+
+[MemoryPackable(GenerateType.VersionTolerant)]
+public sealed partial record MinecraftStopStrategy(
+	[property: MemoryPackOrder(0)] ushort Seconds
+) {
+	public static MinecraftStopStrategy Instant => new (0);
+}

Common/Phantom.Common.Data/Minecraft/MinecraftVersion.cs

@@ -1,6 +1,6 @@
 using MemoryPack;
 
-namespace Phantom.Common.Data.Web.Minecraft;
+namespace Phantom.Common.Data.Minecraft;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record MinecraftVersion(

Common/Phantom.Common.Data/Minecraft/MinecraftVersionType.cs

@@ -1,6 +1,6 @@
 using System.Collections.Immutable;
 
-namespace Phantom.Common.Data.Web.Minecraft;
+namespace Phantom.Common.Data.Minecraft;
 
 public enum MinecraftVersionType : ushort {
 	Other = 0,

Common/Phantom.Common.Data/Result.cs

@@ -1,5 +1,6 @@
 using System.Diagnostics.CodeAnalysis;
 using MemoryPack;
+using Phantom.Utils.Monads;
 using Phantom.Utils.Result;
 
 namespace Phantom.Common.Data;
@@ -24,6 +25,9 @@ public sealed partial class Result<TValue, TError> {
 	[MemoryPackIgnore]
 	public TError Error => !hasValue ? error! : throw new InvalidOperationException("Attempted to get error from a success result.");
 	
+	[MemoryPackIgnore]
+	public Either<TValue, TError> AsEither => hasValue ? Either.Left(value!) : Either.Right(error!);
+	
 	private Result(bool hasValue, TValue? value, TError? error) {
 		this.hasValue = hasValue;
 		this.value = value;
@@ -34,11 +38,6 @@ public sealed partial class Result<TValue, TError> {
 		return hasValue && EqualityComparer<TValue>.Default.Equals(value, expectedValue);
 	}
 	
-	[return: NotNullIfNotNull(nameof(valueIfError))]
-	public TValue? OrElse(TValue? valueIfError) {
-		return hasValue ? Value : valueIfError;
-	}
-	
 	public TOutput Into<TOutput>(Func<TValue, TOutput> valueConverter, Func<TError, TOutput> errorConverter) {
 		return hasValue ? valueConverter(value!) : errorConverter(error!);
 	}

Common/Phantom.Common.Messages.Agent/Phantom.Common.Messages.Agent.csproj

@@ -6,7 +6,7 @@
   </PropertyGroup>
   
   <ItemGroup>
-    <ProjectReference Include="..\Phantom.Common.Data.Agent\Phantom.Common.Data.Agent.csproj" />
+    <ProjectReference Include="..\Phantom.Common.Data\Phantom.Common.Data.csproj" />
     <ProjectReference Include="..\..\Utils\Phantom.Utils.Logging\Phantom.Utils.Logging.csproj" />
     <ProjectReference Include="..\..\Utils\Phantom.Utils.Rpc\Phantom.Utils.Rpc.csproj" />
   </ItemGroup>

Common/Phantom.Common.Messages.Agent/ToAgent/ConfigureInstanceMessage.cs

@@ -1,8 +1,7 @@
 using MemoryPack;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Agent.Instance;
+using Phantom.Common.Data.Instance;
-using Phantom.Common.Data.Agent.Instance.Launch;
+using Phantom.Common.Data.Instance.Launch;
-using Phantom.Common.Data.Agent.Instance.Stop;
 using Phantom.Common.Data.Replies;
 using Phantom.Utils.Actor;
 
@@ -13,6 +12,5 @@ public sealed partial record ConfigureInstanceMessage(
 	[property: MemoryPackOrder(0)] Guid InstanceGuid,
 	[property: MemoryPackOrder(1)] InstanceInfo Info,
 	[property: MemoryPackOrder(2)] InstanceLaunchRecipe? LaunchRecipe,
-	[property: MemoryPackOrder(3)] bool LaunchNow,
+	[property: MemoryPackOrder(3)] bool LaunchNow = false
-	[property: MemoryPackOrder(4)] InstanceStopRecipe StopRecipe
 ) : IMessageToAgent, ICanReply<Result<ConfigureInstanceResult, InstanceActionFailure>>;

Common/Phantom.Common.Messages.Agent/ToAgent/StopInstanceMessage.cs

@@ -1,6 +1,6 @@
 using MemoryPack;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Agent.Instance.Stop;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Utils.Actor;
 
@@ -9,5 +9,5 @@ namespace Phantom.Common.Messages.Agent.ToAgent;
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record StopInstanceMessage(
 	[property: MemoryPackOrder(0)] Guid InstanceGuid,
-	[property: MemoryPackOrder(1)] InstanceStopRecipe StopRecipe
+	[property: MemoryPackOrder(1)] MinecraftStopStrategy StopStrategy
 ) : IMessageToAgent, ICanReply<Result<StopInstanceResult, InstanceActionFailure>>;

Common/Phantom.Common.Messages.Web/ToController/CreateOrUpdateInstanceMessage.cs

@@ -1,6 +1,7 @@
 using System.Collections.Immutable;
 using MemoryPack;
 using Phantom.Common.Data;
+using Phantom.Common.Data.Instance;
 using Phantom.Common.Data.Web.Instance;
 using Phantom.Common.Data.Web.Users;
 using Phantom.Utils.Actor;

Common/Phantom.Common.Messages.Web/ToController/GetMinecraftVersionsMessage.cs

@@ -1,6 +1,6 @@
 using System.Collections.Immutable;
 using MemoryPack;
-using Phantom.Common.Data.Web.Minecraft;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Utils.Actor;
 
 namespace Phantom.Common.Messages.Web.ToController;

Common/Phantom.Common.Messages.Web/ToController/StopInstanceMessage.cs

@@ -1,6 +1,7 @@
 using System.Collections.Immutable;
 using MemoryPack;
 using Phantom.Common.Data;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Data.Web.Users;
 using Phantom.Utils.Actor;
@@ -12,5 +13,5 @@ public sealed partial record StopInstanceMessage(
 	[property: MemoryPackOrder(0)] ImmutableArray<byte> AuthToken,
 	[property: MemoryPackOrder(1)] Guid AgentGuid,
 	[property: MemoryPackOrder(2)] Guid InstanceGuid,
-	[property: MemoryPackOrder(3)] ushort AfterSeconds
+	[property: MemoryPackOrder(3)] MinecraftStopStrategy StopStrategy
 ) : IMessageToController, ICanReply<Result<StopInstanceResult, UserInstanceActionFailure>>;

Common/Phantom.Common.Messages.Web/WebMessageRegistries.cs

@@ -1,12 +1,12 @@
 using System.Collections.Immutable;
 using Phantom.Common.Data;
 using Phantom.Common.Data.Java;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Data.Web.Agent;
 using Phantom.Common.Data.Web.AuditLog;
 using Phantom.Common.Data.Web.EventLog;
 using Phantom.Common.Data.Web.Instance;
-using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Common.Data.Web.Users;
 using Phantom.Common.Messages.Web.ToController;
 using Phantom.Common.Messages.Web.ToWeb;

Controller/Phantom.Controller.Database/ApplicationDbContext.cs

@@ -2,9 +2,9 @@
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
 using Phantom.Common.Data;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Web.AuditLog;
 using Phantom.Common.Data.Web.EventLog;
-using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Controller.Database.Converters;
 using Phantom.Controller.Database.Entities;
 using Phantom.Controller.Database.Factories;

Controller/Phantom.Controller.Database/Entities/InstanceEntity.cs

@@ -2,7 +2,7 @@
 using System.ComponentModel.DataAnnotations.Schema;
 using System.Diagnostics.CodeAnalysis;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Web.Minecraft;
+using Phantom.Common.Data.Minecraft;
 
 namespace Phantom.Controller.Database.Entities;
 

Controller/Phantom.Controller.Database/Repositories/AuditLogRepository.Writer.cs

@@ -87,8 +87,10 @@ sealed partial class AuditLogRepository {
 			});
 		}
 		
-		public void InstanceStopped(Guid instanceGuid) {
+		public void InstanceStopped(Guid instanceGuid, int stopInSeconds) {
-			AddItem(AuditLogEventType.InstanceStopped, instanceGuid.ToString());
+			AddItem(AuditLogEventType.InstanceStopped, instanceGuid.ToString(), new Dictionary<string, object?> {
+				{ "stop_in_seconds", stopInSeconds.ToString() },
+			});
 		}
 	}
 }

Controller/Phantom.Controller.Minecraft/MinecraftInstanceRecipes.Stop.cs

@@ -1,41 +0,0 @@
-using System.Collections.Immutable;
-using Phantom.Common.Data.Agent.Instance;
-using Phantom.Common.Data.Agent.Instance.Stop;
-
-namespace Phantom.Controller.Minecraft;
-
-public sealed partial class MinecraftInstanceRecipes {
-	private static readonly ushort[] Stops = [60, 30, 10, 5, 4, 3, 2, 1];
-	private static readonly IInstanceValue StopCommand = new InstanceValues.Text("stop");
-	
-	private static InstanceValues.Text SayCommand(string message) {
-		return new InstanceValues.Text($"say {message}");
-	}
-	
-	private static InstanceValues.Text SayCountDownAnnouncementCommand(ushort seconds) {
-		return SayCommand("Server shutting down in " + seconds + (seconds == 1 ? " second." : " seconds."));
-	}
-	
-	public InstanceStopRecipe Stop(ushort afterSeconds) {
-		var steps = ImmutableArray.CreateBuilder<IInstanceStopStep>();
-		
-		if (afterSeconds > 0) {
-			steps.Add(new InstanceStopStep.SendToStandardInput(SayCountDownAnnouncementCommand(afterSeconds)));
-			
-			int remainingSeconds = afterSeconds;
-			
-			for (int stopIndex = Array.FindIndex(Stops, stop => stop < afterSeconds); stopIndex != -1 && stopIndex < Stops.Length; stopIndex++) {
-				ushort currentStop = Stops[stopIndex];
-				
-				steps.Add(new InstanceStopStep.Wait(TimeSpan.FromSeconds(remainingSeconds - currentStop)));
-				steps.Add(new InstanceStopStep.SendToStandardInput(SayCountDownAnnouncementCommand(currentStop)));
-				
-				remainingSeconds = currentStop;
-			}
-			
-			steps.Add(new InstanceStopStep.Wait(TimeSpan.FromSeconds(remainingSeconds)));
-		}
-		
-		return new InstanceStopRecipe(steps.ToImmutable(), StopCommand);
-	}
-}

Controller/Phantom.Controller.Minecraft/MinecraftLaunchRecipeCreationFailReason.cs

@@ -1,6 +0,0 @@
-namespace Phantom.Controller.Minecraft;
-
-public enum MinecraftLaunchRecipeCreationFailReason : byte {
-	UnknownError             = 0,
-	MinecraftVersionNotFound = 1,
-}

Controller/Phantom.Controller.Minecraft/MinecraftLaunchRecipes.cs

@@ -1,29 +1,34 @@
 using System.Collections.Immutable;
 using System.Text.RegularExpressions;
-using Phantom.Common.Data;
+using Phantom.Common.Data.Instance;
-using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Instance.Launch;
-using Phantom.Common.Data.Agent.Instance;
+using Phantom.Common.Data.Minecraft;
-using Phantom.Common.Data.Agent.Instance.Launch;
-using Phantom.Common.Data.Web.Instance;
-using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Utils.Collections;
 
 namespace Phantom.Controller.Minecraft;
 
-public sealed partial class MinecraftInstanceRecipes(MinecraftVersions minecraftVersions) {
+public sealed partial class MinecraftLaunchRecipes(MinecraftVersions minecraftVersions) {
-	public async Task<Result<InstanceLaunchRecipe, MinecraftLaunchRecipeCreationFailReason>> Launch(InstanceConfiguration configuration, CancellationToken cancellationToken) {
+	[GeneratedRegex(@"[^a-zA-Z0-9_\-\.]", RegexOptions.Compiled)]
-		string minecraftVersion = configuration.MinecraftVersion;
+	private static partial Regex SanitizePathRegex();
 	
-		var serverExecutableInfo = await minecraftVersions.GetServerExecutableInfo(minecraftVersion, cancellationToken);
+	private static readonly ImmutableDictionary<string, string> Eula = ImmutableDictionary.From([
-		if (serverExecutableInfo == null) {
+		("eula", "true"),
-			return MinecraftLaunchRecipeCreationFailReason.MinecraftVersionNotFound;
+	]);
-		}
+	
+	public async Task<InstanceLaunchRecipe?> Create(InstanceConfiguration configuration, CancellationToken cancellationToken) {
+		string minecraftVersion = configuration.MinecraftVersion;
+		string minecraftVersionPath = SanitizePathRegex().IsMatch(minecraftVersion) ? SanitizePathRegex().Replace(minecraftVersion, "_") : minecraftVersion;
 		
 		uint maximumHeapSizeMegabvtes = configuration.MemoryAllocation.InMegabytes;
 		uint initialHeapSizeMegabytes = maximumHeapSizeMegabvtes / 2;
 		
-		var minecraftVersionPathSegment = SanitizePath(minecraftVersion);
+		var serverExecutableInfo = await minecraftVersions.GetServerExecutableInfo(minecraftVersion, cancellationToken);
-		var vanillaJarFilePath = new InstancePath.Global(["minecraft", "vanilla", minecraftVersionPathSegment, "server.jar"]);
+		if (serverExecutableInfo == null) {
+			// TODO
+			return null;
+		}
+		
+		var vanillaJarFilePath = new InstancePath.Global(["minecraft", "vanilla", minecraftVersionPath, "server.jar"]);
 		
 		var steps = ImmutableArray.CreateBuilder<IInstanceLaunchStep>();
 		
@@ -35,8 +40,12 @@ public sealed partial class MinecraftInstanceRecipes(MinecraftVersions minecraft
 		var additionalJvmArguments = new List<IInstanceValue>();
 		
 		if (configuration.MinecraftServerKind == MinecraftServerKind.Fabric) {
-			var fabricLauncherFilePath = new InstancePath.Global(["minecraft", "fabric", minecraftVersionPathSegment, "loader.jar"]);
+			// TODO customizable loader version
-			AddFabricDownloadStep(steps, configuration, fabricLauncherFilePath);
+			string fabricLauncherUrl = $"https://meta.fabricmc.net/v2/versions/loader/{minecraftVersion}/stable/stable/server/jar";
+			
+			var fabricLauncherFilePath = new InstancePath.Global(["minecraft", "fabric", minecraftVersionPath, "loader.jar"]);
+			
+			steps.Add(new InstanceLaunchStep.DownloadFile(new FileDownloadInfo(fabricLauncherUrl), fabricLauncherFilePath));
 			
 			serverExecutableFilePath = fabricLauncherFilePath;
 			
@@ -49,25 +58,14 @@ public sealed partial class MinecraftInstanceRecipes(MinecraftVersions minecraft
 		return new InstanceLaunchRecipe(steps.ToImmutable(), new InstancePath.Runtime(configuration.JavaRuntimeGuid), [
 			..configuration.JvmArguments.Select(static arg => new InstanceValues.Text(arg)),
 			..additionalJvmArguments,
-			new InstanceValues.Text($"-Xms{initialHeapSizeMegabytes}M"),
+			new InstanceValues.Text("-Xms" + initialHeapSizeMegabytes + "M"),
-			new InstanceValues.Text($"-Xmx{maximumHeapSizeMegabvtes}M"),
+			new InstanceValues.Text("-Xmx" + maximumHeapSizeMegabvtes + "M"),
 			new InstanceValues.Text("-jar"),
 			new InstanceValues.Path(serverExecutableFilePath),
 			new InstanceValues.Text("-nogui"),
 		]);
 	}
 	
-	[GeneratedRegex(@"[^a-zA-Z0-9_\-\.]", RegexOptions.Compiled)]
-	private static partial Regex SanitizePathRegex();
-	
-	private static string SanitizePath(string path) {
-		return SanitizePathRegex().IsMatch(path) ? SanitizePathRegex().Replace(path, "_") : path;
-	}
-	
-	private static readonly ImmutableDictionary<string, string> Eula = ImmutableDictionary.From([
-		("eula", "true"),
-	]);
-	
 	private static ImmutableDictionary<string, string> ServerProperties(InstanceConfiguration configuration) {
 		return ImmutableDictionary.From([
 			("server-port", configuration.ServerPort.ToString()),
@@ -76,11 +74,4 @@ public sealed partial class MinecraftInstanceRecipes(MinecraftVersions minecraft
 			("sync-chunk-writes", "false"),
 		]);
 	}
-	
-	private static void AddFabricDownloadStep(ImmutableArray<IInstanceLaunchStep>.Builder steps, InstanceConfiguration configuration, InstancePath.Global filePath) {
-		// TODO customizable loader version
-		string minecraftVersion = configuration.MinecraftVersion;
-		string fabricLauncherUrl = $"https://meta.fabricmc.net/v2/versions/loader/{minecraftVersion}/stable/stable/server/jar";
-		steps.Add(new InstanceLaunchStep.DownloadFile(new FileDownloadInfo(fabricLauncherUrl), filePath));
-	}
 }

Controller/Phantom.Controller.Minecraft/MinecraftVersionApi.cs

@@ -1,8 +1,7 @@
 using System.Collections.Immutable;
 using System.Net.Http.Json;
 using System.Text.Json;
-using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Minecraft;
-using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Utils.Cryptography;
 using Phantom.Utils.Logging;
 using Phantom.Utils.Runtime;

Controller/Phantom.Controller.Minecraft/MinecraftVersions.cs

@@ -1,7 +1,6 @@
 using System.Collections.Immutable;
 using System.Diagnostics;
-using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Minecraft;
-using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Utils.Logging;
 using Serilog;
 
@@ -35,7 +34,7 @@ public sealed class MinecraftVersions : IDisposable {
 		return versions;
 	}
 	
-	internal async Task<FileDownloadInfo?> GetServerExecutableInfo(string version, CancellationToken cancellationToken) {
+	public async Task<FileDownloadInfo?> GetServerExecutableInfo(string version, CancellationToken cancellationToken) {
 		var versions = await GetVersions(cancellationToken);
 		return await GetCachedObject(() => cachedServerExecutables.ContainsKey(version), () => cachedServerExecutables[version], v => cachedServerExecutables[version] = v, ct => LoadServerExecutableInfo(versions, version, ct), cancellationToken);
 	}
@@ -47,7 +46,7 @@ public sealed class MinecraftVersions : IDisposable {
 			Logger.Information("Refreshed Minecraft {Version} server executable cache, no file found.", version);
 		}
 		else {
-			Logger.Information("Refreshed Minecraft {Version} server executable cache, found file: {Url}.", version, info.Url);
+			Logger.Information("Refreshed Minecraft {Version} server executable cache, found file: {Url}.", version, info.DownloadUrl);
 		}
 		
 		return info;

Controller/Phantom.Controller.Minecraft/Phantom.Controller.Minecraft.csproj

@@ -6,8 +6,7 @@
   </PropertyGroup>
   
   <ItemGroup>
-    <ProjectReference Include="..\..\Common\Phantom.Common.Data.Agent\Phantom.Common.Data.Agent.csproj" />
+    <ProjectReference Include="..\..\Common\Phantom.Common.Data\Phantom.Common.Data.csproj" />
-    <ProjectReference Include="..\..\Common\Phantom.Common.Data.Web\Phantom.Common.Data.Web.csproj" />
     <ProjectReference Include="..\..\Utils\Phantom.Utils\Phantom.Utils.csproj" />
     <ProjectReference Include="..\..\Utils\Phantom.Utils.Logging\Phantom.Utils.Logging.csproj" />
   </ItemGroup>

Controller/Phantom.Controller.Services/Agents/AgentActor.cs

@@ -2,13 +2,15 @@
 using Akka.Actor;
 using Microsoft.EntityFrameworkCore;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Agent.Instance.Launch;
+using Phantom.Common.Data.Agent;
 using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Instance.Launch;
 using Phantom.Common.Data.Java;
+using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Data.Web.Agent;
 using Phantom.Common.Data.Web.Instance;
-using Phantom.Common.Data.Web.Java;
+using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Common.Messages.Agent;
 using Phantom.Common.Messages.Agent.ToAgent;
 using Phantom.Controller.Database;
@@ -40,7 +42,7 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 		AgentRuntimeInfo AgentRuntimeInfo,
 		AgentConnectionKeys AgentConnectionKeys,
 		ControllerState ControllerState,
-		MinecraftInstanceRecipes MinecraftInstanceRecipes,
+		MinecraftLaunchRecipes LaunchRecipes,
 		IDbContextProvider DbProvider,
 		CancellationToken CancellationToken
 	);
@@ -53,7 +55,7 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 	
 	private readonly AgentConnectionKeys agentConnectionKeys;
 	private readonly ControllerState controllerState;
-	private readonly MinecraftInstanceRecipes minecraftInstanceRecipes;
+	private readonly MinecraftLaunchRecipes launchRecipes;
 	private readonly IDbContextProvider dbProvider;
 	private readonly CancellationToken cancellationToken;
 	
@@ -94,7 +96,7 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 	private AgentActor(Init init) {
 		this.agentConnectionKeys = init.AgentConnectionKeys;
 		this.controllerState = init.ControllerState;
-		this.minecraftInstanceRecipes = init.MinecraftInstanceRecipes;
+		this.launchRecipes = init.LaunchRecipes;
 		this.dbProvider = init.DbProvider;
 		this.cancellationToken = init.CancellationToken;
 		
@@ -189,15 +191,17 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 		var configurationMessages = ImmutableArray.CreateBuilder<ConfigureInstanceMessage>();
 		
 		foreach (var (instanceGuid, instanceConfiguration, _, _, launchAutomatically) in instanceDataByGuid.Values.ToImmutableArray()) {
-			var launchRecipe = await minecraftInstanceRecipes.Launch(instanceConfiguration, cancellationToken);
+			var launchRecipe = await CreateInstanceLaunchRecipe(instanceConfiguration);
-			var stopRecipe = minecraftInstanceRecipes.Stop(0);
+			configurationMessages.Add(new ConfigureInstanceMessage(instanceGuid, instanceConfiguration.AsInfo, launchRecipe, launchAutomatically));
-			var configurationMessage = new ConfigureInstanceMessage(instanceGuid, instanceConfiguration.AsInfo, launchRecipe.OrElse(null), launchAutomatically, stopRecipe);
-			configurationMessages.Add(configurationMessage);
 		}
 		
 		return configurationMessages.ToImmutable();
 	}
 	
+	private async Task<InstanceLaunchRecipe?> CreateInstanceLaunchRecipe(InstanceConfiguration configuration) {
+		return await launchRecipes.Create(configuration, cancellationToken);
+	}
+	
 	public interface ICommand;
 	
 	private sealed record InitializeCommand : ICommand;
@@ -226,7 +230,7 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 	
 	public sealed record LaunchInstanceCommand(Guid LoggedInUserGuid, Guid InstanceGuid) : ICommand, ICanReply<Result<LaunchInstanceResult, InstanceActionFailure>>;
 	
-	public sealed record StopInstanceCommand(Guid LoggedInUserGuid, Guid InstanceGuid, ushort AfterSeconds) : ICommand, ICanReply<Result<StopInstanceResult, InstanceActionFailure>>;
+	public sealed record StopInstanceCommand(Guid LoggedInUserGuid, Guid InstanceGuid, MinecraftStopStrategy StopStrategy) : ICommand, ICanReply<Result<StopInstanceResult, InstanceActionFailure>>;
 	
 	public sealed record SendCommandToInstanceCommand(Guid LoggedInUserGuid, Guid InstanceGuid, string Command) : ICommand, ICanReply<Result<SendCommandToInstanceResult, InstanceActionFailure>>;
 	
@@ -341,35 +345,30 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 			return Task.FromResult<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>>(CreateOrUpdateInstanceResult.InstanceMemoryMustNotBeZero);
 		}
 		
-		return minecraftInstanceRecipes.Launch(instanceConfiguration, cancellationToken)
+		return launchRecipes.Create(instanceConfiguration, cancellationToken)
 		                    .ContinueOnActor(CreateOrUpdateInstance1, command)
 		                    .Unwrap();
 	}
 	
-	private Task<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>> CreateOrUpdateInstance1(Result<InstanceLaunchRecipe, MinecraftLaunchRecipeCreationFailReason> launchRecipe, CreateOrUpdateInstanceCommand command) {
+	private Task<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>> CreateOrUpdateInstance1(InstanceLaunchRecipe? launchRecipe, CreateOrUpdateInstanceCommand command) {
-		if (!launchRecipe) {
+		if (launchRecipe == null) {
-			return Task.FromResult<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>>(launchRecipe.Error switch {
+			// TODO
-				MinecraftLaunchRecipeCreationFailReason.MinecraftVersionNotFound => CreateOrUpdateInstanceResult.MinecraftVersionNotFound,
+			return Task.FromResult<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>>(CreateOrUpdateInstanceResult.MinecraftVersionDownloadInfoNotFound);
-				_                                                                => CreateOrUpdateInstanceResult.UnknownError,
-			});
 		}
 		
-		var instanceGuid = command.InstanceGuid;
 		var instanceConfiguration = command.Configuration;
 		
-		bool isCreatingInstance = !instanceActorByGuid.TryGetValue(instanceGuid, out var instanceActorRef);
+		bool isCreatingInstance = !instanceActorByGuid.TryGetValue(command.InstanceGuid, out var instanceActorRef);
 		if (isCreatingInstance) {
-			instanceActorRef = CreateNewInstance(Instance.Offline(instanceGuid, instanceConfiguration));
+			instanceActorRef = CreateNewInstance(Instance.Offline(command.InstanceGuid, instanceConfiguration));
 		}
 		
-		var stopRecipe = minecraftInstanceRecipes.Stop(afterSeconds: 0);
+		var configureInstanceCommand = new InstanceActor.ConfigureInstanceCommand(command.LoggedInUserGuid, command.InstanceGuid, instanceConfiguration, launchRecipe, isCreatingInstance);
-		var configureInstanceCommand = new InstanceActor.ConfigureInstanceCommand(command.LoggedInUserGuid, instanceGuid, instanceConfiguration, launchRecipe.Value, stopRecipe, isCreatingInstance);
 		
 		return instanceActorRef.Request(configureInstanceCommand, cancellationToken)
 		                       .ContinueOnActor(CreateOrUpdateInstance2, configureInstanceCommand);
 	}
 	
-	#pragma warning disable CA2254
 	private Result<CreateOrUpdateInstanceResult, InstanceActionFailure> CreateOrUpdateInstance2(Result<ConfigureInstanceResult, InstanceActionFailure> result, InstanceActor.ConfigureInstanceCommand command) {
 		var instanceGuid = command.InstanceGuid;
 		var instanceName = command.Configuration.InstanceName;
@@ -389,7 +388,6 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 			return CreateOrUpdateInstanceResult.UnknownError;
 		}
 	}
-	#pragma warning restore CA2254
 	
 	private void UpdateInstanceStatus(UpdateInstanceStatusCommand command) {
 		TellInstance(command.InstanceGuid, new InstanceActor.SetStatusCommand(command.Status));
@@ -404,8 +402,7 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 	}
 	
 	private Task<Result<StopInstanceResult, InstanceActionFailure>> StopInstance(StopInstanceCommand command) {
-		var stopRecipe = minecraftInstanceRecipes.Stop(command.AfterSeconds);
+		return RequestInstance<InstanceActor.StopInstanceCommand, StopInstanceResult>(command.InstanceGuid, new InstanceActor.StopInstanceCommand(command.LoggedInUserGuid, command.StopStrategy));
-		return RequestInstance<InstanceActor.StopInstanceCommand, StopInstanceResult>(command.InstanceGuid, new InstanceActor.StopInstanceCommand(command.LoggedInUserGuid, stopRecipe));
 	}
 	
 	private Task<Result<SendCommandToInstanceResult, InstanceActionFailure>> SendMinecraftCommand(SendCommandToInstanceCommand command) {

Controller/Phantom.Controller.Services/Agents/AgentManager.cs

@@ -21,7 +21,7 @@ sealed class AgentManager(
 	IActorRefFactory actorSystem,
 	AgentConnectionKeys agentConnectionKeys,
 	ControllerState controllerState,
-	MinecraftInstanceRecipes minecraftInstanceRecipes,
+	MinecraftLaunchRecipes launchRecipes,
 	IDbContextProvider dbProvider,
 	CancellationToken cancellationToken
 ) {
@@ -42,7 +42,7 @@ sealed class AgentManager(
 	}
 	
 	private bool AddAgent(Guid? loggedInUserGuid, Guid agentGuid, AgentConfiguration configuration, AuthSecret authSecret, AgentRuntimeInfo runtimeInfo) {
-		var init = new AgentActor.Init(loggedInUserGuid, agentGuid, configuration, authSecret, runtimeInfo, agentConnectionKeys, controllerState, minecraftInstanceRecipes, dbProvider, cancellationToken);
+		var init = new AgentActor.Init(loggedInUserGuid, agentGuid, configuration, authSecret, runtimeInfo, agentConnectionKeys, controllerState, launchRecipes, dbProvider, cancellationToken);
 		var name = "Agent:" + agentGuid;
 		return agentsByAgentGuid.TryAdd(agentGuid, actorSystem.ActorOf(AgentActor.Factory(init), name));
 	}
@@ -52,11 +52,8 @@ sealed class AgentManager(
 			return null;
 		}
 		
-		var agentInfo = registration.AgentInfo;
+		var runtimeInfo = AgentRuntimeInfo.From(registration.AgentInfo);
-		var agentVersionInfo = new AgentVersionInfo(agentInfo.ProtocolVersion, agentInfo.BuildVersion);
+		return await agentActor.Request(new AgentActor.RegisterCommand(runtimeInfo, registration.JavaRuntimes), cancellationToken);
-		var agentRuntimeInfo = new AgentRuntimeInfo(agentVersionInfo, agentInfo.MaxInstances, agentInfo.MaxMemory, agentInfo.AllowedServerPorts, agentInfo.AllowedAdditionalPorts);
-		
-		return await agentActor.Request(new AgentActor.RegisterCommand(agentRuntimeInfo, registration.JavaRuntimes), cancellationToken);
 	}
 	
 	public async Task<AuthSecret?> GetAgentAuthSecret(Guid agentGuid) {

Controller/Phantom.Controller.Services/ControllerServices.cs

@@ -20,7 +20,7 @@ public sealed class ControllerServices : IDisposable {
 	
 	private ControllerState ControllerState { get; }
 	private MinecraftVersions MinecraftVersions { get; }
-	private MinecraftInstanceRecipes MinecraftInstanceRecipes { get; }
+	private MinecraftLaunchRecipes LaunchRecipes { get; }
 	
 	private AuthenticatedUserCache AuthenticatedUserCache { get; }
 	private UserManager UserManager { get; }
@@ -51,7 +51,7 @@ public sealed class ControllerServices : IDisposable {
 		
 		this.ControllerState = new ControllerState();
 		this.MinecraftVersions = new MinecraftVersions();
-		this.MinecraftInstanceRecipes = new MinecraftInstanceRecipes(MinecraftVersions);
+		this.LaunchRecipes = new MinecraftLaunchRecipes(MinecraftVersions);
 		
 		this.AuthenticatedUserCache = new AuthenticatedUserCache();
 		this.UserManager = new UserManager(AuthenticatedUserCache, ControllerState, dbProvider);
@@ -60,7 +60,7 @@ public sealed class ControllerServices : IDisposable {
 		this.UserLoginManager = new UserLoginManager(AuthenticatedUserCache, dbProvider);
 		this.PermissionManager = new PermissionManager(dbProvider);
 		
-		this.AgentManager = new AgentManager(ActorSystem, new AgentConnectionKeys(agentCertificateThumbprint), ControllerState, MinecraftInstanceRecipes, dbProvider, cancellationToken);
+		this.AgentManager = new AgentManager(ActorSystem, new AgentConnectionKeys(agentCertificateThumbprint), ControllerState, LaunchRecipes, dbProvider, cancellationToken);
 		this.InstanceLogManager = new InstanceLogManager();
 		
 		this.AuditLogManager = new AuditLogManager(dbProvider);