[Conversation UX] Remove tooltip on assistant mention (#5892)

* [Conversation UX] Remove tooltip on assistant mention * clean * clean 2
dust-tt · Jun 26, 2024 · 0f4123b · 0f4123b
1 parent 13f0b1d
commit 0f4123b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 42 deletions.
diff --git a/front/components/assistant/RenderMessageMarkdown.tsx b/front/components/assistant/RenderMessageMarkdown.tsx
@@ -3,13 +3,9 @@ import {
   ClipboardIcon,
   IconButton,
   SparklesIcon,
-  Tooltip,
   WrenchIcon,
 } from "@dust-tt/sparkle";
-import type {
-  LightAgentConfigurationType,
-  WebsearchResultType,
-} from "@dust-tt/types";
+import type { WebsearchResultType } from "@dust-tt/types";
 import type { RetrievalDocumentType } from "@dust-tt/types";
 import mermaid from "mermaid";
 import dynamic from "next/dynamic";
@@ -175,12 +171,10 @@ export const CitationsContext = React.createContext<CitationsContextType>({
 export function RenderMessageMarkdown({
   content,
   isStreaming,
-  agentConfigurations,
   citationsContext,
 }: {
   content: string;
   isStreaming: boolean;
-  agentConfigurations?: LightAgentConfigurationType[];
   citationsContext?: CitationsContextType;
 }) {
   // Memoize markdown components to avoid unnecessary re-renders that disrupt text selection
@@ -236,19 +230,11 @@ export function RenderMessageMarkdown({
       ),
       // @ts-expect-error - `mention` is a custom tag, currently refused by
       // react-markdown types although the functionality is supported
-      mention: ({ agentName, agentSId }) => {
-        const agentConfiguration = agentConfigurations?.find(
-          (agentConfiguration) => agentConfiguration.sId === agentSId
-        );
-        return (
-          <MentionBlock
-            agentConfiguration={agentConfiguration}
-            agentName={agentName}
-          />
-        );
+      mention: ({ agentName }) => {
+        return <MentionBlock agentName={agentName} />;
       },
     }),
-    [agentConfigurations, isStreaming]
+    [isStreaming]
   );
 
   const markdownPlugins = useMemo(
@@ -281,25 +267,10 @@ export function RenderMessageMarkdown({
   );
 }
 
-function MentionBlock({
-  agentName,
-  agentConfiguration,
-}: {
-  agentName: string;
-  agentConfiguration?: LightAgentConfigurationType;
-}) {
-  const statusText =
-    !agentConfiguration || agentConfiguration?.status === "archived"
-      ? "(This assistant was deleted)"
-      : agentConfiguration?.status === "active"
-        ? ""
-        : "(This assistant is either deactivated or being tested)";
-  const tooltipLabel = agentConfiguration?.description || "" + " " + statusText;
+function MentionBlock({ agentName }: { agentName: string }) {
   return (
     <span className="inline-block cursor-default font-medium text-brand">
-      <Tooltip label={tooltipLabel} position="below">
-        @{agentName}
-      </Tooltip>
+      @{agentName}
     </span>
   );
 }

diff --git a/front/components/assistant/conversation/UserMessage.tsx b/front/components/assistant/conversation/UserMessage.tsx
@@ -9,7 +9,6 @@ import { AgentSuggestion } from "@app/components/assistant/conversation/AgentSug
 import type { MessageSizeType } from "@app/components/assistant/conversation/ConversationMessage";
 import { ConversationMessage } from "@app/components/assistant/conversation/ConversationMessage";
 import { RenderMessageMarkdown } from "@app/components/assistant/RenderMessageMarkdown";
-import { useAgentConfigurations } from "@app/lib/swr";
 
 interface UserMessageProps {
   conversationId: string;
@@ -36,11 +35,6 @@ export function UserMessage({
   contentFragments,
   size,
 }: UserMessageProps) {
-  const { agentConfigurations } = useAgentConfigurations({
-    workspaceId: owner.sId,
-    agentsGetView: { conversationId },
-  });
-
   return (
     <ConversationMessage
       owner={owner}
@@ -61,7 +55,6 @@ export function UserMessage({
           <RenderMessageMarkdown
             content={message.content}
             isStreaming={false}
-            agentConfigurations={agentConfigurations}
           />
         </div>
         {message.mentions.length === 0 && isLastMessage && (