@omercnet/paseo-connected-hosts 0.0.0 → 0.1.0-next.305.1

package/OVERVIEW.md

@@ -1,17 +1,17 @@
 Connected hosts manages the hosts saved on the daemon you are working in, from a **Connected hosts** screen. It adds a sidebar row and a Command Center action, "Manage connected hosts".
 
-Each saved host shows its label, connection status, the safe description of where it connects, the server ID once learned, who registered it, its metadata, when it was last online, when the next retry happens, and the last connection error. Status updates live. From the screen you can add a host, edit its label, metadata and connection, disable and re-enable it, and remove it.
+Each saved host shows its label, connection status, the safe description of where it connects, the server ID once learned, what registered it, its metadata, when it was last online, when the next retry happens, and the last connection error. Status updates live. From the screen you can add a host, edit its label, metadata and connection, disable and re-enable it, and remove it.
 
 The list is the daemon's own registry, the same one `paseo host` and agent tools such as `list_hosts` use. It is separate from the connections saved in the app, and it always describes the host selected in the screen's host picker. Changing the picker shows that daemon's registry.
 
-Connections are written as the CLI accepts them, for example `host:6767` or `ssh://user@host`. A host that cannot be reached is still saved and retried. A connection is write-only: the daemon never returns saved credentials, so the edit form shows the safe description and leaves the connection untouched unless you type a new one.
+Connections are written as the CLI accepts them, for example `host:6767` or `ssh://user@host`. A host that cannot be reached is still saved and retried automatically. A host that is reachable but rejected (failed authentication, a different daemon identity, this daemon itself, a duplicate, or a daemon without the feature) stays in an error state until you disable and re-enable it, or save a new connection. A connection is write-only: the daemon never returns saved credentials, so the edit form shows the safe description and leaves the connection untouched unless you type a new one.
 
 Replacing a connection keeps the host's pinned identity. If the new connection reaches a different daemon, the new connection is saved but the host shows an identity error and stays disconnected. To accept the new daemon, edit the host, enter the connection again and tick **Accept a different server identity**. The daemon refuses an identity change without a connection, so the field cannot be left blank.
 
-Metadata is a set of `key=value` lines and saving replaces the whole map.
+Metadata is a set of `key=value` lines and saving replaces the whole map. Only fields you change are sent, so editing a label never rewrites metadata.
 
-Who may change what follows the daemon's own rules. Label and access changes need tunnel management, and connection and metadata changes need the registering producer or access management. A refused change shows the daemon's message and leaves the host as it was. Removing a host that another process registered lets that process add it again; disabling keeps the host saved and switched off. Removing a host does not stop agents already running on it.
+Who may change what follows the daemon's own rules. Adding, editing and removing need tunnel management. Changing a connection or metadata, and removing, also need the registering producer or access management. The screen cannot see your grants, so every action is shown, and a refused change shows the daemon's message and leaves the host as it was. Removing a host that another process registered lets that process add it again; disabling keeps the host saved and switched off. Removing a host does not stop agents already running on it, but this daemon can no longer reach them.
 
-Both daemons need the connected hosts feature.
+Both daemons need the connected hosts feature, and the link between them needs host multiplexing.
 
 Requires Paseo ^0.13.0.

package/client/hosts.ts

@@ -1,7 +1,21 @@
-import type { HostAdd, HostDirectory, HostSummary, HostUpdate, PaseoApi } from "@getpaseo/client";
+import type { HostAdd, HostDirectory, HostSummary, HostUpdate } from "@getpaseo/protocol/hosts";
 import type { SessionOutboundMessage } from "@getpaseo/protocol/messages";
 import { useCallback, useEffect, useState } from "react";
 
+/** The slice of the plugin's `usePaseo()` client that follows the saved hosts. */
+export interface HostsApi {
+  hosts: {
+    observe(): {
+      subscribe(observer: {
+        snapshot(directory: HostDirectory): void;
+        update(message: SessionOutboundMessage): void;
+        error(cause: unknown): void;
+      }): () => void;
+      release(): Promise<void>;
+    };
+  };
+}
+
 export type Built<T> = { ok: true; value: T } | { ok: false; error: string };
 
 /** Prefers the daemon's `userMessage` over the transport message, which appends request details. */
@@ -64,15 +78,23 @@ export interface EditForm {
   replaceIdentity: boolean;
 }
 
