@vellumai/assistant 0.7.1 → 0.7.2

package/src/runtime/btw-sidechain.ts

@@ -1,5 +1,4 @@
 import type { LLMCallSite } from "../config/schemas/llm.js";
-import { buildToolDefinitions } from "../daemon/conversation-tool-setup.js";
 import { buildSystemPrompt } from "../prompts/system-prompt.js";
 import {
   createTimeout,
@@ -27,7 +26,7 @@ export interface RunBtwSidechainParams {
   provider?: Provider;
   messages?: Message[];
   systemPrompt?: string;
-  tools?: ToolDefinition[];
+  tools: ToolDefinition[];
   maxTokens?: number;
   /**
    * Unified call-site identifier. The provider layer resolves
@@ -68,7 +67,7 @@ export async function runBtwSidechain(
     throw new Error("BTW side-chain requires a provider");
   }
 
-  const tools = params.tools ?? buildToolDefinitions();
+  const tools = params.tools;
   const history = params.messages ?? params.conversation?.getMessages() ?? [];
   const messages = [...history, userMessage(trimmedContent)];
   const systemPrompt =

package/src/runtime/channel-invite-transport.ts

@@ -20,57 +20,11 @@ import { slackInviteAdapter } from "./channel-invite-transports/slack.js";
 import { telegramInviteAdapter } from "./channel-invite-transports/telegram.js";
 import { voiceInviteAdapter } from "./channel-invite-transports/voice.js";
 import { whatsappInviteAdapter } from "./channel-invite-transports/whatsapp.js";
-
 // ---------------------------------------------------------------------------
 // Types
 // ---------------------------------------------------------------------------
-
-export interface InviteShareLink {
-  /** The full URL the recipient can open to redeem the invite. */
-  url: string;
-  /** Human-readable text suitable for display alongside the link. */
-  displayText: string;
-}
-
-export interface ChannelInviteAdapter {
-  /** The channel this adapter handles. */
-  channel: ChannelId;
-
-  /**
-   * Build a channel-specific shareable link (e.g. Telegram deep link).
-   * Optional — not all channels support link-based invites.
-   */
-  buildShareLink?(params: {
-    rawToken: string;
-    sourceChannel: ChannelId;
-  }): InviteShareLink;
-
-  /**
-   * Extract a channel-specific invite token from an inbound message
-   * (e.g. Telegram `/start iv_<token>`). Optional — only needed for
-   * channels with link-based invites.
-   */
-  extractInboundToken?(params: {
-    commandIntent?: Record<string, unknown>;
-    content: string;
-    sourceMetadata?: Record<string, unknown>;
-  }): string | undefined;
-
-  /**
-   * Resolve the channel-specific handle to reach the assistant (e.g.
-   * "@botName", "+15551234567", "hello@vellum.me").
-   * Returns `undefined` when the handle cannot be resolved (e.g.
-   * credentials not yet configured).
-   */
-  resolveChannelHandle?(): string | undefined;
-
-  /**
-   * Async variant of `resolveChannelHandle` for adapters that need to
-   * perform I/O (e.g. querying a provider API for the assigned address).
-   * When both are present, `resolveAdapterHandle()` prefers this method.
-   */
-  resolveChannelHandleAsync?(): Promise<string | undefined>;
-}
+import type { ChannelInviteAdapter } from "./channel-invite-types.js";
+export type { ChannelInviteAdapter, InviteShareLink } from "./channel-invite-types.js";
 
 // ---------------------------------------------------------------------------
 // Registry

package/src/runtime/channel-invite-transports/email.ts

@@ -12,7 +12,7 @@
  */
 
 import { getNestedValue, loadRawConfig } from "../../config/loader.js";
-import type { ChannelInviteAdapter } from "../channel-invite-transport.js";
+import type { ChannelInviteAdapter } from "../channel-invite-types.js";
 
 // ---------------------------------------------------------------------------
 // Adapter implementation

package/src/runtime/channel-invite-transports/slack.ts

@@ -9,7 +9,7 @@
 
 import type { ChannelId } from "../../channels/types.js";
 import { getConfig } from "../../config/loader.js";
-import type { ChannelInviteAdapter } from "../channel-invite-transport.js";
+import type { ChannelInviteAdapter } from "../channel-invite-types.js";
 
 // ---------------------------------------------------------------------------
 // Slack bot info resolution

package/src/runtime/channel-invite-transports/telegram.ts

@@ -26,7 +26,7 @@ import { getLogger } from "../../util/logger.js";
 import type {
   ChannelInviteAdapter,
   InviteShareLink,
-} from "../channel-invite-transport.js";
+} from "../channel-invite-types.js";
 
 // ---------------------------------------------------------------------------
 // Bot username resolution

package/src/runtime/channel-invite-transports/voice.ts

@@ -15,7 +15,7 @@ import type { ChannelId } from "../../channels/types.js";
 import type {
   ChannelInviteAdapter,
   InviteShareLink,
-} from "../channel-invite-transport.js";
+} from "../channel-invite-types.js";
 
 // ---------------------------------------------------------------------------
 // Adapter implementation

package/src/runtime/channel-invite-transports/whatsapp.ts

@@ -10,7 +10,7 @@
 
 import type { ChannelId } from "../../channels/types.js";
 import { getConfig } from "../../config/loader.js";
-import type { ChannelInviteAdapter } from "../channel-invite-transport.js";
+import type { ChannelInviteAdapter } from "../channel-invite-types.js";
 
 // ---------------------------------------------------------------------------
 // Phone number resolution

package/src/runtime/channel-invite-types.ts

