From aaf65fcf09d3a1ca6afc03b9c95cc4fa42479e70 Mon Sep 17 00:00:00 2001
From: Pat Hartl <pat@pathar.tl>
Date: Sat, 27 Sep 2025 16:39:54 -0500
Subject: [PATCH] Runnable launcher using new client

---
 .../ProfileService.cs                         |  2 +-
 LANCommander.Launcher/UI/MainLayout.razor     |  4 ++
 LANCommander.Launcher/wwwroot/index.html      |  2 +-
 LANCommander.SDK/Client.cs                    |  4 +-
 .../IServiceCollectionExtensions.cs           | 38 +++++++++----------
 LANCommander.SDK/Services/ChatClient.cs       |  2 +-
 LANCommander.SDK/Services/ConnectionClient.cs | 17 +++++----
 .../{MediaService.cs => MediaClient.cs}       |  2 +-
 LANCommander.Server.Services/MediaService.cs  |  4 +-
 .../Components/SafeImage/SafeImage.razor      |  4 +-
 10 files changed, 43 insertions(+), 36 deletions(-)
 rename LANCommander.SDK/Services/{MediaService.cs => MediaClient.cs} (98%)

diff --git a/LANCommander.Launcher.Services/ProfileService.cs b/LANCommander.Launcher.Services/ProfileService.cs
index 0d34c823..08985c1e 100644
--- a/LANCommander.Launcher.Services/ProfileService.cs
+++ b/LANCommander.Launcher.Services/ProfileService.cs
@@ -85,7 +85,7 @@ namespace LANCommander.Launcher.Services
                             FileId = Guid.NewGuid(),
                             Type = SDK.Enums.MediaType.Avatar,
                             MimeType = MediaTypeNames.Image.Png,
-                            Crc32 = await SDK.Services.MediaService.CalculateChecksumAsync(tempAvatarPath),
+                            Crc32 = await SDK.Services.MediaClient.CalculateChecksumAsync(tempAvatarPath),
                             UserId = remoteProfile.Id,
                         };
 
diff --git a/LANCommander.Launcher/UI/MainLayout.razor b/LANCommander.Launcher/UI/MainLayout.razor
index 1b8b227a..32afceac 100644
--- a/LANCommander.Launcher/UI/MainLayout.razor
+++ b/LANCommander.Launcher/UI/MainLayout.razor
@@ -1,6 +1,7 @@
 @using System.Management.Automation.Remoting
 @using LANCommander.Launcher.Models
 @using LANCommander.SDK
+@using LANCommander.SDK.Rpc.Client
 @using Photino.Blazor.CustomWindow.Components
 @using ConnectionState = LANCommander.Launcher.Models.ConnectionState
 @inherits LayoutComponentBase
@@ -16,6 +17,7 @@
 @inject ILogger<MainLayout> Logger
 @inject ImportManagerService ImportManagerService
 @inject LocalizationService LocalizationService
+@inject IRpcClient RpcClient
 
     <CustomWindow HeaderHeight="37">
         <HeaderExtraControlsLayout>
@@ -141,6 +143,8 @@
             await ImportManagerService.RequestImport();
             await InvokeAsync(StateHasChanged);
         }