-/** Sends only what changed, so an untouched field is never rewritten or erased. */
+/**
+ * Sends only what the user changed. A field still equal to its prefilled text is never parsed or
+ * sent, so values the form cannot round-trip (padding, `=` in a key) are not rewritten and a
+ * label-only edit never asks for metadata permission.
+ */
 export function buildUpdatePatch(host: HostSummary, form: EditForm): Built<HostUpdate> {
   const patch: HostUpdate = {};
-  const label = form.label.trim() || null;
-  if (label !== host.label) patch.label = label;
-  const metadata = parseMetadata(form.metadata);
-  if (!metadata.ok) return metadata;
-  if (formatMetadata(metadata.value) !== formatMetadata(host.metadata)) {
-    patch.metadata = metadata.value;
+  if (form.label !== (host.label ?? "")) {
+    const label = form.label.trim() || null;
+    if (label !== host.label) patch.label = label;
+  }
+  if (form.metadata !== formatMetadata(host.metadata)) {
+    const metadata = parseMetadata(form.metadata);
+    if (!metadata.ok) return metadata;
+    if (formatMetadata(metadata.value) !== formatMetadata(host.metadata)) {
+      patch.metadata = metadata.value;
+    }
   }
   const connection = form.connection.trim();
   if (connection) {
@@ -86,47 +108,61 @@ export function directoryFromMessage(message: SessionOutboundMessage): HostDirec
   return message.type === "hosts.changed" ? message.payload.directory : null;
 }
 
+export interface DirectoryHandlers {
+  onDirectory(directory: HostDirectory): void;
+  onError(message: string): void;
+}
+
+/**
+ * Follows the selected daemon's saved hosts and returns the cleanup. `observe()` throws
+ * synchronously when the daemon is offline or too old, so that is reported like any other failure.
+ */
+export function observeDirectory(paseo: HostsApi, handlers: DirectoryHandlers): () => void {
+  try {
+    const subscription = paseo.hosts.observe();
+    const stop = subscription.subscribe({
+      snapshot: handlers.onDirectory,
+      update: (message) => {
+        const directory = directoryFromMessage(message);
+        if (directory) handlers.onDirectory(directory);
+      },
+      error: (cause) => handlers.onError(errorMessage(cause)),
+    });
+    return () => {
+      stop();
+      void subscription.release();
+    };
+  } catch (cause) {
+    handlers.onError(errorMessage(cause));
+    return () => {};
+  }
+}
+
 export interface HostDirectoryState {
   directory: HostDirectory | null;
   error: string | null;
-  refresh(): Promise<void>;
+  /** Subscribes again after a failure. */
+  retry(): void;
 }
 
 /** Live view of the selected daemon's saved hosts. Resets when the selected host changes. */
-export function useHostDirectory(paseo: PaseoApi): HostDirectoryState {
+export function useHostDirectory(paseo: HostsApi): HostDirectoryState {
   const [directory, setDirectory] = useState<HostDirectory | null>(null);
   const [error, setError] = useState<string | null>(null);
+  const [attempt, setAttempt] = useState(0);
 
   useEffect(() => {
     setDirectory(null);
     setError(null);
-    const subscription = paseo.hosts.observe();
-    const stop = subscription.subscribe({
-      snapshot: (snapshot) => {
-        setDirectory(snapshot);
+    return observeDirectory(paseo, {
+      onDirectory: (next) => {
+        setDirectory(next);
         setError(null);
       },
-      update: (message) => {
-        const next = directoryFromMessage(message);
-        if (next) setDirectory(next);
-      },
-      error: (cause) => setError(errorMessage(cause)),
+      onError: setError,
     });
-    subscription.ready.catch((cause) => setError(errorMessage(cause)));
-    return () => {
-      stop();
-      void subscription.release();
-    };
-  }, [paseo]);
-
-  const refresh = useCallback(async () => {
-    try {
-      setDirectory(await paseo.hosts.list());
-      setError(null);
-    } catch (cause) {
-      setError(errorMessage(cause));
-    }
-  }, [paseo]);
+  }, [paseo, attempt]);
 
-  return { directory, error, refresh };
+  const retry = useCallback(() => setAttempt((count) => count + 1), []);
+  return { directory, error, retry };
 }

package/client/screen.tsx

@@ -1,7 +1,7 @@
-import type { HostSummary } from "@getpaseo/client";
 import type { PluginTheme } from "@getpaseo/plugin";
 import { type PluginScreenProps, usePaseo } from "@getpaseo/plugin/client";
 import { Icon, Modal, ScrollView, TextInput, useToast } from "@getpaseo/plugin/client/ui";
+import type { HostSummary } from "@getpaseo/protocol/hosts";
 import { useMemo, useState } from "react";
 import { Pressable, Text, type TextStyle, View, type ViewStyle } from "react-native";
 import {
@@ -57,7 +57,7 @@ type Styles = Record<StyleKey, ViewStyle & TextStyle>;
 function makeStyles(theme: PluginTheme, compact: boolean): Styles {
   const c = theme.colors;
   const button = {
-    minHeight: 40,
+    minHeight: compact ? 44 : 40,
     paddingHorizontal: 14,
     borderRadius: 10,
     alignItems: "center" as const,
@@ -125,17 +125,20 @@ function Action({
   style,
   textStyle,
   disabled,
+  accessibilityLabel,
 }: {
   label: string;
   onPress(): void;
   style: object;
   textStyle: object;
   disabled?: boolean;
+  /** Defaults to `label`; set it when several buttons share a label. */
+  accessibilityLabel?: string;
 }) {
   return (
     <Pressable
       accessibilityRole="button"
-      accessibilityLabel={label}
+      accessibilityLabel={accessibilityLabel ?? label}
       accessibilityState={{ disabled: !!disabled }}
       disabled={disabled}
       onPress={onPress}
@@ -189,7 +192,7 @@ function HostForm({
   styles: Styles;
   theme: PluginTheme;
   onCancel(): void;
-  onDone(message: string): Promise<void>;
+  onDone(message: string): void;
 }) {
   const paseo = usePaseo();
   const host = target.kind === "edit" ? target.host : null;
@@ -198,7 +201,7 @@ function HostForm({
   const [metadata, setMetadata] = useState(host ? formatMetadata(host.metadata) : "");
   const [sourceKey, setSourceKey] = useState("");
   const mismatch = host?.error?.code === "server_id_mismatch";
-  const [replaceIdentity, setReplaceIdentity] = useState(mismatch);
+  const [replaceIdentity, setReplaceIdentity] = useState(false);
   const [error, setError] = useState<string | null>(null);
   const [pending, setPending] = useState(false);
 
@@ -217,12 +220,12 @@ function HostForm({
     try {
       if (add?.ok) {
         await paseo.hosts.add(add.value);
-        await onDone("Host added");
+        onDone("Host added");
       } else if (host && update?.ok && Object.keys(update.value).length > 0) {
         await paseo.hosts.update(host.registrationId, update.value);
-        await onDone("Host updated");
+        onDone("Host updated");
       } else {
-        await onDone("No changes to save");
+        onCancel();
       }
     } catch (cause) {
       setError(errorMessage(cause));
@@ -245,7 +248,7 @@ function HostForm({
           host
             ? `Currently ${host.targetDescription}. Saved credentials are never shown.${
                 mismatch
-                  ? " This host reached a different daemon. Enter the connection again and accept the new identity to keep it."
+                  ? " This host reached a different daemon. Enter the connection again and tick the box below to accept the new identity."
                   : ""
               }`
             : "Saved on this daemon. Unreachable hosts are kept and retried."
@@ -254,6 +257,7 @@ function HostForm({
       {host && connection.trim() ? (
         <Pressable
           accessibilityRole="checkbox"
+          accessibilityLabel="Accept a different server identity"
           accessibilityState={{ checked: replaceIdentity }}
           onPress={() => setReplaceIdentity((value) => !value)}
           style={styles.row}
@@ -295,7 +299,7 @@ function HostForm({
           value={sourceKey}
           onChangeText={setSourceKey}
           placeholder="Optional"
-          hint="Names this registration so repeating the same add returns the saved entry."
+          hint="Names this registration. Repeating an add with the same connection, label and metadata returns the saved entry."
         />
       )}
       {error ? (
@@ -353,7 +357,7 @@ function HostCard({
       </View>
       <Text style={styles.mono}>{host.targetDescription}</Text>
       <Text style={styles.mono}>
-        Server {host.serverId ?? "not learned yet"} | Source {host.source.key}
+        Server {host.serverId ?? "not learned yet"} | Source {host.source.id}
       </Text>
       {host.lastOnlineAt ? (
         <Text style={styles.muted}>Last online {new Date(host.lastOnlineAt).toLocaleString()}</Text>
@@ -380,6 +384,7 @@ function HostCard({
       <View style={styles.row}>
         <Action
           label={host.enabled ? "Disable" : "Enable"}
+          accessibilityLabel={`${host.enabled ? "Disable" : "Enable"} ${name}`}
           onPress={onToggle}
           disabled={busy}
           style={styles.button}
@@ -387,6 +392,7 @@ function HostCard({
         />
         <Action
           label="Edit"
+          accessibilityLabel={`Edit ${name}`}
           onPress={onEdit}
           disabled={busy}
           style={styles.button}
@@ -394,6 +400,7 @@ function HostCard({
         />
         <Action
           label="Remove"
+          accessibilityLabel={`Remove ${name}`}
           onPress={onRemove}
           disabled={busy}
           style={styles.danger}
@@ -407,7 +414,7 @@ function HostCard({
 export function ConnectedHostsScreen({ theme, layout, host: selected }: PluginScreenProps) {
   const paseo = usePaseo();
   const toast = useToast();
-  const { directory, error, refresh } = useHostDirectory(paseo);
+  const { directory, error, retry } = useHostDirectory(paseo);
   const styles = useMemo(() => makeStyles(theme, layout.compact), [theme, layout.compact]);
   const [form, setForm] = useState<FormTarget | null>(null);
   const [removing, setRemoving] = useState<HostSummary | null>(null);
@@ -418,7 +425,6 @@ export function ConnectedHostsScreen({ theme, layout, host: selected }: PluginSc
     try {
       await action();
       toast.show(done, { variant: "success" });
-      await refresh();
     } catch (cause) {
       toast.error(errorMessage(cause));
     } finally {
@@ -426,10 +432,9 @@ export function ConnectedHostsScreen({ theme, layout, host: selected }: PluginSc
     }
   }
 
-  async function finishForm(message: string) {
+  function finishForm(message: string) {
     toast.show(message, { variant: "success" });
     setForm(null);
-    await refresh();
   }
 
   const hosts = directory?.hosts ?? [];
@@ -456,7 +461,7 @@ export function ConnectedHostsScreen({ theme, layout, host: selected }: PluginSc
           </Text>
           <Action
             label="Retry"
-            onPress={refresh}
+            onPress={retry}
             style={styles.button}
             textStyle={styles.buttonText}
           />
@@ -522,8 +527,9 @@ export function ConnectedHostsScreen({ theme, layout, host: selected }: PluginSc
           <View style={{ gap: 12 }}>
             <Text style={styles.body}>
               Remove {removing?.label ?? removing?.targetDescription} from this daemon? Agents
-              already running on it keep running. If another process registered this host, it can
-              add it again. Disable the host instead to keep it saved and switched off.
+              already running on it keep running on their own daemon, but this daemon can no longer
+              reach them. If another process registered this host, it can add it again. Disable the
+              host instead to keep it saved and switched off.
             </Text>
             <View style={[styles.row, { justifyContent: "flex-end" }]}>
               <Action

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omercnet/paseo-connected-hosts",
-  "version": "0.0.0",
+  "version": "0.1.0-next.305.1",
   "type": "module",
   "description": "A Paseo plugin that manages the daemon's connected hosts from the UI.",
   "license": "MIT",
@@ -42,17 +42,17 @@
     "typecheck": "tsc --noEmit"
   },
   "devDependencies": {
-    "@biomejs/biome": "catalog:",
-    "@getpaseo/cli": "catalog:",
-    "@getpaseo/client": "catalog:",
-    "@getpaseo/plugin": "catalog:",
-    "@getpaseo/protocol": "catalog:",
-    "@tanstack/react-query": "catalog:",
-    "@types/node": "catalog:",
-    "@types/react": "catalog:",
-    "react": "catalog:",
-    "react-native": "catalog:",
-    "typescript": "catalog:",
-    "vitest": "catalog:"
+    "@biomejs/biome": "^2.5.10",
+    "@getpaseo/cli": "0.13.0",
+    "@getpaseo/client": "0.13.0",
+    "@getpaseo/plugin": "0.13.0",
+    "@getpaseo/protocol": "0.13.0",
+    "@tanstack/react-query": "^5.102.3",
+    "@types/node": "^24.10.1",
+    "@types/react": "^19.2.18",
+    "react": "19.1.0",
+    "react-native": "0.81.5",
+    "typescript": "^7.0.2",
+    "vitest": "^5.0.0"
   }
 }