@djangocfg/ui-tools 2.1.444 → 2.1.445

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@djangocfg/ui-tools",
-  "version": "2.1.444",
+  "version": "2.1.445",
   "description": "Heavy React tools with lazy loading - for Electron, Vite, CRA, Next.js apps",
   "keywords": [
     "ui-tools",
@@ -334,8 +334,8 @@
     "test:watch": "vitest"
   },
   "peerDependencies": {
-    "@djangocfg/i18n": "^2.1.444",
-    "@djangocfg/ui-core": "^2.1.444",
+    "@djangocfg/i18n": "^2.1.445",
+    "@djangocfg/ui-core": "^2.1.445",
     "consola": "^3.4.2",
     "lodash-es": "^4.18.1",
     "lucide-react": "^0.545.0",
@@ -418,9 +418,9 @@
     "@maplibre/maplibre-gl-geocoder": "^1.7.0"
   },
   "devDependencies": {
-    "@djangocfg/i18n": "^2.1.444",
-    "@djangocfg/typescript-config": "^2.1.444",
-    "@djangocfg/ui-core": "^2.1.444",
+    "@djangocfg/i18n": "^2.1.445",
+    "@djangocfg/typescript-config": "^2.1.445",
+    "@djangocfg/ui-core": "^2.1.445",
     "@types/lodash-es": "^4.17.12",
     "@types/mapbox__mapbox-gl-draw": "^1.4.8",
     "@types/node": "^25.2.3",

package/src/tools/chat/hooks/useChat.ts

@@ -272,6 +272,16 @@ export function useChat(config: UseChatConfig): UseChatReturn {
       let serverMessageId: string | null = null;
       let chunkCount = 0;
       let charsReceived = 0;
+      // Tracks whether the stream already delivered a TERMINAL event
+      // (`message_end` or `error`) via handleEvent. Guards the manual-finalize
+      // below. Without it, the finalize reads `stateRef.current.isStreaming` —
+      // a snapshot NOT updated synchronously after a `dispatch` — so right
+      // after a STREAM_ERROR it still sees `isStreaming: true` and fires a
+      // spurious STREAM_DONE, whose reducer clears `state.error`. Net effect:
+      // the error is set then wiped in the same tick and the ErrorBanner never
+      // paints. `sawTerminal` is a closure flag, so it reflects the truth the
+      // ref can't. See @dev/web2/chat/reports/COMPARE-new-vs-old-chat-error-handling.md (layer 3).
+      let sawTerminal = false;
       const t0 = performance.now();
 
       try {
@@ -311,8 +321,12 @@ export function useChat(config: UseChatConfig): UseChatReturn {
         }
         tokenBuffer.flush();
 
-        // If transport never emitted message_end, finalize manually.
-        if (stateRef.current.isStreaming) {
+        // If transport never emitted a terminal event (message_end / error),
+        // finalize manually. Gated on `sawTerminal` — NOT on
+        // `stateRef.current.isStreaming`, which is a stale snapshot right after
+        // a STREAM_ERROR dispatch and would otherwise fire a STREAM_DONE that
+        // wipes `state.error`.
+        if (!sawTerminal && stateRef.current.isStreaming) {
           dispatch({ type: 'STREAM_DONE', id: targetId });
         }
 
@@ -397,6 +411,7 @@ export function useChat(config: UseChatConfig): UseChatReturn {
             log.tools.info('call_end', { toolId: ev.toolId, status: ev.status });
             return;
           case 'message_end':
+            sawTerminal = true;
             tokenBuffer.flush();
             dispatch({
               type: 'STREAM_DONE',
@@ -440,6 +455,7 @@ export function useChat(config: UseChatConfig): UseChatReturn {
             });
             return;
           case 'error':
+            sawTerminal = true;
             tokenBuffer.flush();
             dispatch({
               type: 'STREAM_ERROR',