@droppii-org/chat-sdk 0.5.1 → 0.5.3

package/dist/hooks/conversation/useActiveConversation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useActiveConversation.d.ts","sourceRoot":"","sources":["../../../src/hooks/conversation/useActiveConversation.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAEjE,eAAO,MAAM,qBAAqB,GAChC,mBAAmB,oBAAoB,EAAE,SAuC1C,CAAC"}
+{"version":3,"file":"useActiveConversation.d.ts","sourceRoot":"","sources":["../../../src/hooks/conversation/useActiveConversation.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAEjE,eAAO,MAAM,qBAAqB,GAChC,mBAAmB,oBAAoB,EAAE,SAoD1C,CAAC"}

package/dist/hooks/conversation/useActiveConversation.js

@@ -3,12 +3,23 @@ import { useEffect } from "react";
 import { useSearchParams } from "next/navigation";
 import useConversationStore from "../../store/conversation";
 export const useActiveConversation = (conversationItems) => {
+    // Only used as an effect trigger on navigation — Next's Pages Router
+    // compat shim for useSearchParams isn't a live subscription, so its
+    // snapshot can still be one render behind a just-completed router.push
+    // when this effect re-fires for an unrelated reason (e.g. conversationItems
+    // changing after an async group-info fetch). Re-reading window.location
+    // directly below (instead of trusting this snapshot's value) avoids ever
+    // acting on that staleness — see the "URL clobbered right after selecting
+    // a group" regression this guards.
     const searchParams = useSearchParams();
-    const threadId = searchParams.get("threadId");
+    const searchParamsSnapshot = searchParams.toString();
     const setSelectedConversationId = useConversationStore((state) => state.setSelectedConversationId);
     const setConversationData = useConversationStore((state) => state.setConversationData);
     const clearSelectedConversation = useConversationStore((state) => state.clearSelectedConversation);
     useEffect(() => {
+        const threadId = typeof window !== "undefined"
+            ? new URLSearchParams(window.location.search).get("threadId")
+            : null;
         if (!threadId) {
             clearSelectedConversation();
             return;
@@ -22,8 +33,8 @@ export const useActiveConversation = (conversationItems) => {
             setConversationData(matchedThread);
         }
     }, [
-        threadId,
         conversationItems,
+        searchParamsSnapshot,
         setSelectedConversationId,
         setConversationData,
         clearSelectedConversation,

package/dist/hooks/conversation/useResolveConversation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useResolveConversation.d.ts","sourceRoot":"","sources":["../../../src/hooks/conversation/useResolveConversation.ts"],"names":[],"mappings":"AAaA,MAAM,WAAW,4BAA4B;IAC3C,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAqED,eAAO,MAAM,sBAAsB,GAAI,6CAKpC,4BAA4B;;;CA2E9B,CAAC"}
+{"version":3,"file":"useResolveConversation.d.ts","sourceRoot":"","sources":["../../../src/hooks/conversation/useResolveConversation.ts"],"names":[],"mappings":"AAaA,MAAM,WAAW,4BAA4B;IAC3C,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AA6ED,eAAO,MAAM,sBAAsB,GAAI,6CAKpC,4BAA4B;;;CA2H9B,CAAC"}

package/dist/hooks/conversation/useResolveConversation.js

@@ -1,5 +1,5 @@
 "use client";
-import { useEffect, useState } from "react";
+import { useEffect, useRef, useState } from "react";
 import { SessionType } from "@droppii-org/wasm-client-sdk";
 import { DChatSDK } from "../../constants/sdk";
 import { apiInstance } from "../../services/api";
@@ -48,18 +48,54 @@ const matchesTarget = (conversation, { conversationId, userId, groupId, botId })
         (!!groupId && conversation.groupID === groupId) ||
         (!!botId && resolveConversationBotId(conversation) === botId));
 };
+const targetKey = ({ conversationId, userId, groupId, botId, }) => `${conversationId !== null && conversationId !== void 0 ? conversationId : ""}|${userId !== null && userId !== void 0 ? userId : ""}|${groupId !== null && groupId !== void 0 ? groupId : ""}|${botId !== null && botId !== void 0 ? botId : ""}`;
 export const useResolveConversation = ({ conversationId, userId, groupId, botId, }) => {
     const [isFetching, setIsFetching] = useState(false);
     const [error, setError] = useState(null);
+    // A target the store rejected (no access) clears conversationData back to
+    // null, which would otherwise re-satisfy shouldResolve and refetch forever.
+    const rejectedTargetRef = useRef(null);
+    // The conversationID + target key currently in flight through
+    // setConversationData — setConversationData sets conversationData to it
+    // optimistically before updateCurrentConversation's async group-membership
+    // check has a chance to reject it, so this can't be cleared just because
+    // conversationData looks truthy; only a rename to something else (a newer
+    // resolve superseding this one) or an explicit reject below clears it.
+    const pendingTargetRef = useRef(null);
     const { connectStatus, syncStatus } = useChatContext();
     const isSdkReady = connectStatus === ConnectStatus.Connected &&
         syncStatus !== SyncStatus.Loading;
     const conversationData = useConversationStore((state) => state.conversationData);
+    const removedFromGroupModalOpen = useConversationStore((state) => state.removedFromGroupModalOpen);
     const setConversationData = useConversationStore((state) => state.setConversationData);
     const setSelectedConversationId = useConversationStore((state) => state.setSelectedConversationId);
     const target = { conversationId, userId, groupId, botId };
     const hasTarget = !!(conversationId || userId || groupId || botId);
-    const shouldResolve = hasTarget && !matchesTarget(conversationData, target);
+    const currentTargetKey = targetKey(target);
+    const wasRejected = rejectedTargetRef.current === currentTargetKey;
+    const shouldResolve = hasTarget && !matchesTarget(conversationData, target) && !wasRejected;
+    // Catches updateCurrentConversation's async, silent rejection (blocked/no
+    // group access) of the target this hook just resolved and handed to
+    // setConversationData — that path clears conversationData back to null
+    // via clearSelectedConversation instead of throwing, which would otherwise
+    // re-satisfy shouldResolve above and refetch this target forever.
+    // showRemovedFromGroupModal is only ever called from that one rejection
+    // branch, immediately after clearSelectedConversation — react to it
+    // opening at all. pendingTargetRef is deliberately NOT cleared on a
+    // matching conversationData: setConversationData sets that optimistically,
+    // synchronously, before updateCurrentConversation's async membership check
+    // has a chance to reject it — clearing early on that transient match would
+    // just recreate the loop. It's overwritten by the next resolve() call
+    // instead, once a newer/different target supersedes this one.
+    useEffect(() => {
+        if (!removedFromGroupModalOpen)
+            return;
+        const pending = pendingTargetRef.current;
+        if (!pending)
+            return;
+        rejectedTargetRef.current = pending.key;
+        pendingTargetRef.current = null;
+    }, [removedFromGroupModalOpen]);
     useEffect(() => {
         if (!shouldResolve || !isSdkReady)
             return;
@@ -79,6 +115,14 @@ export const useResolveConversation = ({ conversationId, userId, groupId, botId,
                 if (!resolved)
                     throw new Error(CONVERSATION_NOT_FOUND_ERROR);
                 // Entry point for notifications/deep links — force a fresh check.
+                // The store's own group-membership rejection (blocked/no access)
+                // lands asynchronously after this call returns, via
+                // clearSelectedConversation + showRemovedFromGroupModal — a separate
+                // effect below watches removedFromGroupModalOpen to catch it.
+                pendingTargetRef.current = {
+                    key: currentTargetKey,
+                    conversationID: resolved.conversationID,
+                };
                 setConversationData(resolved, undefined, { forceRevalidate: true });
                 setSelectedConversationId(resolved.conversationID);
             }
@@ -106,6 +150,7 @@ export const useResolveConversation = ({ conversationId, userId, groupId, botId,
         isSdkReady,
         setConversationData,
         setSelectedConversationId,
+        currentTargetKey,
     ]);
     return {
         isResolving: shouldResolve && (isFetching || !isSdkReady),

package/dist/screens/conversation-inbox/index.d.ts

@@ -4,11 +4,7 @@ export interface DChatConversationInboxProps {
     onSelectThread?: (conversation: ConversationListItem) => void;
     className?: string;
     moreOptions?: MoreOptionConfig[];
-    /**
-     * Scopes the conversation list to a single BE chat category (e.g. Mall
-     * buyer inbox = Droppii Support + linked sellers only). Omit to keep the
-     * existing unscoped behavior (all categories for the application type).
-     */
+    /** Scopes the conversation list to a single BE chat category. Omit for unscoped behavior. */
     chatCategory?: ChatCategory;
     /** Passed through to ConversationInboxHeader — see its own prop docs. */
     showSearch?: boolean;

package/dist/screens/conversation-inbox/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/screens/conversation-inbox/index.tsx"],"names":[],"mappings":"AAeA,OAAO,KAAK,EAAE,YAAY,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAC/E,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAG/D,MAAM,WAAW,2BAA2B;IAC1C,cAAc,CAAC,EAAE,CAAC,YAAY,EAAE,oBAAoB,KAAK,IAAI,CAAC;IAC9D,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,gBAAgB,EAAE,CAAC;IACjC;;;;OAIG;IACH,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,yEAAyE;IACzE,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,eAAe,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;CACnC;AAED,QAAA,MAAM,sBAAsB,GAAI,qGAQ7B,2BAA2B,4CA8E7B,CAAC;AAEF,eAAe,sBAAsB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/screens/conversation-inbox/index.tsx"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EAAE,YAAY,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAC/E,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAG/D,MAAM,WAAW,2BAA2B;IAC1C,cAAc,CAAC,EAAE,CAAC,YAAY,EAAE,oBAAoB,KAAK,IAAI,CAAC;IAC9D,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,gBAAgB,EAAE,CAAC;IACjC,6FAA6F;IAC7F,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,yEAAyE;IACzE,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,eAAe,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;CACnC;AAED,QAAA,MAAM,sBAAsB,GAAI,qGAQ7B,2BAA2B,4CA2F7B,CAAC;AAEF,eAAe,sBAAsB,CAAC"}

package/dist/screens/conversation-inbox/index.js

@@ -6,6 +6,7 @@ import { Empty } from "antd";
 import { useBoolean } from "ahooks";
 import clsx from "clsx";
 import useAuthStore from "../../store/auth";
+import useConversationStore from "../../store/conversation";
 import { useConversationList } from "../../hooks/conversation/useConversationList";
 import { useActiveConversation } from "../../hooks/conversation/useActiveConversation";
 import { ConversationInboxHeader } from "../../components/conversation-inbox";
@@ -23,6 +24,7 @@ const DChatConversationInbox = ({ onSelectThread, className, moreOptions, chatCa
     const groupId = (_b = searchParams.get("groupId")) !== null && _b !== void 0 ? _b : undefined;
     const botId = (_c = searchParams.get("botId")) !== null && _c !== void 0 ? _c : undefined;
     const applicationType = useAuthStore((state) => state.applicationType);
+    const selectedConversationId = useConversationStore((state) => state.selectedConversationId);
     const [isSearchOpen, { setTrue: openSearch, setFalse: closeSearch }] = useBoolean(false);
     const isApplicationTypeValid = Object.values(DChatApplicationType).includes(applicationType);
     const conversationList = useConversationList({
@@ -33,7 +35,15 @@ const DChatConversationInbox = ({ onSelectThread, className, moreOptions, chatCa
     // Any of the four targets opens the detail screen — useResolveConversation
     // turns userId/groupId/botId into a thread (creating the bot one when
     // missing), so entry points that only know the peer still work.
-    const hasConversationTarget = !!(threadId || userId || groupId || botId);
+    // threadId alone must also match selectedConversationId: clearSelectedConversation
+    // strips threadId from the URL via history.replaceState, which Next's Pages
+    // Router compat shim for useSearchParams doesn't react to (only router.push/
+    // replace do) — without this check, a stale threadId here would keep
+    // rendering the detail screen against an already-cleared store forever.
+    const hasConversationTarget = !!((threadId && threadId === selectedConversationId) ||
+        userId ||
+        groupId ||
+        botId);
     if (hasConversationTarget) {
         return (_jsx(DChatConversationDetail, { className: className, moreOptions: moreOptions, conversationId: threadId !== null && threadId !== void 0 ? threadId : undefined, userId: userId, groupId: groupId, botId: botId }));
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@droppii-org/chat-sdk",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Droppii React Chat SDK",
   "repository": {
     "type": "git",