@vellumai/assistant 0.11.0 → 0.11.1-staging.2

package/src/workflows/engine.ts

@@ -198,7 +198,9 @@ class AbortedSignal extends Error {
  * this helper just covers a stray late `AbortError` after the signal cleared.
  */
 function isAbortError(err: unknown): boolean {
-  if (err instanceof AbortedSignal) return true;
+  if (err instanceof AbortedSignal) {
+    return true;
+  }
   if (typeof DOMException !== "undefined" && err instanceof DOMException) {
     return err.name === "AbortError";
   }
@@ -368,11 +370,15 @@ function lineStartsInCode(source: string, lineCount: number): boolean[] {
     if (c === "\n") {
       // A string survives the newline only if it was line-continued (`\`); an
       // otherwise-unterminated string is a syntax error the transpiler catches.
-      if (stringQuote !== null && !escaped) stringQuote = null;
+      if (stringQuote !== null && !escaped) {
+        stringQuote = null;
+      }
       inLineComment = false;
       escaped = false;
       line++;
-      if (line < lineCount) result[line] = carryIsCode();
+      if (line < lineCount) {
+        result[line] = carryIsCode();
+      }
       continue;
     }
 
@@ -380,7 +386,9 @@ function lineStartsInCode(source: string, lineCount: number): boolean[] {
       escaped = false;
       continue;
     }
-    if (inLineComment) continue;
+    if (inLineComment) {
+      continue;
+    }
     if (inBlockComment) {
       if (c === "*" && next === "/") {
         inBlockComment = false;
@@ -389,16 +397,21 @@ function lineStartsInCode(source: string, lineCount: number): boolean[] {
       continue;
     }
     if (stringQuote !== null) {
-      if (c === "\\") escaped = true;
-      else if (c === stringQuote) stringQuote = null;
+      if (c === "\\") {
+        escaped = true;
+      } else if (c === stringQuote) {
+        stringQuote = null;
+      }
       continue;
     }
 
     const top = stack[stack.length - 1]!;
     if (top.kind === "template") {
-      if (c === "\\") escaped = true;
-      else if (c === "`") stack.pop();
-      else if (c === "$" && next === "{") {
+      if (c === "\\") {
+        escaped = true;
+      } else if (c === "`") {
+        stack.pop();
+      } else if (c === "$" && next === "{") {
         stack.push({ kind: "code", braceDepth: 0 });
         i++;
       }
@@ -406,7 +419,9 @@ function lineStartsInCode(source: string, lineCount: number): boolean[] {
     }
 
     // --- code frame ---
-    if (c === " " || c === "\t" || c === "\r") continue; // not significant
+    if (c === " " || c === "\t" || c === "\r") {
+      continue;
+    } // not significant
     if (c === "/" && next === "/") {
       inLineComment = true;
       i++;
@@ -438,8 +453,11 @@ function lineStartsInCode(source: string, lineCount: number): boolean[] {
     if (c === "{") {
       top.braceDepth++;
     } else if (c === "}") {
-      if (top.braceDepth === 0 && stack.length > 1) stack.pop();
-      else if (top.braceDepth > 0) top.braceDepth--;
+      if (top.braceDepth === 0 && stack.length > 1) {
+        stack.pop();
+      } else if (top.braceDepth > 0) {
+        top.braceDepth--;
+      }
     }
     prevSignificant = c;
   }
@@ -456,7 +474,9 @@ function lineStartsInCode(source: string, lineCount: number): boolean[] {
  * contains a backtick or `/*`), so this conservative table suffices.
  */
 function regexAllowedAfter(prev: string): boolean {
-  if (prev === "") return true;
+  if (prev === "") {
+    return true;
+  }
   return "(,=:[!&|?{};+-*/%^~<>".includes(prev);
 }
 
@@ -470,14 +490,20 @@ function skipRegexLiteral(source: string, start: number): number {
   let inClass = false;
   for (let j = start + 1; j < source.length; j++) {
     const c = source[j]!;
-    if (c === "\n") return j - 1; // regex can't span a newline; bail before it
+    if (c === "\n") {
+      return j - 1;
+    } // regex can't span a newline; bail before it
     if (c === "\\") {
       j++;
       continue;
     }
-    if (c === "[") inClass = true;
-    else if (c === "]") inClass = false;
-    else if (c === "/" && !inClass) return j;
+    if (c === "[") {
+      inClass = true;
+    } else if (c === "]") {
+      inClass = false;
+    } else if (c === "/" && !inClass) {
+      return j;
+    }
   }
   return source.length - 1;
 }
@@ -661,7 +687,9 @@ export async function executeWorkflow(
     leafOpts: LeafCallOptions,
     leafSignal?: AbortSignal,
   ): Promise<{ output: unknown; failed: boolean }> => {
-    if (signal?.aborted) throw new AbortedSignal();
+    if (signal?.aborted) {
+      throw new AbortedSignal();
+    }
 
     const hash = callHashOf(prompt, leafOpts);
 
@@ -816,7 +844,9 @@ export async function executeWorkflow(
     labelPrefix: string,
     leafOpts: LeafCallOptions,
   ): LeafCallOptions => {
-    if (!labelPrefix) return leafOpts;
+    if (!labelPrefix) {
+      return leafOpts;
+    }
     const base = leafOpts.label ?? "";
     return {
       ...leafOpts,
@@ -914,8 +944,12 @@ export async function executeWorkflow(
       nameArg: unknown,
       childArgs?: unknown,
     ): Promise<unknown> => {
-      if (depth >= 1) throw new WorkflowNestingDepthError();
-      if (signal?.aborted) throw new AbortedSignal();
+      if (depth >= 1) {
+        throw new WorkflowNestingDepthError();
+      }
+      if (signal?.aborted) {
+        throw new AbortedSignal();
+      }
       const childName = String(nameArg);
 
       // Snapshot the resolved child source under its own journal `seq` so a
@@ -940,7 +974,9 @@ export async function executeWorkflow(
         source = (cached.result as { source: string }).source;
       } else {
         const saved = library.getWorkflow(childName);
-        if (!saved) throw new WorkflowNotFoundError(childName);
+        if (!saved) {
+          throw new WorkflowNotFoundError(childName);
+        }
         source = saved.source;
         journal.appendJournalEntry({
           runId,
@@ -979,7 +1015,9 @@ export async function executeWorkflow(
     // PRs; here we expose the names so an undeclared call is a ReferenceError and
     // a declared-but-unbound call fails loudly rather than silently.)
     for (const name of capabilities.hostFunctions) {
-      if (name in hostFunctions) continue;
+      if (name in hostFunctions) {
+        continue;
+      }
       hostFunctions[name] = () => {
         throw new WorkflowScriptError(
           `Host function "${name}" is declared but not bound in this engine build.`,
@@ -1075,14 +1113,26 @@ export async function executeWorkflow(
 }
 
 function normalizeLeafOpts(optsArg: unknown): LeafCallOptions {
-  if (!optsArg || typeof optsArg !== "object") return {};
+  if (!optsArg || typeof optsArg !== "object") {
+    return {};
+  }
   const o = optsArg as Record<string, unknown>;
   const out: LeafCallOptions = {};
-  if (o.schema !== undefined) out.schema = o.schema;
-  if (typeof o.label === "string") out.label = o.label;
-  if (typeof o.profile === "string") out.profile = o.profile;
-  if (typeof o.persona === "boolean") out.persona = o.persona;
-  if (typeof o.phase === "string") out.phase = o.phase;
+  if (o.schema !== undefined) {
+    out.schema = o.schema;
+  }
+  if (typeof o.label === "string") {
+    out.label = o.label;
+  }
+  if (typeof o.profile === "string") {
+    out.profile = o.profile;
+  }
+  if (typeof o.persona === "boolean") {
+    out.persona = o.persona;
+  }
+  if (typeof o.phase === "string") {
+    out.phase = o.phase;
+  }
   return out;
 }
 
@@ -1155,9 +1205,13 @@ async function runWithConcurrency<T, R>(
     for (;;) {
       // Once any task has thrown, stop pulling NEW work; let the in-flight
       // leaves (which `onUnwind` has asked to cancel) drain to settlement.
-      if (failed) return;
+      if (failed) {
+        return;
+      }
       const index = cursor++;
-      if (index >= tasks.length) return;
+      if (index >= tasks.length) {
+        return;
+      }
       try {
         results[index] = await run(tasks[index]!);
       } catch (err) {
@@ -1177,6 +1231,8 @@ async function runWithConcurrency<T, R>(
   // Each worker catches its own errors and resolves, so this awaits ALL of them
   // to settle — an all-settled drain, not a fail-fast `Promise.all`.
   await Promise.all(Array.from({ length: width }, () => worker()));
-  if (failed) throw firstError;
+  if (failed) {
+    throw firstError;
+  }
   return results;
 }

package/src/workflows/journal-store.ts

@@ -158,7 +158,9 @@ interface WorkflowJournalRow {
 
 /** Parse a nullable JSON column; malformed values collapse to null. */
 function parseJsonColumn(value: string | null): unknown {
-  if (value === null) return null;
+  if (value === null) {
+    return null;
+  }
   try {
     return JSON.parse(value);
   } catch {

package/src/workflows/leaf-runner.ts

@@ -43,7 +43,7 @@ import { z } from "zod";
 
 import { type AgentEvent, AgentLoop } from "../agent/loop.js";
 import type { AssistantEvent } from "../api/index.js";
-import { getEffectiveProfile } from "../config/default-profile-catalog.js";
+import { resolveDefaultProfileForProvider } from "../config/default-profile-catalog.js";
 import { getConfig } from "../config/loader.js";
 import { isMemoryEnabled } from "../config/memory-v3-gate.js";
 import { isPersonalMemoryAllowed } from "../daemon/trust-context.js";
@@ -328,7 +328,14 @@ function validateProfile(profile: string): string {
 }
 
 function profileExists(profile: string): boolean {
-  return getEffectiveProfile(getConfig().llm.profiles, profile) != null;
+  const { llm } = getConfig();
+  return (
+    resolveDefaultProfileForProvider(
+      llm.profiles,
+      profile,
+      llm.defaultProvider ?? null,
+    ) != null
+  );
 }
 
 /**

package/src/workflows/library.test.ts

@@ -17,8 +17,11 @@ beforeEach(() => {
 });
 
 afterEach(() => {
-  if (prevOverride === undefined) delete process.env.VELLUM_WORKSPACE_DIR;
-  else process.env.VELLUM_WORKSPACE_DIR = prevOverride;
+  if (prevOverride === undefined) {
+    delete process.env.VELLUM_WORKSPACE_DIR;
+  } else {
+    process.env.VELLUM_WORKSPACE_DIR = prevOverride;
+  }
   rmSync(workspaceDir, { recursive: true, force: true });
 });
 

package/src/workflows/library.ts

@@ -101,7 +101,9 @@ function logDuplicateMetaName(name: string, paths: string[]): void {
  */
 function* readWorkflowFiles(): Generator<RawWorkflowFile> {
   const dir = getWorkspaceWorkflowsDir();
-  if (!existsSync(dir)) return;
+  if (!existsSync(dir)) {
+    return;
+  }
   const entries = readdirSync(dir).sort();
 
   // Resolve each form once, keyed by base name (sorted entries keep Map
@@ -114,13 +116,17 @@ function* readWorkflowFiles(): Generator<RawWorkflowFile> {
       continue;
     }
     const entrypoint = join(dir, entry, DIRECTORY_ENTRY);
-    if (existsSync(entrypoint)) dirPaths.set(entry, entrypoint);
+    if (existsSync(entrypoint)) {
+      dirPaths.set(entry, entrypoint);
+    }
   }
 
   // Fail closed on a base-name collision (both forms present).
   const collisions = new Set<string>();
   for (const baseName of dirPaths.keys()) {
-    if (flatPaths.has(baseName)) collisions.add(baseName);
+    if (flatPaths.has(baseName)) {
+      collisions.add(baseName);
+    }
   }
   for (const baseName of collisions) {
     log.error(
@@ -132,9 +138,13 @@ function* readWorkflowFiles(): Generator<RawWorkflowFile> {
 
   // Directory-style first, then flat; a colliding base name is skipped in both.
   for (const [baseName, path] of [...dirPaths, ...flatPaths]) {
-    if (collisions.has(baseName)) continue;
+    if (collisions.has(baseName)) {
+      continue;
+    }
     const raw = readRawWorkflow(baseName, path);
-    if (raw) yield raw;
+    if (raw) {
+      yield raw;
+    }
   }
 }
 

package/src/workflows/run-manager.test.ts

@@ -42,7 +42,9 @@ function makeFakeJournal() {
   const rows = new Map<string, WorkflowRun>();
   const setStats = (id: string, patch: Partial<WorkflowRun>): void => {
     const existing = rows.get(id);
-    if (existing) rows.set(id, { ...existing, ...patch });
+    if (existing) {
+      rows.set(id, { ...existing, ...patch });
+    }
   };
   return {
     rows,
@@ -77,7 +79,9 @@ function makeFakeJournal() {
         patch: Partial<WorkflowRun>,
       ): WorkflowRun | null => {
         const existing = rows.get(id);
-        if (!existing) return null;
+        if (!existing) {
+          return null;
+        }
         const updated = { ...existing, ...patch, updatedAt: Date.now() };
         rows.set(id, updated);
         return updated;
@@ -208,7 +212,9 @@ function makeHarness(opts?: {
     fake,
     resolveLatest: async (result) => {
       const resolve = resolvers[resolvers.length - 1];
-      if (!resolve) throw new Error("no in-flight engine run to resolve");
+      if (!resolve) {
+        throw new Error("no in-flight engine run to resolve");
+      }
       resolve(result);
       // Let the manager's post-completion microtasks (broadcast + wake) settle.
       await new Promise((r) => setTimeout(r, 0));

package/src/workflows/run-manager.ts

@@ -344,9 +344,13 @@ export class WorkflowRunManager {
    * does not resolve.
    */
   private resolveScriptSource(opts: StartWorkflowOptions): string {
-    if (opts.scriptSource !== undefined) return opts.scriptSource;
+    if (opts.scriptSource !== undefined) {
+      return opts.scriptSource;
+    }
     const saved = this.deps.getWorkflow(opts.name);
-    if (!saved) throw new WorkflowNotFoundError(opts.name);
+    if (!saved) {
+      throw new WorkflowNotFoundError(opts.name);
+    }
     return saved.source;
   }
 
@@ -502,7 +506,9 @@ export class WorkflowRunManager {
     ctx: RunContext,
     summary: string,
   ): Promise<void> {
-    if (!ctx.conversationId) return;
+    if (!ctx.conversationId) {
+      return;
+    }
     try {
       await this.deps.wake({
         conversationId: ctx.conversationId,
@@ -600,7 +606,9 @@ const MAX_SUMMARY_RESULT_CHARS = 2000;
 
 /** Bound a result tail for the summary; the full value stays on the run row. */
 function truncateForSummary(s: string, runId: string): string {
-  if (s.length <= MAX_SUMMARY_RESULT_CHARS) return s;
+  if (s.length <= MAX_SUMMARY_RESULT_CHARS) {
+    return s;
+  }
   const omitted = s.length - MAX_SUMMARY_RESULT_CHARS;
   return (
     `${s.slice(0, MAX_SUMMARY_RESULT_CHARS)}… ` +
@@ -611,8 +619,12 @@ function truncateForSummary(s: string, runId: string): string {
 
 /** Compact a workflow result for the summary — JSON for objects, string otherwise. */
 function stringifyResult(value: unknown): string {
-  if (value == null) return "(no result)";
-  if (typeof value === "string") return value;
+  if (value == null) {
+    return "(no result)";
+  }
+  if (typeof value === "string") {
+    return value;
+  }
   try {
     return JSON.stringify(value);
   } catch {

package/src/workflows/sandbox.ts

@@ -279,7 +279,9 @@ export function createWorkflowSandbox(
       // CPU guard). `signal.aborted` remains the hard, total-runtime stop.
       const deadline = { at: Date.now() + interruptDeadlineMs };
       runtime.setInterruptHandler(() => {
-        if (opts.signal?.aborted) return true;
+        if (opts.signal?.aborted) {
+          return true;
+        }
         return Date.now() > deadline.at;
       });
 
@@ -342,7 +344,9 @@ function installNativeBridge(
         const result = await fn(...parsedArgs);
         // `undefined` JSON-stringifies to `undefined`; map it to the VM's
         // undefined so the sync wrapper returns `undefined` rather than NaN.
-        if (result === undefined) return vm.undefined;
+        if (result === undefined) {
+          return vm.undefined;
+        }
         return vm.newString(deterministicStringify(result));
       } finally {
         // The VM is about to resume script execution. Reset the CPU budget so the

package/src/workspace/adaptive-thinking-repair.ts

@@ -51,22 +51,30 @@ export function repairAdaptiveThinkingOnManagedProfiles(
   workspaceDir: string,
 ): void {
   const configPath = join(workspaceDir, "config.json");
-  if (!existsSync(configPath)) return;
+  if (!existsSync(configPath)) {
+    return;
+  }
 
   let config: Record<string, unknown>;
   try {
     const raw = JSON.parse(readFileSync(configPath, "utf-8"));
-    if (!raw || typeof raw !== "object" || Array.isArray(raw)) return;
+    if (!raw || typeof raw !== "object" || Array.isArray(raw)) {
+      return;
+    }
     config = raw as Record<string, unknown>;
   } catch {
     return;
   }
 
   const llm = readObject(config.llm);
-  if (llm === null) return;
+  if (llm === null) {
+    return;
+  }
 
   const profiles = readObject(llm.profiles);
-  if (profiles === null) return;
+  if (profiles === null) {
+    return;
+  }
 
   // Profiles without an explicit provider and without a model-implied provider
   // inherit llm.default.provider at resolution time; an absent
@@ -81,7 +89,9 @@ export function repairAdaptiveThinkingOnManagedProfiles(
 
   for (const name of TARGET_PROFILES) {
     const profile = readObject(profiles[name]);
-    if (profile === null) continue;
+    if (profile === null) {
+      continue;
+    }
 
     // Only patch managed Anthropic profiles.
     // Legacy profiles created before the `source` metadata field was introduced
@@ -91,17 +101,25 @@ export function repairAdaptiveThinkingOnManagedProfiles(
     // explicit `provider`, else the provider implied by a known catalog
     // `model`, else the inherited llm.default.provider. Explicit
     // `source: "user"` profiles are always skipped.
-    if (profile.source === "user") continue;
-    if (profile.source !== undefined && profile.source !== "managed") continue;
+    if (profile.source === "user") {
+      continue;
+    }
+    if (profile.source !== undefined && profile.source !== "managed") {
+      continue;
+    }
     const effectiveProvider = resolveEffectiveProvider(
       profile,
       defaultProvider,
     );
-    if (effectiveProvider !== "anthropic") continue;
+    if (effectiveProvider !== "anthropic") {
+      continue;
+    }
 
     // Skip if thinking is already enabled.
     const thinking = readObject(profile.thinking);
-    if (thinking !== null && thinking.enabled === true) continue;
+    if (thinking !== null && thinking.enabled === true) {
+      continue;
+    }
 
     profile.thinking = { ...ADAPTIVE_THINKING };
     profiles[name] = profile;
@@ -119,10 +137,14 @@ function resolveEffectiveProvider(
   profile: Record<string, unknown>,
   defaultProvider: string,
 ): string {
-  if (typeof profile.provider === "string") return profile.provider;
+  if (typeof profile.provider === "string") {
+    return profile.provider;
+  }
   if (typeof profile.model === "string") {
     const implied = getCatalogProviderForModel(profile.model);
-    if (implied !== undefined) return implied;
+    if (implied !== undefined) {
+      return implied;
+    }
   }
   return defaultProvider;
 }