@indexnetwork/protocol 4.3.6 → 4.3.7-rc.317.1

package/dist/opportunity/opportunity.tools.js

@@ -3,7 +3,8 @@ import { requestContext } from "../shared/observability/request-context.js";
 import { success, error, UUID_REGEX } from "../shared/agent/tool.helpers.js";
 import { deriveDiscoveryNetworkIds, focusedIntentId, focusedNetworkId, focusedNetworkLabel } from "../shared/agent/tool.scope.js";
 import { MINIMAL_MAIN_TEXT_MAX_CHARS, getPrimaryActionLabel, SECONDARY_ACTION_LABEL } from "./opportunity.labels.js";
-import { viewerCentricCardSummary, narratorRemarkFromReasoning, stripUuids } from "./opportunity.presentation.js";
+import { narratorRemarkFromReasoning, stripUuids } from "./opportunity.presentation.js";
+import { safeFallbackSummary, getSafePresentationOrSkip } from "./opportunity.safe-presentation.js";
 import { runDiscoverFromQuery, continueDiscovery } from "./opportunity.discover.js";
 import { OpportunityPresenter, gatherPresenterContext } from "./opportunity.presenter.js";
 import { loadNegotiationContext } from "./negotiation-context.loader.js";
@@ -61,17 +62,18 @@ const logger = protocolLogger("ChatTools:Opportunity");
  *   accepted opp and can engage or ignore.
  */
 export function resolveActionableLinkKind(input) {
-    const { status, viewerRole, viewerApproved } = input;
+    const { status, viewerRole, viewerApproved, viewerActedAt } = input;
     const isIntroducer = viewerRole === "introducer";
+    const hasViewerActed = !!viewerActedAt;
     if (status === "accepted") {
         return isIntroducer ? null : "outreach";
     }
     if (status === "pending") {
-        return isIntroducer ? null : "connect";
+        return isIntroducer || hasViewerActed ? null : "connect";
     }
     if (status === "draft" || status === "latent") {
         if (!isIntroducer)
-            return "send_direct";
+            return hasViewerActed ? null : "send_direct";
         return viewerApproved === true ? null : "approve_introduction";
     }
     return null;
@@ -136,12 +138,14 @@ export async function attachActionableLinks(card, opts) {
         status: card.status,
         viewerRole: card.viewerRole,
         viewerApproved: opts.viewerApproved,
+        viewerActedAt: opts.viewerActedAt,
     });
     logger.info("Opportunity actionability decision", {
         opportunityId: card.opportunityId,
         status: card.status,
         viewerRole: card.viewerRole,
         viewerApproved: opts.viewerApproved,
+        viewerActedAt: opts.viewerActedAt,
         kind: kind ?? "none",
     });
     if (kind === null)
@@ -229,7 +233,14 @@ export function buildMinimalOpportunityCard(opp, viewerId, counterpartUserId, co
     const introducerActor = opp.actors.find((a) => a.role === "introducer" && a.userId !== viewerId);
     const viewerIsIntroducer = opp.actors.some((a) => a.role === "introducer" && a.userId === viewerId);
     const reasoning = opp.interpretation?.reasoning ?? "";
-    const mainText = viewerCentricCardSummary(reasoning, counterpartName, MINIMAL_MAIN_TEXT_MAX_CHARS, viewerName, introducerName ?? undefined);
+    // Shared sanitization standard — see opportunity.safe-presentation.ts.
+    const mainText = safeFallbackSummary(reasoning, {
+        counterpartName,
+        viewerName,
+        introducerName: introducerName ?? undefined,
+        maxChars: MINIMAL_MAIN_TEXT_MAX_CHARS,
+        emptyText: "A suggested connection.",
+    });
     const score = typeof opp.interpretation?.confidence === "number"
         ? opp.interpretation.confidence
         : undefined;
@@ -647,7 +658,7 @@ export function createOpportunityTools(defineTool, deps) {
                     userId: opp.userId,
                     name: opp.name,
                     avatar: opp.avatar,
-                    mainText: opp.homeCardPresentation?.personalizedSummary ?? opp.matchReason ?? "",
+                    mainText: getSafePresentationOrSkip(opp, { counterpartName: opp.name })?.summary ?? "",
                     cta: opp.homeCardPresentation?.suggestedAction,
                     headline: opp.homeCardPresentation?.headline,
                     primaryActionLabel: opp.homeCardPresentation?.primaryActionLabel,
@@ -658,6 +669,7 @@ export function createOpportunityTools(defineTool, deps) {
                     isGhost: opp.isGhost ?? false,
                     score: opp.score,
                     status: opp.status,
+                    viewerActedAt: opp.viewerActedAt,
                 }));
                 const displayedCards = allCardData.slice(0, CHAT_DISPLAY_LIMIT);
                 const extraFromCap = allCardData.length - displayedCards.length;
@@ -791,8 +803,13 @@ export function createOpportunityTools(defineTool, deps) {
                         firstEntity?.profile
                             ?.avatar ??
                         null,
-                    mainText: viewerCentricCardSummary(reasoning, counterpartName, MINIMAL_MAIN_TEXT_MAX_CHARS, undefined, // viewerName not available in this context; introducer name passed separately
-                    introducerUser?.name ?? undefined),
+                    mainText: safeFallbackSummary(reasoning, {
+                        counterpartName,
+                        // viewerName not available in this context; introducer name passed separately
+                        introducerName: introducerUser?.name ?? undefined,
+                        maxChars: MINIMAL_MAIN_TEXT_MAX_CHARS,
+                        emptyText: "A suggested connection.",
+                    }),
                     cta: "Start a conversation to connect.",
                     headline,
                     primaryActionLabel,
@@ -1111,9 +1128,7 @@ export function createOpportunityTools(defineTool, deps) {
                 userId: opp.userId,
                 name: opp.name,
                 avatar: opp.avatar,
-                mainText: opp.homeCardPresentation?.personalizedSummary ??
-                    opp.matchReason ??
-                    "",
+                mainText: getSafePresentationOrSkip(opp, { counterpartName: opp.name })?.summary ?? "",
                 cta: opp.homeCardPresentation?.suggestedAction,
                 headline: opp.homeCardPresentation?.headline,
                 primaryActionLabel: opp.homeCardPresentation?.primaryActionLabel,
@@ -1135,6 +1150,7 @@ export function createOpportunityTools(defineTool, deps) {
                     await attachActionableLinks(card, {
                         viewerId: context.userId,
                         viewerApproved: source?.viewerApproved,
+                        viewerActedAt: source?.viewerActedAt,
                         counterpartUserId: source?.userId ?? card.userId,
                         mintConnectLink,
                         frontendUrl: deps.frontendUrl,
@@ -1672,6 +1688,7 @@ export function createOpportunityTools(defineTool, deps) {
                                     await attachActionableLinks(card, {
                                         viewerId: context.userId,
                                         viewerApproved,
+                                        viewerActedAt: viewerActor?.actedAt ?? null,
                                         counterpartUserId,
                                         mintConnectLink: deps.mintConnectLink,
                                         frontendUrl: deps.frontendUrl,
@@ -1823,6 +1840,7 @@ export function createOpportunityTools(defineTool, deps) {
                                 await attachActionableLinks(cardData, {
                                     viewerId: context.userId,
                                     viewerApproved,
+                                    viewerActedAt: viewerActor?.actedAt ?? null,
                                     counterpartUserId,
                                     mintConnectLink: deps.mintConnectLink,
                                     frontendUrl: deps.frontendUrl,