diff --git a/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx b/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx index e2df3d9c..a320194b 100644 --- a/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx +++ b/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx @@ -80,6 +80,7 @@ export const ToolGroupMessage: React.FC = ({ marginLeft={1} borderDimColor={hasPending} borderColor={borderColor} + gap={1} > {toolCalls.map((tool) => { const isConfirming = toolAwaitingApproval?.callId === tool.callId; diff --git a/packages/cli/src/ui/components/messages/ToolMessage.test.tsx b/packages/cli/src/ui/components/messages/ToolMessage.test.tsx index c9bed003..843ebf03 100644 --- a/packages/cli/src/ui/components/messages/ToolMessage.test.tsx +++ b/packages/cli/src/ui/components/messages/ToolMessage.test.tsx @@ -71,19 +71,19 @@ describe('', () => { StreamingState.Idle, ); const output = lastFrame(); - expect(output).toContain('✔'); // Success indicator + expect(output).toContain('√'); // Success indicator expect(output).toContain('test-tool'); expect(output).toContain('A tool for testing'); expect(output).toContain('MockMarkdown:Test result'); }); describe('ToolStatusIndicator rendering', () => { - it('shows ✔ for Success status', () => { + it('shows √ for Success status', () => { const { lastFrame } = renderWithContext( , StreamingState.Idle, ); - expect(lastFrame()).toContain('✔'); + expect(lastFrame()).toContain('√'); }); it('shows o for Pending status', () => { @@ -125,7 +125,7 @@ describe('', () => { ); expect(lastFrame()).toContain('⊷'); expect(lastFrame()).not.toContain('MockRespondingSpinner'); - expect(lastFrame()).not.toContain('✔'); + expect(lastFrame()).not.toContain('√'); }); it('shows paused spinner for Executing status when streamingState is WaitingForConfirmation', () => { @@ -135,7 +135,7 @@ describe('', () => { ); expect(lastFrame()).toContain('⊷'); expect(lastFrame()).not.toContain('MockRespondingSpinner'); - expect(lastFrame()).not.toContain('✔'); + expect(lastFrame()).not.toContain('√'); }); it('shows MockRespondingSpinner for Executing status when streamingState is Responding', () => { @@ -144,7 +144,7 @@ describe('', () => { StreamingState.Responding, // Simulate app still responding ); expect(lastFrame()).toContain('MockRespondingSpinner'); - expect(lastFrame()).not.toContain('✔'); + expect(lastFrame()).not.toContain('√'); }); }); diff --git a/packages/cli/src/ui/components/messages/ToolMessage.tsx b/packages/cli/src/ui/components/messages/ToolMessage.tsx index e1eb75b8..a4d24529 100644 --- a/packages/cli/src/ui/components/messages/ToolMessage.tsx +++ b/packages/cli/src/ui/components/messages/ToolMessage.tsx @@ -127,7 +127,7 @@ const ToolStatusIndicator: React.FC = ({ /> )} {status === ToolCallStatus.Success && ( - + )} {status === ToolCallStatus.Confirming && ( ?