@xapp/chat-widget 1.98.4 → 1.99.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)));
@@ -35435,11 +35460,23 @@ 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.
+     */
     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) {
+            container.scrollTo({ top: container.scrollHeight, behavior: behavior });
         }
     }, []);
     // Always scroll to bottom when messages change (unless disabled)
@@ -35472,16 +35509,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,