pycodex-ws 0.4.5__tar.gz → 0.4.6__tar.gz

{pycodex_ws-0.4.5 → pycodex_ws-0.4.6}/PKG-INFO

@@ -1,11 +1,11 @@
 Metadata-Version: 2.1
 Name: pycodex-ws
-Version: 0.4.5
+Version: 0.4.6
 Summary: Browser workspace and optional Feishu integration for python-codex
 License-File: LICENSE
 Requires-Python: >=3.7
 Requires-Dist: pillow>=9.0.0
-Requires-Dist: python-codex==0.4.5
+Requires-Dist: python-codex==0.4.6
 Requires-Dist: websockets<16,>=11
 Description-Content-Type: text/markdown
 
@@ -96,12 +96,26 @@ Creating a workspace through the management page or API without a prefix
 generates a `/tmp/pcws-*` prefix and saves it in the configuration.
 
 The router directory collects unread workspace shortcuts and their device names
-in a distinct Unread panel at the top. Read workspaces leave that panel; the
-complete device groups keep their device id and workspace configuration order.
+in a distinct Unread panel at the top. Read workspaces leave that panel.
+Drag a handle in Online devices to reorder devices; both lists share the saved
+order. The focused handle also supports the up and down arrow keys. Escape or
+dropping outside the list cancels a drag. On touch screens, drag the handle;
+the rest of the card still scrolls normally. List updates wait until the drag
+and save finish.
+The router saves the order in the existing `CODEX_HOME/router-aliases.json`,
+shared by all browsers and retained across router restarts. Its device entry
+order is the display order; an empty alias means to show the original name.
+Existing alias files remain readable. Offline device positions are
+remembered, newly registered devices go last, and each device's workspaces keep
+their configuration order. Reordering requires router login; if connected
+devices change during a drag, refresh and try again.
 Device groups start collapsed, retain their expanded state during updates, and
-show Manage and the number of working workspaces in their headings. Click the
-device name to rename it; aliases must be nonblank and contain 1 to 128
-characters. Device headings and the Online devices list share an unread dot
+show Manage and the number of working workspaces in their headings. Click a
+device heading or its name to expand or collapse the group. To rename a device,
+use its pencil button in the Online devices list, then press Enter or Save;
+Escape or Cancel discards the edit. Leaving the field does not save it.
+Aliases must be nonblank and contain 1 to 128 characters.
+Device headings and the Online devices list share an unread dot
 whenever any of that device's workspaces is unread. Final replies and failed
 turns both trigger shared unread dots; viewing the reply or error marks it as read.
 

{pycodex_ws-0.4.5 → pycodex_ws-0.4.6}/README.md

@@ -85,12 +85,26 @@ Creating a workspace through the management page or API without a prefix
 generates a `/tmp/pcws-*` prefix and saves it in the configuration.
 
 The router directory collects unread workspace shortcuts and their device names
-in a distinct Unread panel at the top. Read workspaces leave that panel; the
-complete device groups keep their device id and workspace configuration order.
+in a distinct Unread panel at the top. Read workspaces leave that panel.
+Drag a handle in Online devices to reorder devices; both lists share the saved
+order. The focused handle also supports the up and down arrow keys. Escape or
+dropping outside the list cancels a drag. On touch screens, drag the handle;
+the rest of the card still scrolls normally. List updates wait until the drag
+and save finish.
+The router saves the order in the existing `CODEX_HOME/router-aliases.json`,
+shared by all browsers and retained across router restarts. Its device entry
+order is the display order; an empty alias means to show the original name.
+Existing alias files remain readable. Offline device positions are
+remembered, newly registered devices go last, and each device's workspaces keep
+their configuration order. Reordering requires router login; if connected
+devices change during a drag, refresh and try again.
 Device groups start collapsed, retain their expanded state during updates, and
