@vellumai/assistant 0.7.1 → 0.7.2

package/src/messaging/providers/gmail/types.ts

@@ -95,52 +95,3 @@ export interface GmailModifyRequest {
 
 /** Message format for GET requests */
 export type GmailMessageFormat = "minimal" | "full" | "raw" | "metadata";
-
-/** Attachment data from the Gmail API */
-export interface GmailAttachment {
-  size: number;
-  data: string; // base64url-encoded
-}
-
-/** Gmail filter criteria */
-export interface GmailFilterCriteria {
-  from?: string;
-  to?: string;
-  subject?: string;
-  query?: string;
-  hasAttachment?: boolean;
-  excludeChats?: boolean;
-  size?: number;
-  sizeComparison?: "larger" | "smaller";
-}
-
-/** Gmail filter action */
-export interface GmailFilterAction {
-  addLabelIds?: string[];
-  removeLabelIds?: string[];
-  forward?: string;
-}
-
-/** Gmail filter */
-export interface GmailFilter {
-  id: string;
-  criteria: GmailFilterCriteria;
-  action: GmailFilterAction;
-}
-
-/** Gmail filters list response */
-export interface GmailFiltersListResponse {
-  filter?: GmailFilter[];
-}
-
-/** Gmail vacation auto-reply settings */
-export interface GmailVacationSettings {
-  enableAutoReply: boolean;
-  responseSubject?: string;
-  responseBodyPlainText?: string;
-  responseBodyHtml?: string;
-  restrictToContacts?: boolean;
-  restrictToDomain?: boolean;
-  startTime?: string; // epoch ms as string
-  endTime?: string; // epoch ms as string
-}

package/src/messaging/providers/slack/adapter.ts

@@ -11,7 +11,6 @@ import {
 } from "@vellumai/slack-text";
 
 import { findContactChannel } from "../../../contacts/contact-store.js";
-import { upsertContactChannel } from "../../../contacts/contacts-write.js";
 import type { OAuthConnection } from "../../../oauth/connection.js";
 import { resolveOAuthConnection } from "../../../oauth/connection-resolver.js";
 import { isProviderConnected } from "../../../oauth/oauth-store.js";
@@ -163,18 +162,6 @@ async function resolveUserName(
       resp.user.real_name ||
       resp.user.name;
     userNameCache.set(userId, name);
-
-    // Persist to contacts for future sessions
-    try {
-      upsertContactChannel({
-        sourceChannel: "slack",
-        externalUserId: userId,
-        displayName: name,
-      });
-    } catch {
-      // Non-fatal — caching failure shouldn't break messaging
-    }
-
     return name;
   } catch {
     return userId;
@@ -433,24 +420,7 @@ export const slackProvider: MessagingProvider = {
         const dmUserId = conv.metadata.dmUserId as string;
         conv.name = await resolveUserName(auth, dmUserId);
 
-        // Persist the DM channel ID so future sends skip conversations.open
-        try {
-          const existing = findContactChannel({
-            channelType: "slack",
-            externalUserId: dmUserId,
-          });
-          if (existing && !existing.channel.externalChatId) {
-            upsertContactChannel({
-              contactId: existing.contact.id,
-              sourceChannel: "slack",
-              externalUserId: dmUserId,
-              externalChatId: conv.id,
-              displayName: conv.name,
-            });
-          }
-        } catch {
-          // Non-fatal
-        }
+
       }
     }
 

package/src/messaging/providers/slack/types.ts

@@ -76,11 +76,6 @@ export interface SlackUser {
   deleted?: boolean;
 }
 
-export interface SlackUsersListResponse extends SlackApiResponse {
-  members: SlackUser[];
-  response_metadata?: { next_cursor?: string };
-}
-
 export interface SlackUserInfoResponse extends SlackApiResponse {
   user: SlackUser;
 }
@@ -110,35 +105,8 @@ export interface SlackSearchMatch {
   thread_ts?: string;
 }
 
-export interface SlackConversationInfoResponse extends SlackApiResponse {
-  channel: SlackConversation;
-}
-
 export interface SlackConversationsOpenResponse extends SlackApiResponse {
   channel: { id: string };
 }
 
-export type SlackReactionAddResponse = SlackApiResponse;
-
-export type SlackConversationJoinResponse = SlackApiResponse & {
-  channel?: SlackConversation;
-};
-
-export type SlackConversationLeaveResponse = SlackApiResponse;
-
-export interface SlackChatDeleteResponse extends SlackApiResponse {
-  channel: string;
-  ts: string;
-}
-
-export interface SlackChatUpdateResponse extends SlackApiResponse {
-  channel: string;
-  ts: string;
-  text: string;
-}
-
 export type SlackConversationMarkResponse = SlackApiResponse;
-
-export interface SlackPostEphemeralResponse extends SlackApiResponse {
-  message_ts: string;
-}

package/src/notifications/README.md

@@ -133,7 +133,7 @@ The system produces **three distinct copy outputs** per notification:
 
 | Output                    | Purpose                                          | Verbosity                |
 | ------------------------- | ------------------------------------------------ | ------------------------ |
-| `title` + `body`          | Native notification popup (macOS/iOS banner)     | Short and glanceable     |
+| `title` + `body`          | Native notification popup (macOS banner)         | Short and glanceable     |
 | `deliveryText`            | Channel-native chat message text (Telegram)      | Natural chat phrasing    |
 | Conversation seed message | Opening message in the notification conversation | Richer and context-aware |
 
@@ -185,7 +185,7 @@ Reminder. Take out the trash. Action required.
 
 ## Conversation Surfacing via `notification_conversation_created` Event (Creation-Only)
 
-The `notification_conversation_created` SSE event is emitted **only when a brand-new conversation is actually created** by the broadcaster. Reused conversations do not trigger this event — the macOS/iOS client already knows about the conversation from the original creation.
+The `notification_conversation_created` SSE event is emitted **only when a brand-new conversation is actually created** by the broadcaster. Reused conversations do not trigger this event — the macOS client already knows about the conversation from the original creation.
 
 This is enforced in `broadcaster.ts` by gating the event emission on `pairing.createdNewConversation === true`:
 
@@ -214,7 +214,7 @@ The SSE event payload:
 }
 ```
 
-The macOS/iOS client listens for this event and surfaces the conversation in the sidebar, enabling deep-link navigation to the notification conversation.
+The macOS client listens for this event and surfaces the conversation in the sidebar, enabling deep-link navigation to the notification conversation.
 
 ### Per-Dispatch Conversation Callback
 
@@ -223,7 +223,7 @@ The macOS/iOS client listens for this event and surfaces the conversation in the
 **Important distinction between the two callbacks:**
 
 - **Per-dispatch `options.onConversationCreated`**: Fires for **both** new and reused vellum conversation pairings. Callers like `dispatchGuardianQuestion` rely on this to create delivery bookkeeping rows before `emitNotificationSignal()` returns, regardless of whether the conversation was newly created or reused.
-- **Class-level `this.onConversationCreated` (SSE broadcast)**: Fires **only** when a brand-new conversation is created (`createdNewConversation === true && strategy === 'start_new_conversation'`). This emits the `notification_conversation_created` SSE event so macOS/iOS clients surface the new conversation in the sidebar. Reused conversations do not trigger this event because the client already knows about the conversation.
+- **Class-level `this.onConversationCreated` (SSE broadcast)**: Fires **only** when a brand-new conversation is created (`createdNewConversation === true && strategy === 'start_new_conversation'`). This emits the `notification_conversation_created` SSE event so macOS clients surface the new conversation in the sidebar. Reused conversations do not trigger this event because the client already knows about the conversation.
 
 ## Schedule Routing Metadata and Trigger-Time Enforcement
 
@@ -295,7 +295,7 @@ Local SSE via the daemon's broadcast mechanism. The `VellumAdapter` emits a `not
 - `title` and `body` -- rendered notification copy
 - `deepLinkMetadata` -- optional metadata for navigating to the relevant context (e.g. `{ conversationId }`)
 
