Fix Unexpected Intercept result when multiple plugin register the global hotkey event

Author: taooceros

Flow.Launcher.Infrastructure/Hotkey/GlobalHotkey.cs

@@ -79,7 +79,7 @@ private static IntPtr LowLevelKeyboardProc(int nCode, UIntPtr wParam, IntPtr lPa
             {
                 return InterceptKeys.CallNextHookEx(hookId, nCode, wParam, lParam);
             }
-            return (IntPtr)1;
+            return (IntPtr)(-1);
         }
 
         public static void Dispose()

Flow.Launcher.Plugin/Interfaces/IPublicAPI.cs

@@ -104,7 +104,21 @@ public interface IPublicAPI
         /// Fired after global keyboard events
         /// if you want to hook something like Ctrl+R, you should use this event
         /// </summary>
+        [Obsolete("Unable to Retrieve correct return value")]
         event FlowLauncherGlobalKeyboardEventHandler GlobalKeyboardEvent;
+        
+        /// <summary>
+        /// Register a callback for Global Keyboard Event
+        /// </summary>
+        /// <param name="callback"></param>
+        public void RegisterGlobalKeyboardCallback(Func<int, int, SpecialKeyState, bool> callback);
+        
+        /// <summary>
+        /// Remove a callback for Global Keyboard Event
+        /// </summary>
+        /// <param name="callback"></param>
+        public void RemoveGlobalKeyboardCallback(Func<int, int, SpecialKeyState, bool> callback);
+
 
         /// <summary>
         /// Fuzzy Search the string with the given query. This is the core search mechanism Flow uses

Flow.Launcher/PublicAPIInstance.cs

@@ -40,7 +40,7 @@ public PublicAPIInstance(SettingWindowViewModel settingsVM, MainViewModel mainVM
             _settingsVM = settingsVM;
             _mainVM = mainVM;
             _alphabet = alphabet;
-            GlobalHotkey.hookedKeyboardCallback += KListener_hookedKeyboardCallback;
+            GlobalHotkey.hookedKeyboardCallback = KListener_hookedKeyboardCallback;
             WebRequest.RegisterPrefix("data", new DataWebRequestFactory());
         }
 
@@ -190,25 +190,35 @@ public void OpenDirectory(string DirectoryPath, string FileName = null)
                 Arguments = FileName is null ?
                     explorerInfo.DirectoryArgument.Replace("%d", DirectoryPath) :
                     explorerInfo.FileArgument.Replace("%d", DirectoryPath).Replace("%f",
-                    Path.IsPathRooted(FileName) ? FileName : Path.Combine(DirectoryPath, FileName))
+                        Path.IsPathRooted(FileName) ? FileName : Path.Combine(DirectoryPath, FileName))
             };
             explorer.Start();
         }
 
         public event FlowLauncherGlobalKeyboardEventHandler GlobalKeyboardEvent;
 
+        private readonly List<Func<int, int, SpecialKeyState, bool>> _globalKeyboardHandlers = new();
+
+        public void RegisterGlobalKeyboardCallback(Func<int, int, SpecialKeyState, bool> callback) => _globalKeyboardHandlers.Add(callback);
+        public void RemoveGlobalKeyboardCallback(Func<int, int, SpecialKeyState, bool> callback) => _globalKeyboardHandlers.Remove(callback);
+
         #endregion
 
         #region Private Methods
 
         private bool KListener_hookedKeyboardCallback(KeyEvent keyevent, int vkcode, SpecialKeyState state)
         {
+            var continueHook = true;
             if (GlobalKeyboardEvent != null)
             {
-                return GlobalKeyboardEvent((int)keyevent, vkcode, state);
+                continueHook = GlobalKeyboardEvent((int)keyevent, vkcode, state);
+            }
+            foreach (var x in _globalKeyboardHandlers)
+            {
+                continueHook &= x((int)keyevent, vkcode, state);
             }
 
-            return true;
+            return continueHook;
         }
 
         #endregion

Plugins/Flow.Launcher.Plugin.Shell/Main.cs

@@ -300,7 +300,7 @@ public void Init(PluginInitContext context)
         {
             this.context = context;
             _settings = context.API.LoadSettingJsonStorage<Settings>();
-            context.API.GlobalKeyboardEvent += API_GlobalKeyboardEvent;
+            context.API.RegisterGlobalKeyboardCallback(API_GlobalKeyboardEvent);
         }
 
         bool API_GlobalKeyboardEvent(int keyevent, int vkcode, SpecialKeyState state)