bunnyquery 1.3.2 → 1.3.5

package/bunnyquery.js

@@ -196,6 +196,36 @@ ${options.inlineContentPlaceholder}
     }
     return false;
   }
+  function isNonRetryableRequestError(input) {
+    if (!input || typeof input !== "object") return false;
+    var status = typeof input.status_code === "number" ? input.status_code : typeof input.status === "number" ? input.status : void 0;
+    var param = void 0;
+    var blobs = [];
+    var sources = [input.error, input.body && input.body.error, input.body, input];
+    for (var i = 0; i < sources.length; i++) {
+      var e = sources[i];
+      if (!e) continue;
+      if (typeof e === "string") {
+        blobs.push(e);
+        continue;
+      }
+      if (typeof e !== "object") continue;
+      if (param === void 0 && e.param != null) param = e.param;
+      if (typeof e.code === "string") blobs.push(e.code);
+      if (typeof e.type === "string") blobs.push(e.type);
+      if (typeof e.message === "string") blobs.push(e.message);
+    }
+    var hay = blobs.join(" | ").toLowerCase();
+    if (hay.indexOf("unknown_parameter") !== -1 || hay.indexOf("unknown parameter") !== -1 || hay.indexOf("unsupported_parameter") !== -1 || hay.indexOf("unsupported parameter") !== -1) {
+      return true;
+    }
+    var isClientReqStatus = status === 400 || status === 422;
+    if (isClientReqStatus && param != null && param !== "") return true;
+    if (isClientReqStatus && hay.indexOf("invalid_request") !== -1 && (hay.indexOf("parameter") !== -1 || hay.indexOf("param") !== -1)) {
+      return true;
+    }
+    return false;
+  }
   function isAuthExpiredError(input) {
     if (!input) return false;
     var blobs = [];
@@ -950,10 +980,10 @@ ${options.inlineContentPlaceholder}
         });
       };
       var run = sendAndPoll().catch(function(err) {
-        if (isAuthExpiredError(err)) return self.host.refreshSession().then(sendAndPoll);
+        if (isAuthExpiredError(err) && !isNonRetryableRequestError(err)) return self.host.refreshSession().then(sendAndPoll);
         throw err;
       }).then(function(response) {
-        if (isErrorResponseBody(response) && isAuthExpiredError(response)) {
+        if (isErrorResponseBody(response) && isAuthExpiredError(response) && !isNonRetryableRequestError(response)) {
           return self.host.refreshSession().then(sendAndPoll);
         }
         return response;
@@ -1191,6 +1221,7 @@ ${options.inlineContentPlaceholder}
           this.enqueueTypewrite(aiIdx, answer, lid);
         }
       }
+      this._removeStrayPendingAssistants();
       this.promoteNextQueuedToRunning();
       this.updateHistoryCache();
       this.host.notify();
@@ -1233,6 +1264,7 @@ ${options.inlineContentPlaceholder}
           if (thPos2 !== -1) this.state.messages.splice(thPos2, 1);
         }
         if (serverId) this.cancelledServerIds.delete(serverId);
+        this._removeStrayPendingAssistants();
         this.promoteNextQueuedToRunning();
         this.updateHistoryCache();
         this.host.notify();
@@ -1246,6 +1278,7 @@ ${options.inlineContentPlaceholder}
         return;
       }
       this.insertAtTarget({ role: "assistant", content: getErrorMessage(err), isError: true }, targetIdx);
+      this._removeStrayPendingAssistants();
       this.promoteNextQueuedToRunning();
       this.updateHistoryCache();
       this.host.notify();
@@ -1407,16 +1440,48 @@ ${options.inlineContentPlaceholder}
       if (pendingIdx === -1) return Promise.resolve();
       if (latest.isError || !latest.content) {
         this.state.messages[pendingIdx] = { role: "assistant", content: latest.content || "", isError: !!latest.isError };
+        this._removeStrayPendingAssistants();
         this.host.notify();
         this.promoteNextQueuedToRunning();
         return Promise.resolve();
       }
       var lid = this._newLocalId();
       this.state.messages[pendingIdx] = { role: "assistant", content: "", isPending: false, _localId: lid };
+      this._removeStrayPendingAssistants();
       this.host.notify();
       this.promoteNextQueuedToRunning();
       return this.enqueueTypewrite(pendingIdx, latest.content, lid);
     }
