@vellumai/assistant 0.8.5 → 0.8.7

package/src/plugins/defaults/index.ts

@@ -7,47 +7,45 @@
  *
  * Consumers:
  *
- * - `daemon/external-plugins-bootstrap.ts` — production/registry boot path;
- *   calls {@link registerDefaultPlugins} inside `bootstrapPlugins()`.
+ * - `daemon/external-plugins-bootstrap.ts` — the daemon's `initializePlugins()`
+ *   calls {@link registerDefaultPlugins} explicitly at startup, before
+ *   `loadUserPlugins()` closes the registration window, so the defaults compose
+ *   innermost (ahead of any user plugins). `bootstrapPlugins()` replays it;
+ *   idempotent, so already-registered defaults are skipped.
  * - integration tests that reset the registry and then need a
  *   production-parity state (e.g. `conversation-agent-loop.test.ts`); those
  *   call {@link resetPluginRegistryAndRegisterDefaults}.
  *
- * Each `defaults/<name>.ts` module self-registers at module load via a local
- * side effect. Importing this aggregator (or any individual default file)
- * populates the registry — the self-registration is idempotent, and so are
- * {@link registerDefaultPlugins} and {@link resetPluginRegistryAndRegisterDefaults}.
- * Per-file self-registration is what keeps registration attached to each
- * file's own already-initialized plugin identifier, so importing
- * `defaults/index.ts` mid-cycle (e.g. through the
- * `memory-retrieval.ts` → … → `pipeline.ts` → `defaults/index.ts`
- * chain) does not trip a TDZ.
+ * Each `defaults/<name>/register.ts` module only builds and exports its
+ * `Plugin` object; registration is centralized here. The plugin identifiers
+ * are dereferenced inside {@link registerDefaultPlugins} at call time, once
+ * every module has finished initializing, so importing this aggregator does no
+ * registration work.
  */
 
-import { memoryV3ShadowPlugin } from "../../memory/v3/shadow-middleware.js";
+import { memoryV3ShadowPlugin } from "../../memory/v3/shadow-plugin.js";
 import { registerPlugin, resetPluginRegistryForTests } from "../registry.js";
 import { type Plugin, PluginExecutionError } from "../types.js";
