@xapp/chat-widget 1.98.4 → 1.100.0

package/dist/index.js

@@ -4491,23 +4491,48 @@ var MCPChat = /** @class */ (function () {
         log("SERVER: account_status: ".concat(JSON.stringify(status)));
         this.dispatch(setAccountStatus(status));
     };
-    MCPChat.prototype.getEndpoint = function () {
+    /**
+     * The paths this mode hangs off its base URL: [stream, close].
+     */
+    MCPChat.prototype.getEndpointPaths = function () {
+        return (this.config.mode || "homeowner") === "business_owner"
+            ? [BUSINESS_OWNER_ENDPOINT_PATH, BUSINESS_OWNER_CLOSE_ENDPOINT_PATH]
+            : [HOMEOWNER_ENDPOINT_PATH, HOMEOWNER_CLOSE_ENDPOINT_PATH];
+    };
+    /**
+     * The base the endpoint paths are built from.
+     *
+     * A configured `serverUrl` arrives in either of two shapes: the base
+     * (`https://agentic.xapp.ai/<appId>`), which is what this class was written for, or the
+     * full stream endpoint (`https://agentic.xapp.ai/<appId>/chat/stream`), which is the form
+     * issue #1403 documents as the Dev and Prod defaults. Appending unconditionally turns the
+     * second into `.../chat/stream/chat/stream`.
+     *
+     * Taking `serverUrl` verbatim instead is not the fix: `/chat/close` has to be derived
+     * from something, and a widget configured with the full stream URL would then send every
+     * close request to the stream endpoint. So normalise back to the base and build both
+     * paths from it, which leaves either configured form working.
+     *
+     * Only the paths belonging to the CONFIGURED mode come off. A homeowner URL handed to a
+     * business_owner widget is a misconfiguration, not something to quietly reinterpret, and
+     * an appId that merely ends in `/stream` is not an endpoint path.
+     */
+    MCPChat.prototype.getBaseUrl = function () {
         // Normalize serverUrl to remove trailing slashes
         var serverUrl = this.config.serverUrl.replace(/\/+$/, "");
-        var mode = this.config.mode || "homeowner";
-        if (mode === "business_owner") {
-            return "".concat(serverUrl).concat(BUSINESS_OWNER_ENDPOINT_PATH);
+        for (var _i = 0, _a = this.getEndpointPaths(); _i < _a.length; _i++) {
+            var path = _a[_i];
+            if (serverUrl.endsWith(path)) {
+                return serverUrl.slice(0, -path.length);
+            }
         }
-        // homeowner mode - serverUrl should already include appId
-        return "".concat(serverUrl).concat(HOMEOWNER_ENDPOINT_PATH);
+        return serverUrl;
+    };
+    MCPChat.prototype.getEndpoint = function () {
+        return "".concat(this.getBaseUrl()).concat(this.getEndpointPaths()[0]);
     };
     MCPChat.prototype.getCloseEndpoint = function () {
-        var serverUrl = this.config.serverUrl.replace(/\/+$/, "");
-        var mode = this.config.mode || "homeowner";
-        if (mode === "business_owner") {
-            return "".concat(serverUrl).concat(BUSINESS_OWNER_CLOSE_ENDPOINT_PATH);
-        }
-        return "".concat(serverUrl).concat(HOMEOWNER_CLOSE_ENDPOINT_PATH);
+        return "".concat(this.getBaseUrl()).concat(this.getEndpointPaths()[1]);
     };
     MCPChat.prototype.sendNewMessage = function (msg, user) {
         log("SERVER: new message: ".concat(JSON.stringify(msg)));
@@ -12953,7 +12978,7 @@ var ChatMessage = function (props) {
                 return (require$$0.jsxs(require$$0.Fragment, { children: [showToolProgressWithAvatar && (require$$0.jsx(ChatMessagePart, { showAvatar: true, user: user, avatarPosition: (_g = (_f = (_e = chatConfig === null || chatConfig === void 0 ? void 0 : chatConfig.env) === null || _e === void 0 ? void 0 : _e.theme) === null || _f === void 0 ? void 0 : _f.messages) === null || _g === void 0 ? void 0 : _g.avatarPosition, hideUserInfo: hideUserInfo, isStreaming: true, children: require$$0.jsx("div", { className: "chat-msg", children: require$$0.jsx(ToolCallDisplay, { toolCalls: toolCalls, debugMode: debugMode }) }) })), msg.text && (require$$0.jsx(ChatMessagePart, { showAvatar: avaKey === "text", user: user, avatarPosition: (_k = (_j = (_h = chatConfig === null || chatConfig === void 0 ? void 0 : chatConfig.env) === null || _h === void 0 ? void 0 : _h.theme) === null || _j === void 0 ? void 0 : _j.messages) === null || _k === void 0 ? void 0 : _k.avatarPosition, hideUserInfo: hideUserInfo, isStreaming: isStreaming_1 || wasStreamed, children: require$$0.jsx(ChatTextMessage, { message: props.message, sibling: props.sibling, user: user, toolCalls: hasToolCalls ? toolCalls : undefined, debugMode: debugMode }) })), msg.html &&
                             require$$0.jsx(ChatMessagePart, { showAvatar: avaKey === "html", user: user, avatarPosition: (_o = (_m = (_l = chatConfig === null || chatConfig === void 0 ? void 0 : chatConfig.env) === null || _l === void 0 ? void 0 : _l.theme) === null || _m === void 0 ? void 0 : _m.messages) === null || _o === void 0 ? void 0 : _o.avatarPosition, hideUserInfo: hideUserInfo, children: require$$0.jsx(ChatMarkdownMessage, { message: props.message, sibling: props.sibling, onOpenUrl: (_p = props.middlewareContext) === null || _p === void 0 ? void 0 : _p.openUrl }) }), msg.displays && middleware && msg.displays.map(function (display, index) {
                             if (display.type === "ScheduleButton") {
-                                return (require$$0.jsx(ChatScheduleWidget, { minimizeOnClick: props.minimizeOnClick, display: display }));
+                                return (require$$0.jsx(ChatScheduleWidget, { minimizeOnClick: props.minimizeOnClick, display: display }, index));
                             }
                             var Middleware = middleware;
                             return (require$$0.jsx(Middleware, { msg: display, ctx: props.middlewareContext }, index));
@@ -13451,11 +13476,14 @@ var Suggestions = function (props) {
     var actions = React.useMemo(function () { return createActions(onItemUse); }, [onItemUse]);
     var transformContent = function (content) {
         var lines = content.split('\n');
-        return lines.map(function (line) {
+        // Keyed by position: the lines have no id of their own, and a blank line between
+        // paragraphs is not distinguishable by content either. `<></>` cannot carry a key,
+        // hence the explicit Fragment.
+        return lines.map(function (line, index) {
             if (line) {
-                return require$$0.jsx(MarkdownBuilder, { text: line });
+                return require$$0.jsx(MarkdownBuilder, { text: line }, "line-".concat(index));
             }
-            return require$$0.jsx(require$$0.Fragment, {});
+            return require$$0.jsx(React.Fragment, {}, "line-".concat(index));
         });
     };
     var containerClass = "xappw-suggestions ".concat(props.className || "", " ").concat(props.hasWsButton ? "xappw-suggestions--with-ws-button" : "").trim();
@@ -35435,12 +35463,37 @@ var MessageList = function (_a) {
         var _a = containerRef.current, scrollTop = _a.scrollTop, scrollHeight = _a.scrollHeight, clientHeight = _a.clientHeight;
         return Math.abs(scrollHeight - clientHeight - scrollTop) < 10;
     }, []);
-    // Scroll to bottom
+    /**
+     * Pin the list to its newest message.
+     *
+     * Scrolls THIS container, deliberately - not `messagesEndRef.scrollIntoView()`.
+     * scrollIntoView walks every scrollable ancestor, and its default `block: "start"` puts
+     * the target at the top of each one. On first open the greeting and its chips arrive,
+     * this effect fires, and the whole chat panel was smoothly dragged upward until the end
+     * marker reached the top - leaving only the suggestion chips and the input on screen
+     * until the user scrolled back by hand. Device, 2026-09-10.
+     *
+     * A container scroll cannot touch an ancestor, which is the entire point.
+     *
+     * `scrollTo` is feature-detected rather than assumed. It is absent in jsdom, so every
+     * test that mounts this component through a passive effect throws on it -- but the
+     * reason to guard is not the test runner: this widget renders on whatever browser a
+     * customer's visitor brought, and a missing `Element.prototype.scrollTo` would throw
+     * inside a passive effect and take the message list down with it. Assigning
+     * `scrollTop` is the universally supported equivalent; it loses the smooth animation
+     * and nothing else.
+     */
     var scrollToBottom = React.useCallback(function (behavior) {
         if (behavior === void 0) { behavior = "smooth"; }
-        if (messagesEndRef.current) {
-            messagesEndRef.current.scrollIntoView({ behavior: behavior });
+        var container = containerRef.current;
+        if (!container) {
+            return;
+        }
+        if (typeof container.scrollTo === "function") {
+            container.scrollTo({ top: container.scrollHeight, behavior: behavior });
+            return;
         }
+        container.scrollTop = container.scrollHeight;
     }, []);
     // Always scroll to bottom when messages change (unless disabled)
     React.useEffect(function () {
@@ -35472,16 +35525,16 @@ var MessageList = function (_a) {
     }, [hasWsButton, isAtBottom, scrollToBottom, disableAutoScroll]);
     var handleSend = React.useCallback(function (msg) {
         onSend(msg);
-        if (!disableAutoScroll && messagesEndRef.current) {
-            messagesEndRef.current.scrollIntoView({ behavior: "smooth" });
+        if (!disableAutoScroll) {
+            scrollToBottom("smooth");
         }
-    }, [onSend, disableAutoScroll]);
+    }, [onSend, disableAutoScroll, scrollToBottom]);
     var handleWrite = React.useCallback(function (msg) {
         onWrite(msg);
-        if (!disableAutoScroll && messagesEndRef.current) {
-            messagesEndRef.current.scrollIntoView({ behavior: "smooth" });
+        if (!disableAutoScroll) {
+            scrollToBottom("smooth");
         }
-    }, [onWrite, disableAutoScroll]);
+    }, [onWrite, disableAutoScroll, scrollToBottom]);
     var ctxCache = React.useMemo(function () {
         return new MiddlewareContextFactory({
             send: handleSend,
@@ -35531,7 +35584,7 @@ var MessageList = function (_a) {
             case "chat.prechat":
                 return require$$0.jsx(PrechatFormContainer, { time: time }, "prechat");
             default:
-                return (require$$0.jsxs("span", { children: [require$$0.jsx("span", { className: "message-sr-only", children: "at " + time + " system message" }), require$$0.jsxs("div", { children: ["Unhandled message: ", JSON.stringify(msg)] }, "uhm-".concat(+new Date()))] }, "default-".concat(msg.timestamp)));
+                return (require$$0.jsxs("span", { children: [require$$0.jsx("span", { className: "message-sr-only", children: "at " + time + " system message" }), require$$0.jsxs("div", { "data-testid": "unhandled-message", children: ["Unhandled message: ", JSON.stringify(msg)] }, "uhm-".concat(msg.timestamp))] }, "default-".concat(msg.timestamp)));
         }
     }
     /**