fix(mship): tool names

Author: Sg312

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

@@ -485,6 +485,40 @@ describe('completed tool titles', () => {
     expect(firstToolTitle(modelToContentBlocks(model))).toBe('Compared workflows')
   })
 
+  it('humanizes an internal read target through the full wire lifecycle', () => {
+    const model = createTurnModel()
+    reduceEvent(
+      model,
+      toolEnvelope(
+        1,
+        {
+          phase: 'call',
+          toolCallId: 'read-oauth-integrations',
+          toolName: 'read',
+          arguments: { path: 'environment/oauth-integrations.json' },
+        },
+        'auth'
+      )
+    )
+    reduceEvent(
+      model,
+      toolEnvelope(
+        2,
+        {
+          phase: 'result',
+          toolCallId: 'read-oauth-integrations',
+          toolName: 'read',
+          success: true,
+          status: 'success',
+          output: {},
+        },
+        'auth'
+      )
+    )
+
+    expect(firstToolTitle(modelToContentBlocks(model))).toBe('Read OAuth integrations')
+  })
+
   it('renders the completed title through the full wire lifecycle for every visible tool', () => {
     const hiddenToolNames = getHiddenToolNames()
     const failures: string[] = []

apps/sim/lib/copilot/tools/client/store-utils.test.ts

@@ -149,12 +149,35 @@ describe('resolveToolDisplay', () => {
       resolveToolDisplay(ReadTool.id, ClientToolCallState.success, {
         path: 'components/blocks/unknown_block.json',
       })?.text
-    ).toBe('Read unknown_block')
+    ).toBe('Read Unknown block')
+  })
+
+  it('humanizes internal VFS resource identifiers', () => {
+    expect(
+      resolveToolDisplay(ReadTool.id, ClientToolCallState.executing, {
+        path: 'environment/oauth-integrations.json',
+      })?.text
+    ).toBe('Reading OAuth integrations')
+
+    expect(
+      resolveToolDisplay(ReadTool.id, ClientToolCallState.success, {
+        path: 'environment/oauth-integrations.json',
+      })?.text
+    ).toBe('Read OAuth integrations')
+
+    expect(
+      resolveToolDisplay(ReadTool.id, ClientToolCallState.success, {
+        path: 'environment/api-key-integrations.json',
+      })?.text
+    ).toBe('Read API key integrations')
   })
 
   it('falls back to a humanized tool label for generic tools', () => {
     expect(resolveToolDisplay('deploy_api', ClientToolCallState.success)?.text).toBe(
-      'Executed Deploy Api'
+      'Executed Deploy API'
+    )
+    expect(resolveToolDisplay('oauth-integrations', ClientToolCallState.success)?.text).toBe(
+      'Executed OAuth Integrations'
     )
   })
 

apps/sim/lib/copilot/tools/client/store-utils.ts

@@ -6,6 +6,7 @@ import { VFS_DIR_TO_RESOURCE } from '@/lib/copilot/resources/types'
 import { isToolHiddenInUi } from '@/lib/copilot/tools/client/hidden-tools'
 import { getReadTargetBlock } from '@/lib/copilot/tools/client/read-block'
 import { ClientToolCallState } from '@/lib/copilot/tools/client/tool-call-state'
+import { humanizeDisplayIdentifier, humanizeToolName } from '@/lib/copilot/tools/tool-display'
 import { decodeVfsSegmentSafe } from '@/lib/copilot/vfs/path-utils'
 
 /** Respond tools are internal handoff tools shown with a friendly generic label. */
