@vellumai/assistant 0.8.2 → 0.8.4

package/src/__tests__/guardian-action-conversation-turn.test.ts

@@ -1,441 +0,0 @@
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
-import {
-  createCallSession,
-  createPendingQuestion,
-} from "../calls/call-store.js";
-import { getDb } from "../memory/db-connection.js";
-import { initializeDb } from "../memory/db-init.js";
-import {
-  createGuardianActionDelivery,
-  createGuardianActionRequest,
-  finalizeFollowup,
-  getFollowupDeliveriesByConversation,
-  getFollowupDeliveriesByDestination,
-  getGuardianActionRequest,
-  markTimedOutWithReason,
-  progressFollowupState,
-  startFollowupFromExpiredRequest,
-  updateDeliveryStatus,
-} from "../memory/guardian-action-store.js";
-import { conversations } from "../memory/schema.js";
-import { processGuardianFollowUpTurn } from "../runtime/guardian-action-conversation-turn.js";
-import type {
-  GuardianFollowUpConversationContext,
-  GuardianFollowUpConversationGenerator,
-  GuardianFollowUpTurnResult,
-} from "../runtime/http-types.js";
-
-initializeDb();
-
-function ensureConversation(id: string): void {
-  const db = getDb();
-  const now = Date.now();
-  db.insert(conversations)
-    .values({
-      id,
-      title: `Conversation ${id}`,
-      createdAt: now,
-      updatedAt: now,
-    })
-    .run();
-}
-
-function resetTables(): void {
-  const db = getDb();
-  db.run("DELETE FROM guardian_action_deliveries");
-  db.run("DELETE FROM guardian_action_requests");
-  db.run("DELETE FROM call_pending_questions");
-  db.run("DELETE FROM call_events");
-  db.run("DELETE FROM call_sessions");
-  db.run("DELETE FROM messages");
-  db.run("DELETE FROM conversations");
-}
-
-function createAwaitingChoiceRequest(
-  convId: string,
-  opts?: {
-    chatId?: string;
-    externalUserId?: string;
-    conversationId?: string;
-  },
-) {
-  ensureConversation(convId);
-  const session = createCallSession({
-    conversationId: convId,
-    provider: "twilio",
-    fromNumber: "+15550001111",
-    toNumber: "+15550002222",
-  });
-  const pq = createPendingQuestion(session.id, "What is the gate code?");
-  const request = createGuardianActionRequest({
-    kind: "ask_guardian",
-    sourceChannel: "phone",
-    sourceConversationId: convId,
-    callSessionId: session.id,
-    pendingQuestionId: pq.id,
-    questionText: pq.questionText,
-    expiresAt: Date.now() - 10_000,
-  });
-
-  const deliveryConvId = opts?.conversationId ?? `delivery-conv-${request.id}`;
-  if (opts?.conversationId) {
-    ensureConversation(opts.conversationId);
-  } else {
-    ensureConversation(deliveryConvId);
-  }
-  const delivery = createGuardianActionDelivery({
-    requestId: request.id,
-    destinationChannel: "telegram",
-    destinationChatId: opts?.chatId ?? "chat-123",
-    destinationExternalUserId: opts?.externalUserId ?? "user-456",
-    destinationConversationId: deliveryConvId,
-  });
-  updateDeliveryStatus(delivery.id, "sent");
-
-  // Expire the request
-  markTimedOutWithReason(request.id, "call_timeout");
-
-  // Start follow-up (transitions to awaiting_guardian_choice)
-  startFollowupFromExpiredRequest(request.id, "The gate code is 1234");
-
-  return {
-    request: getGuardianActionRequest(request.id)!,
-    delivery,
-    deliveryConvId,
-  };
-}
-
-// ---------------------------------------------------------------------------
-// Helpers for creating mock generators
-// ---------------------------------------------------------------------------
-
-function createMockGenerator(
-  result: GuardianFollowUpTurnResult,
-): GuardianFollowUpConversationGenerator {
-  return async (_context: GuardianFollowUpConversationContext) => result;
-}
-
-function createFailingGenerator(): GuardianFollowUpConversationGenerator {
-  return async () => {
-    throw new Error("LLM provider unavailable");
-  };
-}
-
-describe("guardian-action-conversation-turn", () => {
-  beforeEach(() => {
-    resetTables();
-  });
-
-  // ── processGuardianFollowUpTurn: classification ─────────────────────
-
-  test('classifies "call them back" as call_back', async () => {
-    const generator = createMockGenerator({
-      disposition: "call_back",
-      replyText: "Sure, I'll call them back right away.",
-    });
-
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Yes, call them back",
-      },
-      generator,
-    );
-
-    expect(result.disposition).toBe("call_back");
-    expect(result.replyText).toBe("Sure, I'll call them back right away.");
-  });
-
-  test('classifies "never mind" as decline', async () => {
-    const generator = createMockGenerator({
-      disposition: "decline",
-      replyText: "No problem. Let me know if you change your mind.",
-    });
-
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Never mind, forget it",
-      },
-      generator,
-    );
-
-    expect(result.disposition).toBe("decline");
-    expect(result.replyText).toBe(
-      "No problem. Let me know if you change your mind.",
-    );
-  });
-
-  test("classifies ambiguous input as keep_pending with clarification", async () => {
-    const generator = createMockGenerator({
-      disposition: "keep_pending",
-      replyText: "Would you like to call them back or send a text message?",
-    });
-
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "hmm I dunno",
-      },
-      generator,
-    );
-
-    expect(result.disposition).toBe("keep_pending");
-    expect(result.replyText).toContain("call them back");
-  });
-
-  // ── Failure modes ───────────────────────────────────────────────────
-
-  test("generator failure returns keep_pending with safe fallback", async () => {
-    const generator = createFailingGenerator();
-
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Call them back please",
-      },
-      generator,
-    );
-
-    expect(result.disposition).toBe("keep_pending");
-    expect(result.replyText.length).toBeGreaterThan(0);
-  });
-
-  test("no generator returns keep_pending with safe fallback", async () => {
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Call them back",
-      },
-      undefined,
-    );
-
-    expect(result.disposition).toBe("keep_pending");
-    expect(result.replyText.length).toBeGreaterThan(0);
-  });
-
-  test("generator returning empty replyText falls back to keep_pending", async () => {
-    const generator = createMockGenerator({
-      disposition: "call_back",
-      replyText: "",
-    });
-
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Call them back",
-      },
-      generator,
-    );
-
-    expect(result.disposition).toBe("keep_pending");
-  });
-
-  test("generator returning invalid disposition falls back to keep_pending", async () => {
-    const generator: GuardianFollowUpConversationGenerator = async () => {
-      return {
-        disposition:
-          "invalid_value" as GuardianFollowUpTurnResult["disposition"],
-        replyText: "Some reply",
-      };
-    };
-
-    const result = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Call them back",
-      },
-      generator,
-    );
-
-    expect(result.disposition).toBe("keep_pending");
-  });
-
-  test("reply text is always present in the result", async () => {
-    // With generator
-    const generatorResult = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Call them",
-      },
-      createMockGenerator({
-        disposition: "call_back",
-        replyText: "Calling now!",
-      }),
-    );
-    expect(typeof generatorResult.replyText).toBe("string");
-    expect(generatorResult.replyText.length).toBeGreaterThan(0);
-
-    // Without generator
-    const fallbackResult = await processGuardianFollowUpTurn({
-      questionText: "What is the gate code?",
-      lateAnswerText: "The gate code is 1234",
-      guardianReply: "Call them",
-    });
-    expect(typeof fallbackResult.replyText).toBe("string");
-    expect(fallbackResult.replyText.length).toBeGreaterThan(0);
-
-    // With failing generator
-    const failResult = await processGuardianFollowUpTurn(
-      {
-        questionText: "What is the gate code?",
-        lateAnswerText: "The gate code is 1234",
-        guardianReply: "Call them",
-      },
-      createFailingGenerator(),
-    );
-    expect(typeof failResult.replyText).toBe("string");
-    expect(failResult.replyText.length).toBeGreaterThan(0);
-  });
-
-  // ── Store queries for awaiting_guardian_choice ───────────────────────
-
-  test("getFollowupDeliveriesByDestination returns deliveries in awaiting_guardian_choice", () => {
-    const { request } = createAwaitingChoiceRequest("conv-turn-1", {
-      chatId: "chat-abc",
-      externalUserId: "user-xyz",
-    });
-
-    const deliveries = getFollowupDeliveriesByDestination(
-      "telegram",
-      "chat-abc",
-    );
-    expect(deliveries).toHaveLength(1);
-    expect(deliveries[0].requestId).toBe(request.id);
-  });
-
-  test("getFollowupDeliveriesByDestination returns empty for non-matching channel", () => {
-    createAwaitingChoiceRequest("conv-turn-2", { chatId: "chat-abc" });
-
-    const deliveries = getFollowupDeliveriesByDestination("phone", "chat-abc");
-    expect(deliveries).toHaveLength(0);
-  });
-
-  test("getFollowupDeliveriesByDestination returns empty for expired with followup_state=none", () => {
-    // Create expired request WITHOUT starting follow-up
-    ensureConversation("conv-turn-3");
-    const session = createCallSession({
-      conversationId: "conv-turn-3",
-      provider: "twilio",
-      fromNumber: "+15550001111",
-      toNumber: "+15550002222",
-    });
-    const pq = createPendingQuestion(session.id, "Question?");
-    const request = createGuardianActionRequest({
-      kind: "ask_guardian",
-      sourceChannel: "phone",
-      sourceConversationId: "conv-turn-3",
-      callSessionId: session.id,
-      pendingQuestionId: pq.id,
-      questionText: pq.questionText,
-      expiresAt: Date.now() - 10_000,
-    });
-    ensureConversation(`delivery-conv-${request.id}`);
-    const delivery = createGuardianActionDelivery({
-      requestId: request.id,
-      destinationChannel: "telegram",
-      destinationChatId: "chat-none",
-      destinationExternalUserId: "user-none",
-      destinationConversationId: `delivery-conv-${request.id}`,
-    });
-    updateDeliveryStatus(delivery.id, "sent");
-    markTimedOutWithReason(request.id, "call_timeout");
-
-    // followup_state is 'none' — should not appear
-    const deliveries = getFollowupDeliveriesByDestination(
-      "telegram",
-      "chat-none",
-    );
-    expect(deliveries).toHaveLength(0);
-  });
-
-  test("getFollowupDeliveriesByConversation returns delivery in awaiting_guardian_choice", () => {
-    const { delivery, deliveryConvId } = createAwaitingChoiceRequest(
-      "conv-turn-4",
-      {
-        conversationId: "mac-conv-1",
-      },
-    );
-
-    const found = getFollowupDeliveriesByConversation(deliveryConvId);
-    expect(found).toHaveLength(1);
-    expect(found[0].id).toBe(delivery.id);
-  });
-
-  test("getFollowupDeliveriesByConversation returns empty for non-matching conversation", () => {
-    createAwaitingChoiceRequest("conv-turn-5", {
-      conversationId: "mac-conv-2",
-    });
-
-    const found = getFollowupDeliveriesByConversation("nonexistent-conv");
-    expect(found).toHaveLength(0);
-  });
-
-  // ── State transitions from conversation engine results ──────────────
-
-  test("call_back disposition transitions to dispatching with call_back action", () => {
-    const { request } = createAwaitingChoiceRequest("conv-turn-6");
-
-    // Simulate what the handler does with a call_back disposition
-    const updated = progressFollowupState(
-      request.id,
-      "dispatching",
-      "call_back",
-    );
-    expect(updated).not.toBeNull();
-    expect(updated!.followupState).toBe("dispatching");
-    expect(updated!.followupAction).toBe("call_back");
-  });
-
-  test("decline disposition finalizes to declined", () => {
-    const { request } = createAwaitingChoiceRequest("conv-turn-8");
-
-    const updated = finalizeFollowup(request.id, "declined");
-    expect(updated).not.toBeNull();
-    expect(updated!.followupState).toBe("declined");
-    expect(updated!.followupCompletedAt).toBeGreaterThan(0);
-  });
-
-  test("keep_pending disposition does not change state", () => {
-    const { request } = createAwaitingChoiceRequest("conv-turn-9");
-
-    // No state change for keep_pending — just verify the state is still awaiting_guardian_choice
-    const reloaded = getGuardianActionRequest(request.id);
-    expect(reloaded!.followupState).toBe("awaiting_guardian_choice");
-  });
-
-  test("state transitions are atomic: second call_back after dispatching fails", () => {
-    const { request } = createAwaitingChoiceRequest("conv-turn-10");
-
-    const first = progressFollowupState(request.id, "dispatching", "call_back");
-    expect(first).not.toBeNull();
-
-    // Second attempt: already in dispatching, cannot re-enter dispatching
-    const second = progressFollowupState(
-      request.id,
-      "dispatching",
-      "call_back",
-    );
-    expect(second).toBeNull();
-
-    // Original action preserved
-    const reloaded = getGuardianActionRequest(request.id);
-    expect(reloaded!.followupAction).toBe("call_back");
-  });
-});

