Merge branch 'main' into fix/sharepoint-folders

Author: mfortman11

.coderabbit.yaml

@@ -33,14 +33,8 @@ reviews:
       instructions: "Apply when the PR only changes docs, README, AGENTS.md, or CLAUDE.md."
     - label: "refactor"
       instructions: "Apply when the PR restructures existing code without changing behaviour."
-    - label: "performance"
-      instructions: "Apply when the PR improves runtime speed, memory usage, or resource efficiency."
-    - label: "test"
+    - label: "tests"
       instructions: "Apply when the PR only adds or updates tests."
-    - label: "chore"
-      instructions: "Apply for maintenance tasks: dependency bumps, config changes, tooling updates."
-    - label: "build"
-      instructions: "Apply when the PR changes the build system, Dockerfiles, CI workflows, or release process."
   path_filters: []
   path_instructions:
     - path: "src/**/*.py"
@@ -156,6 +150,19 @@ chat:
       usage: auto
     linear:
       usage: auto
+issue_enrichment:
+  auto_enrich:
+    enabled: true
+  planning:
+    enabled: true
+    auto_planning:
+      enabled: true
+      labels:
+        - "enhancement"
+        - "bug"
+        - "refactor"
+        - "!documentation"
+        - "!tests"
 knowledge_base:
   opt_out: false
   code_guidelines:

frontend/app/chat/_components/assistant-message.tsx

@@ -11,13 +11,15 @@ const MarkdownRenderer = dynamic(
   { ssr: false },
 );
 
+import { trackButton } from "@/lib/analytics";
 import { cn } from "@/lib/utils";
 import type {
   FunctionCall,
   TokenUsage as TokenUsageType,
 } from "../_types/types";
 import { FunctionCalls } from "./function-calls";
 import { Message } from "./message";
