Project summary

src/__fixtures__/chat_links_response.ts

@@ -6,10 +6,19 @@ export const STUB_LINKS_FOR_CHAT_RESPONSE: LinksForChatResponse = {
       text: "Save and return",
       action: "patch-all",
       goto: "SETTINGS:/path/to/config/file.yaml",
+      link_tooltip: "",
+    },
+    {
+      text: "Can you fix it?",
+      action: "follow-up",
+      link_tooltip: "a nice tool tip message",
+    },
+    { text: 'git commit -m "message"', action: "commit", link_tooltip: "" },
+    { text: "Save and return", goto: "SETTINGS:postgres", link_tooltip: "" },
+    {
+      text: "Investigate Project",
+      action: "summarize-project",
+      link_tooltip: "",
     },
-    { text: "Can you fix it?", action: "follow-up" },
-    { text: 'git commit -m "message"', action: "commit" },
-    { text: "Save and return", goto: "SETTINGS:postgres" },
-    { text: "Investigate Project", action: "summarize-project" },
   ],
 };

src/components/ChatContent/ChatContent.tsx

@@ -1,8 +1,6 @@
-import React, { useCallback, useRef, useMemo } from "react";
+import React, { useCallback, useRef } from "react";
 import {
   ChatMessages,
-  diffApi,
-  isAssistantMessage,
   isChatContextFileMessage,
   isDiffMessage,
   isToolMessage,
@@ -122,20 +120,6 @@ export const ChatContent: React.FC<ChatContentProps> = ({
   const thread = useAppSelector(selectThread);
   const isConfig = !!thread.integration;
   const isWaiting = useAppSelector(selectIsWaiting);
-  const [applyAll, applyAllResult] =
-    diffApi.useApplyAllPatchesInMessagesMutation();
-
-  const hasPins = useMemo(
-    () =>
-      messages.some((message) => {
-        if (!isAssistantMessage(message)) return false;
-        if (!message.content) return false;
-        return message.content
-          .split("\n")
-          .some((line) => line.startsWith("📍"));
-      }),
-    [messages],
-  );
 
   const {
     handleScroll,
@@ -170,13 +154,6 @@ export const ChatContent: React.FC<ChatContentProps> = ({
     thread.integration?.path,
   ]);
 
-  const handleSaveAndReturn = useCallback(async () => {
-    const result = await applyAll(messages);
-    if (!result.error) {
-      handleReturnToConfigurationClick();
-    }
-  }, [applyAll, handleReturnToConfigurationClick, messages]);
-
   return (
     <ScrollArea
       ref={scrollRef}
@@ -221,20 +198,6 @@ export const ChatContent: React.FC<ChatContentProps> = ({
             Return
           </Button>
         )}
-
-        {isConfig && hasPins && (
-          <Button
-            ml="auto"
-            color="green"
-            title="Save and return"
-            disabled={isStreaming || applyAllResult.isLoading}
-            onClick={() => {
-              void handleSaveAndReturn();
-            }}
-          >
-            Save
-          </Button>
-        )}
       </Flex>
     </ScrollArea>
   );

src/components/ChatLinks/ChatLinks.tsx

@@ -20,6 +20,7 @@ import {
   setIntegrationData,
 } from "../../features/Chat";
 import { popBackTo } from "../../features/Pages/pagesSlice";
+import { Spinner } from "@radix-ui/themes";
 
 function maybeConcatActionAndGoToStrings(link: ChatLink): string | undefined {
   const hasAction = "action" in link;
@@ -176,32 +177,51 @@ export const ChatLinks: React.FC = () => {
 
   // TODO: waiting, errors, maybe add a title
 
-  if (!linksResult.data || isStreaming || isWaiting || unCalledTools) {
+  if (isStreaming || isWaiting || unCalledTools) {
     return null;
   }
 
   const Wrapper = messages.length === 0 ? Box : Container;
-  return (
-    <Wrapper position="relative" mt="6">
-      <Heading as="h4" size="2" mb="2">
-        Available Actions:{" "}
-      </Heading>
-
-      <Flex gap="2" wrap="wrap" direction="column" align="start">
-        {linksResult.data.links.map((link, index) => {
-          const key = `chat-link-${index}`;
-          return <ChatLinkButton key={key} link={link} onClick={handleClick} />;
-        })}
-      </Flex>
-    </Wrapper>
-  );
+
+  if (linksResult.isLoading) {
+    return (
+      <Wrapper position="relative" mt="6">
+        <Heading as="h4" size="2" mb="2">
+          <Button variant="surface" disabled>
+            <Spinner loading />
+            Checking for actions
+          </Button>
+        </Heading>
+      </Wrapper>
+    );
+  }
+
+  if (linksResult.data && linksResult.data.links.length > 0)
+    return (
+      <Wrapper position="relative" mt="6">
+        <Heading as="h4" size="2" mb="2">
+          Available Actions:{" "}
+        </Heading>
+
+        <Flex gap="2" wrap="wrap" direction="column" align="start">
+          {linksResult.data.links.map((link, index) => {
+            const key = `chat-link-${index}`;
+            return (
+              <ChatLinkButton key={key} link={link} onClick={handleClick} />
+            );
+          })}
+        </Flex>
+      </Wrapper>
+    );
+
+  return null;
 };
 
 const ChatLinkButton: React.FC<{
   link: ChatLink;
   onClick: (link: ChatLink) => void;
 }> = ({ link, onClick }) => {
-  const title = maybeConcatActionAndGoToStrings(link);
+  const title = link.link_tooltip || maybeConcatActionAndGoToStrings(link);
   const handleClick = React.useCallback(() => onClick(link), [link, onClick]);
 
   return (

src/services/refact/links.ts

@@ -6,11 +6,20 @@ import { CHAT_LINKS_URL } from "./consts";
 
 // goto: can be an integration file to open in settings, a file to open in an idea or a global integration.
 export type ChatLink =
-  | { text: string; goto: string; action: string }
-  | { text: string; goto: string /* action: undefined */ }
-  | { text: string; /* goto: undefined; */ action: string }
-  | { text: string; goto: string; action: "go-to" }
-  | { text: string; action: "summarize-project"; current_config_file?: string };
+  | { text: string; goto: string; action: string; link_tooltip: string }
+  | { text: string; goto: string; link_tooltip: string /* action: undefined */ }
+  | {
+      text: string;
+      /* goto: undefined; */ action: string;
+      link_tooltip: string;
+    }
+  | { text: string; goto: string; action: "go-to"; link_tooltip: string }
+  | {
+      text: string;
+      action: "summarize-project";
+      current_config_file?: string;
+      link_tooltip: string;
+    };
 
 function isChatLink(json: unknown): json is ChatLink {
   if (!json || typeof json !== "object") return false;