@vellumai/assistant 0.9.1-staging.1 → 0.10.0-staging.2

package/src/workspace/migrations/registry.ts

@@ -102,6 +102,10 @@ import { upgradeBalancedEconomyToMinimaxM3Migration } from "./101-upgrade-balanc
 import { preserveHeartbeatEnabledForExistingWorkspacesMigration } from "./102-preserve-heartbeat-enabled-for-existing-workspaces.js";
 import { upgradeQualityProfileToOpus48Migration } from "./103-upgrade-quality-profile-to-opus-4-8.js";
 import { recheckAdaptiveThinkingModelImpliedAnthropicMigration } from "./104-recheck-adaptive-thinking-model-implied-anthropic.js";
+import { enableMemoryV3LiveForNewWorkspacesMigration } from "./105-enable-memory-v3-live-for-new-workspaces.js";
+import { dropCollectUsageDataMigration } from "./106-drop-collect-usage-data.js";
+import { dropSendDiagnosticsMigration } from "./107-drop-send-diagnostics.js";
+import { dropBalancedEconomyProfileMigration } from "./108-drop-balanced-economy-profile.js";
 import { migrateToWorkspaceVolumeMigration } from "./migrate-to-workspace-volume.js";
 import type { WorkspaceMigration } from "./types.js";
 
@@ -215,4 +219,8 @@ export const WORKSPACE_MIGRATIONS: WorkspaceMigration[] = [
   preserveHeartbeatEnabledForExistingWorkspacesMigration,
   upgradeQualityProfileToOpus48Migration,
   recheckAdaptiveThinkingModelImpliedAnthropicMigration,
+  enableMemoryV3LiveForNewWorkspacesMigration,
+  dropCollectUsageDataMigration,
+  dropSendDiagnosticsMigration,
+  dropBalancedEconomyProfileMigration,
 ];

package/src/notifications/tool-approval-copy.ts

@@ -1,142 +0,0 @@
-/**
- * Deterministic helpers for building guardian-facing tool-approval copy.
- *
- * Produces Surface card seed content blocks for `tool_approval`,
- * `tool_grant_request`, and voice/call `pending_question` (with `toolName`)
- * guardian questions, enabling Approve/Reject buttons in the Vellum in-app
- * channel (web/macOS/iOS).
- */
-
-import { buildApprovalCardBlocks } from "./approval-card-builder.js";
-import {
-  buildGuardianRequestCodeInstruction,
-  type GuardianQuestionPayload,
-  type LenientToolApprovalPayload,
-  LenientToolApprovalPayloadSchema,
-  parseGuardianQuestionPayload,
-  resolveGuardianInstructionModeFromFields,
-  resolveGuardianInstructionModeFromPayload,
-} from "./guardian-question-mode.js";
-import { nonEmpty, sanitizeIdentityField } from "./notification-utils.js";
-
-// ── Approval detection ──────────────────────────────────────────────────────
-
-/**
- * Determine whether a typed guardian.question payload represents a tool
- * approval (as opposed to a free-text answer). Uses the canonical mode
- * resolver in `guardian-question-mode.ts` — the single source of truth
- * for the requestKind → instructionMode mapping.
- */
-function isToolApprovalPayload(payload: GuardianQuestionPayload): boolean {
-  const { mode } = resolveGuardianInstructionModeFromPayload(payload);
-  return mode === "approval" && payload.requestKind !== "access_request";
-}
-
-/**
- * Lenient approval detection for partially-constructed payloads that don't
- * satisfy the strict discriminated union schema.
- */
-function isLenientToolApproval(payload: LenientToolApprovalPayload): boolean {
-  const modeResolution = resolveGuardianInstructionModeFromFields(
-    payload.requestKind,
-    payload.toolName,
-  );
-  if (!modeResolution) return false;
-  return (
-    modeResolution.mode === "approval" &&
-    payload.requestKind !== "access_request"
-  );
-}
-
-// ── Seed content blocks (Surface-based rendering) ───────────────────────────
-
-/**
- * Build structured content blocks for a tool approval/grant notification seed
- * message. Returns `null` when the payload does not represent a tool approval.
- *
- * Accepts both strict `GuardianQuestionPayload` (Zod-validated) and raw
- * `Record<string, unknown>` payloads. For raw payloads, attempts strict
- * parsing first, then falls back to lenient field extraction so cards
- * still render when optional fields are absent.
- */
-export function buildToolApprovalSeedContentBlocks(
-  payload: GuardianQuestionPayload,
-): unknown[] | null;
-export function buildToolApprovalSeedContentBlocks(
-  payload: Record<string, unknown>,
-): unknown[] | null;
-export function buildToolApprovalSeedContentBlocks(
-  payload: GuardianQuestionPayload | Record<string, unknown>,
-): unknown[] | null {
-  // Try strict Zod parsing first (full discriminated union).
-  const strict = parseGuardianQuestionPayload(
-    payload as Record<string, unknown>,
-  );
-  if (strict) {
-    if (!isToolApprovalPayload(strict)) return null;
-    return buildCardFromFields(strict);
-  }
-
-  // Fall back to lenient parsing — requires only `requestKind`.
-  const lenient = LenientToolApprovalPayloadSchema.safeParse(payload);
-  if (!lenient.success) return null;
-  if (!isLenientToolApproval(lenient.data)) return null;
-  return buildCardFromFields(lenient.data);
-}
-
-// ── Card construction (shared between strict and lenient paths) ─────────────
-
-function buildCardFromFields(
-  p: GuardianQuestionPayload | LenientToolApprovalPayload,
-): unknown[] {
-  const toolName =
-    ("toolName" in p ? nonEmpty(p.toolName) : undefined) ?? "unknown tool";
-  const rawRequester = nonEmpty(p.requesterIdentifier);
-  const requester = rawRequester
-    ? sanitizeIdentityField(rawRequester)
-    : "Someone";
-
-  const isGrant = p.requestKind === "tool_grant_request";
-
-  const metadata: Array<{ label: string; value: string }> = [];
-  metadata.push({ label: "Tool", value: toolName });
-  const sourceChannel = nonEmpty(p.sourceChannel);
-  if (sourceChannel) {
-    metadata.push({ label: "Source", value: sourceChannel });
-  }
-
-  const body = p.questionText
-    ? `> ${p.questionText}`
-    : "No additional context available.";
-
-  // Fallback text with request-code instructions for older clients.
-  const baseFallback =
-    p.questionText ?? `${requester} is requesting approval to use ${toolName}`;
-  let fallbackText = baseFallback;
-  const requestCode = nonEmpty(p.requestCode);
-  if (requestCode) {
-    const modeResolution = resolveGuardianInstructionModeFromFields(
-      p.requestKind,
-      "toolName" in p ? (p.toolName ?? undefined) : undefined,
-    );
-    const mode = modeResolution?.mode ?? "approval";
-    const instruction = buildGuardianRequestCodeInstruction(
-      requestCode.trim().toUpperCase(),
-      mode,
-    );
-    fallbackText = `${baseFallback}\n\n${instruction}`;
-  }
-
-  return buildApprovalCardBlocks({
-    surfaceIdPrefix: "tool-approval",
-    cardTitle: isGrant ? "Tool Grant Request" : "Tool Approval",
-    requesterName: requester,
-    subtitle: isGrant
-      ? "Requesting permission to use this tool"
-      : "Requesting approval to run this tool",
-    body,
-    metadata,
-    requestId: nonEmpty(p.requestId),
-    fallbackText,
-  });
-}