-show Manage and the number of working workspaces in their headings. Click the
-device name to rename it; aliases must be nonblank and contain 1 to 128
-characters. Device headings and the Online devices list share an unread dot
+show Manage and the number of working workspaces in their headings. Click a
+device heading or its name to expand or collapse the group. To rename a device,
+use its pencil button in the Online devices list, then press Enter or Save;
+Escape or Cancel discards the edit. Leaving the field does not save it.
+Aliases must be nonblank and contain 1 to 128 characters.
+Device headings and the Online devices list share an unread dot
 whenever any of that device's workspaces is unread. Final replies and failed
 turns both trigger shared unread dots; viewing the reply or error marks it as read.
 

{pycodex_ws-0.4.5 → pycodex_ws-0.4.6}/pycodex_router/index.html

@@ -40,7 +40,7 @@
     * { box-sizing: border-box; }
     body { margin: 0; min-height: 100vh; background: var(--bg); color: var(--ink); font-size: 13px; }
     a { color: inherit; text-decoration: none; }
-    a:focus-visible, .device-label:focus-visible, .workspace-group-heading:focus-visible { outline: 2px solid var(--accent); outline-offset: 5px; border-radius: 3px; }
+    a:focus-visible, button:focus-visible, .workspace-group-heading:focus-visible { outline: 2px solid var(--accent); outline-offset: 5px; border-radius: 3px; }
     svg { width: 20px; height: 20px; flex: 0 0 auto; fill: none; stroke: currentColor; stroke-width: 1.6; stroke-linecap: round; stroke-linejoin: round; }
     h1, h2, h3, h4, p { margin: 0; }
     code { font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace; }
@@ -64,8 +64,8 @@
     .registration-content { display: grid; gap: 8px; margin-top: 12px; }
     .registration input { width: 100%; min-width: 0; padding: 8px 10px; border: 1px solid var(--line); border-radius: 6px; background: var(--panel); color: var(--ink); font: 12px ui-monospace, SFMono-Regular, Menlo, Consolas, monospace; }
     .registration p { color: var(--muted); font-size: 12px; line-height: 1.6; overflow-wrap: anywhere; }
-    #alias-status { margin-bottom: 12px; color: var(--danger); font-size: 12px; }
-    #alias-status:empty { display: none; }
+    #alias-status, #order-status { margin-bottom: 12px; color: var(--danger); font-size: 12px; }
+    #alias-status:empty, #order-status:empty { display: none; }
     .layout { display: grid; grid-template-columns: minmax(0, 1fr) 280px; gap: 20px; align-items: start; }
     .layout > * { min-width: 0; }
     .section-heading { display: flex; align-items: center; justify-content: space-between; gap: 12px; margin-bottom: 12px; }
@@ -120,9 +120,19 @@
     .device-heading { display: flex; align-items: start; gap: 9px; }
     .device-heading svg { width: 17px; height: 17px; margin-top: 1px; color: var(--muted); }
     .device-name { flex: 1; min-width: 0; font-size: 11px; font-weight: 500; line-height: 1.6; overflow-wrap: anywhere; }
-    .device-label { display: inline-block; max-width: 100%; padding: 0; border: 0; background: none; color: inherit; font: inherit; font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace; text-align: left; overflow-wrap: anywhere; cursor: text; }
-    .device-label[hidden] { display: none; }
+    .device-label { font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace; }
+    .device-rename, .device-drag, .device-alias-actions button { display: inline-flex; align-items: center; justify-content: center; min-height: 28px; padding: 4px 8px; border: 1px solid var(--line); border-radius: 4px; background: var(--panel); color: var(--muted); font: inherit; cursor: pointer; }
+    .device-rename:hover, .device-drag:hover, .device-alias-actions button:hover { background: var(--soft); color: var(--ink); }
+    .device-rename, .device-drag { flex: 0 0 28px; padding: 4px; border-color: transparent; }
+    .device-rename[hidden] { display: none; }
+    .device-rename svg, .device-drag svg { width: 14px; height: 14px; margin: 0; color: inherit; }
+    .device-drag { touch-action: none; user-select: none; cursor: grab; }
+    .device-drag:disabled { opacity: .35; cursor: default; }
+    .device[data-dragging] { background: var(--soft); opacity: .65; }
+    .device-drop-before { box-shadow: 0 -2px var(--accent); }
+    .nodes.device-drop-end { box-shadow: inset 0 -2px var(--accent); }
     .device-alias-input { width: 100%; min-width: 0; padding: 3px 6px; border: 1px solid var(--accent); border-radius: 4px; background: var(--panel); color: var(--ink); font: inherit; outline: none; }
