@vellumai/assistant 0.8.3 → 0.8.4

package/src/cli/commands/conversations.ts

@@ -2,10 +2,15 @@ import { existsSync, readFileSync } from "node:fs";
 
 import type { Command } from "commander";
 
-import { cliIpcCall, exitFromIpcResult } from "../../ipc/cli-client.js";
+import {
+  cliIpcCall,
+  exitCodeFromIpcResult,
+  exitFromIpcResult,
+} from "../../ipc/cli-client.js";
 import { registerCommand } from "../lib/register-command.js";
 import { timeAgo } from "../lib/time-ago.js";
 import { log } from "../logger.js";
+import { tryResolveConversationId } from "../utils/conversation-id.js";
 import { registerConversationsDeferCommand } from "./conversations-defer.js";
 import { registerConversationsImportCommand } from "./conversations-import.js";
 
@@ -14,6 +19,23 @@ type ConversationSeedMessage = {
   content: string;
 };
 
+type SlackDetachCliResult = {
+  detached: boolean;
+  channelId: string;
+  threadTs: string;
+  source: "explicit" | "conversation_binding";
+  conversationId?: string;
+};
+
+function outputSlackDetachError(message: string, json?: boolean): void {
+  if (json) {
+    process.stdout.write(JSON.stringify({ ok: false, error: message }) + "\n");
+  } else {
+    log.error(`Error: ${message}`);
+  }
+  process.exitCode = 1;
+}
+
 function readSeedMessages(
   contentFile?: string,
 ): ConversationSeedMessage[] | undefined {
@@ -318,6 +340,111 @@ Examples:
           },
         );
 
+      // -------------------------------------------------------------------
+      // slack
+      // -------------------------------------------------------------------
+
+      const slack = conversations
+        .command("slack")
+        .description("Manage Slack conversation bindings");
+
+      slack
+        .command("detach [conversationId]")
+        .alias("mute")
+        .description("Detach the assistant from a Slack thread")
+        .option("--channel <id>", "Slack channel ID")
+        .option("--thread <ts>", "Slack thread timestamp")
+        .option("--json", "Output result as JSON")
+        .addHelpText(
+          "after",
+          `
+Arguments:
+  conversationId   Optional conversation ID. Defaults to the current skill or
+                   tool conversation when available.
+
+Detaches the assistant from Socket Mode listening for the Slack thread bound
+to a conversation, or for explicit --channel and --thread identifiers.
+
+Examples:
+  $ assistant conversations slack detach
+  $ assistant conversations slack detach conv-123
+  $ assistant conversations slack mute --channel C123 --thread 1700000000.000100
+  $ assistant conversations slack detach --json`,
+        )
+        .action(
+          async (
+            conversationIdArg: string | undefined,
+            opts: { channel?: string; thread?: string; json?: boolean },
+          ) => {
+            const channelId = opts.channel?.trim();
+            const threadTs = opts.thread?.trim();
+            const hasExplicitSlackTarget =
+              opts.channel !== undefined || opts.thread !== undefined;
+            const body: Record<string, string> = {};
+
+            if (hasExplicitSlackTarget) {
+              if (!channelId || !threadTs) {
+                outputSlackDetachError(
+                  "Both --channel and --thread are required when using explicit Slack identifiers.",
+                  opts.json,
+                );
+                return;
+              }
+              body.channelId = channelId;
+              body.threadTs = threadTs;
+            } else {
+              const conversationId = tryResolveConversationId({
+                explicit: conversationIdArg,
+              });
+              if (!conversationId) {
+                outputSlackDetachError(
+                  "No conversation ID available. Pass a conversation ID, provide --channel and --thread, or run this command from a skill or bash tool context.",
+                  opts.json,
+                );
+                return;
+              }
+              body.conversationId = conversationId;
+            }
+
+            const result = await cliIpcCall<SlackDetachCliResult>(
+              "conversation_slack_detach_cli",
+              { body },
+            );
+
+            if (!result.ok) {
+              if (opts.json) {
+                process.stdout.write(
+                  JSON.stringify({
+                    ok: false,
+                    error: result.error ?? "Failed to detach Slack thread",
+                  }) + "\n",
+                );
+                process.exitCode = exitCodeFromIpcResult(result);
+                return;
+              }
+              return exitFromIpcResult(result);
+            }
+
+            const detach = result.result!;
+            if (opts.json) {
+              process.stdout.write(
+                JSON.stringify({ ok: true, result: detach }) + "\n",
+              );
+              return;
+            }
+
+            if (detach.detached) {
+              log.info(
+                `Detached Slack thread ${detach.threadTs} in channel ${detach.channelId} from assistant listening.`,
+              );
+            } else {
+              log.info(
+                `Slack thread ${detach.threadTs} in channel ${detach.channelId} was already detached from assistant listening.`,
+              );
+            }
+          },
+        );
+
       // -------------------------------------------------------------------
       // clear
       // -------------------------------------------------------------------

