ConsoleImprovements/ConsoleImprovements.csproj

@@ -38,14 +38,5 @@
         <Reference Include="Unity.TextMeshPro">
             <HintPath>$(LogicWorldGameLocation)\Logic_World_Data\Managed\Unity.TextMeshPro.dll</HintPath>
         </Reference>
-        <Reference Include="UnityEngine.CoreModule">
-            <HintPath>$(LogicWorldGameLocation)\Logic_World_Data\Managed\UnityEngine.CoreModule.dll</HintPath>
-        </Reference>
-        <Reference Include="UnityEngine.UI">
-            <HintPath>$(LogicWorldGameLocation)\Logic_World_Data\Managed\UnityEngine.UI.dll</HintPath>
-        </Reference>
-        <Reference Include="LICC">
-            <HintPath>$(LogicWorldGameLocation)\Logic_World_Data\Managed\LICC.dll</HintPath>
-        </Reference>
     </ItemGroup>
 </Project>

ConsoleImprovements/ConsoleImprovements/manifest.jecs

@@ -1,8 +1,8 @@
 ID: D4VID_ConsoleImprovements
 Name: ConsoleImprovements
 Author: D4VID
-Version: 1.2.1
+Version: 1.1.0
-Priority: -90
+Priority: 0
 ClientOnly: true
 Dependencies:
   - HarmonyForLogicWorld

ConsoleImprovements/ConsoleImprovements/src/client/CommandHistoryPatch.cs

@@ -1,17 +1,14 @@
-using System.Collections;
 using System.Reflection;
 using System.Collections.Generic;
 using EccsLogicWorldAPI.Shared.AccessHelper;
 using FancyInput;
 using HarmonyLib;