-import { defaultCircuitBreakerPlugin } from "./circuit-breaker.js";
-import { defaultCompactionPlugin } from "./compaction.js";
-import { defaultEmptyResponsePlugin } from "./empty-response.js";
-import { defaultHistoryRepairPlugin } from "./history-repair.js";
-import { defaultInjectorsPlugin } from "./injectors.js";
-import { defaultLlmCallPlugin } from "./llm-call.js";
-import { defaultMemoryRetrievalPlugin } from "./memory-retrieval.js";
-import { defaultOverflowReducePlugin } from "./overflow-reduce.js";
-import { defaultPersistencePlugin } from "./persistence.js";
-import { defaultTitleGeneratePlugin } from "./title-generate.js";
-import { defaultTokenEstimatePlugin } from "./token-estimate.js";
-import { defaultToolErrorPlugin } from "./tool-error.js";
-import { defaultToolExecutePlugin } from "./tool-execute.js";
-import { defaultToolResultTruncatePlugin } from "./tool-result-truncate.js";
+import { defaultCircuitBreakerPlugin } from "./circuit-breaker/register.js";
+import { defaultCompactionPlugin } from "./compaction/register.js";
+import { defaultEmptyResponsePlugin } from "./empty-response/register.js";
+import { defaultHistoryRepairPlugin } from "./history-repair/register.js";
+import { defaultInjectorsPlugin } from "./injectors/register.js";
+import { defaultLlmCallPlugin } from "./llm-call/register.js";
+import { defaultMemoryRetrievalPlugin } from "./memory-retrieval/register.js";
+import { defaultOverflowReducePlugin } from "./overflow-reduce/register.js";
+import { defaultPersistencePlugin } from "./persistence/register.js";
+import { defaultTitleGeneratePlugin } from "./title-generate/register.js";
+import { defaultTokenEstimatePlugin } from "./token-estimate/register.js";
+import { defaultToolErrorPlugin } from "./tool-error/register.js";
+import { defaultToolExecutePlugin } from "./tool-execute/register.js";
+import { defaultToolResultTruncatePlugin } from "./tool-result-truncate/register.js";
 
 /**
  * Full set of first-party default plugins. Used by
- * {@link registerDefaultPlugins} to drive the idempotent re-registration
- * loop; actual registration-order in the registry is determined by the
- * module-load side effects in each per-file default (whichever loader
- * evaluates a file first wins, later attempts are swallowed as duplicates).
+ * {@link registerDefaultPlugins} to drive the registration loop; the array
+ * order is the registration order, which determines onion order for middleware
+ * composition (defaults innermost, user plugins outermost).
  *
  * Returned by a function rather than a top-level `const` so the array
  * contents are read at call time, after every imported plugin identifier is
@@ -61,11 +59,6 @@ function getAllDefaultPlugins(): readonly Plugin[] {
     defaultEmptyResponsePlugin,
     defaultToolErrorPlugin,
     defaultMemoryRetrievalPlugin,
-    // Live-shadow v3 retrieval. Always registered; inert unless both
-    // `memory.v3.enabled` and `memory.v3.shadow` are on (gated inside the
-    // middleware). Ordered after the default so the default terminal still
-    // produces the injected (v2) `MemoryResult`.
-    memoryV3ShadowPlugin,
     defaultInjectorsPlugin,
     defaultTokenEstimatePlugin,
     defaultOverflowReducePlugin,
@@ -74,6 +67,7 @@ function getAllDefaultPlugins(): readonly Plugin[] {
     defaultCircuitBreakerPlugin,
     defaultPersistencePlugin,
     defaultTitleGeneratePlugin,
+    memoryV3ShadowPlugin,
   ];
 }
 
@@ -83,11 +77,6 @@ function getAllDefaultPlugins(): readonly Plugin[] {
  * an "already registered" message) are swallowed so repeat bootstrap or test
  * setup calls do not throw. Every other error (shape failure, version
  * mismatch) re-throws.
- *
- * In practice every call after the first is a no-op: each default's
- * module-load side effect registers itself the first time its file is
- * imported, which for production happens via `pipeline.ts`'s side-effect
- * import of this aggregator.
  */
 export function registerDefaultPlugins(): void {
   for (const plugin of getAllDefaultPlugins()) {

package/src/plugins/defaults/injectors/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "default-injectors",
+  "version": "1.0.0",
+  "description": "First-party default plugin wrapping the assistant's built-in injectors pipeline with a passthrough implementation.",
+  "private": true,
+  "license": "MIT",
+  "type": "module",
+  "main": "./register.ts",
+  "engines": {
+    "node": ">=20.12.0"
+  },
+  "peerDependencies": {
+    "@vellumai/plugin-api": "^0.8.0"
+  }
+}

package/src/plugins/defaults/{injectors.ts → injectors/register.ts}

@@ -39,31 +39,28 @@
  * sorted ascending, so a plugin-registered injector at `order: 25`
  * reliably slots between `unified-turn-context` (20) and `pkb` (30).
  *
- * Registration happens via a module-load side effect at the bottom of this
- * file — importing the module is enough to populate the registry. The
- * explicit `registerDefaultPlugins()` call in `plugins/defaults/index.ts`
- * (invoked from `daemon/external-plugins-bootstrap.ts`) re-registers the
- * same plugin idempotently, so either entry point alone is sufficient.
+ * This module only builds and exports the `Plugin` object; the defaults
+ * aggregator in `plugins/defaults/index.ts` registers it centrally, either
+ * explicitly from `daemon/external-plugins-bootstrap.ts` or lazily via the
+ * registry's default registrar the first time a query reads the registry.
  */
 
 import { resolve } from "node:path";
 
-import { isAssistantFeatureFlagEnabled } from "../../config/assistant-feature-flags.js";
-import { getConfig } from "../../config/loader.js";
-import { getInContextPkbPaths } from "../../daemon/pkb-context-tracker.js";
-import { buildPkbReminder } from "../../daemon/pkb-reminder-builder.js";
-import { listComments } from "../../documents/document-comments-store.js";
-import { searchPkbFiles } from "../../memory/pkb/pkb-search.js";
-import { getLogger } from "../../util/logger.js";
-import { registerPlugin } from "../registry.js";
+import { getConfig } from "../../../config/loader.js";
+import { getInContextPkbPaths } from "../../../daemon/pkb-context-tracker.js";
+import { buildPkbReminder } from "../../../daemon/pkb-reminder-builder.js";
+import { listComments } from "../../../documents/document-comments-store.js";
+import { searchPkbFiles } from "../../../memory/pkb/pkb-search.js";
+import { getLogger } from "../../../util/logger.js";
 import {
   type InjectionBlock,
   type Injector,
   type Plugin,
-  PluginExecutionError,
   type TurnContext,
   type TurnInjectionInputs,
-} from "../types.js";
+} from "../../types.js";
+import pkg from "./package.json" with { type: "json" };
 
 const pkbReminderLog = getLogger("pkb-reminder");
 
