From 165d99fab734bc0f89e8a852e8238e1df3ff2bbe Mon Sep 17 00:00:00 2001
From: Louis Knight-Webb <louis@bloop.ai>
Date: Tue, 1 Jul 2025 00:03:57 +0100
Subject: [PATCH] Shrink follow ups (#30)

* Task attempt bc164e47-b15c-4550-abd3-1f980a2750c9 - Final changes

* Task attempt bc164e47-b15c-4550-abd3-1f980a2750c9 - Final changes

* Task attempt bc164e47-b15c-4550-abd3-1f980a2750c9 - Final changes

* Task attempt bc164e47-b15c-4550-abd3-1f980a2750c9 - Final changes

* Task attempt bc164e47-b15c-4550-abd3-1f980a2750c9 - Final changes

* Lint

* Prettier
---
 .../src/components/tasks/TaskDetailsPanel.tsx |  1 -
 .../components/tasks/TaskFollowUpSection.tsx  | 57 +++++++------------
 .../components/ui/file-search-textarea.tsx    |  4 +-
 3 files changed, 25 insertions(+), 37 deletions(-)

diff --git a/frontend/src/components/tasks/TaskDetailsPanel.tsx b/frontend/src/components/tasks/TaskDetailsPanel.tsx
index ebccdbcf..2104a04e 100644
--- a/frontend/src/components/tasks/TaskDetailsPanel.tsx
+++ b/frontend/src/components/tasks/TaskDetailsPanel.tsx
@@ -191,7 +191,6 @@ export function TaskDetailsPanel({
                   followUpError={followUpError}
                   setFollowUpError={setFollowUpError}
                   canSendFollowUp={canSendFollowUp}
-                  isAttemptRunning={isAttemptRunning}
                   projectId={projectId}
                   onSendFollowUp={handleSendFollowUp}
                 />
diff --git a/frontend/src/components/tasks/TaskFollowUpSection.tsx b/frontend/src/components/tasks/TaskFollowUpSection.tsx
index 11116b22..a3e320cc 100644
--- a/frontend/src/components/tasks/TaskFollowUpSection.tsx
+++ b/frontend/src/components/tasks/TaskFollowUpSection.tsx
@@ -1,7 +1,6 @@
 import { Send, AlertCircle } from 'lucide-react';
 import { Button } from '@/components/ui/button';
 import { Alert, AlertDescription } from '@/components/ui/alert';
-import { Label } from '@/components/ui/label';
 import { FileSearchTextarea } from '@/components/ui/file-search-textarea';
 
 interface TaskFollowUpSectionProps {
@@ -11,7 +10,6 @@ interface TaskFollowUpSectionProps {
   followUpError: string | null;
   setFollowUpError: (error: string | null) => void;
   canSendFollowUp: boolean;
-  isAttemptRunning: boolean;
   projectId: string;
   onSendFollowUp: () => void;
 }
@@ -23,23 +21,21 @@ export function TaskFollowUpSection({
   followUpError,
   setFollowUpError,
   canSendFollowUp,
-  isAttemptRunning,
   projectId,
   onSendFollowUp,
 }: TaskFollowUpSectionProps) {
   return (
-    <div className="border-t p-6">
-      <div className="space-y-3">
-        <Label className="text-sm font-medium">Follow-up question</Label>
+    <div className="border-t p-4">
+      <div className="space-y-2">
         {followUpError && (
           <Alert variant="destructive">
             <AlertCircle className="h-4 w-4" />
             <AlertDescription>{followUpError}</AlertDescription>
           </Alert>
         )}
-        <div className="space-y-3">
+        <div className="flex gap-2 items-start">
           <FileSearchTextarea
-            placeholder="Ask a follow-up question about this task... Type @ to search files."
+            placeholder="Ask a follow-up question... Type @ to search files."
             value={followUpMessage}
             onChange={(value) => {
               setFollowUpMessage(value);
@@ -57,37 +53,28 @@ export function TaskFollowUpSection({
                 }
               }
             }}
-            className="w-full min-h-[80px] resize-none"
+            className="flex-1 min-h-[40px] resize-none"
             disabled={!canSendFollowUp}
             projectId={projectId}
-            rows={4}
+            rows={1}
           />
-          <div className="flex justify-end">
-            <Button
-              onClick={onSendFollowUp}
-              disabled={
-                !canSendFollowUp || !followUpMessage.trim() || isSendingFollowUp
-              }
-              size="sm"
-            >
-              {isSendingFollowUp ? (
-                <div className="animate-spin rounded-full h-4 w-4 border-b-2 border-current" />
-              ) : (
-                <>
-                  <Send className="h-4 w-4 mr-2" />
-                  Send
-                </>
-              )}
-            </Button>
-          </div>
+          <Button
+            onClick={onSendFollowUp}
+            disabled={
+              !canSendFollowUp || !followUpMessage.trim() || isSendingFollowUp
+            }
+            size="sm"
+          >
+            {isSendingFollowUp ? (
+              <div className="animate-spin rounded-full h-4 w-4 border-b-2 border-current" />
+            ) : (
+              <>
+                <Send className="h-4 w-4 mr-2" />
+                Send
+              </>
+            )}
+          </Button>
         </div>
-        <p className="text-xs text-muted-foreground">
-          {!canSendFollowUp
-            ? isAttemptRunning
-              ? 'Wait for current execution to complete before asking follow-up questions'
-              : 'Complete at least one coding agent execution to enable follow-up questions'
-            : 'Continue the conversation with the most recent executor session'}
-        </p>
       </div>
     </div>
   );
diff --git a/frontend/src/components/ui/file-search-textarea.tsx b/frontend/src/components/ui/file-search-textarea.tsx
index 01316d1b..66289291 100644
--- a/frontend/src/components/ui/file-search-textarea.tsx
+++ b/frontend/src/components/ui/file-search-textarea.tsx
@@ -265,7 +265,9 @@ export function FileSearchTextarea({
   const dropdownPosition = getDropdownPosition();
 
   return (
-    <div className="relative">
+    <div
+      className={`relative ${className?.includes('flex-1') ? 'flex-1' : ''}`}
+    >
       <Textarea
         ref={textareaRef}
         value={value}