summaryrefslogtreecommitdiff
path: root/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx
diff options
context:
space:
mode:
authorTaylor Mullen <[email protected]>2025-04-25 17:11:08 -0700
committerN. Taylor Mullen <[email protected]>2025-04-26 16:08:05 -0700
commit5be89befeff9c4d4f3ab9f508f030bc153fdd06b (patch)
tree9d4f679a0e7292132cab04fdd3c24062fcd66ce8 /packages/cli/src/ui/components/messages/ToolGroupMessage.tsx
parentaa65a4a1fc3f51589c7633217f9d3c8bd0141abb (diff)
feat: Fix flickering in iTerm + scrolling + performance issues.
- Refactors history display using Ink's <Static> component to prevent flickering and improve performance by rendering completed items statically. - Introduces ConsolePatcher component to capture and display console.log, console.warn, and console.error output within the Ink UI, addressing native handling issues. - Introduce a new content splitting mechanism to work better for static items. Basically when content gets too long we will now split content into multiple blocks for Gemini messages to ensure that we can statically cache larger pieces of history. Fixes: - https://b.corp.google.com/issues/411450097 - https://b.corp.google.com/issues/412716309
Diffstat (limited to 'packages/cli/src/ui/components/messages/ToolGroupMessage.tsx')
-rw-r--r--packages/cli/src/ui/components/messages/ToolGroupMessage.tsx16
1 files changed, 14 insertions, 2 deletions
diff --git a/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx b/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx
index 0675411f..2d4982c2 100644
--- a/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx
+++ b/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx
@@ -13,12 +13,14 @@ import { ToolConfirmationMessage } from './ToolConfirmationMessage.js';
import { Colors } from '../../colors.js';
interface ToolGroupMessageProps {
+ groupId: number;
toolCalls: IndividualToolCallDisplay[];
onSubmit: (value: PartListUnion) => void;
}
// Main component renders the border and maps the tools using ToolMessage
export const ToolGroupMessage: React.FC<ToolGroupMessageProps> = ({
+ groupId,
toolCalls,
onSubmit,
}) => {
@@ -29,13 +31,23 @@ export const ToolGroupMessage: React.FC<ToolGroupMessageProps> = ({
return (
<Box
+ key={groupId}
flexDirection="column"
borderStyle="round"
+ /*
+ This width constraint is highly important and protects us from an Ink rendering bug.
+ Since the ToolGroup can typically change rendering states frequently, it can cause
+ Ink to render the border of the box incorrectly and span multiple lines and even
+ cause tearing.
+ */
+ width="100%"
+ marginLeft={1}
borderDimColor={hasPending}
borderColor={borderColor}
+ marginBottom={1}
>
{toolCalls.map((tool) => (
- <React.Fragment key={tool.callId}>
+ <Box key={groupId + '-' + tool.callId} flexDirection="column">
<ToolMessage
key={tool.callId} // Use callId as the key
callId={tool.callId} // Pass callId
@@ -52,7 +64,7 @@ export const ToolGroupMessage: React.FC<ToolGroupMessageProps> = ({
onSubmit={onSubmit}
></ToolConfirmationMessage>
)}
- </React.Fragment>
+ </Box>
))}
{/* Optional: Add padding below the last item if needed,
though ToolMessage already has some vertical space implicitly */}