Merge pull request microsoft#12 from Roopan-Microsoft/PSL-BUG-10171

fix: Chat history template name is accepting empty strings as well

app.py

@@ -682,7 +682,7 @@ async def rename_conversation():
 
     ## update the title
     title = request_json.get("title", None)
-    if not title:
+    if not title or title.strip() == "":
         return jsonify({"error": "title is required"}), 400
     conversation["title"] = title
     updated_conversation = await cosmos_conversation_client.upsert_conversation(

frontend/src/components/ChatHistory/ChatHistoryListItem.tsx

@@ -25,6 +25,7 @@ import { AppStateContext } from '../../state/AppProvider'
 import { GroupedChatHistory } from './ChatHistoryList'
 
 import styles from './ChatHistoryPanel.module.css'
+import _ from 'lodash'
 
 interface ChatHistoryListItemCellProps {
   item?: Conversation
@@ -125,6 +126,17 @@ export const ChatHistoryListItemCell: React.FC<ChatHistoryListItemCellProps> = (
     if (errorRename || renameLoading) {
       return
     }
+    if (_.trim(editTitle) === "") {
+      setErrorRename('Error: Title is required.')
+      setTimeout(() => {
+        setErrorRename(undefined)
+        setTextFieldFocused(true)
+        if (textFieldRef.current) {
+          textFieldRef.current.focus()
+        }
+      }, 5000)
+      return
+    }
     if (editTitle == item.title) {
       setErrorRename('Error: Enter a new title to proceed.')
       setTimeout(() => {