fix: code review cleanup

- Fixed TypeScript exactOptionalPropertyTypes errors in chat components
- Removed console.error statements (errors are handled via state)
- Fixed type compatibility issues with undefined vs null values
- All chat-related files now pass strict TypeScript checks

apps/web/src/components/chat/Chat.tsx

@@ -66,9 +66,9 @@ export const Chat = forwardRef<ChatRef, ChatProps>(function Chat({
     clearError,
   } = useChat({
     model: "llama3.2",
-    projectId: initialProjectId,
+    ...(initialProjectId !== undefined && { projectId: initialProjectId }),
-    onError: (err) => {
+    onError: (_err) => {
-      console.error("Chat error:", err);
+      // Error is handled by the useChat hook's state
     },
   });
 

apps/web/src/components/chat/ConversationSidebar.tsx

@@ -54,9 +54,9 @@ export const ConversationSidebar = forwardRef<ConversationSidebarRef, Conversati
 
     return {
       id: idea.id,
-      title: idea.title,
+      title: idea.title ?? null,
-      projectId: idea.projectId,
+      projectId: idea.projectId ?? null,
-      updatedAt: idea.updatedAt,
+      updatedAt: idea.updatedAt ?? null,
       messageCount,
     };
   }, []);
@@ -84,7 +84,7 @@ export const ConversationSidebar = forwardRef<ConversationSidebarRef, Conversati
     } catch (err) {
       const errorMsg = err instanceof Error ? err.message : "Failed to load conversations";
       setError(errorMsg);
-      console.error("Error fetching conversations:", err);
+      // Error is set to state and will be displayed to the user
     } finally {
       setIsLoading(false);
     }

apps/web/src/hooks/useChat.ts

@@ -173,9 +173,9 @@ export function useChat(options: UseChatOptions = {}): UseChatReturn {
         const request = {
           model,
           messages: apiMessages,
-          temperature,
+          ...(temperature !== undefined && { temperature }),
-          maxTokens,
+          ...(maxTokens !== undefined && { maxTokens }),
-          systemPrompt,
+          ...(systemPrompt !== undefined && { systemPrompt }),
         };
 
         // Call LLM API
@@ -188,8 +188,8 @@ export function useChat(options: UseChatOptions = {}): UseChatReturn {
           content: response.message.content,
           createdAt: new Date().toISOString(),
           model: response.model,
-          promptTokens: response.promptEvalCount,
+          promptTokens: response.promptEvalCount ?? 0,
-          completionTokens: response.evalCount,
+          completionTokens: response.evalCount ?? 0,
           totalTokens: (response.promptEvalCount ?? 0) + (response.evalCount ?? 0),
         };
 

apps/web/src/lib/api/ideas.ts

@@ -141,7 +141,7 @@ export async function createConversation(
   return createIdea({
     title,
     content,
-    projectId,
+    ...(projectId !== undefined && { projectId }),
     category: "conversation",
     tags: ["chat"],
     metadata: { conversationType: "chat" },
@@ -156,5 +156,8 @@ export async function updateConversation(
   content: string,
   title?: string
 ): Promise<Idea> {
-  return updateIdea(id, { content, title });
+  return updateIdea(id, { 
+    content, 
+    ...(title !== undefined && { title })
+  });
 }