Skip to content

Commit 1ed1929

Browse files
j15zclaude
andcommitted
fix(review): hide the retired get_platform_actions chip like search_documentation
The two retirement shims disagreed. search_documentation is in HIDDEN_TOOL_NAMES with no TOOL_TITLES entry, so a call from an older build renders nothing. get_platform_actions was only hidden:true in the generated catalog, which isToolHiddenInUi never consults, and it kept a live title — so the same situation surfaced a "Getting platform actions" chip for a capability that no longer exists. Hiding also drops the row from historical transcripts, since isToolHiddenInUi runs at transcript build time. That is accepted and is the existing behavior for search_documentation. Verified it cannot error: every consumer skips via continue or returns undefined, nothing indexes off the omitted entry, and neither id is a subagent trigger, so no tool group is orphaned. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 7be0522 commit 1ed1929

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

apps/sim/lib/copilot/tools/client/hidden-tools.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,16 @@
66
// resolvable for one release so a mixed-version deploy works. A call only ever
77
// arrives from an older Mothership build and renders as the search_docs it maps
88
// onto, so it needs no chip of its own. Remove with the rest of the shim.
9+
// get_platform_actions is retired the same way: the tool is gone, the id stays
10+
// in the catalog for one release, and a call from an older build should not
11+
// surface a chip for a capability that no longer exists.
912
const HIDDEN_TOOL_NAMES = new Set([
1013
'load_agent_skill',
1114
'load_custom_tool',
1215
'load_integration_tool',
1316
'search_integration_tools',
1417
'search_documentation',
18+
'get_platform_actions',
1519
])
1620

1721
export function isToolHiddenInUi(toolName: string | undefined): boolean {

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -443,7 +443,6 @@ const TOOL_TITLES: Record<string, string> = {
443443
get_block_upstream_references: 'Getting block references',
444444
get_deployed_workflow_state: 'Getting deployed workflow',
445445
get_deployment_log: 'Getting deployment logs',
446-
get_platform_actions: 'Getting platform actions',
447446
get_scheduled_task_logs: 'Getting scheduled task logs',
448447
get_workflow_data: 'Getting workflow data',
449448
get_workflow_run_options: 'Getting run options',

0 commit comments

Comments
 (0)