package/src/config/bundled-skills/document/SKILL.md

@@ -1,54 +0,0 @@
----
-name: document
-description: Write, draft, or compose long-form text (blog posts, articles, essays, reports, guides)
-compatibility: "Designed for Vellum personal assistants"
-metadata:
-  emoji: "📄"
-  vellum:
-    display-name: "Document"
-    activation-hints:
-      - "User asks to write, draft, or collaborate on long-form content — use the document editor for a better editing experience"
-      - "When content will be iterated on, reviewed, or exported, prefer the document editor over inline markdown"
-      - "When a file attachment contains a draft or document the user wants to iterate on, open it in the editor"
----
-
-Create and edit long-form documents using the built-in rich text editor. Documents open in workspace mode with chat docked to the side.
-
-## Tools
-
-- **document_create** - Opens a new document editor with an optional title and initial Markdown content. Returns a `surface_id` for subsequent updates.
-- **document_update** - Updates content in an open document editor by `surface_id`. Supports `replace` (overwrite) and `append` (add to end) modes.
-- **document_read** - Reads the current content of a document by `surface_id` when it belongs to the current conversation, or when the current actor is the guardian/local user. Use to verify content before editing.
-- **document_list** - Lists documents. Without `query`, lists the current conversation's documents. With `query`, searches by title; guardian/local users can search across conversations, while other actors are scoped to the current conversation.
-- **document_delete** - Deletes a document by `surface_id`. Use to clean up unwanted documents.
-
-## Retrieving existing documents
-
-When the user asks to see, open, or pull up a document:
-
-1. Check the `<active_documents>` block in your context — it lists all documents in this conversation with their `surface_id` and title. If the document is there, call `document_read` with its `surface_id`. Done in one call.
-2. If the document is NOT in `<active_documents>`, call `document_list` with a `query` matching the document title. For guardian/local users, this searches across previous conversations and sessions.
-3. Once you have the `surface_id`, call `document_read` to retrieve the content.
-
-**Never** search the filesystem, conversation history, or archives to find a document. Always use `document_list` with a `query`.
-
-## Creating a new document
-
-1. **Create the document**: Call `document_create` with a title (inferred from the request). Call the tool immediately, not after conversational preamble.
-2. **Write content in Markdown**: Use proper structure (`#` for titles, `##` for sections), **bold**, _italic_, code blocks, tables, lists, blockquotes as appropriate.
-3. **CRITICAL - Stream content in chunks**: Call `document_update` MULTIPLE times, not just once. Break content into logical chunks (paragraphs, sections, or every 200-300 words). Call `document_update` with `mode: "append"` for EACH chunk separately. The user experiences real-time content appearing as you write.
-
-## Editing an existing document
-
-When the user requests changes to a document:
-
-1. Find the `surface_id` from the `<active_documents>` context block.
-2. Use `document_update` with the existing `surface_id` — do NOT call `document_create` again.
-3. Use `mode: "replace"` for full rewrites or `mode: "append"` for additions.
-
-## Usage Notes
-
-- The `mode` parameter on `document_update` defaults to `append`.
-- Documents are automatically saved and accessible via the Generated panel.
-- Users can manually edit documents at any time.
-- Write in clear, engaging prose. Use active voice, vary sentence structure, and break content into logical sections with descriptive headings.

