Fix Actions freeze when SmartKey is used (#1643)

Author: BartoszCichecki

LenovoLegionToolkit.Lib.Automation/AutomationProcessor.cs

@@ -123,17 +123,17 @@ public async Task RunNowAsync(AutomationPipeline pipeline)
         if (Log.Instance.IsTraceEnabled)
             Log.Instance.Trace($"Pipeline run now pending...");
 
-        List<AutomationPipeline> pipelines;
-        using (await _ioLock.LockAsync().ConfigureAwait(false))
-            pipelines = _pipelines.ToList();
-
         using (await _runLock.LockAsync().ConfigureAwait(false))
         {
             if (Log.Instance.IsTraceEnabled)
                 Log.Instance.Trace($"Pipeline run starting...");
 
             try
             {
+                List<AutomationPipeline> pipelines;
+                using (await _ioLock.LockAsync().ConfigureAwait(false))
+                    pipelines = _pipelines.ToList();
+
                 var otherPipelines = pipelines.Where(p => p.Id != pipeline.Id).ToList();
                 await pipeline.DeepCopy().RunAsync(otherPipelines).ConfigureAwait(false);
 
@@ -152,14 +152,14 @@ public async Task RunNowAsync(AutomationPipeline pipeline)
 
     public async Task RunNowAsync(Guid pipelineId)
     {
-        using (await _runLock.LockAsync().ConfigureAwait(false))
-        {
-            var pipeline = _pipelines.Where(p => p.Trigger is null).FirstOrDefault(p => p.Id == pipelineId);
-            if (pipeline is null)
-                return;
+        AutomationPipeline? pipeline;
+        using (await _ioLock.LockAsync().ConfigureAwait(false))
+            pipeline = _pipelines.Where(p => p.Trigger is null).FirstOrDefault(p => p.Id == pipelineId);
 
-            await RunNowAsync(pipeline).ConfigureAwait(false);
-        }
+        if (pipeline is null)
+            return;
+
+        await RunNowAsync(pipeline).ConfigureAwait(false);
     }
 
     private async Task RunAsync(IAutomationEvent automationEvent)

LenovoLegionToolkit.WPF/Utils/SmartKeyHelper.cs

@@ -106,7 +106,8 @@ private async Task ProcessSpecialKey(bool isDoublePress)
 
         try
         {
-            var pipeline = (await _automationProcessor.GetPipelinesAsync()).FirstOrDefault(p => p.Id == currentGuid);
+            var pipelines = await _automationProcessor.GetPipelinesAsync();
+            var pipeline = pipelines.FirstOrDefault(p => p.Id == currentGuid);
             if (pipeline is not null)
             {
                 if (Log.Instance.IsTraceEnabled)