From 04d020f3efcca10f71c17d3d66669523fd51c2fe Mon Sep 17 00:00:00 2001
From: "github-actions[bot]"
 <41898282+github-actions[bot]@users.noreply.github.com>
Date: Sat, 18 Jan 2025 10:09:45 +0000
Subject: [PATCH 1/5] @TheRealLyonic has signed the CLA in
 SubnauticaModding/Nautilus#583

---
 CLA/Signatures.json | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/CLA/Signatures.json b/CLA/Signatures.json
index 4eda0689..5226e154 100644
--- a/CLA/Signatures.json
+++ b/CLA/Signatures.json
@@ -103,6 +103,14 @@
       "created_at": "2024-12-06T23:01:55Z",
       "repoId": 123711758,
       "pullRequestNo": 568
+    },
+    {
+      "name": "TheRealLyonic",
+      "id": 67993317,
+      "comment_id": 2599656575,
+      "created_at": "2025-01-18T10:09:32Z",
+      "repoId": 123711758,
+      "pullRequestNo": 583
     }
   ]
 }
\ No newline at end of file

From 2ff809975fe07505b1282b96644ff87d9acf4ab5 Mon Sep 17 00:00:00 2001
From: Lee23 <31892011+LeeTwentyThree@users.noreply.github.com>
Date: Sat, 1 Feb 2025 02:51:23 -0500
Subject: [PATCH 2/5] Commit the forgotten pre.34 version tag

---
 Version.targets | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Version.targets b/Version.targets
index 60adffa4..9c035281 100644
--- a/Version.targets
+++ b/Version.targets
@@ -3,7 +3,7 @@
     <PropertyGroup>
         <!-- The assembly uses this version number. !-->
         <VersionPrefix>1.0.0</VersionPrefix>
-        <SuffixNumber>33</SuffixNumber>
+        <SuffixNumber>34</SuffixNumber>
         <VersionSuffix>pre.$(SuffixNumber)</VersionSuffix>
     </PropertyGroup>
 </Project>
\ No newline at end of file

From c70f4086b27e3d6fabd432357417580c56198a1a Mon Sep 17 00:00:00 2001
From: Lee23 <31892011+LeeTwentyThree@users.noreply.github.com>
Date: Sun, 2 Feb 2025 20:14:19 -0500
Subject: [PATCH 3/5] feat: Add a dependency warning popup and a user-friendly
 menu to view errors (#586)

* Add dependency warning messages

* Completely overhaul the system

* Move button to bottom of screen

* Fix formatting in BZ

* Fix old options existing over menu in BZ

* Organize error messages better
---
 Nautilus/Initializer.cs                       |   1 +
 .../MonoBehaviours/PluginLoadErrorMenu.cs     |   9 +
 Nautilus/Patchers/DependencyWarningPatcher.cs | 202 ++++++++++++++++++
 3 files changed, 212 insertions(+)
 create mode 100644 Nautilus/MonoBehaviours/PluginLoadErrorMenu.cs
 create mode 100644 Nautilus/Patchers/DependencyWarningPatcher.cs

diff --git a/Nautilus/Initializer.cs b/Nautilus/Initializer.cs
index 0b7b4ac7..3ff1461b 100644
--- a/Nautilus/Initializer.cs
+++ b/Nautilus/Initializer.cs
@@ -85,5 +85,6 @@ static Initializer()
         WaterParkPatcher.Patch(_harmony);
         ModMessageSystem.Patch();
         BiomePatcher.Patch(_harmony);
+        DependencyWarningPatcher.Patch(_harmony);
     }
 }
