@memberjunction/server 6.2.0-edge.0 → 6.2.0-edge.2

package/src/realtimeWidget/widgetCore.ts

@@ -12,7 +12,7 @@
  * @module @memberjunction/server/widget
  */
 
-import { buildSessionClaims } from '../auth/magicLink/magicLinkCore.js';
+import { BuildSessionClaims } from '../auth/magicLink/magicLinkCore.js';
 import type { MagicLinkJWTClaims } from '../auth/magicLink/types.js';
 
 /**
@@ -46,7 +46,7 @@ export interface WidgetInstanceEligibilityInput {
  * comma-separated string (tolerated). Returns an empty array for null/blank/garbage
  * — which makes the allowlist FAIL-CLOSED (no origin is allowed) by construction.
  */
-export function parseAllowedOrigins(allowedOrigins: string | null | undefined): string[] {
+export function ParseAllowedOrigins(allowedOrigins: string | null | undefined): string[] {
   const raw = allowedOrigins?.trim();
   if (!raw) {
     return [];
@@ -64,6 +64,11 @@ export function parseAllowedOrigins(allowedOrigins: string | null | undefined):
   return list.filter((s): s is string => typeof s === 'string').map(normalizeOrigin).filter((s): s is string => !!s);
 }
 
+/** @deprecated Use {@link ParseAllowedOrigins}. */
+export function parseAllowedOrigins(allowedOrigins: string | null | undefined): string[] {
+  return ParseAllowedOrigins(allowedOrigins);
+}
+
 /** Lowercases + trims an origin and strips a trailing slash for stable comparison. */
 function normalizeOrigin(origin: string): string {
   return origin.trim().toLowerCase().replace(/\/+$/, '');
@@ -74,7 +79,7 @@ function normalizeOrigin(origin: string): string {
  * JSON array (preferred, e.g. `["Whiteboard"]`) or a comma-separated string (tolerated). Returns an
  * empty array for null/blank/garbage — the backwards-compatible default (no channels attached).
  */
-export function parseEnabledChannels(enabledChannels: string | null | undefined): string[] {
+export function ParseEnabledChannels(enabledChannels: string | null | undefined): string[] {
   const raw = enabledChannels?.trim();
   if (!raw) {
     return [];
@@ -89,12 +94,17 @@ export function parseEnabledChannels(enabledChannels: string | null | undefined)
   }
 }
 
+/** @deprecated Use {@link ParseEnabledChannels}. */
+export function parseEnabledChannels(enabledChannels: string | null | undefined): string[] {
+  return ParseEnabledChannels(enabledChannels);
+}
+
 /**
  * FAIL-CLOSED origin check. The request's `Origin` header must exactly match one of
  * the widget's allowed origins (after normalization). A missing request origin, or
  * an empty allowlist, is rejected — a public mint endpoint must never accept "*".
  */
-export function isOriginAllowed(requestOrigin: string | null | undefined, allowedOrigins: readonly string[]): boolean {
+export function IsOriginAllowed(requestOrigin: string | null | undefined, allowedOrigins: readonly string[]): boolean {
   if (!requestOrigin || allowedOrigins.length === 0) {
     return false;
   }
@@ -102,8 +112,13 @@ export function isOriginAllowed(requestOrigin: string | null | undefined, allowe
   return allowedOrigins.some((o) => o === candidate);
 }
 
+/** @deprecated Use {@link IsOriginAllowed}. */
+export function isOriginAllowed(requestOrigin: string | null | undefined, allowedOrigins: readonly string[]): boolean {
+  return IsOriginAllowed(requestOrigin, allowedOrigins);
+}
+
 /** True when the requested modality (or the widget's default render) is enabled by the instance. */
-export function isModalityEnabled(widgetModality: string, requested: 'Text' | 'Voice'): boolean {
+export function IsModalityEnabled(widgetModality: string, requested: 'Text' | 'Voice'): boolean {
   const m = widgetModality.trim().toLowerCase();
   if (m === 'both') {
     return true;
@@ -111,25 +126,38 @@ export function isModalityEnabled(widgetModality: string, requested: 'Text' | 'V
   return m === requested.toLowerCase();
 }
 
+/** @deprecated Use {@link IsModalityEnabled}. */
+export function isModalityEnabled(widgetModality: string, requested: 'Text' | 'Voice'): boolean {
+  return IsModalityEnabled(widgetModality, requested);
+}
+
 /**
  * Pure eligibility check for a mint: the widget must be Active and the request's
  * origin must be on the allowlist. Returns an error code on the first failure so the
  * router can map it to a precise HTTP status. Modality is checked separately by the
  * voice path (W4) since a text mint is always permitted for an enabled widget.
  */
-export function evaluateWidgetMint(
+export function EvaluateWidgetMint(
   widget: WidgetInstanceEligibilityInput,
   requestOrigin: string | null | undefined,
 ): { ok: boolean; errorCode?: WidgetMintErrorCode } {
   if (widget.Status.trim().toLowerCase() !== 'active') {
     return { ok: false, errorCode: 'disabled' };
   }
-  if (!isOriginAllowed(requestOrigin, parseAllowedOrigins(widget.AllowedOrigins))) {
+  if (!IsOriginAllowed(requestOrigin, ParseAllowedOrigins(widget.AllowedOrigins))) {
     return { ok: false, errorCode: 'origin_not_allowed' };
   }
   return { ok: true };
 }
 
+/** @deprecated Use {@link EvaluateWidgetMint}. */
+export function evaluateWidgetMint(
+  widget: WidgetInstanceEligibilityInput,
+  requestOrigin: string | null | undefined,
+): { ok: boolean; errorCode?: WidgetMintErrorCode } {
+  return EvaluateWidgetMint(widget, requestOrigin);
+}
+
 /**
  * Known automated-client User-Agent substrings (lowercased). A public mint endpoint is a cheap
  * target for crawlers and scripted abuse; this catches the obvious non-browser callers. It is a
@@ -148,7 +176,7 @@ const BOT_USER_AGENT_MARKERS = [
  * known automation marker. Deliberately conservative to avoid false positives on real browsers; pair
  * with the origin allowlist + rate limits, which are the hard controls.
  */
-export function looksLikeBot(userAgent: string | null | undefined): boolean {
+export function LooksLikeBot(userAgent: string | null | undefined): boolean {
   const ua = (userAgent ?? '').trim().toLowerCase();
   if (!ua) {
     return true;
@@ -156,6 +184,11 @@ export function looksLikeBot(userAgent: string | null | undefined): boolean {
   return BOT_USER_AGENT_MARKERS.some((marker) => ua.includes(marker));
 }
 
+/** @deprecated Use {@link LooksLikeBot}. */
+export function looksLikeBot(userAgent: string | null | undefined): boolean {
+  return LooksLikeBot(userAgent);
+}
+
 /**
  * Builds the guest-session JWT claims for a widget visitor. Reuses magic-link's
  * `buildSessionClaims` (anonymous mode → the shared Anonymous principal + claims-based
@@ -163,7 +196,7 @@ export function looksLikeBot(userAgent: string | null | undefined): boolean {
  * principal is locked to one widget instance. The role name carried here is the
  * widget's restricted guest role (D5 backstop).
  */
-export function buildWidgetGuestClaims(args: {
+export function BuildWidgetGuestClaims(args: {
   issuer: string;
   audience: string;
   widgetId: string;
@@ -191,7 +224,7 @@ export function buildWidgetGuestClaims(args: {
     linkedRecordId?: string;
   };
 }): MagicLinkJWTClaims {
-  const claims = buildSessionClaims({
+  const claims = BuildSessionClaims({
     issuer: args.issuer,
     audience: args.audience,
     // No invite row for a direct-mint guest; the opaque session id stands in as the
@@ -229,3 +262,35 @@ export function buildWidgetGuestClaims(args: {
   }
   return claims;
 }
+
+/** @deprecated Use {@link BuildWidgetGuestClaims}. */
+export function buildWidgetGuestClaims(args: {
+  issuer: string;
+  audience: string;
+  widgetId: string;
+  sessionId: string;
+  anonymousEmail: string;
+  applicationId: string;
+  guestRoleName: string;
+  nowSeconds: number;
+  ttlSeconds: number;
+  /**
+   * Host-asserted identity for a `host-identity` session. Its email/name are carried as
+   * INFORMATIONAL claims only — the principal-resolving `email`/`sub` remain the shared
+   * Anonymous principal, so a host can never escalate a guest into a real account.
+   */
+  hostIdentity?: { email: string; firstName?: string; lastName?: string };
+  /**
+   * Returning-visitor anchor + linked identity (RV1/RV2/RV4) carried as claims so the VOICE path
+   * (server-created conversation) stamps the same fields the text path stamps client-side. Set only
+   * when the widget remembers returning visitors. Omitted ⇒ no returning-visitor claims (default off).
+   */
+  returningVisitor?: {
+    visitorKey?: string;
+    lastConversationId?: string;
+    linkedEntityId?: string;
+    linkedRecordId?: string;
+  };
+}): MagicLinkJWTClaims {
+  return BuildWidgetGuestClaims(args);
+}

package/src/realtimeWidget/widgetGuestElevation.ts

@@ -22,19 +22,19 @@ const WIDGET_ENTITY = 'MJ: Conversation Widget Instances';
 
 /**
  * Resolved elevation context for a widget-guest agent run. Returned only when the request is a
- * widget guest; the dispatch resolver runs the agent under {@link WidgetGuestRunContext.elevatedUser}
- * with {@link WidgetGuestRunContext.pinnedAgentId} (authoritative), keeping ownership checks under
+ * widget guest; the dispatch resolver runs the agent under {@link WidgetGuestRunContext.ElevatedUser}
+ * with {@link WidgetGuestRunContext.PinnedAgentId} (authoritative), keeping ownership checks under
  * the guest principal.
  */
 export interface WidgetGuestRunContext {
   /** The trusted server principal under which the agent run (and its run-entity writes) executes. */
-  elevatedUser: UserInfo;
+  ElevatedUser: UserInfo;
   /** The widget instance the guest session is bound to (authoritative config). */
-  widget: MJConversationWidgetInstanceEntity;
+  Widget: MJConversationWidgetInstanceEntity;
   /** The authoritative pinned support agent id — overrides any client-supplied agent id (D5). */
-  pinnedAgentId: string;
+  PinnedAgentId: string;
   /** The guest's per-session scope id (Conversation.ExternalID), used to validate conversation ownership. */
-  sessionScopeId?: string;
+  SessionScopeId?: string;
 }
 
 /**
@@ -47,7 +47,7 @@ export interface WidgetGuestRunContext {
  * Never throws: a resolution failure (no system user, widget not found, read error) returns `null`,
  * so the caller falls back to the normal (guest-principal) path rather than failing the request.
  */
-export async function resolveWidgetGuestRunContext(
+export async function ResolveWidgetGuestRunContext(
   userPayload: UserPayload,
   provider: DatabaseProviderBase,
 ): Promise<WidgetGuestRunContext | null> {
@@ -70,13 +70,21 @@ export async function resolveWidgetGuestRunContext(
   }
 
   return {
-    elevatedUser,
-    widget,
-    pinnedAgentId: widget.PinnedAgentID,
-    sessionScopeId: guest.MagicLinkScope?.ResourceID,
+    ElevatedUser: elevatedUser,
+    Widget: widget,
+    PinnedAgentId: widget.PinnedAgentID,
+    SessionScopeId: guest.MagicLinkScope?.ResourceID,
   };
 }
 
+/** @deprecated Use {@link ResolveWidgetGuestRunContext}. */
+export async function resolveWidgetGuestRunContext(
+  userPayload: UserPayload,
+  provider: DatabaseProviderBase,
+): Promise<WidgetGuestRunContext | null> {
+  return ResolveWidgetGuestRunContext(userPayload, provider);
+}
+
 /**
  * Returns the identity a realtime session's AI-RUN-ENTITY work (delegated tool dispatch +
  * observability run creation/append/finalize) should execute as: the trusted SYSTEM principal when
@@ -123,7 +131,7 @@ export function ResolveScopedAnonymousRunUser(contextUser: UserInfo): UserInfo {
  * preserving the guest's `sessionId` — so progress/streaming PubSub still routes to the guest's
  * live websocket, but all AI run-entity writes happen under the trusted server principal.
  */
-export function elevateUserPayload(userPayload: UserPayload, elevatedUser: UserInfo): UserPayload {
+export function ElevateUserPayload(userPayload: UserPayload, elevatedUser: UserInfo): UserPayload {
   return {
     ...userPayload,
     email: elevatedUser.Email,
@@ -132,6 +140,11 @@ export function elevateUserPayload(userPayload: UserPayload, elevatedUser: UserI
   };
 }
 
+/** @deprecated Use {@link ElevateUserPayload}. */
+export function elevateUserPayload(userPayload: UserPayload, elevatedUser: UserInfo): UserPayload {
+  return ElevateUserPayload(userPayload, elevatedUser);
+}
+
 /** Loads a single widget instance by id under the elevated principal (read-only). */
 async function loadWidgetById(
   widgetId: string,

package/src/resolvers/AdhocQueryResolver.ts

@@ -7,7 +7,7 @@ import { GetReadOnlyDataSource, GetReadOnlyProvider } from '../util.js';
 import { ResolverBase } from '../generic/ResolverBase.js';
 import { IsScopeLimitedPrincipal } from '../auth/scopeLimitedPrincipal.js';
 import { RunQueryResultType } from './QueryResolver.js';
-import { exactTotalFromPage, resolveAdhocTotalRowCount } from './adhoc-query-helpers.js';
+import { ExactTotalFromPage, ResolveAdhocTotalRowCount } from './adhoc-query-helpers.js';
 import sql from 'mssql';
 
 /**
@@ -200,7 +200,7 @@ export class AdhocQueryResolver extends ResolverBase {
         const recordset = (dataResult.recordset ?? []) as Record<string, unknown>[];
 
         // Total already known from the page alone (unpaged, or a short page)? Skip the count.
-        const exact = exactTotalFromPage(startRow, recordset.length, maxRows);
+        const exact = ExactTotalFromPage(startRow, recordset.length, maxRows);
         if (exact != null || !countSQL) {
             return { recordset, totalRowCount: exact ?? recordset.length };
         }
@@ -209,7 +209,7 @@ export class AdhocQueryResolver extends ResolverBase {
         const lowerBound = startRow + recordset.length;
         try {
             const countResult = await this.runSqlWithDeadline<{ TotalRowCount: number }>(ds, countSQL, deadline);
-            return { recordset, totalRowCount: resolveAdhocTotalRowCount(countResult.recordset, lowerBound) };
+            return { recordset, totalRowCount: ResolveAdhocTotalRowCount(countResult.recordset, lowerBound) };
         } catch (countErr) {
             const msg = countErr instanceof Error ? countErr.message : String(countErr);
             LogError(`Ad-hoc query row-count failed; reporting a lower-bound total (${lowerBound}). ${msg}`);

package/src/resolvers/AgentRunStatusPublisher.ts

@@ -0,0 +1,175 @@
+/**
+ * Publishes one agent run's live status on the caller's session.
+ *
+ * `RunAIAgentResolver` uses this for every status it sends. A host whose
+ * `AgentTurnHandler` answers with an In-Progress reply uses the same publisher,
+ * on that caller's session, so the chat follows the turn: significant-step
+ * progress, streamed text, the partial result, and the completion. When the
+ * turn throws instead of returning a result, call `PublishFailure` with the
+ * reply row's id and the error.
+ */
+import type { PubSubEngine } from 'type-graphql';
+import type {
+    AgentExecutionProgressCallback,
+    AgentExecutionStreamingCallback,
+    ExecuteAgentResult,
+} from '@memberjunction/ai-core-plus';
+import type { UserPayload } from '../types.js';
+import { PublishStatusUpdate } from '../generic/PushStatusResolver.js';
+
+const SIGNIFICANT_STEPS = ['prompt_execution', 'action_execution', 'subagent_execution', 'decision_processing'];
+
+/** The run fields the published messages read. Entities and test fakes both satisfy this. */
+interface StatusRun {
+    ID?: string;
+    ConversationDetailID?: string;
+    ErrorMessage?: string;
+    Steps?: Array<{ StepName?: string }>;
+    GetAll?: () => unknown;
+}
+
+export class AgentRunStatusPublisher {
+    private run: StatusRun | null = null;
+
+    /**
+     * @param sessionId Session stamped on the message body. Defaults to the payload's session.
+     *   The push itself is always addressed to `userPayload.sessionId`, matching the resolver.
+     */
+    constructor(
+        private readonly pubSub: PubSubEngine,
+        private readonly userPayload: UserPayload,
+        private readonly sessionId: string = userPayload.sessionId,
+    ) {}
+
+    /** Latest run observed from a progress event, including steps that are not published. */
+    public get AgentRun(): StatusRun | null {
+        return this.run;
+    }
+
+    /** AgentRunner's progress callback. Noise steps are kept for {@link AgentRun} and not published. */
+    public get OnProgress(): AgentExecutionProgressCallback {
+        return (progress) => {
+            const fromEvent = progress.metadata?.agentRun as StatusRun | undefined;
+            if (fromEvent) {
+                this.run = fromEvent;
+            }
+            if (!SIGNIFICANT_STEPS.includes(progress.step)) {
+                return;
+            }
+            const agentRun = fromEvent || this.run;
+            if (!agentRun?.ID) {
+                // A host that streams before the first progress event otherwise sees nothing and no reason.
+                console.error('❌ No agent run available for progress callback');
+                return;
+            }
+            this.publish('ExecutionProgress', {
+                sessionId: this.sessionId,
+                agentRunId: agentRun.ID,
+                type: 'progress',
+                agentRun: agentRun.GetAll ? agentRun.GetAll() : agentRun,
+                progress: {
+                    currentStep: progress.step,
+                    percentage: progress.percentage,
+                    message: progress.message,
+                    agentName: progress.metadata?.agentName,
+                    agentType: progress.metadata?.agentType,
+                    stepCount: progress.metadata?.stepCount,
+                    hierarchicalStep: progress.metadata?.hierarchicalStep,
+                },
+                timestamp: new Date(),
+            });
+        };
+    }
+
+    /** AgentRunner's streaming callback. Publishes nothing until a progress event has carried the run. */
+    public get OnStreaming(): AgentExecutionStreamingCallback {
+        return (chunk) => {
+            const agentRun = this.run;
+            if (!agentRun?.ID) {
+                console.error('❌ No agent run available for streaming callback');
+                return;
+            }
+            this.publish('StreamingContent', {
+                sessionId: this.sessionId,
+                agentRunId: agentRun.ID,
+                type: 'streaming',
+                agentRun: agentRun.GetAll ? agentRun.GetAll() : agentRun,
+                streaming: {
+                    content: chunk.content,
+                    isPartial: !chunk.isComplete,
+                    stepName: chunk.stepType,
+                    agentName: chunk.modelName,
+                    kind: chunk.kind,
+                },
+                timestamp: new Date(),
+            });
+        };
+    }
+
+    /**
+     * The partial result, then the completion. `conversationDetailId` is the reply row the
+     * chat should update. When omitted, the completion uses the run's own detail id.
+     */
+    public PublishFinal(result: ExecuteAgentResult, conversationDetailId?: string, resultJson?: string): void {
+        const agentRun = result.agentRun as StatusRun | undefined;
+        if (agentRun?.ID) {
+            let lastStep = 'Completed';
+            const steps = agentRun.Steps;
+            if (steps && steps.length > 0) {
+                lastStep = steps[steps.length - 1]?.StepName || 'Completed';
+            }
+            this.publish('StreamingContent', {
+                sessionId: this.sessionId,
+                agentRunId: agentRun.ID,
+                type: 'partial_result',
+                partialResult: {
+                    currentStep: lastStep,
+                    partialOutput: result.payload || undefined,
+                },
+                timestamp: new Date(),
+            });
+        }
+        this.publish('StreamingContent', {
+            sessionId: this.sessionId,
+            agentRunId: agentRun?.ID || 'unknown',
+            type: 'complete',
+            timestamp: new Date(),
+            conversationDetailId: conversationDetailId || agentRun?.ConversationDetailID,
+            success: result.success,
+            errorMessage: agentRun?.ErrorMessage || undefined,
+            result: resultJson || undefined,
+        });
+    }
+
+    /**
+     * Completion for a failure that has no run. The fire-and-forget path uses this when
+     * `executeAIAgent` rejects. A host uses it the same way: when the turn throws
+     * instead of returning a result, call `PublishFailure` with the reply row's id
+     * and the error. That completion is what tells the chat the reply finished.
+     */
+    public PublishFailure(conversationDetailId: string | undefined, errorMessage: string): void {
+        this.publish('StreamingContent', {
+            sessionId: this.sessionId,
+            agentRunId: 'unknown',
+            type: 'complete',
+            timestamp: new Date(),
+            conversationDetailId,
+            success: false,
+            errorMessage,
+            result: JSON.stringify({ success: false, errorMessage }),
+        });
+    }
+
+    private publish(type: 'ExecutionProgress' | 'StreamingContent', data: unknown): void {
+        PublishStatusUpdate(this.pubSub, {
+            sessionId: this.userPayload.sessionId,
+            ownerUserId: this.userPayload?.userRecord?.ID ?? '',
+            message: JSON.stringify({
+                resolver: 'RunAIAgentResolver',
+                type,
+                status: 'ok',
+                data,
+            }),
+        });
+    }
+}

package/src/resolvers/AgentSessionResolver.ts

@@ -25,15 +25,15 @@ import { SessionManager, SessionAuthorizationError } from '../agentSessions/inde
 export class StartAgentSessionResult {
     /** ID of the newly created session. */
     @Field()
-    agentSessionId: string;
+    agentSessionId: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     /** Lifecycle status of the session (always `Active` on a successful start). */
     @Field()
-    status: string;
+    status: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     /** ID of the conversation the session is attached to (supplied or freshly created). */
     @Field()
-    conversationId: string;
+    conversationId: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 /**

package/src/resolvers/ColorResolver.ts

@@ -10,22 +10,22 @@ export class Color {
 
   @Field(() => String)
   @Public()
-  name: string;
+  name: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String)
   @Public()
-  createdZ: string;
+  createdZ: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class ColorNotification {
   @Public()
   @Field(() => String, { nullable: true })
-  message?: string;
+  message?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Public()
   @Field((_type) => Date)
-  date!: Date;
+  date!: Date;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 export interface ColorNotificationPayload {
@@ -36,13 +36,13 @@ export interface ColorNotificationPayload {
 export class ColorResolver {
   @Subscription(() => ColorNotification, { topics: 'COLOR' })
   @Public()
-  colorSubscription(@Root() { message }: ColorNotificationPayload): ColorNotification {
+  colorSubscription(@Root() { message }: ColorNotificationPayload): ColorNotification {  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
     return { message, date: new Date() };
   }
 
   @Query(() => [Color])
   @Public()
-  async colors(@Ctx() _ctx: AppContext, @PubSub() pubSub: PubSubEngine) {
+  async colors(@Ctx() _ctx: AppContext, @PubSub() pubSub: PubSubEngine) {  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
     const createdZ = new Date().toISOString();
 
     pubSub.publish('COLOR', {