refactor(frontend): remove dead code, deduplicate error handling, tighten utility exports

Author: Shubhangi-Microsoft

content-gen/src/app/frontend/src/api/index.ts

@@ -122,8 +122,6 @@ export async function* streamGenerateContent(
   }, { signal });
   const taskId = startData.task_id;
 
-  console.debug(`Generation started with task ID: ${taskId}`);
-  
   // Yield initial status
   yield {
     type: 'status',

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

@@ -75,8 +75,8 @@ export const ChatHistory = memo(function ChatHistory({
     try {
       await dispatch(clearAllConversations()).unwrap();
       onNewConversation();
-    } catch (err) {
-      console.error('Error clearing all conversations:', err);
+    } catch {
+      // Error clearing all conversations
     }
   }, [dispatch, onNewConversation]);
 
@@ -86,16 +86,16 @@ export const ChatHistory = memo(function ChatHistory({
       if (conversationId === currentConversationId) {
         onNewConversation();
       }
-    } catch (err) {
-      console.error('Error deleting conversation:', err);
+    } catch {
+      // Error deleting conversation
     }
   }, [dispatch, currentConversationId, onNewConversation]);
 
   const handleRenameConversation = useCallback(async (conversationId: string, newTitle: string) => {
     try {
       await dispatch(renameConversation({ conversationId, newTitle })).unwrap();
-    } catch (err) {
-      console.error('Error renaming conversation:', err);
+    } catch {
+      // Error renaming conversation
     }
   }, [dispatch]);
 

content-gen/src/app/frontend/src/hooks/useChatOrchestrator.ts

@@ -1,7 +1,7 @@
 import { useCallback, type MutableRefObject } from 'react';
 
 import type { GeneratedContent } from '../types';
-import { createMessage, matchesAnyKeyword, createNameSwapper } from '../utils';
+import { createMessage, createErrorMessage, matchesAnyKeyword, createNameSwapper } from '../utils';
 import {
   useAppDispatch,
   useAppSelector,
@@ -467,14 +467,11 @@ export function useChatOrchestrator(
         }
       } catch (error) {
         if (error instanceof Error && error.name === 'AbortError') {
-          console.debug('Request cancelled by user');
           dispatch(addMessage(createMessage('assistant', 'Generation stopped.')));
         } else {
-          console.error('Error sending message:', error);
           dispatch(
             addMessage(
-              createMessage(
-                'assistant',
+              createErrorMessage(
                 'Sorry, there was an error processing your request. Please try again.',
               ),
             ),

content-gen/src/app/frontend/src/hooks/useContentGeneration.ts

@@ -1,6 +1,6 @@
 import { useCallback, type MutableRefObject } from 'react';
 
-import { createMessage, buildGeneratedContent } from '../utils';
+import { createMessage, createErrorMessage, buildGeneratedContent } from '../utils';
 import {
   useAppDispatch,
   useAppSelector,
@@ -66,25 +66,21 @@ export function useContentGeneration(
             const genContent = buildGeneratedContent(rawContent);
             dispatch(setGeneratedContent(genContent));
             dispatch(setGenerationStatus(''));
-          } catch (parseError) {
-            console.error('Error parsing generated content:', parseError);
+          } catch {
+            // Content parse failure — non-critical, generation result may be malformed
           }
         } else if (response.type === 'error') {
           dispatch(setGenerationStatus(''));
-          dispatch(addMessage(createMessage(
-            'assistant',
+          dispatch(addMessage(createErrorMessage(
             `Error generating content: ${response.content}`,
           )));
         }
       }
     } catch (error) {
       if (error instanceof Error && error.name === 'AbortError') {
-        console.debug('Content generation cancelled by user');
         dispatch(addMessage(createMessage('assistant', 'Content generation stopped.')));
       } else {
-        console.error('Error generating content:', error);
-        dispatch(addMessage(createMessage(
-          'assistant',
+        dispatch(addMessage(createErrorMessage(
           'Sorry, there was an error generating content. Please try again.',
         )));
       }

content-gen/src/app/frontend/src/hooks/useConversationActions.ts

@@ -96,11 +96,8 @@ export function useConversationActions() {
               products?: Product[];
             }>('/products');
             dispatch(setAvailableProducts(productsData.products || []));
-          } catch (err) {
-            console.error(
-              'Error loading products for restored conversation:',
-              err,
-            );
+          } catch {
+            // Non-critical — product load failure for restored conversation
           }
         }
 
@@ -122,8 +119,8 @@ export function useConversationActions() {
           dispatch(setGeneratedContent(null));
           dispatch(setSelectedProducts([]));
         }
-      } catch (error) {
-        console.error('Error loading conversation:', error);
+      } catch {
+        // Error loading conversation — swallowed silently
       }
     },
     [userId, dispatch],
@@ -164,8 +161,8 @@ export function useConversationActions() {
           ),
         ),
       );
-    } catch (error) {
-      console.error('Error confirming brief:', error);
+    } catch {
+      // Error confirming brief — swallowed silently
     }
   }, [conversationId, userId, pendingBrief, dispatch]);
 

content-gen/src/app/frontend/src/hooks/useCopyToClipboard.ts

@@ -13,8 +13,8 @@ export function useCopyToClipboard(resetTimeout = 2000) {
 
   const copy = useCallback(
     (text: string) => {
-      navigator.clipboard.writeText(text).catch((err) => {
-        console.error('Failed to copy text:', err);
+      navigator.clipboard.writeText(text).catch(() => {
+        // Clipboard write failure — non-critical
       });
       setCopied(true);
       clearTimeout(timerRef.current);

content-gen/src/app/frontend/src/store/chatHistorySlice.ts

@@ -78,17 +78,6 @@ const chatHistorySlice = createSlice({
     setShowAll(state, action: PayloadAction<boolean>) {
       state.showAll = action.payload;
     },
-    setConversations(state, action: PayloadAction<ConversationSummary[]>) {
-      state.conversations = action.payload;
-    },
-    upsertConversation(state, action: PayloadAction<ConversationSummary>) {
-      const idx = state.conversations.findIndex((c) => c.id === action.payload.id);
-      if (idx >= 0) {
-        state.conversations[idx] = action.payload;
-      } else {
-        state.conversations.unshift(action.payload);
-      }
-    },
     setIsClearAllDialogOpen(state, action: PayloadAction<boolean>) {
       state.isClearAllDialogOpen = action.payload;
     },
@@ -133,6 +122,6 @@ const chatHistorySlice = createSlice({
   },
 });
 
-export const { setShowAll, setConversations, upsertConversation, setIsClearAllDialogOpen } =
+export const { setShowAll, setIsClearAllDialogOpen } =
   chatHistorySlice.actions;
 export default chatHistorySlice.reducer;

content-gen/src/app/frontend/src/store/index.ts

@@ -44,8 +44,6 @@ export {
   renameConversation,
   clearAllConversations,
   setShowAll,
-  setConversations,
-  upsertConversation,
   setIsClearAllDialogOpen,
 } from './chatHistorySlice';
 export type { ConversationSummary } from './chatHistorySlice';

content-gen/src/app/frontend/src/styles/images/SamplePrompt.png

@@ -5,7 +5,7 @@
  */
 
 // Message factories & formatting
-export { createMessage, formatContentForClipboard } from './messageUtils';
+export { createMessage, createErrorMessage } from './messageUtils';
 
 // Content parsing (raw API → typed domain objects)
 export { parseTextContent, resolveImageUrl, buildGeneratedContent } from './contentParsing';
@@ -21,7 +21,7 @@ export type { GenerationStage } from './generationStages';
 export { BRIEF_FIELD_LABELS, BRIEF_DISPLAY_ORDER, BRIEF_FIELD_KEYS } from './briefFields';
 
 // String utilities
-export { escapeRegex, createNameSwapper, matchesAnyKeyword } from './stringUtils';
+export { createNameSwapper, matchesAnyKeyword } from './stringUtils';
 
 // Production API utilities
 export { retryRequest, RequestCache, throttle } from './apiUtils';