@@ -0,0 +1,54 @@
+/**
+ * Types extracted from channel-invite-transport.ts to break the
+ * transport ↔ channel-invite-transports/* cycles (×5).
+ */
+
+import type { ChannelId } from "../channels/types.js";
+
+export interface InviteShareLink {
+  /** The full URL the recipient can open to redeem the invite. */
+  url: string;
+  /** Human-readable text suitable for display alongside the link. */
+  displayText: string;
+}
+
+export interface ChannelInviteAdapter {
+  /** The channel this adapter handles. */
+  channel: ChannelId;
+
+  /**
+   * Build a channel-specific shareable link (e.g. Telegram deep link).
+   * Optional — not all channels support link-based invites.
+   */
+  buildShareLink?(params: {
+    rawToken: string;
+    sourceChannel: ChannelId;
+  }): InviteShareLink;
+
+  /**
+   * Extract a channel-specific invite token from an inbound message
+   * (e.g. Telegram `/start iv_<token>`). Optional — only needed for
+   * channels with link-based invites.
+   */
+  extractInboundToken?(params: {
+    commandIntent?: Record<string, unknown>;
+    content: string;
+    sourceMetadata?: Record<string, unknown>;
+  }): string | undefined;
+
+  /**
+   * Resolve the channel-specific handle to reach the assistant (e.g.
+   * // generic-examples:ignore-next-line — reason: illustrative docstring examples, not real data
+   * "@botName", "+15551234567", "hello@vellum.me").
+   * Returns `undefined` when the handle cannot be resolved (e.g.
+   * credentials not yet configured).
+   */
+  resolveChannelHandle?(): string | undefined;
+
+  /**
+   * Async variant of `resolveChannelHandle` for adapters that need to
+   * perform I/O (e.g. querying a provider API for the assigned address).
+   * When both are present, `resolveAdapterHandle()` prefers this method.
+   */
+  resolveChannelHandleAsync?(): Promise<string | undefined>;
+}

package/src/runtime/channel-readiness-service.ts

@@ -1,3 +1,8 @@
+import {
+  normalizePublicBaseUrl,
+  resolveTwilioPublicBaseUrl,
+} from "@vellumai/service-contracts/twilio-ingress";
+
 import { resolveTwilioPhoneNumber } from "../calls/twilio-config.js";
 import { hasTwilioCredentials } from "../calls/twilio-rest.js";
 import { getChannelInvitePolicy } from "../channels/config.js";
@@ -19,31 +24,35 @@ import type {
 /** Remote check results are cached for 5 minutes before being considered stale. */
 export const REMOTE_TTL_MS = 5 * 60 * 1000;
 
-function hasIngressConfigured(): boolean {
+function hasIngressConfigured(options: { twilio?: boolean } = {}): boolean {
   try {
     const raw = loadRawConfig();
     const ingress = (raw?.ingress ?? {}) as Record<string, unknown>;
-    const publicBaseUrl = (ingress.publicBaseUrl as string) ?? "";
+    const effectiveBaseUrl = options.twilio
+      ? (resolveTwilioPublicBaseUrl(ingress) ?? "")
+      : (normalizePublicBaseUrl(ingress.publicBaseUrl) ?? "");
     const enabled =
       (ingress.enabled as boolean | undefined) ??
-      (publicBaseUrl ? true : false);
-    return enabled && publicBaseUrl.length > 0;
+      (effectiveBaseUrl ? true : false);
+    return enabled && effectiveBaseUrl.trim().length > 0;
   } catch {
     return false;
   }
 }
 
-function hasWebhookRoutingConfigured(allowManagedCallbacks = false): {
+function hasWebhookRoutingConfigured(
+  allowManagedCallbacks = false,
+  options: { twilio?: boolean } = {},
+): {
   configured: boolean;
   usesManagedCallbacks: boolean;
 } {
-  const ingressConfigured = hasIngressConfigured();
+  const ingressConfigured = hasIngressConfigured(options);
   if (ingressConfigured) {
     return { configured: true, usesManagedCallbacks: false };
   }
 
-  const usesManagedCallbacks =
-    allowManagedCallbacks && getIsPlatform();
+  const usesManagedCallbacks = allowManagedCallbacks && getIsPlatform();
   return {
     configured: usesManagedCallbacks,
     usesManagedCallbacks,
@@ -79,19 +88,29 @@ async function checkCredential(
 }
 
 /** Check that public ingress is configured and enabled. */
-function checkIngress(allowManagedCallbacks = false): ReadinessCheckResult {
+function checkIngress(
+  allowManagedCallbacks = false,
+  options: { twilio?: boolean } = {},
+): ReadinessCheckResult {
   const { configured, usesManagedCallbacks } = hasWebhookRoutingConfigured(
     allowManagedCallbacks,
+    options,
   );
   return check(
     "ingress",
     configured,
     usesManagedCallbacks
       ? "Managed platform callback routing is configured"
-      : "Public ingress URL is configured",
+      : options.twilio
+        ? "Twilio public ingress URL is configured"
+        : "Public ingress URL is configured",
     allowManagedCallbacks
-      ? "No public ingress URL or managed callback route is configured"
-      : "Public ingress URL is not configured or disabled",
+      ? options.twilio
+        ? "No Twilio public ingress URL or managed callback route is configured"
+        : "No public ingress URL or managed callback route is configured"
+      : options.twilio
+        ? "Twilio public ingress URL is not configured or disabled"
+        : "Public ingress URL is not configured or disabled",
   );
 }
 
@@ -102,7 +121,7 @@ const voiceProbe: ChannelProbe = {
   async runLocalChecks(): Promise<ReadinessCheckResult[]> {
     const hasCreds = await hasTwilioCredentials();
     const hasPhone = !!resolveTwilioPhoneNumber();
-    const ingress = checkIngress(true);
+    const ingress = checkIngress(true, { twilio: true });
 
     return [
       check(