@ai-matrx/messaging 0.13.1 → 0.15.0

package/dist/react.cjs

@@ -869,7 +869,7 @@ function projectMessage(row, fallbackOrganizationId) {
     id: requiredStr(row, "id", operation),
     conversationId: requiredStr(row, "conversation_id", operation),
     senderId: requiredStr(row, "sender_id", operation),
-    organizationId: str(row, "organization_id") ?? fallbackOrganizationId,
+    organizationId: str(row, "organization_id") ?? fallbackOrganizationId ?? "",
     content: typeof row["content"] === "string" ? row["content"] : "",
     kind: MESSAGE_KINDS.has(kindRaw) ? kindRaw : "text",
     // A row read from the DB is at least `sent`. `sending`/`failed` are outbox
@@ -917,7 +917,7 @@ function projectConversationSummary(row, viewerId, fallbackOrganizationId) {
       groupName,
       groupImageUrl,
       createdBy: createdBy === null ? null : createdBy,
-      organizationId: str(row, "organization_id") ?? fallbackOrganizationId,
+      organizationId: str(row, "organization_id") ?? fallbackOrganizationId ?? "",
       createdAt,
       updatedAt,
       metadata: jsonObject(row["metadata"])
@@ -983,6 +983,7 @@ function createMessagingStore() {
   let threads = /* @__PURE__ */ new Map();
   let activeConversationId = null;
   let conversationReadFailures = /* @__PURE__ */ new Map();
+  let conversationsLoadError = null;
   const listeners = /* @__PURE__ */ new Set();
   let cached = null;
   function snapshot() {
@@ -996,7 +997,8 @@ function createMessagingStore() {
       totalUnreadConversations: conversations.filter((item) => item.unreadCount > 0).length,
       archiveFilter,
       archivedCount,
-      conversationReadFailures
+      conversationReadFailures,
+      conversationsLoadError
     };
     return cached;
   }
@@ -1041,6 +1043,12 @@ function createMessagingStore() {
       conversations = normalizeConversations(items);
       hasMoreConversations = hasMore;
       hasLoadedConversations = true;
+      conversationsLoadError = null;
+      emit();
+    },
+    setConversationsLoadError(message) {
+      if (message === conversationsLoadError) return;
+      conversationsLoadError = message;
       emit();
     },
     setArchiveFilter(next) {
@@ -1250,7 +1258,11 @@ function createMessagingEngine(options) {
           optimisticMessage({
             conversationId: entry.draft.conversationId,
             senderId: identity.userId,
-            organizationId: identity.organizationId,
+            // Client-only until the confirmed row replaces it: the active
+            // organization, else the conversation's own.
+            organizationId: identity.organizationId ?? store.snapshot().conversations.find(
+              (item) => item.conversation.id === entry.draft.conversationId
+            )?.conversation.organizationId ?? "",
             content: entry.draft.content,
             clientMessageId: entry.clientMessageId,
             ...entry.draft.kind !== void 0 ? { kind: entry.draft.kind } : {},
@@ -1269,10 +1281,20 @@ function createMessagingEngine(options) {
   }
   async function reloadInbox() {
     if (disposed) return;
-    const page = await repository.listConversations({
-      limit: conversationPageSize,
-      archived: store.snapshot().archiveFilter
-    });
+    let page;
+    try {
+      page = await repository.listConversations({
+        limit: conversationPageSize,
+        archived: store.snapshot().archiveFilter
+      });
+    } catch (error) {
+      if (!disposed) {
+        store.setConversationsLoadError(
+          error instanceof Error ? error.message : typeof error === "string" ? error : "The conversation list could not be read."
+        );
+      }
+      throw error;
+    }
     if (disposed) return;
     conversationCursor = page.nextCursor;
     store.setConversations(page.items, page.hasMore);
@@ -1357,6 +1379,13 @@ function createMessagingEngine(options) {
         }
       });
     },
+    async refreshInbox() {
+      try {
+        await reloadInbox();
+      } catch (error) {
+        reportError(error, "inboxRefresh");
+      }
+    },
     async setArchiveFilter(next) {
       if (next === store.snapshot().archiveFilter) return;
       store.setArchiveFilter(next);
@@ -1739,12 +1768,15 @@ var RPCS = {
 };
 var MESSAGE_COLUMNS = "id,conversation_id,sender_id,organization_id,content,message_type,status,reply_to_id,client_message_id,action_data,media_url,media_thumbnail_url,media_metadata,created_at,edited_at,deleted_at,deleted_for_everyone,metadata";
 var CONVERSATION_COLUMNS = "id,type,group_name,group_image_url,created_by,organization_id,created_at,updated_at,metadata";
+function optionalOrg(organizationId) {
+  return typeof organizationId === "string" && organizationId.length > 0 ? organizationId : null;
+}
 function requireOrg(organizationId, operation) {
-  if (typeof organizationId !== "string" || organizationId.length === 0) {
+  if (organizationId === null) {
     throw new MessagingError(
-      "misconfigured",
-      `${operation}: no organization_id`,
-      "Every conversation and message write carries an explicit organization_id (R5). Pass a real org on <MessagingProvider>; the package refuses the write rather than letting an unscoped row reach the database."
+      "organization-required",
+      `${operation}: no organization selected`,
+      "A new conversation is filed under one organization. Ask the person to choose one, then pass it as organizationId on <MessagingProvider>. Reading and replying to existing conversations never needs it."
     );
   }
   return organizationId;
@@ -1756,7 +1788,7 @@ function sleep(ms) {
   return new Promise((resolve) => setTimeout(resolve, jittered));
 }
 function createMessagingRepository(options) {
-  const org = requireOrg(options.organizationId, "createMessagingRepository");
+  const org = optionalOrg(options.organizationId);
   const handedIn = options.identity;
   if (handedIn !== void 0 && handedIn !== null && handedIn.userId !== void 0) {
     options.onDiagnostic?.(
@@ -1826,6 +1858,11 @@ function createMessagingRepository(options) {
     });
     return found;
   }
+  async function organizationOfExisting(conversationId) {
+    if (org !== null) return org;
+    const conversation = await repository.getConversation(conversationId);
+    return conversation.organizationId;
+  }
   const repository = {
     organizationId: org,
     actingUserId,
@@ -1958,6 +1995,7 @@ function createMessagingRepository(options) {
     },
     async getOrCreateDirectConversation(otherUserId) {
       const operation = "getOrCreateDirectConversation";
+      const destination = requireOrg(org, operation);
       const me = await actingUser(operation);
       const id = await withSessionRetry(
         operation,
@@ -1969,7 +2007,7 @@ function createMessagingRepository(options) {
             // not a preference.
             p_user1_id: me,
             p_user2_id: otherUserId,
-            p_organization_id: org
+            p_organization_id: destination
           },
           operation
         )
@@ -1981,13 +2019,14 @@ function createMessagingRepository(options) {
     },
     async createGroupConversation({ name, memberIds }) {
       const operation = "createGroupConversation";
+      const destination = requireOrg(org, operation);
       const me = await actingUser(operation);
       const { data, error } = await withSessionRetry(
         operation,
         async () => db().from(TABLES.conversations).insert({
           type: "group",
           group_name: name,
-          organization_id: org,
+          organization_id: destination,
           created_by: me
         }).select("id").single()
       );
@@ -2002,7 +2041,7 @@ function createMessagingRepository(options) {
           conversation_id: conversationId,
           user_id: userId,
           role: userId === me ? "owner" : "member",
-          organization_id: org,
+          organization_id: destination,
           created_by: me
         }))
       );
@@ -2012,10 +2051,11 @@ function createMessagingRepository(options) {
     async insertMessage(draft, clientMessageId) {
       const operation = "insertMessage";
       const me = await actingUser(operation);
+      const destination = await organizationOfExisting(draft.conversationId);
       const { data, error } = await db().from(TABLES.messages).insert({
         conversation_id: draft.conversationId,
         sender_id: me,
-        organization_id: org,
+        organization_id: destination,
         created_by: me,
         content: draft.content,
         message_type: draft.kind ?? "text",
@@ -2076,12 +2116,13 @@ function createMessagingRepository(options) {
     async addMembers(conversationId, memberIds) {
       const operation = "addMembers";
       const me = await actingUser(operation);
+      const destination = await organizationOfExisting(conversationId);
       const { error } = await db().from(TABLES.participants).insert(
         memberIds.map((userId) => ({
           conversation_id: conversationId,
           user_id: userId,
           role: "member",
-          organization_id: org,
+          organization_id: destination,
           created_by: me
         }))
       );
@@ -2192,7 +2233,7 @@ function MessagingProvider(props) {
   }, [strayUserId]);
   const session = useSessionUserId(client, props.onDiagnostic);
   const userId = session.snapshot.userId;
-  const ready = client != null && userId !== null && typeof organizationId === "string" && organizationId.length > 0;
+  const ready = client != null && userId !== null;
   const hostProvidesRealtime = (0, import_react2.useIsRealtimeProviderMounted)();
   const runtime = /* @__PURE__ */ (0, import_jsx_runtime.jsx)(
     MessagingRuntime,
@@ -2228,12 +2269,13 @@ function MessagingRuntime(props) {
       actionRegistry.register(handler);
     });
   }, [actionRegistry, handlers]);
-  const ready = client != null && manager !== null && sessionIdentity !== null && userId !== null && typeof organizationId === "string" && organizationId.length > 0;
+  const ready = client != null && manager !== null && sessionIdentity !== null && userId !== null;
+  const activeOrganizationId = typeof organizationId === "string" && organizationId.length > 0 ? organizationId : null;
   const engine = (0, import_react.useMemo)(() => {
     if (!ready || manager === null || sessionIdentity === null || userId === null) return null;
     const identity = {
       userId,
-      organizationId
+      organizationId: activeOrganizationId
     };
     const report = (event) => {
       (diagnosticRef.current ?? defaultDiagnostics)(event);
@@ -2244,7 +2286,7 @@ function MessagingRuntime(props) {
       // supabase-shape.ts); everything below this line works against the rich
       // contract, which `supabase-shape.test.ts` proves a real client honors.
       client,
-      organizationId,
+      organizationId: activeOrganizationId,
       // THE SAME READER the render loop above uses. Two readers would be two
       // answers during the tick an account switch takes.
       sessionIdentity,
@@ -2277,14 +2319,14 @@ function MessagingRuntime(props) {
       }
     });
     return built;
-  }, [ready, manager, sessionIdentity, userId, organizationId]);
+  }, [ready, manager, sessionIdentity, userId, activeOrganizationId]);
   (0, import_react.useEffect)(() => {
     if (engine === null) return void 0;
     void engine.start().catch((error) => {
       (diagnosticRef.current ?? defaultDiagnostics)({
         level: "error",
         message: `Messaging failed to start: ${String(error?.message ?? error)}`,
-        remedy: "Check the signed-in session and organization, then reload the surface."
+        remedy: "Check the signed-in session, then reload the surface."
       });
     });
     return () => {
@@ -2298,9 +2340,10 @@ function MessagingRuntime(props) {
   const sourceFeature = props.sourceFeature;
   const ai = (0, import_react.useMemo)(() => {
     if (transport === void 0 || agents === void 0 || !ready) return null;
+    if (activeOrganizationId === null) return null;
     return createMessagingAi({
       transport,
-      organizationId,
+      organizationId: activeOrganizationId,
       agents,
       ...maxTranscriptMessages !== void 0 ? { maxTranscriptMessages } : {},
       ...sourceApp !== void 0 ? { sourceApp } : {},
@@ -2310,7 +2353,7 @@ function MessagingRuntime(props) {
     transport,
     agents,
     ready,
-    organizationId,
+    activeOrganizationId,
     maxTranscriptMessages,
     sourceApp,
     sourceFeature
@@ -2361,7 +2404,7 @@ function useRequiredMessagingHost() {
   const host = useMessagingHost();
   if (host === null) {
     throw new Error(
-      "[@ai-matrx/messaging] No messaging host. Wrap this tree in <MessagingProvider client={supabase} userId={userId} organizationId={orgId}> and render it only once auth has resolved."
+      "[@ai-matrx/messaging] No messaging host. Wrap this tree in <MessagingProvider client={supabase} organizationId={orgId}> and render it only once auth has resolved."
     );
   }
   return host;
@@ -2486,7 +2529,11 @@ function useConversations() {
     hasMore: snapshot?.hasMoreConversations ?? false,
     // "Not loaded yet" is a FACT from the store, never a guess. Showing
     // "No conversations yet" before the first read is a screen telling a lie.
-    isInitialLoading: snapshot === null || !snapshot.hasLoadedConversations,
+    isInitialLoading: snapshot === null || !snapshot.hasLoadedConversations && snapshot.conversationsLoadError === null,
+    loadError: snapshot?.conversationsLoadError ?? null,
+    retry: () => {
+      void host?.engine.refreshInbox();
+    },
     totalUnreadConversations: snapshot?.totalUnreadConversations ?? 0,
     loadMore: () => {
       void host?.engine.loadMoreConversations();
@@ -2751,7 +2798,8 @@ function useMessageAction(message) {
       const context = {
         messageId: message.id,
         actorId: host.identity.userId,
-        organizationId: host.identity.organizationId,
+        // The active organization, else the organization the message lives in.
+        organizationId: host.identity.organizationId ?? message.organizationId,
         choice
       };
       host.actions.execute(action, context).then(setReceipt).catch((cause) => {
@@ -3067,6 +3115,7 @@ function EmptyState(props) {
 }
 
 // src/react/components.tsx
+var import_design_system = require("@ai-matrx/design-system");
 var import_jsx_runtime4 = require("react/jsx-runtime");
 var AI_LABELS = {
   catchUp: "Catch me up",
@@ -3084,7 +3133,9 @@ function ConversationList(props) {
     activeConversationId,
     archiveFilter,
     setArchiveFilter,
-    archivedCount
+    archivedCount,
+    loadError,
+    retry
   } = useConversations();
   const RowChrome = useMessagingHost()?.wrapConversationRow ?? null;
   const [query, setQuery] = (0, import_react5.useState)("");
@@ -3149,7 +3200,20 @@ function ConversationList(props) {
       }
     ) }) : null,
     /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "mx-msg__scroll", children: [
-      isInitialLoading ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(ConversationSkeleton, {}) : visible.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
+      isInitialLoading ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(ConversationSkeleton, {}) : loadError !== null && visible.length === 0 ? (
+        // A failed read resolves to the failure — never a skeleton forever,
+        // never "No conversations yet" (RC-B12 round 11). ErrorBox is the
+        // design-system error slot: the host fills it with its menu.
+        /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
+          ReadFailureBox,
+          {
+            title: "Couldn't load your conversations",
+            message: loadError,
+            operation: "List your conversations",
+            onRetry: retry
+          }
+        )
+      ) : visible.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
         EmptyState,
         {
           title: query.length > 0 ? "No matches" : showingArchive ? "No archived conversations" : "No conversations yet",
@@ -3326,10 +3390,11 @@ function ConversationView(props) {
         body: "It doesn't exist, or you aren't a member of it."
       }
     ) : conversation.error !== null && messages.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
-      EmptyState,
+      ReadFailureBox,
       {
         title: "Couldn't load this conversation",
-        body: "Check your connection, then open it again."
+        message: conversation.error.message || "Check your connection, then open it again.",
+        operation: "Open this conversation"
       }
     ) : messages.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(EmptyState, { title: "No messages yet", body: "Say something to get started." }) : /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "mx-msg__messages", children: [
       conversation.hasMoreOlder ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
@@ -3660,4 +3725,27 @@ function MessagingInbox(props) {
     }
   );
 }
+function ReadFailureBox({
+  title,
+  message,
+  operation,
+  onRetry
+}) {
+  return /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "mx-msg__empty", role: "alert", children: [
+    /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
+      import_design_system.ErrorBox,
+      {
+        as: "div",
+        title,
+        message,
+        operation,
+        origin: "messaging",
+        style: { color: "var(--mx-msg-danger)", fontWeight: 600 },
+        children: title
+      }
+    ),
+    /* @__PURE__ */ (0, import_jsx_runtime4.jsx)("p", { children: message }),
+    onRetry ? /* @__PURE__ */ (0, import_jsx_runtime4.jsx)("button", { type: "button", className: "mx-msg__button mx-msg__button--ghost", onClick: onRetry, children: "Try again" }) : null
+  ] });
+}
 //# sourceMappingURL=react.cjs.map