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_RCON_PORTS" value="25575,27000,27001" />
+      <env name="ALLOWED_ADDITIONAL_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="net9.0" />
+    <option name="PROJECT_TFM" value="net10.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_RCON_PORTS" value="27002-27006" />
+      <env name="ALLOWED_ADDITIONAL_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="net9.0" />
+    <option name="PROJECT_TFM" value="net10.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_RCON_PORTS" value="27007" />
+      <env name="ALLOWED_ADDITIONAL_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="net9.0" />
+    <option name="PROJECT_TFM" value="net10.0" />
     <method v="2">
       <option name="Build" />
     </method>

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

@@ -30,7 +30,7 @@ static class JavaPropertiesStream {
 		}
 		
 		public Reader(string path) {
-			this.reader = new StreamReader(path, Encoding, detectEncodingFromByteOrderMarks: false, CreateFileStreamOptions(FileMode.Open, FileAccess.Read));
+			this.reader = new StreamReader(path, Encoding, detectEncodingFromByteOrderMarks: true, CreateFileStreamOptions(FileMode.Open, FileAccess.Read));
 		}
 		
 		public async IAsyncEnumerable<KeyValuePair<string, string>> ReadProperties([EnumeratorCancellation] CancellationToken cancellationToken) {

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

@@ -1,25 +0,0 @@
-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

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

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

@@ -2,9 +2,10 @@
 using Phantom.Agent.Minecraft.Instance;
 using Phantom.Agent.Minecraft.Java;
 using Phantom.Agent.Minecraft.Server;
+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.Common.Data.Instance.Launch;
-using Phantom.Common.Data.Minecraft;
 using Phantom.Utils.Processes;
 using Serilog;
 
@@ -20,19 +21,19 @@ public sealed class InstanceLauncher(
 	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);
+			logger.Error("Could not resolve server executable path");
 			return new LaunchResult.CouldNotFindServerExecutable();
 		}
 		
-		var stepVisitor = new StepExecutor(downloadManager, pathResolver, reportStatus, cancellationToken);
+		var stepExecutor = 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);
+				await step.Run(stepExecutor);
 			} catch (Exception e) {
-				logger.Error(e, "Failed preparation step {StepIndex} out of {StepCount}: {Step}", stepIndex, steps.Length, step);
+				logger.Error(e, "Failed preparation step {StepIndex} out of {StepCount}: {Step}", stepIndex, steps.Length, step.GetType().Name);
 				return new LaunchResult.CouldNotPrepareServerInstance();
 			}
 		}