package/src/cli/commands/inference-providers.ts

@@ -16,7 +16,12 @@
 
 import type { Command } from "commander";
 
+import { isAssistantFeatureFlagEnabled } from "../../config/assistant-feature-flags.js";
+import { getConfig } from "../../config/loader.js";
 import { cliIpcCall } from "../../ipc/cli-client.js";
+import type { OAuth2Config } from "../../security/oauth2.js";
+import { startOAuth2Flow } from "../../security/oauth2.js";
+import { setSecureKeyAsync } from "../../security/secure-keys.js";
 import { log } from "../logger.js";
 
 // ---------------------------------------------------------------------------
@@ -165,7 +170,11 @@ function buildAuthInput(
     if (credential) return "--credential is not accepted with --auth none";
     return { type: "none" };
   }
-  return `Unknown auth type "${authType}". Use: api_key, platform, none`;
+  if (authType === "oauth_subscription") {
+    if (!credential) return "--credential is required when --auth oauth_subscription";
+    return { type: "oauth_subscription", credential };
+  }
+  return `Unknown auth type "${authType}". Use: api_key, platform, none, oauth_subscription`;
 }
 
 function writeCliError(msg: string, json?: boolean): void {
@@ -311,6 +320,141 @@ function attachDeleteSubcommand(connections: Command): void {
     });
 }
 
