perf(tests): stop copilot handler tests from loading the executor, block, and trigger registries

The router test stubs every handler implementation module so the handler map
binds without loading the executor and block registry; the 90s timeout goes.
Handler, VFS, payload, and request tests mock the execution-side leaves and
catalog registries they never exercise. Source edges that dragged in barrels
for a single symbol now import the defining module: the Mothership block icon
from @sim/emcn/icons, contract schemas from their contract file, and autolayout
constants from new @sim/workflow-renderer/dimensions and /note-content subpaths.

lib/copilot: 141 files, per-file collect+run 74.2s -> 37.4s, wall 19.1s -> 15.0s
router.test.ts: 15.1s -> 0.45s

Author: waleedlatif1

apps/sim/blocks/blocks/mothership.ts

@@ -1,4 +1,4 @@
-import { Blimp } from '@sim/emcn'
+import { Blimp } from '@sim/emcn/icons'
 import type { BlockConfig } from '@/blocks/types'
 import type { ToolResponse } from '@/tools/types'
 

apps/sim/hooks/queries/environment.ts

@@ -6,7 +6,7 @@ import {
   removeWorkspaceEnvironmentContract,
   savePersonalEnvironmentContract,
   upsertWorkspaceEnvironmentContract,
-} from '@/lib/api/contracts'
+} from '@/lib/api/contracts/environment'
 import type { WorkspaceEnvironmentData } from '@/lib/environment/api'
 import { fetchPersonalEnvironment, fetchWorkspaceEnvironment } from '@/lib/environment/api'
 import { invalidateSelectorQueries } from '@/hooks/queries/utils/selector-keys'

apps/sim/lib/copilot/chat/payload.test.ts

@@ -59,6 +59,9 @@ vi.mock('@/tools/registry', () => ({
   },
 }))
 
