Remove footer with agent status from AgentBlockGrid

Author: jahooma

cli/src/components/__tests__/agent-grid.test.tsx

@@ -139,7 +139,7 @@ describe('AgentBlockGrid (via MessageBlock)', () => {
   })
 
   describe('multiple agents rendering', () => {
-    test('renders multiple agents with count header', () => {
+    test('renders multiple agents without footer label', () => {
       const blocks: ContentBlock[] = [
         createAgentBlock('agent-1', 'File Picker', 'file-picker'),
         createAgentBlock('agent-2', 'Code Searcher', 'code-searcher'),
@@ -153,10 +153,11 @@ describe('AgentBlockGrid (via MessageBlock)', () => {
       expect(markup).toContain('File Picker')
       expect(markup).toContain('Code Searcher')
       expect(markup).toContain('Commander')
-      expect(markup).toContain('3 agents completed')
+      // Footer label was removed as redundant
+      expect(markup).not.toContain('agents completed')
     })
 
-    test('shows running count when agents are running', () => {
+    test('renders running agents without footer label', () => {
       const blocks: ContentBlock[] = [
         createAgentBlock('agent-1', 'File Picker', 'file-picker', 'running'),
         createAgentBlock('agent-2', 'Code Searcher', 'code-searcher', 'running'),
@@ -166,37 +167,10 @@ describe('AgentBlockGrid (via MessageBlock)', () => {
         <MessageBlock {...baseMessageBlockProps} blocks={blocks} />,
       )
 
-      expect(markup).toContain('2 agents running')
-    })
-
-    test('shows running when at least one agent is running', () => {
-      const blocks: ContentBlock[] = [
-        createAgentBlock('agent-1', 'File Picker', 'file-picker', 'complete'),
-        createAgentBlock('agent-2', 'Code Searcher', 'code-searcher', 'running'),
-      ]
-
-      const markup = renderToStaticMarkup(
-        <MessageBlock {...baseMessageBlockProps} blocks={blocks} />,
-      )
-
-      expect(markup).toContain('2 agents running')
-    })
-
-    test('shows running when agent is in streamingAgents set', () => {
-      const blocks: ContentBlock[] = [
-        createAgentBlock('agent-1', 'File Picker', 'file-picker', 'complete'),
-        createAgentBlock('agent-2', 'Code Searcher', 'code-searcher', 'complete'),
-      ]
-
-      const markup = renderToStaticMarkup(
-        <MessageBlock
-          {...baseMessageBlockProps}
-          blocks={blocks}
-          streamingAgents={new Set(['agent-1'])}
-        />,
-      )
-
-      expect(markup).toContain('2 agents running')
+      expect(markup).toContain('File Picker')
+      expect(markup).toContain('Code Searcher')
+      // Footer label was removed as redundant
+      expect(markup).not.toContain('agents running')
     })
   })
 
@@ -237,7 +211,6 @@ describe('AgentBlockGrid (via MessageBlock)', () => {
       expect(markup).toContain('File Picker')
       expect(markup).toContain('Code Searcher')
       expect(markup).toContain('After agents')
-      expect(markup).toContain('2 agents completed')
     })
 
     test('groups only consecutive non-implementor agents', () => {
@@ -252,9 +225,9 @@ describe('AgentBlockGrid (via MessageBlock)', () => {
         <MessageBlock {...baseMessageBlockProps} blocks={blocks} />,
       )
 
-      // First group of 2 agents
-      expect(markup).toContain('2 agents completed')
-      // Single agent after separator shouldn't have header
+      expect(markup).toContain('File Picker 1')
+      expect(markup).toContain('File Picker 2')
+      expect(markup).toContain('Separator')
       expect(markup).toContain('Commander')
     })
   })
@@ -529,7 +502,6 @@ describe('Grid layout width handling', () => {
 
     expect(markup).toContain('Agent 1')
     expect(markup).toContain('Agent 2')
-    expect(markup).toContain('2 agents completed')
   })
 
   test('renders with medium width (up to 2 columns)', () => {
@@ -562,6 +534,5 @@ describe('Grid layout width handling', () => {
     expect(markup).toContain('Agent 1')
     expect(markup).toContain('Agent 2')
     expect(markup).toContain('Agent 3')
-    expect(markup).toContain('3 agents completed')
   })
 })

cli/src/components/blocks/agent-block-grid.tsx

@@ -1,9 +1,6 @@
-import { pluralize } from '@codebuff/common/util/string'
-import { TextAttributes } from '@opentui/core'
 import React, { memo, useCallback } from 'react'
 
 import { GridLayout } from '../grid-layout'
-import { useTheme } from '../../hooks/use-theme'
 import type { AgentContentBlock } from '../../types/chat'
 
 export interface AgentBlockGridProps {
@@ -18,41 +15,13 @@ export interface AgentBlockGridProps {
   ) => React.ReactNode
 }
 
-export function getAgentStatusSummary(
-  agentBlocks: AgentContentBlock[],
-  streamingAgents: Set<string>,
-): string {
-  const running = agentBlocks.filter(
-    (agent) => agent.status === 'running' || streamingAgents.has(agent.agentId),
-  ).length
-  const failed = agentBlocks.filter((agent) => agent.status === 'failed').length
-  const completed = agentBlocks.filter((agent) => agent.status === 'complete').length
-
-  if (running > 0) {
-    return `${pluralize(agentBlocks.length, 'agent')} running`
-  }
-
-  if (failed > 0 && completed > 0) {
-    return `${failed} failed, ${completed} completed`
-  }
-
-  if (failed > 0) {
-    return `${pluralize(failed, 'agent')} failed`
-  }
-
-  return `${pluralize(agentBlocks.length, 'agent')} completed`
-}
-
 export const AgentBlockGrid = memo(
   ({
     agentBlocks,
     keyPrefix,
     availableWidth,
-    streamingAgents,
     renderAgentBranch,
   }: AgentBlockGridProps) => {
-    const theme = useTheme()
-
     const getItemKey = useCallback(
       (agentBlock: AgentContentBlock) => agentBlock.agentId,
       [],
@@ -66,26 +35,12 @@ export const AgentBlockGrid = memo(
 
     if (agentBlocks.length === 0) return null
 
-    const headerText = getAgentStatusSummary(agentBlocks, streamingAgents)
-    const hasFailed = agentBlocks.some((agent) => agent.status === 'failed')
-    const showHeader = agentBlocks.length > 1
-
-    const footer = showHeader ? (
-      <text
-        fg={hasFailed ? theme.error : theme.muted}
-        attributes={TextAttributes.DIM}
-      >
-        {headerText}
-      </text>
-    ) : undefined
-
     return (
       <GridLayout
         items={agentBlocks}
         availableWidth={availableWidth}
         getItemKey={getItemKey}
         renderItem={renderItem}
-        footer={footer}
         marginTop={1}
       />
     )