agent cli grep: the searchable text leads with the resource's name and description

Author: Sg312

apps/sim/lib/mothership/agent-cli/engines/universal-grep.test.ts

@@ -133,6 +133,22 @@ describe('universal grep', () => {
     expect(result.stdout).not.toContain('sk-should-never-appear')
   })
 
+  it('matches a resource by its own name, not only its body', async () => {
+    // `grep fx-` in an fx-* workspace found nothing: a workflow's state carries no name.
+    const result = await runEngine(
+      'grep',
+      ['fx-etl'],
+      runtimeWith({
+        '/api/v2/workflows': { data: [{ id: 'wf-1', name: 'fx-etl' }], nextCursor: null },
+        '/api/v2/workflows/wf-1/state': { data: { blocks: {}, edges: [] } },
+      }),
+      { scope: 'workflows' }
+    )
+    expect(result.exitCode).toBe(0)
+    expect(result.stdout).toContain('workflows/fx-etl (wf-1):')
+    expect(result.stdout).toContain('name: fx-etl')
+  })
+
   it('reports no matches honestly', async () => {
     const result = await runEngine('grep', ['zzz-nope'], runtimeWith(CATALOG), { scope: 'blocks' })
     expect(result.exitCode).toBe(0)

apps/sim/lib/mothership/agent-cli/engines/universal-grep.ts

@@ -1,3 +1,4 @@
+import { isRecordLike } from '@sim/utils/object'
 import { LRUCache } from 'lru-cache'
 import type { ReadFileTextResponse } from 'sim/embed'
 import {
@@ -95,8 +96,17 @@ async function mapConcurrent<T, R>(
   return results
 }
 
+/**
+ * The searchable text leads with the resource's own name and description: a workflow's
+ * state carries neither, so `grep fx-` in an `fx-*` workspace found nothing.
+ */
 function render(scope: Scope, id: string, label: string, value: unknown): Materialized {
-  return { scope, id, label, text: JSON.stringify(value, null, 2) }
+  const description =
+    isRecordLike(value) && typeof (value as Record<string, unknown>).description === 'string'
+      ? (value as Record<string, unknown>).description
+      : ''
+  const header = `name: ${label}${description ? `\ndescription: ${description}` : ''}\n`
+  return { scope, id, label, text: `${header}${JSON.stringify(value, null, 2)}` }
 }
 
 /** One materializer per scope: the list, then each resource as its `get` returns it. */