+/** Denied-operation projection walks the block map only for blocks the mocked tool list never names. */
+vi.mock('@/blocks/registry-maps', () => ({ BLOCK_REGISTRY: {}, BLOCK_META_REGISTRY: {} }))
+
 vi.mock('@/tools/utils', () => ({
   getLatestVersionTools: vi.fn((input) => input),
   stripVersionSuffix: vi.fn((toolId: string) => toolId),

apps/sim/lib/copilot/chat/process-contents-log-projection.test.ts

@@ -21,12 +21,16 @@ import {
   permissionGroupScopeMockFns,
   resetPermissionGroupScopeMock,
   workflowAuthzMockFns,
+  workflowsUtilsMock,
 } from '@sim/testing'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import type { ChatContext } from '@/stores/panel'
 
 vi.mock('@/lib/permission-groups/config-scope.server', () => permissionGroupScopeMock)
 
+/** Folder listing is untouched by `@log` mentions; the real module drags in the block and trigger registries. */
+vi.mock('@/lib/workflows/utils', () => workflowsUtilsMock)
+
 import { processContextsServer } from '@/lib/copilot/chat/process-contents'
 import { DEFAULT_PERMISSION_GROUP_CONFIG } from '@/lib/permission-groups/fields'
 

apps/sim/lib/copilot/request/go/stream.test.ts

@@ -11,6 +11,12 @@ import {
   MothershipStreamV1ToolPhase,
 } from '@/lib/copilot/generated/mothership-stream-v1'
 
+/** Table side effects are not exercised here, and the real module loads the table application layer. */
+vi.mock('@/lib/copilot/request/tools/tables', () => ({
+  maybeWriteOutputToTable: vi.fn(async (_toolName, _params, result) => result),
+  maybeWriteReadCsvToTable: vi.fn(async (_toolName, _params, result) => result),
+}))
+
 vi.mock('@/lib/copilot/request/session', async () => {
   const actual = await vi.importActual<typeof import('@/lib/copilot/request/session')>(
     '@/lib/copilot/request/session'

apps/sim/lib/copilot/request/handlers/handlers.test.ts

@@ -65,6 +65,12 @@ vi.mock('@/lib/copilot/async-runs/repository', () => ({
   claimWorkflowToolExecution,
 }))
 
+/** Table side effects are not exercised here, and the real module loads the table application layer. */
+vi.mock('@/lib/copilot/request/tools/tables', () => ({
+  maybeWriteOutputToTable: vi.fn(async (_toolName, _params, result) => result),
+  maybeWriteReadCsvToTable: vi.fn(async (_toolName, _params, result) => result),
+}))
+
 vi.mock('@/lib/copilot/request/tools/client', () => ({
   waitForClientToolCompletion,
   waitForToolCompletion,

apps/sim/lib/copilot/request/lifecycle/resume-leg-context.test.ts

@@ -1,6 +1,13 @@
-import { describe, expect, it } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 import { MothershipStreamV1CompletionStatus } from '@/lib/copilot/generated/mothership-stream-v1'
 import { createStreamingContext } from '@/lib/copilot/request/context/request-context'
+
+/** Table side effects are not exercised here, and the real module loads the table application layer. */
+vi.mock('@/lib/copilot/request/tools/tables', () => ({
+  maybeWriteOutputToTable: vi.fn(async (_toolName, _params, result) => result),
+  maybeWriteReadCsvToTable: vi.fn(async (_toolName, _params, result) => result),
+}))
+
 import { makeResumeLegContext, mergeResumeLegOutputs } from '@/lib/copilot/request/lifecycle/run'
 
 // Guards the makeResumeLegContext / mergeResumeLegOutputs contract: the two MUST

apps/sim/lib/copilot/server/agent-url.ts

@@ -1,7 +1,7 @@
 import { db } from '@sim/db'
 import { settings, user } from '@sim/db/schema'
 import { eq } from 'drizzle-orm'
-import { type MothershipEnvironment, mothershipEnvironmentSchema } from '@/lib/api/contracts'
+import { type MothershipEnvironment, mothershipEnvironmentSchema } from '@/lib/api/contracts/user'
 import { SIM_AGENT_API_URL, SIM_AGENT_API_URL_DEFAULT } from '@/lib/copilot/constants'
 import { env } from '@/lib/core/config/env'
 

apps/sim/lib/copilot/tool-executor/router.test.ts

@@ -2,7 +2,50 @@
  * @vitest-environment node
  */
 
-import { describe, expect, it } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
+
+/**
+ * The handler map is a wiring table from tool id to implementation. Only its
+ * shape is asserted here, so every implementation module it imports is stubbed
+ * — loading them for real reaches the block registry, the executor, and most
+ * of `lib/`. Every export resolves to a mock function, which is all the table
+ * needs to bind.
+ */
+const { stubHandlerModule } = vi.hoisted(() => ({
+  stubHandlerModule: () =>
+    new Proxy(
+      {},
+      {
+        get: (_target, name) => (typeof name === 'string' && name !== 'then' ? vi.fn() : undefined),
+        has: (_target, name) => typeof name === 'string' && name !== 'then',
+      }
+    ),
+}))
+
+vi.mock('@/lib/copilot/tools/handlers/deployment/custom-block', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/deployment/deploy', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/deployment/manage', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/function-execute', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/integration-tools', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/management/connect-slack-bot', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/management/manage-credential', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/management/manage-custom-tool', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/management/manage-mcp-tool', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/management/manage-sandbox', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/management/manage-skill', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/materialize-file', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/oauth', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/resources', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/restore-resource', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/run-code', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/vfs', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/vfs-mutate', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/workflow/mutations', stubHandlerModule)
+vi.mock('@/lib/copilot/tools/handlers/workflow/queries', stubHandlerModule)
+
+/** Server-router tools are appended to the map from their own registry, which this test does not cover. */
+vi.mock('@/lib/copilot/tools/server/router', () => ({ getRegisteredServerToolNames: () => [] }))
+
 import { hasHandler } from '@/lib/copilot/tool-executor/executor'
 import { ensureHandlersRegistered } from '@/lib/copilot/tool-executor/register-handlers'
 import {
@@ -21,11 +64,9 @@ describe('workflow-run cancellation tool routing', () => {
     expect(toolRequiresApproval('cancel_workflow_run')).toBe(true)
   })
 
-  // Registration loads the whole handler map on first use, which is most of
-  // `lib/` — well past the default 10s under a fully parallel run.
   it('registers the Sim cancellation handler', async () => {
     await ensureHandlersRegistered()
 
     expect(hasHandler('cancel_workflow_run')).toBe(true)
-  }, 90_000)
+  })
 })

apps/sim/lib/copilot/tools/handlers/materialize-file.test.ts

@@ -108,6 +108,11 @@ vi.mock('@/lib/copilot/vfs/path-utils', () => ({
 }))
 
 vi.mock('@/lib/workflows/operations/import-export', () => ({ parseWorkflowJson: vi.fn() }))
+/** Only the import size cap is read from `import-workflow`; its orchestration dependency is the whole deploy graph. */
+vi.mock('@/lib/workflows/orchestration', () => ({
+  performCreateWorkflow: vi.fn(),
+  performCreateWorkflowTransition: vi.fn(),
+}))
 vi.mock('@/lib/workflows/persistence/utils', () => ({ saveWorkflowToNormalizedTables: vi.fn() }))
 vi.mock('@/lib/workflows/utils', () => ({ deduplicateWorkflowName: vi.fn() }))
 vi.mock('@/app/api/v1/admin/types', () => ({ extractWorkflowMetadata: vi.fn() }))