@vellumai/cli 0.7.0 → 0.7.2

package/src/components/DefaultMainScreen.tsx

@@ -1,8 +1,6 @@
 import { spawn } from "child_process";
-import { createHash, randomBytes, randomUUID } from "crypto";
-import { hostname, userInfo } from "os";
+import { randomUUID } from "crypto";
 import { basename } from "path";
-import qrcode from "qrcode-terminal";
 import {
   useCallback,
   useEffect,
@@ -19,6 +17,7 @@ import { SPECIES_CONFIG, type Species } from "../lib/constants";
 import { callDoctorDaemon, type ChatLogEntry } from "../lib/doctor-client";
 import { checkHealth } from "../lib/health-check";
 import { appendHistory, loadHistory } from "../lib/input-history";
+import { tuiLog } from "../lib/tui-log";
 import { statusEmoji, withStatusEmoji } from "../lib/status-emoji";
 import {
   getTerminalCapabilities,
@@ -45,7 +44,6 @@ export const SLASH_COMMANDS = [
   "/doctor",
   "/exit",
   "/help",
-  "/pair",
   "/q",
   "/quit",
   "/retire",
@@ -172,14 +170,14 @@ async function runtimeRequest<T>(
   assistantId: string,
   path: string,
   init?: RequestInit,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<T> {
   const url = `${baseUrl}/v1/assistants/${assistantId}${path}`;
   const response = await fetch(url, {
     ...init,
     headers: {
       "Content-Type": "application/json",
-      ...(bearerToken ? { Authorization: `Bearer ${bearerToken}` } : {}),
+      ...auth,
       ...(init?.headers as Record<string, string> | undefined),
     },
   });
@@ -218,7 +216,7 @@ async function checkHealthRuntime(baseUrl: string): Promise<HealthResponse> {
 async function pollMessages(
   baseUrl: string,
   assistantId: string,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<ListMessagesResponse> {
   const params = new URLSearchParams({ conversationKey: assistantId });
   return runtimeRequest<ListMessagesResponse>(
@@ -226,7 +224,7 @@ async function pollMessages(
     assistantId,
     `/messages?${params.toString()}`,
     undefined,
-    bearerToken,
+    auth,
   );
 }
 
@@ -235,7 +233,7 @@ async function sendMessage(
   assistantId: string,
   content: string,
   signal?: AbortSignal,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<SendMessageResponse> {
   return runtimeRequest<SendMessageResponse>(
     baseUrl,
@@ -251,7 +249,7 @@ async function sendMessage(
       }),
       signal,
     },
-    bearerToken,
+    auth,
   );
 }
 
@@ -260,7 +258,7 @@ async function submitDecision(
   assistantId: string,
   requestId: string,
   decision: "allow" | "deny",
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<SubmitDecisionResponse> {
   return runtimeRequest<SubmitDecisionResponse>(
     baseUrl,
@@ -270,7 +268,7 @@ async function submitDecision(
       method: "POST",
       body: JSON.stringify({ requestId, decision }),
     },
-    bearerToken,
+    auth,
   );
 }
 
@@ -281,7 +279,7 @@ async function addTrustRule(
   pattern: string,
   scope: string,
   decision: "allow" | "deny",
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<AddTrustRuleResponse> {
   return runtimeRequest<AddTrustRuleResponse>(
     baseUrl,
@@ -291,7 +289,7 @@ async function addTrustRule(
       method: "POST",
       body: JSON.stringify({ requestId, pattern, scope, decision }),
     },
-    bearerToken,
+    auth,
   );
 }
 
@@ -350,26 +348,39 @@ async function* streamEvents(
   assistantId: string,
   conversationKey: string,
   signal: AbortSignal,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): AsyncGenerator<SseEvent> {
   const params = new URLSearchParams({ conversationKey });
   const url = `${baseUrl}/v1/assistants/${assistantId}/events?${params.toString()}`;
+  const clientHeaders = getClientRegistrationHeaders();
+  tuiLog.info("sse connect", { url, clientHeaders });
   const response = await fetch(url, {
     headers: {
       Accept: "text/event-stream",
-      ...(bearerToken ? { Authorization: `Bearer ${bearerToken}` } : {}),
-      ...getClientRegistrationHeaders(),
+      ...auth,
+      ...clientHeaders,
     },
     signal,
   });
 
+  tuiLog.info("sse response", {
+    status: response.status,
+    statusText: response.statusText,
+    contentType: response.headers.get("content-type"),
+  });
+
   if (!response.ok) {
     const body = await response.text().catch(() => "");
+    tuiLog.error("sse connection failed", {
+      status: response.status,
+      body: body.slice(0, 500),
+    });
     throw new Error(
       `SSE connection failed (${response.status}): ${body || response.statusText}`,
     );
   }
   if (!response.body) {
+    tuiLog.error("sse response has no body");
     throw new Error("No response body from SSE endpoint");
   }
 
@@ -443,7 +454,7 @@ async function handleConfirmationPrompt(
   requestId: string,
   confirmation: PendingConfirmation,
   chatApp: ChatAppHandle,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<void> {
   const preview = formatConfirmationPreview(
     confirmation.toolName,
@@ -469,7 +480,7 @@ async function handleConfirmationPrompt(
   const index = await chatApp.showSelection("Tool Approval", options);
 
   if (index === 0) {
-    await submitDecision(baseUrl, assistantId, requestId, "allow", bearerToken);
+    await submitDecision(baseUrl, assistantId, requestId, "allow", auth);
     chatApp.addStatus("\u2714 Allowed", "green");
     return;
   }
@@ -481,7 +492,7 @@ async function handleConfirmationPrompt(
       confirmation,
       chatApp,
       "always_allow",
-      bearerToken,
+      auth,
     );
     return;
   }
@@ -493,12 +504,12 @@ async function handleConfirmationPrompt(
       confirmation,
       chatApp,
       "always_deny",
-      bearerToken,
+      auth,
     );
     return;
   }
 
-  await submitDecision(baseUrl, assistantId, requestId, "deny", bearerToken);
+  await submitDecision(baseUrl, assistantId, requestId, "deny", auth);
   chatApp.addStatus("\u2718 Denied", "yellow");
 }
 
@@ -509,7 +520,7 @@ async function handlePatternSelection(
   confirmation: PendingConfirmation,
   chatApp: ChatAppHandle,
   trustDecision: TrustDecision,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<void> {
   const allowlistOptions = confirmation.allowlistOptions ?? [];
   const label = trustDecision === "always_deny" ? "Denylist" : "Allowlist";
@@ -530,12 +541,12 @@ async function handlePatternSelection(
       chatApp,
       selectedPattern,
       trustDecision,
-      bearerToken,
+      auth,
     );
     return;
   }
 
-  await submitDecision(baseUrl, assistantId, requestId, "deny", bearerToken);
+  await submitDecision(baseUrl, assistantId, requestId, "deny", auth);
   chatApp.addStatus("\u2718 Denied", "yellow");
 }
 
@@ -547,7 +558,7 @@ async function handleScopeSelection(
   chatApp: ChatAppHandle,
   selectedPattern: string,
   trustDecision: TrustDecision,
-  bearerToken?: string,
+  auth?: Record<string, string>,
 ): Promise<void> {
   const scopeOptions = confirmation.scopeOptions ?? [];
   const label = trustDecision === "always_deny" ? "Denylist" : "Allowlist";
@@ -564,14 +575,14 @@ async function handleScopeSelection(
       selectedPattern,
       scopeOptions[index].scope,
       ruleDecision,
-      bearerToken,
+      auth,
     );
     await submitDecision(
       baseUrl,
       assistantId,
       requestId,
       ruleDecision === "deny" ? "deny" : "allow",
-      bearerToken,
+      auth,
     );
     const ruleLabel =
       trustDecision === "always_deny" ? "Denylisted" : "Allowlisted";
@@ -583,7 +594,7 @@ async function handleScopeSelection(
     return;
   }
 
-  await submitDecision(baseUrl, assistantId, requestId, "deny", bearerToken);
+  await submitDecision(baseUrl, assistantId, requestId, "deny", auth);
   chatApp.addStatus("\u2718 Denied", "yellow");
 }
 
@@ -739,10 +750,6 @@ function HelpDisplay(): ReactElement {
         {"  /clear            "}
         <Text dimColor>Clear the screen</Text>
       </Text>
-      <Text>
-        {"  /pair             "}
-        <Text dimColor>Generate a QR code for mobile device pairing</Text>
-      </Text>
       <Text>
         {"  /help, ?          "}
         <Text dimColor>Show this help</Text>
@@ -1354,7 +1361,9 @@ interface ChatAppProps {
   runtimeUrl: string;
   assistantId: string;
   species: Species;
-  bearerToken?: string;
+  /** Pre-built auth headers (e.g. { Authorization: "Bearer ..." } for local,
+   *  { "X-Session-Token": "...", "Vellum-Organization-Id": "..." } for platform). */
+  auth?: Record<string, string>;
   project?: string;
   zone?: string;
   onExit: () => void;
@@ -1365,7 +1374,7 @@ function ChatApp({
   runtimeUrl,
   assistantId,
   species,
-  bearerToken,
+  auth,
   project,
   zone,
   onExit,
@@ -1653,6 +1662,10 @@ function ChatApp({
 
     try {
       const health = await checkHealthRuntime(runtimeUrl);
+      tuiLog.info("health check", {
+        status: health.status,
+        message: health.message,
+      });
       h.hideSpinner();
       h.updateHealthStatus(health.status);
       if (health.status === "healthy" || health.status === "ok") {
@@ -1674,7 +1687,7 @@ function ChatApp({
         const historyResponse = await pollMessages(
           runtimeUrl,
           assistantId,
-          bearerToken,
+          auth,
         );
         h.hideSpinner();
         if (historyResponse.messages.length > 0) {
@@ -1699,7 +1712,7 @@ function ChatApp({
             assistantId,
             assistantId,
             sseAc.signal,
-            bearerToken,
+            auth,
           )) {
             const hRef = handleRef_.current;
             if (!hRef) continue;
@@ -1770,7 +1783,7 @@ function ChatApp({
                       event.persistentDecisionsAllowed,
                   },
                   hRef,
-                  bearerToken,
+                  auth,
                 );
                 hRef.showSpinner("Working...");
                 break;
@@ -1801,7 +1814,7 @@ function ChatApp({
                           delivery,
                         }),
                       },
-                      bearerToken,
+                      auth,
                     );
                   },
                 );
@@ -1850,9 +1863,12 @@ function ChatApp({
                 break;
             }
           }
-        } catch {
+        } catch (sseErr) {
           // Stream ended — only report if not intentionally aborted
           if (!sseAc.signal.aborted) {
+            tuiLog.warn("sse stream disconnected", {
+              error: String(sseErr),
+            });
             handleRef_.current?.addStatus(
               "SSE stream disconnected — will reconnect on next message",
               "yellow",
@@ -1869,10 +1885,11 @@ function ChatApp({
       setConnectionState("connected");
       return true;
     } catch (err) {
+      const msg = err instanceof Error ? err.message : String(err);
+      tuiLog.error("connection failed", { error: msg });
       h.hideSpinner();
       connectingRef.current = false;
       h.updateHealthStatus("unreachable");
-      const msg = err instanceof Error ? err.message : String(err);
       setConnectionState("error");
       setConnectionError(msg);
       h.addStatus(
@@ -1881,7 +1898,7 @@ function ChatApp({
       );
       return false;
     }
-  }, [runtimeUrl, assistantId, bearerToken]);
+  }, [runtimeUrl, assistantId, auth]);
 
   const handleInput = useCallback(
     async (input: string): Promise<void> => {
@@ -2088,9 +2105,7 @@ function ChatApp({
               method: "POST",
               headers: {
                 "Content-Type": "application/json",
-                ...(bearerToken
-                  ? { Authorization: `Bearer ${bearerToken}` }
-                  : {}),
+                ...auth,
               },
               body: JSON.stringify({
                 conversationKey: assistantId,
@@ -2149,85 +2164,6 @@ function ChatApp({
         return;
       }
 
-      if (trimmed === "/pair") {
-        h.showSpinner("Generating pairing credentials...");
-
-        const isConnected = await ensureConnected();
-        if (!isConnected) {
-          h.hideSpinner();
-          h.showError("Cannot pair — not connected to the assistant runtime.");
-          return;
-        }
-
-        try {
-          const pairingRequestId = randomUUID();
-          const pairingSecret = randomBytes(32).toString("hex");
-          const gatewayUrl = runtimeUrl;
-
-          // Call /pairing/register on the gateway (dedicated pairing proxy route)
-          const registerUrl = `${runtimeUrl}/pairing/register`;
-          const registerRes = await fetch(registerUrl, {
-            method: "POST",
-            headers: {
-              "Content-Type": "application/json",
-              ...(bearerToken
-                ? { Authorization: `Bearer ${bearerToken}` }
-                : {}),
-            },
-            body: JSON.stringify({
-              pairingRequestId,
-              pairingSecret,
-              gatewayUrl,
-            }),
-          });
-
-          if (!registerRes.ok) {
-            const body = await registerRes.text().catch(() => "");
-            throw new Error(
-              `HTTP ${registerRes.status}: ${body || registerRes.statusText}`,
-            );
-          }
-
-          let username: string;
-          try {
-            username = userInfo().username;
-          } catch {
-            username = "";
-          }
-          const hostId = createHash("sha256")
-            .update(hostname() + username)
-            .digest("hex");
-          const payload = JSON.stringify({
-            type: "vellum-assistant",
-            v: 4,
-            id: hostId,
-            g: gatewayUrl,
-            pairingRequestId,
-            pairingSecret,
-          });
-
-          const qrString = await new Promise<string>((resolve) => {
-            qrcode.generate(payload, { small: true }, (code: string) => {
-              resolve(code);
-            });
-          });
-
-          h.hideSpinner();
-          h.addStatus(
-            `Pairing Ready\n\n` +
-              `Scan this QR code with the Vellum iOS app:\n\n` +
-              `${qrString}\n` +
-              `This pairing request expires in 5 minutes. Run /pair again to generate a new one.`,
-          );
-        } catch (err) {
-          h.hideSpinner();
-          h.showError(
-            `Pairing failed: ${err instanceof Error ? err.message : err}`,
-          );
-        }
-        return;
-      }
-
       if (busyRef.current) {
         // /btw is already handled above this block
         if (!trimmed.startsWith("/")) {
@@ -2256,7 +2192,7 @@ function ChatApp({
             assistantId,
             trimmed,
             controller.signal,
-            bearerToken,
+            auth,
           );
           clearTimeout(timeoutId);
           if (sendResult.accepted) {
@@ -2307,7 +2243,7 @@ function ChatApp({
           assistantId,
           trimmed,
           controller.signal,
-          bearerToken,
+          auth,
         );
         clearTimeout(timeoutId);
         if (!sendResult.accepted) {
@@ -2334,7 +2270,7 @@ function ChatApp({
     [
       runtimeUrl,
       assistantId,
-      bearerToken,
+      auth,
       project,
       zone,
       cleanup,
@@ -2648,7 +2584,7 @@ export function renderChatApp(
   assistantId: string,
   species: Species,
   onExit: () => void,
-  options?: { bearerToken?: string; project?: string; zone?: string },
+  options?: { auth?: Record<string, string>; project?: string; zone?: string },
 ): ChatAppInstance {
   let chatHandle: ChatAppHandle | null = null;
 
@@ -2657,7 +2593,7 @@ export function renderChatApp(
       runtimeUrl={runtimeUrl}
       assistantId={assistantId}
       species={species}
-      bearerToken={options?.bearerToken}
+      auth={options?.auth}
       project={options?.project}
       zone={options?.zone}
       onExit={onExit}

package/src/index.ts

@@ -11,7 +11,6 @@ import { hatch } from "./commands/hatch";
 import { login, logout, whoami } from "./commands/login";
 import { logs } from "./commands/logs";
 import { message } from "./commands/message";
-import { pair } from "./commands/pair";
 import { ps } from "./commands/ps";
 import { recover } from "./commands/recover";
 import { restore } from "./commands/restore";
@@ -26,12 +25,7 @@ import { tunnel } from "./commands/tunnel";
 import { upgrade } from "./commands/upgrade";
 import { use } from "./commands/use";
 import { wake } from "./commands/wake";
-import {
-  getActiveAssistant,
-  findAssistantByName,
-  loadLatestAssistant,
-  setActiveAssistant,
-} from "./lib/assistant-config";
+import { resolveAssistant, setActiveAssistant } from "./lib/assistant-config";
 import { loadGuardianToken } from "./lib/guardian-token";
 import { checkHealth } from "./lib/health-check";
 
@@ -47,7 +41,6 @@ const commands = {
   logout,
   logs,
   message,
-  pair,
   ps,
   recover,
   restore,
@@ -82,7 +75,6 @@ function printHelp(): void {
   console.log("  login    Log in to the Vellum platform");
   console.log("  logout   Log out of the Vellum platform");
   console.log("  message  Send a message to a running assistant");
-  console.log("  pair     Pair with a remote assistant via QR code");
   console.log(
     "  ps       List assistants (or processes for a specific assistant)",
   );
@@ -129,10 +121,7 @@ function applyNoColorFlags(argv: string[]): void {
  * Otherwise return false so the caller can fall back to help text.
  */
 async function tryLaunchClient(): Promise<boolean> {
-  const activeName = getActiveAssistant();
-  const entry = activeName
-    ? findAssistantByName(activeName)
-    : loadLatestAssistant();
+  const entry = resolveAssistant();
 
   if (!entry) return false;
 

package/src/lib/__tests__/docker.test.ts

@@ -29,50 +29,41 @@ function buildAssistantArgs(
   return builders.assistant();
 }
 
-describe("serviceDockerRunArgs — assistant", () => {
-  test("grants the minimum capability set needed for DinD (SYS_ADMIN + NET_ADMIN) rather than --privileged", () => {
-    const args = buildAssistantArgs();
-    expect(args).not.toContain("--privileged");
-    // --cap-add SYS_ADMIN and --cap-add NET_ADMIN are each passed as two
-    // adjacent args: "--cap-add" followed by the capability name.
-    const sysAdminIdx = args.indexOf("SYS_ADMIN");
-    expect(sysAdminIdx).toBeGreaterThan(0);
-    expect(args[sysAdminIdx - 1]).toBe("--cap-add");
-    const netAdminIdx = args.indexOf("NET_ADMIN");
-    expect(netAdminIdx).toBeGreaterThan(0);
-    expect(args[netAdminIdx - 1]).toBe("--cap-add");
+function buildGatewayArgs(
+  overrides: Partial<Parameters<typeof serviceDockerRunArgs>[0]> = {},
+): string[] {
+  const res = dockerResourceNames(instanceName);
+  const builders = serviceDockerRunArgs({
+    gatewayPort: 7830,
+    imageTags,
+    instanceName,
+    res,
+    ...overrides,
   });
+  return builders.gateway();
+}
 
-  test("disables the default seccomp and AppArmor profiles so the inner dockerd can mount overlayfs and run pivot_root", () => {
+describe("serviceDockerRunArgs — assistant", () => {
+  test("does not grant elevated capabilities or disable security profiles", () => {
     const args = buildAssistantArgs();
-    const seccompIdx = args.indexOf("seccomp=unconfined");
-    expect(seccompIdx).toBeGreaterThan(0);
-    expect(args[seccompIdx - 1]).toBe("--security-opt");
-    const apparmorIdx = args.indexOf("apparmor=unconfined");
-    expect(apparmorIdx).toBeGreaterThan(0);
-    expect(args[apparmorIdx - 1]).toBe("--security-opt");
+    expect(args).not.toContain("--privileged");
+    expect(args).not.toContain("--cap-add");
+    expect(args).not.toContain("SYS_ADMIN");
+    expect(args).not.toContain("NET_ADMIN");
+    expect(args).not.toContain("seccomp=unconfined");
+    expect(args).not.toContain("apparmor=unconfined");
   });
 
-  test("mounts a dedicated named volume at /var/lib/docker for the inner dockerd data store", () => {
+  test("does not mount a dockerd data volume", () => {
     const args = buildAssistantArgs();
-    const spec = `${instanceName}-dockerd-data:/var/lib/docker`;
-    const mountIndex = args.indexOf(spec);
-    expect(mountIndex).toBeGreaterThan(0);
-    expect(args[mountIndex - 1]).toBe("-v");
+    expect(args.some((a) => a.includes("/var/lib/docker"))).toBe(false);
   });
 
-  test("does NOT bind-mount the host Docker socket (DinD replaces host-socket access)", () => {
+  test("does NOT bind-mount the host Docker socket", () => {
     const args = buildAssistantArgs();
     expect(args).not.toContain("/var/run/docker.sock:/var/run/docker.sock");
   });
 
-  test("does NOT set VELLUM_WORKSPACE_VOLUME_NAME (legacy Phase 1.8 hint, no longer needed in DinD)", () => {
-    const args = buildAssistantArgs();
-    expect(
-      args.some((a) => a.startsWith("VELLUM_WORKSPACE_VOLUME_NAME=")),
-    ).toBe(false);
-  });
-
   test("keeps existing workspace and socket volume mounts intact", () => {
     const args = buildAssistantArgs();
     expect(args).toContain(`${instanceName}-workspace:/workspace`);
@@ -112,6 +103,33 @@ describe("serviceDockerRunArgs — assistant", () => {
   });
 });
 
+describe("serviceDockerRunArgs — gateway", () => {
+  const savedVelayBaseUrl = process.env.VELAY_BASE_URL;
+
+  beforeEach(() => {
+    delete process.env.VELAY_BASE_URL;
+  });
+
+  afterEach(() => {
+    if (savedVelayBaseUrl === undefined) delete process.env.VELAY_BASE_URL;
+    else process.env.VELAY_BASE_URL = savedVelayBaseUrl;
+  });
+
+  test("passes VELAY_BASE_URL into the gateway container when set", () => {
+    process.env.VELAY_BASE_URL = "http://host.docker.internal:8501";
+
+    expect(buildGatewayArgs()).toContain(
+      "VELAY_BASE_URL=http://host.docker.internal:8501",
+    );
+  });
+
+  test("omits VELAY_BASE_URL from gateway args when unset", () => {
+    expect(
+      buildGatewayArgs().some((arg) => arg.startsWith("VELAY_BASE_URL=")),
+    ).toBe(false);
+  });
+});
+
 describe("VELLUM_AVATAR_DEVICE passthrough", () => {
   const savedValue = process.env[AVATAR_DEVICE_ENV_VAR];