refactor(orchestration): move the shared error contract out of lib/workflows

OrchestrationErrorCode and statusForOrchestrationError are the contract every
lib/[resource]/orchestration module returns against, but they lived inside the
workflows module, so resource-neutral code (lib/folders) already had to import
from a workflow path. Moved to lib/core/orchestration/types so the table,
knowledge, workspace, and organization modules landing next don't cement that.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: TheodoreSpeaks

apps/sim/app/api/tools/deployments/deploy/route.ts

@@ -3,10 +3,10 @@ import { assertWorkflowMutable, WorkflowLockedError } from '@sim/platform-authz/
 import { type NextRequest, NextResponse } from 'next/server'
 import { deploymentsDeployContract } from '@/lib/api/contracts/tools/deployments'
 import { getValidationErrorMessage, parseRequest } from '@/lib/api/server'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 import { generateRequestId } from '@/lib/core/utils/request'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { performFullDeploy } from '@/lib/workflows/orchestration'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
 import {
   authenticateDeploymentToolRequest,
   authorizeDeploymentWorkflow,

apps/sim/app/api/tools/deployments/promote/route.ts

@@ -3,10 +3,10 @@ import { assertWorkflowMutable, WorkflowLockedError } from '@sim/platform-authz/
 import { type NextRequest, NextResponse } from 'next/server'
 import { deploymentsPromoteContract } from '@/lib/api/contracts/tools/deployments'
 import { getValidationErrorMessage, parseRequest } from '@/lib/api/server'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 import { generateRequestId } from '@/lib/core/utils/request'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { performActivateVersion } from '@/lib/workflows/orchestration'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
 import {
   authenticateDeploymentToolRequest,
   authorizeDeploymentWorkflow,

apps/sim/app/api/v1/workflows/[id]/deploy/route.ts

@@ -8,11 +8,11 @@ import {
   v1UndeployWorkflowContract,
 } from '@/lib/api/contracts/v1/workflows'
 import { parseOptionalJsonBody, parseRequest } from '@/lib/api/server'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 import { generateRequestId } from '@/lib/core/utils/request'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { captureServerEvent } from '@/lib/posthog/server'
 import { performFullDeploy, performFullUndeploy } from '@/lib/workflows/orchestration'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
 import { createApiResponse, getUserLimits } from '@/app/api/v1/logs/meta'
 import {
   checkRateLimit,

apps/sim/app/api/v1/workflows/[id]/rollback/route.ts

@@ -7,10 +7,10 @@ import {
   v1RollbackWorkflowContract,
 } from '@/lib/api/contracts/v1/workflows'
 import { parseOptionalJsonBody, parseRequest } from '@/lib/api/server'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 import { generateRequestId } from '@/lib/core/utils/request'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { performActivateVersion } from '@/lib/workflows/orchestration'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
 import { findPreviousDeploymentVersion } from '@/lib/workflows/persistence/utils'
 import { createApiResponse, getUserLimits } from '@/app/api/v1/logs/meta'
 import {

apps/sim/app/api/workflows/[id]/deploy/route.ts

@@ -7,6 +7,7 @@ import { eq } from 'drizzle-orm'
 import type { NextRequest } from 'next/server'
 import { updatePublicApiContract } from '@/lib/api/contracts/deployments'
 import { parseRequest } from '@/lib/api/server'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 import { generateRequestId } from '@/lib/core/utils/request'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { captureServerEvent } from '@/lib/posthog/server'
@@ -15,7 +16,6 @@ import {
   performFullDeploy,
   performFullUndeploy,
 } from '@/lib/workflows/orchestration'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
 import { validateWorkflowPermissions } from '@/lib/workflows/utils'
 import {
   checkNeedsRedeployment,

apps/sim/app/api/workflows/[id]/deployments/[version]/route.ts

@@ -4,10 +4,10 @@ import { and, eq } from 'drizzle-orm'
 import type { NextRequest } from 'next/server'
 import { updateDeploymentVersionMetadataContract } from '@/lib/api/contracts/deployments'
 import { getValidationErrorMessage, parseRequest } from '@/lib/api/server'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 import { generateRequestId } from '@/lib/core/utils/request'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { performActivateVersion } from '@/lib/workflows/orchestration'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
 import {
   getWorkflowDeploymentVersion,
   updateDeploymentVersionMetadata,

…ib/workflows/orchestration/types.test.ts …sim/lib/core/orchestration/types.test.tsapps/sim/lib/workflows/orchestration/types.test.ts renamed to apps/sim/lib/core/orchestration/types.test.ts apps/sim/lib/workflows/orchestration/types.test.ts renamed to apps/sim/lib/core/orchestration/types.test.ts

@@ -2,7 +2,7 @@
  * @vitest-environment node
  */
 import { describe, expect, it } from 'vitest'
-import { statusForOrchestrationError } from '@/lib/workflows/orchestration/types'
+import { statusForOrchestrationError } from '@/lib/core/orchestration/types'
 
 describe('statusForOrchestrationError', () => {
   it.each([

…sim/lib/workflows/orchestration/types.ts apps/sim/lib/core/orchestration/types.tsapps/sim/lib/workflows/orchestration/types.ts renamed to apps/sim/lib/core/orchestration/types.ts apps/sim/lib/workflows/orchestration/types.ts renamed to apps/sim/lib/core/orchestration/types.ts

@@ -1,8 +1,9 @@
 export type OrchestrationErrorCode = 'validation' | 'not_found' | 'conflict' | 'internal'
 
 /**
- * Maps an orchestration error code to its HTTP status. Shared by every route
- * surface (UI, v1, tool routes) so deployment errors map identically.
+ * Transport-neutral failure classes returned by every `lib/[resource]/orchestration`
+ * module, so the UI routes, the public API, and the copilot tools map the same
+ * failure to the same status.
  */
 export function statusForOrchestrationError(code: OrchestrationErrorCode | undefined): number {
   if (code === 'validation') return 400

apps/sim/lib/folders/lifecycle.ts

@@ -6,6 +6,7 @@ import { getPostgresErrorCode } from '@sim/utils/errors'
 import { generateId } from '@sim/utils/id'
 import { and, eq, isNull, min } from 'drizzle-orm'
 import type { FolderCascadeCountsApi, FolderResourceType } from '@/lib/api/contracts/folders'
+import type { OrchestrationErrorCode } from '@/lib/core/orchestration/types'
 import type { DbOrTx } from '@/lib/db/types'
 import {
   archiveFolderCascade,
@@ -19,7 +20,6 @@ import { folderResourceConfig } from '@/lib/folders/config'
 import { deduplicateFolderName } from '@/lib/folders/naming'
 import { wouldCreateFolderCycle } from '@/lib/folders/queries'
 import type { FolderMutationErrorCode } from '@/lib/folders/status'
-import type { OrchestrationErrorCode } from '@/lib/workflows/orchestration/types'
 
 const logger = createLogger('FolderLifecycle')
 

apps/sim/lib/folders/status.ts

@@ -1,4 +1,4 @@
-import type { OrchestrationErrorCode } from '@/lib/workflows/orchestration/types'
+import type { OrchestrationErrorCode } from '@/lib/core/orchestration/types'
 
 /**
  * Folder mutations can fail for one reason the shared orchestration vocabulary has no word