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>

Author: j15z

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

@@ -16,7 +16,6 @@ import {
   GetBlockUpstreamReferences,
   GetDeployedWorkflowState,
   GetDeploymentLog,
-  GetPlatformActions,
   GetWorkflowData,
   GetWorkflowRunOptions,
   Glob as GlobTool,
@@ -81,7 +80,6 @@ import { executeManageSandbox } from '../tools/handlers/management/manage-sandbo
 import { executeManageSkill } from '../tools/handlers/management/manage-skill'
 import { executeMaterializeFile } from '../tools/handlers/materialize-file'
 import { executeOAuthGetAuthLink, executeOAuthRequestAccess } from '../tools/handlers/oauth'
-import { executeGetPlatformActions } from '../tools/handlers/platform'
 import { executeOpenResource } from '../tools/handlers/resources'
 import { executeRestoreResource } from '../tools/handlers/restore-resource'
 import { executeRunCode } from '../tools/handlers/run-code'
@@ -192,7 +190,6 @@ function buildHandlerMap(): Record<string, ToolHandler> {
     [OauthRequestAccess.id]: h(executeOAuthRequestAccess),
     [OpenResource.id]: h(executeOpenResource),
     [RestoreResource.id]: h(executeRestoreResource),
-    [GetPlatformActions.id]: h(executeGetPlatformActions),
     [ListIntegrationTools.id]: h(executeListIntegrationTools),
     [MaterializeFile.id]: h(executeMaterializeFile),
     [FunctionExecute.id]: h(executeFunctionExecute),

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

@@ -0,0 +1,45 @@
+/**
+ * @vitest-environment node
+ */
+import { describe, expect, it } from 'vitest'
+import { TOOL_CATALOG } from '@/lib/copilot/generated/tool-catalog-v1'
+import { isKnownTool, isSimExecuted } from '@/lib/copilot/tool-executor/router'
+import { getHiddenToolNames } from '@/lib/copilot/tools/client/hidden-tools'
+import { getRegisteredServerToolNames } from '@/lib/copilot/tools/server/router'
+
+/**
+ * `executeTool` gates on `isKnownTool` (catalog membership) before it ever
+ * consults the handler registry, so a sim-routed tool needs every link of this
+ * chain or dispatch rejects it before the handler is reached. These assertions
+ * pin that chain for search_docs.
+ */
+describe('search_docs dispatch chain', () => {
+  it('is in the catalog, so dispatch does not reject it as unknown', () => {
+    expect(isKnownTool('search_docs')).toBe(true)
+  })
+
+  it('routes to sim, so dispatch reaches the server tool registry', () => {
+    expect(isSimExecuted('search_docs')).toBe(true)
+  })
+
+  it('has a registered server handler', () => {
+    expect(getRegisteredServerToolNames()).toContain('search_docs')
+  })
+})
+
+/**
+ * The retired ids are fully unregistered server-side — no catalog entry, no
+ * handler, no alias. Only the client-side chip suppression survives, forever,
+ * so historical persisted chats replay without rendering chips for tools that
+ * no longer exist (the load_agent_skill precedent).
+ */
+describe('retired docs-tool ids', () => {
+  for (const retired of ['search_documentation', 'get_platform_actions']) {
+    it(`${retired} is gone from the catalog and server registry but stays chip-hidden`, () => {
+      expect(TOOL_CATALOG[retired]).toBeUndefined()
+      expect(isKnownTool(retired)).toBe(false)
+      expect(getRegisteredServerToolNames()).not.toContain(retired)
+      expect(getHiddenToolNames().has(retired)).toBe(true)
+    })
+  }
+})

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

@@ -164,11 +164,6 @@ const baseServerToolRegistry: Record<string, BaseServerTool> = {
   [editWorkflowServerTool.name]: editWorkflowServerTool,
   [queryLogsServerTool.name]: queryLogsServerTool,
   [searchDocsServerTool.name]: searchDocsServerTool,
-  // Transitional alias: sim and mothership deploy independently, so during the
-  // rollout of the search_documentation -> search_docs rename one side is still
-  // emitting the old id. The old params are a subset of the new, so routing them
-  // here is safe. Remove once both repos have shipped the rename.
-  search_documentation: searchDocsServerTool,
   [searchOnlineServerTool.name]: searchOnlineServerTool,
   [setEnvironmentVariablesServerTool.name]: setEnvironmentVariablesServerTool,
   [getCredentialsServerTool.name]: getCredentialsServerTool,