-using JimmysUnityUtilities;
-using LICC;
 using LogicLog;
 using TMPro;
 using Console = FancyPantsConsole.Console;
 
 namespace ConsoleImprovements.Client {
-    public class CommandInputPatch {
+    public class CommandHistoryPatch {
         const int HistLength = 1000;
 
         private static ILogicLogger _logger = null!;
@@ -19,32 +16,14 @@ namespace ConsoleImprovements.Client {
 
         private static LinkedList<string> _history = new LinkedList<string>();
         private static LinkedListNode<string>? _command;
-        private static Trie _commandRegistry = new Trie();
 
         public static bool Prepare(ILogicLogger logger) {
             _logger = logger;
 
-            try {
             // Get access to the rich text string inside the MessageData class
-                _commandInputFieldField = Fields.getPrivate(typeof(Console), "CommandInputField");
+            _commandInputFieldField = AccessTools.Field(typeof(Console), "CommandInputField");
-
+            if (_commandInputFieldField == null) {
-                CommandConsole commandConsole = CommandConsole.Current;
+                _logger.Error("Cannot get field CommandInputField of Console");
-                // ICommandRegistryInternal
-                var registry = Fields.getPrivate(typeof(CommandConsole), "CommandRegistry").GetValue(commandConsole);
-                // CommandRegistry.CommandCollectionByName
-                var commandCollection = Properties.get(registry, "AllRegisteredCommands").GetValue(registry);
-                // IEnumerable<Command>
-                var commands = (IEnumerable) Methods.getPublic(commandCollection, "EnumerateAllCommands")
-                    .Invoke(commandCollection, null)!;
-
-                foreach (var command in commands) {
-                    var nameProperty = Properties.getPublic(command, "Name");
-                    var hiddenProperty = Properties.getPublic(command, "Hidden");
-                    if ((bool) hiddenProperty.GetValue(command)!) continue;
-                    _commandRegistry.AddEntry((string) nameProperty.GetValue(command)!);
-                }
-            } catch (AccessHelperException e) {
-                _logger.Error($"Error occured while preparing command history patch: {e}");
                 return false;
             }
 
@@ -58,13 +37,13 @@ namespace ConsoleImprovements.Client {
 
         private static void PatchUpdate(Harmony harmony) {
             var methodTarget = Methods.getPrivate(typeof(Console), "Update");
-            var methodHook = Methods.get(typeof(CommandInputPatch), nameof(HookUpdate));
+            var methodHook = Methods.get(typeof(CommandHistoryPatch), nameof(HookUpdate));
             harmony.Patch(methodTarget, prefix: new HarmonyMethod(methodHook));
         }
 
         private static void PatchCommandSubmit(Harmony harmony) {
             var methodTarget = Methods.getPrivate(typeof(Console), "OnCommandInputFieldSubmit");
-            var methodHook = Methods.get(typeof(CommandInputPatch), nameof(HookCommandSubmit));
+            var methodHook = Methods.get(typeof(CommandHistoryPatch), nameof(HookCommandSubmit));
             harmony.Patch(methodTarget, prefix: new HarmonyMethod(methodHook));
         }
 
@@ -89,27 +68,6 @@ namespace ConsoleImprovements.Client {
                 }
             }
 
-            if (RawInput.Tab.DownThisFrame()) {
-                TMP_InputField? commandInputField = _commandInputFieldField!.GetValue(__instance) as TMP_InputField;
-                if (commandInputField == null) {
-                    _logger.Error("Cannot get CommandInputField value");
-                    return true;
-                }
-
-                string current = commandInputField.text;
-                string? completed = _commandRegistry.LookupNext(current, out Trie.Node? node);
-                if (completed != null) {
-                    if (completed == current) {
-                        List<string> commands = node?.CollectChildren(completed)!;
-                        if (commands.Count > 0) {
-                            _logger.Info($"Available commands: {string.Join(", ", commands)}");
-                        }
-                    } else {
-                        UpdateCommandTextField(__instance, completed);
-                    }
-                }
-            }
-
             return true; // Resume original functionality
         }
 
@@ -121,7 +79,6 @@ namespace ConsoleImprovements.Client {
             }
 
             commandInputField.text = text;
-            CoroutineUtility.RunAfterOneFrame(() => commandInputField.MoveTextEnd(false));
         }
 
         // ReSharper disable once InconsistentNaming

ConsoleImprovements/ConsoleImprovements/src/client/ConsoleImprovementsMod.cs

@@ -11,8 +11,8 @@ namespace ConsoleImprovements.Client {
                 Logger.Info("Message Copy Patch Successful");
             }
             
-            if (CommandInputPatch.Prepare(Logger)) {
+            if (CommandHistoryPatch.Prepare(Logger)) {
-                CommandInputPatch.Inject(harmony);
+                CommandHistoryPatch.Inject(harmony);
                 Logger.Info("Command History Patch Successful");
             }
 

ConsoleImprovements/ConsoleImprovements/src/client/MessageCopyPatch.cs

@@ -9,27 +9,42 @@ namespace ConsoleImprovements.Client {
     public class MessageCopyPatch {
         private static ILogicLogger _logger = null!;
         private static MethodInfo? _removeRichTextTagsMethod;
-        private static FieldInfo? _dataField;
+        private static FieldInfo? _dataProperty;
         private static FieldInfo? _messageRichTextField;
         private static FieldInfo? _messageTypeField;
 
         public static bool Prepare(ILogicLogger logger) {
             _logger = logger;
 
-            try {
             // Get the private RemoveRichTextTags method of Message
-                _removeRichTextTagsMethod = Methods.getPrivate(typeof(Message), "RemoveRichTextTags");
+            _removeRichTextTagsMethod = AccessTools.Method(
+                typeof(Message),
+                "RemoveRichTextTags",
+                new[] {typeof(string)}
+            );
+            if (_removeRichTextTagsMethod == null) {
+                _logger.Error("Cannot get method RemoveRichTextTags of message");
+                return false;
+            }
 
-                // Get access to the Data field of Message, which is of internal type MessageData
+            // Get access to the Data property of Message, which is of internal type MessageData
-                _dataField = Fields.getPrivate(typeof(Message), "Data");
+            _dataProperty = AccessTools.Field(typeof(Message), "Data");
+            if (_dataProperty == null) {
+                _logger.Error("Cannot get field Data of message");
+                return false;
+            }
 
             // Get access to the rich text string inside the MessageData class
-                _messageRichTextField = Fields.getPublic(_dataField.FieldType, "messageRichText");
+            _messageRichTextField = AccessTools.Field(_dataProperty.FieldType, "messageRichText");
+            if (_messageRichTextField == null) {
+                _logger.Error("Cannot get field messageRichText of message data");
+                return false;
+            }
 
             // Get access to the message type enum inside the MessageData class
-                _messageTypeField = Fields.getPublic(_dataField.FieldType, "messageType");
+            _messageTypeField = AccessTools.Field(_dataProperty.FieldType, "messageType");
-            } catch (AccessHelperException e) {
+            if (_messageTypeField == null) {
-                _logger.Error($"Error occured while preparing message copy patch: {e}");
+                _logger.Error("Cannot get field messageRichText of message data");
                 return false;
             }
 
@@ -45,7 +60,7 @@ namespace ConsoleImprovements.Client {
         // Prefix the CopyMessageToClipboard method with this one
         // ReSharper disable once InconsistentNaming
         public static bool Hook(Message __instance) {
-            var dataInstance = _dataField!.GetValue(__instance);
+            var dataInstance = _dataProperty!.GetValue(__instance);
             if (dataInstance == null) {
                 _logger.Error("Cannot get instance of MessageData");
                 return true; // Resume original functionality

ConsoleImprovements/ConsoleImprovements/src/client/Trie.cs

@@ -1,87 +0,0 @@
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-
-namespace ConsoleImprovements.Client {
-    
-    /// <summary>
-    /// A crude implementation of the trie data structure for fast lookups
-    /// </summary>
-    public class Trie {
-        private readonly Node _root = new Node() {
-            Children = new Dictionary<char, Node>()
-        };
-
-        /// <summary>
-        /// Add a new string entry to the structure
-        /// </summary>
-        public void AddEntry(string str) {
-            Node current = _root;
-            foreach (char ch in str.ToLower()) {
-                if (!current.Children.TryGetValue(ch, out Node? next)) {
-                    next = new Node();
-                    current.Children[ch] = next;
-                }
-
-                current = next;
-            }
-
-            current.End = true;
-        }
-
-        /// <summary>
-        /// Look up how the given string may continue
-        /// </summary>
-        /// <param name="prefix">The string to base the search on</param>
-        /// <param name="node">outputs the reached node which is either an end node or has multiple children</param>
-        /// <returns>Continuation of the prefix (including the prefix)</returns>
-        public string? LookupNext(string prefix, out Node? node) {
-            string lowercase = prefix.ToLower();
-            Node current = _root;
-            foreach (char ch in lowercase) {
-                if (!current.Children.TryGetValue(ch, out Node? next)) {
-                    node = null;
-                    return null;
-                }
-
-                current = next;
-            }
-
-            var commonBuilder = new StringBuilder(lowercase);
-            while (current.Children.Count == 1) {
-                if (current.End) {
-                    break;
-                }
-
-                KeyValuePair<char, Node> single = current.Children.First();
-                commonBuilder.Append(single.Key);
-                current = single.Value;
-            }
-
-            node = current;
-            return commonBuilder.ToString();
-        }
-
-        public class Node {
-            public bool End;
-            public Dictionary<char, Node> Children = new Dictionary<char, Node>();
-
-            /// <summary>
-            /// Get all possible strings that can be made from the children of this node
-            /// </summary>
-            /// <param name="prefix">Pass in the previously obtained string path to this node</param>
-            public List<string> CollectChildren(string prefix) {
-                var collected = new List<string>();
-                if (End) {
-                    collected.Add(prefix.ToLower());
-                }
-
-                foreach ((char ch, Node node) in Children) {
-                    collected.AddRange(node.CollectChildren(prefix + ch));
-                }
-
-                return collected;
-            }
-        }
-    }
-}

README.md

@@ -1,9 +1,8 @@
 # Console Improvements Mod
 
-This mod does 3 things:
+This mod does 2 things:
 1. It removes the `> ` prefix from copied commands 
 2. Adds a command history that can be cycled using up and down arrow keys
-3. Adds tab completion
 
 ## Dependencies