Merge pull request #719 from microsoft/psl-clearhistoryadd

fix: Fix of Bug #34657

Author: Avijit-Microsoft

content-gen/src/app/frontend/src/components/ChatHistory.tsx

@@ -24,6 +24,7 @@ import {
   Compose20Regular,
   Delete20Regular,
   Edit20Regular,
+  DismissCircle20Regular,
 } from '@fluentui/react-icons';
 
 interface ConversationSummary {
@@ -55,8 +56,30 @@ export function ChatHistory({
   const [isLoading, setIsLoading] = useState(true);
   const [error, setError] = useState<string | null>(null);
   const [showAll, setShowAll] = useState(false);
+  const [isClearAllDialogOpen, setIsClearAllDialogOpen] = useState(false);
+  const [isClearing, setIsClearing] = useState(false);
   const INITIAL_COUNT = 5;
 
+  const handleClearAllConversations = useCallback(async () => {
+    setIsClearing(true);
+    try {
+      const response = await fetch('/api/conversations', {
+        method: 'DELETE',
+      });
+      if (response.ok) {
+        setConversations([]);
+        onNewConversation();
+        setIsClearAllDialogOpen(false);
+      } else {
+        console.error('Failed to clear all conversations');
+      }
+    } catch (err) {
+      console.error('Error clearing all conversations:', err);
+    } finally {
+      setIsClearing(false);
+    }
+  }, [onNewConversation]);
+
   const handleDeleteConversation = useCallback(async (conversationId: string) => {
     try {
       const response = await fetch(`/api/conversations/${conversationId}`, {
@@ -170,17 +193,51 @@ export function ChatHistory({
       backgroundColor: tokens.colorNeutralBackground3,
       overflow: 'hidden',
     }}>
-      <Text 
-        weight="semibold" 
-        size={300}
-        style={{ 
-          marginBottom: '12px',
-          color: tokens.colorNeutralForeground1,
-          flexShrink: 0,
-        }}
-      >
-        Chat History
-      </Text>
+      <div style={{
+        display: 'flex',
+        justifyContent: 'space-between',
+        alignItems: 'center',
+        marginBottom: '12px',
+        flexShrink: 0,
+      }}>
+        <Text 
+          weight="semibold" 
+          size={300}
+          style={{ 
+            color: tokens.colorNeutralForeground1,
+          }}
+        >
+          Chat History
+        </Text>
+        <Menu>
+          <MenuTrigger disableButtonEnhancement>
+            <Button
+              appearance="subtle"
+              icon={<MoreHorizontal20Regular />}
+              size="small"
+              title="More options"
+              disabled={isGenerating}
+              style={{ 
+                minWidth: '24px', 
+                height: '24px',
+                padding: '2px',
+                color: tokens.colorNeutralForeground3,
+              }}
+            />
+          </MenuTrigger>
+          <MenuPopover>
+            <MenuList>
+              <MenuItem 
+                icon={<DismissCircle20Regular />}
+                onClick={() => setIsClearAllDialogOpen(true)}
+                disabled={displayConversations.length === 0}
+              >
+                Clear all chat history
+              </MenuItem>
+            </MenuList>
+          </MenuPopover>
+        </Menu>
+      </div>
 
       <div style={{ 
         borderBottom: `1px solid ${tokens.colorNeutralStroke2}`,
@@ -295,6 +352,28 @@ export function ChatHistory({
           </Link>
         </div>
       </div>
+
+      {/* Clear All Confirmation Dialog */}
+      <Dialog open={isClearAllDialogOpen} onOpenChange={(_, data) => !isClearing && setIsClearAllDialogOpen(data.open)}>
+        <DialogSurface>
+          <DialogTitle>Clear all chat history</DialogTitle>
+          <DialogBody>
+            <DialogContent>
+              <Text>
+                Are you sure you want to delete all chat history? This action cannot be undone and all conversations will be permanently removed.
+              </Text>
+            </DialogContent>
+          </DialogBody>
+          <DialogActions>
+            <Button appearance="secondary" onClick={() => setIsClearAllDialogOpen(false)} disabled={isClearing}>
+              Cancel
+            </Button>
+            <Button appearance="primary" onClick={handleClearAllConversations} disabled={isClearing}>
+              {isClearing ? 'Clearing...' : 'Clear All'}
+            </Button>
+          </DialogActions>
+        </DialogSurface>
+      </Dialog>
     </div>
   );
 }

content-gen/src/backend/app.py

@@ -1319,6 +1319,29 @@ async def update_conversation(conversation_id: str):
         return jsonify({"error": "Failed to rename conversation"}), 500
 
 
+@app.route("/api/conversations", methods=["DELETE"])
+async def delete_all_conversations():
+    """
+    Delete all conversations for the current user.
+    
+    Uses authenticated user from EasyAuth headers.
+    """
+    auth_user = get_authenticated_user()
+    user_id = auth_user["user_principal_id"]
+    
+    try:
+        cosmos_service = await get_cosmos_service()
+        deleted_count = await cosmos_service.delete_all_conversations(user_id)
+        return jsonify({
+            "success": True,
+            "message": f"Deleted {deleted_count} conversations",
+            "deleted_count": deleted_count
+        })
+    except Exception as e:
+        logger.warning(f"Failed to delete all conversations: {e}")
+        return jsonify({"error": "Failed to delete conversations"}), 500
+
+
 # ==================== Brand Guidelines Endpoints ====================
 
 @app.route("/api/brand-guidelines", methods=["GET"])

content-gen/src/backend/services/cosmos_service.py

@@ -591,6 +591,35 @@ async def rename_conversation(
         result = await self._conversations_container.upsert_item(conversation)
         return result
 
+    async def delete_all_conversations(
+        self,
+        user_id: str
+    ) -> int:
+        """
+        Delete all conversations for a user.
+        
+        Args:
+            user_id: User ID to delete conversations for
+        
+        Returns:
+            Number of conversations deleted
+        """
+        await self.initialize()
+        
+        # First get all conversations for the user
+        conversations = await self.get_user_conversations(user_id, limit=1000)
+        
+        deleted_count = 0
+        for conv in conversations:
+            try:
+                await self.delete_conversation(conv["id"], user_id)
+                deleted_count += 1
+            except Exception as e:
+                logger.warning(f"Failed to delete conversation {conv['id']}: {e}")
+        
+        logger.info(f"Deleted {deleted_count} conversations for user {user_id}")
+        return deleted_count
+
 
 # Singleton instance
 _cosmos_service: Optional[CosmosDBService] = None