@agent-native/core 0.125.0 → 0.126.0

package/src/sharing/actions/share-resource.ts

@@ -8,6 +8,7 @@ import { renderEmail, emailStrong } from "../../server/email-template.js";
 import { sendEmail, isEmailConfigured } from "../../server/email.js";
 import { invalidateCollabAccessCache } from "../../server/poll.js";
 import { getRequestUserEmail } from "../../server/request-context.js";
+import { getUserProfile } from "../../user-profile/store.js";
 import { assertAccess, ForbiddenError } from "../access.js";
 import { requireShareableResource } from "../registry.js";
 import {
@@ -298,10 +299,9 @@ export default defineAction({
         const appName =
           process.env.APP_NAME || process.env.VITE_APP_NAME || "Agent Native";
         let brandLogoUrl: string | undefined;
-        if (reg.getShareEmailLogoUrl) {
+        if (reg.getLogoUrl) {
           try {
-            brandLogoUrl =
-              (await reg.getShareEmailLogoUrl(resource)) ?? undefined;
+            brandLogoUrl = (await reg.getLogoUrl(resource)) ?? undefined;
           } catch (err) {
             console.error(
               "[share-resource] brand logo resolver failed; using default logo:",
@@ -309,9 +309,51 @@ export default defineAction({
             );
           }
         }
+        let brandName = appName;
+        if (reg.getBrandName) {
+          try {
+            brandName = (await reg.getBrandName(resource))?.trim() || appName;
+          } catch (err) {
+            console.error(
+              "[share-resource] brand name resolver failed; using app name:",
+              err,
+            );
+          }
+        }
+        let fromName: string | undefined;
+        let replyTo: string | undefined;
+        if (reg.getSender) {
+          try {
+            const sender = await reg.getSender(resource, {
+              sender: await getUserProfile(actor),
+            });
+            fromName = sender?.fromName?.trim() || undefined;
+            replyTo = sender?.replyTo?.trim() || undefined;
+          } catch (err) {
+            console.error(
+              "[share-resource] sender resolver failed; using default sender:",
+              err,
+            );
+          }
+        }
+        let heroHtml: string | undefined;
+        if (reg.getHeroHtml) {
+          try {
+            heroHtml =
+              (await reg.getHeroHtml(resource, {
+                href: notificationUrl,
+                alt: resourceTitle,
+              })) ?? undefined;
+          } catch (err) {
+            console.error(
+              "[share-resource] hero html resolver failed; omitting preview:",
+              err,
+            );
+          }
+        }
         const subject = `${actor} shared "${resourceTitle}" with you on ${appName}`;
         const { html, text } = renderEmail({
-          brandName: appName,
+          brandName,
           brandLogoUrl,
           preheader: subject,
           heading: "You've been given access",
@@ -319,10 +361,18 @@ export default defineAction({
             `${emailStrong(actor)} has shared the ${reg.displayName} ${emailStrong(resourceTitle)} with you as a ${emailStrong(args.role)}.`,
             `Use the button below to open it. If prompted, sign in with ${emailStrong(principalId)}.`,
           ],
+          heroHtml,
           cta: { label: `Open ${reg.displayName}`, url: notificationUrl },
           footer: `You received this because ${actor} granted you ${args.role} access.`,
         });
-        await sendEmail({ to: principalId, subject, html, text });
+        await sendEmail({
+          to: principalId,
+          subject,
+          html,
+          text,
+          fromName,
+          replyTo,
+        });
       } catch (err) {
         console.error(
           "[share-resource] failed to send share notification:",

package/src/sharing/registry.ts

@@ -17,6 +17,8 @@
  *   });
  */
 
+import type { UserProfile } from "../user-profile/shared.js";
+
 export interface ShareableResourceRegistration {
   /** Stable identifier used across actions, UI, and analytics. e.g. "document". */
   type: string;
@@ -42,8 +44,46 @@ export interface ShareableResourceRegistration {
    * (e.g. the sharing org's own logo), or undefined to fall back. Receives the
    * resource row being shared so the template can scope the logo to its org.
    */
-  getShareEmailLogoUrl?: (
+  getLogoUrl?: (
+    resource: any,
+  ) => string | undefined | Promise<string | undefined>;
+  /**
+   * Optional resolver for the brand name shown beside the logo in the
+   * share-notification email header. Return a display name (e.g. the sharing
+   * org's name) to override the app name, or undefined to keep the default.
+   * Receives the resource row being shared.
+   */
+  getBrandName?: (
+    resource: any,
+  ) => string | undefined | Promise<string | undefined>;
+  /**
+   * Optional resolver for who the share-notification email appears to come
+   * from. `fromName` changes only the display name and keeps the configured,
+   * domain-verified sending address (e.g. "Alice via Clips"); `replyTo` routes
+   * replies to the sharer. `ctx.sender` is the account doing the sharing, so the
+   * template decides how to present them (name, email, avatar).
+   *
+   * Do not put a user's address in the sending address itself — SPF/DKIM sign
+   * the app's domain, so recipients would bounce or spam-filter it.
+   */
+  getSender?: (
+    resource: any,
+    ctx: { sender: UserProfile },
+  ) =>
+    | { fromName?: string; replyTo?: string }
+    | undefined
+    | Promise<{ fromName?: string; replyTo?: string } | undefined>;
+  /**
+   * Optional resolver for a preview block shown above the CTA in the
+   * share-notification email — e.g. a recording thumbnail with a play badge.
+   * Return trusted HTML (built by template code, dynamic values escaped) that
+   * is injected verbatim, or undefined to omit it. `ctx.href` is the resolved
+   * link to the resource; `ctx.alt` is its title. The template owns the entire
+   * preview markup so the generic share action stays app-agnostic.
+   */
+  getHeroHtml?: (
     resource: any,
+    ctx: { href: string; alt?: string },
   ) => string | undefined | Promise<string | undefined>;
   /**
    * Drizzle DB accessor from the template's server/db/index.ts. Required —