@vellumai/assistant 0.11.3 → 0.11.4-dev.202608190019.b94dbf2

package/src/plugins/defaults/memory/v3/card.ts

@@ -1,5 +1,6 @@
 import { FRONTMATTER_REGEX, parseFrontmatterFields } from "../frontmatter.js";
 import { injectedConceptHeader } from "../substrate/injected-block-slugs.js";
+import { LINK_SEPARATOR, parseLinkEntry } from "../substrate/page-links.js";
 import type { Slug } from "./types.js";
 
 /**
@@ -31,16 +32,15 @@ const SECTION_HEADING_REGEX = /^## (.*)$/m;
  * Entries with no ` — ` separator are bare target slugs and pass through.
  */
 function renderLinkEntry(entry: string): string {
-  const sep = entry.indexOf(" — ");
-  if (sep === -1) {
-    return entry.trim();
+  const { target, description } = parseLinkEntry(entry);
+  if (description === undefined) {
+    return target;
   }
-  const target = entry.slice(0, sep).trim();
-  let note = entry.slice(sep + " — ".length).trim();
-  if (note.length > LINK_NOTE_MAX_CHARS) {
-    note = `${note.slice(0, LINK_NOTE_MAX_CHARS).trimEnd()}…`;
-  }
-  return note.length > 0 ? `${target} — ${note}` : target;
+  const note =
+    description.length > LINK_NOTE_MAX_CHARS
+      ? `${description.slice(0, LINK_NOTE_MAX_CHARS).trimEnd()}…`
+      : description;
+  return `${target}${LINK_SEPARATOR}${note}`;
 }
 
 /**

package/src/plugins/defaults/memory/v3/core-set.test.ts

@@ -30,6 +30,13 @@ describe("loadCoreSet", () => {
     expect(loadCoreSet(workspaceDir)).toEqual(["page-a", "page-b"]);
   });
 
+  test("a wikilink's |display and #anchor suffixes are not part of the slug", () => {
+    writeCorePages(
+      ["- [[people/alice|Alice]]", "- [[page-b#current]] keep"].join("\n"),
+    );
+    expect(loadCoreSet(workspaceDir)).toEqual(["people/alice", "page-b"]);
+  });
+
   test("accepts inline annotations after the slug", () => {
     writeCorePages(
       [

package/src/plugins/defaults/memory/v3/core-set.ts

@@ -1,6 +1,7 @@
 import { readFileSync } from "node:fs";
 import { join } from "node:path";
 
+import { wikilinkTarget } from "../substrate/page-links.js";
 import type { Slug } from "./types.js";
 
 /**
@@ -52,7 +53,10 @@ export function loadCoreSet(workspaceDir: string): Slug[] {
     if (!match) {
       continue;
     }
-    const slug = (match[1] ?? match[2] ?? "").trim();
+    const slug =
+      match[1] !== undefined
+        ? wikilinkTarget(match[1])
+        : (match[2] ?? "").trim();
     if (!SLUG_SHAPE.test(slug) || seen.has(slug)) {
       continue;
     }

package/src/plugins/defaults/memory/v3/edge.ts

@@ -1,5 +1,9 @@
 import { parseFrontmatterFields } from "../frontmatter.js";
 import type { PageIndexEntry } from "../substrate/page-index.js";
+import {
+  extractWikilinkTargets,
+  parseLinkEntry,
+} from "../substrate/page-links.js";
 import type { Slug } from "./types.js";
 
 /**
@@ -8,19 +12,20 @@ import type { Slug } from "./types.js";
  *
  * The graph unions THREE outbound-edge sources per article:
  *
- *   (a) The optional `links:` frontmatter — a YAML list of
- *       `"<target-slug> — <description>"` strings (split on the first
- *       ` — `, space-emdash-space). This is the authored, first-class edge
+ *   (a) The optional `links:` frontmatter, a YAML list of annotated
+ *       `"<target-slug> <sep> <description>"` strings (see
+ *       `substrate/page-links.ts`). This is the authored, first-class edge
  *       source: the curated description is carried through expansion so the
  *       orchestrator can use it directly as a select descriptor.
  *   (b) Inline `[[wikilink]]` targets parsed from the body.
- *   (c) `PageIndexEntry.edges` numeric ids resolved to slugs — the fallback
+ *   (c) `PageIndexEntry.edges` numeric ids resolved to slugs, the fallback
  *       for pages with no frontmatter `links`.
  *
  * All targets are resolved against the corpus slug set; unknown/dangling
- * targets are dropped. A curated description is stored per (source → target)
- * edge only when the `links` source supplied one; wikilink/numeric edges carry
- * no description (the orchestrator falls back to a section descriptor).
+ * targets are dropped (`PageIndex.danglingLinks` reports them). A curated
+ * description is stored per (source → target) edge only when the `links`
+ * source supplied one; wikilink/numeric edges carry no description (the
+ * orchestrator falls back to a section descriptor).
  *
  * The build is pure given a `pageRaw` reader callback (no hard-coded I/O), so
  * it is built once at lane-init and rebuilt by the maintain job, and is
@@ -35,13 +40,6 @@ const DEFAULT_SEED_COUNT = 18;
 const DEFAULT_PER_SEED = 6;
 const DEFAULT_CAP = 45;
 
-/** Matches the space-emdash-space separator in a `links:` entry. */
-const LINK_SEPARATOR = " — ";
-
-/** Matches `[[target]]` wikilinks; captures the raw target before any `|`
- * display-text or `#section` anchor. */
-const WIKILINK_REGEX = /\[\[([^\]]+)\]\]/g;
-
 /**
  * The directed link-graph. `adjacency` maps each source slug to its outbound
  * edges (target slug → curated description, or `undefined` when the edge came
@@ -79,48 +77,6 @@ interface BuildEdgeGraphOptions {
   hubDegree?: number;
 }
 
-/**
- * Split one `links:` entry (`"<target-slug> — <description>"`) into its target
- * slug and curated description on the FIRST ` — ` (space-emdash-space).
- * Entries with no separator are bare target slugs and carry no description.
- */
-function parseLinkEntry(entry: string): {
-  target: Slug;
-  description: string | undefined;
-} {
-  const sep = entry.indexOf(LINK_SEPARATOR);
-  if (sep === -1) {
-    return { target: entry.trim(), description: undefined };
-  }
-  return {
-    target: entry.slice(0, sep).trim(),
-    description: entry.slice(sep + LINK_SEPARATOR.length).trim() || undefined,
-  };
-}
-
-/** Parse inline `[[wikilink]]` targets from a body. Strips `|display` and
- * `#anchor` suffixes; returns trimmed target slugs (possibly with duplicates,
- * deduped by the caller's map insertion). */
-function parseWikilinks(body: string): string[] {
-  const targets: string[] = [];
-  for (const match of body.matchAll(WIKILINK_REGEX)) {
-    let target = match[1];
-    const pipe = target.indexOf("|");
-    if (pipe !== -1) {
-      target = target.slice(0, pipe);
-    }
-    const hash = target.indexOf("#");
-    if (hash !== -1) {
-      target = target.slice(0, hash);
-    }
-    target = target.trim();
-    if (target) {
-      targets.push(target);
-    }
-  }
-  return targets;
-}
-
 /**
  * Build the directed article link-graph from the page-index entries and a raw
  * page reader. See the module docstring for the three unioned edge sources.
@@ -200,7 +156,7 @@ export async function buildEdgeGraph(
     // the whole raw text is the body.
     const body = parsed ? parsed.body : raw;
     if (body !== null) {
-      for (const target of parseWikilinks(body)) {
+      for (const target of extractWikilinkTargets(body)) {
         addEdge(source, target, undefined);
       }
     }

package/src/plugins/defaults/memory/v3/injector.ts

@@ -286,6 +286,11 @@ export const memoryV3Injector: Injector = {
     if (!isPersonalMemoryAllowed(ctx.trust)) {
       return null;
     }
+    // The front door keeps carried cards from history but defers current-turn
+    // retrieval to the escalated leg so memory cannot delay its first token.
+    if (ctx.callSite === "voiceFrontDoor") {
+      return null;
+    }
 
     let observed: OrchestrateResult | null;
     try {
@@ -412,6 +417,9 @@ export const memoryV3SpotlightInjector: Injector = {
     if (!isPersonalMemoryAllowed(ctx.trust)) {
       return null;
     }
+    if (ctx.callSite === "voiceFrontDoor") {
+      return null;
+    }
 
     try {
       const result = await observeTurnOnce(ctx.conversationId, ctx.turnIndex);

package/src/plugins/defaults/memory/v3/prune.ts

@@ -293,6 +293,8 @@ export function collectPersistedV3Cards(conversationId: string): Set<string> {
   const rows = getSqliteFrom(getDb())
     .query(
       /*sql*/ `
+      -- Any-state scan: only metadata markers are read, and the marker is
+      -- written by the injection path on rows it owns regardless of state.
       SELECT metadata FROM messages
       WHERE conversation_id = ? AND metadata LIKE '%' || ? || '%'
     `,