@@ -98,7 +99,7 @@ function describeReadTarget(path: string | undefined): string | undefined {
 
   const resourceType = VFS_DIR_TO_RESOURCE[segments[0]]
   if (!resourceType) {
-    return stripExtension(segments[segments.length - 1])
+    return humanizeDisplayIdentifier(stripExtension(segments[segments.length - 1]), 'sentence')
   }
 
   if (resourceType === 'file') {
@@ -159,9 +160,9 @@ function humanizedFallback(
   toolName: string,
   state: ClientToolCallState
 ): ClientToolDisplay | undefined {
-  const titleCaseName = toolName.replace(/_/g, ' ').replace(/\b\w/g, (c) => c.toUpperCase())
+  const titleCaseName = humanizeToolName(toolName)
   if (state === ClientToolCallState.error) {
-    const lowerCaseName = toolName.replace(/_/g, ' ').toLowerCase()
+    const lowerCaseName = humanizeDisplayIdentifier(toolName, 'sentence')
     return { text: `Attempted to ${lowerCaseName}`, icon: Loader }
   }
   const stateVerb =

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

@@ -58,6 +58,10 @@ describe('humanizeToolName', () => {
     expect(humanizeToolName('manage_folder')).toBe('Manage Folder')
   })
 
+  it('title-cases kebab-case names', () => {
+    expect(humanizeToolName('read-oauth-integrations')).toBe('Read OAuth Integrations')
+  })
+
   it('keeps canonical acronym casing', () => {
     expect(humanizeToolName('create_workspace_mcp_server')).toBe('Create Workspace MCP Server')
     expect(humanizeToolName('deploy_api')).toBe('Deploy API')
@@ -456,11 +460,15 @@ describe('getToolDisplayTitle for operation-driven tools', () => {
 describe('getToolDisplayTitle for request-scoped MCP tools', () => {
   it('hides the internal server id and humanizes the tool name', () => {
     expect(getToolDisplayTitle('mcp-363de040-web_search_exa')).toBe('Web Search Exa')
+    expect(getToolDisplayTitle('mcp-363de040-read-oauth-integrations')).toBe(
+      'Read OAuth Integrations'
+    )
   })
 })
 
 describe('getToolDisplayTitle for context management', () => {
   it('describes compaction in user-facing language', () => {
     expect(getToolDisplayTitle('context_compaction')).toBe('Summarizing context')
+    expect(getToolStatusDisplayTitle('Summarizing context', 'success')).toBe('Summarized context')
   })
 })

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

@@ -503,17 +503,35 @@ const ACRONYM_CASING: Record<string, string> = {
 }
 
 /**
- * Final fallback: humanize a raw tool name (e.g. `manage_folder` -> "Manage
- * Folder"), matching the legacy client humanizer so labels never render blank.
+ * Humanize an internal identifier without leaking snake_case or kebab-case into
+ * the UI. Sentence case is useful for resource names appended to a verb, while
+ * title case is used for standalone tool-name fallbacks.
  */
-export function humanizeToolName(name: string): string {
-  const words = stripVersionSuffix(name).split('_').filter(Boolean)
+export function humanizeDisplayIdentifier(
+  name: string,
+  casing: 'sentence' | 'title' = 'title'
+): string {
+  const words = stripVersionSuffix(name).split(/[-_]+/).filter(Boolean)
   if (words.length === 0) return name
   return words
-    .map((word) => ACRONYM_CASING[word] ?? word.charAt(0).toUpperCase() + word.slice(1))
+    .map((word, index) => {
+      const normalized = word.toLowerCase()
+      const acronym = ACRONYM_CASING[normalized]
+      if (acronym) return acronym
+      if (casing === 'sentence' && index > 0) return normalized
+      return normalized.charAt(0).toUpperCase() + normalized.slice(1)
+    })
     .join(' ')
 }
 
+/**
+ * Final fallback: humanize a raw tool name (e.g. `manage_folder` -> "Manage
+ * Folder"), matching the legacy client humanizer so labels never render blank.
+ */
+export function humanizeToolName(name: string): string {
+  return humanizeDisplayIdentifier(name)
+}
+
 /**
  * Resolve a tool-call display title from its name and arguments. Argument-aware
  * cases come first, then the static map, then a humanized fallback. This never
@@ -846,6 +864,7 @@ const COMPLETED_VERB_REWRITES: Record<string, string> = {
   Scraping: 'Scraped',
   Searching: 'Searched',
   Setting: 'Set',
+  Summarizing: 'Summarized',
   Syncing: 'Synced',
   Toggling: 'Toggled',
   Trimming: 'Trimmed',