fix(copilot): keep model activity outcomes authoritative

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/tool-call-item.test.tsx

@@ -128,7 +128,8 @@ describe('ToolCallItem', () => {
       />
     )
 
-    expect(markup).toContain('Failed: Stopped checking invoices')
+    expect(markup).toContain('Failed checking invoices')
+    expect(markup).not.toContain('Stopped checking invoices')
   })
 
   it('defensively applies the completed verb for every successful tool row', () => {

apps/sim/lib/copilot/tools/tool-display.test.ts

@@ -843,3 +843,25 @@ describe('normalizeToolActivityDescription', () => {
     expect(normalizeToolActivityDescription('🧪'.repeat(161))).toBeUndefined()
   })
 })
+
+describe('model-authored activity outcomes', () => {
+  it.each([
+    ['success', 'Checking invoices', 'Checked invoices'],
+    ['success', 'Check invoices', 'Completed: Check invoices'],
+    ['success', 'Reconciling invoices', 'Completed: Reconciling invoices'],
+    ['success', 'Revisando facturas', 'Completed: Revisando facturas'],
+    ['success', 'Stopped checking invoices', 'Completed checking invoices'],
+    ['success', 'Completed: Check invoices', 'Completed: Check invoices'],
+    ['error', 'Failed: Fetching invoices', 'Failed: Fetching invoices'],
+    ['error', 'Stopped checking invoices', 'Failed checking invoices'],
+    ['error', 'Completed checking invoices', 'Failed checking invoices'],
+    ['rejected', 'Failed checking invoices', 'Failed checking invoices'],
+    ['cancelled', 'Stopped reading notes', 'Stopped reading notes'],
+    ['interrupted', 'Completed: Check invoices', 'Stopped: Check invoices'],
+    ['skipped', 'Failed: Checking invoices', 'Skipped: Checking invoices'],
+  ])('projects %s once onto "%s"', (status, description, expected) => {
+    const title = getToolStatusDisplayTitle('Fallback', status, 'read', description)
+    expect(title).toBe(expected)
+    expect(getToolStatusDisplayTitle(title, status, 'read', description)).toBe(expected)
+  })
+})

apps/sim/lib/copilot/tools/tool-display.ts

@@ -1408,11 +1408,8 @@ const COMPLETED_VERB_REWRITES: Record<string, string> = {
 /**
  * Rewrite a resolved display title to its past-tense form for a successfully
  * completed tool call (e.g. "Querying logs for X" -> "Queried logs for X").
- * Operates on the already-resolved title so enriched and persisted titles both
- * work. Returns undefined when the title has no leading gerund rewrite — the
- * caller keeps the original. Integration gateway descriptions are base-form
- * verb phrases ("Read recent emails") whose first word never matches a gerund
- * key, so they intentionally pass through unchanged.
+ * Returns undefined when no leading gerund rewrite is known; status formatting
+ * handles the fallback for model-authored and legacy titles.
  */
 export function getToolCompletedTitle(title: string): string | undefined {
   const spaceIndex = title.indexOf(' ')
@@ -1447,11 +1444,18 @@ function statesTerminalOutcome(title: string): boolean {
 /** Apply one terminal outcome prefix while preserving already-resolved titles. */
 function getToolOutcomeTitle(
   title: string,
-  outcome: 'Failed' | 'Stopped' | 'Skipped',
+  outcome: 'Completed' | 'Failed' | 'Stopped' | 'Skipped',
   preserveExistingOutcome: boolean
 ): string {
   if (preserveExistingOutcome && statesTerminalOutcome(title)) return title
   const firstWord = firstWordOf(title)
+  const statedOutcome = firstWord.replace(/:$/, '')
+  if (
+    !preserveExistingOutcome &&
+    (TERMINAL_TITLE_PREFIXES.has(statedOutcome) || statedOutcome === 'Completed')
+  ) {
+    return outcome + title.slice(statedOutcome.length)
+  }
   if (COMPLETED_VERB_REWRITES[firstWord]) {
     return `${outcome} ${firstWord.charAt(0).toLowerCase()}${firstWord.slice(1)}${title.slice(firstWord.length)}`
   }
@@ -1475,7 +1479,12 @@ export function getToolStatusDisplayTitle(
   if (status === 'success' && toolName === 'browser_request_takeover') {
     return 'Resumed browser control'
   }
-  if (status === 'success') return getToolCompletedTitle(title) ?? title
+  if (status === 'success') {
+    return (
+      getToolCompletedTitle(title) ??
+      (description ? getToolOutcomeTitle(title, 'Completed', false) : title)
+    )
+  }
   if (status === 'error' || status === 'rejected') {
     return getToolOutcomeTitle(title, 'Failed', !description)
   }