Improve nested tool status presentation

Author: Sg312

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

@@ -236,20 +236,13 @@ export function ToolCallItem({
   }
 
   return (
-    /* Same left geometry as the agent-group header (16px slot + gap-2): every
-       row's text lands on the 24px-per-level grid. The slot renders even when
-       the tool has no icon — internal tools (grep, read, edit_workflow) were
-       starting 20px left of integration rows, giving a mixed lane three
-       different text edges. */
-    <div className='flex min-w-0 items-center gap-2 pl-6'>
-      <div className='flex size-[16px] flex-shrink-0 items-center justify-center'>
-        {BlockIcon && (
-          <BlockIcon
-            className='size-[14px] text-[var(--text-icon)]'
-            style={getBareIconStyle(BlockIcon)}
-          />
-        )}
-      </div>
+    <div className='flex min-w-0 items-center gap-[6px] pl-6'>
+      {BlockIcon && (
+        <BlockIcon
+          className='size-[14px] flex-shrink-0 text-[var(--text-icon)]'
+          style={getBareIconStyle(BlockIcon)}
+        />
+      )}
       {isExecuting ? (
         <ShimmerText className='min-w-0 truncate text-[13px] leading-[18px] [--shimmer-rest:var(--text-secondary)]'>
           {title}

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

@@ -64,7 +64,7 @@ describe('resolveToolDisplay', () => {
       resolveToolDisplay(ReadTool.id, ClientToolCallState.error, {
         path: 'workflows/Elder v2/The Elder/lint.json',
       })?.text
-    ).toBe('Attempted to read lint results for The Elder')
+    ).toBe('Attempted to validate The Elder')
     expect(
       resolveToolDisplay(ReadTool.id, ClientToolCallState.success, {
         path: 'tables/CRM/Leads/views.json',
@@ -233,3 +233,24 @@ describe('resolveToolDisplay', () => {
     expect(resolveToolDisplay('load_custom_tool', ClientToolCallState.executing)).toBeUndefined()
   })
 })
+
+describe('lint reads render as validation', () => {
+  it.each([
+    [ClientToolCallState.generating, 'Validating The Elder'],
+    [ClientToolCallState.success, 'Validated The Elder'],
+    [ClientToolCallState.error, 'Attempted to validate The Elder'],
+    [ClientToolCallState.aborted, 'Skipped validating The Elder'],
+  ])('state %s -> %s', (state, expected) => {
+    expect(
+      resolveToolDisplay('read', state, { path: 'workflows/Elder v2/The Elder/lint.json' })?.text
+    ).toBe(expected)
+  })
+
+  it('only workflow lint artifacts get the verb; other reads keep Reading', () => {
+    expect(
+      resolveToolDisplay('read', ClientToolCallState.success, {
+        path: 'workflows/Elder v2/The Elder/meta.json',
+      })?.text
+    ).toBe('Read metadata for The Elder')
+  })
+})

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

@@ -45,7 +45,15 @@ function specialToolDisplay(
   }
 
   if (toolName === ReadTool.id) {
-    const target = describeReadTarget(readStringParam(params, 'path'))
+    const path = readStringParam(params, 'path')
+    // lint.json is computed at read time, so reading it IS running the checks —
+    // "Validating X" describes the outcome where "Reading issues in X" would
+    // describe the mechanism.
+    const validated = describeValidationReadTarget(path)
+    if (validated) {
+      return { text: formatValidatingLabel(validated, state), icon: FileText }
+    }
+    const target = describeReadTarget(path)
     return {
       text: formatReadingLabel(target, state),
       icon: FileText,
@@ -83,6 +91,33 @@ function formatReadingLabel(target: string | undefined, state: ClientToolCallSta
   }
 }
 
+/** The workflow name when `path` is a lint artifact; undefined otherwise. */
+function describeValidationReadTarget(path: string | undefined): string | undefined {
+  if (!path) return undefined
+  const segments = path
+    .split('/')
+    .map((segment) => segment.trim())
+    .filter(Boolean)
+    .map(decodeVfsSegmentSafe)
+  if (segments.length < 2 || segments[segments.length - 1] !== 'lint.json') return undefined
+  if (VFS_DIR_TO_RESOURCE[segments[0]] !== 'workflow') return undefined
+  return stripExtension(getLeafResourceSegment(segments))
+}
+
+function formatValidatingLabel(target: string, state: ClientToolCallState): string {
+  switch (state) {
+    case ClientToolCallState.success:
+      return `Validated ${target}`
+    case ClientToolCallState.error:
+      return `Attempted to validate ${target}`
+    case ClientToolCallState.rejected:
+    case ClientToolCallState.aborted:
+      return `Skipped validating ${target}`
+    default:
+      return `Validating ${target}`
+  }
+}
+
 function describeReadTarget(path: string | undefined): string | undefined {
   if (!path) return undefined
 
@@ -127,7 +162,6 @@ function describeReadTarget(path: string | undefined): string | undefined {
 const RESOURCE_ARTIFACT_LABELS: Record<string, string> = {
   'state.json': '',
   'meta.json': 'metadata for',
-  'lint.json': 'lint results for',
   'deployment.json': 'deployment status for',
   'versions.json': 'versions of',
   'executions.json': 'runs of',