`c9a8dd6`

fix: add cooldown after entering ReceivedControl to prevent double navigation

When Super+Arrow triggers a transfer, the keypress gets forwarded to the
receiving machine and causes Hyprland to fire IPC Move. This resulted in
focus moving an extra window after the transfer.

Add a 1-second cooldown after entering ReceivedControl during which all
IPC Move events are ignored.

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 4 months ago

SHA: c9a8dd6ad5e20da128595c2f198fd6effae877d8
Parents: 6d08dd0
Tree: d8f072b

1 changed file

Status	File	+	-
M	`hyprkvm-daemon/src/main.rs`	13	0

hyprkvm-daemon/src/main.rsmodified

                          let current_state = transfer_manager.state().await;
                          info!("IPC Move {:?}: state={:?}", direction, current_state);
 +                        // Early exit: if in ReceivedControl and within cooldown, ignore
 +                        // (prevents the Super+Arrow keypress that triggered the transfer from
 +                        // causing a double navigation on the receiving machine)
 +                        if let transfer::TransferState::ReceivedControl { entered_at, .. } = &current_state {
 +                            const RECEIVED_CONTROL_IPC_COOLDOWN_MS: u128 = 1000;
 +                            let time_in_state = entered_at.elapsed().as_millis();
 +                            if time_in_state < RECEIVED_CONTROL_IPC_COOLDOWN_MS {
 +                                tracing::info!("IPC Move {:?}: in ReceivedControl cooldown ({}ms), ignoring", direction, time_in_state);
 +                                response_tx.send(IpcResponse::Ok { message: "in cooldown".to_string() }).ok();
 +                                continue;
 +                            }
 +                        }
++
                          // If we're already initiating a transfer in this direction, skip entirely
                          // (prevents double-action when both RECOVERY hotkey and IPC Move fire)
                          if let transfer::TransferState::Initiating { target, .. } = &current_state {