Move non-UI components to new Refresher.Core project, move to NotEnoughLogs

Refresher/Accessors/ConsolePatchAccessor.cs → Refresher.Core/Accessors/ConsolePatchAccessor.cs

@@ -1,7 +1,7 @@
 using FluentFTP;
-using Refresher.Exceptions;
+using Refresher.Core.Exceptions;
 
-namespace Refresher.Accessors;
+namespace Refresher.Core.Accessors;
 
 public class ConsolePatchAccessor : PatchAccessor, IDisposable
 {
@@ -27,7 +27,7 @@ public ConsolePatchAccessor(string remoteIp)
 
     private byte[]? GetIdps()
     {
-        Program.Log("Getting IDPS...", "IDPS");
+        State.Logger.LogInfo(IDPS, "Getting IDPS from the console...");
         UriBuilder idpsPs3 = new("http", this._remoteIp, 80, "idps.ps3");
         UriBuilder idpsHex = new("http", this._remoteIp, 80, "dev_hdd0/idps.hex");
         UriBuilder idpsHexUsb = new("http", this._remoteIp, 80, "dev_usb000/idps.hex");
@@ -48,7 +48,7 @@ public ConsolePatchAccessor(string remoteIp)
     {
         HttpResponseMessage response;
 
-        Program.Log($"  {stepName} ({uri.AbsolutePath})", "IDPS");
+        State.Logger.LogDebug(IDPS, $"  {stepName} ({uri.AbsolutePath})");
         try
         {
             response = client.GetAsync(uri).Result;
@@ -62,17 +62,17 @@ public ConsolePatchAccessor(string remoteIp)
         {
             if (!HandleIdpsRequestError(e))
             {
-                Program.Log($"Couldn't fetch the IDPS from the PS3 because of an unknown error: {e}", "IDPS", BreadcrumbLevel.Error);
+                State.Logger.LogError(IDPS, $"Couldn't fetch the IDPS from the PS3 because of an unknown error: {e}");
                 SentrySdk.CaptureException(e);
             }
             return null;
         }
-        Program.Log($"    {(int)response.StatusCode} {response.StatusCode} (success: {response.IsSuccessStatusCode})", "IDPS");
+        State.Logger.LogDebug(IDPS, $"    {(int)response.StatusCode} {response.StatusCode} (success: {response.IsSuccessStatusCode})");
 
         if (!response.IsSuccessStatusCode)
         {
-            Program.Log("Couldn't fetch the IDPS from the PS3 because of a bad status code.", "IDPS", BreadcrumbLevel.Error);
-            Program.Log(response.Content.ReadAsStringAsync().Result, "IDPS", BreadcrumbLevel.Debug);
+            State.Logger.LogError(IDPS, "Couldn't fetch the IDPS from the PS3 because of a bad status code.");
+            State.Logger.LogDebug(IDPS, response.Content.ReadAsStringAsync().Result);
             return null;
         }
 
@@ -83,7 +83,7 @@ private static bool HandleIdpsRequestError(Exception inner)
     {
         if (inner is HttpRequestException httpException)
         {
-            Program.Log($"Couldn't fetch the IDPS from the PS3 because we couldn't make the request: {httpException.Message}", "IDPS", BreadcrumbLevel.Error);
+            State.Logger.LogError(IDPS, $"Couldn't fetch the IDPS from the PS3 because we couldn't make the request: {httpException.Message}");
             return true;
         }
 

Refresher/Accessors/EmulatorPatchAccessor.cs → Refresher.Core/Accessors/EmulatorPatchAccessor.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Accessors;
+namespace Refresher.Core.Accessors;
 
 public class EmulatorPatchAccessor : PatchAccessor
 {

Refresher/Accessors/GameCacheAccessor.cs → Refresher.Core/Accessors/GameCacheAccessor.cs

@@ -1,6 +1,4 @@
-using Eto.Forms;
-
-namespace Refresher.Accessors;
+namespace Refresher.Core.Accessors;
 
 public static class GameCacheAccessor
 {
@@ -17,10 +15,11 @@
         {
             Directory.CreateDirectory(FullCacheDirectory);
         }
         catch (IOException e)
         {
-            MessageBox.Show($"Couldn't create the directory for the games cache: {e.Message}\n" +
-                            $"This error is rare and I don't know how to cleanly handle this scenario so Refresher is just gonna exit.");
+            // FIXME: no native MessageBox in Refresher.Core
+            // MessageBox.Show($"Couldn't create the directory for the games cache: {e.Message}\n" +
+            //                 $"This error is rare and I don't know how to cleanly handle this scenario so Refresher is just gonna exit.");
             Environment.Exit(1); // shrug
         }
     }

Refresher/Accessors/PatchAccessor.cs → Refresher.Core/Accessors/PatchAccessor.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Accessors;
+namespace Refresher.Core.Accessors;
 
 public abstract class PatchAccessor
 {

Refresher/Exceptions/FTPConnectionFailureException.cs → Refresher.Core/Exceptions/FTPConnectionFailureException.cs

@@ -1,5 +1,3 @@
-using System.Runtime.CompilerServices;
-
-namespace Refresher.Exceptions;
+namespace Refresher.Core.Exceptions;
 
 public class FTPConnectionFailureException() : Exception("Could not connect to the FTP server.");

Refresher.Core/GlobalUsings.cs

@@ -0,0 +1 @@
+global using static Refresher.Core.LogType;

Refresher.Core/LogType.cs

@@ -0,0 +1,18 @@
+namespace Refresher.Core;
+
+public enum LogType : byte
+{
+    Patcher,
+    PPU,
+    Verify,
+    CLI,
+    PatchAccessor,
+    PatchForm,
+    IntegratedPatchForm,
+    InfoRetrieval,
+    Crypto,
+    IDPS,
+    OSIntegration,
+    AutoDiscover,
+    PSP,
+}

Refresher.Core/Logging/SentryBreadcrumbSink.cs

@@ -0,0 +1,31 @@
+using NotEnoughLogs;
+using NotEnoughLogs.Sinks;
+
+namespace Refresher.Core.Logging;
+
+public class SentryBreadcrumbSink : ILoggerSink
+{
+    private static BreadcrumbLevel GetLevel(LogLevel level)
+    {
+        return level switch
+        {
+            LogLevel.Critical => BreadcrumbLevel.Critical,
+            LogLevel.Error => BreadcrumbLevel.Error,
+            LogLevel.Warning => BreadcrumbLevel.Warning,
+            LogLevel.Info => BreadcrumbLevel.Info,
+            LogLevel.Debug => BreadcrumbLevel.Debug,
+            LogLevel.Trace => BreadcrumbLevel.Debug,
+            _ => throw new ArgumentOutOfRangeException(nameof(level), level, null),
+        };
+    }
+
+    public void Log(LogLevel level, ReadOnlySpan<char> category, ReadOnlySpan<char> content)
+    {
+        SentrySdk.AddBreadcrumb(content.ToString(), category.ToString(), level: GetLevel(level));
+    }
+
+    public void Log(LogLevel level, ReadOnlySpan<char> category, ReadOnlySpan<char> format, params object[] args)
+    {
+        SentrySdk.AddBreadcrumb(string.Format(format.ToString(), args), category.ToString(), level: GetLevel(level));
+    }
+}

Refresher/Patching/EbootPatcher.cs → Refresher.Core/Patching/EbootPatcher.cs

@@ -7,14 +7,14 @@
 using System.Text.RegularExpressions;
 using ELFSharp.ELF;
 using ELFSharp.ELF.Segments;
-using Refresher.Verification;
+using Refresher.Core.Verification;
 
-namespace Refresher.Patching;
+namespace Refresher.Core.Patching;
 
 public partial class EbootPatcher : IPatcher
 {
     private readonly Lazy<List<PatchTargetInfo>> _targets;
     private readonly Lazy<string?> _ppuHash = null;
 
     public bool GenerateRpcs3Patch = false;
     public string? Rpcs3PatchFolder = null;
@@ -141,7 +141,7 @@
         FindLbpkDomains(reader, lbpkPositions, foundItems);
 
         long end = Stopwatch.GetTimestamp();
-        Program.Log($"Detecting patchables took {(double)(end - start) / (double)Stopwatch.Frequency} seconds!");
+        State.Logger.LogDebug(Patcher, $"Detecting patchables took {(end - start) / (double)Stopwatch.Frequency} seconds!");
         return foundItems;
     }
 
@@ -245,7 +245,7 @@
 
             if (UrlMatch().Matches(str).Count != 0)
             {
-                Program.Log($"Found URL at offset {foundPosition}: '{str}'");
+                State.Logger.LogTrace(Patcher, $"Found URL at offset {foundPosition}: '{str}'");
                 foundItems.Add(new PatchTargetInfo
                 {
                     Length = len,
@@ -344,7 +344,7 @@
 
         string ppuHash = BitConverter.ToString(hash.Hash!).Replace("-", "").ToLower();
 
-        Program.Log($"PPU hash: PPU-{ppuHash}", "PPU", BreadcrumbLevel.Debug);
+        State.Logger.LogDebug(PPU, $"PPU hash: PPU-{ppuHash}");
         return ppuHash;
     }
 
@@ -359,7 +359,7 @@
 
         this.Stream.Position = 0;
 
-        Program.Log($"URL: {url}", "Verify");
+        State.Logger.LogInfo(LogType.Verify, $"Verifying EBOOT against URL: {url}");
         // Check url
         if (url.EndsWith('/'))
             messages.Add(new Message(MessageLevel.Error,
@@ -373,7 +373,7 @@
         {
             Class output = ELFReader.CheckELFType(this.Stream);
 
-            Program.Log($"ELF class: {output}", "Verify");
+            State.Logger.LogDebug(LogType.Verify, $"ELF class: {output}");
             if (output == Class.NotELF)
             {
                 messages.Add(new Message(MessageLevel.Error, "EBOOT is not a valid ELF file."));

Refresher/Patching/IPatcher.cs → Refresher.Core/Patching/IPatcher.cs

@@ -1,6 +1,6 @@
-using Refresher.Verification;
+using Refresher.Core.Verification;
 
-namespace Refresher.Patching;
+namespace Refresher.Core.Patching;
 
 public interface IPatcher
 {

Refresher/Patching/PSP/PSPPluginListEntry.cs → Refresher.Core/Patching/PSP/PSPPluginListEntry.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Patching.PSP;
+namespace Refresher.Core.Patching.PSP;
 
 public class PSPPluginListEntry
 {

Refresher/Patching/PSP/PSPPluginListParser.cs → Refresher.Core/Patching/PSP/PSPPluginListParser.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Patching.PSP;
+namespace Refresher.Core.Patching.PSP;
 
 public static class PSPPluginListParser
 {

Refresher/Patching/PSPPatcher.cs → Refresher.Core/Patching/PSPPatcher.cs

@@ -1,8 +1,8 @@
 using System.Reflection;
-using Refresher.Patching.PSP;
-using Refresher.Verification;
+using Refresher.Core.Patching.PSP;
+using Refresher.Core.Verification;
 
-namespace Refresher.Patching;
+namespace Refresher.Core.Patching;
 
 public class PSPPatcher : IPatcher
 {

Refresher/Patching/PatchTargetInfo.cs → Refresher.Core/Patching/PatchTargetInfo.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Patching;
+namespace Refresher.Core.Patching;
 
 public struct PatchTargetInfo
 {

Refresher/Patching/PatchTargetType.cs → Refresher.Core/Patching/PatchTargetType.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Patching;
+namespace Refresher.Core.Patching;
 
 public enum PatchTargetType
 {

Refresher.Core/Refresher.Core.csproj

@@ -0,0 +1,17 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+    <PropertyGroup>
+        <TargetFramework>net8.0</TargetFramework>
+        <ImplicitUsings>enable</ImplicitUsings>
+        <Nullable>enable</Nullable>
+    </PropertyGroup>
+
+    <ItemGroup>
+        <PackageReference Include="FluentFTP" Version="49.0.2" />
+        <PackageReference Include="NotEnoughLogs" Version="2.0.3" />
+        <PackageReference Include="SCEToolSharp" Version="1.2.2" />
+        <PackageReference Include="ELFSharp" Version="2.17.3" />
+        <PackageReference Include="Sentry" Version="4.4.0" />
+    </ItemGroup>
+
+</Project>

Refresher.Core/State.cs

@@ -0,0 +1,23 @@
+using NotEnoughLogs;
+using NotEnoughLogs.Behaviour;
+using NotEnoughLogs.Sinks;
+using Refresher.Core.Logging;
+
+namespace Refresher.Core;
+
+public static class State
+{
+    public static readonly Logger Logger = InitializeLogger([new ConsoleSink(), new SentryBreadcrumbSink()]);
+
+    private static Logger InitializeLogger(IEnumerable<ILoggerSink> sinks)
+    {
+        // if(Logger != null)
+            // Logger.Dispose();
+
+        return new Logger(sinks, new LoggerConfiguration
+        {
+            Behaviour = new DirectLoggingBehaviour(),
+            MaxLevel = LogLevel.Trace,
+        });
+    }
+}

Refresher/Verification/Autodiscover/AutodiscoverResponse.cs → Refresher.Core/Verification/Autodiscover/AutodiscoverResponse.cs

@@ -1,22 +1,22 @@
-using Newtonsoft.Json;
+using System.Text.Json.Serialization;
 
-namespace Refresher.Verification.Autodiscover;
+namespace Refresher.Core.Verification.Autodiscover;
 
 #nullable disable
 
 public class AutodiscoverResponse
 {
     private const int SupportedVersion = 2;
 
-    [JsonProperty("version")]
+    [JsonPropertyName("version")]
     public int Version { get; set; }
 
-    [JsonProperty("serverBrand")]
+    [JsonPropertyName("serverBrand")]
     public string ServerBrand { get; set; }
 
-    [JsonProperty("url")]
+    [JsonPropertyName("url")]
     public string Url { get; set; }
 
-    [JsonProperty("usesCustomDigestKey")]
+    [JsonPropertyName("usesCustomDigestKey")]
     public bool? UsesCustomDigestKey { get; set; } = false; // We mark as nullable, as this was added in version 2
 }

Refresher/Verification/Message.cs → Refresher.Core/Verification/Message.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Verification;
+namespace Refresher.Core.Verification;
 
 public readonly struct Message
 {

Refresher/Verification/MessageLevel.cs → Refresher.Core/Verification/MessageLevel.cs

@@ -1,4 +1,4 @@
-namespace Refresher.Verification;
+namespace Refresher.Core.Verification;
 
 public enum MessageLevel
 {

Refresher/Verification/ParamSfo.cs → Refresher.Core/Verification/ParamSfo.cs

@@ -1,6 +1,6 @@
 using System.Text;
 
-namespace Refresher.Verification;
+namespace Refresher.Core.Verification;
 
 // TODO: Move to separate repository with its own nuget package
 // Maybe combine with NPTicket?

Refresher.sln

@@ -5,6 +5,8 @@ VisualStudioVersion = 17.0.31903.59
 MinimumVisualStudioVersion = 10.0.40219.1
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Refresher", "Refresher\Refresher.csproj", "{0A67E8A2-0E7C-4BB7-90E5-8D8EA59631D4}"
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Refresher.Core", "Refresher.Core\Refresher.Core.csproj", "{49CE141A-8BD7-4CD5-BE86-257BDA5C5796}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -18,5 +20,9 @@ Global
 		{0A67E8A2-0E7C-4BB7-90E5-8D8EA59631D4}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{0A67E8A2-0E7C-4BB7-90E5-8D8EA59631D4}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{0A67E8A2-0E7C-4BB7-90E5-8D8EA59631D4}.Release|Any CPU.Build.0 = Release|Any CPU
+		{49CE141A-8BD7-4CD5-BE86-257BDA5C5796}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{49CE141A-8BD7-4CD5-BE86-257BDA5C5796}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{49CE141A-8BD7-4CD5-BE86-257BDA5C5796}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{49CE141A-8BD7-4CD5-BE86-257BDA5C5796}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 EndGlobal