@vellumai/assistant 0.10.8-dev.202607130516.157ae8a → 0.10.8-dev.202607131107.073396f

package/openapi.yaml

@@ -17607,7 +17607,7 @@ paths:
     post:
       operationId: memory_worker_start_post
       summary: Start the memory worker
-      description: Spawns (or reuses) the memory worker process as a child of the daemon and enables memory.worker.enabled.
+      description: Spawns (or reuses) the memory worker process as a child of the daemon.
       tags:
         - system
       responses:
@@ -17622,15 +17622,11 @@ paths:
                     type: number
                   alreadyRunning:
                     type: boolean
-                  workerEnabled:
-                    type: boolean
-                    const: true
                   pidPath:
                     type: string
                 required:
                   - pid
                   - alreadyRunning
-                  - workerEnabled
                   - pidPath
                 additionalProperties: false
   /v1/memory/worker/status:
@@ -17638,8 +17634,8 @@ paths:
       operationId: memory_worker_status_get
       summary: Memory worker status
       description:
-        Reports the memory worker process state, memory.worker.enabled, the synchronous in-process runner, and the
-        embedding-backend status (including a degraded flag and reason when no backend resolves).
+        Reports the memory worker process liveness and the embedding-backend status (including a degraded flag and
+        reason when no backend resolves).
       tags:
         - system
       responses:
@@ -17657,21 +17653,6 @@ paths:
                       - not_running
                   pid:
                     type: number
-                  workerEnabled:
-                    type: boolean
-                  syncRunner:
-                    type: object
-                    properties:
-                      status:
-                        type: string
-                        enum:
-                          - running
-                          - not_running
-                      pid:
-                        type: number
-                    required:
-                      - status
-                    additionalProperties: false
                   embedding:
                     type: object
                     properties:
@@ -17705,15 +17686,13 @@ paths:
                     additionalProperties: false
                 required:
                   - status
-                  - workerEnabled
-                  - syncRunner
                   - embedding
                 additionalProperties: false
   /v1/memory/worker/stop:
     post:
       operationId: memory_worker_stop_post
       summary: Stop the memory worker
-      description: Disables memory.worker.enabled and SIGTERMs the memory worker process if it is running.
+      description: SIGTERMs the memory worker process if it is running.
       tags:
         - system
       responses:
@@ -17728,12 +17707,8 @@ paths:
                     type: boolean
                   pid:
                     type: number
-                  workerEnabled:
-                    type: boolean
-                    const: false
                 required:
                   - workerWasRunning
-                  - workerEnabled
                 additionalProperties: false
   /v1/messages:
     get:
@@ -25522,9 +25497,7 @@ paths:
     post:
       operationId: schedules_worker_start_post
       summary: Start the schedule worker
-      description:
-        Spawns (or reuses) the schedule worker process as a child of the daemon and enables
-        schedules.worker.enabled, so scheduled jobs run out of process.
+      description: Spawns (or reuses) the schedule worker process as a child of the daemon.
       tags:
         - system
       responses:
@@ -25539,24 +25512,18 @@ paths:
                     type: number
                   alreadyRunning:
                     type: boolean
-                  workerEnabled:
-                    type: boolean
-                    const: true
                   pidPath:
                     type: string
                 required:
                   - pid
                   - alreadyRunning
-                  - workerEnabled
                   - pidPath
                 additionalProperties: false
   /v1/schedules/worker/status:
     get:
       operationId: schedules_worker_status_get
       summary: Schedule worker status
-      description:
-        Reports the schedule worker process state, schedules.worker.enabled, and whether the daemon's in-process
-        scheduler is currently executing schedules.
+      description: Reports the schedule worker process liveness.
       tags:
         - system
       responses:
@@ -25574,33 +25541,14 @@ paths:
                       - not_running
                   pid:
                     type: number
-                  workerEnabled:
-                    type: boolean
-                  inProcessScheduler:
-                    type: object
-                    properties:
-                      status:
-                        type: string
-                        enum:
-                          - running
-                          - not_running
-                      pid:
-                        type: number
-                    required:
-                      - status
-                    additionalProperties: false
                 required:
                   - status
