@vellumai/assistant 0.7.3 → 0.8.1

package/src/cli/commands/bash.ts

@@ -1,6 +1,7 @@
 import type { Command } from "commander";
 
 import { cliIpcCall } from "../../ipc/cli-client.js";
+import { registerCommand } from "../lib/register-command.js";
 import { log } from "../logger.js";
 
 const DEFAULT_TIMEOUT_MS = 30_000;
@@ -14,11 +15,12 @@ interface DebugBashResult {
 }
 
 export function registerBashCommand(program: Command): void {
-  program
-    .command("bash <command>")
-    .description(
-      "Execute a shell command through the assistant process for debugging",
-    )
+  registerCommand(program, {
+    name: "bash <command>",
+    transport: "ipc",
+    description: "Execute a shell command through the assistant process for debugging",
+    build: (cmd) => {
+      cmd
     .option(
       "-t, --timeout <ms>",
       "Timeout in milliseconds for command execution",
@@ -100,4 +102,6 @@ Examples:
 
       process.exitCode = data.exitCode ?? 1;
     });
+    },
+  });
 }

package/src/cli/commands/browser.ts

@@ -17,6 +17,7 @@ import type {
   OperationField,
 } from "../../browser/types.js";
 import { cliIpcCall } from "../../ipc/cli-client.js";
+import { registerCommand } from "../lib/register-command.js";
 import { log } from "../logger.js";
 
 // ── Naming helpers ───────────────────────────────────────────────────
@@ -164,22 +165,31 @@ function buildSubcommand(parent: Command, meta: BrowserOperationMeta): void {
       session?: string;
       json?: boolean;
       browserMode?: string;
+      targetClientId?: string;
     };
     const sessionId = parentOpts.session ?? "default";
     const jsonMode = parentOpts.json ?? false;
     const conversationId = resolveContextConversationId();
 
     // Map Commander camelCase options back to snake_case input keys,
-    // filtering out parent-level options (session, json, browserMode)
-    // and screenshot ergonomics (output).
+    // filtering out parent-level options (session, json, browserMode,
+    // targetClientId) and screenshot ergonomics (output).
     const input: Record<string, unknown> = {};
-    const excludeKeys = new Set(["session", "json", "output", "browserMode"]);
-
-    // Inject parent-level --browser-mode into the operation input so
-    // the backend receives the mode override for backend pinning.
+    const excludeKeys = new Set([
+      "session",
+      "json",
+      "output",
+      "browserMode",
+      "targetClientId",
+    ]);
+
+    // Inject parent-level flags into the operation input.
     if (parentOpts.browserMode) {
       input.browser_mode = parentOpts.browserMode;
     }
+    if (parentOpts.targetClientId) {
+      input.target_client_id = parentOpts.targetClientId;
+    }
 
     for (const [key, value] of Object.entries(opts)) {
       if (excludeKeys.has(key)) continue;
@@ -348,9 +358,12 @@ const BROWSER_MODES = [
 ] as const;
 
 export function registerBrowserCommand(program: Command): void {
-  const browser = program
-    .command("browser")
-    .description("Control the browser via the running assistant.")
+  registerCommand(program, {
+    name: "browser",
+    transport: "ipc",
+    description: "Control the browser via the running assistant.",
+    build: (browser) => {
+      browser
     .option(
       "--session <id>",
       "Session ID to preserve browser state across invocations.",
@@ -362,6 +375,10 @@ export function registerBrowserCommand(program: Command): void {
         "--browser-mode <mode>",
         "Browser backend to use. Overrides automatic selection.",
       ).choices([...BROWSER_MODES]),
+    )
+    .option(
+      "--target-client-id <id>",
+      "Route browser operations to a specific client. Obtain IDs from `assistant clients list --capability host_browser`.",
     );
 
   browser.addHelpText(
@@ -395,4 +412,6 @@ Examples:
   for (const meta of BROWSER_OPERATION_META) {
     buildSubcommand(browser, meta);
   }
+    },
+  });
 }

package/src/cli/commands/cache.ts

@@ -8,8 +8,9 @@
 import type { Command } from "commander";
 
 import { cliIpcCall } from "../../ipc/cli-client.js";
+import { existsSync, readFileSync } from "../lib/cache-fs.js";
+import { registerCommand } from "../lib/register-command.js";
 import { log } from "../logger.js";
-import { existsSync, readFileSync } from "./cache-fs.js";
 
 // ── Constants ─────────────────────────────────────────────────────────
 
@@ -167,9 +168,11 @@ function resolvePayload(opts: { value?: string; file?: string }): unknown {
 // ── Registration ──────────────────────────────────────────────────────
 
 export function registerCacheCommand(program: Command): void {
-  const cache = program
-    .command("cache")
-    .description("Interact with the assistant's in-memory key/value cache");
+  registerCommand(program, {
+    name: "cache",
+    transport: "ipc",
+    description: "Interact with the assistant's in-memory key/value cache",
+    build: (cache) => {
 
   cache.addHelpText(
     "after",
@@ -411,4 +414,6 @@ Examples:
         }
       }
     });
+    },
+  });
 }

