From 8258b044b194812a80b8d438f58bb729f36fa31f Mon Sep 17 00:00:00 2001
From: Chili Palmer <hugoms@me.com>
Date: Sun, 26 Apr 2026 06:45:37 +0200
Subject: [PATCH] fix: hotkeys back working

Co-authored-by: Copilot <copilot@github.com>
---
 lib/bds/desktop/automation.ex              |  9 +++++++
 lib/bds/desktop/shell_live.ex              | 17 +++++++++++++
 lib/bds/desktop/shell_live/index.html.heex |  2 +-
 lib/bds/ui/commands.ex                     |  4 ++-
 priv/ui/live.js                            | 13 +++++++++-
 scripts/desktop_automation_runner.mjs      |  7 ++++++
 test/bds/desktop/automation_test.exs       | 15 +++++++++++
 test/bds/desktop/shell_live_test.exs       | 29 ++++++++++++++++++++++
 8 files changed, 93 insertions(+), 3 deletions(-)

diff --git a/lib/bds/desktop/automation.ex b/lib/bds/desktop/automation.ex
index 8f2c9e2..e907fba 100644
--- a/lib/bds/desktop/automation.ex
+++ b/lib/bds/desktop/automation.ex
@@ -24,6 +24,10 @@ defmodule BDS.Desktop.Automation do
     GenServer.call(session, {:click, selector}, @request_timeout)
   end
 
+  def press(session, shortcut) when is_binary(shortcut) do
+    GenServer.call(session, {:press, shortcut}, @request_timeout)
+  end
+
   def capture_screenshot(session, destination) when is_binary(destination) do
     GenServer.call(session, {:capture_screenshot, destination}, @request_timeout)
   end
@@ -78,6 +82,11 @@ defmodule BDS.Desktop.Automation do
     {:reply, normalize_simple_reply(reply), state}
   end
 
+  def handle_call({:press, shortcut}, _from, state) do
+    {reply, state} = driver_request(state, %{"command" => "press", "shortcut" => shortcut})
+    {:reply, normalize_simple_reply(reply), state}
+  end
+
   def handle_call({:capture_screenshot, destination}, _from, state) do
     File.mkdir_p!(Path.dirname(destination))
 
diff --git a/lib/bds/desktop/shell_live.ex b/lib/bds/desktop/shell_live.ex
index 57a8f8f..3392f42 100644
--- a/lib/bds/desktop/shell_live.ex
+++ b/lib/bds/desktop/shell_live.ex
@@ -6,6 +6,7 @@ defmodule BDS.Desktop.ShellLive do
   import Phoenix.HTML
 
   alias BDS.Desktop.ShellData
+  alias BDS.UI.Commands
   alias BDS.UI.Registry
   alias BDS.UI.Workbench
 
@@ -65,6 +66,14 @@ defmodule BDS.Desktop.ShellLive do
     {:noreply, open_sidebar_item(socket, params, :pin)}
   end
 
+  def handle_event("shortcut", params, socket) do
+    if ignore_shortcut?(params) do
+      {:noreply, socket}
+    else
+      {:noreply, reload_shell(socket, Commands.handle_shortcut(socket.assigns.workbench, params))}
+    end
+  end
+
   def handle_event("select_tab", %{"type" => type, "id" => id}, socket) do
     workbench =
       Workbench.open_tab(socket.assigns.workbench, String.to_existing_atom(type), id, :preview)
@@ -430,6 +439,14 @@ defmodule BDS.Desktop.ShellLive do
     Enum.find(tabs, &(&1.type == type and &1.id == id))
   end
 
+  defp ignore_shortcut?(params) do
+    Map.get(params, "alt", false) or
+      Map.get(params, "contentEditable", false) or
+      Map.get(params, "content_editable", false) or
+      Map.get(params, "tag") in ["INPUT", "TEXTAREA", "SELECT"] or
+      Map.get(params, :tag) in ["INPUT", "TEXTAREA", "SELECT"]
+  end
+
   defp open_sidebar_item(socket, params, intent) do
     route_atom = sidebar_route_atom(Map.fetch!(params, "route"))
     tab_id = tab_id_for_route(route_atom, Map.fetch!(params, "id"))
diff --git a/lib/bds/desktop/shell_live/index.html.heex b/lib/bds/desktop/shell_live/index.html.heex
index 1c4534f..631eb3e 100644
--- a/lib/bds/desktop/shell_live/index.html.heex
+++ b/lib/bds/desktop/shell_live/index.html.heex
@@ -1,4 +1,4 @@
-<div class="app" id="bds-shell-app">
+<div class="app" id="bds-shell-app" phx-window-keydown="shortcut">
   <div class="window-titlebar" data-region="title-bar">
     <div class="window-titlebar-menu-bar is-hidden">
       <button class="window-titlebar-menu-button" type="button">File</button>
