From 6b4a4410d33f70c7cc082bfe427c68f1736ec7f7 Mon Sep 17 00:00:00 2001
From: Hermes <hermes@niklashmotion.art>
Date: Sun, 12 Jul 2026 13:47:06 +0000
Subject: [PATCH] =?UTF-8?q?feat:=20shelf=20button=20opens=20PySide6=20dial?=
 =?UTF-8?q?og=20with=20port=20spinner,=20toggle=20button,=20copy=20?=
 =?UTF-8?q?=E2=80=94=20per=20SideFX=20HOM=20cookbook?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 toolbar/hermes_bridge.shelf | 202 ++++++++++++++++++++++++++----------
 1 file changed, 146 insertions(+), 56 deletions(-)

diff --git a/toolbar/hermes_bridge.shelf b/toolbar/hermes_bridge.shelf
index eb86e86..f57fedb 100644
--- a/toolbar/hermes_bridge.shelf
+++ b/toolbar/hermes_bridge.shelf
@@ -1,80 +1,170 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <shelfDocument>
   <toolshelf name="hermes_bridge" label="Hermes Bridge">
-    <memberTool name="hermes_bridge_toggle"/>
+    <memberTool name="hermes_bridge_dialog"/>
   </toolshelf>
-  <tool name="hermes_bridge_toggle" label="Toggle Bridge" icon="MISC_python">
-    <helpText><![CDATA[Toggle the Hermes-Houdini Bridge.
-Right-click: Set port.]]></helpText>
+  <tool name="hermes_bridge_dialog" label="Hermes Bridge" icon="MISC_python">
+    <helpText><![CDATA[Open the Hermes Bridge control panel.
+Manage port, open/close the bridge, copy connection info.]]></helpText>
     <script scriptType="python"><![CDATA[
+from PySide6 import QtWidgets, QtCore, QtGui
 import hou
 import socket
 import json
 import os
 
-# ── Config ──────────────────────────────────────────
 CONFIG_FILE = os.path.join(hou.homeHoudiniDirectory(), "hermes_bridge.json")
 DEFAULT_PORT = 12345
 
-def load_port():
+def load_config():
     try:
         with open(CONFIG_FILE) as f:
-            return json.load(f).get("port", DEFAULT_PORT)
+            return json.load(f)
     except Exception:
-        return DEFAULT_PORT
+        return {"port": DEFAULT_PORT}
 
-def save_port(p):
+def save_config(cfg):
     with open(CONFIG_FILE, "w") as f:
-        json.dump({"port": p}, f, indent=2)
+        json.dump(cfg, f, indent=2)
 
-def is_open(port):
-    result = hou.hscript("openport")[0]
-    return f"Port {port} is open" in result
+def bridge_is_open(port):
+    return f"Port {port} is open" in hou.hscript("openport")[0]
 
-def toggle(port):
-    if is_open(port):
-        hou.hscript(f"closeport {port}")
-        hou.ui.displayMessage(
-            f"Bridge CLOSED on port {port}",
-            severity=hou.severityType.ImportantMessage,
-            title="Hermes Bridge"
-        )
-    else:
-        hou.hscript(f"closeport {port}")
-        hou.hscript(f"openport {port} 0.0.0.0")
-        host = socket.gethostname()
-        hou.ui.displayMessage(
-            f"Bridge OPEN \u2014 {host}:{port}\n\n"
-            f"Tell Hermes:\n"
-            f'"I have Houdini open on {host} (port {port})"',
-            severity=hou.severityType.ImportantMessage,
-            title="Hermes Bridge"
-        )
+# ── Dialog ──────────────────────────────────────────
+class HermesBridgeDialog(QtWidgets.QDialog):
+    def __init__(self, parent=None):
+        super().__init__(parent)
+        self.cfg = load_config()
+        self.port = self.cfg.get("port", DEFAULT_PORT)
+        self.hostname = socket.gethostname()
+        self._build_ui()
+        self._refresh()
 
-# ── Right-click: set port ───────────────────────────
-kwargs = kwargs if "kwargs" in dir() else {}
-if kwargs.get("altclick") or kwargs.get("ctrlclick"):
-    port = load_port()
-    new = hou.ui.readInput(
-        "Bridge port number",
-        buttons=("OK", "Cancel"),
-        default_choice=0,
-        title="Hermes Bridge \u2014 Port",
-        initial_contents=str(port)
-    )
-    if new and new[0]:
-        try:
-            new_port = int(new[1].strip())
-            save_port(new_port)
-            hou.ui.displayMessage(
-                f"Port set to {new_port}",
-                severity=hou.severityType.Message,
-                title="Hermes Bridge"
-            )
-        except ValueError:
-            pass
-else:
-    toggle(load_port())
+    def _build_ui(self):
+        self.setWindowTitle("Hermes Bridge")
+        self.setMinimumWidth(420)
+        self.setWindowFlags(self.windowFlags() | QtCore.Qt.WindowStaysOnTopHint)
+
+        # Dark theme
+        self.setStyleSheet("""
+            QDialog { background: #1e1e1e; color: #e0e0e0; }
+            QLabel { color: #ccc; }
+            QLabel#title { font-size: 16px; font-weight: bold; color: #fff; }
+            QLabel#status { font-size: 14px; font-weight: bold; }
+            QPushButton { border: none; border-radius: 4px; padding: 8px 16px;
+                          font-weight: bold; font-size: 13px; }
+            QPushButton#btnToggle { padding: 12px; font-size: 15px; }
+            QPushButton#btnToggle[active="true"] {
+                background: #5c1a1a; color: #ff7777; border: 1px solid #a44; }
+            QPushButton#btnToggle[active="false"] {
+                background: #1a4a1a; color: #77cc77; border: 1px solid #4a4; }
+            QPushButton#btnToggle:hover { opacity: 0.9; }
+            QPushButton#btnCopy { background: #1565c0; color: #fff; }
+            QPushButton#btnCopy:hover { background: #1976d2; }
+            QSpinBox { background: #2d2d2d; border: 1px solid #555; color: #e0e0e0;
+                       padding: 5px 8px; border-radius: 3px; }
+            QLineEdit { background: #2d2d2d; border: 1px solid #555; color: #aaa;
+                        padding: 5px 8px; border-radius: 3px; }
+        """)
+
+        layout = QtWidgets.QVBoxLayout(self)
+        layout.setSpacing(12)
+        layout.setContentsMargins(20, 20, 20, 20)
+
+        # Title
+        title = QtWidgets.QLabel("\U0001f916  Hermes Bridge")
+        title.setObjectName("title")
+        layout.addWidget(title)
+
+        # Status
+        self.status_label = QtWidgets.QLabel()
+        self.status_label.setObjectName("status")
+        layout.addWidget(self.status_label)
+
+        # Port
+        port_row = QtWidgets.QHBoxLayout()
+        port_row.addWidget(QtWidgets.QLabel("Port:"))
+        self.port_spin = QtWidgets.QSpinBox()
+        self.port_spin.setRange(1024, 65535)
+        self.port_spin.setValue(self.port)
+        self.port_spin.setFixedWidth(100)
+        self.port_spin.valueChanged.connect(self._on_port_changed)
+        port_row.addWidget(self.port_spin)
+        port_row.addStretch()
+        layout.addLayout(port_row)
+
+        # Toggle button
+        self.btn_toggle = QtWidgets.QPushButton()
+        self.btn_toggle.setObjectName("btnToggle")
+        self.btn_toggle.clicked.connect(self._toggle)
+        layout.addWidget(self.btn_toggle)
+
+        # Prompt text
+        layout.addWidget(QtWidgets.QLabel("Tell Hermes:"))
+        prompt_row = QtWidgets.QHBoxLayout()
+        self.prompt_field = QtWidgets.QLineEdit()
+        self.prompt_field.setReadOnly(True)
+        prompt_row.addWidget(self.prompt_field)
+        self.btn_copy = QtWidgets.QPushButton("Copy")
+        self.btn_copy.setObjectName("btnCopy")
+        self.btn_copy.clicked.connect(self._copy)
+        prompt_row.addWidget(self.btn_copy)
+        layout.addLayout(prompt_row)
+
+        layout.addStretch()
+
+    def _refresh(self):
+        is_open = bridge_is_open(self.port)
+        if is_open:
+            self.status_label.setText(
+                f"\u25cf  Connected \u2014 {self.hostname}:{self.port}")
+            self.status_label.setStyleSheet(
+                "color: #4caf50; font-size: 14px; font-weight: bold;")
+            self.btn_toggle.setText("Close Bridge")
+            self.btn_toggle.setProperty("active", "true")
+        else:
+            self.status_label.setText("\u25cf  Not connected")
+            self.status_label.setStyleSheet(
+                "color: #ff5555; font-size: 14px; font-weight: bold;")
+            self.btn_toggle.setText("Open Bridge")
+            self.btn_toggle.setProperty("active", "false")
+        # Force style refresh
+        self.btn_toggle.style().unpolish(self.btn_toggle)
+        self.btn_toggle.style().polish(self.btn_toggle)
+        self.prompt_field.setText(
+            f"Hey Hermes, I have a Houdini session running "
+            f"on {self.hostname} (port {self.port}).")
+
+    def _toggle(self):
+        if bridge_is_open(self.port):
+            hou.hscript(f"closeport {self.port}")
+        else:
+            hou.hscript(f"closeport {self.port}")
+            hou.hscript(f"openport {self.port} 0.0.0.0")
+        self._refresh()
+
+    def _on_port_changed(self, value):
+        was_open = bridge_is_open(self.port)
+        if was_open:
+            hou.hscript(f"closeport {self.port}")
+        self.port = value
+        self.cfg["port"] = value
+        save_config(self.cfg)
+        if was_open:
+            hou.hscript(f"closeport {self.port}")
+            hou.hscript(f"openport {self.port} 0.0.0.0")
+        self._refresh()
+
+    def _copy(self):
+        QtWidgets.QApplication.clipboard().setText(
+            self.prompt_field.text())
+        self.btn_copy.setText("\u2713 Copied!")
+        QtCore.QTimer.singleShot(2000,
+            lambda: self.btn_copy.setText("Copy"))
+
+# ── Show dialog ─────────────────────────────────────
+dialog = HermesBridgeDialog(hou.ui.mainQtWindow())
+dialog.show()
 ]]></script>
   </tool>
 </shelfDocument>