package/src/config/bundled-skills/document/TOOLS.json

@@ -1,106 +0,0 @@
-{
-  "version": 1,
-  "tools": [
-    {
-      "name": "document_create",
-      "description": "Create a new long-form document with a rich text editor. Use this when the user asks to write a blog post, article, or any long-form content. The editor opens in workspace mode with chat docked to the side.",
-      "category": "document",
-      "risk": "low",
-      "input_schema": {
-        "type": "object",
-        "properties": {
-          "title": {
-            "type": "string",
-            "description": "Initial title for the document (optional, can be updated later)"
-          },
-          "initial_content": {
-            "type": "string",
-            "description": "Initial Markdown content to populate the editor (optional)"
-          }
-        }
-      },
-      "executor": "tools/document-create.ts",
-      "execution_target": "host"
-    },
-    {
-      "name": "document_update",
-      "description": "Update content in an open document editor. Use this to stream generated content or apply edits.",
-      "category": "document",
-      "risk": "low",
-      "input_schema": {
-        "type": "object",
-        "properties": {
-          "surface_id": {
-            "type": "string",
-            "description": "The ID of the document surface to update"
-          },
-          "content": {
-            "type": "string",
-            "description": "Markdown content to set or append"
-          },
-          "mode": {
-            "type": "string",
-            "enum": ["replace", "append"],
-            "description": "Whether to replace all content or append to the end. Defaults to append."
-          }
-        },
-        "required": ["surface_id", "content"]
-      },
-      "executor": "tools/document-update.ts",
-      "execution_target": "host"
-    },
-    {
-      "name": "document_read",
-      "description": "Read the current content of a document by its surface_id when it belongs to the current conversation, or when the current actor is the guardian/local user. Use this to verify document state before making edits.",
-      "category": "document",
-      "risk": "low",
-      "input_schema": {
-        "type": "object",
-        "properties": {
-          "surface_id": {
-            "type": "string",
-            "description": "The ID of the document to read"
-          }
-        },
-        "required": ["surface_id"]
-      },
-      "executor": "tools/document-read.ts",
-      "execution_target": "host"
-    },
-    {
-      "name": "document_list",
-      "description": "List documents. Without a query, lists documents in the current conversation. With a query, searches documents by title; guardian/local users can search across conversations, while other actors are scoped to the current conversation.",
-      "category": "document",
-      "risk": "low",
-      "input_schema": {
-        "type": "object",
-        "properties": {
-          "query": {
-            "type": "string",
-            "description": "Search documents by title. Omit to list only the current conversation's documents."
-          }
-        }
-      },
-      "executor": "tools/document-list.ts",
-      "execution_target": "host"
-    },
-    {
-      "name": "document_delete",
-      "description": "Delete a document by its surface_id.",
-      "category": "document",
-      "risk": "low",
-      "input_schema": {
-        "type": "object",
-        "properties": {
-          "surface_id": {
-            "type": "string",
-            "description": "The ID of the document to delete"
-          }
-        },
-        "required": ["surface_id"]
-      },
-      "executor": "tools/document-delete.ts",
-      "execution_target": "host"
-    }
-  ]
-}