+import MessageActions from "./message-actions";
 import { TokenUsage } from "./token-usage";
 
 interface AssistantMessageProps {
@@ -35,6 +37,8 @@ interface AssistantMessageProps {
   delay?: number;
   isInitialGreeting?: boolean;
   usage?: TokenUsageType;
+  timestamp?: Date;
+  showFeedback?: boolean;
 }
 
 export function AssistantMessage({
@@ -52,7 +56,19 @@ export function AssistantMessage({
   delay = 0.2,
   isInitialGreeting = false,
   usage,
+  timestamp,
+  showFeedback = true,
 }: AssistantMessageProps) {
+  const trackFeedback = (feedback: "like" | "dislike") => {
+    trackButton({
+      action: feedback,
+      elementId: "message-feedback",
+      namespace: "chat",
+      CTA: feedback === "like" ? "Like Message" : "Dislike Message",
+      timestamp: timestamp?.getTime(),
+    });
+  };
+
   return (
     <motion.div
       initial={animate ? { opacity: 0, y: -20 } : { opacity: 1, y: 0 }}
@@ -151,6 +167,9 @@ export function AssistantMessage({
               }
             />
             {usage && !isStreaming && <TokenUsage usage={usage} />}
+            {!isInitialGreeting && showFeedback && !isStreaming && (
+              <MessageActions trackFeedback={trackFeedback} />
+            )}
           </motion.div>
         </div>
       </Message>

frontend/app/chat/_components/message-actions.tsx

@@ -0,0 +1,58 @@
+import { ThumbsDown, ThumbsUp } from "lucide-react";
+import { useState } from "react";
+import { Button } from "@/components/ui/button";
+
+interface MessageActionsProps {
+  trackFeedback: (feedback: "like" | "dislike") => void;
+}
+
+const MessageActions = ({ trackFeedback }: MessageActionsProps) => {
+  const [feedbackSelected, setFeedbackSelected] = useState<
+    "like" | "dislike" | null
+  >(null);
+
+  const handleFeedback = (feedback: "like" | "dislike") => {
+    if (feedbackSelected === feedback) return; // Prevent multiple tracking events for the same feedback
+    trackFeedback(feedback);
+    setFeedbackSelected(feedback);
+  };
+
+  return (
+    <div className="flex space-x-2 mt-2">
+      <Button
+        variant="ghost"
+        size="icon"
+        aria-label="Like"
+        aria-pressed={feedbackSelected === "like"}
+        className={
+          feedbackSelected !== "like"
+            ? "text-muted-foreground hover:text-foreground"
+            : ""
+        }
+        onClick={() => handleFeedback("like")}
+      >
+        <ThumbsUp
+          className={`h-4 w-4 ${feedbackSelected === "like" ? "fill-current" : ""}`}
+        />
+      </Button>
+      <Button
+        variant="ghost"
+        size="icon"
+        aria-label="Dislike"
+        aria-pressed={feedbackSelected === "dislike"}
+        className={
+          feedbackSelected !== "dislike"
+            ? "text-muted-foreground hover:text-foreground"
+            : ""
+        }
+        onClick={() => handleFeedback("dislike")}
+      >
+        <ThumbsDown
+          className={`h-4 w-4 ${feedbackSelected === "dislike" ? "fill-current" : ""}`}
+        />
+      </Button>
+    </div>
+  );
+};
+
+export default MessageActions;

frontend/app/chat/_types/types.ts

@@ -21,6 +21,12 @@ export interface Message {
   usage?: TokenUsage;
 }
 
+export const INITIAL_ASSISTANT_MESSAGE: Message = {
+  role: "assistant",
+  content: "How can I assist?",
+  timestamp: new Date(),
+};
+
 export interface FunctionCall {
   name: string;
   arguments?: Record<string, unknown>;

frontend/app/chat/page.tsx

@@ -13,7 +13,6 @@ import { trackLLMCall } from "@/lib/analytics";
 import { FILE_CONFIRMATION, FILES_REGEX } from "@/lib/constants";
 import { buildSearchPayloadFilters } from "@/lib/filter-normalization";
 import { cn } from "@/lib/utils";
-import { useLoadingStore } from "@/stores/loadingStore";
 import { useGetConversationsQuery } from "../api/queries/useGetConversationsQuery";
 import { useGetNudgesQuery } from "../api/queries/useGetNudgesQuery";
 import { useGetSettingsQuery } from "../api/queries/useGetSettingsQuery";
@@ -29,10 +28,10 @@ import type {
   RequestBody,
   ToolCallResult,
 } from "./_types/types";
+import { INITIAL_ASSISTANT_MESSAGE } from "./_types/types";
 
 function ChatPage() {
   const isDebugMode = process.env.NEXT_PUBLIC_OPENRAG_DEBUG === "true";
-  const isCloudBrand = useIsCloudBrand();
   const {
     endpoint,
     setEndpoint,
@@ -50,16 +49,13 @@ function ChatPage() {
     placeholderConversation,
     conversationFilter,
     setConversationFilter,
+    loading,
+    setLoading,
   } = useChat();
   const [messages, setMessages] = useState<Message[]>([
-    {
-      role: "assistant",
-      content: "How can I assist?",
-      timestamp: new Date(),
-    },
+    INITIAL_ASSISTANT_MESSAGE,
   ]);
   const [input, setInput] = useState("");
-  const { loading, setLoading } = useLoadingStore();
   const { setChatError } = useChat();
   const [asyncMode, setAsyncMode] = useState(true);
   const [expandedFunctionCalls, setExpandedFunctionCalls] = useState<
@@ -109,7 +105,7 @@ function ChatPage() {
     streamingMessage,
     sendMessage: sendStreamingMessage,
     abortStream,
-    isLoading: isStreamLoading,
+    isLoading: isChatStreaming,
   } = useChatStreaming({
     endpoint: apiEndpoint,
     onComplete: (message, responseId) => {
@@ -169,7 +165,7 @@ function ChatPage() {
   useEffect(() => {
     let timeoutId: NodeJS.Timeout | null = null;
 
-    if (isStreamLoading && !streamingMessage) {
+    if (isChatStreaming && !streamingMessage) {
       timeoutId = setTimeout(() => {
         setWaitingTooLong(true);
       }, 20000); // 20 seconds
@@ -180,7 +176,7 @@ function ChatPage() {
     return () => {
       if (timeoutId) clearTimeout(timeoutId);
     };
-  }, [isStreamLoading, streamingMessage]);
+  }, [isChatStreaming, streamingMessage]);
 
   const handleEndpointChange = (newEndpoint: EndpointType) => {
     setEndpoint(newEndpoint);
@@ -328,13 +324,7 @@ function ChatPage() {
       // Abort any in-flight streaming so it doesn't bleed into new chat
       abortStream();
       // Reset chat UI even if context state was already 'new'
-      setMessages([
-        {
-          role: "assistant",
-          content: "How can I assist?",
-          timestamp: new Date(),
-        },
-      ]);
+      setMessages([INITIAL_ASSISTANT_MESSAGE]);
       setInput("");
       setExpandedFunctionCalls(new Set());
       setIsFilterHighlighted(false);
@@ -372,7 +362,7 @@ function ChatPage() {
 
     if (
       conversationData?.messages &&
-      (isNewConversation || (!isStreamLoading && hasMessageCountChanged)) &&
+      (isNewConversation || (!isChatStreaming && hasMessageCountChanged)) &&
       !isUserInteracting &&
       !isForkingInProgress
     ) {
@@ -564,21 +554,15 @@ function ChatPage() {
     isUserInteracting,
     isForkingInProgress,
     setPreviousResponseIds,
-    isStreamLoading,
+    isChatStreaming,
     messages.length,
   ]);
 
   // Handle new conversation creation - only reset messages when placeholderConversation is set
   useEffect(() => {
     if (placeholderConversation && currentConversationId === null) {
       console.log("Starting new conversation");
-      setMessages([
-        {
-          role: "assistant",
-          content: "How can I assist?",
-          timestamp: new Date(),
-        },
-      ]);
+      setMessages([INITIAL_ASSISTANT_MESSAGE]);
       lastLoadedConversationRef.current = null;
 
       // Focus input when starting a new conversation
@@ -588,93 +572,6 @@ function ChatPage() {
     }
   }, [placeholderConversation, currentConversationId]);
 
-  // Listen for file upload events from navigation
-  useEffect(() => {
-    const handleFileUploadStart = (event: CustomEvent) => {
-      const { filename } = event.detail;
-      console.log("Chat page received file upload start event:", filename);
-
-      setLoading(true);
-      setIsUploading(true);
-      setUploadedFile(null); // Clear previous file
-    };
-
-    const handleFileUploaded = (event: CustomEvent) => {
-      const { result } = event.detail;
-      console.log("Chat page received file upload event:", result);
-
-      setUploadedFile(null); // Clear file after upload
-
-      // Update the response ID for this endpoint
-      if (result.response_id) {
-        setPreviousResponseIds((prev) => ({
-          ...prev,
-          [endpoint]: result.response_id,
-        }));
-      }
-    };
-
-    const handleFileUploadComplete = () => {
-      console.log("Chat page received file upload complete event");
-      setLoading(false);
-      setIsUploading(false);
-    };
-
-    const handleFileUploadError = (event: CustomEvent) => {
-      const { filename, error } = event.detail;
-      console.log(
-        "Chat page received file upload error event:",
-        filename,
-        error,
-      );
-
-      // Replace the last message with error message
-      const errorMessage: Message = {
-        role: "assistant",
-        content: `❌ Upload failed for **${filename}**: ${error}`,
-        timestamp: new Date(),
-      };
-      setMessages((prev) => [...prev.slice(0, -1), errorMessage]);
-      setUploadedFile(null); // Clear file on error
-    };
-
-    window.addEventListener(
-      "fileUploadStart",
-      handleFileUploadStart as EventListener,
-    );
-    window.addEventListener(
-      "fileUploaded",
-      handleFileUploaded as EventListener,
-    );
-    window.addEventListener(
-      "fileUploadComplete",
-      handleFileUploadComplete as EventListener,
-    );
-    window.addEventListener(
-      "fileUploadError",
-      handleFileUploadError as EventListener,
-    );
-
-    return () => {
-      window.removeEventListener(
-        "fileUploadStart",
-        handleFileUploadStart as EventListener,
-      );
-      window.removeEventListener(
-        "fileUploaded",
-        handleFileUploaded as EventListener,
-      );
-      window.removeEventListener(
-        "fileUploadComplete",
-        handleFileUploadComplete as EventListener,
-      );
-      window.removeEventListener(
-        "fileUploadError",
-        handleFileUploadError as EventListener,
-      );
-    };
-  }, [endpoint, setPreviousResponseIds, setLoading]);
-
   // Check onboarding completion
 
   // Check if onboarding is complete (current_step >= 4 means complete)
@@ -1142,9 +1039,11 @@ function ChatPage() {
                             isInitialGreeting={
                               index === 0 &&
                               messages.length === 1 &&
-                              message.content === "How can I assist?"
+                              message.content ===
+                                INITIAL_ASSISTANT_MESSAGE.content
                             }
                             usage={message.usage}
+                            timestamp={message.timestamp}
                           />
                         )}
                       </div>

frontend/app/onboarding/_components/onboarding-content.tsx

@@ -287,6 +287,7 @@ export function OnboardingContent({
                 onToggle={() => {}}
                 isStreaming={!!streamingMessage}
                 isCompleted={currentStep > 3}
+                showFeedback={false}
               />
             )}