+
+        await RpcClient.ConnectAsync();
         
         _crashQuips = new[]
         {
diff --git a/LANCommander.Launcher/wwwroot/index.html b/LANCommander.Launcher/wwwroot/index.html
index 65faf74d..cfaebaa2 100644
--- a/LANCommander.Launcher/wwwroot/index.html
+++ b/LANCommander.Launcher/wwwroot/index.html
@@ -28,13 +28,13 @@
     </div>
 
     <script src="/_content/AntDesign/js/ant-design-blazor.js"></script>
-    <script src="/_framework/blazor.webview.js"></script>
     <script src="/_content/XtermBlazor/XtermBlazor.min.js"></script>
     <script src="/_content/Photino.Blazor.CustomWindow/js/js-extensions.js"></script>
     <script src="/_content/Photino.Blazor.CustomWindow/js/pb-custom-window.js"></script>
     <script src="/_content/LANCommander.UI/bundle.js"></script>
     <script src="/_content/PSC.Blazor.Components.MarkdownEditor/js/easymde.min.js"></script>
     <script src="/_content/PSC.Blazor.Components.MarkdownEditor/js/markdownEditor.js"></script>
+    <script src="/_framework/blazor.webview.js"></script>
 
     <script>
         XtermBlazor.registerAddons({
diff --git a/LANCommander.SDK/Client.cs b/LANCommander.SDK/Client.cs
index 788dee4c..58a9098f 100644
--- a/LANCommander.SDK/Client.cs
+++ b/LANCommander.SDK/Client.cs
@@ -13,7 +13,7 @@ public class Client(
     LauncherClient launcherClient,
     LibraryClient libraryClient,
     LobbyClient lobbyClient,
-    MediaService mediaClient,
+    MediaClient mediaClient,
     PlaySessionClient playSessionClient,
     ProfileClient profileClient,
     RedistributableClient redistributableClient,
@@ -32,7 +32,7 @@ public class Client(
     public LauncherClient Launcher = launcherClient;
     public LibraryClient Library = libraryClient;
     public LobbyClient Lobbies = lobbyClient;
-    public MediaService Media = mediaClient;
+    public MediaClient Media = mediaClient;
     public PlaySessionClient PlaySessions = playSessionClient;
     public ProfileClient Profile = profileClient;
     public RedistributableClient Redistributables = redistributableClient;
diff --git a/LANCommander.SDK/Extensions/IServiceCollectionExtensions.cs b/LANCommander.SDK/Extensions/IServiceCollectionExtensions.cs
index 7e0e0369..94e61f63 100644
--- a/LANCommander.SDK/Extensions/IServiceCollectionExtensions.cs
+++ b/LANCommander.SDK/Extensions/IServiceCollectionExtensions.cs
@@ -15,29 +15,29 @@ public static class IServiceCollectionExtensions
         services.AddSingleton<ITokenProvider, TokenProvider>();
         services.AddSingleton<INetworkInformationProvider, NetworkInformationProvider>();
         services.AddSingleton<IRpcClient, RpcClient>();
-        services.AddScoped<ApiRequestFactory>();
-        services.AddScoped<ProcessExecutionContextFactory>();
+        services.AddSingleton<ApiRequestFactory>();
+        services.AddSingleton<ProcessExecutionContextFactory>();
 
-        services.AddScoped<AuthenticationClient>();
+        services.AddSingleton<AuthenticationClient>();
         services.AddSingleton<BeaconClient>();
-        services.AddScoped<ChatClient>();
+        services.AddSingleton<ChatClient>();
         services.AddSingleton<IConnectionClient, ConnectionClient>();
-        services.AddScoped<DepotClient>();
-        services.AddScoped<GameClient>();
-        services.AddScoped<IssueClient>();
-        services.AddScoped<LauncherClient>();
-        services.AddScoped<LibraryClient>();
-        services.AddScoped<LobbyClient>();
-        services.AddScoped<MediaService>();
-        services.AddScoped<PlaySessionClient>();
-        services.AddScoped<ProfileClient>();
-        services.AddScoped<RedistributableClient>();
-        services.AddScoped<SaveClient>();
-        services.AddScoped<ScriptClient>();
-        services.AddScoped<ServerClient>();
-        services.AddScoped<TagClient>();
+        services.AddSingleton<DepotClient>();
+        services.AddSingleton<GameClient>();
+        services.AddSingleton<IssueClient>();
+        services.AddSingleton<LauncherClient>();
+        services.AddSingleton<LibraryClient>();
+        services.AddSingleton<LobbyClient>();
+        services.AddSingleton<MediaClient>();
+        services.AddSingleton<PlaySessionClient>();
+        services.AddSingleton<ProfileClient>();
+        services.AddSingleton<RedistributableClient>();
+        services.AddSingleton<SaveClient>();
+        services.AddSingleton<ScriptClient>();
+        services.AddSingleton<ServerClient>();
+        services.AddSingleton<TagClient>();
 
-        services.AddScoped<Client>();
+        services.AddSingleton<Client>();
         
         return services;
     }
diff --git a/LANCommander.SDK/Services/ChatClient.cs b/LANCommander.SDK/Services/ChatClient.cs
index bafe1ea0..4113f8ab 100644
--- a/LANCommander.SDK/Services/ChatClient.cs
+++ b/LANCommander.SDK/Services/ChatClient.cs
@@ -7,7 +7,7 @@ using LANCommander.SDK.Rpc.Client;
 
 namespace LANCommander.SDK.Services;
 
-public class ChatClient(IRpcClient rpc)
+public class ChatClient
 {
     private readonly Dictionary<Guid, ChatThread> _threads = new();
     public ChatThread GetThread(Guid threadId)
diff --git a/LANCommander.SDK/Services/ConnectionClient.cs b/LANCommander.SDK/Services/ConnectionClient.cs
index 19302b94..77f1f9ea 100644
--- a/LANCommander.SDK/Services/ConnectionClient.cs
+++ b/LANCommander.SDK/Services/ConnectionClient.cs
@@ -6,21 +6,24 @@ using System.Threading.Tasks;
 using LANCommander.SDK.Exceptions;
 using LANCommander.SDK.Extensions;
 using LANCommander.SDK.Factories;
+using LANCommander.SDK.Models;
 using LANCommander.SDK.Rpc;
 using LANCommander.SDK.Rpc.Client;
 using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Options;
 
 namespace LANCommander.SDK.Services;
 
 public class ConnectionClient(
     ILogger<ConnectionClient> logger,
-    IRpcClient rpc) : IConnectionClient
+    IOptions<Settings> settings) : IConnectionClient
 {
-    private Uri _serverAddress;
+    private Uri _serverAddress = settings.Value.Authentication.ServerAddress;
     
     public bool IsConnected()
     {
-        return rpc.IsConnected();
+        return true;
+        //return rpc.IsConnected();
     }
 
     public Uri GetServerAddress() => _serverAddress;
@@ -54,7 +57,7 @@ public class ConnectionClient(
 
                     logger?.LogInformation("Successfully discovered server at {ServerAddress}", uri.ToString());
 
-                    await rpc.ConnectAsync();
+                    //await rpc.ConnectAsync();
 
                     return;
                 }
@@ -63,14 +66,14 @@ public class ConnectionClient(
             {
                 logger?.LogError("Failed to discover server at {ServerAddress}", uri.ToString());
             }
-            
-            throw new InvalidAddressException("Could not find a server at that address");
         }
+        
+        throw new InvalidAddressException("Could not find a server at that address");
     }
 
     public async Task<bool> DisconnectAsync()
     {
-        return await rpc.DisconnectAsync();
+        return true;//return await rpc.DisconnectAsync();
     }
 
     public async Task<bool> PingAsync(Uri serverAddress = null)
diff --git a/LANCommander.SDK/Services/MediaService.cs b/LANCommander.SDK/Services/MediaClient.cs
similarity index 98%
rename from LANCommander.SDK/Services/MediaService.cs
rename to LANCommander.SDK/Services/MediaClient.cs
index f4cf3140..9fd2745e 100644
--- a/LANCommander.SDK/Services/MediaService.cs
+++ b/LANCommander.SDK/Services/MediaClient.cs
@@ -8,7 +8,7 @@ using LANCommander.SDK.Factories;
 
 namespace LANCommander.SDK.Services
 {
-    public class MediaService(
+    public class MediaClient(
         ApiRequestFactory apiRequestFactory,
         IConnectionClient connectionClient)
     {
diff --git a/LANCommander.Server.Services/MediaService.cs b/LANCommander.Server.Services/MediaService.cs
index 2857dd17..a88d75b2 100644
--- a/LANCommander.Server.Services/MediaService.cs
+++ b/LANCommander.Server.Services/MediaService.cs
@@ -19,7 +19,7 @@ using SixLabors.ImageSharp.PixelFormats;
 namespace LANCommander.Server.Services
 {
     public sealed class MediaService(
-        ILogger<SDK.Services.MediaService> logger,
+        ILogger<SDK.Services.MediaClient> logger,
         IFusionCache cache,
         IMapper mapper,
         IHttpContextAccessor httpContextAccessor,
@@ -153,7 +153,7 @@ namespace LANCommander.Server.Services
                 await stream.CopyToAsync(fs);
             }
 
-            media.Crc32 = await SDK.Services.MediaService.CalculateChecksumAsync(path);
+            media.Crc32 = await SDK.Services.MediaClient.CalculateChecksumAsync(path);
 
             await GenerateThumbnailAsync(media);
 
diff --git a/LANCommander.UI/Components/SafeImage/SafeImage.razor b/LANCommander.UI/Components/SafeImage/SafeImage.razor
index 29882d69..766ed51c 100644
--- a/LANCommander.UI/Components/SafeImage/SafeImage.razor
+++ b/LANCommander.UI/Components/SafeImage/SafeImage.razor
@@ -1,6 +1,6 @@
 @using LANCommander.SDK.Models
 @using LANCommander.SDK.Services
-@inject MediaService MediaService
+@inject MediaClient MediaClient
 @namespace LANCommander.UI.Components
 
 <Spin Spinning="_loading" WrapperClassName="@_class">
@@ -12,7 +12,7 @@
         }
         else if (Media != null)
         {
-            <img src="@(MediaService.GetAbsoluteThumbnailUrl(Media))" @onload="OnLoad" @onerror="OnError" style="@Style" />
+            <img src="@(MediaClient.GetAbsoluteThumbnailUrl(Media))" @onload="OnLoad" @onerror="OnError" style="@Style" />
         }
         else
         {