Make ChatWarningService part of the MessageService.

Author: Ottermandias

OtterGui

@@ -4,23 +4,26 @@
 using Penumbra.GameData.Files;
 using Penumbra.GameData.Files.Utility;
 using Penumbra.Interop.Hooks.ResourceLoading;
+using Penumbra.Mods.Manager;
+using Penumbra.Services;
 using Penumbra.String;
 using Penumbra.String.Classes;
-using Penumbra.UI;
 
 namespace Penumbra.Interop.Processing;
 
 /// <summary>
 /// Path pre-processor for shader packages that reverts redirects to known invalid files, as bad ShPks can crash the game.
 /// </summary>
-public sealed class ShpkPathPreProcessor(ResourceManagerService resourceManager, ChatWarningService chatWarningService) : IPathPreProcessor
+public sealed class ShpkPathPreProcessor(ResourceManagerService resourceManager, MessageService messager, ModManager modManager)
+    : IPathPreProcessor
 {
     public ResourceType Type
         => ResourceType.Shpk;
 
-    public unsafe FullPath? PreProcess(ResolveData resolveData, CiByteString path, Utf8GamePath originalGamePath, bool nonDefault, FullPath? resolved)
+    public unsafe FullPath? PreProcess(ResolveData resolveData, CiByteString path, Utf8GamePath originalGamePath, bool nonDefault,
+        FullPath? resolved)
     {
-        chatWarningService.CleanLastFileWarnings(false);
+        messager.CleanTaggedMessages(false);
 
         if (!resolved.HasValue)
             return null;
@@ -31,16 +34,16 @@ public ResourceType Type
             return resolvedPath;
 
         // If the ShPk is already loaded, it means that it already passed the sanity check.
-        var existingResource = resourceManager.FindResource(ResourceCategory.Shader, ResourceType.Shpk, unchecked((uint)resolvedPath.InternalName.Crc32));
+        var existingResource =
+            resourceManager.FindResource(ResourceCategory.Shader, ResourceType.Shpk, unchecked((uint)resolvedPath.InternalName.Crc32));
         if (existingResource != null)
             return resolvedPath;
 
         var checkResult = SanityCheck(resolvedPath.FullName);
         if (checkResult == SanityCheckResult.Success)
             return resolvedPath;
 
-        Penumbra.Log.Warning($"Refusing to honor file redirection because of failed sanity check (result: {checkResult}). Original path: {originalGamePath} Redirected path: {resolvedPath}");
-        chatWarningService.PrintFileWarning(resolvedPath.FullName, originalGamePath, WarningMessageComplement(checkResult));
+        messager.PrintFileWarning(modManager, resolvedPath.FullName, originalGamePath, WarningMessageComplement(checkResult));
 
         return null;
     }
@@ -49,8 +52,8 @@ private static SanityCheckResult SanityCheck(string path)
     {
         try
         {
-            using var file = MmioMemoryManager.CreateFromFile(path);
-            var bytes = file.GetSpan();
+            using var file  = MmioMemoryManager.CreateFromFile(path);
+            var       bytes = file.GetSpan();
 
             return ShpkFile.FastIsLegacy(bytes)
                 ? SanityCheckResult.Legacy
@@ -69,9 +72,9 @@ private static SanityCheckResult SanityCheck(string path)
     private static string WarningMessageComplement(SanityCheckResult result)
         => result switch
         {
-            SanityCheckResult.IoError  => "cannot read the modded file.",
-            SanityCheckResult.NotFound => "the modded file does not exist.",
-            SanityCheckResult.Legacy   => "this mod is not compatible with Dawntrail. Get an updated version, if possible, or disable it.",
+            SanityCheckResult.IoError  => "Cannot read the modded file.",
+            SanityCheckResult.NotFound => "The modded file does not exist.",
+            SanityCheckResult.Legacy   => "This mod is not compatible with Dawntrail. Get an updated version, if possible, or disable it.",
             _                          => string.Empty,
         };
 

Penumbra/Interop/Processing/ShpkPathPreProcessor.cs

@@ -2,10 +2,14 @@
 using Dalamud.Game.Text.SeStringHandling;
 using Dalamud.Game.Text.SeStringHandling.Payloads;
 using Dalamud.Interface;
+using Dalamud.Interface.ImGuiNotification;
 using Dalamud.Plugin.Services;
 using Lumina.Excel.GeneratedSheets;
 using OtterGui.Log;
 using OtterGui.Services;
+using Penumbra.Mods.Manager;
+using Penumbra.String.Classes;
+using Notification = OtterGui.Classes.Notification;
 
 namespace Penumbra.Services;
 
@@ -38,4 +42,16 @@ public void LinkItem(Item item)
             Message = payload,
         });
     }
+
+    public void PrintFileWarning(ModManager modManager, string fullPath, Utf8GamePath originalGamePath, string messageComplement)
+    {
+        // Don't warn for files managed by other plugins, or files we aren't sure about.
+        if (!modManager.TryIdentifyPath(fullPath, out var mod, out _))
+            return;
+
+        AddTaggedMessage($"{fullPath}.{messageComplement}",
+            new Notification(
+                $"Cowardly refusing to load replacement for {originalGamePath.Filename().ToString().ToLowerInvariant()} by {mod.Name}{(messageComplement.Length > 0 ? ":\n" : ".")}{messageComplement}",
+                NotificationType.Warning, 10000));
+    }
 }