Skip to content

Commit ab0421e

Browse files
committed
Name the block that ran; never fall back to a raw block id
run_block and set_block_enabled carry only a blockId, so their titles would have printed an opaque UUID ('Running 7f3a2b91-… in Invoice Sync'). The enrichment layer now resolves blockId against the workflow store the same way it already did for run_from_block's startBlockId, and the base titles no longer accept an id as a name — an unresolved block reads 'Running block' rather than a UUID.
1 parent 5ea4b68 commit ab0421e

2 files changed

Lines changed: 13 additions & 5 deletions

File tree

apps/sim/app/workspace/[workspaceId]/home/hooks/stream/stream-helpers.ts

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,15 @@ export function resolveToolDisplayTitle(name: string, args?: Record<string, unkn
252252
// reads, so deployments, reads, and block work all say WHICH workflow.
253253
if (WORKFLOW_SCOPED_TOOL_IDS.has(name) && !stringParam(args?.workflowName)) {
254254
const workflowName = resolveTargetWorkflowName(args)
255-
if (workflowName) return getToolDisplayTitle(name, { ...args, workflowName })
255+
// Block-scoped tools carry a blockId for the same reason; resolve it too,
256+
// so a row says which block ran rather than an opaque id (or nothing).
257+
const blockName = stringParam(args?.blockName) ?? resolveBlockNameForDisplay(args?.blockId)
258+
const enriched = {
259+
...args,
260+
...(workflowName ? { workflowName } : {}),
261+
...(blockName ? { blockName } : {}),
262+
}
263+
if (workflowName || blockName) return getToolDisplayTitle(name, enriched)
256264
}
257265

258266
return getToolDisplayTitle(name, args)

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1105,25 +1105,25 @@ export function getToolDisplayTitle(name: string, args?: Record<string, unknown>
11051105
return workflow ? `Running ${workflow}` : 'Running workflow'
11061106
}
11071107
case 'run_from_block': {
1108-
const block = firstStringArg(args, 'blockName', 'block_name', 'startBlockName', 'blockId')
1108+
const block = firstStringArg(args, 'blockName', 'block_name', 'startBlockName')
11091109
const workflow = firstStringArg(args, 'workflowName', 'name')
11101110
if (!block) return 'Running workflow'
11111111
return workflow ? `Running from ${block} in ${workflow}` : `Running from ${block}`
11121112
}
11131113
case 'run_workflow_until_block': {
1114-
const block = firstStringArg(args, 'blockName', 'block_name', 'untilBlockName', 'blockId')
1114+
const block = firstStringArg(args, 'blockName', 'block_name', 'untilBlockName')
11151115
const workflow = firstStringArg(args, 'workflowName', 'name')
11161116
if (!block) return workflow ? `Running ${workflow}` : 'Running workflow'
11171117
return `Running ${workflow || 'workflow'} until ${block}`
11181118
}
11191119
case 'run_block': {
1120-
const block = firstStringArg(args, 'blockName', 'block_name', 'blockId')
1120+
const block = firstStringArg(args, 'blockName', 'block_name')
11211121
const workflow = firstStringArg(args, 'workflowName', 'name')
11221122
if (!block) return 'Running block'
11231123
return workflow ? `Running ${block} in ${workflow}` : `Running ${block}`
11241124
}
11251125
case 'set_block_enabled': {
1126-
const block = firstStringArg(args, 'blockName', 'block_name', 'blockId')
1126+
const block = firstStringArg(args, 'blockName', 'block_name')
11271127
const workflow = firstStringArg(args, 'workflowName', 'name')
11281128
const verb =
11291129
args?.enabled === false ? 'Disabling' : args?.enabled === true ? 'Enabling' : 'Toggling'

0 commit comments

Comments
 (0)