fix(m9/T24b): try/finally around _busy in move dialog (circuit-teardown lock)

2026-06-18 11:59:44 -04:00
parent b3d99248fa
commit 1ca2e0b130
1 changed files with 15 additions and 9 deletions
@@ -88,18 +88,24 @@

        _busy = true;
        _error = null;
-        var result = await MoveService.MoveAsync(ConnectionId, target);
-        _busy = false;
+        try
+        {
+            var result = await MoveService.MoveAsync(ConnectionId, target);

-        if (result.Success)
-        {
-            await IsVisibleChanged.InvokeAsync(false);
-            await OnMoved.InvokeAsync();
+            if (result.Success)
+            {
+                await IsVisibleChanged.InvokeAsync(false);
+                await OnMoved.InvokeAsync();
+            }
+            else
+            {
+                // Surface the server guard error inline; keep the dialog open.
+                _error = result.Error ?? "Move failed.";
+            }
        }
-        else
+        finally
        {
-            // Surface the server guard error inline; keep the dialog open.
-            _error = result.Error ?? "Move failed.";
+            _busy = false;
        }
    }
 }