@vellumai/cli 0.10.11-dev.202607242127.a6bf2b2 → 0.10.11-staging.2

package/knip.json

@@ -11,7 +11,6 @@
   "ignoreDependencies": [
     "@vellumai/environments",
     "@vellumai/local-mode",
-    "@vellumai/service-contracts",
     "@vellumai/web"
   ]
 }

package/node_modules/@vellumai/local-mode/src/lockfile-contract.test.ts

@@ -24,7 +24,6 @@ describe("parseLockfile", () => {
           platformAssistantId: "platform-assistant-1",
           platformBaseUrl: "https://platform.example.com",
           platformOrganizationId: "org_1",
-          ingressUrl: "https://tunnel.example.ts.net",
           resources: { gatewayPort: 7777, daemonPort: 7778 },
         },
       ],
@@ -187,30 +186,6 @@ describe("parseLockfile", () => {
     ]);
   });
 
-  test("keeps a well-typed ingressUrl and drops it when mistyped", () => {
-    // The tunnel-recorded public URL rides the entry so remote-web pairing
-    // surfaces can prefill it; a mistyped value is dropped, entry preserved.
-    const raw = {
-      assistants: [
-        {
-          assistantId: "asst_1",
-          cloud: "local",
-          ingressUrl: "https://tunnel.example.ts.net",
-        },
-        { assistantId: "asst_2", cloud: "local", ingressUrl: 7 }, // mistyped
-      ],
-      activeAssistant: null,
-    };
-    expect(parseLockfile(raw).assistants).toEqual([
-      {
-        assistantId: "asst_1",
-        cloud: "local",
-        ingressUrl: "https://tunnel.example.ts.net",
-      },
-      { assistantId: "asst_2", cloud: "local" },
-    ]);
-  });
-
   test("drops a resources object missing its numeric ports", () => {
     const raw = {
       assistants: [

package/node_modules/@vellumai/local-mode/src/lockfile-contract.ts

@@ -115,13 +115,6 @@ export const LockfileAssistantSchema = z.object({
   platformBaseUrl: z.string().optional(),
   /** Platform organization UUID used for a self-hosted local assistant registration. */
   platformOrganizationId: z.string().optional(),
-  /**
-   * Public https URL a `vellum tunnel` provider recorded for this assistant
-   * (mirrored from the workspace ingress config). A public address, so it is
-   * renderer-safe: remote-web pairing surfaces (the CLI `--qr` flow, the web
-   * "Pair a device" card) default to it instead of asking the user to retype it.
-   */
-  ingressUrl: z.string().optional(),
   resources: LocalAssistantResourcesSchema.optional(),
 });
 

package/node_modules/@vellumai/local-mode/src/lockfile.test.ts

@@ -81,30 +81,6 @@ describe("getLockfileData", () => {
     }
   });
 
-  test("surfaces a tunnel-recorded ingressUrl to the renderer", () => {
-    // The exact read the "Pair a device" card relies on: `vellum tunnel` stamps
-    // ingressUrl onto the entry, and the host read must carry it through.
-    writeOnDisk({
-      activeAssistant: "asst_1",
-      assistants: [
-        {
-          assistantId: "asst_1",
-          cloud: "local",
-          runtimeUrl: "http://127.0.0.1:7777",
-          ingressUrl: "https://tunnel.example.ts.net",
-        },
-      ],
-    });
-
-    const result = getLockfileData([lockfilePath]);
-    expect(result.ok).toBe(true);
-    if (result.ok) {
-      expect(result.data.assistants[0]?.ingressUrl).toBe(
-        "https://tunnel.example.ts.net",
-      );
-    }
-  });
-
   test("fails with status 500 on malformed JSON", () => {
     fs.writeFileSync(lockfilePath, "{ not json");
     const result = getLockfileData([lockfilePath]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vellumai/cli",
-  "version": "0.10.11-dev.202607242127.a6bf2b2",
+  "version": "0.10.11-staging.2",
   "description": "CLI tools for vellum-assistant",
   "type": "module",
   "exports": {
@@ -31,21 +31,17 @@
   "dependencies": {
     "@vellumai/environments": "file:../packages/environments",
     "@vellumai/local-mode": "file:../packages/local-mode",
-    "@vellumai/service-contracts": "file:../packages/service-contracts",
     "chalk": "5.6.2",
     "ink": "6.8.0",
     "nanoid": "5.1.7",
-    "qrcode-terminal": "0.12.0",
     "react": "19.2.4"
   },
   "bundledDependencies": [
     "@vellumai/environments",
-    "@vellumai/local-mode",
-    "@vellumai/service-contracts"
+    "@vellumai/local-mode"
   ],
   "devDependencies": {
     "@types/bun": "1.3.11",
-    "@types/qrcode-terminal": "0.12.2",
     "@types/react": "19.2.14",
     "eslint": "10.1.0",
     "knip": "5.88.1",

package/src/__tests__/assistant-config.test.ts

@@ -1,19 +1,5 @@
-import {
-  describe,
-  test,
-  expect,
-  beforeEach,
-  afterEach,
-  afterAll,
-  spyOn,
-} from "bun:test";
-import {
-  mkdirSync,
-  mkdtempSync,
-  readFileSync,
-  rmSync,
-  writeFileSync,
-} from "node:fs";
+import { describe, test, expect, beforeEach, afterAll } from "bun:test";
+import { mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 
@@ -30,7 +16,6 @@ import {
   lookupAssistantByIdentifier,
   removeAssistantEntry,
   loadAllAssistants,
-  resolveTargetAssistant,
   saveAssistantEntry,
   getActiveAssistant,
   migrateLegacyEntry,
@@ -715,104 +700,3 @@ describe("env-scoped lockfile and migration", () => {
     }
   });
 });
-
-describe("cross-environment hint on resolution failure", () => {
-  let dataHome: string;
-  let prevXdg: string | undefined;
-  let prevEnv: string | undefined;
-
-  beforeEach(() => {
-    prevXdg = process.env.XDG_DATA_HOME;
-    prevEnv = process.env.VELLUM_ENVIRONMENT;
-    dataHome = mkdtempSync(join(tmpdir(), "cli-config-xdg-"));
-    process.env.XDG_DATA_HOME = dataHome;
-    // Pin the current env to production so the detector skips it and the hint
-    // copy is deterministic. The current-env lockfile is emptied so resolution
-    // genuinely finds nothing.
-    process.env.VELLUM_ENVIRONMENT = "production";
-    rmSync(join(testDir, ".vellum.lock.json"), { force: true });
-    rmSync(join(testDir, ".vellum.lockfile.json"), { force: true });
-  });
-
-  afterEach(() => {
-    if (prevXdg === undefined) {
-      delete process.env.XDG_DATA_HOME;
-    } else {
-      process.env.XDG_DATA_HOME = prevXdg;
-    }
-    if (prevEnv === undefined) {
-      delete process.env.VELLUM_ENVIRONMENT;
-    } else {
-      process.env.VELLUM_ENVIRONMENT = prevEnv;
-    }
-    rmSync(dataHome, { recursive: true, force: true });
-  });
-
-  // A dev-env assistant lives under the dev data dir, invisible to this
-  // production CLI — the exact split the hint explains.
-  function seedDevAssistant(): void {
-    mkdirSync(join(dataHome, "vellum-dev", "assistants", "dev-bot"), {
-      recursive: true,
-    });
-  }
-
-  function captureResolveTargetAssistantError(): string {
-    const errors: string[] = [];
-    const errorSpy = spyOn(console, "error").mockImplementation(
-      (...args: unknown[]) => {
-        errors.push(args.join(" "));
-      },
-    );
-    const exitSpy = spyOn(process, "exit").mockImplementation(((
-      _code?: number,
-    ) => {
-      throw new Error("exit");
-    }) as typeof process.exit);
-    try {
-      expect(() => resolveTargetAssistant()).toThrow("exit");
-    } finally {
-      errorSpy.mockRestore();
-      exitSpy.mockRestore();
-    }
-    return errors.join("\n");
-  }
-
-  test("formatAssistantLookupError appends the hint when another env has assistants", () => {
-    seedDevAssistant();
-    const msg = formatAssistantLookupError("ghost", { status: "not_found" });
-    expect(msg).toContain("No assistant found with name or ID 'ghost'.");
-    expect(msg).toContain("Install vellum Command");
-    expect(msg).toContain("VELLUM_ENVIRONMENT=dev");
-  });
-
-  test("formatAssistantLookupError stays bare when no other env has assistants", () => {
-    // dataHome is empty — detection finds nothing, so the message is intact.
-    const msg = formatAssistantLookupError("ghost", { status: "not_found" });
-    expect(msg).toBe("No assistant found with name or ID 'ghost'.");
-  });
-
-  test("ambiguous lookup never gets the hint", () => {
-    seedDevAssistant();
-    const msg = formatAssistantLookupError("Alice", {
-      status: "ambiguous",
-      matches: [
-        makeEntry("assistant-1", "http://localhost:7821", { name: "Alice" }),
-        makeEntry("assistant-2", "http://localhost:7822", { name: "Alice" }),
-      ],
-    });
-    expect(msg).not.toContain("Install vellum Command");
-  });
-
-  test("resolveTargetAssistant zero-assistants error carries the hint", () => {
-    seedDevAssistant();
-    const combined = captureResolveTargetAssistantError();
-    expect(combined).toContain("No assistant found. Run 'vellum hatch' first.");
-    expect(combined).toContain("Install vellum Command");
-  });
-
-  test("resolveTargetAssistant zero-assistants error stays bare without other envs", () => {
-    const combined = captureResolveTargetAssistantError();
-    expect(combined).toContain("No assistant found. Run 'vellum hatch' first.");
-    expect(combined).not.toContain("Install vellum Command");
-  });
-});

package/src/__tests__/flags.test.ts

@@ -188,7 +188,14 @@ describe("vellum flags --assistant routing", () => {
       ],
       "alice-1",
     );
-    process.argv = ["bun", "vellum", "flags", "set", "voice-mode", "true"];
+    process.argv = [
+      "bun",
+      "vellum",
+      "flags",
+      "set",
+      "voice-mode",
+      "true",
+    ];
 
     await flags();
 
@@ -239,80 +246,3 @@ describe("vellum flags --assistant routing", () => {
     expect(fetchCalls.length).toBe(0);
   });
 });
-
-describe("vellum flags cross-environment hint", () => {
-  let xdgDataHome: string;
-  let prevXdg: string | undefined;
-  let prevEnv: string | undefined;
-
-  beforeEach(() => {
-    process.env.VELLUM_LOCKFILE_DIR = testDir;
-    rmSync(join(testDir, ".vellum.lock.json"), { force: true });
-    prevXdg = process.env.XDG_DATA_HOME;
-    prevEnv = process.env.VELLUM_ENVIRONMENT;
-    xdgDataHome = mkdtempSync(join(tmpdir(), "cli-flags-hint-xdg-"));
-    process.env.XDG_DATA_HOME = xdgDataHome;
-    // Pin production so the detector skips it and the lockfile routes to
-    // testDir; the VELLUM_ENVIRONMENT var beats this machine's env config.
-    process.env.VELLUM_ENVIRONMENT = "production";
-    // An unreachable gateway: every request throws like a dead socket would.
-    // Cast through `unknown` since an always-throwing stub returns
-    // `Promise<never>`, which does not structurally overlap with `fetch`.
-    globalThis.fetch = (async () => {
-      throw new TypeError("fetch failed");
-    }) as unknown as typeof globalThis.fetch;
-    process.exit = ((code?: number) => {
-      throw new Error(`process.exit:${code}`);
-    }) as typeof process.exit;
-    consoleLogSpy = spyOn(console, "log").mockImplementation(() => {});
-    consoleErrorSpy = spyOn(console, "error").mockImplementation(() => {});
-  });
-
-  afterEach(() => {
-    process.argv = originalArgv;
-    process.exit = originalExit;
-    globalThis.fetch = originalFetch;
-    consoleLogSpy.mockRestore();
-    consoleErrorSpy.mockRestore();
-    if (prevXdg === undefined) {
-      delete process.env.XDG_DATA_HOME;
-    } else {
-      process.env.XDG_DATA_HOME = prevXdg;
-    }
-    if (prevEnv === undefined) {
-      delete process.env.VELLUM_ENVIRONMENT;
-    } else {
-      process.env.VELLUM_ENVIRONMENT = prevEnv;
-    }
-    rmSync(xdgDataHome, { recursive: true, force: true });
-  });
-
-  // A dev-env assistant lives under the dev data dir, invisible to this
-  // production CLI — the split the hint explains.
-  function seedDevAssistant(): void {
-    mkdirSync(join(xdgDataHome, "vellum-dev", "assistants", "dev-bot"), {
-      recursive: true,
-    });
-  }
-
-  test("gateway-unreachable error gains the hint when another env has assistants", async () => {
-    writeLockfile([makeEntry("alice-1", { name: "Alice" })], "alice-1");
-    seedDevAssistant();
-    process.argv = ["bun", "vellum", "flags", "set", "voice-mode", "true"];
-    await expect(flags()).rejects.toThrow(/Install vellum Command/);
-  });
-
-  test("gateway-unreachable error stays bare when no other env has assistants", async () => {
-    writeLockfile([makeEntry("alice-1", { name: "Alice" })], "alice-1");
-    // xdgDataHome is empty — detection finds nothing, so the message is intact.
-    process.argv = ["bun", "vellum", "flags", "set", "voice-mode", "true"];
-    let error: Error | undefined;
-    try {
-      await flags();
-    } catch (e) {
-      error = e as Error;
-    }
-    expect(error?.message).toContain("Could not reach the assistant gateway");
-    expect(error?.message).not.toContain("Install vellum Command");
-  });
-});

package/src/__tests__/input-history.test.ts

@@ -1,5 +1,10 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { existsSync, mkdtempSync, readFileSync, rmSync } from "node:fs";
+import {
+  existsSync,
+  mkdtempSync,
+  readFileSync,
+  rmSync,
+} from "node:fs";
 import { homedir, tmpdir } from "node:os";
 import { join } from "node:path";
 

package/src/__tests__/login-loopback.test.ts

@@ -3,9 +3,7 @@ import { describe, expect, test } from "bun:test";
 import { startLoopbackListener } from "../commands/login.js";
 
 /** Resolve "settled"/"pending" — proves whether `waitForCode` resolved. */
-async function settleState(
-  p: Promise<unknown>,
-): Promise<"settled" | "pending"> {
+async function settleState(p: Promise<unknown>): Promise<"settled" | "pending"> {
   return Promise.race([
     p.then(
       () => "settled" as const,
@@ -51,9 +49,7 @@ describe("startLoopbackListener", () => {
         () => null,
         (e: Error) => e,
       );
-      const res = await fetch(
-        `${listener.redirectUri}?error=access_denied&state=st`,
-      );
+      const res = await fetch(`${listener.redirectUri}?error=access_denied&state=st`);
       expect(res.status).toBe(400);
       const err = await settled;
       expect(err?.message).toMatch(/access_denied/);

package/src/__tests__/nginx-ingress.test.ts

@@ -59,13 +59,6 @@ describe("buildIngressNginxConfig", () => {
     }
   });
 
-  test("emits relative redirects so a fronting proxy keeps scheme and port", () => {
-    for (const config of [conf, remoteConf]) {
-      expect(config).toContain("absolute_redirect off;");
-      expect(config).toContain("port_in_redirect off;");
-    }
-  });
-
   test("proxies requests to the gateway", () => {
     expect(conf).toContain("location / {");
     expect(conf).toContain("proxy_pass http://127.0.0.1:7830;");