diff --git a/Nautilus/MonoBehaviours/PluginLoadErrorMenu.cs b/Nautilus/MonoBehaviours/PluginLoadErrorMenu.cs
new file mode 100644
index 00000000..aa337d07
--- /dev/null
+++ b/Nautilus/MonoBehaviours/PluginLoadErrorMenu.cs
@@ -0,0 +1,9 @@
+using UnityEngine;
+
+namespace Nautilus.MonoBehaviours;
+
+internal class PluginLoadErrorMenu : MonoBehaviour
+{
+    public void OnButtonOpen() => gameObject.SetActive(true);
+    public void OnButtonClose() => gameObject.SetActive(false);
+}
\ No newline at end of file
diff --git a/Nautilus/Patchers/DependencyWarningPatcher.cs b/Nautilus/Patchers/DependencyWarningPatcher.cs
new file mode 100644
index 00000000..1175ed57
--- /dev/null
+++ b/Nautilus/Patchers/DependencyWarningPatcher.cs
@@ -0,0 +1,202 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using BepInEx.Bootstrap;
+using HarmonyLib;
+using Nautilus.MonoBehaviours;
+using Nautilus.Utility;
+using TMPro;
+using UnityEngine;
+using UnityEngine.UI;
+using Object = UnityEngine.Object;
+
+namespace Nautilus.Patchers;
+
+internal static class DependencyWarningPatcher
+{
+    internal static void Patch(Harmony harmony)
+    {
+        harmony.PatchAll(typeof(DependencyWarningPatcher));
+    }
+
+    [HarmonyPostfix]
+    [HarmonyPatch(typeof(uGUI_MainMenu), nameof(uGUI_MainMenu.Start))]
+    private static void MainMenuStartPostfix(uGUI_MainMenu __instance)
+    {
+        try
+        {
+            CreateDependencyWarningUI(__instance);
+        }
+        catch (Exception e)
+        {
+            InternalLogger.Error("Failed to create main menu dependency warning UI! Exception thrown: " + e);
+        }
+    }
+
+    private static void CreateDependencyWarningUI(uGUI_MainMenu mainMenu)
+    {
+        var dependencyErrors = Chainloader.DependencyErrors;
+        if (dependencyErrors.Count == 0)
+            return;
+
+        var missingDependencies = GetMissingDependencies();
+        var formattedMissingDependencies = FormatMissingDependencies(missingDependencies);
+
+        // -- Add warning text --
+        var textReference = mainMenu.transform.Find("Panel/MainMenu/PlayerName").gameObject;
+        var textObject = Object.Instantiate(textReference, textReference.transform.parent, true);
+        textObject.name = "NautilusDependencyWarningText";
+        Object.DestroyImmediate(textObject.GetComponent<MainMenuPlayerName>());
+        textObject.SetActive(true);
+        var transform = textObject.GetComponent<RectTransform>();
+        transform.anchorMin = new Vector2(0, 1);
+        transform.anchorMax = new Vector2(1, 1);
+        transform.pivot = new Vector2(0.5f, 0);
+        transform.sizeDelta = new Vector2(1, 40);
+        transform.offsetMin = new Vector2(-0.5f, 0);
+        transform.offsetMax = new Vector2(-0.5f, 570);
+        var text = textObject.GetComponentInChildren<TextMeshProUGUI>();
+        text.overflowMode = TextOverflowModes.Ellipsis;
+        text.alignment = TextAlignmentOptions.Center;
+        text.color = Color.red;
+        text.richText = true;
+        text.geometrySortingOrder = VertexSortingOrder.Reverse;
+        text.fontStyle = FontStyles.Bold;
+        text.text = "<mark=#000000CC>Some mods failed to load! Please view the mod load errors for more details.\n" +
+                    formattedMissingDependencies + "</mark>";
+
+        // -- Add mod load errors window --
+        var optionsWindowReference = mainMenu.transform.Find("Panel/Options");
+        var errorsMenuPanel =
+            Object.Instantiate(optionsWindowReference.gameObject, optionsWindowReference.parent, true);
+        errorsMenuPanel.name = "NautilusModErrorsWindow";
+        var menuBehaviour = errorsMenuPanel.AddComponent<PluginLoadErrorMenu>();
+        errorsMenuPanel.SetActive(false);
+        var optionsPanel = errorsMenuPanel.GetComponentInChildren<uGUI_OptionsPanel>();
+        var panePrefab = optionsPanel.panePrefab;
+        Object.DestroyImmediate(optionsPanel);
+        var errorsMenuHeader = errorsMenuPanel.transform.Find("Header");
+        errorsMenuHeader.gameObject.GetComponent<TextMeshProUGUI>().text = "BepInEx plugin load errors";
+        Object.DestroyImmediate(errorsMenuHeader.GetComponent<TranslationLiveUpdate>());
+        errorsMenuPanel.transform.Find("Middle/TabsHolder").gameObject.SetActive(false);
+        var panesHolder = errorsMenuPanel.transform.Find("Middle/PanesHolder");
+        foreach (Transform child in panesHolder)
+        {
+            Object.Destroy(child.gameObject);
+        }
+        var mainPaneTransform = Object.Instantiate(panePrefab, panesHolder).transform;
+        var mainPaneContent = mainPaneTransform.Find("Viewport/Content");
+#if BELOWZERO
+        var layoutGroup = mainPaneContent.gameObject.GetComponent<VerticalLayoutGroup>();
+        layoutGroup.spacing = 65;
+        layoutGroup.padding = new RectOffset(15, 15, 40, 15);
+#endif
+        // Create a list of all error messages that should be displayed
+        var errorsToDisplay = new List<string>
+        {
+            $"<color=#FF0000>{formattedMissingDependencies}</color>",
+            "<color=#FFFFFF>Mod load errors:</color>"
+        };
+        errorsToDisplay.AddRange(dependencyErrors.Where(ShouldDisplayError));
+        // Add error messages to menu
+        foreach (var error in errorsToDisplay)
+        {
+            var errorEntryTextObj = Object.Instantiate(textReference, mainPaneContent, true);
+            Object.DestroyImmediate(errorEntryTextObj.GetComponent<MainMenuPlayerName>());
+            var errorEntryText = errorEntryTextObj.GetComponent<TextMeshProUGUI>();
+            errorEntryText.text = error;
+            errorEntryText.enableWordWrapping = true;
+            errorEntryText.fontSize = 25;
+            errorEntryText.richText = true;
+            errorEntryTextObj.SetActive(true);
+            errorEntryTextObj.transform.localScale = Vector3.one;
+        }
+
+        // Fix up buttons and add close window button
+        var closeWindowButton = errorsMenuPanel.transform.Find("Bottom/ButtonBack").gameObject;
+        Object.DestroyImmediate(closeWindowButton.GetComponentInChildren<TranslationLiveUpdate>());
+        closeWindowButton.GetComponentInChildren<TextMeshProUGUI>().text = "Close";
+        var closeWindowButtonComponent = closeWindowButton.GetComponent<Button>();
+        closeWindowButtonComponent.onClick.RemoveAllListeners();
+        closeWindowButtonComponent.onClick.AddListener(menuBehaviour.OnButtonClose);
+        errorsMenuPanel.transform.Find("Bottom/ButtonApply").gameObject.SetActive(false);
+
+        // -- Add open button --
+        var openButtonParent = mainMenu.transform.Find("Panel/MainMenu");
+        var openButtonReference = mainMenu.transform.Find("Panel/Options/Bottom/ButtonBack");
+        var openButton = Object.Instantiate(openButtonReference.gameObject, openButtonParent, true);
+        openButton.name = "NautilusViewModLoadErrorsButton";
+        var buttonTransform = openButton.GetComponent<RectTransform>();
+        buttonTransform.SetSiblingIndex(0);
+        buttonTransform.anchorMin = new Vector2(0.65f, 0.01f);
+        buttonTransform.anchorMax = new Vector2(0.99f, 0.12f);
+        buttonTransform.offsetMin = new Vector2(-0.5f, -0.5f);
+        buttonTransform.offsetMax = new Vector2(0.5f, 0.5f);
+        buttonTransform.sizeDelta = Vector2.one;
+        Object.DestroyImmediate(openButton.GetComponentInChildren<TranslationLiveUpdate>());
+        buttonTransform.GetComponentInChildren<TextMeshProUGUI>().text =
+            $"View mod load errors ({dependencyErrors.Count})";
+        var openWindowButtonComponent = openButton.GetComponent<Button>();
+        openWindowButtonComponent.onClick.RemoveAllListeners();
+        openWindowButtonComponent.onClick.AddListener(menuBehaviour.OnButtonOpen);
+        openButton.SetActive(true);
+    }
+
+    private static bool ShouldDisplayError(string errorMessage)
+    {
+        // These errors occur for perfectly working mods, don't do any good for people, and will confuse users
+        if (errorMessage.Contains("targets a wrong version of BepInEx"))
+            return false;
+        return true;
+    }
+
+    private static string FormatMissingDependencies(IList<string> missingDependencies)
+    {
+        var sb = new StringBuilder();
+        sb.Append("Missing ");
+        sb.Append(missingDependencies.Count);
+        sb.Append(" dependenc");
+        sb.Append(missingDependencies.Count == 1 ? "y" : "ies");
+        sb.Append(": ");
+        for (int i = 0; i < missingDependencies.Count; i++)
+        {
+            sb.Append(missingDependencies[i]);
+            if (i < missingDependencies.Count - 1)
+            {
+                sb.Append(", ");
+            }
+        }
+
+        return sb.ToString();
+    }
+
+    private static List<string> GetMissingDependencies()
+    {
+        var missingDependencies = new HashSet<string>();
+        // Get the list of dependency log warning messages
+        // The format is as follows: "Could not load [{0}] because it has missing dependencies: {1}"
+        var dependencyErrors = new List<string>(Chainloader.DependencyErrors);
+        foreach (var error in dependencyErrors)
+        {
+            // Plugin GUIDs CANNOT contain colons, so this will only search for the colon in the log warning format
+            var indexOfColon = error.LastIndexOf(':');
+            // If the message didn't have a colon, it was of the wrong format
+            if (indexOfColon <= 0) continue;
+            // If the message contained the word "incompatible," it was an incompatibility error and should be ignored
+            var indexOfIncompatible = error.IndexOf("incompatible", StringComparison.Ordinal);
+            if (indexOfIncompatible >= 0 && indexOfIncompatible < indexOfColon) continue;
+            // Otherwise, get the remainder of the string, which should be the dependency's GUID
+            var dependencyGuidText = error[(indexOfColon + 2)..];
+            var dependencyGuids = dependencyGuidText.Split(new[] {", "}, StringSplitOptions.RemoveEmptyEntries);
+            foreach (var guid in dependencyGuids)
+            {
+                missingDependencies.Add(guid);
+            }
+        }
+
+        var list = missingDependencies.ToList();
+        list.Sort();
+        return list;
+    }
+}
\ No newline at end of file

