@kodaris/krubble-app-components 1.0.125 → 1.0.126

package/dist/krubble-app.umd.js

@@ -3347,11 +3347,6 @@
       // --- Lifecycle ---
       connectedCallback() {
           super.connectedCallback();
-          // Capture the consumer's targeting BEFORE we default `load` below: a custom
-          // `load` or an explicitly-set `conversation-id` means the consumer owns which
-          // conversation shows, so the tab-scoped resume must not override it.
-          const usesBuiltInLoad = !this.load;
-          const consumerSetConversationId = this.conversationId != null;
           // Default to loading a conversation's messages when the consumer hasn't
           // provided their own `load`. With `conversationId` set we rebuild that
           // conversation; with none set the user wants a NEW conversation, so we start
@@ -3439,49 +3434,49 @@
           // most-recent one. The pinned id lives in sessionStorage (per-tab — unlike
           // localStorage it isn't shared across tabs, so two tabs can sit on different
           // conversations). We resolve the id FIRST, then run the built-in load so it
-          // rebuilds the right transcript. Skipped when the consumer owns targeting (a
-          // custom `load` or an explicit `conversation-id`) or in inline mode (embedded
-          // as a page — the host route, not the tab, decides the conversation).
-          const resolveConversationId = () => {
-              if (this.variant !== 'floating' || !usesBuiltInLoad || consumerSetConversationId) {
-                  return Promise.resolve();
-              }
+          // rebuilds the right transcript. Skipped when the consumer owns targeting (an
+          // explicit `conversation-id`) or in inline mode (embedded as a page — the host
+          // route, not the tab, decides the conversation).
+          let resolveConversationId = Promise.resolve();
+          if (this.variant === 'floating' && this.conversationId == null) {
               const stored = Number(sessionStorage.getItem('kr-chatbot-conversation-id'));
               if (stored > 0) {
                   this.conversationId = stored;
                   // Fetch the conversation so the header shows its title on reload (the
                   // transcript load doesn't carry it). Best-effort — the header falls back
                   // to the default label if this fails.
-                  return krubbleHttp.Http.fetch({ url: `/api/system/conversation/${stored}` })
+                  resolveConversationId = krubbleHttp.Http.fetch({ url: `/api/system/conversation/${stored}` })
                       .then((r) => r.json())
                       .then((res) => {
                       this._conversationTitle = res?.data?.title || '';
                   })
                       .catch(() => { });
               }
-              // Fresh tab, nothing pinned yet — resume the user's most recent conversation
-              // (404 when they have none, which leaves us blank for a new one).
-              return krubbleHttp.Http.fetch({ url: '/api/system/conversation/last' })
-                  .then((r) => {
-                  if (!r.ok) {
-                      return;
-                  }
-                  return r.json().then((res) => {
-                      const id = res?.data?.conversationID;
-                      if (typeof id === 'number') {
-                          this.conversationId = id;
-                          this._conversationTitle = res?.data?.title || '';
-                          this._persistConversationId();
+              else {
+                  // Fresh tab, nothing pinned yet — resume the user's most recent conversation
+                  // (404 when they have none, which leaves us blank for a new one).
+                  resolveConversationId = krubbleHttp.Http.fetch({ url: '/api/system/conversation/last' })
+                      .then((r) => {
+                      if (!r.ok) {
+                          return;
                       }
-                  });
-              })
-                  .catch(() => { });
-          };
+                      return r.json().then((res) => {
+                          const id = res?.data?.conversationID;
+                          if (typeof id === 'number') {
+                              this.conversationId = id;
+                              this._conversationTitle = res?.data?.title || '';
+                              this._persistConversationId();
+                          }
+                      });
+                  })
+                      .catch(() => { });
+              }
+          }
           // `this.load` is always set by here (defaulted above when the consumer didn't
           // supply one), so capture it for the async chain — TS can't narrow the nullable
           // property across the promise boundary.
           const load = this.load;
-          resolveConversationId().then(() => load()).then((messages) => {
+          resolveConversationId.then(() => load()).then((messages) => {
               if (messages && messages.length) {
                   this.messages = messages;
               }
@@ -3731,6 +3726,13 @@
        * embeds and custom-`load` consumers own their own targeting.
        */
       _persistConversationId() {
+          // Floating-only: inline embeds (and custom-`load` consumers) own their own
+          // targeting and never resume from this key, so they must not write it either —
+          // otherwise an inline chat sharing the page/origin would clobber the floating
+          // widget's pinned id.
+          if (this.variant !== 'floating') {
+              return;
+          }
           try {
               if (this.conversationId != null) {
                   sessionStorage.setItem('kr-chatbot-conversation-id', String(this.conversationId));