package/src/daemon/seed-files.ts

@@ -1,18 +0,0 @@
-import { existsSync, mkdirSync } from "node:fs";
-
-import { getLogger } from "../util/logger.js";
-import { getInterfacesDir } from "../util/platform.js";
-
-const log = getLogger("seed-files");
-
-/**
- * Ensures interface directories exist so the runtime can serve files
- * immediately. Called during daemon startup.
- */
-export function seedInterfaceFiles(): void {
-  const tuiDir = getInterfacesDir();
-  if (!existsSync(tuiDir)) {
-    mkdirSync(tuiDir, { recursive: true });
-    log.info("Created interfaces directory");
-  }
-}

package/src/memory/graph/__tests__/remember-description.test.ts

@@ -1,55 +0,0 @@
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-
-let flagEnabled = false;
-
-mock.module("../../../config/assistant-feature-flags.js", () => ({
-  isAssistantFeatureFlagEnabled: (_key: string, _config: unknown) =>
-    flagEnabled,
-}));
-
-import { getRememberDescription } from "../tools.js";
-
-const stubConfig = {} as unknown as Parameters<
-  typeof getRememberDescription
->[0];
-
-describe("getRememberDescription", () => {
-  beforeEach(() => {
-    flagEnabled = false;
-  });
-
-  test("flag off — returns the default high-pressure description", () => {
-    const desc = getRememberDescription(stubConfig);
-    expect(desc).toContain("**CRITICAL:**");
-    expect(desc).toContain("most frequently used tool");
-    expect(desc).toContain("almost every turn");
-  });
-
-  test("flag on — returns the relaxed judgment-framing description", () => {
-    flagEnabled = true;
-    const desc = getRememberDescription(stubConfig);
-    expect(desc).not.toContain("**CRITICAL:**");
-    expect(desc).not.toContain("almost every turn");
-    expect(desc).toContain("a retrospective pass");
-    expect(desc).toContain("Use judgment");
-  });
-
-  test("the two variants differ", () => {
-    flagEnabled = false;
-    const off = getRememberDescription(stubConfig);
-    flagEnabled = true;
-    const on = getRememberDescription(stubConfig);
-    expect(off).not.toBe(on);
-  });
-
-  test("corrections-are-priority language is preserved in BOTH variants", () => {
-    flagEnabled = false;
-    expect(getRememberDescription(stubConfig)).toMatch(
-      /Corrections are.*highest priority/i,
-    );
-    flagEnabled = true;
-    expect(getRememberDescription(stubConfig)).toMatch(
-      /Corrections are.*highest priority/i,
-    );
-  });
-});