feedback + changelog

Author: brendan-kellam

CHANGELOG.md

@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - Removed prefix from structured log output. [#443](https://github.com/sourcebot-dev/sourcebot/pull/443)
+- [ask sb] Fixed long generation times for first message in a chat thread. [#447](https://github.com/sourcebot-dev/sourcebot/pull/447)
 
 ### Changed
 - Bumped AI SDK and associated packages version. [#444](https://github.com/sourcebot-dev/sourcebot/pull/444)

packages/web/src/features/chat/components/chatThread/chatThread.tsx

@@ -141,6 +141,8 @@ export const ChatThread = ({
                         variant: "destructive",
                     });
                 }
+                // Refresh the page to update the chat name.
+                router.refresh();
             });
         }
     }, [
@@ -151,14 +153,25 @@ export const ChatThread = ({
         toast,
         chatId,
         domain,
+        router,
     ]);
 
 
     const messagePairs = useMessagePairs(messages);
 
     useNavigationGuard({
-        enabled: status === "streaming" || status === "submitted",
-        confirm: () => window.confirm("You have unsaved changes that will be lost.")
+        enabled: ({ type }) => {
+            // @note: a "refresh" in this context means we have triggered a client side
+            // refresh via `router.refresh()`, and not the user pressing "CMD+R"
+            // (that would be a "beforeunload" event). We can safely peform refreshes
+            // without loosing any unsaved changes.
+            if (type === "refresh") {
+                return false;
+            }
+
+            return status === "streaming" || status === "submitted";
+        },
+        confirm: () => window.confirm("You have unsaved changes that will be lost."),
     });
 
     // When the chat is finished, refresh the page to update the chat history.