+    .device-alias-actions { display: flex; gap: 6px; margin-top: 6px; }
     .online-dot { margin-top: 7px; }
     .device-heading > .unread-dot { margin-top: 7px; }
     .device-footer { display: flex; flex-wrap: wrap; align-items: center; justify-content: space-between; gap: 2px 8px; margin-top: 6px; }
@@ -146,6 +156,8 @@
       .workspace-paths { grid-column: 1 / -1; grid-row: 2; }
       .workspace-status { grid-column: 2; grid-row: 1; }
       .open-link { grid-column: 3; grid-row: 1; min-height: 36px; }
+      .device-rename, .device-drag, .device-alias-actions button { min-height: 36px; }
+      .device-rename, .device-drag { flex-basis: 36px; }
     }
   </style>
 </head>
@@ -177,6 +189,7 @@
         </div>
       </details>
       <p id="alias-status" role="alert"></p>
+      <p id="order-status" role="alert"></p>
       <div class="layout">
         <section aria-labelledby="workspace-heading">
           <div class="section-heading">
@@ -211,7 +224,7 @@
       <summary class="workspace-group-heading">
         <svg class="disclosure-icon" viewBox="0 0 24 24" aria-hidden="true"><path d="m9 6 6 6-6 6"/></svg>
         <h3 class="workspace-group-name">
-          <button type="button" class="device-label"></button>
+          <span class="device-label"></span>
           <span class="activity-badge device-working" data-state="working" hidden></span>
         </h3>
         <span class="unread-dot device-unread" hidden role="img" aria-label="Unread workspaces" title="Unread workspaces"></span>
@@ -244,8 +257,13 @@
   <template id="device-template">
     <article class="device">
       <div class="device-heading">
-        <svg viewBox="0 0 24 24" aria-hidden="true"><rect x="3" y="4" width="18" height="12" rx="2"/><path d="M8 20h8m-4-4v4"/></svg>
-        <h3 class="device-name"><button type="button" class="device-label"></button></h3>
+        <button type="button" class="device-drag" title="Drag to reorder, or use the up and down arrow keys">
+          <svg viewBox="0 0 24 24" aria-hidden="true"><path d="M9 5h.01M15 5h.01M9 12h.01M15 12h.01M9 19h.01M15 19h.01" stroke-width="3"/></svg>
+        </button>
+        <h3 class="device-name"><span class="device-label"></span></h3>
+        <button type="button" class="device-rename" title="Rename device">
+          <svg viewBox="0 0 24 24" aria-hidden="true"><path d="m16 4 4 4M4 20l4-1L20 7a2.1 2.1 0 0 0-3-3L5 16l-1 4Z"/></svg>
+        </button>
         <span class="unread-dot device-unread" hidden role="img" aria-label="Unread workspaces" title="Unread workspaces"></span>
         <span class="online-dot" aria-label="Online"></span>
       </div>
@@ -255,6 +273,15 @@
       </div>
     </article>
   </template>
+  <template id="device-alias-template">
+    <span class="device-alias-editor">
+      <input type="text" class="device-alias-input" minlength="1" maxlength="128" required aria-label="Device alias" title="Use 1 to 128 characters.">
+      <span class="device-alias-actions">
+        <button type="button" class="device-alias-save">Save</button>
+        <button type="button" class="device-alias-cancel">Cancel</button>
+      </span>
+    </span>
+  </template>
   <template id="empty-template">
     <div class="empty">
       <svg viewBox="0 0 24 24" aria-hidden="true"><rect x="3" y="4" width="18" height="12" rx="2"/><path d="M8 20h8m-4-4v4"/></svg>
