address comments

Author: icecrasher321

apps/sim/app/api/copilot/chat/resources/route.ts

@@ -38,7 +38,15 @@ export const POST = withRouteHandler(async (req: NextRequest) => {
       return createUnauthorizedResponse()
     }
 
-    const parsed = await parseRequest(addCopilotChatResourceContract, req, {})
+    const parsed = await parseRequest(
+      addCopilotChatResourceContract,
+      req,
+      {},
+      {
+        validationErrorResponse: (error) =>
+          createBadRequestResponse(error.issues.map((e) => e.message).join(', ')),
+      }
+    )
     if (!parsed.success) return parsed.response
     const { chatId, resource } = parsed.data.body
 
@@ -99,7 +107,15 @@ export const PATCH = withRouteHandler(async (req: NextRequest) => {
       return createUnauthorizedResponse()
     }
 
-    const parsed = await parseRequest(reorderCopilotChatResourcesContract, req, {})
+    const parsed = await parseRequest(
+      reorderCopilotChatResourcesContract,
+      req,
+      {},
+      {
+        validationErrorResponse: (error) =>
+          createBadRequestResponse(error.issues.map((e) => e.message).join(', ')),
+      }
+    )
     if (!parsed.success) return parsed.response
     const { chatId, resources: newOrder } = parsed.data.body
 
@@ -142,7 +158,15 @@ export const DELETE = withRouteHandler(async (req: NextRequest) => {
       return createUnauthorizedResponse()
     }
 
-    const parsed = await parseRequest(removeCopilotChatResourceContract, req, {})
+    const parsed = await parseRequest(
+      removeCopilotChatResourceContract,
+      req,
+      {},
+      {
+        validationErrorResponse: (error) =>
+          createBadRequestResponse(error.issues.map((e) => e.message).join(', ')),
+      }
+    )
     if (!parsed.success) return parsed.response
     const { chatId, resourceType, resourceId } = parsed.data.body
 

apps/sim/app/api/copilot/checkpoints/revert/route.ts

@@ -98,20 +98,11 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
         : new Date(rawDeployedAt as string | number | Date)
 
     const cleanedState: CleanedWorkflowState = {
-      blocks:
-        rawBlocks && typeof rawBlocks === 'object' && !Array.isArray(rawBlocks)
-          ? (rawBlocks as Record<string, unknown>)
-          : {},
-      edges: Array.isArray(rawEdges) ? rawEdges : [],
-      loops:
-        rawLoops && typeof rawLoops === 'object' && !Array.isArray(rawLoops)
-          ? (rawLoops as Record<string, unknown>)
-          : {},
-      parallels:
-        rawParallels && typeof rawParallels === 'object' && !Array.isArray(rawParallels)
-          ? (rawParallels as Record<string, unknown>)
-          : {},
-      isDeployed: checkpointState.isDeployed === true,
+      blocks: (rawBlocks ?? {}) as Record<string, unknown>,
+      edges: (rawEdges ?? []) as unknown[],
+      loops: (rawLoops ?? {}) as Record<string, unknown>,
+      parallels: (rawParallels ?? {}) as Record<string, unknown>,
+      isDeployed: Boolean(checkpointState.isDeployed),
       lastSaved: Date.now(),
       ...(parsedDeployedAt && !Number.isNaN(parsedDeployedAt.getTime())
         ? { deployedAt: parsedDeployedAt }