@vellumai/assistant 0.6.4 → 0.6.6

package/src/cli/commands/memory.ts

@@ -2,6 +2,7 @@ import type { Command } from "commander";
 
 import {
   cleanupShortSegments,
+  compactLongMemoryNodes,
   findReextractTarget,
   findReextractTargets,
   getMemorySystemStatus,
@@ -35,7 +36,7 @@ Key concepts:
 Examples:
   $ assistant memory status
   $ assistant memory query "What is the project deadline?"
-  $ assistant memory backfill`
+  $ assistant memory backfill`,
   );
 
   memory
@@ -57,7 +58,7 @@ Fields shown:
   jobs               Status of background jobs (backfill, cleanup, rebuild-index)
 
 Examples:
-  $ assistant memory status`
+  $ assistant memory status`,
     )
     .action(async () => {
       initializeDb();
@@ -97,7 +98,7 @@ useful after bulk imports or if the incremental state has become inconsistent.
 
 Examples:
   $ assistant memory backfill
-  $ assistant memory backfill --force`
+  $ assistant memory backfill --force`,
     )
     .action((opts: { force?: boolean }) => {
       initializeDb();
@@ -121,20 +122,20 @@ existing short segments that were stored before the filter was added.
 
 Examples:
   $ assistant memory cleanup-segments
-  $ assistant memory cleanup-segments --dry-run`
+  $ assistant memory cleanup-segments --dry-run`,
     )
     .action(async (opts: { dryRun?: boolean }) => {
       initializeDb();
       const result = await cleanupShortSegments({ dryRun: opts.dryRun });
       if (opts.dryRun) {
         log.info(
-          `Dry run: ${result.dryRunCount} short segment(s) would be removed.`
+          `Dry run: ${result.dryRunCount} short segment(s) would be removed.`,
         );
       } else {
         log.info(`Removed ${result.removed} short segment(s).`);
         if (result.failed > 0) {
           log.warn(
-            `${result.failed} segment(s) skipped — Qdrant deletion failed. Re-run when Qdrant is available.`
+            `${result.failed} segment(s) skipped — Qdrant deletion failed. Re-run when Qdrant is available.`,
           );
         }
       }
@@ -143,7 +144,7 @@ Examples:
   memory
     .command("query <text>")
     .description(
-      "Run a memory recall query and print the injected memory payload"
+      "Run a memory recall query and print the injected memory payload",
     )
     .option("-c, --conversation <id>", "Optional conversation ID")
     .addHelpText(
@@ -164,7 +165,7 @@ context-aware recall. If omitted, the most recent conversation is used.
 Examples:
   $ assistant memory query "What is the project deadline?"
   $ assistant memory query "preferred communication style" --conversation conv_abc123
-  $ assistant memory query "API rate limits"`
+  $ assistant memory query "API rate limits"`,
     )
     .action(async (text: string, opts?: { conversation?: string }) => {
       initializeDb();
@@ -181,8 +182,8 @@ Examples:
         for (const r of result.results) {
           log.info(
             `[${r.type}] (confidence: ${r.confidence.toFixed(
-              2
-            )}, score: ${r.score.toFixed(3)})`
+              2,
+            )}, score: ${r.score.toFixed(3)})`,
           );
           log.info(r.content);
           log.info("");
@@ -208,7 +209,7 @@ status" to monitor job progress.
 
 Examples:
   $ assistant memory rebuild-index
-  $ assistant memory status`
+  $ assistant memory status`,
     )
     .action(() => {
       initializeDb();
@@ -219,13 +220,13 @@ Examples:
   memory
     .command("re-extract")
     .description(
-      "Re-extract memories from conversations using the latest extraction prompt"
+      "Re-extract memories from conversations using the latest extraction prompt",
     )
     .option(
       "-c, --conversation <id>",
       "Target a specific conversation by ID (repeatable)",
       (val: string, prev: string[]) => [...prev, val],
-      [] as string[]
+      [] as string[],
     )
     .option("-t, --top <n>", "Auto-select top N conversations by message count")
     .option("--dry-run", "Show what would be re-extracted without doing it")
@@ -248,7 +249,7 @@ background worker).
 Examples:
   $ assistant memory re-extract --top 20
   $ assistant memory re-extract --conversation conv_abc123
-  $ assistant memory re-extract --top 10 --dry-run`
+  $ assistant memory re-extract --top 10 --dry-run`,
     )
     .action(
       (opts: { conversation?: string[]; top?: string; dryRun?: boolean }) => {
@@ -288,7 +289,7 @@ Examples:
 
         if (targets.length === 0) {
           log.info(
-            "No targets specified. Use --conversation <id> or --top <n>."
+            "No targets specified. Use --conversation <id> or --top <n>.",
           );
           return;
         }
@@ -307,8 +308,117 @@ Examples:
 
         const { jobIds } = requestReextract(targets);
         log.info(
-          `\nQueued ${jobIds.length} re-extraction job(s). The assistant will process them in the background.`
+          `\nQueued ${jobIds.length} re-extraction job(s). The assistant will process them in the background.`,
         );
-      }
+      },
+    );
+
+  memory
+    .command("compact")
+    .description(
+      "Rewrite memory nodes whose content exceeds the length cap (backfill)",
+    )
+    .option(
+      "--threshold <n>",
+      "Content length threshold — nodes longer than this are candidates (default: 400)",
+      (v: string) => Number.parseInt(v, 10),
+      400,
+    )
+    .option(
+      "--limit <n>",
+      "Maximum number of candidates to process (default: no limit)",
+      (v: string) => Number.parseInt(v, 10),
+    )
+    .option("--apply", "Rewrite content (default is a candidate-only preview)")
+    .addHelpText(
+      "after",
+      `
+One-off backfill for memory graphs that accumulated over-long content before
+the extraction prompt was tightened to enforce the 1-3 sentence / ~300
+character cap. Scans memory_graph_nodes (skipping fidelity=gone) for entries
+whose content length exceeds --threshold, then either lists them (default)
+or rewrites each via the memoryConsolidation LLM call site (--apply).
+
+Only the content field is rewritten; significance, emotionalCharge, edges,
+triggers, and image_refs are preserved. Each rewrite is logged in
+memory_graph_node_edits with source="manual" so it is reversible.
+
+Start with a bounded spot-check before processing the whole graph:
+
+  $ assistant memory compact --limit 3 --apply
+
+Examples:
+  $ assistant memory compact                     # preview candidates
+  $ assistant memory compact --threshold 500     # tighter threshold
+  $ assistant memory compact --limit 5 --apply   # rewrite first 5
+  $ assistant memory compact --apply             # rewrite everything`,
+    )
+    .action(
+      async (opts: { threshold: number; limit?: number; apply?: boolean }) => {
+        initializeDb();
+        const apply = Boolean(opts.apply);
+
+        if (!Number.isFinite(opts.threshold) || opts.threshold <= 0) {
+          log.info("--threshold must be a positive integer");
+          return;
+        }
+        if (
+          opts.limit !== undefined &&
+          (!Number.isFinite(opts.limit) || opts.limit <= 0)
+        ) {
+          log.info("--limit must be a positive integer");
+          return;
+        }
+
+        log.info(
+          `Scanning memory_graph_nodes for content > ${opts.threshold} chars${
+            apply ? "" : " (preview — no changes will be written)"
+          }...`,
+        );
+
+        const result = await compactLongMemoryNodes({
+          threshold: opts.threshold,
+          limit: opts.limit,
+          apply,
+          onCandidates: (candidates) => {
+            log.info(`Found ${candidates.length} candidate node(s)`);
+            if (!apply) {
+              for (const c of candidates) {
+                log.info(`  ${c.id}  ${c.beforeLen} chars`);
+              }
+            }
+          },
+          onProgress: (evt) => {
+            const tag = `[${evt.action}]`;
+            log.info(
+              `${tag} ${evt.nodeId}: ${evt.beforeLen} → ${evt.afterLen} chars${
+                evt.reason ? ` (${evt.reason})` : ""
+              }`,
+            );
+            if (evt.newContent && evt.action === "compacted") {
+              log.info(`  new: ${evt.newContent}`);
+            }
+          },
+        });
+
+        log.info("");
+        log.info(`Scanned above threshold: ${result.scanned}`);
+        log.info(`Processed:               ${result.processed}`);
+        if (apply) {
+          log.info(`Compacted:               ${result.compacted}`);
+          log.info(`Skipped:                 ${result.skipped}`);
+          log.info(`Failures:                ${result.failures}`);
+          if (result.processed > 0) {
+            log.info(
+              `Chars: ${result.beforeChars} → ${result.afterChars} (saved ${
+                result.beforeChars - result.afterChars
+              })`,
+            );
+          }
+        } else if (result.processed > 0) {
+          log.info("");
+          log.info("Preview only. Re-run with --apply to rewrite.");
+        }
+      },
     );
 }

package/src/cli/commands/notifications.ts

@@ -1,41 +1,9 @@
 import type { Command } from "commander";
 
-import { getDeliverableChannels } from "../../channels/config.js";
-import { emitNotificationSignal } from "../../notifications/emit-signal.js";
-import { listEvents } from "../../notifications/events-store.js";
-import {
-  isNotificationSourceChannel,
-  isNotificationSourceEventName,
-  NOTIFICATION_SOURCE_CHANNELS,
-  NOTIFICATION_SOURCE_EVENT_NAMES,
-} from "../../notifications/signal.js";
-import type { NotificationChannel } from "../../notifications/types.js";
-import {
-  initAuthSigningKey,
-  resolveSigningKey,
-} from "../../runtime/auth/token-service.js";
-import { initializeDb } from "../db.js";
+import { cliIpcCall } from "../../ipc/cli-client.js";
 import { log } from "../logger.js";
 import { shouldOutputJson, writeOutput } from "../output.js";
 
-// ---------------------------------------------------------------------------
-// Help text builders
-// ---------------------------------------------------------------------------
-
-function buildSourceChannelsHelpBlock(): string {
-  const lines = NOTIFICATION_SOURCE_CHANNELS.map(
-    (c) => `  ${c.id.padEnd(20)} ${c.description}`,
-  );
-  return `\nSource channels:\n${lines.join("\n")}`;
-}
-
-function buildSourceEventNamesHelpBlock(): string {
-  const lines = NOTIFICATION_SOURCE_EVENT_NAMES.map(
-    (e) => `  ${e.id.padEnd(50)} ${e.description}`,
-  );
-  return `\nSource event names:\n${lines.join("\n")}`;
-}
-
 // ---------------------------------------------------------------------------
 // Command registration
 // ---------------------------------------------------------------------------
@@ -55,8 +23,6 @@ Notifications flow through a unified pipeline: a signal is emitted with a
 source channel, event name, and attention hints. The decision engine evaluates
 whether and where to deliver the notification based on connected channels,
 urgency, and user preferences.
-${buildSourceChannelsHelpBlock()}
-${buildSourceEventNamesHelpBlock()}
 
 Examples:
   $ assistant notifications send --source-channel assistant_tool --source-event-name user.send_notification --message "Build finished"
@@ -129,6 +95,10 @@ Examples:
       "--dedupe-key <key>",
       "Optional dedupe key to suppress duplicate notifications",
     )
+    .option(
+      "--deep-link-metadata <json>",
+      "Optional JSON metadata clients can use for deep linking",
+    )
     .addHelpText(
       "after",
       `
@@ -165,37 +135,12 @@ Examples:
           preferredChannels?: string;
           sessionId?: string;
           dedupeKey?: string;
+          deepLinkMetadata?: string;
         },
         cmd: Command,
       ) => {
         try {
-          // Validate --source-channel
-          if (!isNotificationSourceChannel(opts.sourceChannel)) {
-            const validChannels = NOTIFICATION_SOURCE_CHANNELS.map(
-              (c) => c.id,
-            ).join(", ");
-            writeOutput(cmd, {
-              ok: false,
-              error: `Invalid source channel "${opts.sourceChannel}". Valid values: ${validChannels}`,
-            });
-            process.exitCode = 1;
-            return;
-          }
-
-          // Validate --source-event-name
-          if (!isNotificationSourceEventName(opts.sourceEventName)) {
-            const validEvents = NOTIFICATION_SOURCE_EVENT_NAMES.map(
-              (e) => e.id,
-            ).join(", ");
-            writeOutput(cmd, {
-              ok: false,
-              error: `Invalid source event name "${opts.sourceEventName}". Valid values: ${validEvents}`,
-            });
-            process.exitCode = 1;
-            return;
-          }
-
-          // Validate --message
+          // Validate --message (keep basic validation for immediate CLI feedback)
           const message = opts.message.trim();
           if (message.length === 0) {
             writeOutput(cmd, {
@@ -217,7 +162,7 @@ Examples:
             return;
           }
 
-          // Validate --deadline-at
+          // Parse --deadline-at
           let deadlineAt: number | undefined;
           if (opts.deadlineAt != null) {
             const parsed = Number(opts.deadlineAt);
@@ -232,36 +177,43 @@ Examples:
             deadlineAt = parsed;
           }
 
-          // Validate --preferred-channels
-          let preferredChannels: NotificationChannel[] | undefined;
+          // Parse --preferred-channels
+          let preferredChannels: string[] | undefined;
           if (opts.preferredChannels) {
-            const deliverable = getDeliverableChannels();
-            const requested = opts.preferredChannels
+            preferredChannels = opts.preferredChannels
               .split(",")
               .map((ch) => ch.trim())
               .filter((ch) => ch.length > 0);
+          }
 
-            for (const ch of requested) {
-              if (!deliverable.includes(ch as NotificationChannel)) {
-                writeOutput(cmd, {
-                  ok: false,
-                  error: `Invalid preferred channel "${ch}". Valid deliverable channels: ${deliverable.join(", ")}`,
-                });
-                process.exitCode = 1;
-                return;
-              }
+          // Parse --deep-link-metadata
+          let deepLinkMetadata: Record<string, unknown> | undefined;
+          if (opts.deepLinkMetadata != null) {
+            try {
+              deepLinkMetadata = JSON.parse(opts.deepLinkMetadata) as Record<
+                string,
+                unknown
+              >;
+            } catch {
+              writeOutput(cmd, {
+                ok: false,
+                error: `Invalid deep-link-metadata: must be a valid JSON string`,
+              });
+              process.exitCode = 1;
+              return;
             }
-            preferredChannels = requested as NotificationChannel[];
           }
 
-          initializeDb();
-          initAuthSigningKey(resolveSigningKey());
-
           const sourceContextId = opts.sessionId ?? `cli-${Date.now()}`;
 
-          const result = await emitNotificationSignal({
-            sourceEventName: opts.sourceEventName,
+          const result = await cliIpcCall<{
+            signalId: string;
+            dispatched: boolean;
+            deduplicated: boolean;
+            reason: string;
+          }>("emit_notification_signal", {
             sourceChannel: opts.sourceChannel,
+            sourceEventName: opts.sourceEventName,
             sourceContextId,
             attentionHints: {
               requiresAction: opts.requiresAction ?? true,
@@ -275,24 +227,33 @@ Examples:
               requestedBySource: opts.sourceChannel,
               ...(opts.title ? { requestedTitle: opts.title } : {}),
               ...(preferredChannels?.length ? { preferredChannels } : {}),
+              ...(deepLinkMetadata ? { deepLinkMetadata } : {}),
             },
             ...(opts.dedupeKey ? { dedupeKey: opts.dedupeKey } : {}),
             throwOnError: true,
           });
 
+          if (!result.ok) {
+            writeOutput(cmd, { ok: false, error: result.error });
+            process.exitCode = 1;
+            return;
+          }
+
+          const signal = result.result!;
+
           writeOutput(cmd, {
             ok: true,
-            signalId: result.signalId,
-            dispatched: result.dispatched,
-            reason: result.reason,
+            signalId: signal.signalId,
+            dispatched: signal.dispatched,
+            reason: signal.reason,
           });
 
           if (!shouldOutputJson(cmd)) {
             log.info(
-              `Signal ${result.signalId} emitted (dispatched: ${result.dispatched})`,
+              `Signal ${signal.signalId} emitted (dispatched: ${signal.dispatched})`,
             );
-            if (result.reason) {
-              log.info(`  Reason: ${result.reason}`);
+            if (signal.reason) {
+              log.info(`  Reason: ${signal.reason}`);
             }
           }
         } catch (err) {
@@ -318,7 +279,6 @@ Examples:
 Reads from the local notification events store, ordered by creation time
 (newest first). Each event represents a signal that was emitted through the
 notification pipeline.
-${buildSourceEventNamesHelpBlock()}
 
 Examples:
   $ assistant notifications list
@@ -327,7 +287,7 @@ Examples:
   $ assistant notifications list --source-event-name schedule.notify --limit 10 --json`,
     )
     .action(
-      (
+      async (
         opts: {
           limit?: string;
           sourceEventName?: string;
@@ -368,23 +328,28 @@ Examples:
             limit = parsed;
           }
 
-          initializeDb();
-
-          const rows = listEvents({
+          const result = await cliIpcCall<
+            Array<{
+              id: string;
+              sourceEventName: string;
+              sourceChannel: string;
+              sourceContextId: string;
+              urgency: string;
+              dedupeKey: string | null;
+              createdAt: string;
+            }>
+          >("list_notification_events", {
             limit,
             sourceEventName: opts.sourceEventName,
           });
 
-          const events = rows.map((row) => ({
-            id: row.id,
-            sourceEventName: row.sourceEventName,
-            sourceChannel: row.sourceChannel,
-            sourceContextId: row.sourceContextId,
-            urgency: (JSON.parse(row.attentionHintsJson) as { urgency: string })
-              .urgency,
-            dedupeKey: row.dedupeKey,
-            createdAt: new Date(row.createdAt).toISOString(),
-          }));
+          if (!result.ok) {
+            writeOutput(cmd, { ok: false, error: result.error });
+            process.exitCode = 1;
+            return;
+          }
+
+          const events = result.result!;
 
           writeOutput(cmd, { ok: true, events });
 

package/src/cli/commands/oauth/__tests__/providers-register.test.ts

@@ -148,7 +148,7 @@ const sampleProviderRow = {
   userinfoUrl: null,
   baseUrl: null,
   defaultScopes: "[]",
-  scopePolicy: "{}",
+  availableScopes: null,
   scopeSeparator: null,
   authorizeParams: null,
   managedServiceConfigKey: null,

package/src/cli/commands/oauth/__tests__/providers-update.test.ts

@@ -148,7 +148,7 @@ const sampleProviderRow = {
   userinfoUrl: null,
   baseUrl: null,
   defaultScopes: "[]",
-  scopePolicy: "{}",
+  availableScopes: null,
   scopeSeparator: null,
   authorizeParams: null,
   managedServiceConfigKey: null,

package/src/cli/commands/oauth/connect.ts

@@ -105,8 +105,8 @@ Arguments:
   provider   Provider name (e.g. google, slack, notion).
              Run 'assistant oauth providers list' to see available providers.
 
-In managed mode, --scopes must be in the provider's allowed set (use full
-scope URLs). In BYO mode, --scopes are appended to the provider's defaults.
+When --scopes is provided, the specified scopes replace the provider's
+defaults entirely (use full scope URLs).
 By default, the browser opens automatically and the command waits for
 completion. Use --no-browser to print the URL instead (useful for headless
 or SSH sessions).