From 0077093dc76d5578c52965be163bdeeb49ca8acb Mon Sep 17 00:00:00 2001
From: Lee23 <31892011+LeeTwentyThree@users.noreply.github.com>
Date: Sun, 2 Feb 2025 20:17:25 -0500
Subject: [PATCH 4/5] Increase version to pre.35

---
 Version.targets | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Version.targets b/Version.targets
index 9c035281..b1b1c0b4 100644
--- a/Version.targets
+++ b/Version.targets
@@ -3,7 +3,7 @@
     <PropertyGroup>
         <!-- The assembly uses this version number. !-->
         <VersionPrefix>1.0.0</VersionPrefix>
-        <SuffixNumber>34</SuffixNumber>
+        <SuffixNumber>35</SuffixNumber>
         <VersionSuffix>pre.$(SuffixNumber)</VersionSuffix>
     </PropertyGroup>
 </Project>
\ No newline at end of file

From 2940f1e27040ff05cf20c35f38eae4f57ae12ac4 Mon Sep 17 00:00:00 2001
From: Lee23 <31892011+LeeTwentyThree@users.noreply.github.com>
Date: Sun, 2 Feb 2025 21:18:09 -0500
Subject: [PATCH 5/5] Address possible edge cases w/ dependency warnings

---
 Nautilus/Patchers/DependencyWarningPatcher.cs | 75 +++++++++++++++++--
 1 file changed, 67 insertions(+), 8 deletions(-)

