Change parsing arguments

Author: emoacht

Source/Monitorian.Core/AppKeeper.cs

@@ -37,7 +37,7 @@ public async Task<bool> StartAsync(StartupEventArgs e, IEnumerable<string> addit
 
 		SubscribeExceptions();
 
-		var (success, response) = StartupAgent.Start(ProductInfo.Product, ProductInfo.StartupTaskId, OtherArguments);
+		var (success, response) = StartupAgent.Start(ProductInfo.Product, ProductInfo.StartupTaskId, ForwardingArguments);
 		if (!success && (response is not null))
 		{
 			ConsoleService.WriteLine(response);
@@ -66,6 +66,9 @@ public void Write(string content)
 	public static IReadOnlyList<string> OtherArguments => _otherArguments?.ToArray() ?? [];
 	private static string[] _otherArguments;
 
+	public static IReadOnlyList<string> ForwardingArguments => _forwardingArguments?.ToArray() ?? [];
+	private static string[] _forwardingArguments;
+
 	public static IEnumerable<string> EnumerateStandardOptions() =>
 		new[]
 		{
@@ -78,26 +81,32 @@ public static IEnumerable<string> EnumerateStandardOptions() =>
 
 	private async Task ParseArgumentsAsync(StartupEventArgs e, string[] standardOptions)
 	{
-		// Load persistent arguments.
-		var args = (await LoadArgumentsAsync())?.Split() ?? [];
-
-		// Concatenate current and persistent arguments.
+		// Divide current arguments.
 		// The first element of StartupEventArgs.Args is not executing assembly's path unlike
 		// that of arguments provided by Environment.GetCommandLineArgs method.
-		args = e.Args.Concat(args.Select(x => x.Trim('"'))).ToArray();
-		if (args is not { Length: > 0 })
-			return;
+		var (currentStandard, currentOther) = Divide(e.Args);
 
-		const char optionMark = '/';
-		var isStandard = false;
+		// Divide persistent arguments.
+		var persistentArgs = (await LoadArgumentsAsync())?.Split().Select(x => x.Trim('"'));
+		var (persistentStandard, persistentOther) = Divide(persistentArgs);
 
-		var buffer = args
-			.Where(x => !string.IsNullOrWhiteSpace(x))
-			.GroupBy(x => (x[0] == optionMark) ? (isStandard = standardOptions.Contains(x.ToLower())) : isStandard)
-			.ToArray();
+		_standardArguments = persistentStandard.Concat(currentStandard).ToArray();
+		_forwardingArguments = currentOther.ToArray();
+		_otherArguments = persistentOther.Concat(_forwardingArguments).ToArray();
 
-		_standardArguments = buffer.SingleOrDefault(x => x.Key)?.ToArray();
-		_otherArguments = buffer.SingleOrDefault(x => !x.Key)?.ToArray();
+		(IEnumerable<string> standard, IEnumerable<string> other) Divide(IEnumerable<string> args)
+		{
+			const char optionMark = '/';
+			var isStandard = false;
+
+			var buffer = args
+				.Where(x => !string.IsNullOrWhiteSpace(x))
+				.GroupBy(x => (x[0] == optionMark) ? (isStandard = standardOptions.Contains(x.ToLower())) : isStandard)
+				.ToArray() ?? [];
+
+			return (standard: buffer.SingleOrDefault(x => x.Key) ?? Enumerable.Empty<string>(),
+					other: buffer.SingleOrDefault(x => !x.Key) ?? Enumerable.Empty<string>());
+		}
 	}
 
 	private const string ArgumentsFileName = "arguments.txt";

Source/StartupAgency/PipeHolder.cs

@@ -37,7 +37,7 @@ public PipeHolder(string name, Func<string[], Task<string>> handleRequestAsync)
 	/// <summary>
 	/// Creates <see cref="System.Threading.Semaphore"/> to start named pipes.
 	/// </summary>
-	/// <param name="args">Arguments to another instance</param>
+	/// <param name="args">Arguments being forwarded to another instance</param>
 	/// <returns>
 	///	<para>success: True if no other instance exists and this instance successfully creates</para>
 	/// <para>response: Response from another instance if that instance exists and returns an response</para>

Source/StartupAgency/StartupAgent.cs

@@ -22,7 +22,7 @@ public class StartupAgent : IDisposable
 	/// </summary>
 	/// <param name="name">Name</param>
 	/// <param name="startupTaskId">Startup task ID</param>
-	/// <param name="args">Arguments to another instance</param>
+	/// <param name="args">Arguments being forwarded to another instance</param>
 	/// <returns>
 	/// <para>success: True if no other instance exists and this instance successfully starts</para>
 	/// <para>response: Response from another instance if that instance exists and returns an response</para>