Optimise AFK module

Author: VolcanicArts

VRCOSC.Game/Processes/ProcessExtensions.cs

@@ -27,7 +27,7 @@ public static async Task PressKeys(IEnumerable<User32.VirtualKey> keys, int pres
     {
         var buffer = new char[256];
         var handle = User32.GetForegroundWindow();
-        return User32.GetWindowText(handle, buffer, 256) > 0 ? Encoding.UTF8.GetString(Encoding.UTF8.GetBytes(buffer)) : null;
+        return User32.GetWindowText(handle, buffer, 256) > 0 ? Encoding.UTF8.GetString(Encoding.UTF8.GetBytes(buffer)).TrimEnd('\u0000') : null;
     }
 
     public static void ShowMainWindow(this Process process, User32.WindowShowStyle style) => User32.ShowWindow(process.MainWindowHandle, style);

VRCOSC.Modules/AFK/AFKModule.cs

@@ -2,7 +2,6 @@
 // See the LICENSE file in the repository root for full license text.
 
 using VRCOSC.Game.Modules.ChatBox;
-using VRCOSC.Game.OSC.VRChat;
 using VRCOSC.Game.Processes;
 
 namespace VRCOSC.Modules.AFK;
@@ -13,7 +12,7 @@ public class AFKModule : ChatBoxModule
     public override string Description => "Display text and time since going AFK";
     public override string Author => "VolcanicArts";
     public override ModuleType Type => ModuleType.General;
-    protected override TimeSpan DeltaUpdate => VRChatOscConstants.UPDATE_TIME_SPAN;
+    protected override TimeSpan DeltaUpdate => TimeSpan.FromSeconds(1);
 
     private DateTime? afkBegan;
 
@@ -38,6 +37,14 @@ protected override void OnModuleStart()
     }
 
     protected override void OnModuleUpdate()
+    {
+        SetVariableValue(AFKVariable.FocusedWindow, ProcessExtensions.GetActiveWindowTitle() ?? "None");
+        SetVariableValue(AFKVariable.Duration, afkBegan is null ? null : (DateTime.Now - afkBegan.Value).ToString(@"hh\:mm\:ss"));
+        SetVariableValue(AFKVariable.Since, afkBegan?.ToString(@"hh\:mm"));
+        ChangeStateTo(afkBegan is null ? AFKState.NotAFK : AFKState.AFK);
+    }
+
+    protected override void OnPlayerUpdate()
     {
         if (Player.AFK is null)
         {
@@ -48,7 +55,6 @@ protected override void OnModuleUpdate()
         if (Player.AFK.Value && afkBegan is null)
         {
             afkBegan = DateTime.Now;
-            SetVariableValue(AFKVariable.FocusedWindow, ProcessExtensions.GetActiveWindowTitle() ?? "None");
             TriggerEvent(AFKEvent.AFKStarted);
         }
 
@@ -57,10 +63,6 @@ protected override void OnModuleUpdate()
             afkBegan = null;
             TriggerEvent(AFKEvent.AFKStopped);
         }
-
-        SetVariableValue(AFKVariable.Duration, afkBegan is null ? null : (DateTime.Now - afkBegan.Value).ToString(@"hh\:mm\:ss"));
-        SetVariableValue(AFKVariable.Since, afkBegan?.ToString(@"hh\:mm"));
-        ChangeStateTo(afkBegan is null ? AFKState.NotAFK : AFKState.AFK);
     }
 
     private enum AFKVariable