diff --git a/lib/bds/ui/commands.ex b/lib/bds/ui/commands.ex
index 980cc40..7a173e8 100644
--- a/lib/bds/ui/commands.ex
+++ b/lib/bds/ui/commands.ex
@@ -33,7 +33,9 @@ defmodule BDS.UI.Commands do
 
   def handle_shortcut(state, shortcut) when is_map(shortcut) do
     key = shortcut |> Map.get(:key, Map.get(shortcut, "key", "")) |> String.downcase()
-    primary = Map.get(shortcut, :meta, false) or Map.get(shortcut, :ctrl, false)
+    primary =
+      Map.get(shortcut, :meta, Map.get(shortcut, "meta", false)) or
+        Map.get(shortcut, :ctrl, Map.get(shortcut, "ctrl", false))
 
     case Enum.find(@menu_shortcuts, &shortcut_match?(&1, key, primary)) do
       %{id: command_id} -> MenuBar.execute(state, command_id)
diff --git a/priv/ui/live.js b/priv/ui/live.js
index cfc168e..9801c2b 100644
--- a/priv/ui/live.js
+++ b/priv/ui/live.js
@@ -32,7 +32,18 @@ document.addEventListener("DOMContentLoaded", () => {
 
   const liveSocket = new LiveView.LiveSocket("/live", Phoenix.Socket, {
     params: { _csrf_token: csrfToken },
-    hooks: Hooks
+    hooks: Hooks,
+    metadata: {
+      keydown: (event) => ({
+        key: event.key,
+        meta: event.metaKey,
+        ctrl: event.ctrlKey,
+        alt: event.altKey,
+        shift: event.shiftKey,
+        tag: event.target?.tagName || null,
+        contentEditable: event.target?.isContentEditable || false
+      })
+    }
   });
 
   liveSocket.connect();
diff --git a/scripts/desktop_automation_runner.mjs b/scripts/desktop_automation_runner.mjs
index c82e67a..a68e792 100644
--- a/scripts/desktop_automation_runner.mjs
+++ b/scripts/desktop_automation_runner.mjs
@@ -63,6 +63,13 @@ for await (const line of rl) {
       continue;
     }
 
+    if (message.command === "press") {
+      await page.keyboard.press(message.shortcut);
+      await page.waitForTimeout(50);
+      console.log(JSON.stringify({ ref, status: "ok", result: "ok" }));
+      continue;
+    }
+
     if (message.command === "screenshot") {
       await page.screenshot({ path: message.path, fullPage: false });
       console.log(JSON.stringify({ ref, status: "ok", result: message.path }));
diff --git a/test/bds/desktop/automation_test.exs b/test/bds/desktop/automation_test.exs
index 8383bef..871fcec 100644
--- a/test/bds/desktop/automation_test.exs
+++ b/test/bds/desktop/automation_test.exs
@@ -45,6 +45,21 @@ defmodule BDS.Desktop.AutomationTest do
     snapshot = Automation.snapshot(session)
     assert snapshot.sidebar_visible == false
 
+    assert :ok = Automation.press(session, "Meta+B")
+
+    snapshot = Automation.snapshot(session)
+    assert snapshot.sidebar_visible == true
+
+    assert :ok = Automation.press(session, "Meta+J")
+
+    snapshot = Automation.snapshot(session)
+    assert snapshot.panel_visible == true
+
+    assert :ok = Automation.press(session, "Meta+J")
+
+    snapshot = Automation.snapshot(session)
+    assert snapshot.panel_visible == false
+
     assert :ok = Automation.click(session, "[data-testid='toggle-assistant']")
 
     snapshot = Automation.snapshot(session)
diff --git a/test/bds/desktop/shell_live_test.exs b/test/bds/desktop/shell_live_test.exs
index 7d5e1b5..5c03dae 100644
--- a/test/bds/desktop/shell_live_test.exs
+++ b/test/bds/desktop/shell_live_test.exs
@@ -125,4 +125,33 @@ defmodule BDS.Desktop.ShellLiveTest do
     assert String.contains?(html, ">hero.png<")
     assert String.contains?(html, ">cover.png<")
   end
+
+  test "global shortcuts route through the shared command model" do
+    {:ok, view, html} = live_isolated(build_conn(), BDS.Desktop.ShellLive)
+
+    assert html =~ ~s(data-testid="sidebar-shell")
+    assert html =~ ~s(class="panel-shell is-hidden")
+
+    html = render_keydown(view, "shortcut", %{key: "b", meta: true})
+    assert html =~ ~s(class="sidebar-shell is-hidden")
+
+    html = render_keydown(view, "shortcut", %{key: "j", meta: true})
+    refute html =~ ~s(class="panel-shell is-hidden")
+
+    html = render_keydown(view, "shortcut", %{key: "2", meta: true})
+    assert html =~ ~s(data-view="media")
+
+    html =
+      render_click(view, "pin_sidebar_item", %{
+        "route" => "media",
+        "id" => "media-1",
+        "title" => "hero.png",
+        "subtitle" => "12 KB"
+      })
+
+    assert html =~ ~s(data-tab-id="media-1")
+
+    html = render_keydown(view, "shortcut", %{key: "w", meta: true})
+    refute html =~ ~s(data-tab-id="media-1")
+  end
 end