@vellumai/assistant 0.5.7 → 0.5.9

package/Dockerfile

@@ -92,6 +92,7 @@ ENV IS_CONTAINERIZED=true
 
 # Copy from builder
 COPY --from=builder /app /app
+RUN chmod +x /app/assistant/docker-entrypoint.sh
 
 # Run the daemon + http server
-CMD ["bun", "run", "src/daemon/main.ts"]
+CMD ["/app/assistant/docker-entrypoint.sh"]

package/docker-entrypoint.sh

@@ -0,0 +1,9 @@
+#!/usr/bin/env sh
+set -eu
+
+if [ "$(id -u)" = "0" ] && [ "${VELLUM_WORKSPACE_DIR:-}" = "/workspace" ] && [ -d /workspace ]; then
+  git config --global --add safe.directory /workspace >/dev/null 2>&1 || true
+  git config --global --add safe.directory '/workspace/*' >/dev/null 2>&1 || true
+fi
+
+exec bun run src/daemon/main.ts

package/docs/architecture/memory.md

@@ -273,7 +273,7 @@ The key distinction: normal compaction is a cost-optimized background process th
 
 1. Run a recall-heavy turn and inspect `memory_recalled` events in the client trace stream.
 2. Validate baseline counters:
-   - `semanticHits`, `recencyHits`
+   - `semanticHits`
    - `tier1Count`, `tier2Count`
    - `hybridSearchLatencyMs`
    - `mergedCount`, `selectedCount`, `injectedTokens`, `latencyMs`