@@ -107,24 +104,19 @@ function readInjectionInputs(ctx: TurnContext): TurnInjectionInputs {
 }
 
 export const DISK_PRESSURE_WARNING_PROMPT = `<disk_pressure_warning>
-Disk usage is critically low: this assistant is in storage cleanup mode because the workspace volume is at least 95% full.
+Disk usage is critically low: this assistant is in storage cleanup mode because the workspace volume is critically full.
 
 In your first paragraph, warn the user that storage is critically low and that normal work is suspended until space is freed.
 
 Then help the user clean up storage. Prefer safe inspection steps first, such as checking available space and finding large directories. Ask before deleting files or caches unless the user has already clearly approved the specific cleanup action.
 
-Do not work on unrelated tasks until disk usage drops below the critical threshold or the user explicitly overrides the lock. Background processes and messages from trusted contacts are blocked while this cleanup mode is active.
+Do not work on unrelated tasks until enough space is freed to clear the lock or the user explicitly overrides it. Background processes and messages from trusted contacts are blocked while this cleanup mode is active.
 </disk_pressure_warning>`;
 
-function isSafeStorageLimitsEnabled(): boolean {
-  return isAssistantFeatureFlagEnabled("safe-storage-limits", getConfig());
-}
-
 const diskPressureWarningInjector: Injector = {
   name: "disk-pressure-warning",
   order: DEFAULT_INJECTOR_ORDER.diskPressureWarning,
   async produce(ctx: TurnContext): Promise<InjectionBlock | null> {
-    if (!isSafeStorageLimitsEnabled()) return null;
     const inputs = readInjectionInputs(ctx);
     if (!inputs.diskPressureContext?.cleanupModeActive) return null;
     return {
@@ -692,8 +684,8 @@ const threadFocusInjector: Injector = {
  */
 export const defaultInjectorsPlugin: Plugin = {
   manifest: {
-    name: "default-injectors",
-    version: "1.0.0",
+    name: pkg.name,
+    version: pkg.version,
   },
   injectors: [
     diskPressureWarningInjector,
@@ -711,25 +703,3 @@ export const defaultInjectorsPlugin: Plugin = {
     threadFocusInjector,
   ],
 };
-
-// Module-load side effect: register this default at import time so
-// downstream consumers (including tests that skip `bootstrapPlugins()`)
-// observe a populated registry by default. Idempotent via the swallowed
-// duplicate-name check. Kept local to this module (rather than iterating
-// an array in `defaults/index.ts`) so the registration only references
-// the already-initialized `defaultInjectorsPlugin` identifier —
-// avoiding a TDZ crash when tests `mock.module(...)` a dependency of any
-// other default plugin and directly import this file.
-try {
-  registerPlugin(defaultInjectorsPlugin);
-} catch (err) {
-  if (
-    err instanceof PluginExecutionError &&
-    err.message.includes("already registered")
-  ) {
-    // already registered — expected when both index.ts and the direct
-    // file are imported in the same process
-  } else {
-    throw err;
-  }
-}

package/src/plugins/defaults/llm-call/middlewares/llmCall.ts

@@ -0,0 +1,17 @@
+import type { LLMCallArgs, LLMCallResult, Middleware } from "../../../types.js";
+
+/**
+ * Passthrough middleware for the `llmCall` pipeline. Forwards to `next(args)`
+ * unchanged so any user-registered middleware (registered later, inner in the
+ * onion) still runs and the terminal at the call site (`agent/loop.ts`)
+ * performs the actual `provider.sendMessage(...)` call.
+ *
+ * Defaults register at the OUTERMOST onion position; forwarding
+ * unconditionally keeps user-registered middleware reachable.
+ */
+const defaultLlmCall: Middleware<LLMCallArgs, LLMCallResult> =
+  async function defaultLlmCall(args, next, _ctx) {
+    return next(args);
+  };
+
+export default defaultLlmCall;

package/src/plugins/defaults/llm-call/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "default-llm-call",
+  "version": "1.0.0",
+  "description": "First-party default plugin wrapping the assistant's built-in llm-call pipeline with a passthrough implementation.",
+  "private": true,
+  "license": "MIT",
+  "type": "module",
+  "main": "./register.ts",
+  "engines": {
+    "node": ">=20.12.0"
+  },
+  "peerDependencies": {
+    "@vellumai/plugin-api": "^0.8.0"
+  }
+}

package/src/plugins/defaults/{llm-call.ts → llm-call/register.ts}

@@ -20,13 +20,9 @@
  * Design doc: `.private/plans/agent-plugin-system.md` (PR 15).
  */
 
-import { registerPlugin } from "../registry.js";
-import {
-  type LLMCallArgs,
-  type LLMCallResult,
-  type Plugin,
-  PluginExecutionError,
-} from "../types.js";
+import { type Plugin } from "../../types.js";
+import llmCall from "./middlewares/llmCall.js";
+import pkg from "./package.json" with { type: "json" };
 
 /**
  * The default LLM-call plugin. Its `llmCall` middleware is a passthrough that
@@ -40,38 +36,10 @@ import {
  */
 export const defaultLlmCallPlugin: Plugin = {
   manifest: {
-    name: "default-llm-call",
-    version: "1.0.0",
+    name: pkg.name,
+    version: pkg.version,
   },
   middleware: {
-    llmCall: async function defaultLlmCall(
-      args: LLMCallArgs,
-      next,
-      _ctx,
-    ): Promise<LLMCallResult> {
-      return next(args);
-    },
+    llmCall,
   },
 };
-
-// Module-load side effect: register this default at import time so
-// downstream consumers (including tests that skip `bootstrapPlugins()`)
-// observe a populated registry by default. Idempotent via the swallowed
-// duplicate-name check. Kept local to this module (rather than iterating
-// an array in `defaults/index.ts`) so the registration only references
-// the already-initialized `defaultLlmCallPlugin` identifier —
-// avoiding a TDZ crash when tests `mock.module(...)` a dependency of any
-// other default plugin and directly import this file.
-try {
-  registerPlugin(defaultLlmCallPlugin);
-} catch (err) {
-  if (
-    err instanceof PluginExecutionError &&
-    err.message.includes("already registered")
-  ) {
-    // already registered — expected when both index.ts and the direct
-    // file are imported in the same process
-  } else {
-    throw err;
-  }
-}

package/src/plugins/defaults/memory-retrieval/middlewares/memoryRetrieval.ts

@@ -0,0 +1,17 @@
+import type { MemoryArgs, MemoryResult, Middleware } from "../../../types.js";
+
+/**
+ * Passthrough middleware for the `memoryRetrieval` pipeline.
+ *
+ * Keeping a real middleware registered (rather than an empty list) makes the
+ * pipeline observable in `plugin.pipeline` logs with a non-empty `chain` field
+ * and lets third-party plugins rely on the default slot being present even
+ * when nothing is overriding it. The work happens in the terminal supplied by
+ * the agent loop, which calls `runDefaultMemoryRetrieval`.
+ */
+const defaultMemoryRetrievalMiddleware: Middleware<MemoryArgs, MemoryResult> =
+  async function defaultMemoryRetrieval(args, next) {
+    return next(args);
+  };
+
+export default defaultMemoryRetrievalMiddleware;

package/src/plugins/defaults/memory-retrieval/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "default-memory-retrieval",
+  "version": "0.0.1",
+  "description": "First-party default plugin wrapping the assistant's built-in memory-retrieval pipeline with a passthrough implementation.",
+  "private": true,
+  "license": "MIT",
+  "type": "module",
+  "main": "./register.ts",
+  "engines": {
+    "node": ">=20.12.0"
+  },
+  "peerDependencies": {
+    "@vellumai/plugin-api": "^0.8.0"
+  }
+}

package/src/plugins/defaults/{memory-retrieval.ts → memory-retrieval/register.ts}

@@ -24,22 +24,21 @@
  * milestone.
  */
 
-import type { AssistantConfig } from "../../config/schema.js";
+import type { AssistantConfig } from "../../../config/schema.js";
 import {
   readNowScratchpad,
   readPkbContext,
-} from "../../daemon/conversation-runtime-assembly.js";
-import type { ServerMessage } from "../../daemon/message-protocol.js";
-import type { ConversationGraphMemory } from "../../memory/graph/conversation-graph-memory.js";
-import type { Message } from "../../providers/types.js";
-import { registerPlugin } from "../registry.js";
+} from "../../../daemon/conversation-runtime-assembly.js";
+import type { ServerMessage } from "../../../daemon/message-protocol.js";
+import type { ConversationGraphMemory } from "../../../memory/graph/conversation-graph-memory.js";
+import type { Message } from "../../../providers/types.js";
 import {
   type MemoryArgs,
   type MemoryResult,
-  type Middleware,
   type Plugin,
-  PluginExecutionError,
-} from "../types.js";
+} from "../../types.js";
+import defaultMemoryRetrievalMiddleware from "./middlewares/memoryRetrieval.js";
+import pkg from "./package.json" with { type: "json" };
 
 /**
  * Discriminator the agent loop uses to narrow `MemoryResult.memoryGraphBlocks`
@@ -166,21 +165,6 @@ export function asDefaultGraphPayload(
   return null;
 }
 
-/**
- * Default `memoryRetrieval` middleware — a pure pass-through.
- *
- * Keeping a real middleware registered (rather than an empty list) makes
- * the pipeline observable in `plugin.pipeline` logs with a non-empty
- * `chain` field and lets third-party plugins rely on the default slot
- * being present even when nothing is overriding it. The work happens in
- * the terminal supplied by the agent loop, which calls
- * {@link runDefaultMemoryRetrieval}.
- */
-const defaultMemoryRetrievalMiddleware: Middleware<MemoryArgs, MemoryResult> =
-  async function defaultMemoryRetrieval(args, next) {
-    return next(args);
-  };
-
 /**
  * Default plugin exposing the `memoryRetrieval` pipeline slot. Registered
  * by {@link registerDefaultMemoryRetrievalPlugin} from the plugin
@@ -188,32 +172,10 @@ const defaultMemoryRetrievalMiddleware: Middleware<MemoryArgs, MemoryResult> =
  */
 export const defaultMemoryRetrievalPlugin: Plugin = {
   manifest: {
-    name: "default-memory-retrieval",
-    version: "0.0.1",
+    name: pkg.name,
+    version: pkg.version,
   },
   middleware: {
     memoryRetrieval: defaultMemoryRetrievalMiddleware,
   },
 };
-
-// Module-load side effect: register this default at import time so
-// downstream consumers (including tests that skip `bootstrapPlugins()`)
-// observe a populated registry by default. Idempotent via the swallowed
-// duplicate-name check. Kept local to this module (rather than iterating
-// an array in `defaults/index.ts`) so the registration only references
-// the already-initialized `defaultMemoryRetrievalPlugin` identifier —
-// avoiding a TDZ crash when tests `mock.module(...)` a dependency of any
-// other default plugin and directly import this file.
-try {
-  registerPlugin(defaultMemoryRetrievalPlugin);
-} catch (err) {
-  if (
-    err instanceof PluginExecutionError &&
-    err.message.includes("already registered")
-  ) {
-    // already registered — expected when both index.ts and the direct
-    // file are imported in the same process
-  } else {
-    throw err;
-  }
-}

package/src/plugins/defaults/{overflow-reduce.ts → overflow-reduce/middlewares/overflowReduce.ts}

@@ -1,54 +1,29 @@
-/**
- * Default `overflowReduce` plugin — extracted verbatim from the inline
- * preflight reducer loop that previously lived in
- * `daemon/conversation-agent-loop.ts` (the `while (preflightAttempts < …)`
- * block around lines 1045–1156 before PR 23).
- *
- * The plugin owns the reducer tier-loop (forced compaction, tool-result
- * truncation, media stubbing, injection downgrade) and the post-step
- * re-injection / re-estimation dance. Orchestrator-specific coupling
- * (activity emission, circuit-breaker tracking, compaction-result
- * application, runtime injection reassembly) is threaded in through the
- * callbacks carried on {@link OverflowReduceArgs}; the plugin itself has no
- * access to the agent-loop context object.
- *
- * The forced-compaction tier runs through the orchestrator-supplied
- * `compactFn`, which routes into the `compaction` plugin pipeline so
- * registered compaction middleware observes reducer-initiated invocations
- * alongside the orchestrator-owned call sites. Non-compaction tiers
- * (tool-result truncation, media stubbing, injection downgrade) remain
- * in-process: they mutate message arrays directly without crossing a
- * pipeline boundary. The reducer itself runs under the `overflowReduce`
- * pipeline, so the full layering is `overflowReduce` → reducer tier loop
- * → (for the forced-compaction tier only) nested `compaction` pipeline.
- */
-
-import type { ContextWindowCompactOptions } from "../../context/window-manager.js";
+import type { ContextWindowCompactOptions } from "../../../../context/window-manager.js";
 import {
   createInitialReducerState,
   reduceContextOverflow,
   type ReducerState,
-} from "../../daemon/context-overflow-reducer.js";
-import { registerPlugin } from "../registry.js";
-import {
-  type Middleware,
-  type OverflowReduceArgs,
-  type OverflowReduceResult,
-  type Plugin,
-  PluginExecutionError,
-} from "../types.js";
+} from "../../../../daemon/context-overflow-reducer.js";
+import type {
+  Middleware,
+  OverflowReduceArgs,
+  OverflowReduceResult,
+} from "../../../types.js";
 
 /**
- * Default middleware — implements the historical tier-loop semantics.
+ * Default middleware for the `overflowReduce` pipeline — implements the
+ * historical tier-loop semantics (forced compaction, tool-result truncation,
+ * media stubbing, injection downgrade) plus the post-step re-injection /
+ * re-estimation dance.
  *
  * The middleware intentionally ignores `next`. Overflow reduction is a
- * *terminal* behavior: there is no downstream implementation to defer to
- * when a user-supplied middleware short-circuits. Later plugins may still
- * wrap this one (outer middleware can observe each reduction iteration via
- * their own `next` callback) but the default never delegates to a
- * hypothetical base handler — the inline loop was the base.
+ * *terminal* behavior: there is no downstream implementation to defer to when
+ * a user-supplied middleware short-circuits. Later plugins may still wrap this
+ * one (outer middleware can observe each reduction iteration via their own
+ * `next` callback) but the default never delegates to a hypothetical base
+ * handler — the inline loop was the base.
  */