+    // Remove any leftover non-background pending ("Thinking…") assistant bubbles.
+    // There is normally at most ONE such bubble at a time (promoteNext* refuses to
+    // add a second), so any extra is a duplicate — it appears when a concurrent
+    // history refetch re-maps the still-"running" turn into a pending placeholder
+    // (with a real _serverItemId) while the local pending bubble (no _serverItemId)
+    // is rescued and re-appended (see loadHistory rescue below). Each resolve path
+    // only replaces the FIRST pending bubble, so without this a stray "Thinking…"
+    // survives next to the reply/error. MUST run AFTER the resolved bubble has been
+    // made non-pending and BEFORE promoteNext*() (so a freshly-promoted Thinking,
+    // which is added only once no pending assistant remains, is preserved).
+    _removeStrayPendingAssistants() {
+      for (var k = this.state.messages.length - 1; k >= 0; k--) {
+        var m = this.state.messages[k];
+        if (m.isPending && m.role === "assistant" && !m.isBackgroundTask) this.state.messages.splice(k, 1);
+      }
+    }
+    // Drop the pending flags on the resolved turn's USER bubble (preserving its
+    // content + background-task marker). Needed because a bg "Indexing:" turn's user
+    // bubble carries isPendingInProcess; leaving it set keeps the bubble visually
+    // stuck and keeps its bgTaskQueue entry alive forever.
+    _clearPendingUserBubble(itemId) {
+      var uIdx = this.state.messages.findIndex(function(m) {
+        return m.role === "user" && m._serverItemId === itemId && (m.isPendingInProcess || m.isPendingQueued || m.isSendingToServer);
+      });
+      if (uIdx === -1) return;
+      var u = this.state.messages[uIdx];
+      var cleaned = { role: "user", content: u.content, _serverItemId: itemId };
+      if (u.isBackgroundTask) cleaned.isBackgroundTask = true;
+      this.state.messages[uIdx] = cleaned;
+    }
     // If an immediate-send request for the current cache key is still in flight
     // (e.g. the view unmounted then remounted mid-request), show the sending
     // state, await it, then render the reply from the cache. Skipped when the