@@ -273,15 +300,19 @@
       const workspaceGroupTemplate = document.getElementById("workspace-group-template");
       const workspaceTemplate = document.getElementById("workspace-template");
       const deviceTemplate = document.getElementById("device-template");
+      const deviceAliasTemplate = document.getElementById("device-alias-template");
       const emptyTemplate = document.getElementById("empty-template");
       const deviceConnection = document.getElementById("device-connection");
       const registrationKey = document.getElementById("registration-key");
       const registrationStatus = document.getElementById("registration-status");
       const aliasStatus = document.getElementById("alias-status");
+      const orderStatus = document.getElementById("order-status");
       let refreshing = false;
       let signature = "";
       let editingDevice = null;
-      let aliasRevision = 0;
+      let inventoryRevision = 0;
+      let reordering = false;
+      let drag = null;
 
       registrationKey.addEventListener("focus", () => registrationKey.select());
       deviceConnection.addEventListener("toggle", async () => {
@@ -318,36 +349,30 @@
         layout.dataset.stale = String(state === "error");
       }
 
-      function bindDeviceLabel(label, node, data) {
+      function setDeviceLabel(label, node) {
         label.textContent = node.alias || node.name;
-        label.title = `${node.name}\nClick to rename device`;
-        label.setAttribute("aria-label", `Rename device ${node.alias || node.name}`);
-        label.addEventListener("click", () => editDeviceAlias(label, node, data));
+        label.title = node.name;
       }
 
-      function editDeviceAlias(label, node, data) {
-        if (editingDevice !== null) return;
+      function editDeviceAlias(label, button, node, data) {
+        if (editingDevice !== null || reordering) return;
         editingDevice = node.id;
         aliasStatus.textContent = "";
-        const input = document.createElement("input");
-        input.type = "text";
-        input.className = "device-alias-input";
-        input.minLength = 1;
-        input.maxLength = 128;
-        input.required = true;
-        input.setAttribute("aria-label", "Device alias");
-        input.title = "Use 1 to 128 characters.";
+        const editor = deviceAliasTemplate.content.firstElementChild.cloneNode(true);
+        const input = editor.querySelector("input");
         input.value = node.alias || "";
         input.placeholder = node.name;
         label.hidden = true;
-        label.before(input);
+        button.hidden = true;
+        label.before(editor);
         let finished = false;
         const finish = async save => {
           if (finished) return;
           finished = true;
           const alias = input.value.trim();
-          input.remove();
+          editor.remove();
           label.hidden = false;
+          button.hidden = false;
           try {
             if (save && !alias) throw new Error("Device alias must contain at least one character.");
             if (save && alias !== (node.alias || "")) {
@@ -359,7 +384,7 @@
               if (!response.ok) throw new Error(await response.text());
               const result = await response.json();
               node.alias = result.alias;
-              aliasRevision += 1;
+              inventoryRevision += 1;
               render(data);
               signature = JSON.stringify(data);
             }
@@ -367,23 +392,153 @@
             aliasStatus.textContent = `Could not rename device: ${error.message}`;
           } finally {
             editingDevice = null;
+            nodes.querySelector(`.device[data-node-id="${CSS.escape(node.id)}"] .device-rename`).focus();
             refresh();
           }
         };
-        input.addEventListener("keydown", event => {
+        editor.addEventListener("keydown", event => {
           if (event.isComposing) return;
-          if (event.key === "Enter" || event.key === "Escape") {
+          if (event.key === "Escape" || (event.key === "Enter" && event.target === input)) {
             event.preventDefault();
             finish(event.key === "Enter");
-            label.focus();
           }
         });
-        input.addEventListener("blur", () => finish(true));
+        editor.querySelector(".device-alias-save").addEventListener("click", () => finish(true));
+        editor.querySelector(".device-alias-cancel").addEventListener("click", () => finish(false));
         input.focus();
         input.select();
       }
 
+      async function saveDeviceOrder(data, order, nodeId) {
+        reordering = true;
+        orderStatus.textContent = "";
+        try {
+          if (order.some((id, index) => id !== data.nodes[index].id)) {
+            const response = await fetch("api/nodes/order", {
+              method: "POST",
+              headers: {"Content-Type": "application/json"},
+              body: JSON.stringify({order}),
+            });
+            if (!response.ok) throw new Error(await response.text());
+            const result = await response.json();
+            const byId = new Map(data.nodes.map(node => [node.id, node]));
+            data.nodes = result.order.map(id => byId.get(id));
+            data.workspaces = result.order.flatMap(
+              id => data.workspaces.filter(workspace => workspace.node_id === id));
+            inventoryRevision += 1;
+            render(data);
+            signature = JSON.stringify(data);
+          }
+        } catch (error) {
+          orderStatus.textContent = `Could not reorder devices: ${error.message}`;
+        } finally {
+          reordering = false;
+          nodes.querySelector(`.device[data-node-id="${CSS.escape(nodeId)}"] .device-drag`).focus({preventScroll: true});
+          refresh();
+        }
+      }
+
+      function clearDropMarker() {
+        nodes.classList.remove("device-drop-end");
+        for (const card of nodes.querySelectorAll(".device-drop-before")) {
+          card.classList.remove("device-drop-before");
+        }
+      }
+
+      function markDropTarget() {
+        clearDropMarker();
+        drag.before = Array.from(nodes.children).find(card => {
+          const rect = card.getBoundingClientRect();
+          return card !== drag.card && drag.y < (rect.top + rect.bottom) / 2;
+        });
+        (drag.before || nodes).classList.add(drag.before ? "device-drop-before" : "device-drop-end");
+      }
+
+      function scrollDrag() {
+        const step = drag.y < 56 ? -12 : drag.y > innerHeight - 56 ? 12 : 0;
+        if (step) {
+          window.scrollBy(0, step);
+          markDropTarget();
+        }
+        drag.frame = requestAnimationFrame(scrollDrag);
+      }
+
+      function finishDrag(commit, event) {
+        const current = drag;
+        if (!current) return;
+        drag = null;
+        cancelAnimationFrame(current.frame);
+        current.card.removeAttribute("data-dragging");
+        clearDropMarker();
+        if (nodes.hasPointerCapture(current.pointerId)) {
+          nodes.releasePointerCapture(current.pointerId);
+        }
+        const rect = nodes.getBoundingClientRect();
+        if (commit && current.moved && event.clientX >= rect.left
+            && event.clientX <= rect.right && event.clientY >= rect.top && event.clientY <= rect.bottom) {
+          const cards = Array.from(nodes.children).filter(card => card !== current.card);
+          const index = current.before ? cards.indexOf(current.before) : cards.length;
+          cards.splice(index, 0, current.card);
+          saveDeviceOrder(current.data, cards.map(card => card.dataset.nodeId), current.card.dataset.nodeId);
+        } else {
+          reordering = false;
+          refresh();
+        }
+      }
+
+      nodes.addEventListener("pointermove", event => {
+        if (!drag || event.pointerId !== drag.pointerId) return;
+        drag.y = event.clientY;
+        if (!drag.moved) {
+          if (Math.hypot(event.clientX - drag.x, event.clientY - drag.startY) < 5) return;
+          drag.moved = true;
+          drag.card.setAttribute("data-dragging", "");
+          drag.frame = requestAnimationFrame(scrollDrag);
+        }
+        markDropTarget();
+      });
+      nodes.addEventListener("pointerup", event => {
+        if (drag && event.pointerId === drag.pointerId) finishDrag(true, event);
+      });
+      nodes.addEventListener("pointercancel", () => finishDrag(false));
+      nodes.addEventListener("lostpointercapture", () => finishDrag(false));
+      nodes.addEventListener("keydown", event => {
+        if (drag && event.key === "Escape") {
+          event.preventDefault();
+          finishDrag(false);
+        }
+      });
+
+      function bindDeviceDrag(card, node, data) {
+        const handle = card.querySelector(".device-drag");
+        handle.disabled = data.nodes.length < 2;
+        handle.setAttribute("aria-label", `Reorder device ${node.alias || node.name}`);
+        handle.addEventListener("pointerdown", event => {
+          if (event.button !== 0 || reordering || editingDevice !== null) return;
+          event.preventDefault();
+          handle.focus({preventScroll: true});
+          reordering = true;
+          orderStatus.textContent = "";
+          drag = {card, data, pointerId: event.pointerId, x: event.clientX,
+            startY: event.clientY, y: event.clientY, moved: false, frame: null, before: null};
+          nodes.setPointerCapture(event.pointerId);
+        });
+        handle.addEventListener("keydown", event => {
+          if (reordering || editingDevice !== null || !["ArrowUp", "ArrowDown"].includes(event.key)) return;
+          event.preventDefault();
+          const order = data.nodes.map(item => item.id);
+          const index = order.indexOf(node.id);
+          const target = index + (event.key === "ArrowUp" ? -1 : 1);
+          if (target < 0 || target >= order.length) return;
+          [order[index], order[target]] = [order[target], order[index]];
+          saveDeviceOrder(data, order, node.id);
+        });
+      }
+
       function render(data) {
+        const focusedHandle = nodes.contains(document.activeElement)
+          && document.activeElement.matches(".device-drag")
+          ? document.activeElement.closest(".device").dataset.nodeId : null;
         const expanded = new Set(Array.from(
           workspaces.querySelectorAll(".workspace-group[open]"), group => group.dataset.nodeId));
         const workspaceList = document.createDocumentFragment();
@@ -395,7 +550,7 @@
           group.dataset.nodeId = node.id;
           group.open = expanded.has(node.id);
           group.setAttribute("aria-label", node.alias || node.name);
-          bindDeviceLabel(group.querySelector(".device-label"), node, data);
+          setDeviceLabel(group.querySelector(".device-label"), node);
           const link = group.querySelector(".manage-link");
           link.href = node.url;
           link.setAttribute("aria-label", `Manage workspaces on ${node.alias || node.name}`);
@@ -454,7 +609,13 @@
         const nodeList = document.createDocumentFragment();
         for (const node of data.nodes) {
           const card = deviceTemplate.content.firstElementChild.cloneNode(true);
-          bindDeviceLabel(card.querySelector(".device-label"), node, data);
+          card.dataset.nodeId = node.id;
+          bindDeviceDrag(card, node, data);
+          const label = card.querySelector(".device-label");
+          setDeviceLabel(label, node);
+          const rename = card.querySelector(".device-rename");
+          rename.setAttribute("aria-label", `Rename device ${node.alias || node.name}`);
+          rename.addEventListener("click", () => editDeviceAlias(label, rename, node, data));
           const owned = data.workspaces.filter(workspace => workspace.node_id === node.id);
           card.querySelector(".device-unread").hidden = !owned.some(workspace => workspace.unread);
           const sessions = owned.reduce((total, workspace) => total + workspace.session_count, 0);
@@ -472,6 +633,10 @@
         }
         workspaces.replaceChildren(workspaceList);
         nodes.replaceChildren(nodeList);
+        if (focusedHandle) {
+          const handle = nodes.querySelector(`.device[data-node-id="${CSS.escape(focusedHandle)}"] .device-drag`);
+          if (handle) handle.focus({preventScroll: true});
+        }
         document.getElementById("workspace-count").textContent = data.workspaces.length;
         document.getElementById("device-count").textContent = data.nodes.length;
         const running = data.workspaces.filter(workspace => workspace.running).length;
@@ -481,7 +646,7 @@
       async function refresh() {
         if (refreshing) return;
         refreshing = true;
-        const revision = aliasRevision;
+        const revision = inventoryRevision;
         try {
           const response = await fetch("api/workspaces", {cache: "no-store"});
           if (response.status === 401) {
@@ -491,7 +656,7 @@
           if (!response.ok) throw new Error(`HTTP ${response.status}`);
           const data = await response.json();
           const next = JSON.stringify(data);
-          if (editingDevice === null && revision === aliasRevision && next !== signature) {
+          if (editingDevice === null && !reordering && revision === inventoryRevision && next !== signature) {
             render(data);
             signature = next;
           }

{pycodex_ws-0.4.5 → pycodex_ws-0.4.6}/pycodex_router/server.py

@@ -176,8 +176,8 @@ class Router:
             aliases = json.loads(self.aliases_path.read_text(encoding="utf-8"))
             if not isinstance(aliases, dict) or any(
                 not re.fullmatch("[a-z2-7]{52}", node_id)
-                or not alias
-                or normalize_alias(alias) != alias
+                or not isinstance(alias, str)
+                or (alias and normalize_alias(alias) != alias)
                 for node_id, alias in aliases.items()
             ):
                 raise ValueError("invalid router aliases file")
@@ -201,6 +201,33 @@ class Router:
                         temporary_path.unlink()
         self.aliases = aliases
 
+    async def reorder_devices(self, request):
+        if not self.browser_token or not _auth_cookie_matches(
+            self.browser_token, request.cookies.get("pycodex_router_auth")
+        ):
+            raise HTTPException(401, "Authentication required.")
+        payload = await _json_object_body(request, max_bytes=64 * 1024)
+        order = payload.get("order")
+        if (
+            not isinstance(order, list)
+            or any(
+                not isinstance(node_id, str)
+                or not re.fullmatch("[a-z2-7]{52}", node_id)
+                for node_id in order
+            )
+            or len(set(order)) != len(order)
+        ):
+            raise HTTPException(422, "Device order must be a list of unique device IDs.")
+        if set(order) != set(self.nodes):
+            raise HTTPException(409, "Connected devices changed. Refresh and try again.")
+        online_order = iter(order)
+        saved_order = [
+            next(online_order) if node_id in self.nodes else node_id
+            for node_id in self.aliases
+        ]
+        self._save_aliases({node_id: self.aliases[node_id] for node_id in saved_order})
+        return JSONResponse({"order": order})
+
     async def rename_device(self, request):
         if not self.browser_token or not _auth_cookie_matches(
             self.browser_token, request.cookies.get("pycodex_router_auth")
@@ -263,6 +290,10 @@ class Router:
                 name,
                 validate_inventory(message["workspaces"]),
             )
+            if node_id not in self.aliases:
+                aliases = dict(self.aliases)
+                aliases[node_id] = ""
+                self._save_aliases(aliases)
             self.nodes[node_id] = node
             await websocket.send_text(encode({"type": "registered", "id": node_id}))
             while True:
@@ -423,7 +454,10 @@ class Router:
     async def workspaces(self, request):
         nodes = []
         workspaces = []
-        for node in sorted(self.nodes.values(), key=lambda item: item.id):
+        for node_id in self.aliases:
+            if node_id not in self.nodes:
+                continue
+            node = self.nodes[node_id]
             base = request.scope.get("root_path", "") + "/n/" + node.id + "/"
             nodes.append({
                 "id": node.id,
@@ -478,6 +512,7 @@ def create_app(
             Route("/", index),
             Route("/api/workspaces", router.workspaces),
             Route("/api/registration-key", registration_key_info),
+            Route("/api/nodes/order", router.reorder_devices, methods=["POST"]),
             Route("/api/nodes/{node_id}/alias", router.rename_device, methods=["POST"]),
         ]
     )

{pycodex_ws-0.4.5 → pycodex_ws-0.4.6}/pyproject.toml

@@ -4,12 +4,12 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pycodex-ws"
-version = "0.4.5"
+version = "0.4.6"
 description = "Browser workspace and optional Feishu integration for python-codex"
 readme = "README.md"
 requires-python = ">=3.7"
 dependencies = [
-    "python-codex==0.4.5",
+    "python-codex==0.4.6",
     "pillow>=9.0.0",
     "websockets>=11,<16",
 ]