package/src/cli/commands/changelog.ts

@@ -0,0 +1,414 @@
+/**
+ * `assistant changelog`
+ *
+ * Lazy, fetched-on-demand release-notes surface. Pulls release bodies from
+ * the public GitHub Releases API for `vellum-ai/vellum-assistant` and caches
+ * them locally so repeat invocations don't hit the network.
+ *
+ * Subcommands:
+ *   - default action — show the latest stable release
+ *   - `--since <version>` — concatenate every stable release newer than
+ *     <version>, newest first
+ *   - `show <version>` — show a single specific release tag
+ *   - `list` — print the recent release tags and dates
+ *
+ * The CLI is intentionally a pure read-side surface; nothing is pushed to
+ * the user at startup. A follow-up PR will add a tiny "doorbell" wake when
+ * the running version's `release_id` advances.
+ */
+
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { dirname, join } from "node:path";
+
+import type { Command } from "commander";
+
+import { getWorkspaceDir } from "../../util/platform.js";
+import { registerCommand } from "../lib/register-command.js";
+import { log } from "../logger.js";
+
+// ── Types ────────────────────────────────────────────────────────────
+
+/**
+ * Subset of the GitHub Releases API payload we care about. Any field the
+ * upstream API may add stays compatible because we read by name and ignore
+ * the rest.
+ */
+interface GitHubRelease {
+  tag_name: string;
+  name: string | null;
+  body: string | null;
+  published_at: string | null;
+  html_url: string;
+  draft: boolean;
+  prerelease: boolean;
+}
+
+interface CacheStore {
+  fetchedAt: string;
+  releases: GitHubRelease[];
+}
+
+// ── Config ───────────────────────────────────────────────────────────
+
+const REPO = "vellum-ai/vellum-assistant";
+const LIST_TTL_MS = 60 * 60 * 1000;
+const DEFAULT_LIST_LIMIT = 30;
+const MAX_LIST_LIMIT = 100;
+const FETCH_TIMEOUT_MS = 10_000;
+const USER_AGENT = "vellum-assistant-cli";
+
+// ── Cache plumbing ───────────────────────────────────────────────────
+
+function getCachePath(): string {
+  return join(getWorkspaceDir(), "data", "changelog-cache.json");
+}
+
+function readCache(): CacheStore | null {
+  try {
+    const path = getCachePath();
+    if (!existsSync(path)) return null;
+    const parsed = JSON.parse(
+      readFileSync(path, "utf-8"),
+    ) as Partial<CacheStore>;
+    if (
+      typeof parsed.fetchedAt !== "string" ||
+      !Array.isArray(parsed.releases)
+    ) {
+      return null;
+    }
+    return parsed as CacheStore;
+  } catch {
+    return null;
+  }
+}
+
+function writeCache(store: CacheStore): void {
+  try {
+    const path = getCachePath();
+    mkdirSync(dirname(path), { recursive: true });
+    writeFileSync(path, JSON.stringify(store, null, 2));
+  } catch (err) {
+    // Cache is best-effort; never fail the command because of a write error.
+    log.warn(`Failed to write changelog cache: ${(err as Error).message}`);
+  }
+}
+
+function isStale(cache: CacheStore): boolean {
+  const fetchedAt = Date.parse(cache.fetchedAt);
+  if (Number.isNaN(fetchedAt)) return true;
+  return Date.now() - fetchedAt > LIST_TTL_MS;
+}
+
+// ── Network ──────────────────────────────────────────────────────────
+
+async function githubFetch(url: string): Promise<Response> {
+  return fetch(url, {
+    headers: {
+      Accept: "application/vnd.github+json",
+      "User-Agent": USER_AGENT,
+      "X-GitHub-Api-Version": "2022-11-28",
+    },
+    signal: AbortSignal.timeout(FETCH_TIMEOUT_MS),
+  });
+}
+
+function describeGithubError(status: number, body: string): string {
+  if (status === 403 || status === 429) {
+    return "GitHub API rate limit reached. Wait a few minutes and retry, or pass --no-cache after the limit resets.";
+  }
+  if (status === 404) {
+    return "Release not found on GitHub.";
+  }
+  return `GitHub API error ${status}: ${body || "(no body)"}`;
+}
+
+async function fetchReleaseList(limit: number): Promise<GitHubRelease[]> {
+  const url = `https://api.github.com/repos/${REPO}/releases?per_page=${limit}`;
+  const res = await githubFetch(url);
+  if (!res.ok) {
+    const text = await res.text().catch(() => "");
+    throw new Error(describeGithubError(res.status, text));
+  }
+  return (await res.json()) as GitHubRelease[];
+}
+
+async function fetchReleaseByTag(tag: string): Promise<GitHubRelease | null> {
+  const url = `https://api.github.com/repos/${REPO}/releases/tags/${encodeURIComponent(tag)}`;
+  const res = await githubFetch(url);
+  if (res.status === 404) return null;
+  if (!res.ok) {
+    const text = await res.text().catch(() => "");
+    throw new Error(describeGithubError(res.status, text));
+  }
+  return (await res.json()) as GitHubRelease;
+}
+
+// ── Cache-aware loaders ──────────────────────────────────────────────
+
+interface LoadOpts {
+  noCache: boolean;
+  limit: number;
+}
+
+async function loadReleases(opts: LoadOpts): Promise<GitHubRelease[]> {
+  if (!opts.noCache) {
+    const cache = readCache();
+    if (cache && !isStale(cache) && cache.releases.length >= opts.limit) {
+      return cache.releases.slice(0, opts.limit);
+    }
+  }
+  const releases = await fetchReleaseList(opts.limit);
+  writeCache({ fetchedAt: new Date().toISOString(), releases });
+  return releases;
+}
+
+async function loadReleaseByTag(
+  tag: string,
+  opts: { noCache: boolean },
+): Promise<GitHubRelease | null> {
+  if (!opts.noCache) {
+    const cache = readCache();
+    const hit = cache?.releases.find((r) => r.tag_name === tag);
+    if (hit) return hit;
+  }
+  return fetchReleaseByTag(tag);
+}
+
+// ── Filtering / version utilities ────────────────────────────────────
+
+/** Drop drafts and pre-releases; release notes only show shipped versions. */
+function stableReleases(releases: GitHubRelease[]): GitHubRelease[] {
+  return releases.filter((r) => !r.draft && !r.prerelease);
+}
+
+/** Accept `0.8.0` or `v0.8.0` from the user; canonicalize to the tag form. */
+function normalizeTag(input: string): string {
+  return input.startsWith("v") ? input : `v${input}`;
+}
+
+/**
+ * Lightweight semver compare. Only handles `vMAJOR.MINOR.PATCH` (the format
+ * the release pipeline emits in `.github/workflows/release.yml`). Prerelease
+ * suffixes are ignored because we filter prereleases out before comparing.
+ */
+function compareTags(a: string, b: string): number {
+  const parse = (tag: string): number[] =>
+    tag
+      .replace(/^v/, "")
+      .split(".")
+      .map((part) => Number.parseInt(part, 10))
+      .map((n) => (Number.isNaN(n) ? 0 : n));
+  const pa = parse(a);
+  const pb = parse(b);
+  const len = Math.max(pa.length, pb.length);
+  for (let i = 0; i < len; i++) {
+    const diff = (pa[i] ?? 0) - (pb[i] ?? 0);
+    if (diff !== 0) return diff;
+  }
+  return 0;
+}
+
+// ── Rendering ────────────────────────────────────────────────────────
+
+function renderRelease(r: GitHubRelease): string {
+  const heading = r.name && r.name.trim().length > 0 ? r.name : r.tag_name;
+  const date = r.published_at ? r.published_at.slice(0, 10) : "";
+  const lines = [`# ${heading}`];
+  if (date) lines.push(`Published: ${date}`);
+  if (r.html_url) lines.push(r.html_url);
+  lines.push("");
+  if (r.body && r.body.trim().length > 0) lines.push(r.body.trim());
+  else lines.push("(no release body)");
+  return lines.join("\n");
+}
+
+function renderList(releases: GitHubRelease[]): string {
+  if (releases.length === 0) return "No releases found.";
+  const tagWidth = Math.max(...releases.map((r) => r.tag_name.length), 6);
+  return releases
+    .map((r) => {
+      const date = r.published_at ? r.published_at.slice(0, 10) : "          ";
+      const name = r.name && r.name.trim().length > 0 ? r.name : r.tag_name;
+      return `${r.tag_name.padEnd(tagWidth)}  ${date}  ${name}`;
+    })
+    .join("\n");
+}
+
+// ── Action helpers ───────────────────────────────────────────────────
+
+function parseLimit(input: string | undefined, fallback: number): number {
+  const parsed = Number.parseInt(String(input ?? ""), 10);
+  if (Number.isNaN(parsed) || parsed <= 0) return fallback;
+  return Math.min(parsed, MAX_LIST_LIMIT);
+}
+
+function emit(output: string): void {
+  process.stdout.write(output.endsWith("\n") ? output : output + "\n");
+}
+
+function emitError(message: string): never {
+  process.stderr.write(`${message}\n`);
+  process.exit(1);
+}
+
+interface CommonOpts {
+  cache?: boolean;
+  json?: boolean;
+  limit?: string;
+}
+
+interface DefaultOpts extends CommonOpts {
+  since?: string;
+}
+
+async function runDefault(opts: DefaultOpts): Promise<void> {
+  const noCache = opts.cache === false;
+  const useJson = opts.json === true;
+  const limit = parseLimit(opts.limit, DEFAULT_LIST_LIMIT);
+
+  if (opts.since) {
+    const floor = normalizeTag(opts.since);
+    const all = stableReleases(await loadReleases({ noCache, limit }));
+    const newer = all
+      .filter((r) => compareTags(r.tag_name, floor) > 0)
+      .sort((a, b) => compareTags(b.tag_name, a.tag_name));
+    if (newer.length === 0) {
+      if (useJson) emit(JSON.stringify({ releases: [] }));
+      else log.info(`No releases newer than ${floor}.`);
+      return;
+    }
+    if (useJson) {
+      emit(JSON.stringify({ releases: newer }, null, 2));
+    } else {
+      emit(newer.map(renderRelease).join("\n\n---\n\n"));
+    }
+    return;
+  }
+
+  const releases = stableReleases(await loadReleases({ noCache, limit }));
+  if (releases.length === 0) {
+    emitError("No releases found.");
+  }
+  const latest = releases[0];
+  if (useJson) emit(JSON.stringify(latest, null, 2));
+  else emit(renderRelease(latest));
+}
+
+async function runShow(version: string, opts: CommonOpts): Promise<void> {
+  const noCache = opts.cache === false;
+  const useJson = opts.json === true;
+  const tag = normalizeTag(version);
+  const release = await loadReleaseByTag(tag, { noCache });
+  if (!release) {
+    emitError(`No release found for tag ${tag}.`);
+  }
+  if (useJson) emit(JSON.stringify(release, null, 2));
+  else emit(renderRelease(release));
+}
+
+async function runList(opts: CommonOpts): Promise<void> {
+  const noCache = opts.cache === false;
+  const useJson = opts.json === true;
+  const limit = parseLimit(opts.limit, DEFAULT_LIST_LIMIT);
+  const releases = stableReleases(await loadReleases({ noCache, limit })).slice(
+    0,
+    limit,
+  );
+  if (useJson) {
+    emit(JSON.stringify({ releases }, null, 2));
+    return;
+  }
+  emit(renderList(releases));
+}
+
+async function withErrorHandling(fn: () => Promise<void>): Promise<void> {
+  try {
+    await fn();
+  } catch (err) {
+    emitError((err as Error).message);
+  }
+}
+
+// ── Registration ─────────────────────────────────────────────────────
+
+export function registerChangelogCommand(program: Command): void {
+  registerCommand(program, {
+    name: "changelog",
+    transport: "local",
+    description: "Show release notes (fetched on demand from GitHub Releases)",
+    build: (cmd) => {
+      cmd.addHelpText(
+        "after",
+        `
+Release notes are fetched on demand from the public GitHub Releases of
+${REPO}. Results are cached locally for ${LIST_TTL_MS / 60_000} minutes;
+pass --no-cache to bypass the cache. Specific tags are cached without TTL
+because release tags are immutable.
+
+Examples:
+  $ assistant changelog                       Show the latest release
+  $ assistant changelog --since 0.7.0         Show every release since 0.7.0
+  $ assistant changelog show 0.8.0            Show a specific release
+  $ assistant changelog list                  List recent release tags
+  $ assistant changelog --json                JSON output for tooling`,
+      );
+
+      // Shared flags live on the parent so they're inherited by subcommands
+      // via `command.optsWithGlobals()`. Declaring the same flag on both
+      // parent and subcommand confuses commander's option resolution — the
+      // parent wins and the subcommand's copy never gets populated.
+      cmd
+        .option(
+          "--since <version>",
+          "Show notes for every stable release newer than this version (e.g. 0.7.0)",
+        )
+        .option("--no-cache", "Bypass the local cache")
+        .option("--json", "Output structured JSON")
+        .option(
+          "--limit <n>",
+          "Max releases to consider when listing or filtering (1-100)",
+          String(DEFAULT_LIST_LIMIT),
+        )
+        .action(async (opts: DefaultOpts) => {
+          await withErrorHandling(() => runDefault(opts));
+        });
+
+      cmd
+        .command("show <version>")
+        .description("Show release notes for a specific version tag")
+        .action(async (version: string, _opts, command: Command) => {
+          const merged = command.optsWithGlobals() as CommonOpts;
+          await withErrorHandling(() => runShow(version, merged));
+        });
+
+      cmd
+        .command("list")
+        .description("List recent release tags")
+        .action(async (_opts, command: Command) => {
+          const merged = command.optsWithGlobals() as CommonOpts;
+          await withErrorHandling(() => runList(merged));
+        });
+    },
+  });
+}
+
+// ── Test-only exports ────────────────────────────────────────────────
+
+/**
+ * Internal helpers exported for unit tests only. Not part of the public CLI
+ * contract — do not import from outside `cli/commands/__tests__`.
+ */
+export const __testing = {
+  compareTags,
+  normalizeTag,
+  stableReleases,
+  renderRelease,
+  renderList,
+  parseLimit,
+  readCache,
+  writeCache,
+  isStale,
+  loadReleases,
+  loadReleaseByTag,
+  getCachePath,
+};