-The macOS/iOS client posts a native `UNUserNotificationCenter` notification from this payload. When the user taps the notification, the client uses `deepLinkMetadata` to navigate to the relevant conversation.
+The macOS client posts a native `UNUserNotificationCenter` notification from this payload. When the user taps the notification, the client uses `deepLinkMetadata` to navigate to the relevant conversation.
 
 ### Telegram (when guardian binding exists)
 
@@ -480,7 +480,7 @@ Three SQLite tables form the audit chain:
 
 ### Client Delivery Ack
 
-For vellum (macOS/iOS) deliveries, the audit trail now extends past the SSE broadcast to the actual OS notification post. The `notification_intent` message carries an optional `deliveryId` that the client echoes back in a `notification_intent_result` ack after `UNUserNotificationCenter.add()` completes (or fails).
+For vellum (macOS) deliveries, the audit trail now extends past the SSE broadcast to the actual OS notification post. The `notification_intent` message carries an optional `deliveryId` that the client echoes back in a `notification_intent_result` ack after `UNUserNotificationCenter.add()` completes (or fails).
 
 The ack populates three columns on `notification_deliveries`:
 
@@ -541,10 +541,10 @@ Preferences are sanitized against prompt injection (angle brackets replaced with
 The decision engine and preference extractor pick their per-call LLM config
 from the unified `llm` block. Override defaults by setting either of:
 
-| Key                                  | Type    | Default        | Description                                                                 |
-| ------------------------------------ | ------- | -------------- | --------------------------------------------------------------------------- |
-| `llm.callSites.notificationDecision` | object  | _(unset)_      | Provider/model/effort/etc. override for the decision engine call site       |
-| `llm.callSites.preferenceExtraction` | object  | _(unset)_      | Provider/model/effort/etc. override for the preference extractor call site  |
+| Key                                  | Type   | Default   | Description                                                                |
+| ------------------------------------ | ------ | --------- | -------------------------------------------------------------------------- |
+| `llm.callSites.notificationDecision` | object | _(unset)_ | Provider/model/effort/etc. override for the decision engine call site      |
+| `llm.callSites.preferenceExtraction` | object | _(unset)_ | Provider/model/effort/etc. override for the preference extractor call site |
 
 When a call site override is unset, the resolver falls back to `llm.default`.
 

package/src/notifications/broadcaster.ts

@@ -211,7 +211,7 @@ export class NotificationBroadcaster {
       );
 
       // For the vellum channel, merge the conversationId into deep-link metadata
-      // so the macOS/iOS client can navigate directly to the notification conversation.
+      // so the macOS client can navigate directly to the notification conversation.
       let deepLinkTarget = decision.deepLinkTarget;
       if (channel === "vellum" && pairing.conversationId) {
         deepLinkTarget = {

package/src/notifications/guardian-question-mode.ts

@@ -6,12 +6,12 @@
  * fields like `toolName`.
  */
 
-export type GuardianQuestionRequestKind =
+type GuardianQuestionRequestKind =
   | "pending_question"
   | "tool_approval"
   | "tool_grant_request"
   | "access_request";
-export type GuardianQuestionInstructionMode = "approval" | "answer";
+type GuardianQuestionInstructionMode = "approval" | "answer";
 
 interface GuardianRequestKindModeConfig {
   defaultMode: GuardianQuestionInstructionMode;
@@ -48,12 +48,12 @@ interface GuardianQuestionPayloadBaseWithDiscriminator extends GuardianQuestionP
   [key: string]: unknown;
 }
 
-export interface GuardianRequestModeInput {
+interface GuardianRequestModeInput {
   kind: unknown;
   toolName?: unknown;
 }
 
-export interface GuardianRequestTextInput {
+interface GuardianRequestTextInput {
   requestCode: string;
   questionText?: string | null;
   toolName?: string | null;
@@ -136,7 +136,7 @@ export type GuardianQuestionPayload =
   | ToolGrantGuardianPayload
   | AccessRequestGuardianPayload;
 
-export interface GuardianQuestionModeResolution {
+interface GuardianQuestionModeResolution {
   mode: GuardianQuestionInstructionMode;
   requestKind: GuardianQuestionRequestKind | null;
 }

package/src/oauth/connect-orchestrator.ts

@@ -19,6 +19,7 @@
  * - Running identity verifiers
  */
 
+import { emitPostConnectNudge } from "../home/post-connect-feed.js";
 import type { TokenEndpointAuthMethod } from "../security/oauth2.js";
 import { prepareOAuth2Flow, startOAuth2Flow } from "../security/oauth2.js";
 import { getLogger } from "../util/logger.js";
@@ -250,6 +251,7 @@ export async function orchestrateOAuthConnect(
               },
               "Deferred OAuth2 flow completed — tokens stored",
             );
+            void emitPostConnectNudge(options.service);
             options.onDeferredComplete?.({
               success: true,
               service: options.service,
@@ -371,6 +373,8 @@ export async function orchestrateOAuthConnect(
       "orchestrateOAuthConnect: tokens stored, connect complete",
     );
 
+    void emitPostConnectNudge(options.service);
+
     return {
       success: true,
       deferred: false,

package/src/oauth/credential-token-resolver.ts

@@ -43,9 +43,7 @@ export interface ConnectionAccessTokenResult {
  * providers (e.g. credential-health checks) must resolve the path through here
  * rather than assuming the OAuth access-token path.
  */
-export function manualTokenAccessCredentialKey(
-  provider: string,
-): string | null {
+function manualTokenAccessCredentialKey(provider: string): string | null {
   switch (provider) {
     case "slack_channel":
       return credentialKey("slack_channel", "bot_token");

package/src/oauth/manual-token-connection.ts

@@ -19,10 +19,6 @@ import {
   upsertApp,
 } from "./oauth-store.js";
 
-// Re-export from the centralized resolver so existing callers that import
-// from this module continue to work without changes.
-export { manualTokenAccessCredentialKey } from "./credential-token-resolver.js";
-
 const log = getLogger("manual-token-connection");
 
 /** Sentinel client_id used for non-OAuth providers that don't have a real app. */

package/src/outbound-proxy/index.ts

@@ -3,40 +3,11 @@
 // ---------------------------------------------------------------------------
 
 export type {
-  CredentialInjectionTemplate,
-  CredentialInjectionType,
   PolicyDecision,
-  PolicyDecisionAmbiguous,
-  PolicyDecisionAskMissingCredential,
-  PolicyDecisionAskUnauthenticated,
-  PolicyDecisionMatched,
-  PolicyDecisionMissing,
-  PolicyDecisionUnauthenticated,
   ProxyApprovalCallback,
   ProxyApprovalRequest,
-  ProxyEnvVars,
-  ProxySession,
-  ProxySessionConfig,
-  ProxySessionId,
-  ProxySessionStatus,
-  RequestTargetContext,
 } from "@vellumai/egress-proxy";
 
-// ---------------------------------------------------------------------------
-// Conversation core — re-exported from @vellumai/egress-proxy shared package
-// ---------------------------------------------------------------------------
-
-export type { ManagedSession, SessionStartHooks } from "@vellumai/egress-proxy";
-
-// ---------------------------------------------------------------------------
-// Host pattern matching
-// ---------------------------------------------------------------------------
-
-export type {
-  HostMatchKind,
-  MatchHostPatternOptions,
-} from "./host-pattern-match.js";
-
 // Certificate management
 export {
   ensureCombinedCABundle,
@@ -49,7 +20,6 @@ export {
 export type { RewriteCallback } from "./mitm-handler.js";
 
 // Router
-export type { RouteDecision, RouteReason } from "./router.js";
 export { routeConnection } from "./router.js";
 
 // CONNECT tunnel
@@ -60,15 +30,9 @@ export { evaluateRequest, evaluateRequestWithApproval } from "./policy.js";
 // HTTP forwarder
 
 // Proxy server
-export type { MitmHandlerConfig, ProxyServerConfig } from "./server.js";
+export type { ProxyServerConfig } from "./server.js";
 export { createProxyServer } from "./server.js";
 
-// Sidecar configuration
-export type { SidecarConfig } from "./config.js";
-
-// Health / readiness server
-export type { HealthServerOptions } from "./health.js";
-
 // Logging/diagnostics
 export type { CredentialRefTrace, ProxyDecisionTrace } from "./logging.js";
 export {

package/src/outbound-proxy/logging.ts

@@ -7,7 +7,7 @@
  * candidate counts.
  */
 
-import type { PolicyDecision } from "./types.js";
+import type { PolicyDecision } from "@vellumai/egress-proxy";
 
 const REDACTED = "[REDACTED]";
 

package/src/outbound-proxy/policy.ts

@@ -3,16 +3,17 @@
  * injection templates and emits deterministic policy decisions.
  */
 
+import type {
+  CredentialInjectionTemplate,
+  PolicyDecision,
+  RequestTargetContext,
+} from "@vellumai/egress-proxy";
+
 import {
   compareMatchSpecificity,
   type HostMatchKind,
   matchHostPattern,
 } from "./host-pattern-match.js";
-import type {
-  CredentialInjectionTemplate,
-  PolicyDecision,
-  RequestTargetContext,
-} from "./types.js";
 
 interface MatchCandidate {
   credentialId: string;

package/src/outbound-proxy/router.ts

@@ -7,8 +7,9 @@
  * pay the cost of TLS termination, cert issuance, and request rewriting.
  */
 
+import type { CredentialInjectionTemplate } from "@vellumai/egress-proxy";
+
 import { matchHostPattern } from "./host-pattern-match.js";
-import type { CredentialInjectionTemplate } from "./types.js";
 
 // ---- Public types ----------------------------------------------------------