diff --git a/Nautilus/Patchers/DependencyWarningPatcher.cs b/Nautilus/Patchers/DependencyWarningPatcher.cs
index 1175ed57..c0700085 100644
--- a/Nautilus/Patchers/DependencyWarningPatcher.cs
+++ b/Nautilus/Patchers/DependencyWarningPatcher.cs
@@ -36,7 +36,7 @@ private static void MainMenuStartPostfix(uGUI_MainMenu __instance)
 
     private static void CreateDependencyWarningUI(uGUI_MainMenu mainMenu)
     {
-        var dependencyErrors = Chainloader.DependencyErrors;
+        var dependencyErrors = new List<string>(Chainloader.DependencyErrors.Where(ShouldDisplayError));
         if (dependencyErrors.Count == 0)
             return;
 
@@ -85,6 +85,7 @@ private static void CreateDependencyWarningUI(uGUI_MainMenu mainMenu)
         {
             Object.Destroy(child.gameObject);
         }
+
         var mainPaneTransform = Object.Instantiate(panePrefab, panesHolder).transform;
         var mainPaneContent = mainPaneTransform.Find("Viewport/Content");
 #if BELOWZERO
@@ -98,7 +99,7 @@ private static void CreateDependencyWarningUI(uGUI_MainMenu mainMenu)
             $"<color=#FF0000>{formattedMissingDependencies}</color>",
             "<color=#FFFFFF>Mod load errors:</color>"
         };