@@ -1455,6 +1520,7 @@ ${options.inlineContentPlaceholder}
         return m.isPending && m._serverItemId === itemId;
       });
       if (idx !== -1) {
+        this._clearPendingUserBubble(itemId);
         if (isErr) {
           this.state.messages[idx] = { role: "assistant", content: answer, isError: true, _serverItemId: itemId };
           this.host.notify();
@@ -1571,7 +1637,7 @@ ${options.inlineContentPlaceholder}
         return getChatHistory({ service: serviceId, owner, platform }, options);
       };
       return Promise.resolve().then(fetchHistory).catch(function(err) {
-        if (isAuthExpiredError(err)) return self.host.refreshSession().then(fetchHistory);
+        if (isAuthExpiredError(err) && !isNonRetryableRequestError(err)) return self.host.refreshSession().then(fetchHistory);
         throw err;
       }).then(function(history) {
         if (token !== self.state.gateRefreshToken) return;
@@ -1604,12 +1670,16 @@ ${options.inlineContentPlaceholder}
           self.state.messages.forEach(function(m) {
             if (m.isCancelled && m._serverItemId) locallyCancelled[m._serverItemId] = 1;
           });
+          var mappedHasPendingAssistant = mapped.some(function(m) {
+            return m.isPending && m.role === "assistant" && !m.isBackgroundTask;
+          });
           var rescued = [];
           for (var ri = 0; ri < self.state.messages.length; ri++) {
             var mm = self.state.messages[ri];
             if (mm.isBackgroundTask) continue;
             if (mm._serverItemId && serverIds[mm._serverItemId]) continue;
             if (!mm._serverItemId) {
+              if (mappedHasPendingAssistant) continue;
               if (mm.isSendingToServer || mm.isPendingQueued || mm.isPendingInProcess || mm.isPending) rescued.push(mm);
               else if (self.state.sending && mm.role === "user") {
                 var next = self.state.messages[ri + 1];

package/dist/engine.cjs

@@ -195,6 +195,36 @@ function isErrorResponseBody(response) {
   }
   return false;
 }
+function isNonRetryableRequestError(input) {
+  if (!input || typeof input !== "object") return false;
+  var status = typeof input.status_code === "number" ? input.status_code : typeof input.status === "number" ? input.status : void 0;
+  var param = void 0;
+  var blobs = [];
+  var sources = [input.error, input.body && input.body.error, input.body, input];
+  for (var i = 0; i < sources.length; i++) {
+    var e = sources[i];
+    if (!e) continue;
+    if (typeof e === "string") {
+      blobs.push(e);
+      continue;
+    }
+    if (typeof e !== "object") continue;
+    if (param === void 0 && e.param != null) param = e.param;
+    if (typeof e.code === "string") blobs.push(e.code);
+    if (typeof e.type === "string") blobs.push(e.type);
+    if (typeof e.message === "string") blobs.push(e.message);
+  }
+  var hay = blobs.join(" | ").toLowerCase();
+  if (hay.indexOf("unknown_parameter") !== -1 || hay.indexOf("unknown parameter") !== -1 || hay.indexOf("unsupported_parameter") !== -1 || hay.indexOf("unsupported parameter") !== -1) {
+    return true;
+  }
+  var isClientReqStatus = status === 400 || status === 422;
+  if (isClientReqStatus && param != null && param !== "") return true;
+  if (isClientReqStatus && hay.indexOf("invalid_request") !== -1 && (hay.indexOf("parameter") !== -1 || hay.indexOf("param") !== -1)) {
+    return true;
+  }
+  return false;
+}
 function isAuthExpiredError(input) {
   if (!input) return false;
   var blobs = [];
@@ -977,10 +1007,10 @@ var ChatSession = class {
       });
     };
     var run = sendAndPoll().catch(function(err) {
-      if (isAuthExpiredError(err)) return self.host.refreshSession().then(sendAndPoll);
+      if (isAuthExpiredError(err) && !isNonRetryableRequestError(err)) return self.host.refreshSession().then(sendAndPoll);
       throw err;
     }).then(function(response) {
-      if (isErrorResponseBody(response) && isAuthExpiredError(response)) {
+      if (isErrorResponseBody(response) && isAuthExpiredError(response) && !isNonRetryableRequestError(response)) {
         return self.host.refreshSession().then(sendAndPoll);
       }
       return response;
@@ -1218,6 +1248,7 @@ var ChatSession = class {
         this.enqueueTypewrite(aiIdx, answer, lid);
       }
     }
+    this._removeStrayPendingAssistants();
     this.promoteNextQueuedToRunning();
     this.updateHistoryCache();
     this.host.notify();
@@ -1260,6 +1291,7 @@ var ChatSession = class {
         if (thPos2 !== -1) this.state.messages.splice(thPos2, 1);
       }
       if (serverId) this.cancelledServerIds.delete(serverId);
+      this._removeStrayPendingAssistants();
       this.promoteNextQueuedToRunning();
       this.updateHistoryCache();
       this.host.notify();
@@ -1273,6 +1305,7 @@ var ChatSession = class {
       return;
     }
     this.insertAtTarget({ role: "assistant", content: getErrorMessage(err), isError: true }, targetIdx);
+    this._removeStrayPendingAssistants();
     this.promoteNextQueuedToRunning();
     this.updateHistoryCache();
     this.host.notify();
@@ -1434,16 +1467,48 @@ var ChatSession = class {
     if (pendingIdx === -1) return Promise.resolve();
     if (latest.isError || !latest.content) {
       this.state.messages[pendingIdx] = { role: "assistant", content: latest.content || "", isError: !!latest.isError };
+      this._removeStrayPendingAssistants();
       this.host.notify();
       this.promoteNextQueuedToRunning();
       return Promise.resolve();
     }
     var lid = this._newLocalId();
     this.state.messages[pendingIdx] = { role: "assistant", content: "", isPending: false, _localId: lid };
+    this._removeStrayPendingAssistants();
     this.host.notify();
     this.promoteNextQueuedToRunning();
     return this.enqueueTypewrite(pendingIdx, latest.content, lid);
   }
+  // Remove any leftover non-background pending ("Thinking…") assistant bubbles.
+  // There is normally at most ONE such bubble at a time (promoteNext* refuses to
+  // add a second), so any extra is a duplicate — it appears when a concurrent
+  // history refetch re-maps the still-"running" turn into a pending placeholder
+  // (with a real _serverItemId) while the local pending bubble (no _serverItemId)
+  // is rescued and re-appended (see loadHistory rescue below). Each resolve path
+  // only replaces the FIRST pending bubble, so without this a stray "Thinking…"
+  // survives next to the reply/error. MUST run AFTER the resolved bubble has been
+  // made non-pending and BEFORE promoteNext*() (so a freshly-promoted Thinking,
+  // which is added only once no pending assistant remains, is preserved).
+  _removeStrayPendingAssistants() {
+    for (var k = this.state.messages.length - 1; k >= 0; k--) {
+      var m = this.state.messages[k];
+      if (m.isPending && m.role === "assistant" && !m.isBackgroundTask) this.state.messages.splice(k, 1);
+    }
+  }
+  // Drop the pending flags on the resolved turn's USER bubble (preserving its
+  // content + background-task marker). Needed because a bg "Indexing:" turn's user
+  // bubble carries isPendingInProcess; leaving it set keeps the bubble visually
+  // stuck and keeps its bgTaskQueue entry alive forever.
+  _clearPendingUserBubble(itemId) {
+    var uIdx = this.state.messages.findIndex(function(m) {
+      return m.role === "user" && m._serverItemId === itemId && (m.isPendingInProcess || m.isPendingQueued || m.isSendingToServer);
+    });
+    if (uIdx === -1) return;
+    var u = this.state.messages[uIdx];
+    var cleaned = { role: "user", content: u.content, _serverItemId: itemId };
+    if (u.isBackgroundTask) cleaned.isBackgroundTask = true;
+    this.state.messages[uIdx] = cleaned;
+  }
   // If an immediate-send request for the current cache key is still in flight
   // (e.g. the view unmounted then remounted mid-request), show the sending
   // state, await it, then render the reply from the cache. Skipped when the
@@ -1482,6 +1547,7 @@ var ChatSession = class {
       return m.isPending && m._serverItemId === itemId;
     });
     if (idx !== -1) {
+      this._clearPendingUserBubble(itemId);
       if (isErr) {
         this.state.messages[idx] = { role: "assistant", content: answer, isError: true, _serverItemId: itemId };
         this.host.notify();
@@ -1598,7 +1664,7 @@ var ChatSession = class {
       return getChatHistory({ service: serviceId, owner, platform }, options);
     };
     return Promise.resolve().then(fetchHistory).catch(function(err) {
-      if (isAuthExpiredError(err)) return self.host.refreshSession().then(fetchHistory);
+      if (isAuthExpiredError(err) && !isNonRetryableRequestError(err)) return self.host.refreshSession().then(fetchHistory);
       throw err;
     }).then(function(history) {
       if (token !== self.state.gateRefreshToken) return;
@@ -1631,12 +1697,16 @@ var ChatSession = class {
         self.state.messages.forEach(function(m) {
           if (m.isCancelled && m._serverItemId) locallyCancelled[m._serverItemId] = 1;
         });
+        var mappedHasPendingAssistant = mapped.some(function(m) {
+          return m.isPending && m.role === "assistant" && !m.isBackgroundTask;
+        });
         var rescued = [];
         for (var ri = 0; ri < self.state.messages.length; ri++) {
           var mm = self.state.messages[ri];
           if (mm.isBackgroundTask) continue;
           if (mm._serverItemId && serverIds[mm._serverItemId]) continue;
           if (!mm._serverItemId) {
+            if (mappedHasPendingAssistant) continue;
             if (mm.isSendingToServer || mm.isPendingQueued || mm.isPendingInProcess || mm.isPending) rescued.push(mm);
             else if (self.state.sending && mm.role === "user") {
               var next = self.state.messages[ri + 1];
@@ -1955,6 +2025,7 @@ exports.getExpiredAttachmentVisiblePath = getExpiredAttachmentVisiblePath;
 exports.groupAttachmentFailures = groupAttachmentFailures;
 exports.isAuthExpiredError = isAuthExpiredError;
 exports.isErrorResponseBody = isErrorResponseBody;
+exports.isNonRetryableRequestError = isNonRetryableRequestError;
 exports.isOfficeFile = isOfficeFile;
 exports.listClaudeModels = listClaudeModels;
 exports.listOpenAIModels = listOpenAIModels;