@@ -80,6 +81,7 @@ public sealed class InstanceLauncher(
 		public async Task DownloadFile(FileDownloadInfo downloadInfo, IInstancePath path) {
 			string? filePath = path.Resolve(pathResolver);
 			if (filePath == null) {
+				// TODO avoid exc
 				throw new FileNotFoundException("Could not resolve path");
 			}
 			

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

@@ -1,7 +0,0 @@
-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

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

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

@@ -1,4 +1,4 @@
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Agent;
 using Phantom.Utils.IO;
 using Phantom.Utils.Logging;
 using Serilog;

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

@@ -1,5 +1,5 @@
 using System.Security.Cryptography;
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Agent;
 using Phantom.Utils.Cryptography;
 using Phantom.Utils.IO;
 using Phantom.Utils.Logging;

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

@@ -2,6 +2,7 @@
 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.Backups;
 using Phantom.Common.Data.Instance;
 using Phantom.Common.Data.Minecraft;

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

@@ -5,8 +5,9 @@ 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.Instance;
-using Phantom.Common.Data.Instance.Launch;
 using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Utils.Actor;
@@ -56,7 +57,7 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 	
 	public interface ICommand;
 	
-	public sealed record ConfigureInstanceCommand(Guid InstanceGuid, InstanceInfo Info, InstanceLaunchRecipe? LaunchRecipe, bool LaunchNow, bool AlwaysReportStatus) : ICommand, ICanReply<Result<ConfigureInstanceResult, InstanceActionFailure>>;
+	public sealed record ConfigureInstanceCommand(Guid InstanceGuid, InstanceInfo InstanceInfo, InstanceLaunchRecipe? LaunchRecipe, bool LaunchNow, bool AlwaysReportStatus) : ICommand, ICanReply<Result<ConfigureInstanceResult, InstanceActionFailure>>;
 	
 	public sealed record LaunchInstanceCommand(Guid InstanceGuid) : ICommand, ICanReply<Result<LaunchInstanceResult, InstanceActionFailure>>;
 	
@@ -68,7 +69,7 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 	
 	private Result<ConfigureInstanceResult, InstanceActionFailure> ConfigureInstance(ConfigureInstanceCommand command) {
 		var instanceGuid = command.InstanceGuid;
-		var instanceInfo = command.Info;
+		var instanceInfo = command.InstanceInfo;
 		var launchRecipe = command.LaunchRecipe;
 		
 		if (instances.TryGetValue(instanceGuid, out var instance)) {
@@ -85,7 +86,8 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 		}
 		else {
 			var instanceLoggerName = PhantomLogger.ShortenGuid(instanceGuid) + "/" + Interlocked.Increment(ref instanceLoggerSequenceId);
-			var instanceProperties = new InstanceProperties(instanceGuid, Path.Combine(agentFolders.InstancesFolderPath, instanceGuid.ToString()));
+			var instanceFolder = 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);
 			
@@ -94,11 +96,10 @@ sealed class InstanceManagerActor : ReceiveActor<InstanceManagerActor.ICommand>
 			instance.Actor.Tell(new InstanceActor.ReportInstanceStatusCommand());
 		}
 		
-		string instanceFolder = instance.Properties.InstanceFolder;
 		try {
-			Directories.Create(instanceFolder, Chmod.URWX_GRX);
+			Directories.Create(instance.Properties.InstanceFolder, Chmod.URWX_GRX);
 		} catch (Exception e) {
-			Logger.Error(e, "Could not create instance folder: {Path}", instanceFolder);
+			Logger.Error(e, "Could not create instance folder: {Path}", instance.Properties.InstanceFolder);
 			return ConfigureInstanceResult.CouldNotCreateInstanceFolder;
 		}
 		

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

@@ -1,30 +1,41 @@
-using System.Collections.Immutable;
+using System.Buffers;
+using System.Collections.Immutable;
 using Phantom.Agent.Minecraft.Instance;
 using Phantom.Agent.Minecraft.Java;
-using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Agent.Instance;
 
 namespace Phantom.Agent.Services.Instances;
 
 sealed class InstancePathResolver(AgentFolders agentFolders, JavaRuntimeRepository javaRuntimeRepository, InstanceProperties instanceProperties) : IInstancePathResolver {
-	public string Global(ImmutableArray<string> path) {
+	public string? Global(ImmutableArray<string> segments) {
-		return CombinePath(agentFolders.ServerExecutableFolderPath, path);
+		return ValidateAndCombinePath(agentFolders.ServerExecutableFolderPath, segments);
 	}
 	
-	public string Local(ImmutableArray<string> path) {
+	public string? Local(ImmutableArray<string> segments) {
-		return CombinePath(instanceProperties.InstanceFolder, path);
+		return ValidateAndCombinePath(instanceProperties.InstanceFolder, segments);
 	}
 	
 	public string? Runtime(Guid guid) {
-		if (javaRuntimeRepository.TryGetByGuid(guid, out var runtime)) {
+		return javaRuntimeRepository.TryGetByGuid(guid, out var runtime) ? runtime.ExecutablePath : null;
-			return runtime.ExecutablePath;
-		}
-		else {
-			return null;
-		}
 	}
 	
-	private string CombinePath(string basePath, ImmutableArray<string> additionalParts) {
+	private static readonly SearchValues<char> InvalidPathPartChars = SearchValues.Create([
-		// TODO validation
+		Path.DirectorySeparatorChar,
-		return Path.Combine([basePath, ..additionalParts]);
+		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/InstanceTicketManager.cs

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

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

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

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

@@ -1,8 +1,8 @@
 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.State;
 

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

@@ -1,9 +1,9 @@
 using Phantom.Agent.Minecraft.Instance;
 using Phantom.Agent.Minecraft.Launcher;
 using Phantom.Agent.Services.Backups;
+using Phantom.Common.Data.Agent.Instance;
 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;

Agent/Phantom.Agent/Variables.cs

@@ -43,7 +43,7 @@ sealed record Variables(
 		try {
 			return LoadOrThrow();
 		} catch (Exception e) {
-			PhantomLogger.Root.Fatal("{}", e.Message);
+			PhantomLogger.Root.Fatal("{Error}", e.Message);
 			throw StopProcedureException.Instance;
 		}
 	}

Common/Phantom.Common.Data.Agent/FileDownloadInfo.cs

@@ -1,7 +1,7 @@
 using MemoryPack;
 using Phantom.Utils.Cryptography;
 
-namespace Phantom.Common.Data.Minecraft;
+namespace Phantom.Common.Data.Agent;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial class FileDownloadInfo {

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

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

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

@@ -0,0 +1,9 @@
+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/IInstanceValue.cs

@@ -2,13 +2,12 @@
 using System.Text;
 using MemoryPack;
 
-namespace Phantom.Common.Data.Instance;
+namespace Phantom.Common.Data.Agent.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);
 }
@@ -45,13 +44,4 @@ public static partial class InstanceValues {
 			return resolver.Path(Value);
 		}
 	}
-	
-	[MemoryPackable]
-	public sealed partial record Variable(InstanceVariable Value) : IInstanceValue {
-		public string? Resolve(IInstanceValueResolver resolver) {
-			return resolver.Variable(Value);
 }
-	}
-}
-
-public enum InstanceVariable {}

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

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

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

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

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

@@ -1,8 +1,7 @@
 using System.Collections.Immutable;
 using MemoryPack;
-using Phantom.Common.Data.Minecraft;
 
-namespace Phantom.Common.Data.Instance.Launch;
+namespace Phantom.Common.Data.Agent.Instance.Launch;
 
 [MemoryPackable]
 [MemoryPackUnion(tag: 0, typeof(InstanceLaunchStep.DownloadFile))]

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

@@ -1,7 +1,7 @@
 using System.Collections.Immutable;
 using Phantom.Common.Data.Minecraft;
 
-namespace Phantom.Common.Data.Instance.Launch;
+namespace Phantom.Common.Data.Agent.Instance.Launch;
 
 public interface IInstanceLaunchStepExecutor {
 	Task DownloadFile(FileDownloadInfo downloadInfo, IInstancePath path);

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

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

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

@@ -0,0 +1,16 @@
+<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,6 +1,5 @@
 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,5 +1,4 @@
 using MemoryPack;
-using Phantom.Common.Data.Agent;
 
 namespace Phantom.Common.Data.Web.Agent;
 
@@ -10,8 +9,4 @@ 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/Agent/AgentStats.cs

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

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

@@ -5,7 +5,7 @@ public enum CreateOrUpdateInstanceResult : byte {
 	Success,
 	InstanceNameMustNotBeEmpty,
 	InstanceMemoryMustNotBeZero,
-	MinecraftVersionDownloadInfoNotFound,
+	MinecraftVersionNotFound,
 	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.MinecraftVersionDownloadInfoNotFound => "Could not find download information for the selected Minecraft version.",
+			CreateOrUpdateInstanceResult.MinecraftVersionNotFound    => "Minecraft version not found.",
 			CreateOrUpdateInstanceResult.AgentNotFound               => "Agent not found.",
 			_                                                        => "Unknown error.",
 		};

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

@@ -1,8 +1,8 @@
 using System.Collections.Immutable;
 using MemoryPack;
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Web.Minecraft;
 
-namespace Phantom.Common.Data.Instance;
+namespace Phantom.Common.Data.Web.Instance;
 
 [MemoryPackable(GenerateType.VersionTolerant)]
 public sealed partial record InstanceConfiguration(
@@ -15,7 +15,4 @@ 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.Web/Minecraft/MinecraftServerKind.cs

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

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

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

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

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

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

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

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

@@ -1,9 +0,0 @@
-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,5 +1,4 @@
 using MemoryPack;
-using Phantom.Common.Data.Instance.Launch;
 
 namespace Phantom.Common.Data.Instance;
 

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

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

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

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

Common/Phantom.Common.Data/Result.cs

@@ -1,6 +1,5 @@
 using System.Diagnostics.CodeAnalysis;
 using MemoryPack;
-using Phantom.Utils.Monads;
 using Phantom.Utils.Result;
 
 namespace Phantom.Common.Data;
@@ -25,9 +24,6 @@ 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;
@@ -38,6 +34,11 @@ 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\Phantom.Common.Data.csproj" />
+    <ProjectReference Include="..\Phantom.Common.Data.Agent\Phantom.Common.Data.Agent.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,7 +1,7 @@
 using MemoryPack;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Agent.Instance;
-using Phantom.Common.Data.Instance.Launch;
+using Phantom.Common.Data.Agent.Instance.Launch;
 using Phantom.Common.Data.Replies;
 using Phantom.Utils.Actor;
 

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

@@ -1,7 +1,6 @@
 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.Minecraft;
+using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Utils.Actor;
 
 namespace Phantom.Common.Messages.Web.ToController;

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.Minecraft;
+using Phantom.Common.Data.Web.Minecraft;
 
 namespace Phantom.Controller.Database.Entities;
 

Controller/Phantom.Controller.Minecraft/MinecraftLaunchRecipeCreationFailReason.cs

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

Controller/Phantom.Controller.Minecraft/MinecraftLaunchRecipes.cs

@@ -1,34 +1,33 @@
 using System.Collections.Immutable;
 using System.Text.RegularExpressions;
-using Phantom.Common.Data.Instance;
+using Phantom.Common.Data;
-using Phantom.Common.Data.Instance.Launch;
+using Phantom.Common.Data.Agent;
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Agent.Instance;
+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 MinecraftLaunchRecipes(MinecraftVersions minecraftVersions) {
-	[GeneratedRegex(@"[^a-zA-Z0-9_\-\.]", RegexOptions.Compiled)]
-	private static partial Regex SanitizePathRegex();
-	
 	private static readonly ImmutableDictionary<string, string> Eula = ImmutableDictionary.From([
 		("eula", "true"),
 	]);
 	
-	public async Task<InstanceLaunchRecipe?> Create(InstanceConfiguration configuration, CancellationToken cancellationToken) {
+	public async Task<Result<InstanceLaunchRecipe, MinecraftLaunchRecipeCreationFailReason>> Create(InstanceConfiguration configuration, CancellationToken cancellationToken) {
 		string minecraftVersion = configuration.MinecraftVersion;
-		string minecraftVersionPath = SanitizePathRegex().IsMatch(minecraftVersion) ? SanitizePathRegex().Replace(minecraftVersion, "_") : minecraftVersion;
+		
+		var serverExecutableInfo = await minecraftVersions.GetServerExecutableInfo(minecraftVersion, cancellationToken);
+		if (serverExecutableInfo == null) {
+			return MinecraftLaunchRecipeCreationFailReason.MinecraftVersionNotFound;
+		}
 		
 		uint maximumHeapSizeMegabvtes = configuration.MemoryAllocation.InMegabytes;
 		uint initialHeapSizeMegabytes = maximumHeapSizeMegabvtes / 2;
 		
-		var serverExecutableInfo = await minecraftVersions.GetServerExecutableInfo(minecraftVersion, cancellationToken);
+		var minecraftVersionPathSegment = SanitizePath(minecraftVersion);
-		if (serverExecutableInfo == null) {
+		var vanillaJarFilePath = new InstancePath.Global(["minecraft", "vanilla", minecraftVersionPathSegment, "server.jar"]);
-			// TODO
-			return null;
-		}
-		
-		var vanillaJarFilePath = new InstancePath.Global(["minecraft", "vanilla", minecraftVersionPath, "server.jar"]);
 		
 		var steps = ImmutableArray.CreateBuilder<IInstanceLaunchStep>();
 		
@@ -40,12 +39,8 @@ public sealed partial class MinecraftLaunchRecipes(MinecraftVersions minecraftVe
 		var additionalJvmArguments = new List<IInstanceValue>();
 		
 		if (configuration.MinecraftServerKind == MinecraftServerKind.Fabric) {
-			// TODO customizable loader version
+			var fabricLauncherFilePath = new InstancePath.Global(["minecraft", "fabric", minecraftVersionPathSegment, "loader.jar"]);
-			string fabricLauncherUrl = $"https://meta.fabricmc.net/v2/versions/loader/{minecraftVersion}/stable/stable/server/jar";
+			AddFabricDownloadStep(steps, configuration, fabricLauncherFilePath);
-			
-			var fabricLauncherFilePath = new InstancePath.Global(["minecraft", "fabric", minecraftVersionPath, "loader.jar"]);
-			
-			steps.Add(new InstanceLaunchStep.DownloadFile(new FileDownloadInfo(fabricLauncherUrl), fabricLauncherFilePath));
 			
 			serverExecutableFilePath = fabricLauncherFilePath;
 			
@@ -66,6 +61,13 @@ public sealed partial class MinecraftLaunchRecipes(MinecraftVersions minecraftVe
 		]);
 	}
 	
+	[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 ImmutableDictionary<string, string> ServerProperties(InstanceConfiguration configuration) {
 		return ImmutableDictionary.From([
 			("server-port", configuration.ServerPort.ToString()),
@@ -74,4 +76,11 @@ public sealed partial class MinecraftLaunchRecipes(MinecraftVersions minecraftVe
 			("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,7 +1,8 @@
 using System.Collections.Immutable;
 using System.Net.Http.Json;
 using System.Text.Json;
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Agent;
+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,6 +1,7 @@
 using System.Collections.Immutable;
 using System.Diagnostics;
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Utils.Logging;
 using Serilog;
 
@@ -34,7 +35,7 @@ public sealed class MinecraftVersions : IDisposable {
 		return versions;
 	}
 	
-	public async Task<FileDownloadInfo?> GetServerExecutableInfo(string version, CancellationToken cancellationToken) {
+	internal 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);
 	}

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

@@ -6,7 +6,8 @@
   </PropertyGroup>
   
   <ItemGroup>
-    <ProjectReference Include="..\..\Common\Phantom.Common.Data\Phantom.Common.Data.csproj" />
+    <ProjectReference Include="..\..\Common\Phantom.Common.Data.Agent\Phantom.Common.Data.Agent.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,9 +2,8 @@
 using Akka.Actor;
 using Microsoft.EntityFrameworkCore;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Agent;
+using Phantom.Common.Data.Agent.Instance.Launch;
 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;
@@ -191,17 +190,14 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 		var configurationMessages = ImmutableArray.CreateBuilder<ConfigureInstanceMessage>();
 		
 		foreach (var (instanceGuid, instanceConfiguration, _, _, launchAutomatically) in instanceDataByGuid.Values.ToImmutableArray()) {
-			var launchRecipe = await CreateInstanceLaunchRecipe(instanceConfiguration);
+			var launchRecipe = await launchRecipes.Create(instanceConfiguration, cancellationToken);
-			configurationMessages.Add(new ConfigureInstanceMessage(instanceGuid, instanceConfiguration.AsInfo, launchRecipe, launchAutomatically));
+			var configurationMessage = new ConfigureInstanceMessage(instanceGuid, instanceConfiguration.AsInfo, launchRecipe.OrElse(null), launchAutomatically);
+			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;
@@ -350,25 +346,29 @@ sealed class AgentActor : ReceiveActor<AgentActor.ICommand>, IWithTimers {
 		                    .Unwrap();
 	}
 	
-	private Task<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>> CreateOrUpdateInstance1(InstanceLaunchRecipe? launchRecipe, CreateOrUpdateInstanceCommand command) {
+	private Task<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>> CreateOrUpdateInstance1(Result<InstanceLaunchRecipe, MinecraftLaunchRecipeCreationFailReason> launchRecipe, CreateOrUpdateInstanceCommand command) {
-		if (launchRecipe == null) {
+		if (!launchRecipe) {
-			// TODO
+			return Task.FromResult<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>>(launchRecipe.Error switch {
-			return Task.FromResult<Result<CreateOrUpdateInstanceResult, InstanceActionFailure>>(CreateOrUpdateInstanceResult.MinecraftVersionDownloadInfoNotFound);
+				MinecraftLaunchRecipeCreationFailReason.MinecraftVersionNotFound => CreateOrUpdateInstanceResult.MinecraftVersionNotFound,
+				_                                                                => CreateOrUpdateInstanceResult.UnknownError,
+			});
 		}
 		
+		var instanceGuid = command.InstanceGuid;
 		var instanceConfiguration = command.Configuration;
 		
-		bool isCreatingInstance = !instanceActorByGuid.TryGetValue(command.InstanceGuid, out var instanceActorRef);
+		bool isCreatingInstance = !instanceActorByGuid.TryGetValue(instanceGuid, out var instanceActorRef);
 		if (isCreatingInstance) {
-			instanceActorRef = CreateNewInstance(Instance.Offline(command.InstanceGuid, instanceConfiguration));
+			instanceActorRef = CreateNewInstance(Instance.Offline(instanceGuid, instanceConfiguration));
 		}
 		
-		var configureInstanceCommand = new InstanceActor.ConfigureInstanceCommand(command.LoggedInUserGuid, command.InstanceGuid, instanceConfiguration, launchRecipe, isCreatingInstance);
+		var configureInstanceCommand = new InstanceActor.ConfigureInstanceCommand(command.LoggedInUserGuid, instanceGuid, instanceConfiguration, launchRecipe.Value, 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;
@@ -388,6 +388,7 @@ 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));

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

@@ -52,8 +52,11 @@ sealed class AgentManager(
 			return null;
 		}
 		
-		var runtimeInfo = AgentRuntimeInfo.From(registration.AgentInfo);
+		var agentInfo = registration.AgentInfo;
-		return await agentActor.Request(new AgentActor.RegisterCommand(runtimeInfo, registration.JavaRuntimes), cancellationToken);
+		var agentVersionInfo = new AgentVersionInfo(agentInfo.ProtocolVersion, agentInfo.BuildVersion);
+		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/Instances/InstanceActor.cs

@@ -1,6 +1,6 @@
 using Phantom.Common.Data;
+using Phantom.Common.Data.Agent.Instance.Launch;
 using Phantom.Common.Data.Instance;
-using Phantom.Common.Data.Instance.Launch;
 using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Data.Web.Instance;

Controller/Phantom.Controller.Services/Instances/InstanceDatabaseStorageActor.cs

@@ -1,5 +1,5 @@
-using Phantom.Common.Data.Instance;
+using Phantom.Common.Data.Minecraft;
-using Phantom.Common.Data.Minecraft;
+using Phantom.Common.Data.Web.Instance;
 using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Controller.Database;
 using Phantom.Controller.Database.Entities;

Controller/Phantom.Controller.Services/Instances/InstanceExtensions.cs

@@ -0,0 +1,10 @@
+using Phantom.Common.Data.Agent.Instance;
+using Phantom.Common.Data.Web.Instance;
+
+namespace Phantom.Controller.Services.Instances;
+
+static class InstanceExtensions {
+	extension(InstanceConfiguration configuration) {
+		public InstanceInfo AsInfo => new (configuration.InstanceName, configuration.ServerPort, [configuration.RconPort], configuration.MemoryAllocation);
+	}
+}

Controller/Phantom.Controller.Services/Rpc/WebMessageHandlerActor.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;
 using Phantom.Common.Messages.Web.ToController;

PhantomPanel.sln

@@ -26,6 +26,8 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Phantom.Agent.Services", "A
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Phantom.Common.Data", "Common\Phantom.Common.Data\Phantom.Common.Data.csproj", "{6C3DB1E5-F695-4D70-8F3A-78C2957274BE}"
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Phantom.Common.Data.Agent", "Common\Phantom.Common.Data.Agent\Phantom.Common.Data.Agent.csproj", "{6EB98B08-19B2-4726-B313-19C4FDA6F89F}"
+EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Phantom.Common.Data.Tests", "Common\Phantom.Common.Data.Tests\Phantom.Common.Data.Tests.csproj", "{435D7981-DFDA-46A0-8CD8-CD8C117935D7}"
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Phantom.Common.Data.Web", "Common\Phantom.Common.Data.Web\Phantom.Common.Data.Web.csproj", "{BC969D0B-0019-48E0-9FAF-F5CC906AAF09}"
@@ -90,6 +92,10 @@ Global
 		{6C3DB1E5-F695-4D70-8F3A-78C2957274BE}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{6C3DB1E5-F695-4D70-8F3A-78C2957274BE}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{6C3DB1E5-F695-4D70-8F3A-78C2957274BE}.Release|Any CPU.Build.0 = Release|Any CPU
+		{6EB98B08-19B2-4726-B313-19C4FDA6F89F}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{6EB98B08-19B2-4726-B313-19C4FDA6F89F}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{6EB98B08-19B2-4726-B313-19C4FDA6F89F}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{6EB98B08-19B2-4726-B313-19C4FDA6F89F}.Release|Any CPU.Build.0 = Release|Any CPU
 		{435D7981-DFDA-46A0-8CD8-CD8C117935D7}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
 		{435D7981-DFDA-46A0-8CD8-CD8C117935D7}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{435D7981-DFDA-46A0-8CD8-CD8C117935D7}.Release|Any CPU.ActiveCfg = Release|Any CPU
@@ -169,6 +175,7 @@ Global
 		{065FFFA0-DFF4-43DB-AB3D-B92EE9848DDB} = {94C1E464-3F91-49EA-99FF-3A3082C54CE8}
 		{AEE8B77E-AB07-423F-9981-8CD829ACB834} = {F5878792-64C8-4ECF-A075-66341FF97127}
 		{6C3DB1E5-F695-4D70-8F3A-78C2957274BE} = {01CB1A81-8950-471C-BFDF-F135FDDB2C18}
+		{6EB98B08-19B2-4726-B313-19C4FDA6F89F} = {01CB1A81-8950-471C-BFDF-F135FDDB2C18}
 		{95B55357-F8F0-48C2-A1C2-5EA997651783} = {01CB1A81-8950-471C-BFDF-F135FDDB2C18}
 		{6E798DEB-8921-41A2-8AFB-E4416A9E0704} = {01CB1A81-8950-471C-BFDF-F135FDDB2C18}
 		{435D7981-DFDA-46A0-8CD8-CD8C117935D7} = {D781E00D-8563-4102-A0CD-477A679193B5}

Web/Phantom.Web.Services/Instances/InstanceManager.cs

@@ -1,6 +1,5 @@
 using System.Collections.Immutable;
 using Phantom.Common.Data;
-using Phantom.Common.Data.Instance;
 using Phantom.Common.Data.Minecraft;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Data.Web.Instance;

Web/Phantom.Web/Shared/InstanceAddOrEditForm.razor

@@ -2,9 +2,7 @@
 @using System.ComponentModel.DataAnnotations
 @using System.Diagnostics.CodeAnalysis
 @using Phantom.Common.Data
-@using Phantom.Common.Data.Instance
 @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

Web/Phantom.Web/Shared/InstanceLog.razor

@@ -1,9 +1,9 @@
-@using Phantom.Utils.Collections
+@using System.Diagnostics
-@using Phantom.Utils.Events
-@using System.Diagnostics
-@using Phantom.Web.Services.Instances
 @using Phantom.Common.Data.Web.Users
-@inherits Phantom.Web.Components.PhantomComponent
+@using Phantom.Utils.Collections
+@using Phantom.Utils.Events
+@using Phantom.Web.Services.Instances
+@inherits PhantomComponent
 @inject IJSRuntime Js;
 @inject InstanceLogManager InstanceLogManager;
 

Web/Phantom.Web/Utils/Messages.cs

@@ -1,4 +1,4 @@
-using Phantom.Common.Data.Instance.Launch;
+using Phantom.Common.Data.Instance;
 using Phantom.Common.Data.Replies;
 using Phantom.Common.Data.Web.Minecraft;
 using Phantom.Common.Data.Web.Users;