changed div to popover

This commit is contained in:
Lucas Oliveira 2025-10-02 14:25:10 -03:00
parent a7bd2c2bc2
commit 3fae4c0fa3

View file

@ -15,10 +15,16 @@ import {
Zap,
} from "lucide-react";
import { useEffect, useRef, useState } from "react";
import { filterAccentClasses } from "@/components/knowledge-filter-panel";
import { MarkdownRenderer } from "@/components/markdown-renderer";
import { ProtectedRoute } from "@/components/protected-route";
import { Avatar, AvatarFallback, AvatarImage } from "@/components/ui/avatar";
import { Button } from "@/components/ui/button";
import {
Popover,
PopoverContent,
PopoverTrigger,
} from "@/components/ui/popover";
import { useAuth } from "@/contexts/auth-context";
import { type EndpointType, useChat } from "@/contexts/chat-context";
import { useKnowledgeFilter } from "@/contexts/knowledge-filter-context";
@ -26,7 +32,6 @@ import { useTask } from "@/contexts/task-context";
import { useLoadingStore } from "@/stores/loadingStore";
import { useGetNudgesQuery } from "../api/queries/useGetNudgesQuery";
import Nudges from "./nudges";
import { filterAccentClasses } from "@/components/knowledge-filter-panel";
interface Message {
role: "user" | "assistant";
@ -134,7 +139,6 @@ function ChatPage() {
const messagesEndRef = useRef<HTMLDivElement>(null);
const inputRef = useRef<HTMLTextAreaElement>(null);
const fileInputRef = useRef<HTMLInputElement>(null);
const dropdownRef = useRef<HTMLDivElement>(null);
const streamAbortRef = useRef<AbortController | null>(null);
const streamIdRef = useRef(0);
const lastLoadedConversationRef = useRef<string | null>(null);
@ -193,7 +197,7 @@ function ChatPage() {
"Upload failed with status:",
response.status,
"Response:",
errorText
errorText,
);
throw new Error("Failed to process document");
}
@ -314,13 +318,6 @@ function ChatPage() {
}
};
const handleFilterDropdownToggle = () => {
if (!isFilterDropdownOpen) {
loadAvailableFilters();
}
setIsFilterDropdownOpen(!isFilterDropdownOpen);
};
const handleFilterSelect = (filter: KnowledgeFilterData | null) => {
setSelectedFilter(filter);
setIsFilterDropdownOpen(false);
@ -410,7 +407,7 @@ function ChatPage() {
console.log(
"Loading conversation with",
conversationData.messages.length,
"messages"
"messages",
);
// Convert backend message format to frontend Message interface
const convertedMessages: Message[] = conversationData.messages.map(
@ -538,7 +535,7 @@ function ChatPage() {
) === "string"
? toolCall.function?.arguments || toolCall.arguments
: JSON.stringify(
toolCall.function?.arguments || toolCall.arguments
toolCall.function?.arguments || toolCall.arguments,
),
result: toolCall.result,
status: "completed",
@ -557,7 +554,7 @@ function ChatPage() {
}
return message;
}
},
);
setMessages(convertedMessages);
@ -646,7 +643,7 @@ function ChatPage() {
console.log(
"Chat page received file upload error event:",
filename,
error
error,
);
// Replace the last message with error message
@ -660,64 +657,45 @@ function ChatPage() {
window.addEventListener(
"fileUploadStart",
handleFileUploadStart as EventListener
handleFileUploadStart as EventListener,
);
window.addEventListener(
"fileUploaded",
handleFileUploaded as EventListener
handleFileUploaded as EventListener,
);
window.addEventListener(
"fileUploadComplete",
handleFileUploadComplete as EventListener
handleFileUploadComplete as EventListener,
);
window.addEventListener(
"fileUploadError",
handleFileUploadError as EventListener
handleFileUploadError as EventListener,
);
return () => {
window.removeEventListener(
"fileUploadStart",
handleFileUploadStart as EventListener
handleFileUploadStart as EventListener,
);
window.removeEventListener(
"fileUploaded",
handleFileUploaded as EventListener
handleFileUploaded as EventListener,
);
window.removeEventListener(
"fileUploadComplete",
handleFileUploadComplete as EventListener
handleFileUploadComplete as EventListener,
);
window.removeEventListener(
"fileUploadError",
handleFileUploadError as EventListener
handleFileUploadError as EventListener,
);
};
}, [endpoint, setPreviousResponseIds]);
// Handle click outside to close dropdown
useEffect(() => {
const handleClickOutside = (event: MouseEvent) => {
if (
isFilterDropdownOpen &&
dropdownRef.current &&
!dropdownRef.current.contains(event.target as Node) &&
!inputRef.current?.contains(event.target as Node)
) {
setIsFilterDropdownOpen(false);
setFilterSearchTerm("");
setSelectedFilterIndex(0);
}
};
document.addEventListener("mousedown", handleClickOutside);
return () => {
document.removeEventListener("mousedown", handleClickOutside);
};
}, [isFilterDropdownOpen]);
const { data: nudges = [], cancel: cancelNudges } = useGetNudgesQuery(
previousResponseIds[endpoint]
previousResponseIds[endpoint],
);
const handleSSEStream = async (userMessage: Message) => {
@ -822,7 +800,7 @@ function ChatPage() {
console.log(
"Received chunk:",
chunk.type || chunk.object,
chunk
chunk,
);
// Extract response ID if present
@ -838,14 +816,14 @@ function ChatPage() {
if (chunk.delta.function_call) {
console.log(
"Function call in delta:",
chunk.delta.function_call
chunk.delta.function_call,
);
// Check if this is a new function call
if (chunk.delta.function_call.name) {
console.log(
"New function call:",
chunk.delta.function_call.name
chunk.delta.function_call.name,
);
const functionCall: FunctionCall = {
name: chunk.delta.function_call.name,
@ -861,7 +839,7 @@ function ChatPage() {
else if (chunk.delta.function_call.arguments) {
console.log(
"Function call arguments delta:",
chunk.delta.function_call.arguments
chunk.delta.function_call.arguments,
);
const lastFunctionCall =
currentFunctionCalls[currentFunctionCalls.length - 1];
@ -873,14 +851,14 @@ function ChatPage() {
chunk.delta.function_call.arguments;
console.log(
"Accumulated arguments:",
lastFunctionCall.argumentsString
lastFunctionCall.argumentsString,
);
// Try to parse arguments if they look complete
if (lastFunctionCall.argumentsString.includes("}")) {
try {
const parsed = JSON.parse(
lastFunctionCall.argumentsString
lastFunctionCall.argumentsString,
);
lastFunctionCall.arguments = parsed;
lastFunctionCall.status = "completed";
@ -888,7 +866,7 @@ function ChatPage() {
} catch (e) {
console.log(
"Arguments not yet complete or invalid JSON:",
e
e,
);
}
}
@ -921,7 +899,7 @@ function ChatPage() {
else if (toolCall.function.arguments) {
console.log(
"Tool call arguments delta:",
toolCall.function.arguments
toolCall.function.arguments,
);
const lastFunctionCall =
currentFunctionCalls[
@ -935,7 +913,7 @@ function ChatPage() {
toolCall.function.arguments;
console.log(
"Accumulated tool arguments:",
lastFunctionCall.argumentsString
lastFunctionCall.argumentsString,
);
// Try to parse arguments if they look complete
@ -944,7 +922,7 @@ function ChatPage() {
) {
try {
const parsed = JSON.parse(
lastFunctionCall.argumentsString
lastFunctionCall.argumentsString,
);
lastFunctionCall.arguments = parsed;
lastFunctionCall.status = "completed";
@ -952,7 +930,7 @@ function ChatPage() {
} catch (e) {
console.log(
"Tool arguments not yet complete or invalid JSON:",
e
e,
);
}
}
@ -984,7 +962,7 @@ function ChatPage() {
console.log(
"Error parsing function call on finish:",
fc,
e
e,
);
}
}
@ -1000,12 +978,12 @@ function ChatPage() {
console.log(
"🟢 CREATING function call (added):",
chunk.item.id,
chunk.item.tool_name || chunk.item.name
chunk.item.tool_name || chunk.item.name,
);
// Try to find an existing pending call to update (created by earlier deltas)
let existing = currentFunctionCalls.find(
(fc) => fc.id === chunk.item.id
(fc) => fc.id === chunk.item.id,
);
if (!existing) {
existing = [...currentFunctionCalls]
@ -1014,7 +992,7 @@ function ChatPage() {
(fc) =>
fc.status === "pending" &&
!fc.id &&
fc.name === (chunk.item.tool_name || chunk.item.name)
fc.name === (chunk.item.tool_name || chunk.item.name),
);
}
@ -1027,7 +1005,7 @@ function ChatPage() {
chunk.item.inputs || existing.arguments;
console.log(
"🟢 UPDATED existing pending function call with id:",
existing.id
existing.id,
);
} else {
const functionCall: FunctionCall = {
@ -1045,7 +1023,7 @@ function ChatPage() {
currentFunctionCalls.map((fc) => ({
id: fc.id,
name: fc.name,
}))
})),
);
}
}
@ -1056,7 +1034,7 @@ function ChatPage() {
) {
console.log(
"Function args delta (Realtime API):",
chunk.delta
chunk.delta,
);
const lastFunctionCall =
currentFunctionCalls[currentFunctionCalls.length - 1];
@ -1067,7 +1045,7 @@ function ChatPage() {
lastFunctionCall.argumentsString += chunk.delta || "";
console.log(
"Accumulated arguments (Realtime API):",
lastFunctionCall.argumentsString
lastFunctionCall.argumentsString,
);
}
}
@ -1078,26 +1056,26 @@ function ChatPage() {
) {
console.log(
"Function args done (Realtime API):",
chunk.arguments
chunk.arguments,
);
const lastFunctionCall =
currentFunctionCalls[currentFunctionCalls.length - 1];
if (lastFunctionCall) {
try {
lastFunctionCall.arguments = JSON.parse(
chunk.arguments || "{}"
chunk.arguments || "{}",
);
lastFunctionCall.status = "completed";
console.log(
"Parsed function arguments (Realtime API):",
lastFunctionCall.arguments
lastFunctionCall.arguments,
);
} catch (e) {
lastFunctionCall.arguments = { raw: chunk.arguments };
lastFunctionCall.status = "error";
console.log(
"Error parsing function arguments (Realtime API):",
e
e,
);
}
}
@ -1111,14 +1089,14 @@ function ChatPage() {
console.log(
"🔵 UPDATING function call (done):",
chunk.item.id,
chunk.item.tool_name || chunk.item.name
chunk.item.tool_name || chunk.item.name,
);
console.log(
"🔵 Looking for existing function calls:",
currentFunctionCalls.map((fc) => ({
id: fc.id,
name: fc.name,
}))
})),
);
// Find existing function call by ID or name
@ -1126,14 +1104,14 @@ function ChatPage() {
(fc) =>
fc.id === chunk.item.id ||
fc.name === chunk.item.tool_name ||
fc.name === chunk.item.name
fc.name === chunk.item.name,
);
if (functionCall) {
console.log(
"🔵 FOUND existing function call, updating:",
functionCall.id,
functionCall.name
functionCall.name,
);
// Update existing function call with completion data
functionCall.status =
@ -1156,7 +1134,7 @@ function ChatPage() {
"🔴 WARNING: Could not find existing function call to update:",
chunk.item.id,
chunk.item.tool_name,
chunk.item.name
chunk.item.name,
);
}
}
@ -1177,7 +1155,7 @@ function ChatPage() {
fc.name === chunk.item.name ||
fc.name === chunk.item.type ||
fc.name.includes(chunk.item.type.replace("_call", "")) ||
chunk.item.type.includes(fc.name)
chunk.item.type.includes(fc.name),
);
if (functionCall) {
@ -1221,12 +1199,12 @@ function ChatPage() {
"🟡 CREATING tool call (added):",
chunk.item.id,
chunk.item.tool_name || chunk.item.name,
chunk.item.type
chunk.item.type,
);
// Dedupe by id or pending with same name
let existing = currentFunctionCalls.find(
(fc) => fc.id === chunk.item.id
(fc) => fc.id === chunk.item.id,
);
if (!existing) {
existing = [...currentFunctionCalls]
@ -1238,7 +1216,7 @@ function ChatPage() {
fc.name ===
(chunk.item.tool_name ||
chunk.item.name ||
chunk.item.type)
chunk.item.type),
);
}
@ -1254,7 +1232,7 @@ function ChatPage() {
chunk.item.inputs || existing.arguments;
console.log(
"🟡 UPDATED existing pending tool call with id:",
existing.id
existing.id,
);
} else {
const functionCall = {
@ -1275,7 +1253,7 @@ function ChatPage() {
id: fc.id,
name: fc.name,
type: fc.type,
}))
})),
);
}
}
@ -1553,7 +1531,7 @@ function ChatPage() {
const handleForkConversation = (
messageIndex: number,
event?: React.MouseEvent
event?: React.MouseEvent,
) => {
// Prevent any default behavior and stop event propagation
if (event) {
@ -1618,7 +1596,7 @@ function ChatPage() {
const renderFunctionCalls = (
functionCalls: FunctionCall[],
messageIndex?: number
messageIndex?: number,
) => {
if (!functionCalls || functionCalls.length === 0) return null;
@ -1803,8 +1781,7 @@ function ChatPage() {
));
})()}
<div className="text-xs text-muted-foreground">
Found{" "}
{(() => {
Found {(() => {
let resultsToCount = fc.result;
if (
fc.result.length > 0 &&
@ -1815,8 +1792,7 @@ function ChatPage() {
resultsToCount = fc.result[0].results;
}
return resultsToCount.length;
})()}{" "}
result
})()} result
{(() => {
let resultsToCount = fc.result;
if (
@ -1942,7 +1918,11 @@ function ChatPage() {
{message.role === "user" && (
<div className="flex gap-3">
<Avatar className="w-8 h-8 flex-shrink-0 select-none">
<AvatarImage draggable={false} src={user?.picture} alt={user?.name} />
<AvatarImage
draggable={false}
src={user?.picture}
alt={user?.name}
/>
<AvatarFallback className="text-sm bg-primary/20 text-primary">
{user?.name ? (
user.name.charAt(0).toUpperCase()
@ -1967,7 +1947,7 @@ function ChatPage() {
<div className="flex-1 min-w-0">
{renderFunctionCalls(
message.functionCalls || [],
index
index,
)}
<MarkdownRenderer chatMessage={message.content} />
</div>
@ -1996,7 +1976,7 @@ function ChatPage() {
<div className="flex-1">
{renderFunctionCalls(
streamingMessage.functionCalls,
messages.length
messages.length,
)}
<MarkdownRenderer
chatMessage={streamingMessage.content}
@ -2125,7 +2105,7 @@ function ChatPage() {
const filteredFilters = availableFilters.filter((filter) =>
filter.name
.toLowerCase()
.includes(filterSearchTerm.toLowerCase())
.includes(filterSearchTerm.toLowerCase()),
);
if (e.key === "Escape") {
@ -2143,7 +2123,7 @@ function ChatPage() {
if (e.key === "ArrowDown") {
e.preventDefault();
setSelectedFilterIndex((prev) =>
prev < filteredFilters.length - 1 ? prev + 1 : 0
prev < filteredFilters.length - 1 ? prev + 1 : 0,
);
return;
}
@ -2151,7 +2131,7 @@ function ChatPage() {
if (e.key === "ArrowUp") {
e.preventDefault();
setSelectedFilterIndex((prev) =>
prev > 0 ? prev - 1 : filteredFilters.length - 1
prev > 0 ? prev - 1 : filteredFilters.length - 1,
);
return;
}
@ -2169,7 +2149,7 @@ function ChatPage() {
) {
e.preventDefault();
handleFilterSelect(
filteredFilters[selectedFilterIndex]
filteredFilters[selectedFilterIndex],
);
return;
}
@ -2188,7 +2168,7 @@ function ChatPage() {
) {
e.preventDefault();
handleFilterSelect(
filteredFilters[selectedFilterIndex]
filteredFilters[selectedFilterIndex],
);
return;
}
@ -2225,19 +2205,30 @@ function ChatPage() {
className="hidden"
accept=".pdf,.doc,.docx,.txt,.md,.rtf,.odt"
/>
<Popover
open={isFilterDropdownOpen}
onOpenChange={(open) => {
setIsFilterDropdownOpen(open);
if (open) {
loadAvailableFilters();
}
}}
>
<PopoverTrigger asChild>
<Button
type="button"
variant="ghost"
size="sm"
onClick={handleFilterDropdownToggle}
className="absolute bottom-3 left-3 h-8 w-8 p-0 rounded-full hover:bg-muted/50"
>
<AtSign className="h-4 w-4" />
</Button>
{isFilterDropdownOpen && (
<div
ref={dropdownRef}
className="absolute bottom-14 left-0 w-64 bg-popover border border-border rounded-md shadow-md z-50 p-2"
</PopoverTrigger>
<PopoverContent
className="w-64 p-2"
side="top"
align="start"
sideOffset={8}
>
<div className="space-y-1">
{filterSearchTerm && (
@ -2253,6 +2244,7 @@ function ChatPage() {
<>
{!filterSearchTerm && (
<button
type="button"
onClick={() => handleFilterSelect(null)}
className={`w-full text-left px-2 py-2 text-sm rounded hover:bg-muted/50 flex items-center justify-between ${
selectedFilterIndex === -1 ? "bg-muted/50" : ""
@ -2268,11 +2260,12 @@ function ChatPage() {
.filter((filter) =>
filter.name
.toLowerCase()
.includes(filterSearchTerm.toLowerCase())
.includes(filterSearchTerm.toLowerCase()),
)
.map((filter, index) => (
<button
key={filter.id}
type="button"
onClick={() => handleFilterSelect(filter)}
className={`w-full overflow-hidden text-left px-2 py-2 gap-2 text-sm rounded hover:bg-muted/50 flex items-center justify-between ${
index === selectedFilterIndex ? "bg-muted/50" : ""
@ -2296,7 +2289,7 @@ function ChatPage() {
{availableFilters.filter((filter) =>
filter.name
.toLowerCase()
.includes(filterSearchTerm.toLowerCase())
.includes(filterSearchTerm.toLowerCase()),
).length === 0 &&
filterSearchTerm && (
<div className="px-2 py-3 text-sm text-muted-foreground">
@ -2306,8 +2299,8 @@ function ChatPage() {
</>
)}
</div>
</div>
)}
</PopoverContent>
</Popover>
<Button
type="button"
variant="ghost"