-                  - workerEnabled
-                  - inProcessScheduler
                 additionalProperties: false
   /v1/schedules/worker/stop:
     post:
       operationId: schedules_worker_stop_post
       summary: Stop the schedule worker
-      description:
-        Disables schedules.worker.enabled (handing schedule execution back to the in-process scheduler) and
-        SIGTERMs the schedule worker process if it is running.
+      description: SIGTERMs the schedule worker process if it is running.
       tags:
         - system
       responses:
@@ -25615,12 +25563,8 @@ paths:
                     type: boolean
                   pid:
                     type: number
-                  workerEnabled:
-                    type: boolean
-                    const: false
                 required:
                   - workerWasRunning
-                  - workerEnabled
                 additionalProperties: false
   /v1/search:
     get:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vellumai/assistant",
-  "version": "0.10.8-dev.202607130516.157ae8a",
+  "version": "0.10.8-dev.202607131107.073396f",
   "license": "MIT",
   "type": "module",
   "exports": {

package/src/__tests__/memory-jobs-worker-backoff.test.ts

@@ -31,14 +31,12 @@ mock.module("../persistence/checkpoints.js", () => ({
 import {
   POLL_INTERVAL_MAX_MS,
   POLL_INTERVAL_MIN_MS,
-  startMemoryJobsWorker,
+  startMemoryJobsWorkerLoop,
 } from "../plugins/defaults/memory/jobs-worker.js";
 import { setConfig } from "./helpers/set-config.js";
 
-// Memory disabled so each idle tick stays on the mocked-store fast path, and
-// the worker flag off so the supervisor neither spawns the out-of-process
-// worker nor stands the in-process runner down at the slow-poll cap.
-setConfig("memory", { enabled: false, worker: { enabled: false } });
+// Memory disabled so each idle tick stays on the mocked-store fast path.
+setConfig("memory", { enabled: false });
 
 describe("memory jobs worker adaptive poll interval", () => {
   test("exports expected poll interval constants", () => {
@@ -86,7 +84,7 @@ describe("memory jobs worker adaptive poll interval", () => {
     globalThis.clearTimeout = (() => {}) as typeof clearTimeout;
 
     try {
-      const worker = startMemoryJobsWorker();
+      const worker = startMemoryJobsWorkerLoop();
 
       // Wait for the initial tick() promise to settle
       await new Promise((resolve) => originalSetTimeout(resolve, 20));

package/src/__tests__/plugin-import-boundary-guard.test.ts

@@ -135,6 +135,8 @@ const BASELINE: Record<string, readonly string[]> = {
     "../../../../skills/catalog-cache.js",
     "../../../../telemetry/watchdog-events-store.js",
     "../../../../tools/skills/delete-managed.js",
+    "../../../../util/logger.js",
+    "../../../../util/platform.js",
     "../../../channels/types.js",
     "../../../config/loader.js",
     "../../../config/memory-v3-gate.js",
@@ -180,7 +182,6 @@ const BASELINE: Record<string, readonly string[]> = {
     "../../../persistence/jobs-store.js",
     "../../../persistence/raw-query.js",
     "../../../persistence/schema/index.js",
-    "../../../persistence/worker-control.js",
     "../../../prompts/persona-resolver.js",
     "../../../prompts/system-prompt.js",
     "../../../prompts/template-detection.js",
@@ -205,6 +206,7 @@ const BASELINE: Record<string, readonly string[]> = {
     "../../../util/strip-comment-lines.js",
     "../../../util/truncate.js",
     "../../../util/worker-memory.js",
+    "../../../util/worker-process.js",
     "../../types.js",
     "../injection-presence.js",
     "../injector-order.js",

package/src/__tests__/plugin-import-boundary-reverse-guard.test.ts

@@ -102,7 +102,6 @@ const BASELINE: Record<string, readonly string[]> = {
     "src/permissions/checker.ts",
     "src/persistence/conversation-crud.ts",
     "src/persistence/steps.ts",
-    "src/persistence/worker-control.ts",
     "src/prompts/system-prompt.ts",
     "src/runtime/routes/consolidation-routes.ts",
     "src/runtime/routes/conversation-query-routes.ts",

package/src/__tests__/{scheduler-worker-standdown.test.ts → run-due-schedules.test.ts}

@@ -1,8 +1,8 @@
 /**
- * Scheduler ↔ schedule-worker ownership: while `schedules.worker.enabled` is
- * set, the daemon's in-process scheduler must leave every due schedule
- * unclaimed (the worker process owns schedule execution), while
- * `runDueSchedulesOnce` — the worker's tick — claims and executes them.
+ * `runDueSchedulesOnce` — the schedule worker's tick — claims and executes
+ * every due schedule across modes. Schedule execution is owned by the schedule
+ * worker process; the daemon's own scheduler tick runs only watchers and
+ * sequences.
  */
 
 import { beforeEach, describe, expect, mock, test } from "bun:test";
@@ -44,16 +44,7 @@ mock.module("../notifications/emit-signal.js", () => ({
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
 import { createSchedule } from "../schedule/schedule-store.js";
-import {
-  runDueSchedulesOnce,
-  runScheduleDueWorkOnce,
-} from "../schedule/scheduler.js";
-import { setConfig } from "./helpers/set-config.js";
-
-/** Seed `schedules.worker.enabled` for real; the scheduler re-reads it per tick. */
-function setWorkerEnabled(enabled: boolean): void {
-  setConfig("schedules", { worker: { enabled } });
-}
+import { runDueSchedulesOnce } from "../schedule/scheduler.js";
 
 await initializeDb();
 
@@ -63,7 +54,6 @@ function rawDb(): import("bun:sqlite").Database {
 }
 
 beforeEach(() => {
-  setWorkerEnabled(false);
   mockEmitNotificationSignal.mockClear();
   const db = getDb();
   db.run("DELETE FROM cron_runs");
@@ -95,48 +85,8 @@ function runsFor(jobId: string): Array<{ status: string }> {
     .all(jobId) as Array<{ status: string }>;
 }
 
-describe("scheduler stand-down for the schedule worker", () => {
-  test("flag on: leaves every due schedule unclaimed and reports none pending", async () => {
-    setWorkerEnabled(true);
-    const { script } = await createDueScriptAndNotifyJobs();
-    const dueBefore = rawDb()
-      .query("SELECT id, next_run_at FROM cron_jobs")
-      .all() as Array<{ id: string; next_run_at: number }>;
-
-    const result = await runScheduleDueWorkOnce({
-      includeStillPending: true,
-    });
-
-    expect(result.claimed).toBe(0);
-    expect(result.stillPending).toBe(0);
-    expect(mockEmitNotificationSignal).not.toHaveBeenCalled();
-    expect(runsFor(script.id)).toHaveLength(0);
-    // Nothing advanced — every schedule stays due for the worker to claim.
-    const dueAfter = rawDb()
-      .query("SELECT id, next_run_at FROM cron_jobs")
-      .all() as Array<{ id: string; next_run_at: number }>;
-    expect(dueAfter).toEqual(dueBefore);
-  });
-
-  test("flag off: the in-process scheduler claims and runs due schedules itself", async () => {
-    setWorkerEnabled(false);
-    const { script } = await createDueScriptAndNotifyJobs();
-
-    const result = await runScheduleDueWorkOnce();
-
-    expect(result.claimed).toBe(2);
-    const scriptRuns = runsFor(script.id);
-    expect(scriptRuns).toHaveLength(1);
-    expect(scriptRuns[0].status).toBe("ok");
-    expect(mockEmitNotificationSignal).toHaveBeenCalled();
-  });
-});
-
 describe("runDueSchedulesOnce (the schedule worker's tick)", () => {
   test("claims and executes due schedules across modes", async () => {
-    // The worker runs with the flag on; runDueSchedulesOnce itself does not
-    // consult the flag — its caller owns schedule execution.
-    setWorkerEnabled(true);
     const { script } = await createDueScriptAndNotifyJobs();
 
     const result = await runDueSchedulesOnce();

package/src/__tests__/schedule-retry.test.ts

@@ -56,7 +56,6 @@ mock.module("../notifications/emit-signal.js", () => ({
     emitNotificationSignalImpl(payload),
 }));
 
-import { loadRawConfig, saveRawConfig } from "../config/loader.js";
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
 import { applyRetryDecision, decideRetry } from "../schedule/retry-policy.js";
@@ -72,15 +71,19 @@ import {
   scheduleRetry,
 } from "../schedule/schedule-store.js";
 import type { SchedulerHandle } from "../schedule/scheduler.js";
-import { startScheduler as startSchedulerReal } from "../schedule/scheduler.js";
+import {
+  runDueSchedulesOnce,
+  runScheduleDueWorkOnce,
+} from "../schedule/scheduler.js";
 import type { ScheduleMessageProcessor } from "../schedule/scheduler-types.js";
 
 /**
- * Wrap `startScheduler` so a single per-test `processMessage` callback drives
- * both scheduler dispatch paths: the conversation-reuse path (which calls the
- * mocked `daemon/process-message`) and the fresh-bootstrap path (which calls
- * the mocked `runBackgroundJob`). Tests exercise both deterministically through
- * this one callback.
+ * Wire a single per-test `processMessage` callback into both schedule dispatch
+ * paths — the conversation-reuse path (mocked `daemon/process-message`) and the
+ * fresh-bootstrap path (mocked `runBackgroundJob`) — and return a handle that
+ * drives schedule execution directly via `runDueSchedulesOnce` (the schedule
+ * worker's path). Avoids spawning the real worker process in tests while
+ * exercising the exact execution + retry logic.
  */
 function startScheduler(
   processMessage: ScheduleMessageProcessor,
@@ -92,18 +95,22 @@ function startScheduler(
   };
   processMessageImpl = dispatch;
   injectedProcessMessageForRunner = dispatch;
-  return startSchedulerReal();
+  // Mimic the real scheduler's immediate startup tick that fires due
+  // schedules once; tests await a short delay for it to settle before
+  // asserting or driving further runs via `runOnce`.
+  void runDueSchedulesOnce();
+  return {
+    async runOnce(): Promise<number> {
+      const r = await runDueSchedulesOnce();
+      return r.completed + r.failed + r.skipped;
+    },
+    runDueWorkOnce: (options) => runScheduleDueWorkOnce(options),
+    stop(): void {},
+  };
 }
 
 await initializeDb();
 
-// The schedule worker is on by default, which stands the daemon's in-process
-// scheduler down. These tests exercise that in-process execution path directly,
-// so pin the worker flag off for this test process.
-const rawConfig = loadRawConfig();
-rawConfig.schedules = { worker: { enabled: false } };
-saveRawConfig(rawConfig);
-
 /** Access the underlying bun:sqlite Database for raw parameterized queries. */
 function getRawDb(): import("bun:sqlite").Database {
   return (getDb() as unknown as { $client: import("bun:sqlite").Database })

package/src/__tests__/scheduler-recurrence.test.ts

@@ -53,7 +53,6 @@ mock.module("../notifications/emit-signal.js", () => ({
   },
 }));
 
-import { loadRawConfig, saveRawConfig } from "../config/loader.js";
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
 import {
@@ -61,17 +60,10 @@ import {
   getSchedule,
   getScheduleRuns,
 } from "../schedule/schedule-store.js";
-import { runScheduleDueWorkOnce } from "../schedule/scheduler.js";
+import { runDueSchedulesOnce } from "../schedule/scheduler.js";
 
 await initializeDb();
 
-// The schedule worker is on by default, which stands the daemon's in-process
-// scheduler down. These tests exercise that in-process execution path directly,
-// so pin the worker flag off for this test process.
-const rawConfig = loadRawConfig();
-rawConfig.schedules = { worker: { enabled: false } };
-saveRawConfig(rawConfig);
-
 /** Access the underlying bun:sqlite Database for raw parameterized queries. */
 function getRawDb(): import("bun:sqlite").Database {
   return (getDb() as unknown as { $client: import("bun:sqlite").Database })
@@ -150,7 +142,7 @@ describe("scheduler RRULE execution", () => {
       processedMessages.push({ conversationId, message: prompt });
     };
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     // The runner should have been invoked with the RRULE message
     expect(
@@ -198,7 +190,7 @@ describe("scheduler RRULE execution", () => {
     };
 
     // First tick: the expired schedule should fire its final due run
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     // The message IS delivered once
     expect(processedMessages).toContain("Final expired run");
@@ -216,7 +208,7 @@ describe("scheduler RRULE execution", () => {
 
     // Second tick: the disabled schedule must NOT fire again
     processedMessages.length = 0;
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     expect(processedMessages).not.toContain("Final expired run");
     // No additional runs
@@ -244,7 +236,7 @@ describe("scheduler RRULE execution", () => {
       processedMessages.push({ conversationId, message: prompt });
     };
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     // The runner should have been invoked with the cron message
     expect(processedMessages.some((m) => m.message === "Cron message")).toBe(
@@ -275,7 +267,7 @@ describe("scheduler RRULE execution", () => {
       dispatched.push(message);
     };
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     // The raw `run_task:<id>` string must never reach the agent.
     expect(dispatched).toHaveLength(0);
@@ -326,7 +318,7 @@ describe("scheduler RRULE execution", () => {
     // Force the schedule to be due
     forceScheduleDue(schedule.id);
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     // The schedule should have been claimed and nextRunAt advanced
     const after = getSchedule(schedule.id);
@@ -373,7 +365,7 @@ describe("scheduler RRULE execution", () => {
       processedMessages.push(prompt);
     };
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     expect(processedMessages).toContain("Set fire test");
 
@@ -447,7 +439,7 @@ describe("scheduler RRULE execution", () => {
         processedMessages.push(prompt);
       };
 
-      await runScheduleDueWorkOnce();
+      await runDueSchedulesOnce();
 
       // The schedule should have fired
       expect(processedMessages).toContain("EXRULE scheduler fire");
@@ -483,7 +475,7 @@ describe("scheduler RRULE execution", () => {
     forceScheduleDue(schedule.id);
     const forcedDueAt = getSchedule(schedule.id)!.nextRunAt;
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     const after = getSchedule(schedule.id);
     expect(after).not.toBeNull();
@@ -512,7 +504,7 @@ describe("scheduler RRULE execution", () => {
       processedMessages.push({ conversationId, message: prompt });
     };
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     expect(
       processedMessages.some((m) => m.message === "Execute this once"),
@@ -541,7 +533,7 @@ describe("scheduler RRULE execution", () => {
 
     expect(getSchedule(schedule.id)!.status).toBe("active");
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     expect(notifySignalCalls).toHaveLength(1);
     expect(notifySignalCalls[0]).toMatchObject({
@@ -574,7 +566,7 @@ describe("scheduler RRULE execution", () => {
 
     runBackgroundJobShouldFail = true;
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     const after = getSchedule(schedule.id);
     expect(after).not.toBeNull();
@@ -597,7 +589,7 @@ describe("scheduler RRULE execution", () => {
 
     forceScheduleDue(schedule.id);
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     expect(notifySignalCalls).toHaveLength(1);
     expect(notifySignalCalls[0]).toMatchObject({
@@ -635,7 +627,7 @@ describe("scheduler RRULE execution", () => {
       },
     });
 
-    await runScheduleDueWorkOnce();
+    await runDueSchedulesOnce();
 
     expect(notifySignalCalls).toHaveLength(1);
     expect(notifySignalCalls[0].routingIntent).toBe("all_channels");