+// ---------------------------------------------------------------------------
+// OpenAI Codex OAuth config (PKCE, no client secret)
+// ---------------------------------------------------------------------------
+
+const OPENAI_CODEX_OAUTH_CONFIG: OAuth2Config = {
+  authorizeUrl: "https://auth.openai.com/oauth/authorize",
+  tokenExchangeUrl: "https://auth.openai.com/oauth/token",
+  clientId: "app_EMoamEEZ73f0CkXaXp7hrann",
+  scopes: ["openid", "profile", "email", "offline_access"],
+  scopeSeparator: " ",
+  authorizeParams: { id_token_add_organizations: "true" },
+};
+
+// ---------------------------------------------------------------------------
+// Subcommand: login-chatgpt
+// ---------------------------------------------------------------------------
+
+function attachLoginChatgptSubcommand(providers: Command): void {
+  providers
+    .command("login-chatgpt")
+    .description("Authenticate with ChatGPT via browser OAuth flow")
+    .option("--json", "Output as JSON")
+    .action(async (opts: { json?: boolean }) => {
+      const config = getConfig();
+      if (!isAssistantFeatureFlagEnabled("chatgpt-subscription-auth", config)) {
+        writeCliError("This feature is not yet available", opts.json);
+        return;
+      }
+
+      try {
+        // Step 1: Run browser-based PKCE OAuth flow
+        process.stdout.write("Opening browser for ChatGPT authentication...\n");
+        const result = await startOAuth2Flow(
+          OPENAI_CODEX_OAUTH_CONFIG,
+          {
+            openUrl: (url) => {
+              Bun.spawn(["open", url]);
+            },
+          },
+          {
+            callbackTransport: "loopback",
+            loopbackPort: 1455,
+            loopbackCallbackPath: "/auth/callback",
+          },
+        );
+        const tokens = result.tokens;
+
+        // Step 2: Store tokens in CES
+        const accessStored = await setSecureKeyAsync(
+          "credential/chatgpt/access_token",
+          tokens.accessToken,
+        );
+        if (!accessStored) {
+          writeCliError("Failed to store access token", opts.json);
+          return;
+        }
+
+        if (tokens.refreshToken) {
+          const refreshStored = await setSecureKeyAsync(
+            "credential/chatgpt/refresh_token",
+            tokens.refreshToken,
+          );
+          if (!refreshStored) {
+            writeCliError("Failed to store refresh token", opts.json);
+            return;
+          }
+        }
+
+        if (tokens.expiresIn) {
+          const expiresAt = Math.floor(Date.now() / 1000 + tokens.expiresIn);
+          await setSecureKeyAsync(
+            "credential/chatgpt/expires_at",
+            String(expiresAt),
+          );
+        }
+
+        // Step 3: Create (or update) provider connection via IPC
+        const connectionName = "chatgpt-subscription";
+        const authInput = {
+          type: "oauth_subscription",
+          credential: "credential/chatgpt/access_token",
+        };
+
+        // Try to update first; if the connection doesn't exist, create it.
+        const updateResult = await cliIpcCall<ProviderConnection>(
+          "inference_provider_connections_update",
+          {
+            pathParams: { name: connectionName },
+            body: { auth: authInput },
+          },
+        );
+
+        if (!updateResult.ok) {
+          // Connection doesn't exist yet — create it
+          const createResult = await cliIpcCall<ProviderConnection>(
+            "inference_provider_connections_create",
+            {
+              body: {
+                name: connectionName,
+                provider: "openai",
+                auth: authInput,
+              },
+            },
+          );
+
+          if (!createResult.ok) {
+            writeCliError(
+              createResult.error ?? "Failed to create provider connection",
+              opts.json,
+            );
+            return;
+          }
+        }
+
+        if (opts.json) {
+          process.stdout.write(
+            JSON.stringify({
+              ok: true,
+              connection: connectionName,
+              message: "ChatGPT subscription auth configured successfully",
+            }) + "\n",
+          );
+        } else {
+          process.stdout.write(
+            `ChatGPT subscription auth configured successfully.\n` +
+              `Connection "${connectionName}" is ready (provider=openai, auth=oauth_subscription).\n`,
+          );
+        }
+      } catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        writeCliError(message, opts.json);
+      }
+    });
+}
+
 // ---------------------------------------------------------------------------
 // Registration
 // ---------------------------------------------------------------------------
@@ -349,4 +493,6 @@ Examples:
   attachCreateSubcommand(connections);
   attachUpdateSubcommand(connections);
   attachDeleteSubcommand(connections);
+
+  attachLoginChatgptSubcommand(providers);
 }

package/src/cli/commands/memory-v2.ts

@@ -22,7 +22,9 @@ import { cliIpcCall } from "../../ipc/cli-client.js";
 import type {
   MemoryV2BackfillOp,
   MemoryV2BackfillResult,
+  MemoryV2EmaScoresResult,
   MemoryV2ReembedSkillsResult,
+  MemoryV2SimulateRouterResult,
   MemoryV2ValidateResult,
 } from "../../runtime/routes/memory-v2-routes.js";
 import { registerCommand } from "../lib/register-command.js";
@@ -252,6 +254,312 @@ Examples:
             process.exitCode = 1;
           }
         });
