From 4e713682e49698f50c004912f998e5fc14a9554d Mon Sep 17 00:00:00 2001
From: Zedhugh Chen <zedhugh@gmail.com>
Date: Tue, 9 Jun 2026 23:05:54 +0800
Subject: [PATCH] =?UTF-8?q?refactor(policy):=20=E7=BB=9F=E4=B8=80=20output?=
 =?UTF-8?q?=20=E5=88=87=E6=8D=A2=E9=80=BB=E8=BE=91=E5=B9=B6=E6=B7=BB?=
 =?UTF-8?q?=E5=8A=A0=20workspace=20=E5=88=87=E6=8D=A2=E9=80=9A=E7=9F=A5?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/core/policy.c | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/src/core/policy.c b/src/core/policy.c
index abc0842..df739f0 100644
--- a/src/core/policy.c
+++ b/src/core/policy.c
@@ -14,6 +14,7 @@
 #include "core/command_buffer.h"
 #include "core/event.h"
 #include "core/layout.h"
+#include "core/listeners.h"
 #include "core/plan.h"
 #include "core/rules.h"
 #include "core/state.h"
@@ -1306,11 +1307,25 @@ static void command_window_set_fullscreen(
   fullscreen_window(ctx, data->window, data->state, plan);
 }
 
-static void
-switch_workspace(state_t *state, workspace_id_t workspace_id, plan_t *plan) {
+static inline void change_current_output(
+  state_t *state,
+  output_id_t output_id,
+  const listeners_t *listeners
+) {
+  if (!state_set_current_output(state, output_id)) return;
+  listeners_notify_current_output(listeners, output_id);
+}
+
+static void switch_workspace(
+  const policy_context_t *ctx,
+  workspace_id_t workspace_id,
+  plan_t *plan
+) {
+  auto state     = ctx->state;
   auto workspace = state_workspace_get(state, workspace_id);
   if (!workspace) return;
 
+  auto listeners               = ctx->listeners;
   auto output_id               = workspace->output_id;
   workspace_id_t old_workspace = ZDWM_WORKSPACE_ID_INVALID;
 
@@ -1322,17 +1337,17 @@ switch_workspace(state_t *state, workspace_id_t workspace_id, plan_t *plan) {
       )) {
     plan->need_relayout = true;
     add_switch_workspace_effects(state, old_workspace, workspace_id, plan);
+    listeners_notify_workspace_active(listeners, output_id, workspace_id);
   }
 
-  state_set_current_output(state, output_id);
+  change_current_output(state, output_id, listeners);
 }
 
 static void set_current_output(
   const policy_context_t *ctx,
   const set_current_output_command_t *command
 ) {
-  if (!state_set_current_output(ctx->state, command->output)) return;
-  listeners_notify_current_output(ctx->listeners, command->output);
+  change_current_output(ctx->state, command->output, ctx->listeners);
 }
 
 static void set_layout(
@@ -1461,7 +1476,7 @@ void policy_apply_command(
       command_window_set_fullscreen(ctx, &cmd->as.fullscreen, plan);
       break;
     case ZDWM_COMMAND_SWITCH_WORKSPACE:
-      switch_workspace(state, cmd->as.switch_workspace.workspace, plan);
+      switch_workspace(ctx, cmd->as.switch_workspace.workspace, plan);
       break;
     case ZDWM_COMMAND_SET_CURRENT_OUTPUT:
       set_current_output(ctx, &cmd->as.current_output);