@@ -304,16 +304,18 @@ stateDiagram-v2
     Superseded --> Cleanup : cleanup_stale_superseded_items\n(delete from DB + Qdrant)
 ```
 
-**Item extraction** uses LLM-powered extraction (with pattern-based fallback) to identify memorable information from conversation messages. Each extracted item belongs to one of six kinds:
-
-| Kind         | Description                                       | Base Lifetime |
-| ------------ | ------------------------------------------------- | ------------- |
-| `identity`   | Personal info, facts, relationships               | 6 months      |
-| `preference` | Likes, dislikes, preferred approaches/tools       | 3 months      |
-| `constraint` | Rules, requirements, directives                   | 1 month       |
-| `project`    | Project details, repos, tech stacks, action items | 2 weeks       |
-| `decision`   | Choices made, approaches selected                 | 2 weeks       |
-| `event`      | Deadlines, milestones, meetings, dates            | 3 days        |
+**Item extraction** uses LLM-powered extraction (with pattern-based fallback) to identify memorable information from conversation messages. Each extracted item belongs to one of eight kinds:
+
+| Kind         | Description                                                        | Base Lifetime |
+| ------------ | ------------------------------------------------------------------ | ------------- |
+| `identity`   | Personal info, facts, relationships                                | 6 months      |
+| `preference` | Likes, dislikes, preferred approaches/tools                        | 3 months      |
+| `journal`    | Experiential reflections, journal-style notes, forward-looking items | 3 months      |
+| `constraint` | Rules, requirements, directives                                    | 1 month       |
+| `project`    | Project details, repos, tech stacks, action items                  | 2 weeks       |
+| `decision`   | Choices made, approaches selected                                  | 2 weeks       |
+| `event`      | Deadlines, milestones, meetings, dates                             | 3 days        |
+| `capability` | Skill catalog entries (system-generated, not LLM-extracted)        | never expires |
 
 **Supersession chains** replace the old conflict resolution system. When the LLM extracts a new item that updates an existing one, it sets `supersedes` to the old item's ID and `overrideConfidence` to one of three levels:
 

package/eslint.config.mjs

@@ -29,37 +29,6 @@ const eslintConfig = defineConfig([
         { argsIgnorePattern: "^_", varsIgnorePattern: "^_" },
       ],
       "@typescript-eslint/no-explicit-any": "error",
-
-      // Standardize on `undefined` only — avoid `null` in new code.
-      // Prefer `=== undefined`, `?? fallback`, or `?.` optional chaining
-      // instead of `=== null`.
-      "no-restricted-syntax": [
-        "error",
-        {
-          selector:
-            "BinaryExpression[operator='==='][right.type='Literal'][right.raw='null']",
-          message:
-            "Avoid `=== null`. Prefer `=== undefined`, `?? fallback`, or optional chaining `?.` instead.",
-        },
-        {
-          selector:
-            "BinaryExpression[operator='==='][left.type='Literal'][left.raw='null']",
-          message:
-            "Avoid `null ===`. Prefer `=== undefined`, `?? fallback`, or optional chaining `?.` instead.",
-        },
-        {
-          selector:
-            "BinaryExpression[operator='!=='][right.type='Literal'][right.raw='null']",
-          message:
-            "Avoid `!== null`. Prefer `!== undefined`, nullish coalescing `??`, or optional chaining `?.` instead.",
-        },
-        {
-          selector:
-            "BinaryExpression[operator='!=='][left.type='Literal'][left.raw='null']",
-          message:
-            "Avoid `null !==`. Prefer `!== undefined`, nullish coalescing `??`, or optional chaining `?.` instead.",
-        },
-      ],
     },
   },
   {

package/node_modules/@vellumai/ces-contracts/src/error.ts

@@ -3,7 +3,7 @@
  * dependencies between index.ts and rpc.ts.
  */
 
-import { z } from "zod/v4";
+import { z } from "zod";
 
 export const RpcErrorSchema = z.object({
   code: z.string(),

package/node_modules/@vellumai/ces-contracts/src/grants.ts

@@ -9,7 +9,7 @@
  * - Audit record summaries (materialization events)
  */
 
-import { z } from "zod/v4";
+import { z } from "zod";
 
 // ---------------------------------------------------------------------------
 // Grant proposal types

package/node_modules/@vellumai/ces-contracts/src/handles.ts

@@ -20,7 +20,7 @@
  *    is the platform-assigned connection identifier.
  */
 
-import { z } from "zod/v4";
+import { z } from "zod";
 
 // ---------------------------------------------------------------------------
 // Handle type discriminator

package/node_modules/@vellumai/ces-contracts/src/index.ts

@@ -7,7 +7,7 @@
  * module so that both sides can depend on it without circular references.
  */
 
-import { z } from "zod/v4";
+import { z } from "zod";
 import { RpcErrorSchema } from "./error.js";
 
 // ---------------------------------------------------------------------------

package/node_modules/@vellumai/ces-contracts/src/rpc.ts

@@ -33,7 +33,7 @@
  * - `list_credentials` — List all credential account names
  */
 
-import { z } from "zod/v4";
+import { z } from "zod";
 import {
   AuditRecordSummarySchema,
   GrantProposalSchema,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vellumai/assistant",
-  "version": "0.5.7",
+  "version": "0.5.9",
   "license": "MIT",
   "type": "module",
   "exports": {

package/src/__tests__/approval-cascade.test.ts

@@ -162,7 +162,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),

package/src/__tests__/browser-fill-credential.test.ts

@@ -61,7 +61,7 @@ mock.module("../security/secure-keys.js", () => ({
   getSecureKeyAsync: async (...args: unknown[]) => mockGetSecureKey(...args),
   setSecureKeyAsync: async () => true,
   deleteSecureKeyAsync: async () => "deleted",
-  listSecureKeysAsync: async () => [],
+  listSecureKeysAsync: async () => ({ accounts: [], unreachable: false }),
   _resetBackend: () => {},
 }));
 

package/src/__tests__/call-controller.test.ts

@@ -42,7 +42,6 @@ mock.module("../config/loader.js", () => {
     ui: {},
 
     provider: "anthropic",
-    providerOrder: ["anthropic"],
     calls: {
       enabled: true,
       provider: "twilio",

package/src/__tests__/ces-rpc-credential-backend.test.ts

@@ -185,15 +185,15 @@ describe("CesRpcCredentialBackend", () => {
       const result = await backend.list();
 
       expect(callFn).toHaveBeenCalledWith(CesRpcMethod.ListCredentials, {});
-      expect(result).toEqual(["account-a", "account-b"]);
+      expect(result).toEqual({ accounts: ["account-a", "account-b"], unreachable: false });
     });
 
-    test("returns empty array when RPC call throws", async () => {
+    test("returns unreachable when RPC call throws", async () => {
       callFn.mockRejectedValue(new Error("transport error"));
 
       const result = await backend.list();
 
-      expect(result).toEqual([]);
+      expect(result).toEqual({ accounts: [], unreachable: true });
     });
   });
 });

package/src/__tests__/ces-startup-timeout.test.ts

@@ -0,0 +1,40 @@
+import { describe, expect, mock, test } from "bun:test";
+
+import type { CesClient } from "../credential-execution/client.js";
+import {
+  awaitCesClientWithTimeout,
+  DEFAULT_CES_STARTUP_TIMEOUT_MS,
+} from "../credential-execution/startup-timeout.js";
+
+describe("awaitCesClientWithTimeout", () => {
+  test("clears the fallback timer when the CES client resolves first", async () => {
+    const onTimeout = mock(() => {});
+    const client = { isReady: () => true } as unknown as CesClient;
+
+    const result = await awaitCesClientWithTimeout(Promise.resolve(client), {
+      timeoutMs: 25,
+      onTimeout,
+    });
+
+    expect(result).toBe(client);
+
+    await new Promise((resolve) => setTimeout(resolve, 40));
+    expect(onTimeout).not.toHaveBeenCalled();
+  });
+
+  test("returns undefined and runs the fallback handler when the timeout wins", async () => {
+    const onTimeout = mock(() => {});
+
+    const result = await awaitCesClientWithTimeout(new Promise(() => {}), {
+      timeoutMs: 10,
+      onTimeout,
+    });
+
+    expect(result).toBeUndefined();
+    expect(onTimeout).toHaveBeenCalledTimes(1);
+  });
+
+  test("exports the daemon CES startup timeout constant", () => {
+    expect(DEFAULT_CES_STARTUP_TIMEOUT_MS).toBe(20_000);
+  });
+});

package/src/__tests__/config-schema-cmd.test.ts

@@ -204,7 +204,6 @@ describe("z.toJSONSchema integration", () => {
     expect(properties).toBeDefined();
     // Check that top-level keys are present
     expect(properties.services).toBeDefined();
-    expect(properties.providerOrder).toBeDefined();
     expect(properties.maxTokens).toBeDefined();
     expect(properties.calls).toBeDefined();
     expect(properties.memory).toBeDefined();

package/src/__tests__/config-schema.test.ts

@@ -638,6 +638,8 @@ describe("AssistantConfigSchema", () => {
         language: "en-US",
         transcriptionProvider: "Deepgram",
         ttsProvider: "elevenlabs",
+        hints: [],
+        interruptSensitivity: "low",
       },
       callerIdentity: {
         allowPerCallOverride: true,

package/src/__tests__/conversation-abort-tool-results.test.ts

@@ -129,7 +129,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),

package/src/__tests__/conversation-agent-loop-overflow.test.ts

@@ -168,7 +168,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),
@@ -199,7 +198,6 @@ mock.module("../daemon/conversation-memory.js", () => ({
       injectedText: "",
 
       semanticHits: 0,
-      recencyHits: 0,
       injectedTokens: 0,
       latencyMs: 0,
       tier1Count: 0,

package/src/__tests__/conversation-agent-loop.test.ts

@@ -158,7 +158,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),
@@ -189,7 +188,6 @@ mock.module("../daemon/conversation-memory.js", () => ({
       injectedText: "",
 
       semanticHits: 0,
-      recencyHits: 0,
       injectedTokens: 0,
       latencyMs: 0,
       tier1Count: 0,
@@ -614,7 +612,7 @@ describe("session-agent-loop", () => {
   });
 
   describe("LLM request log persistence", () => {
-    test("record request log prefers the actual provider from failover", async () => {
+    test("record request log captures the actual provider name", async () => {
       const events: ServerMessage[] = [];
       const rawRequest = {
         model: "gpt-4.1",
@@ -768,7 +766,7 @@ describe("session-agent-loop", () => {
   });
 
   describe("usage accounting", () => {
-    test("records the actual provider for failover-served usage", async () => {
+    test("records the actual provider for usage accounting", async () => {
       const events: ServerMessage[] = [];
 
       const agentLoopRun: AgentLoopRun = async (messages, onEvent) => {

package/src/__tests__/conversation-confirmation-signals.test.ts

@@ -154,7 +154,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),

package/src/__tests__/conversation-error.test.ts

@@ -6,7 +6,7 @@ import {
   classifyConversationError,
   isUserCancellation,
 } from "../daemon/conversation-error.js";
-import { ProviderError } from "../util/errors.js";
+import { ProviderError, ProviderNotConfiguredError } from "../util/errors.js";
 
 describe("isUserCancellation", () => {
   it("returns false for non-AbortError even when abort flag is set", () => {
@@ -278,6 +278,20 @@ describe("classifyConversationError", () => {
     });
   });
 
+  describe("provider not configured errors", () => {
+    it("classifies ProviderNotConfiguredError as PROVIDER_NOT_CONFIGURED", () => {
+      const err = new ProviderNotConfiguredError("anthropic", []);
+      const result = classifyConversationError(err, baseCtx);
+      expect(result.code).toBe("PROVIDER_NOT_CONFIGURED");
+      expect(result.userMessage).toBe(
+        "No API key configured for inference. Add one in Settings to start chatting.",
+      );
+      expect(result.retryable).toBe(true);
+      expect(result.errorCategory).toBe("provider_not_configured");
+      expect(result.debugDetails).toBeDefined();
+    });
+  });
+
   describe("streaming corruption errors", () => {
     const cases = [
       "Unexpected event order, got message_start before receiving message_stop",

package/src/__tests__/conversation-messaging-secret-redirect.test.ts

@@ -28,7 +28,7 @@ mock.module("../security/secure-keys.js", () => ({
   setSecureKeyAsync: async (key?: string, value?: string) =>
     setSecureKeyMock(key, value),
   deleteSecureKeyAsync: async () => "deleted" as const,
-  listSecureKeysAsync: async () => [],
+  listSecureKeysAsync: async () => ({ accounts: [], unreachable: false }),
   _resetBackend: () => {},
 }));
 

package/src/__tests__/conversation-pre-run-repair.test.ts

@@ -119,7 +119,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),

package/src/__tests__/conversation-provider-retry-repair.test.ts

@@ -188,7 +188,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),

package/src/__tests__/conversation-queue.test.ts

@@ -204,7 +204,6 @@ mock.module("../memory/retriever.js", () => ({
     injectedText: "",
 
     semanticHits: 0,
-    recencyHits: 0,
     injectedTokens: 0,
     latencyMs: 0,
   }),