package/src/plugins/defaults/memory/v3/selection-log-store.ts

@@ -97,6 +97,8 @@ function forkSourceIdsOf(messageIds: string[]): string[] {
     .query(
       /*sql*/ `
       SELECT json_extract(metadata, '$.forkSourceMessageId') AS src
+      -- No completeness predicate: ids come from this store's own selection
+      -- log, written for rows the injection pipeline already processed.
       FROM messages
       WHERE id IN (${placeholders})
     `,

package/src/plugins/defaults/memory/v3/shadow-plugin.ts

@@ -28,7 +28,9 @@ import { existsSync, readFileSync } from "node:fs";
 import {
   getMessages,
   listInstalledSkills,
+  parseMessageMetadata,
   stringifyMessageContent,
+  VOICE_ESCALATION_CONTINUATION_MESSAGE_KIND,
 } from "@vellumai/plugin-api";
 
 import { getConfig } from "../../../../config/loader.js";
@@ -490,9 +492,11 @@ function buildSituationalContext(): string {
 
 /**
  * Build a v3 {@link MemoryRoutingTurn} from the conversation's persisted messages.
- * `currentMessage` is the latest user message; `previousAssistantMessage` is
- * the tail of the last assistant reply BEFORE that message (the reply-query
- * pass's input — absent on a conversation's first turn); `recentContext` is
+ * `currentMessage` is the latest user message, except that the synthetic voice
+ * escalation continuation routes on the preceding caller message;
+ * `previousAssistantMessage` is the tail of the last assistant reply BEFORE
+ * the routed message (the reply-query pass's input, absent on a conversation's
+ * first turn); `recentContext` is
  * the tail of the recent transcript; `situationalContext` carries the current
  * date and the live NOW.md scratchpad. Returns `null` when there is no user
  * message to route on (nothing to shadow this turn).
@@ -509,12 +513,22 @@ async function buildShadowTurn(
   let currentMessage = "";
   let currentIndex = -1;
   for (let i = rows.length - 1; i >= 0; i--) {
-    if (rows[i]!.role === "user") {
-      currentMessage = stringifyMessageContent(rows[i]!.content);
-      if (currentMessage.length > 0) {
-        currentIndex = i;
-        break;
-      }
+    const row = rows[i]!;
+    if (row.role !== "user") {
+      continue;
+    }
+    const candidate = stringifyMessageContent(row.content);
+    const metadata = await parseMessageMetadata(row.metadata);
+    const isVoiceEscalationContinuation =
+      metadata?.hidden === true &&
+      metadata.messageKind === VOICE_ESCALATION_CONTINUATION_MESSAGE_KIND;
+    if (isVoiceEscalationContinuation) {
+      continue;
+    }
+    if (candidate.length > 0) {
+      currentMessage = candidate;
+      currentIndex = i;
+      break;
     }
   }
   if (currentMessage.length === 0) {

package/src/plugins/defaults/memory/worker-control.ts

@@ -52,6 +52,7 @@ export async function spawnMemoryWorkerProcess(
     return await spawnWorkerProcess({
       pidPath: getMemoryWorkerPidPath(),
       entry: new URL("./worker.ts", import.meta.url),
+      packagedEntry: "memory",
       workerLabel: "Memory worker",
       options: opts,
     });

package/src/plugins/defaults/memory/worker.ts

@@ -27,6 +27,7 @@ import {
   cleanupWorkerPidFile,
   startWorkerPidFileGuard,
 } from "../../../util/worker-process.js";
+import { registerWorkerPluginSurface } from "../../worker-plugin-surface.js";
 import { registerMemoryPluginJobHandlers } from "./job-handler-registration.js";
 import { startMemoryJobsWorkerLoop } from "./jobs-worker.js";
 import { getLogger } from "./logging.js";
@@ -59,9 +60,11 @@ async function main(): Promise<void> {
   // and are rejected.
   await rehydratePlatformCredentials();
 
-  // This process does not run plugin bootstrap, so self-register the job
-  // handlers the worker dispatches from before starting it — the memory
-  // plugin's own plus the host's non-plugin domain handlers.
+  // Jobs in this process wake real agent conversations, so it registers the
+  // default plugin hook surface (hooks and injectors, no init hooks) the same
+  // way the daemon does, and the job handlers the worker dispatches from: the
+  // memory plugin's own plus the host's non-plugin domain handlers.
+  registerWorkerPluginSurface();
   registerMemoryPluginJobHandlers();
 
   // Populate the tool registry (core built-ins + workspace tools), exactly as

package/src/plugins/defaults/worker-entrypoints.ts

@@ -0,0 +1,3 @@
+export async function loadDefaultMemoryWorker(): Promise<void> {
+  await import("./memory/worker.js");
+}

package/src/plugins/external-plugin-loader.ts

@@ -92,6 +92,12 @@ const PluginPackageJsonSchema = z
     version: z.string().optional(),
     peerDependencies: z.record(z.string(), z.string()).optional(),
     credentialKeyPatterns: z.unknown().optional(),
+    /** Human title, when the package name is not one ("@vellumai/imessage"). */
+    displayName: z.string().min(1).optional(),
+    /** Standard npm field, reused as the one-line description clients show. */
+    description: z.string().min(1).optional(),
+    /** Lucide icon name without the `lucide-` prefix, matching `ChannelInfo`. */
+    icon: z.string().min(1).optional(),
   })
   .passthrough();
 
@@ -466,6 +472,47 @@ export async function loadExternalPlugin(
  * surface the failure through their own channel (e.g. the schedule
  * reconciler's per-day deduped notification).
  */
+/**
+ * How a plugin presents itself: its title, its one line, its glyph.
+ *
+ * Every field is optional and none of them gate anything. A surface that
+ * shows a plugin falls back rather than hiding it, so a missing `icon`
+ * costs an icon and not the row.
+ */
+export interface PluginPresentation {
+  displayName?: string;
+  description?: string;
+  icon?: string;
+}
+
+/**
+ * Read a plugin's presentation fields from its manifest.
+ *
+ * Separate from {@link parsePluginManifest} because the two answer different
+ * questions: that one decides whether a plugin is loadable, this one only
+ * dresses it for display. An unreadable or invalid manifest yields `undefined`
+ * here rather than an error, since a caller showing a plugin has already
+ * established it exists.
+ */
+export async function parsePluginPresentation(
+  pluginDir: string,
+): Promise<PluginPresentation | undefined> {
+  let rawPkg: unknown;
+  try {
+    rawPkg = JSON.parse(
+      await readFile(join(pluginDir, "package.json"), "utf8"),
+    );
+  } catch {
+    return undefined;
+  }
+  const parsed = PluginPackageJsonSchema.safeParse(rawPkg);
+  if (!parsed.success) {
+    return undefined;
+  }
+  const { displayName, description, icon } = parsed.data;
+  return { displayName, description, icon };
+}
+
 export async function parsePluginManifest(
   pluginDir: string,
   opts: { quiet?: boolean } = {},