-        errorsToDisplay.AddRange(dependencyErrors.Where(ShouldDisplayError));
+        errorsToDisplay.AddRange(dependencyErrors);
         // Add error messages to menu
         foreach (var error in errorsToDisplay)
         {
@@ -173,7 +174,7 @@ private static string FormatMissingDependencies(IList<string> missingDependencie
 
     private static List<string> GetMissingDependencies()
     {
-        var missingDependencies = new HashSet<string>();
+        var missingDependencies = new Dictionary<string, Version>();
         // Get the list of dependency log warning messages
         // The format is as follows: "Could not load [{0}] because it has missing dependencies: {1}"
         var dependencyErrors = new List<string>(Chainloader.DependencyErrors);
@@ -188,15 +189,73 @@ private static List<string> GetMissingDependencies()
             if (indexOfIncompatible >= 0 && indexOfIncompatible < indexOfColon) continue;
             // Otherwise, get the remainder of the string, which should be the dependency's GUID
             var dependencyGuidText = error[(indexOfColon + 2)..];
+            // Split up separate GUIDs if there are multiple
             var dependencyGuids = dependencyGuidText.Split(new[] {", "}, StringSplitOptions.RemoveEmptyEntries);
-            foreach (var guid in dependencyGuids)
+            // Format individual GUIDs and add them to the list
+            foreach (var formattedGuid in dependencyGuids)
             {
-                missingDependencies.Add(guid);
+                // Separate the name and version of each formatted GUID string
+                var dependencyData = SeparateDependencyNameAndVersion(formattedGuid);
+                // Update the dictionary of dependencies, avoiding duplicates and displaying the highest version
+                if (missingDependencies.TryGetValue(dependencyData.Item1, out var previousHighestVersion))
+                {
+                    // Make sure only the highest version requirement is present
+                    // E.g., if we have Nautilus (1.0 or newer) and Nautilus (2.0 or newer), only display the latter
+                    if (previousHighestVersion == null ||
+                        dependencyData.Item2 != null && dependencyData.Item2 > previousHighestVersion)
+                    {
+                        missingDependencies[dependencyData.Item1] = dependencyData.Item2;
+                    }
+                }
+                else
+                {
+                    missingDependencies.Add(dependencyData.Item1, dependencyData.Item2);
+                }
             }
         }
 
-        var list = missingDependencies.ToList();
-        list.Sort();
-        return list;
+        // Finally, make a list of formatted strings:
+        var formattedDependencyStrings = new List<string>();
+        foreach (var dependency in missingDependencies)
+        {
+            var formattedString = dependency.Key;
+            // Only add the version number if the requirement exists and isn't 0.0.0
+            if (dependency.Value != default)
+            {
+                formattedString += $" ({dependency.Value} or newer)";
+            }
+
+            formattedDependencyStrings.Add(formattedString);
+        }
+
+        formattedDependencyStrings.Sort();
+        return formattedDependencyStrings;
+    }
+
+    // Attempts to find a version from the given dependency name and returns the GUID name by itself
+    // Example string of a formatted dependency name with version:
+    // "com.snmodding.nautilus (v1.0.0.34 or newer)"
+    private static (string, Version) SeparateDependencyNameAndVersion(string formattedDependency)
+    {
+        try
+        {
+            // Parentheses cannot exist in GUIDs, so this is ONLY present if the GUID is accompanied by a version
+            if (!formattedDependency.Contains("("))
+            {
+                return (formattedDependency, default);
+            }
+            
+            // Surpass the "(v"
+            var versionBeginIndex = formattedDependency.IndexOf('(') + 2;
+            var length = formattedDependency.LastIndexOf(" or newer)", StringComparison.Ordinal) + 1 -
+                         versionBeginIndex;
+            var rawGuidName = formattedDependency.Substring(0, versionBeginIndex - 3);
+            return (rawGuidName, Version.Parse(formattedDependency.Substring(versionBeginIndex, length)));
+        }
+        catch (Exception e)
+        {
+            InternalLogger.Warn($"Exception getting version from string '{formattedDependency}'" + e);
+            return (formattedDependency, default);
+        }
     }
 }
\ No newline at end of file