-export const defaultOverflowReduceMiddleware: Middleware<
+const defaultOverflowReduceMiddleware: Middleware<
   OverflowReduceArgs,
   OverflowReduceResult
 > = async function defaultOverflowReduceMiddleware(args, _next, _ctx) {
@@ -148,38 +123,4 @@ export const defaultOverflowReduceMiddleware: Middleware<
   };
 };
 
-/**
- * The default plugin registered at bootstrap. No `init`/`onShutdown` —
- * registering the middleware is the only behavior.
- */
-export const defaultOverflowReducePlugin: Plugin = {
-  manifest: {
-    name: "default-overflow-reduce",
-    version: "1.0.0",
-  },
-  middleware: {
-    overflowReduce: defaultOverflowReduceMiddleware,
-  },
-};
-
-// Module-load side effect: register this default at import time so
-// downstream consumers (including tests that skip `bootstrapPlugins()`)
-// observe a populated registry by default. Idempotent via the swallowed
-// duplicate-name check. Kept local to this module (rather than iterating
-// an array in `defaults/index.ts`) so the registration only references
-// the already-initialized `defaultOverflowReducePlugin` identifier —
-// avoiding a TDZ crash when tests `mock.module(...)` a dependency of any
-// other default plugin and directly import this file.
-try {
-  registerPlugin(defaultOverflowReducePlugin);
-} catch (err) {
-  if (
-    err instanceof PluginExecutionError &&
-    err.message.includes("already registered")
-  ) {
-    // already registered — expected when both index.ts and the direct
-    // file are imported in the same process
-  } else {
-    throw err;
-  }
-}
+export default defaultOverflowReduceMiddleware;