+
+      // ── ema ───────────────────────────────────────────────────────────────
+
+      v2.command("ema")
+        .description(
+          "List concept pages by injection-frequency EMA score (read-only)",
+        )
+        .option(
+          "-n, --limit <count>",
+          "Maximum rows to print (default 25; ignored with --all)",
+          "25",
+        )
+        .option("--all", "Print every page, including zero-score pages")
+        .option(
+          "--include-zeros",
+          "Include pages with score 0 in the default-limited view",
+        )
+        .option("--json", "Emit raw JSON instead of a formatted table")
+        .addHelpText(
+          "after",
+          `
+EMA score is the time-decayed sum Σ exp(-λ × (now - tᵢ)) with a 3-day
+half-life, computed from memory_v2_injection_events. A score of 1.0 means
+roughly one router selection in the last few minutes; 0.5 means a single
+selection ~3 days ago. Pages that have never been router-selected since
+EMA tracking began report 0.
+
+Examples:
+  $ assistant memory v2 ema
+  $ assistant memory v2 ema -n 100
+  $ assistant memory v2 ema --all --json | jq '.entries | length'`,
+        )
+        .action(
+          async (opts: {
+            limit: string;
+            all?: boolean;
+            includeZeros?: boolean;
+            json?: boolean;
+          }) => {
+            const result = await cliIpcCall<MemoryV2EmaScoresResult>(
+              "memory_v2_ema_scores",
+              { body: {} },
+            );
+
+            if (!result.ok) {
+              log.error(result.error ?? "Failed to fetch EMA scores");
+              process.exitCode = 1;
+              return;
+            }
+
+            const allEntries = result.result!.entries;
+            const includeZeros =
+              opts.all === true || opts.includeZeros === true;
+            const visible = includeZeros
+              ? allEntries
+              : allEntries.filter((e) => e.score > 0);
+
+            const limit =
+              opts.all === true ? visible.length : Number(opts.limit);
+            if (!opts.all && (!Number.isFinite(limit) || limit < 1)) {
+              log.error(
+                `--limit must be a positive integer (got "${opts.limit}")`,
+              );
+              process.exitCode = 1;
+              return;
+            }
+            const rows = visible.slice(0, limit);
+
+            if (opts.json === true) {
+              log.info(
+                JSON.stringify(
+                  {
+                    entries: rows,
+                    totalScored: allEntries.filter((e) => e.score > 0).length,
+                    totalPages: allEntries.length,
+                  },
+                  null,
+                  2,
+                ),
+              );
+              return;
+            }
+
+            if (rows.length === 0) {
+              log.info(
+                "No concept pages have any EMA signal yet. Send a few turns through the router and try again.",
+              );
+              return;
+            }
+
+            const slugWidth = Math.min(
+              60,
+              Math.max(...rows.map((r) => r.slug.length)),
+            );
+            const header = `${"slug".padEnd(slugWidth)}  ${"score".padStart(8)}  modified`;
+            log.info(header);
+            log.info("-".repeat(header.length));
+            for (const row of rows) {
+              const slug =
+                row.slug.length > slugWidth
+                  ? row.slug.slice(0, slugWidth - 1) + "…"
+                  : row.slug.padEnd(slugWidth);
+              const score = row.score.toFixed(3).padStart(8);
+              const modified =
+                row.modifiedAt > 0
+                  ? new Date(row.modifiedAt).toISOString().slice(0, 10)
+                  : "—";
+              log.info(`${slug}  ${score}  ${modified}`);
+            }
+            const totalScored = allEntries.filter((e) => e.score > 0).length;
+            log.info(
+              `\n${rows.length} of ${visible.length} shown (${totalScored} total with score > 0, ${allEntries.length} pages indexed).`,
+            );
+          },
+        );
+
+      // ── simulate ──────────────────────────────────────────────────────────
+
+      v2.command("simulate")
+        .description(
+          "Dry-run the v4 router against a synthetic query (read-only)",
+        )
+        .requiredOption(
+          "-q, --query <text>",
+          "User query to route the simulated turn against",
+        )
+        .option(
+          "--tier1-size <n>",
+          "Override memory.v2.router.tier1_size for this run (number or 'null')",
+        )
+        .option(
+          "--tier2-size <n>",
+          "Override memory.v2.router.tier2_size for this run (number or 'null')",
+        )
+        .option(
+          "--batch-size <n>",
+          "Override memory.v2.router.batch_size for this run (number or 'null')",
+        )
+        .option("--json", "Emit raw JSON instead of a grouped report")
+        .addHelpText(
+          "after",
+          `
+Runs the v4 router read-only against the live page index + EMA scores, with
+optional tier/batch overrides applied on top of the live config. NO writes:
+no row is appended to memory_v2_injection_events or memory_v2_activation_logs,
+and no activation state is mutated. Use this to preview the effect of a
+config knob change before flipping it in workspace config.json.
+
+Limitations:
+  - priorEverInjected is empty (single-turn simulation; live router dedups
+    against pages already in context).
+  - NOW.md is read at simulate-time, not historical-turn time.
+  - assistantMessage is empty.
+
+Pass 'null' to an override flag to explicitly disable that tier for this run
+(e.g. --tier2-size null reverts to tier1 → tier3). Omitting an override
+inherits the live config value.
+
+Examples:
+  $ assistant memory v2 simulate -q "what should we ship next"
+  $ assistant memory v2 simulate -q "..." --tier1-size 100 --tier2-size 200 --batch-size 50
+  $ assistant memory v2 simulate -q "..." --json | jq '.selectedSlugs'`,
+        )
+        .action(
+          async (opts: {
+            query: string;
+            tier1Size?: string;
+            tier2Size?: string;
+            batchSize?: string;
+            json?: boolean;
+          }) => {
+            const parseOverride = (
+              flag: string,
+              raw: string | undefined,
+            ): number | null | undefined => {
+              if (raw === undefined) return undefined;
+              if (raw === "null") return null;
+              const parsed = Number(raw);
+              if (!Number.isInteger(parsed) || parsed < 1) {
+                log.error(
+                  `${flag} must be a positive integer or 'null' (got "${raw}")`,
+                );
+                process.exitCode = 1;
+                throw new Error("invalid-override");
+              }
+              return parsed;
+            };
+
+            let configOverrides:
+              | {
+                  tier1_size?: number | null;
+                  tier2_size?: number | null;
+                  batch_size?: number | null;
+                }
+              | undefined;
+            try {
+              const t1 = parseOverride("--tier1-size", opts.tier1Size);
+              const t2 = parseOverride("--tier2-size", opts.tier2Size);
+              const bs = parseOverride("--batch-size", opts.batchSize);
+              configOverrides = {
+                ...(t1 !== undefined ? { tier1_size: t1 } : {}),
+                ...(t2 !== undefined ? { tier2_size: t2 } : {}),
+                ...(bs !== undefined ? { batch_size: bs } : {}),
+              };
+              if (Object.keys(configOverrides).length === 0) {
+                configOverrides = undefined;
+              }
+            } catch {
+              return;
+            }
+
+            const result = await cliIpcCall<MemoryV2SimulateRouterResult>(
+              "memory_v2_simulate_router",
+              {
+                body: {
+                  query: opts.query,
+                  ...(configOverrides ? { configOverrides } : {}),
+                },
+              },
+            );
+
+            if (!result.ok) {
+              log.error(result.error ?? "Failed to simulate router");
+              process.exitCode = 1;
+              return;
+            }
+
+            const payload = result.result!;
+
+            if (opts.json === true) {
+              log.info(JSON.stringify(payload, null, 2));
+              return;
+            }
+
+            log.info("Memory Router Simulation");
+            log.info("========================");
+            log.info(`Query: ${JSON.stringify(opts.query)}`);
+            log.info("");
+            log.info("Config (effective):");
+            const formatKnob = (
+              key: keyof MemoryV2SimulateRouterResult["effectiveConfig"],
+            ): string => {
+              const eff = payload.effectiveConfig[key];
+              const override = (
+                payload.overrides as Record<string, number | null | undefined>
+              )[key];
+              const effStr = eff === null ? "null" : String(eff);
+              if (override === undefined) {
+                return `  ${key}: ${effStr}`;
+              }
+              return `  ${key}: ${effStr}  (override)`;
+            };
+            log.info(formatKnob("tier1_size"));
+            log.info(formatKnob("tier2_size"));
+            log.info(formatKnob("batch_size"));
+            log.info(`  max_page_ids: ${payload.effectiveConfig.max_page_ids}`);
+            log.info("");
+            log.info(`Total candidate pages: ${payload.totalCandidatePages}`);
+            log.info(
+              `Selected: ${payload.selectedSlugs.length} / ${payload.effectiveConfig.max_page_ids} pages`,
+            );
+            if (payload.failureReason) {
+              log.info(`Failure: ${payload.failureReason}`);
+            }
+            log.info("");
+
+            const grouped = new Map<string, string[]>();
+            for (const slug of payload.selectedSlugs) {
+              const source = payload.sourceBySlug[slug] ?? "unknown";
+              const bucket = grouped.get(source) ?? [];
+              bucket.push(slug);
+              grouped.set(source, bucket);
+            }
+            const sortedKeys = [...grouped.keys()].sort((a, b) => {
+              const order = (s: string) => {
+                if (s === "tier1") return 0;
+                if (s === "tier2") return 1;
+                if (s.startsWith("tier3:")) {
+                  return 2 + Number(s.slice("tier3:".length));
+                }
+                return Number.MAX_SAFE_INTEGER;
+              };
+              return order(a) - order(b);
+            });
+
+            for (const key of sortedKeys) {
+              const label = key.startsWith("tier3:")
+                ? `tier 3 · b${key.slice("tier3:".length)}`
+                : key === "tier1"
+                  ? "tier 1"
+                  : key === "tier2"
+                    ? "tier 2"
+                    : key;
+              log.info(label);
+              for (const slug of grouped.get(key)!) {
+                if (key === "tier2") {
+                  const score = payload.scores[slug] ?? 0;
+                  log.info(`  - ${slug}  (EMA ${score.toFixed(3)})`);
+                } else {
+                  log.info(`  - ${slug}`);
+                }
+              }
+              log.info("");
+            }
+          },
+        );
     },
   });
 }