package/src/runtime/routes/approval-prompt-ts-tracker.ts

@@ -1,78 +0,0 @@
-/**
- * Persistent tracker for approval prompt message timestamps.
- *
- * Scopes guardian reaction approvals so only reactions on a known approval
- * prompt can resolve a pending request. Without this, a stray 👍/✅ on any
- * message in the guardian chat could approve a pending request (since
- * reactions are now admitted from any subscribed channel, not just tracked
- * bot threads).
- *
- * Entries are stored in the `approval_prompt_ts_tracker` table (created by
- * `createApprovalPromptTsTrackerTable`) so that a daemon restart between
- * prompt delivery and guardian reaction does not silently invalidate
- * reactions that are still within the 30-minute guardian approval TTL.
- * Entries expire after `APPROVAL_PROMPT_TS_TTL_MS` (guardian approval TTL
- * plus grace).
- */
-import { getSqlite } from "../../memory/db-connection.js";
-import { getLogger } from "../../util/logger.js";
-
-const log = getLogger("runtime-http");
-
-const APPROVAL_PROMPT_TS_TTL_MS = 35 * 60 * 1000;
-
-// Swallow errors: callers run this inside their delivery try/catch, so a
-// tracker throw would be misread as a delivery failure and trigger
-// fallback/retry, double-posting the guardian prompt.
-export function trackApprovalPromptTs(
-  channel: string,
-  chatId: string,
-  ts: string,
-): void {
-  try {
-    const now = Date.now();
-    const expiresAt = now + APPROVAL_PROMPT_TS_TTL_MS;
-    const db = getSqlite();
-    db.run(
-      /*sql*/ `DELETE FROM approval_prompt_ts_tracker WHERE expires_at <= ?`,
-      [now],
-    );
-    db.run(
-      /*sql*/ `INSERT OR REPLACE INTO approval_prompt_ts_tracker (channel, chat_id, ts, expires_at) VALUES (?, ?, ?, ?)`,
-      [channel, chatId, ts, expiresAt],
-    );
-  } catch (err) {
-    log.error(
-      { err, channel, chatId, ts },
-      "Failed to persist approval prompt ts tracker entry; continuing without tracking",
-    );
-  }
-}
-
-export function isTrackedApprovalPromptTs(
-  channel: string,
-  chatId: string,
-  ts: string,
-): boolean {
-  const now = Date.now();
-  const db = getSqlite();
-  const row = db
-    .query(
-      /*sql*/ `SELECT expires_at FROM approval_prompt_ts_tracker WHERE channel = ? AND chat_id = ? AND ts = ?`,
-    )
-    .get(channel, chatId, ts) as { expires_at: number } | null;
-  if (!row) return false;
-  if (row.expires_at <= now) {
-    db.run(
-      /*sql*/ `DELETE FROM approval_prompt_ts_tracker WHERE channel = ? AND chat_id = ? AND ts = ?`,
-      [channel, chatId, ts],
-    );
-    return false;
-  }
-  return true;
-}
-
-/** @internal Test-only — clear all tracked entries. */
-export function _clearApprovalPromptTsTrackerForTesting(): void {
-  getSqlite().run(/*sql*/ `DELETE FROM approval_prompt_ts_tracker`);
-}