@vellumai/assistant 0.10.2 → 0.10.3-dev.202606252046.9075fd5

package/src/cli/commands/contacts.ts

@@ -8,13 +8,16 @@ import { shouldOutputJson, writeOutput } from "../output.js";
 // IPC response shapes
 // ---------------------------------------------------------------------------
 
+// ACL fields (role, status, policy) are gateway-owned and not hydrated by the
+// daemon-native filtered reads (`--query`/`--channel-address`/`--channel-type`),
+// so they are optional here. The unfiltered default read carries them.
 interface ContactChannel {
   id: string;
   contactId: string;
   type: string;
   address: string;
-  status: string;
-  policy: string;
+  status?: string;
+  policy?: string;
   isPrimary?: boolean;
   revokedReason?: string | null;
   blockedReason?: string | null;
@@ -23,7 +26,7 @@ interface ContactChannel {
 interface ContactWithChannels {
   id: string;
   displayName: string;
-  role: string;
+  role?: string;
   contactType: string;
   notes?: string;
   principalId?: string;
@@ -56,7 +59,7 @@ function formatContactTable(contacts: ContactWithChannels[]): string {
   const rows = contacts.map((c) => [
     c.id,
     c.displayName,
-    `${c.role}/${c.contactType}`,
+    `${c.role ?? "—"}/${c.contactType}`,
     String(c.channels.length),
   ]);
 
@@ -81,8 +84,8 @@ function formatChannelTable(channels: ContactChannel[]): string {
   const rows = channels.map((ch) => {
     const flags = [
       ch.isPrimary ? "primary" : null,
-      ch.status !== "active" ? ch.status : null,
-      ch.policy !== "allow" ? ch.policy : null,
+      ch.status && ch.status !== "active" ? ch.status : null,
+      ch.policy && ch.policy !== "allow" ? ch.policy : null,
     ]
       .filter(Boolean)
       .join(", ");
@@ -124,7 +127,7 @@ function formatContactDetail(
   const lines: string[] = [];
   lines.push(`ID:           ${c.id}`);
   lines.push(`Display Name: ${c.displayName}`);
-  lines.push(`Role:         ${c.role}`);
+  if (c.role) lines.push(`Role:         ${c.role}`);
   lines.push(`Type:         ${c.contactType}`);
   if (c.notes) lines.push(`Notes:        ${c.notes}`);
   if (c.principalId) lines.push(`Principal:    ${c.principalId}`);

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

@@ -456,6 +456,9 @@ async function initSchema(): Promise<void> {
   clearStoredDb("memory");
 }
 
+// Every test here runs initSchema(), which replays the full production
+// migration suite, plus at least one repair pass. On a loaded machine that
+// comfortably exceeds bun's 5s per-test default, so each gets a 30s ceiling.
 describe("assistant db repair — conversation-backfill step", () => {
   test("backfills a disk-only conversation into SQLite", async () => {
     await initSchema();
@@ -499,7 +502,7 @@ describe("assistant db repair — conversation-backfill step", () => {
     } finally {
       verify.close();
     }
-  });
+  }, 30_000);
 
   test("skips conversations already present (idempotent)", async () => {
     await initSchema();
@@ -516,8 +519,6 @@ describe("assistant db repair — conversation-backfill step", () => {
     expect(parsed.steps[1].result.data.recovered).toBe(0);
     expect(parsed.steps[1].result.data.skipped).toBe(1);
     expect(parsed.steps[1].result.status).toBe("ok");
-    // Two full repair passes (each walks the DB via integrity-check) exceed
-    // bun's 5s per-test default, so raise the ceiling to keep CI stable.
   }, 30_000);
 
   test("reports nothing-to-backfill on an empty conversations dir", async () => {
@@ -529,7 +530,7 @@ describe("assistant db repair — conversation-backfill step", () => {
     expect(parsed.steps[1].result.status).toBe("ok");
     expect(parsed.steps[1].result.summary).toContain("nothing to backfill");
     expect(parsed.steps[1].result.data.recovered).toBe(0);
-  });
+  }, 30_000);
 
   test("surfaces warnings for malformed meta.json without erroring the step", async () => {
     await initSchema();
@@ -547,5 +548,5 @@ describe("assistant db repair — conversation-backfill step", () => {
         w.includes("malformed meta.json"),
       ),
     ).toBe(true);
-  });
+  }, 30_000);
 });

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

@@ -31,6 +31,7 @@ import type { Command } from "commander";
 import { getLogsDbPath } from "../../../util/logs-db-path.js";
 import { getMemoryDbPath } from "../../../util/memory-db-path.js";
 import { getDbPath } from "../../../util/platform.js";
+import { getTelemetryDbPath } from "../../../util/telemetry-db-path.js";
 import { red } from "../../lib/cli-colors.js";
 import { shouldOutputJson, writeOutput } from "../../output.js";
 import {
@@ -99,6 +100,15 @@ interface StatusReport {
    */
   memoryFile?: FileFacts;
   memoryDb?: DbFacts | null;
+  /**
+   * The dedicated telemetry database (`assistant-telemetry.db`). Like
+   * `logsFile` and `memoryFile`, it is omitted from the report until the file
+   * exists on disk — it is created the first time the daemon opens its
+   * connection, so a fresh install that has never run the daemon won't have
+   * one.
+   */
+  telemetryFile?: FileFacts;
+  telemetryDb?: DbFacts | null;
 }
 
 interface MigrationSummary {
@@ -327,6 +337,13 @@ function renderHuman(report: StatusReport): string {
     out += renderDbBlock(report.memoryFile, report.memoryDb ?? null);
   }
 
+  // The secondary telemetry file is likewise only reported once it exists.
+  if (report.telemetryFile?.exists) {
+    out += "\n";
+    out += "Telemetry database (telemetry event tables)\n";
+    out += renderDbBlock(report.telemetryFile, report.telemetryDb ?? null);
+  }
+
   // Cross-database migration summary — the checkpoint ledger lives only in
   // the main DB, so this is a single block at the end covering all three.
   out += renderSummary(report.migration);
@@ -409,7 +426,13 @@ function readMigrationSummary(mainDbPath: string): MigrationSummary | null {
       // No table yet.
     }
 
-    return { latestKey, latestUpdatedAtMs, appliedSteps, appliedRegistry, crashed };
+    return {
+      latestKey,
+      latestUpdatedAtMs,
+      appliedSteps,
+      appliedRegistry,
+      crashed,
+    };
   } finally {
     db.close();
   }
@@ -519,6 +542,17 @@ export function registerDbStatus(parent: Command): void {
         }
       }
 
+      // Same best-effort probe for the dedicated telemetry DB.
+      const telemetryFile = readFileFacts(getTelemetryDbPath());
+      let telemetryDb: DbFacts | null = null;
+      if (telemetryFile.exists) {
+        try {
+          telemetryDb = readDbFacts(telemetryFile.path);
+        } catch {
+          telemetryDb = null;
+        }
+      }
+
       // Cross-database migration summary from the main DB's checkpoint ledger.
       let migration: MigrationSummary | null = null;
       try {
@@ -535,6 +569,8 @@ export function registerDbStatus(parent: Command): void {
         logsDb,
         memoryFile,
         memoryDb,
+        telemetryFile,
+        telemetryDb,
       };
       if (shouldOutputJson(this)) {
         writeOutput(this, report);