@vellumai/assistant 0.8.4 → 0.8.6

package/src/daemon/daemon-control.ts

@@ -33,7 +33,7 @@ const DAEMON_TIMEOUT_DEFAULTS = {
 };
 
 const HEALTH_CHECK_TIMEOUT_MS = 1500;
-const STARTUP_LOCK_STALE_MS = 30_000;
+const STARTUP_LOCK_STALE_MS = 120_000;
 
 function isPositiveInteger(v: unknown): v is number {
   return typeof v === "number" && Number.isInteger(v) && v > 0;

package/src/daemon/daemon-skill-host.ts

@@ -219,8 +219,15 @@ function buildRegistriesFacet(skillId: string): RegistriesFacet {
     // overlay (`assistant/src/tools/types.ts`); the assistant-side
     // registry accepts the daemon flavor. Skills construct tools via
     // helpers that already produce the daemon shape, so a cast at this
-    // boundary is safe.
-    registerTools: (provider) => registerExternalTools(provider as never),
+    // boundary is safe. The contract's `registerTools(provider)` stays
+    // single-arg — skill code never needs to know its own id — and this
+    // adapter pairs the provider with the owner derived from the
+    // surrounding {@link buildRegistriesFacet} closure.
+    registerTools: (provider) =>
+      registerExternalTools(
+        { kind: "skill", id: skillId },
+        provider as never,
+      ),
     registerSkillRoute: (route: SkillRoute): SkillRouteHandle =>
       registerSkillRoute(route) as unknown as SkillRouteHandle,
     // Namespace hook names by skillId so two skills using the same label

package/src/daemon/disk-pressure-guard.ts

@@ -1,12 +1,17 @@
-import { isAssistantFeatureFlagEnabled } from "../config/assistant-feature-flags.js";
-import { getConfig } from "../config/loader.js";
 import { buildAssistantEvent } from "../runtime/assistant-event.js";
 import { assistantEventHub } from "../runtime/assistant-event-hub.js";
 import { cancelBackgroundTools } from "../tools/background-tool-registry.js";
 import { getDiskUsageInfo } from "../util/disk-usage.js";
 import { getLogger } from "../util/logger.js";
 
+export const DISK_PRESSURE_WARNING_THRESHOLD_PERCENT = 80;
 export const DISK_PRESSURE_THRESHOLD_PERCENT = 95;
+// Hysteresis lower bound: once locked, the guard stays locked until usage
+// falls below this clear threshold. The deadband between this and the
+// critical threshold stops the lock from flapping when usage hovers near
+// 95% — otherwise clearing the lock immediately resumes background work,
+// which can refill the disk and re-trip the lock on the next sample.
+export const DISK_PRESSURE_CLEAR_THRESHOLD_PERCENT = 90;
 export const DISK_PRESSURE_CHECK_INTERVAL_MS = 60_000;
 export const DISK_PRESSURE_OVERRIDE_CONFIRMATION = "I understand the risks";
 export const DISK_PRESSURE_BLOCKED_CAPABILITIES = [
@@ -15,7 +20,12 @@ export const DISK_PRESSURE_BLOCKED_CAPABILITIES = [
   "remote-ingress",
 ] as const;
 
-export type DiskPressureState = "disabled" | "ok" | "critical" | "unknown";
+export type DiskPressureState =
+  | "disabled"
+  | "ok"
+  | "warning"
+  | "critical"
+  | "unknown";
 
 export type DiskPressureBlockedCapability =
   (typeof DISK_PRESSURE_BLOCKED_CAPABILITIES)[number];
@@ -118,24 +128,10 @@ function replaceStatus(next: DiskPressureStatus): DiskPressureStatus {
   return cloneStatus(state.status);
 }
 
-function isEnabled(): boolean {
-  return isAssistantFeatureFlagEnabled("safe-storage-limits", getConfig());
-}
-
-function resetToDisabled(): DiskPressureStatus {
-  const previous = cloneStatus(state.status);
-  stopDiskPressureGuard();
-  state.status = cloneStatus(DISABLED_STATUS);
-  publishStatusChangedIfNeeded(previous);
-  return cloneStatus(state.status);
-}
-
-function ensureEnabledStatus(): DiskPressureStatus | null {
-  if (!isEnabled()) return resetToDisabled();
+function ensureEnabledStatus(): void {
   if (!state.status.enabled) {
     state.status = cloneStatus(OPEN_STATUS);
   }
-  return null;
 }
 
 function nextLockId(): string {
@@ -191,8 +187,7 @@ function rejectTransition(
 }
 
 export function startDiskPressureGuard(): DiskPressureStatus {
-  const disabledStatus = ensureEnabledStatus();
-  if (disabledStatus) return disabledStatus;
+  ensureEnabledStatus();
 
   if (!state.timer) {
     state.timer = setInterval(() => {
@@ -211,8 +206,7 @@ export function stopDiskPressureGuard(): void {
 }
 
 export function evaluateDiskPressureNow(): DiskPressureStatus {
-  const disabledStatus = ensureEnabledStatus();
-  if (disabledStatus) return disabledStatus;
+  ensureEnabledStatus();
 
   let usageInfo: ReturnType<typeof getDiskUsageInfo>;
   try {
@@ -228,10 +222,17 @@ export function evaluateDiskPressureNow(): DiskPressureStatus {
   const usagePercent = roundPercent(
     (usageInfo.usedMb / usageInfo.totalMb) * 100,
   );
-  const isCritical = usagePercent >= DISK_PRESSURE_THRESHOLD_PERCENT;
+  // Hysteresis: while locked, hold until usage drops below the lower clear
+  // threshold; otherwise lock at the critical threshold.
+  const criticalThreshold = state.status.locked
+    ? DISK_PRESSURE_CLEAR_THRESHOLD_PERCENT
+    : DISK_PRESSURE_THRESHOLD_PERCENT;
+  const isCritical = usagePercent >= criticalThreshold;
+  const isWarning =
+    !isCritical && usagePercent >= DISK_PRESSURE_WARNING_THRESHOLD_PERCENT;
   const lastCheckedAt = new Date().toISOString();
 
-  if (!isCritical) {
+  if (!isCritical && !isWarning) {
     return replaceStatus({
       ...OPEN_STATUS,
       usagePercent,
@@ -240,6 +241,16 @@ export function evaluateDiskPressureNow(): DiskPressureStatus {
     });
   }
 
+  if (isWarning) {
+    return replaceStatus({
+      ...OPEN_STATUS,
+      state: "warning",
+      usagePercent,
+      path: usageInfo.path,
+      lastCheckedAt,
+    });
+  }
+
   if (!state.status.locked) {
     cancelTerminalBackgroundToolsForLock();
   }
@@ -265,14 +276,13 @@ export function evaluateDiskPressureNow(): DiskPressureStatus {
 }
 
 export function getDiskPressureStatus(): DiskPressureStatus {
-  if (!isEnabled()) return cloneStatus(DISABLED_STATUS);
   if (!state.status.enabled) return cloneStatus(OPEN_STATUS);
   return cloneStatus(state.status);
 }
 
 export function acknowledgeDiskPressureLock(): DiskPressureTransitionResult {
-  const disabledStatus = ensureEnabledStatus();
-  const status = disabledStatus ?? cloneStatus(state.status);
+  ensureEnabledStatus();
+  const status = cloneStatus(state.status);
   if (!status.locked) {
     return rejectTransition(
       "not_locked",
@@ -298,8 +308,8 @@ export function acknowledgeDiskPressureLock(): DiskPressureTransitionResult {
 export function overrideDiskPressureLock(
   confirmation: string,
 ): DiskPressureTransitionResult {
-  const disabledStatus = ensureEnabledStatus();
-  const status = disabledStatus ?? cloneStatus(state.status);
+  ensureEnabledStatus();
+  const status = cloneStatus(state.status);
   if (!status.locked) {
     return rejectTransition(
       "not_locked",

package/src/daemon/first-greeting.ts

@@ -14,12 +14,6 @@ export interface OnboardingGreetingContext {
   googleConnected?: boolean;
 }
 
-export const CANNED_FIRST_GREETING = [
-  "Hey,",
-  "",
-  "We can get into whatever you've got, or just talk first — that tends to go better. Up to you.",
-].join("\n");
-
 /**
  * Returns `true` when all of the following are true:
  * - `conversationMessageCount === 0` (no prior messages in this conversation)
@@ -67,30 +61,54 @@ function buildIntroLine(
   return [greeting, who, close].filter(Boolean).join(" ");
 }
 
-const TONE_INVITE: Record<Tone, string> = {
+// Every greeting variant — the no-onboarding CANNED greeting and all four
+// personalized tones — ends with a migration offer. The opener (task-vs-talk)
+// and the migration offer are kept as separate per-tone maps and composed in
+// `buildInvite`, rather than inlined as one full sentence per variant, so the
+// offer cannot be silently dropped from a variant when the opener copy is
+// edited. The first-greeting test asserts every variant still includes it.
+const TONE_INVITE_OPENER: Record<Tone, string> = {
   grounded:
     "We can get into whatever you've got, or just talk first — that tends to go better. Up to you.",
   warm: "We can start on something specific, or just talk for a bit first — honestly that tends to work out better. Either way, I'm here.",
   energetic:
-    "We can jump straight into whatever you've got, or take a few minutes to just talk first. What sounds right?",
+    "We can jump straight into whatever you've got, or take a few minutes to just talk first.",
   poetic:
     "We can start with whatever's in front of you, or just talk for a bit first. Either way.",
 };
 
+const TONE_MIGRATION_OFFER: Record<Tone, string> = {
+  grounded:
+    "If you have context or workflows from another assistant or harness, bring them over early and I'll help port them.",
+  warm: "If you have context or workflows from another assistant or harness, bring them over early and I can help port them.",
+  energetic:
+    "If you've got context or workflows from another assistant or harness, bring them over early and I'll port them with you. What sounds right?",
+  poetic:
+    "If there's old context or workflows from another assistant or harness, bring them over early and I'll help port them.",
+};
+
 const TONE_GOOGLE_SCAN: Record<Tone, string> = {
   grounded:
-    "I can scan your email, calendar, and drive in the background while we talk — just say the word.",
-  warm: "Also — I can scan your email, calendar, and drive in the background while we chat, if you'd like. Just let me know.",
+    "I can scan Gmail in the background while we talk — just say the word.",
+  warm: "Also — I can scan Gmail in the background while we chat, if you'd like. Just let me know.",
   energetic:
-    "Oh, and I can scan your email, calendar, and drive in the background right now — want me to?",
+    "Oh, and I can scan Gmail in the background right now — want me to?",
   poetic:
-    "I can also look through your email, calendar, and drive quietly in the background — say the word.",
+    "I can also look through Gmail quietly in the background — say the word.",
 };
 
 function buildInvite(tone: Tone = "grounded"): string {
-  return TONE_INVITE[tone];
+  return `${TONE_INVITE_OPENER[tone]} ${TONE_MIGRATION_OFFER[tone]}`;
 }
 
+// Composed from the grounded opener + migration offer so the no-onboarding
+// greeting reuses the same source as the personalized grounded greeting rather
+// than duplicating the copy. Defined after the tone maps so they are
+// initialized before this module-level evaluation runs.
+export const CANNED_FIRST_GREETING = ["Hey,", "", buildInvite("grounded")].join(
+  "\n",
+);
+
 const VALID_TONES = new Set<string>([
   "grounded",
   "warm",

package/src/daemon/handlers/config-model.test.ts

@@ -70,6 +70,7 @@ describe("projectProviderForWire", () => {
       "gemini-3.1-pro-preview-customtools",
       "gemini-3-flash-preview",
       "gemini-3.1-flash-lite-preview",
+      "gemini-3.1-flash-lite",
     ];
 
     expect(modelIds.filter((id) => id.startsWith("gemini-3"))).toEqual(

package/src/daemon/handlers/conversations.ts

@@ -36,7 +36,15 @@ export function handleConfirmationResponse(msg: ConfirmationResponse): void {
         undefined,
         { source: "button" },
       );
-      pendingInteractions.resolve(msg.requestId);
+      // Idempotent cleanup: the prompter's `resolveConfirmation` already
+      // resolved the interaction with the correct approved/rejected state
+      // before we reach here, so the entry is typically gone. This call is
+      // a safety net for paths where the prompter no-ops; map decision to
+      // state to keep the emitted event accurate when it does fire.
+      pendingInteractions.resolve(
+        msg.requestId,
+        decision === "allow" ? "approved" : "rejected",
+      );
       return;
     }
   }
@@ -49,13 +57,13 @@ export function handleConfirmationResponse(msg: ConfirmationResponse): void {
 /**
  * Clear all conversations and DB conversations. Returns the number of conversations cleared.
  */
-export function clearAllConversations(): number {
+export async function clearAllConversations(): Promise<number> {
   const cleared = clearAllActiveConversations();
   // Also clear DB conversations. When a new local connection triggers
   // sendInitialConversation, it auto-creates a conversation if none exist.
   // Without this DB clear, that auto-created row survives, contradicting
   // the "clear all conversations" intent.
-  clearAll();
+  await clearAll();
   return cleared;
 }
 

package/src/daemon/handlers/shared.ts

@@ -85,7 +85,12 @@ export interface HistorySurface {
   surfaceType: string;
   title?: string;
   data: Record<string, unknown>;
-  actions?: Array<{ id: string; label: string; style?: string }>;
+  actions?: Array<{
+    id: string;
+    label: string;
+    style?: string;
+    data?: Record<string, unknown>;
+  }>;
   display?: string;
   persistent?: boolean;
   completed?: boolean;

package/src/daemon/host-browser-proxy.ts

@@ -185,7 +185,7 @@ export class HostBrowserProxy {
       let detachAbort: () => void = () => {};
 
       const timer = setTimeout(() => {
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, cdpMethod: input.cdpMethod },
           "Host browser proxy request timed out",
@@ -200,7 +200,7 @@ export class HostBrowserProxy {
       if (signal) {
         const onAbort = () => {
           if (pendingInteractions.get(requestId)) {
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             try {
               broadcastMessage({
                 type: "host_browser_cancel",
@@ -229,7 +229,7 @@ export class HostBrowserProxy {
 
       try {
         if (!preferredClient) {
-          pendingInteractions.resolve(requestId);
+          pendingInteractions.resolve(requestId, "cancelled");
           reject(
             new Error(
               "host_browser send failed: no active extension connection",
@@ -244,7 +244,7 @@ export class HostBrowserProxy {
           { targetClientId: preferredClient.clientId },
         );
       } catch (err) {
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, cdpMethod: input.cdpMethod, err },
           "Host browser proxy send failed",
@@ -256,7 +256,7 @@ export class HostBrowserProxy {
 
   dispose(): void {
     for (const entry of pendingInteractions.getByKind("host_browser")) {
-      pendingInteractions.resolve(entry.requestId);
+      pendingInteractions.resolve(entry.requestId, "cancelled");
       try {
         broadcastMessage({
           type: "host_browser_cancel",

package/src/daemon/host-cu-proxy.ts

@@ -195,7 +195,7 @@ export class HostCuProxy {
 
       const timer = setTimeout(() => {
         this._ownedRequests.delete(requestId);
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn({ requestId, toolName }, "Host CU proxy request timed out");
         resolve({
           content: "Host CU proxy timed out waiting for client response",
@@ -207,7 +207,7 @@ export class HostCuProxy {
         const onAbort = () => {
           if (pendingInteractions.get(requestId)) {
             this._ownedRequests.delete(requestId);
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             try {
               broadcastMessage(
                 {
@@ -263,7 +263,7 @@ export class HostCuProxy {
         );
       } catch (err) {
         this._ownedRequests.delete(requestId);
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn({ requestId, toolName, err }, "Host CU proxy send failed");
         reject(err instanceof Error ? err : new Error(String(err)));
       }
@@ -447,7 +447,7 @@ export class HostCuProxy {
 
   dispose(): void {
     for (const requestId of this._ownedRequests) {
-      const entry = pendingInteractions.resolve(requestId);
+      const entry = pendingInteractions.resolve(requestId, "cancelled");
       if (!entry) continue;
       try {
         broadcastMessage(

package/src/daemon/host-file-proxy.ts

@@ -136,7 +136,7 @@ export class HostFileProxy {
       let detachAbort: () => void = () => {};
 
       const timer = setTimeout(() => {
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, operation: input.operation },
           "Host file proxy request timed out",
@@ -152,7 +152,7 @@ export class HostFileProxy {
       if (signal) {
         const onAbort = () => {
           if (pendingInteractions.get(requestId)) {
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             try {
               broadcastMessage(
                 {
@@ -210,7 +210,7 @@ export class HostFileProxy {
           { targetClientId: resolvedTargetClientId },
         );
       } catch (err) {
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, operation: input.operation, err },
           "Host file proxy send failed",
@@ -252,7 +252,7 @@ export class HostFileProxy {
 
   dispose(): void {
     for (const entry of pendingInteractions.getByKind("host_file")) {
-      pendingInteractions.resolve(entry.requestId);
+      pendingInteractions.resolve(entry.requestId, "cancelled");
       try {
         broadcastMessage(
           {

package/src/daemon/host-proxy-base.ts

@@ -154,7 +154,7 @@ export abstract class HostProxyBase<TRequest, TResultPayload> {
       const timer = setTimeout(() => {
         this.pending.delete(requestId);
         detachAbort();
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, toolName, kind: this.resultPendingKind },
           "Host proxy request timed out",
@@ -168,7 +168,7 @@ export abstract class HostProxyBase<TRequest, TResultPayload> {
             clearTimeout(timer);
             this.pending.delete(requestId);
             detachAbort();
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             try {
               broadcastDynamic(
                 {
@@ -239,7 +239,7 @@ export abstract class HostProxyBase<TRequest, TResultPayload> {
         clearTimeout(timer);
         this.pending.delete(requestId);
         detachAbort();
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, toolName, kind: this.resultPendingKind, err },
           "Host proxy send failed",
@@ -285,7 +285,7 @@ export abstract class HostProxyBase<TRequest, TResultPayload> {
     for (const [requestId, entry] of this.pending) {
       clearTimeout(entry.timer);
       entry.detachAbort();
-      pendingInteractions.resolve(requestId);
+      pendingInteractions.resolve(requestId, "cancelled");
       try {
         broadcastDynamic(
           {

package/src/daemon/host-transfer-proxy.ts

@@ -205,7 +205,7 @@ export class HostTransferProxy {
 
           const timer = setTimeout(() => {
             this.transfers.delete(transferId);
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             log.warn(
               { requestId, transferId, direction: "to_host" },
               "Host transfer proxy request timed out",
@@ -222,7 +222,7 @@ export class HostTransferProxy {
             const onAbort = () => {
               if (pendingInteractions.get(requestId)) {
                 this.transfers.delete(transferId);
-                pendingInteractions.resolve(requestId);
+                pendingInteractions.resolve(requestId, "cancelled");
                 try {
                   broadcastMessage(
                     {
@@ -301,7 +301,7 @@ export class HostTransferProxy {
             );
           } catch (err) {
             this.transfers.delete(transferId);
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             log.warn(
               { requestId, transferId, err },
               "Host transfer proxy send failed",
@@ -396,7 +396,7 @@ export class HostTransferProxy {
 
       const timer = setTimeout(() => {
         this.transfers.delete(transferId);
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, transferId, direction: "to_sandbox" },
           "Host transfer proxy request timed out",
@@ -413,7 +413,7 @@ export class HostTransferProxy {
         const onAbort = () => {
           if (pendingInteractions.get(requestId)) {
             this.transfers.delete(transferId);
-            pendingInteractions.resolve(requestId);
+            pendingInteractions.resolve(requestId, "cancelled");
             try {
               broadcastMessage(
                 {
@@ -487,7 +487,7 @@ export class HostTransferProxy {
         );
       } catch (err) {
         this.transfers.delete(transferId);
-        pendingInteractions.resolve(requestId);
+        pendingInteractions.resolve(requestId, "cancelled");
         log.warn(
           { requestId, transferId, err },
           "Host transfer proxy send failed",
@@ -622,14 +622,14 @@ export class HostTransferProxy {
 
     if (entry.overwrite !== true && existsSync(entry.filePath)) {
       const errorMsg = `Destination file already exists: ${entry.filePath}. Set overwrite to true to replace it.`;
-      const interaction = pendingInteractions.resolve(requestId);
+      const interaction = pendingInteractions.resolve(requestId, "answered");
       this.transfers.delete(transferId);
       interaction?.rpcResolve?.({ content: errorMsg, isError: true });
       return { accepted: false, error: errorMsg };
     }
 
     const cleanup = () => {
-      pendingInteractions.resolve(requestId);
+      pendingInteractions.resolve(requestId, "answered");
       this.transfers.delete(transferId);
     };
 
@@ -662,7 +662,7 @@ export class HostTransferProxy {
     if (!interaction) return;
     const transferId = interaction.metadata?.transferId as string | undefined;
     if (transferId) this.transfers.delete(transferId);
-    pendingInteractions.resolve(requestId);
+    pendingInteractions.resolve(requestId, "cancelled");
     try {
       broadcastMessage(
         {
@@ -718,7 +718,7 @@ export class HostTransferProxy {
     for (const entry of pendingInteractions.getByKind("host_transfer")) {
       const transferId = entry.metadata?.transferId as string | undefined;
       if (transferId) this.transfers.delete(transferId);
-      pendingInteractions.resolve(entry.requestId);
+      pendingInteractions.resolve(entry.requestId, "cancelled");
       try {
         broadcastMessage(
           {