@droppii-org/chat-sdk 0.6.6 → 0.6.7

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

@@ -1 +1 @@
-{"version":3,"file":"useConversationList.d.ts","sourceRoot":"","sources":["../../../src/hooks/conversation/useConversationList.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,YAAY,EAAE,KAAK,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAI/E,UAAU,yBAAyB;IACjC,OAAO,EAAE,OAAO,CAAC;IACjB,YAAY,CAAC,EAAE,YAAY,CAAC;CAC7B;AAED,UAAU,yBAAyB;IACjC,KAAK,EAAE,oBAAoB,EAAE,CAAC;IAC9B,SAAS,EAAE,OAAO,CAAC;IACnB,kBAAkB,EAAE,OAAO,CAAC;IAC5B,WAAW,EAAE,OAAO,CAAC;IACrB,aAAa,EAAE,MAAM,IAAI,CAAC;CAC3B;AASD,eAAO,MAAM,mBAAmB,GAC9B,QAAQ,yBAAyB,KAChC,yBA0EF,CAAC"}
+{"version":3,"file":"useConversationList.d.ts","sourceRoot":"","sources":["../../../src/hooks/conversation/useConversationList.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,YAAY,EAAE,KAAK,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAI/E,UAAU,yBAAyB;IACjC,OAAO,EAAE,OAAO,CAAC;IACjB,YAAY,CAAC,EAAE,YAAY,CAAC;CAC7B;AAED,UAAU,yBAAyB;IACjC,KAAK,EAAE,oBAAoB,EAAE,CAAC;IAC9B,SAAS,EAAE,OAAO,CAAC;IACnB,kBAAkB,EAAE,OAAO,CAAC;IAC5B,WAAW,EAAE,OAAO,CAAC;IACrB,aAAa,EAAE,MAAM,IAAI,CAAC;CAC3B;AASD,eAAO,MAAM,mBAAmB,GAC9B,QAAQ,yBAAyB,KAChC,yBA8FF,CAAC"}

package/dist/hooks/conversation/useConversationList.js

@@ -1,5 +1,5 @@
 "use client";
-import { useCallback, useEffect, useMemo, useState } from "react";
+import { useCallback, useEffect, useMemo, useRef, useState } from "react";
 import { CbEvents } from "@droppii-org/wasm-client-sdk";
 import { DChatSDK } from "../../constants/sdk";
 import { useChatContext } from "../../context/ChatContext";
@@ -21,16 +21,29 @@ export const useConversationList = (params) => {
         isApplicationTypeValid &&
         connectStatus === ConnectStatus.Connected;
     const appParams = useMemo(() => ({ applicationType, chatCategory }), [applicationType, chatCategory]);
+    // Guards against loadInitial overlapping itself — e.g. OnSyncServerFinish
+    // firing again (common right after connect, while WASM is still catching
+    // up) before the previous fetch has resolved, which would otherwise fire
+    // a second concurrent getConversationListByReq call.
+    const isLoadingInitialRef = useRef(false);
     const loadInitial = useCallback(async () => {
+        if (isLoadingInitialRef.current)
+            return;
+        isLoadingInitialRef.current = true;
         setIsLoading(true);
         try {
             const hasMore = await getConversationListByReq(false, appParams);
             setHasNextPage(hasMore);
         }
         finally {
+            isLoadingInitialRef.current = false;
             setIsLoading(false);
         }
     }, [getConversationListByReq, appParams]);
+    // Always points at the latest loadInitial (fresh appParams) without being
+    // an effect dependency itself — see the effect below.
+    const loadInitialRef = useRef(loadInitial);
+    loadInitialRef.current = loadInitial;
     const fetchNextPage = useCallback(() => {
         if (!hasNextPage || isFetchingNextPage)
             return;
@@ -39,18 +52,24 @@ export const useConversationList = (params) => {
             .then((hasMore) => setHasNextPage(hasMore))
             .finally(() => setIsFetchingNextPage(false));
     }, [hasNextPage, isFetchingNextPage, getConversationListByReq, appParams]);
+    // Deliberately keyed on queryEnabled alone (not loadInitial/appParams) —
+    // a parent re-render passing a new chatCategory reference (or any other
+    // appParams change) shouldn't tear down and resubscribe this listener,
+    // which would otherwise also refire the initial fetch for no real reason.
+    // onSyncFinish still calls the CURRENT loadInitial via the ref above, so
+    // a genuine appParams change is still picked up by the next sync event.
     useEffect(() => {
         if (!queryEnabled)
             return;
-        void loadInitial();
+        void loadInitialRef.current();
         const onSyncFinish = () => {
-            void loadInitial();
+            void loadInitialRef.current();
         };
         DChatSDK.on(CbEvents.OnSyncServerFinish, onSyncFinish);
         return () => {
             DChatSDK.off(CbEvents.OnSyncServerFinish, onSyncFinish);
         };
-    }, [queryEnabled, loadInitial]);
+    }, [queryEnabled]);
     const items = useMemo(() => conversationList.map(toConversationListItem), [conversationList]);
     return {
         items,

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/screens/conversation-detail/index.tsx"],"names":[],"mappings":"AAmBA,OAAO,KAAK,EAAE,4BAA4B,EAAE,MAAM,kCAAkC,CAAC;AAWrF,OAAO,EAEL,KAAK,4BAA4B,EAClC,MAAM,6CAA6C,CAAC;AAGrD,MAAM,WAAW,4BACf,SAAQ,4BAA4B,EACpC,4BAA4B;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;IAC3B,MAAM,CAAC,EAAE,MAAM,IAAI,CAAC;CACrB;AAED,QAAA,MAAM,uBAAuB,GAAI,wFAS9B,4BAA4B,mDA8L9B,CAAC;AAEF,eAAe,uBAAuB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/screens/conversation-detail/index.tsx"],"names":[],"mappings":"AAoBA,OAAO,KAAK,EAAE,4BAA4B,EAAE,MAAM,kCAAkC,CAAC;AAWrF,OAAO,EAEL,KAAK,4BAA4B,EAClC,MAAM,6CAA6C,CAAC;AAGrD,MAAM,WAAW,4BACf,SAAQ,4BAA4B,EACpC,4BAA4B;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;IAC3B,MAAM,CAAC,EAAE,MAAM,IAAI,CAAC;CACrB;AAED,QAAA,MAAM,uBAAuB,GAAI,wFAS9B,4BAA4B,mDAoN9B,CAAC;AAEF,eAAe,uBAAuB,CAAC"}

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

@@ -10,7 +10,7 @@ var __rest = (this && this.__rest) || function (s, e) {
         }
     return t;
 };
-import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { useChatContext } from "../../context/ChatContext";
@@ -18,6 +18,7 @@ import { useChatNavigate } from "../../hooks/conversation/useChatNavigate";
 import { useBoolean } from "ahooks";
 import clsx from "clsx";
 import { Empty, Spin } from "antd";
+import { Icon } from "../../components/icon";
 import useConversationStore from "../../store/conversation";
 import ThreadProfilePanel from "../../components/conversation-detail/ThreadProfilePanel";
 import MediaResourcePanel from "../../components/media-resources/MediaResourcePanel";
@@ -82,7 +83,7 @@ const DChatConversationDetail = (_a) => {
         return (_jsx("div", { className: "flex h-full w-full items-center justify-center bg-white", children: _jsx(Spin, {}) }));
     }
     if (error) {
-        return (_jsx("div", { className: "flex h-full w-full items-center justify-center bg-white", children: _jsx(Empty, { description: t("thread_detail.resolve_failed") }) }));
+        return (_jsxs("div", { "data-testid": "conversation-detail-root", className: clsx("relative flex h-full w-full flex-col overflow-hidden bg-white", className), children: [_jsx("div", { "data-testid": "conversation-detail-header", className: "flex w-full shrink-0 cursor-default items-center border-b border-gray-25 bg-white p-2 shadow-inset-gray-divider", children: _jsx("button", { type: "button", onClick: handleBack, "data-testid": "conversation-detail-back", "data-cp-name": "HeaderIconButton", className: "flex shrink-0 cursor-pointer items-center justify-center p-2 text-[var(--dc-color-primary)]", children: _jsx(Icon, { icon: "arrow-left-o", size: 24 }) }) }), _jsx("div", { className: "flex flex-1 items-center justify-center", children: _jsx(Empty, { description: t("thread_detail.resolve_failed") }) })] }));
     }
     if (!conversation) {
         return null;

package/dist/store/conversation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conversation.d.ts","sourceRoot":"","sources":["../../src/store/conversation.ts"],"names":[],"mappings":"AAUA,OAAO,EAAE,iBAAiB,EAAE,MAAM,QAAQ,CAAC;AAY3C,eAAO,MAAM,kBAAkB,GAAI,SAAS,OAAO,SAElD,CAAC;AAEF,QAAA,MAAM,oBAAoB,gFAiYvB,CAAC;AAEJ,eAAe,oBAAoB,CAAC"}
+{"version":3,"file":"conversation.d.ts","sourceRoot":"","sources":["../../src/store/conversation.ts"],"names":[],"mappings":"AAUA,OAAO,EAAE,iBAAiB,EAAE,MAAM,QAAQ,CAAC;AAY3C,eAAO,MAAM,kBAAkB,GAAI,SAAS,OAAO,SAElD,CAAC;AAEF,QAAA,MAAM,oBAAoB,gFAkYvB,CAAC;AAEJ,eAAe,oBAAoB,CAAC"}

package/dist/store/conversation.js

@@ -143,16 +143,17 @@ const useConversationStore = create((set, get) => ({
                 }
             }
             // Derives view-only/block state from fetch/cache above for fresh
-            // loads and external entries. Unconfirmed (support/bot) threads get
-            // view-only-rejoin when not-a-member but never the block+modal path
-            // — that's reserved for isConfirmedGroup.
-            if (groupID) {
+            // loads and external entries. Unconfirmed (support/bot) threads skip
+            // this membership check entirely — BE already gates access to them
+            // (a non-member's device never syncs the conversation at all), so
+            // there's no "removed from group" state to detect for this kind.
+            if (groupID && isConfirmedGroup) {
                 const notAMember = !get().currentMemberInGroup.has(groupID);
                 const isPublic = ((_a = get().currentGroupInfo.get(groupID)) === null || _a === void 0 ? void 0 : _a.visibility) === GroupVisibility.Public;
-                if (notAMember && (!isConfirmedGroup || isPublic)) {
+                if (notAMember && isPublic) {
                     set({ isViewOnlyMember: true });
                 }
-                else if (isConfirmedGroup && notAMember) {
+                else if (notAMember) {
                     // Blocked: don't fall through to setting currentConversation below
                     // — that would re-open the very conversation just cleared.
                     get().clearSelectedConversation();

package/package.json

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