package/src/cli/commands/notifications.ts

@@ -8,6 +8,7 @@ import {
 import { registerCommand } from "../lib/register-command.js";
 import { log } from "../logger.js";
 import { shouldOutputJson, writeOutput } from "../output.js";
+import { tryResolveConversationId } from "../utils/conversation-id.js";
 
 // ---------------------------------------------------------------------------
 // Command registration
@@ -248,8 +249,6 @@ Examples:
                 }
               }
 
-              const sourceContextId = opts.sessionId ?? `cli-${Date.now()}`;
-
               // Validate --conversation-id if provided
               const conversationId = opts.conversationId?.trim();
               if (opts.conversationId != null && !conversationId) {
@@ -261,6 +260,26 @@ Examples:
                 return;
               }
 
+              // Picks up __CONVERSATION_ID / __SKILL_CONTEXT_JSON env vars
+              // so deferred-emit can buffer notifications when called from a
+              // background job that hasn't confirmed success yet.
+              const originatingConversationId = tryResolveConversationId();
+
+              // The signal's `sourceContextId` doubles as the home-feed's
+              // navigation target — `resolveHomeFeedMirror` looks it up via
+              // `getConversation()` and only renders a "Go to Convo" button
+              // when it resolves to a real row. Prefer the conversation the
+              // CLI was invoked from (env-derived) so notifications emitted
+              // by background jobs and skills link back to their producing
+              // convo; an explicit --session-id still wins to preserve
+              // caller intent, and --conversation-id is the last resort
+              // before the unresolvable `cli-<ts>` sentinel.
+              const sourceContextId =
+                opts.sessionId ??
+                originatingConversationId ??
+                conversationId ??
+                `cli-${Date.now()}`;
+
               const result = await cliIpcCall<{
                 signalId: string;
                 dispatched: boolean;
@@ -289,6 +308,9 @@ Examples:
                   ...(conversationId
                     ? { conversationAffinityHint: { vellum: conversationId } }
                     : {}),
+                  ...(originatingConversationId
+                    ? { originatingConversationId }
+                    : {}),
                   throwOnError: true,
                 },
               });