Skip to content

Commit a30ad25

Browse files
j15zclaude
andcommitted
improvement(copilot): retire search_documentation and get_platform_actions outright, no shims
The transitional apparatus is gone: no search_documentation registry alias, no get_platform_actions handler, and the ids are out of the regenerated catalog/schemas. During the deploy window an old Mothership build calling either id gets the recoverable tool-not-found result. The two ids stay in HIDDEN_TOOL_NAMES forever — like load_agent_skill, historical persisted chats contain their tool calls and must replay without rendering chips for retired tools. The alias test is replaced by a dispatch test pinning search_docs's own catalog -> route -> handler chain and the retired ids' gone-but-chip-hidden state. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent ac8eecc commit a30ad25

9 files changed

Lines changed: 50 additions & 260 deletions

File tree

apps/sim/lib/copilot/generated/tool-catalog-v1.ts

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ export interface ToolCatalogEntry {
6262
| 'get_deployed_workflow_state'
6363
| 'get_deployment_log'
6464
| 'get_page_contents'
65-
| 'get_platform_actions'
6665
| 'get_scheduled_task_logs'
6766
| 'get_workflow_data'
6867
| 'get_workflow_run_options'
@@ -106,7 +105,6 @@ export interface ToolCatalogEntry {
106105
| 'scrape_page'
107106
| 'search'
108107
| 'search_docs'
109-
| 'search_documentation'
110108
| 'search_integration_tools'
111109
| 'search_knowledge_base'
112110
| 'search_library_docs'
@@ -183,7 +181,6 @@ export interface ToolCatalogEntry {
183181
| 'get_deployed_workflow_state'
184182
| 'get_deployment_log'
185183
| 'get_page_contents'
186-
| 'get_platform_actions'
187184
| 'get_scheduled_task_logs'
188185
| 'get_workflow_data'
189186
| 'get_workflow_run_options'
@@ -227,7 +224,6 @@ export interface ToolCatalogEntry {
227224
| 'scrape_page'
228225
| 'search'
229226
| 'search_docs'
230-
| 'search_documentation'
231227
| 'search_integration_tools'
232228
| 'search_knowledge_base'
233229
| 'search_library_docs'
@@ -2478,15 +2474,6 @@ export const GetPageContents: ToolCatalogEntry = {
24782474
},
24792475
}
24802476

2481-
export const GetPlatformActions: ToolCatalogEntry = {
2482-
id: 'get_platform_actions',
2483-
name: 'get_platform_actions',
2484-
route: 'sim',
2485-
mode: 'async',
2486-
parameters: { type: 'object', properties: {} },
2487-
hidden: true,
2488-
}
2489-
24902477
export const GetScheduledTaskLogs: ToolCatalogEntry = {
24912478
id: 'get_scheduled_task_logs',
24922479
name: 'get_scheduled_task_logs',
@@ -4057,22 +4044,6 @@ export const SearchDocs: ToolCatalogEntry = {
40574044
},
40584045
}
40594046

4060-
export const SearchDocumentation: ToolCatalogEntry = {
4061-
id: 'search_documentation',
4062-
name: 'search_documentation',
4063-
route: 'sim',
4064-
mode: 'async',
4065-
parameters: {
4066-
type: 'object',
4067-
properties: {
4068-
query: { type: 'string', description: 'The search query' },
4069-
topK: { type: 'number', description: 'Number of results (default 5, max 25)' },
4070-
},
4071-
required: ['query'],
4072-
},
4073-
hidden: true,
4074-
}
4075-
40764047
export const SearchIntegrationTools: ToolCatalogEntry = {
40774048
id: 'search_integration_tools',
40784049
name: 'search_integration_tools',
@@ -5481,7 +5452,6 @@ export const TOOL_CATALOG: Record<string, ToolCatalogEntry> = {
54815452
[GetDeployedWorkflowState.id]: GetDeployedWorkflowState,
54825453
[GetDeploymentLog.id]: GetDeploymentLog,
54835454
[GetPageContents.id]: GetPageContents,
5484-
[GetPlatformActions.id]: GetPlatformActions,
54855455
[GetScheduledTaskLogs.id]: GetScheduledTaskLogs,
54865456
[GetWorkflowData.id]: GetWorkflowData,
54875457
[GetWorkflowRunOptions.id]: GetWorkflowRunOptions,
@@ -5525,7 +5495,6 @@ export const TOOL_CATALOG: Record<string, ToolCatalogEntry> = {
55255495
[ScrapePage.id]: ScrapePage,
55265496
[Search.id]: Search,
55275497
[SearchDocs.id]: SearchDocs,
5528-
[SearchDocumentation.id]: SearchDocumentation,
55295498
[SearchIntegrationTools.id]: SearchIntegrationTools,
55305499
[SearchKnowledgeBase.id]: SearchKnowledgeBase,
55315500
[SearchLibraryDocs.id]: SearchLibraryDocs,

apps/sim/lib/copilot/generated/tool-schemas-v1.ts

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -2167,13 +2167,6 @@ export const TOOL_RUNTIME_SCHEMAS: Record<string, ToolRuntimeSchemaEntry> = {
21672167
},
21682168
resultSchema: undefined,
21692169
},
2170-
get_platform_actions: {
2171-
parameters: {
2172-
type: 'object',
2173-
properties: {},
2174-
},
2175-
resultSchema: undefined,
2176-
},
21772170
get_scheduled_task_logs: {
21782171
parameters: {
21792172
type: 'object',
@@ -3704,23 +3697,6 @@ export const TOOL_RUNTIME_SCHEMAS: Record<string, ToolRuntimeSchemaEntry> = {
37043697
},
37053698
resultSchema: undefined,
37063699
},
3707-
search_documentation: {
3708-
parameters: {
3709-
type: 'object',
3710-
properties: {
3711-
query: {
3712-
type: 'string',
3713-
description: 'The search query',
3714-
},
3715-
topK: {
3716-
type: 'number',
3717-
description: 'Number of results (default 5, max 25)',
3718-
},
3719-
},
3720-
required: ['query'],
3721-
},
3722-
resultSchema: undefined,
3723-
},
37243700
search_integration_tools: {
37253701
parameters: {
37263702
properties: {

apps/sim/lib/copilot/tool-executor/register-handlers.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import {
1717
GetBlockUpstreamReferences,
1818
GetDeployedWorkflowState,
1919
GetDeploymentLog,
20-
GetPlatformActions,
2120
GetWorkflowData,
2221
GetWorkflowRunOptions,
2322
Glob as GlobTool,
@@ -87,7 +86,6 @@ import { executeManageMcpTool } from '../tools/handlers/management/manage-mcp-to
8786
import { executeManageSkill } from '../tools/handlers/management/manage-skill'
8887
import { executeMaterializeFile } from '../tools/handlers/materialize-file'
8988
import { executeOAuthGetAuthLink, executeOAuthRequestAccess } from '../tools/handlers/oauth'
90-
import { executeGetPlatformActions } from '../tools/handlers/platform'
9189
import { executeOpenResource } from '../tools/handlers/resources'
9290
import { executeRestoreResource } from '../tools/handlers/restore-resource'
9391
import { executeRunCode } from '../tools/handlers/run-code'
@@ -198,7 +196,6 @@ function buildHandlerMap(): Record<string, ToolHandler> {
198196
[OauthRequestAccess.id]: h(executeOAuthRequestAccess),
199197
[OpenResource.id]: h(executeOpenResource),
200198
[RestoreResource.id]: h(executeRestoreResource),
201-
[GetPlatformActions.id]: h(executeGetPlatformActions),
202199
[ListIntegrationTools.id]: h(executeListIntegrationTools),
203200
[MaterializeFile.id]: h(executeMaterializeFile),
204201
[FunctionExecute.id]: h(executeFunctionExecute),

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

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,11 @@
66
// user-meaningful action, only the resolved call_integration_tool row is.
77
// load_skill is the same shape — the agent pulling in a reference guide before
88
// doing the work is a step toward the action, not the action.
9-
// search_documentation is the deprecated pre-rename id of search_docs, kept
10-
// resolvable for one release so a mixed-version deploy works. A call only ever
11-
// arrives from an older Mothership build and renders as the search_docs it maps
12-
// onto, so it needs no chip of its own. Remove with the rest of the shim.
13-
// get_platform_actions is retired the same way: the tool is gone, the id stays
14-
// in the catalog for one release, and a call from an older build should not
15-
// surface a chip for a capability that no longer exists.
9+
// search_documentation (the pre-rename id of search_docs) and
10+
// get_platform_actions (retired; the quick reference lives in the docs corpus)
11+
// are fully unregistered server-side, but historical persisted chats still
12+
// contain their tool calls — like load_agent_skill, these entries stay forever
13+
// so replaying an old transcript never renders a chip for a retired tool.
1614
const HIDDEN_TOOL_NAMES = new Set([
1715
'load_agent_skill',
1816
'load_custom_tool',

apps/sim/lib/copilot/tools/handlers/platform-actions.ts

Lines changed: 0 additions & 118 deletions
This file was deleted.

apps/sim/lib/copilot/tools/handlers/platform.ts

Lines changed: 0 additions & 9 deletions
This file was deleted.

apps/sim/lib/copilot/tools/server/docs/search-docs-alias.test.ts

Lines changed: 0 additions & 63 deletions
This file was deleted.

0 commit comments

Comments
 (0)