fix(tui): distinguish disconnect actions

packages/tui/src/component/dialog-integration.tsx

@@ -123,6 +123,7 @@ function manageConnections(
   onConnected?: OnIntegrationConnected,
 ) {
   dialog.replace(() => {
+    const theme = useTheme("elevated")
     const data = useData()
     const client = useClient()
     const toast = useToast()
@@ -142,6 +143,9 @@ function manageConnections(
           ...credentialConnections(integration).map((connection) => ({
             title: `Disconnect ${connection.label}`,
             value: connection.id,
+            category: "Connected accounts",
+            bg: theme.background.action.destructive.focused,
+            fg: theme.text.action.destructive.focused,
             onSelect: () => {
               void client.api.credential
                 .remove({ credentialID: connection.id, location: location(data) })

packages/tui/src/component/dialog-pair.tsx

@@ -3,7 +3,6 @@ import { useTerminalDimensions } from "@opentui/solid"
 import { createMemo, createResource, createSignal, For, Show } from "solid-js"
 import { renderUnicodeCompact } from "uqr"
 import { useClient } from "../context/client"
-import { Keymap } from "../context/keymap"
 import { useTheme } from "../context/theme"
 import { useDialog } from "../ui/dialog"
 import { errorMessage } from "../util/error"
@@ -21,22 +20,6 @@ export function DialogPair(props: { credentials?: DialogPairCredentials }) {
   const [loadError, setLoadError] = createSignal<unknown>()
   const [showPassword, setShowPassword] = createSignal(false)
   const [passwordHover, setPasswordHover] = createSignal(false)
-  const shortcuts = Keymap.useShortcuts()
-  const togglePassword = () => {
-    setShowPassword((current) => !current)
-  }
-
-  Keymap.createLayer(() => ({
-    mode: "modal",
-    commands: [
-      {
-        id: "dialog.pair.toggle_password",
-        title: showPassword() ? "Hide pairing password" : "Show pairing password",
-        group: "Dialog",
-        run: togglePassword,
-      },
-    ],
-  }))
 
   dialog.setSize("large")
   dialog.setCentered(true)
@@ -78,14 +61,10 @@ export function DialogPair(props: { credentials?: DialogPairCredentials }) {
               wrapMode="word"
               onMouseOver={() => setPasswordHover(true)}
               onMouseOut={() => setPasswordHover(false)}
-              onMouseUp={togglePassword}
+              onMouseUp={() => setShowPassword((current) => !current)}
             >
               {showPassword() ? value.password : "************"}
             </text>
-            <text fg={theme.text.default} onMouseUp={togglePassword}>
-              {shortcuts.get("dialog.pair.toggle_password")} {" "}
-              <span style={{ fg: theme.text.subdued }}>{showPassword() ? "hide password" : "show password"}</span>
-            </text>
           </box>
           <Show when={value.urls.some((url) => ["localhost", "127.0.0.1", "[::1]"].includes(new URL(url).hostname))}>
             <text fg={theme.text.subdued} wrapMode="word">

packages/tui/src/config/v1/keybind.ts

@@ -220,7 +220,6 @@ export const Definitions = {
   "dialog.move_session.new": keybind("ctrl+m", "New project copy"),
   "dialog.move_session.delete": keybind("ctrl+d", "Delete project copy"),
   "dialog.move_session.refresh": keybind("ctrl+r", "Refresh project copies"),
-  "dialog.pair.toggle_password": keybind("space", "Show or hide pairing password"),
   "prompt.autocomplete.prev": keybind("up,ctrl+p", "Move to previous autocomplete item"),
   "prompt.autocomplete.next": keybind("down,ctrl+n", "Move to next autocomplete item"),
   "prompt.autocomplete.hide": keybind("escape", "Hide autocomplete"),

packages/tui/test/keybind.test.ts

@@ -5,7 +5,3 @@ test("binds agent cycling only to shift+tab by default", () => {
   expect(TuiKeybind.Definitions.agent_cycle.default).toBe("shift+tab")
   expect(TuiKeybind.Definitions.agent_cycle_reverse.default).toBe("none")
 })
-
-test("binds pairing password visibility to space by default", () => {
-  expect(TuiKeybind.Definitions["dialog.pair.toggle_password"].default).toBe("space")
-})