@vellumai/assistant 0.10.7 → 0.10.8-dev.202607102228.5945895

package/src/cli/commands/db/repair-step-conversation-backfill.ts

@@ -28,11 +28,6 @@ import { Database } from "bun:sqlite";
 import { eq } from "drizzle-orm";
 import { drizzle } from "drizzle-orm/bun-sqlite";
 
-import {
-  conversations,
-  messages,
-} from "../../../persistence/schema/conversations.js";
-import * as schema from "../../../persistence/schema/index.js";
 import { getWorkspaceDir } from "../../../util/platform.js";
 import type { RepairContext, RepairStep, StepResult } from "./repair-steps.js";
 
@@ -86,7 +81,9 @@ interface DiskMessageRecord {
 // ---------------------------------------------------------------------------
 
 function parseEpochMs(isoString: string | undefined): number | null {
-  if (!isoString) return null;
+  if (!isoString) {
+    return null;
+  }
   const ms = new Date(isoString).getTime();
   return Number.isNaN(ms) ? null : ms;
 }
@@ -137,6 +134,13 @@ function buildContentBlocks(record: DiskMessageRecord): unknown[] {
 async function runConversationBackfill(
   ctx: RepairContext,
 ): Promise<StepResult> {
+  // Lazy-imported so the persistence schema (and the drizzle/DB module graph
+  // it pulls) stays out of the CLI's static import graph — this step only
+  // touches it when `db repair` actually runs the backfill.
+  const { conversations, messages } =
+    await import("../../../persistence/schema/conversations.js");
+  const schema = await import("../../../persistence/schema/index.js");
+
   // Open RW so we can insert recovered rows. Mirror the daemon's pragmas
   // for consistent journal/FK behavior.
   let sqlite: Database;
@@ -171,7 +175,9 @@ async function runConversationBackfill(
     let errors = 0;
     const warnings: string[] = [];
     const pushWarning = (line: string): void => {
-      if (warnings.length < WARNING_CAP_TOTAL) warnings.push(line);
+      if (warnings.length < WARNING_CAP_TOTAL) {
+        warnings.push(line);
+      }
     };
 
     const conversationsDir = join(workspaceDir, "conversations");
@@ -188,7 +194,9 @@ async function runConversationBackfill(
         const dirPath = join(conversationsDir, entry);
 
         try {
-          if (!statSync(dirPath).isDirectory()) continue;
+          if (!statSync(dirPath).isDirectory()) {
+            continue;
+          }
         } catch {
           continue;
         }
@@ -233,7 +241,9 @@ async function runConversationBackfill(
             const raw = readFileSync(messagesPath, "utf-8");
             for (const line of raw.split("\n")) {
               const trimmed = line.trim();
-              if (!trimmed) continue;
+              if (!trimmed) {
+                continue;
+              }
               try {
                 messageRecords.push(JSON.parse(trimmed) as DiskMessageRecord);
               } catch {
@@ -261,7 +271,6 @@ async function runConversationBackfill(
                 conversationType: meta.type ?? "standard",
                 originChannel: meta.channel ?? null,
                 source: "user",
-                memoryScopeId: "default",
                 isAutoTitle: 1,
                 totalInputTokens: 0,
                 totalOutputTokens: 0,

package/src/cli/commands/db/repair.ts

@@ -7,7 +7,7 @@
  * `RepairReport` that renders either as plain text or as a single JSON
  * payload (`--json`).
  *
- * Adding a new step is a one-line edit to the `STEPS` array.
+ * Adding a new step is a one-line edit to the `repairSteps` list.
  *
  * Transport: `local`. The whole point of this surface is that it works when
  * the daemon is down, so it opens the DB file directly and never goes
@@ -15,26 +15,34 @@
  */
 
 import { existsSync } from "node:fs";
+import { createRequire } from "node:module";
 
 import type { Command } from "commander";
 
 import { getDbPath } from "../../../util/platform.js";
 import { dim, green, red } from "../../lib/cli-colors.js";
 import { shouldOutputJson, writeOutput } from "../../output.js";
-import { conversationBackfillStep } from "./repair-step-conversation-backfill.js";
 import { integrityCheckStep } from "./repair-step-integrity.js";
 import type { RepairReport, RepairStep, StepResult } from "./repair-steps.js";
 import { formatDurationMs, runRepairSteps } from "./repair-steps.js";
 
+const loadModule = createRequire(import.meta.url);
+
 // ---------------------------------------------------------------------------
 // Step sequence
 // ---------------------------------------------------------------------------
 
 /**
- * Repair steps run in the order listed here. Integrity check runs first so
- * structural damage surfaces before subsequent steps touch the same pages.
+ * Repair steps run in the order listed here; integrity check first so
+ * structural damage surfaces early. Resolved lazily because the backfill
+ * step pulls the DB graph.
  */
-const STEPS: RepairStep[] = [integrityCheckStep, conversationBackfillStep];
+function repairSteps(): RepairStep[] {
+  const { conversationBackfillStep } = loadModule(
+    "./repair-step-conversation-backfill.js",
+  ) as typeof import("./repair-step-conversation-backfill.js");
+  return [integrityCheckStep, conversationBackfillStep];
+}
 
 // ---------------------------------------------------------------------------
 // Rendering
@@ -85,7 +93,9 @@ function renderSummary(report: RepairReport): string {
   let line =
     `\nDone. ${total} step${total === 1 ? "" : "s"} ran: ` +
     `${okCount} ok, ${errorCount} failed`;
-  if (halted) line += "  (sequence halted)";
+  if (halted) {
+    line += "  (sequence halted)";
+  }
   return line + "\n";
 }
 
@@ -120,7 +130,7 @@ export function registerDbRepair(parent: Command): void {
 
       const report = await runRepairSteps(
         { dbPath },
-        STEPS,
+        repairSteps(),
         isJson
           ? {}
           : {
@@ -136,6 +146,8 @@ export function registerDbRepair(parent: Command): void {
       }
 
       // Exit non-zero if any step failed — makes the command scriptable.
-      if (report.errorCount > 0) process.exit(1);
+      if (report.errorCount > 0) {
+        process.exit(1);
+      }
     });
 }

package/src/cli/commands/default-action.ts

@@ -1,8 +1,5 @@
 import type { Command } from "commander";
 
-import { startCli } from "../../cli.js";
-import { shouldAutoStartDaemon } from "../../daemon/connection-policy.js";
-import { ensureDaemonRunning } from "../../daemon/daemon-control.js";
 import {
   findClosestCommand,
   formatUnknownCommandMessage,
@@ -30,6 +27,14 @@ export function registerDefaultAction(program: Command): void {
       return;
     }
 
+    // Deferred: only the bare `assistant` action needs these graphs.
+    const [{ startCli }, { shouldAutoStartDaemon }, { ensureDaemonRunning }] =
+      await Promise.all([
+        import("../../cli.js"),
+        import("../../daemon/connection-policy.js"),
+        import("../../daemon/daemon-control.js"),
+      ]);
+
     if (shouldAutoStartDaemon()) {
       await ensureDaemonRunning();
     }

package/src/cli/commands/domain.ts

@@ -1,13 +1,28 @@
+import { createRequire } from "node:module";
+
 import type { Command } from "commander";
 
-import { getApexDomain, getAssistantDomain } from "../../config/env.js";
 import { cliIpcCall, exitFromIpcResult } from "../../ipc/cli-client.js";
 import { registerCommand } from "../lib/register-command.js";
 import { getCliLogger } from "../logger.js";
 import { shouldOutputJson, writeOutput } from "../output.js";
 
+const loadModule = createRequire(import.meta.url);
+
 const log = getCliLogger("domain");
 
+/**
+ * Loaded lazily because config/env pulls the full config-loader graph. Sync
+ * require rather than import(): commander invokes help-text callbacks
+ * synchronously, so there is no place to await a module load.
+ */
+function envDomains(): { apexDomain: string; baseDomain: string } {
+  const { getApexDomain, getAssistantDomain } = loadModule(
+    "../../config/env.js",
+  ) as typeof import("../../config/env.js");
+  return { apexDomain: getApexDomain(), baseDomain: getAssistantDomain() };
+}
+
 function handleDomainIpcError(
   r: { ok: false; error?: string; statusCode?: number },
   cmd: Command,
@@ -21,19 +36,17 @@ function handleDomainIpcError(
 }
 
 export function registerDomainCommand(program: Command): void {
-  const apexDomain = getApexDomain();
-  const baseDomain = getAssistantDomain();
   registerCommand(program, {
     name: "domain",
     transport: "ipc",
-    description: `Register and manage this assistant's custom subdomain on ${baseDomain}`,
+    description: "Register and manage this assistant's custom subdomain",
     build: (domain) => {
       domain.option("--json", "Machine-readable compact JSON output");
 
       domain.addHelpText(
         "after",
-        `
-Each assistant can register its own subdomain (e.g. velly.${baseDomain})
+        () => `
+Each assistant can register its own subdomain (e.g. velly.${envDomains().baseDomain})
 for email and web presence. DNS managed by the Vellum platform.
 
 Examples:
@@ -49,12 +62,10 @@ Examples:
           "--email-username <username>",
           "Also register an email address (e.g. --email-username hello → hello@<subdomain>.domain)",
         )
-        .description(
-          `Register a custom subdomain on ${baseDomain} for this assistant`,
-        )
-        .addHelpText(
-          "after",
-          `
+        .description("Register a custom subdomain for this assistant")
+        .addHelpText("after", () => {
+          const { baseDomain } = envDomains();
+          return `
 Arguments:
   subdomain   The subdomain to register (e.g. "velly" → velly.${baseDomain}).
               If omitted, the platform derives it from the assistant's name.
@@ -77,8 +88,8 @@ Examples:
   ✓ Registered my-assistant.${baseDomain}
 
   $ assistant domain register velly --json
-  {"domain":"velly.${baseDomain}","id":"..."}`,
-        )
+  {"domain":"velly.${baseDomain}","id":"..."}`;
+        })
         .action(
           async (
             subdomain: string | undefined,
@@ -102,13 +113,15 @@ Examples:
               email_error?: { detail: string; code: string };
             }>("domain_register", { body });
 
-            if (!r.ok)
+            if (!r.ok) {
               return handleDomainIpcError(
                 { ok: false, error: r.error, statusCode: r.statusCode },
                 cmd,
               );
+            }
 
             const data = r.result!;
+            const { apexDomain } = envDomains();
             const registeredSubdomain =
               data.subdomain ??
               data.domain?.replace(`.${apexDomain}`, "") ??
@@ -145,7 +158,7 @@ Examples:
         )
         .addHelpText(
           "after",
-          `
+          () => `
 Arguments:
   subdomain   The subdomain to check (e.g. "velly").
 
@@ -154,7 +167,7 @@ status from the email provider.
 
 Examples:
   $ assistant domain status velly
-  Domain:       velly.${baseDomain}
+  Domain:       velly.${envDomains().baseDomain}
   Verification: verified
                 DNS records have been verified. Your domain is ready to send and receive email.
   Created:      2026-04-15
@@ -173,17 +186,16 @@ Examples:
             }[];
           }>("domain_status");
 
-          if (!r.ok)
+          if (!r.ok) {
             return handleDomainIpcError(
               { ok: false, error: r.error, statusCode: r.statusCode },
               cmd,
             );
+          }
 
           const data = r.result!;
           const domains = data.results ?? [];
-          const d = domains.find(
-            (entry) => entry.subdomain === subdomain,
-          );
+          const d = domains.find((entry) => entry.subdomain === subdomain);
 
           if (!d) {
             if (shouldOutputJson(cmd)) {
@@ -214,7 +226,9 @@ Examples:
           } else {
             const displayDomain =
               d.domain ??
-              (d.subdomain ? `${d.subdomain}.${apexDomain}` : "unknown");
+              (d.subdomain
+                ? `${d.subdomain}.${envDomains().apexDomain}`
+                : "unknown");
             const createdRaw = d.created_at ?? d.created;
             const createdDate = createdRaw
               ? createdRaw.split("T")[0]

package/src/cli/commands/email.ts

@@ -4,11 +4,11 @@ import {
   readFileSync,
   writeFileSync,
 } from "node:fs";
+import { createRequire } from "node:module";
 import { basename, join } from "node:path";
 
 import type { Command } from "commander";
 
-import { getAssistantDomain } from "../../config/env.js";
 import {
   cliIpcCall,
   cliIpcCallStream,
@@ -19,6 +19,8 @@ import { registerCommand } from "../lib/register-command.js";
 import { getCliLogger } from "../logger.js";
 import { shouldOutputJson, writeOutput } from "../output.js";
 
+const loadModule = createRequire(import.meta.url);
+
 const log = getCliLogger("email");
 
 /**
@@ -49,12 +51,24 @@ function handleEmailIpcError(
   exitFromIpcResult(r, cmd);
 }
 
+/**
+ * Loaded lazily because config/env pulls the full config-loader graph. Sync
+ * require rather than import(): commander invokes help-text callbacks
+ * synchronously, so there is no place to await a module load.
+ */
+function assistantDomain(): string {
+  const { getAssistantDomain } = loadModule(
+    "../../config/env.js",
+  ) as typeof import("../../config/env.js");
+  return getAssistantDomain();
+}
+
 export function registerEmailCommand(program: Command): void {
-  const domain = getAssistantDomain();
   registerCommand(program, {
     name: "email",
     transport: "ipc",
-    description: `Get your own email address (@${domain}) — register, send, receive, and manage email natively`,
+    description:
+      "Get your own email address — register, send, receive, and manage email natively",
     build: (email) => {
       // Keep the --json option at the email namespace level
       email.option("--json", "Machine-readable compact JSON output");
@@ -78,12 +92,12 @@ Examples:
 
       email
         .command("register <username>")
-        .description(`Register an @${domain} email address for this assistant`)
+        .description("Register an email address for this assistant")
         .addHelpText(
           "after",
-          `
+          () => `
 Arguments:
-  username   The local part of the email address (e.g. "mybot" → mybot@${domain})
+  username   The local part of the email address (e.g. "mybot" → mybot@${assistantDomain()})
 
 Registers a new email address on the Vellum platform for the current
 assistant. Each assistant can have one email address. The address is
@@ -91,10 +105,10 @@ immediately active for receiving inbound email.
 
 Examples:
   $ assistant email register mybot
-  ✓ Registered mybot@${domain}
+  ✓ Registered mybot@${assistantDomain()}
 
   $ assistant email register support --json
-  {"address":"support@${domain}","id":"...","created_at":"..."}`,
+  {"address":"support@${assistantDomain()}","id":"...","created_at":"..."}`,
         )
         .action(async (username: string, _opts: unknown, cmd: Command) => {
           const r = await cliIpcCall<{
@@ -102,11 +116,12 @@ Examples:
             address: string;
             created_at: string;
           }>("email_register", { body: { username } });
-          if (!r.ok)
+          if (!r.ok) {
             return handleEmailIpcError(
               { ok: false, error: r.error, statusCode: r.statusCode },
               cmd,
             );
+          }
           if (shouldOutputJson(cmd)) {
             writeOutput(cmd, r.result);
           } else {
@@ -120,7 +135,7 @@ Examples:
         .option("--confirm", "Skip confirmation prompt")
         .addHelpText(
           "after",
-          `
+          () => `
 Removes the email address currently registered for this assistant.
 The address is deactivated immediately — inbound email will no longer
 be delivered. The username enters a cooldown period and is not
@@ -128,14 +143,14 @@ immediately available for reuse.
 
 Examples:
   $ assistant email unregister
-  Remove mybot@${domain}? (y/N) y
-  ✓ Unregistered mybot@${domain}
+  Remove mybot@${assistantDomain()}? (y/N) y
+  ✓ Unregistered mybot@${assistantDomain()}
 
   $ assistant email unregister --confirm
-  ✓ Unregistered mybot@${domain}
+  ✓ Unregistered mybot@${assistantDomain()}
 
   $ assistant email unregister --json
-  {"unregistered":"mybot@${domain}"}`,
+  {"unregistered":"mybot@${assistantDomain()}"}`,
         )
         .action(async (_opts: { confirm?: boolean }, cmd: Command) => {
           if (!_opts.confirm && !shouldOutputJson(cmd)) {
@@ -162,11 +177,12 @@ Examples:
             "email_unregister",
             {},
           );
-          if (!r.ok)
+          if (!r.ok) {
             return handleEmailIpcError(
               { ok: false, error: r.error, statusCode: r.statusCode },
               cmd,
             );
+          }
           if (shouldOutputJson(cmd)) {
             writeOutput(cmd, r.result);
           } else {
@@ -179,13 +195,13 @@ Examples:
         .description("Show email address info and usage for this assistant")
         .addHelpText(
           "after",
-          `
+          () => `
 Shows the email address registered for this assistant along with
 current usage and quota information from the platform.
 
 Examples:
   $ assistant email status
-  Address:  hi@mybot.${domain}
+  Address:  hi@mybot.${assistantDomain()}
   Status:   active
   Since:    2026-04-15
   Sent:     12 / 100 (daily)
@@ -193,7 +209,7 @@ Examples:
   Monthly:  42 sent, 18 received
 
   $ assistant email status --json
-  {"address":"hi@mybot.${domain}","status":"active","created_at":"2026-04-15T...","usage":{...}}`,
+  {"address":"hi@mybot.${assistantDomain()}","status":"active","created_at":"2026-04-15T...","usage":{...}}`,
         )
         .action(async (_opts: unknown, cmd: Command) => {
           const r = await cliIpcCall<{
@@ -208,11 +224,12 @@ Examples:
               received_this_month: number;
             };
           }>("email_status", {});
-          if (!r.ok)
+          if (!r.ok) {
             return handleEmailIpcError(
               { ok: false, error: r.error, statusCode: r.statusCode },
               cmd,
             );
+          }
           const statusData = r.result!;
           if (shouldOutputJson(cmd)) {
             writeOutput(cmd, statusData);
@@ -287,11 +304,12 @@ Examples:
               }[];
               count: number;
             }>("email_list", { queryParams: params });
-            if (!r.ok)
+            if (!r.ok) {
               return handleEmailIpcError(
                 { ok: false, error: r.error, statusCode: r.statusCode },
                 cmd,
               );
+            }
             const data = r.result!;
             if (shouldOutputJson(cmd)) {
               writeOutput(cmd, data);
@@ -327,7 +345,7 @@ Examples:
         .option("-o, --output <path>", "Write to file instead of stdout")
         .addHelpText(
           "after",
-          `
+          () => `
 Arguments:
   message-id   Email message ID (from \`assistant email list --json\`)
 
@@ -338,7 +356,7 @@ or --format json for the full message object.
 Examples:
   $ assistant email download msg_abc123
   From:    user@example.com
-  To:      mybot@${domain}
+  To:      mybot@${assistantDomain()}
   Subject: Hello
   Date:    2026-04-05 12:00:00
 
@@ -371,11 +389,12 @@ Examples:
               references: string[];
               created_at: string;
             }>("email_download", { queryParams: { messageId } });
-            if (!r.ok)
+            if (!r.ok) {
               return handleEmailIpcError(
                 { ok: false, error: r.error, statusCode: r.statusCode },
                 cmd,
               );
+            }
             const msg = r.result!;
 
             const fmt = opts.format ?? "text";
@@ -546,21 +565,32 @@ Examples:
             }
 
             const params: Record<string, unknown> = { to, text };
-            if (opts.subject) params.subject = opts.subject;
-            if (html) params.html = html;
-            if (opts.cc && opts.cc.length > 0) params.cc = opts.cc;
-            if (opts.bcc && opts.bcc.length > 0) params.bcc = opts.bcc;
-            if (opts.replyTo) params.reply_to = opts.replyTo;
+            if (opts.subject) {
+              params.subject = opts.subject;
+            }
+            if (html) {
+              params.html = html;
+            }
+            if (opts.cc && opts.cc.length > 0) {
+              params.cc = opts.cc;
+            }
+            if (opts.bcc && opts.bcc.length > 0) {
+              params.bcc = opts.bcc;
+            }
+            if (opts.replyTo) {
+              params.reply_to = opts.replyTo;
+            }
 
             const r = await cliIpcCall<{ delivery_id: string; status: string }>(
               "email_send",
               { body: params },
             );
-            if (!r.ok)
+            if (!r.ok) {
               return handleEmailIpcError(
                 { ok: false, error: r.error, statusCode: r.statusCode },
                 cmd,
               );
+            }
             const data = r.result!;
             if (shouldOutputJson(cmd)) {
               writeOutput(cmd, data);
@@ -617,11 +647,12 @@ $ assistant email attachment msg_abc1 --list --json`,
                 "email_attachment_list",
                 { queryParams: { messageId } },
               );
-              if (!r.ok)
+              if (!r.ok) {
                 return handleEmailIpcError(
                   { ok: false, error: r.error, statusCode: r.statusCode },
                   cmd,
                 );
+              }
               const data = r.result!;
               if (shouldOutputJson(cmd)) {
                 writeOutput(cmd, data);
@@ -668,7 +699,7 @@ $ assistant email attachment msg_abc1 --list --json`,
                   "email_attachment_list",
                   { queryParams: { messageId } },
                 );
-                if (!listR.ok)
+                if (!listR.ok) {
                   return handleEmailIpcError(
                     {
                       ok: false,
@@ -677,6 +708,7 @@ $ assistant email attachment msg_abc1 --list --json`,
                     },
                     cmd,
                   );
+                }
                 const attachments = listR.result!.results ?? [];
                 if (attachments.length === 0) {
                   log.error("No attachments for this message.");
@@ -717,7 +749,7 @@ $ assistant email attachment msg_abc1 --list --json`,
                   "email_attachment_list",
                   { queryParams: { messageId } },
                 );
-                if (!listR.ok)
+                if (!listR.ok) {
                   return handleEmailIpcError(
                     {
                       ok: false,
@@ -726,6 +758,7 @@ $ assistant email attachment msg_abc1 --list --json`,
                     },
                     cmd,
                   );
+                }
                 const meta = (listR.result!.results ?? []).find(
                   (a) => a.id === attachmentId,
                 );
@@ -772,8 +805,12 @@ interface AttachmentMeta {
 }
 
 function formatBytes(bytes: number): string {
-  if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)} KB`;
+  if (bytes < 1024) {
+    return `${bytes} B`;
+  }
+  if (bytes < 1024 * 1024) {
+    return `${(bytes / 1024).toFixed(1)} KB`;
+  }
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 
@@ -790,14 +827,18 @@ async function streamDownloadAttachment(
   const r = await cliIpcCallStream("email_attachment_get", {
     queryParams: { messageId, attachmentId },
   });
-  if (!r.ok) throw new Error(r.error ?? "Stream failed");
+  if (!r.ok) {
+    throw new Error(r.error ?? "Stream failed");
+  }
 
   const fileStream = createWriteStream(dest);
   const reader = r.body.getReader();
   try {
     while (true) {
       const { done, value } = await reader.read();
-      if (done) break;
+      if (done) {
+        break;
+      }
       await new Promise<void>((resolve, reject) =>
         fileStream.write(value, (err) => (err ? reject(err) : resolve())),
       );