package/src/plugins/defaults/overflow-reduce/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "default-overflow-reduce",
+  "version": "1.0.0",
+  "description": "First-party default plugin wrapping the assistant's built-in overflow-reduce pipeline with a passthrough implementation.",
+  "private": true,
+  "license": "MIT",
+  "type": "module",
+  "main": "./register.ts",
+  "engines": {
+    "node": ">=20.12.0"
+  },
+  "peerDependencies": {
+    "@vellumai/plugin-api": "^0.8.0"
+  }
+}

package/src/plugins/defaults/overflow-reduce/register.ts

@@ -0,0 +1,42 @@
+/**
+ * Default `overflowReduce` plugin — extracted verbatim from the inline
+ * preflight reducer loop that previously lived in
+ * `daemon/conversation-agent-loop.ts` (the `while (preflightAttempts < …)`
+ * block around lines 1045–1156 before PR 23).
+ *
+ * The plugin owns the reducer tier-loop (forced compaction, tool-result
+ * truncation, media stubbing, injection downgrade) and the post-step
+ * re-injection / re-estimation dance. Orchestrator-specific coupling
+ * (activity emission, circuit-breaker tracking, compaction-result
+ * application, runtime injection reassembly) is threaded in through the
+ * callbacks carried on {@link OverflowReduceArgs}; the plugin itself has no
+ * access to the agent-loop context object.
+ *
+ * The forced-compaction tier runs through the orchestrator-supplied
+ * `compactFn`, which routes into the `compaction` plugin pipeline so
+ * registered compaction middleware observes reducer-initiated invocations
+ * alongside the orchestrator-owned call sites. Non-compaction tiers
+ * (tool-result truncation, media stubbing, injection downgrade) remain
+ * in-process: they mutate message arrays directly without crossing a
+ * pipeline boundary. The reducer itself runs under the `overflowReduce`
+ * pipeline, so the full layering is `overflowReduce` → reducer tier loop
+ * → (for the forced-compaction tier only) nested `compaction` pipeline.
+ */
+
+import { type Plugin } from "../../types.js";
+import defaultOverflowReduceMiddleware from "./middlewares/overflowReduce.js";
+import pkg from "./package.json" with { type: "json" };
+
+/**
+ * The default plugin registered at bootstrap. No `init`/`onShutdown` —
+ * registering the middleware is the only behavior.
+ */
+export const defaultOverflowReducePlugin: Plugin = {
+  manifest: {
+    name: pkg.name,
+    version: pkg.version,
+  },
+  middleware: {
+    overflowReduce: defaultOverflowReduceMiddleware,
+  },
+};

