fix: Remove redundant dispatcherqueue from moduleprocess logging

Author: benaclejames

VRCFaceTracking.ModuleProcess/ModuleProcessMain.cs

@@ -7,7 +7,6 @@
 using VRCFaceTracking.Core.Params.Expressions;
 using VRCFaceTracking.Core.Sandboxing;
 using VRCFaceTracking.Core.Sandboxing.IPC;
-using Windows.System;
 
 namespace VRCFaceTracking.ModuleProcess;
 
@@ -106,7 +105,7 @@ static int VrcftMain(string modulePath, int serverPortNumber)
                 // .AddSentry(o =>
                 //     o.Dsn =
                 //     "https://444b0799dd2b670efa85d866c8c12134@o4506152235237376.ingest.us.sentry.io/4506152246575104")
-                .AddProvider(new ProxyLoggerProvider(DispatcherQueue.GetForCurrentThread()))
+                .AddProvider(new ProxyLoggerProvider())
             )
         .BuildServiceProvider();
 
@@ -118,7 +117,7 @@ static int VrcftMain(string modulePath, int serverPortNumber)
         Client = new VrcftSandboxClient(serverPortNumber, LoggerFactory);
 
         // Bind the log function so that we can forward log messages to VRCFT's main process
-        ProxyLogger.OnLog += (LogLevel level, string msg) =>
+        ProxyLogger.OnLog += (level, msg) =>
         {
             var pkt = new EventLogPacket(level, msg);
             Client.SendData(pkt);

VRCFaceTracking.ModuleProcess/ProxyLogger.cs

@@ -1,6 +1,5 @@
 using System.Collections.ObjectModel;
 using Microsoft.Extensions.Logging;
-using Windows.System;
 
 namespace VRCFaceTracking.ModuleProcess;
 
@@ -11,12 +10,10 @@ public class ProxyLogger : ILogger
     private readonly string _categoryName;
     // public static readonly ObservableCollection<string> AllLogs = new();
     public static OnLog OnLog;
-    private static DispatcherQueue? _dispatcher;
 
-    public ProxyLogger(string categoryName, DispatcherQueue? queue)
+    public ProxyLogger(string categoryName)
     {
         _categoryName = categoryName;
-        _dispatcher = queue;
     }
 
     public IDisposable BeginScope<TState>(TState state) where TState : notnull => default!;

VRCFaceTracking.ModuleProcess/ProxyLoggerProvider.cs

@@ -1,22 +1,14 @@
 using System.Collections.Concurrent;
 using Microsoft.Extensions.Logging;
-using Windows.System;
 
 namespace VRCFaceTracking.ModuleProcess;
 public class ProxyLoggerProvider : ILoggerProvider
 {
     private readonly ConcurrentDictionary<string, ProxyLogger> _loggers =
         new(StringComparer.OrdinalIgnoreCase);
 
-    private readonly DispatcherQueue _dispatcher;
-
-    public ProxyLoggerProvider(DispatcherQueue dispatcher)
-    {
-        _dispatcher = dispatcher;
-    }
-
     public ILogger CreateLogger(string categoryName) =>
-        _loggers.GetOrAdd(categoryName, name => new ProxyLogger(name, _dispatcher));
+        _loggers.GetOrAdd(categoryName, name => new ProxyLogger(name));
 
     public void Dispose() => _loggers.Clear();
 }