fix(chat): keep an activity's completed title off runs while it still waits on the user

Every run of a main-lane activity segment received the segment-wide activity,
including its completedTitle. A finished multi-call run before a pending
approval or terminal handoff (which splits the segment into runs) therefore
read the past-tense completed title while the activity was still unfinished.

Runs now receive the completed title only once every call in the segment has
finished; until then they keep the activity's in-progress title and summarize
their own calls. Fully finished activities render exactly as before.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/agent-group.test.ts

@@ -481,6 +481,50 @@ describe('AgentGroup inline main activity', () => {
     expect(container.textContent).not.toContain('Built API')
   })
 
+  function renderBuildActivity(lastStatus: ToolCallStatus) {
+    const read = (id: string): AgentGroupItem => ({
+      type: 'tool',
+      data: { id, toolName: 'read', displayTitle: `Read ${id}`, status: 'success' },
+    })
+    act(() =>
+      root.render(
+        createElement(AgentGroupView, {
+          agentName: 'mothership',
+          agentLabel: 'Sim',
+          activity: { id: 'build', title: 'Building API', completedTitle: 'Built API' },
+          items: [
+            read('config'),
+            read('schema'),
+            {
+              type: 'tool',
+              data: {
+                id: 'create',
+                toolName: 'create',
+                displayTitle: 'Create route',
+                status: lastStatus,
+              },
+            },
+          ],
+          ToolCallComponent: ({ displayTitle, renderStatus }: ToolCallItemProps) =>
+            renderStatus
+              ? renderStatus({ label: displayTitle, activeLabel: displayTitle, isActive: false })
+              : createElement('div', { 'data-pending': 'true' }, displayTitle),
+        })
+      )
+    )
+  }
+
+  it('keeps the completed title off a finished multi-call run while its activity awaits approval', () => {
+    renderBuildActivity('awaiting_approval')
+    expect(container.textContent).not.toContain('Built API')
+    expect(container.querySelector('[data-pending]')?.textContent).toBe('Create route')
+  })
+
+  it('shows the completed title on a multi-call run once its activity has finished', () => {
+    renderBuildActivity('success')
+    expect(container.querySelector('[role="status"]')?.textContent).toBe('Built API')
+  })
+
   it.each([
     [
       'approval',

apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/main-agent-activity.tsx

@@ -1,5 +1,7 @@
 import { type ComponentType, Fragment, type ReactNode } from 'react'
+import { omit } from '@sim/utils/object'
 import type { ToolActivity } from '@/lib/mothership/generated/protocol'
+import { isAgentGroupResolved } from '@/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/agent-group-content'
 import type { AgentGroupItem } from '@/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/agent-group-view'
 import { splitMainLane } from '@/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/lane-activity'
 import { ToolActivityGroup } from '@/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/tool-activity-group'
@@ -18,6 +20,9 @@ interface MainAgentActivityProps {
 /**
  * The main lane's runs and interaction cards;
  * which run is live is decided by the lane, never by a run's position.
+ * A run reads the activity's completed title only once every call of the
+ * activity has finished, so a finished run before a pending approval or
+ * handoff never claims the whole activity is done.
  */
 export function MainAgentActivity({
   activity: groupActivity,
@@ -28,6 +33,10 @@ export function MainAgentActivity({
   liveToolId,
 }: MainAgentActivityProps) {
   const entries = splitMainLane(items)
+  const runActivity =
+    groupActivity && !isAgentGroupResolved(items)
+      ? omit(groupActivity, ['completedTitle'])
+      : groupActivity
   const activity = entries.map((entry) => {
     if (entry.type === 'item') {
       return (
@@ -41,7 +50,7 @@ export function MainAgentActivity({
       <ToolActivityGroup
         key={tools[0].id}
         tools={tools}
-        activity={groupActivity}
+        activity={runActivity}
         isLive={tools.some((tool) => tool.id === liveToolId)}
         ToolCallComponent={ToolCallComponent}
         autoScrollActivity={autoScrollActivity}