package/src/plugins/defaults/persistence/middlewares/persistence.ts

@@ -0,0 +1,19 @@
+import type { Middleware, PersistArgs, PersistResult } from "../../../types.js";
+
+/**
+ * Passthrough middleware for the `persistence` pipeline. Forwards to
+ * `next(args)` unchanged; the actual dispatch lives in the terminal handler
+ * (`../terminal.ts`), wired in at the `runPipeline` call sites in
+ * `daemon/conversation-agent-loop.ts` and
+ * `daemon/conversation-agent-loop-handlers.ts`.
+ *
+ * Defaults register at the OUTERMOST onion position, so deciding here without
+ * calling `next` would shadow every later-registered plugin. Routing through
+ * `next(args)` lets user middleware participate normally.
+ */
+const passthrough: Middleware<PersistArgs, PersistResult> = async (
+  args,
+  next,
+) => next(args);
+
+export default passthrough;

package/src/plugins/defaults/persistence/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "default-persistence",
+  "version": "1.0.0",
+  "description": "First-party default plugin wrapping the assistant's built-in persistence pipeline with a passthrough implementation.",
+  "private": true,
+  "license": "MIT",
+  "type": "module",
+  "main": "./register.ts",
+  "engines": {
+    "node": ">=20.12.0"
+  },
+  "peerDependencies": {
+    "@vellumai/plugin-api": "^0.8.0"
+  }
+}