added new rate limit category, ack PR comments

Author: waleedlatif1

apps/sim/app/api/v1/middleware.ts

@@ -19,7 +19,7 @@ export interface RateLimitResult {
 
 export async function checkRateLimit(
   request: NextRequest,
-  endpoint: 'logs' | 'logs-detail' = 'logs'
+  endpoint: 'logs' | 'logs-detail' | 'workflows' | 'workflow-detail' = 'logs'
 ): Promise<RateLimitResult> {
   try {
     const auth = await authenticateV1Request(request)

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

@@ -15,7 +15,7 @@ export async function GET(request: NextRequest, { params }: { params: Promise<{
   const requestId = crypto.randomUUID().slice(0, 8)
 
   try {
-    const rateLimit = await checkRateLimit(request, 'logs-detail')
+    const rateLimit = await checkRateLimit(request, 'workflow-detail')
     if (!rateLimit.allowed) {
       return createRateLimitResponse(rateLimit)
     }

apps/sim/app/api/v1/workflows/route.ts

@@ -42,7 +42,7 @@ export async function GET(request: NextRequest) {
   const requestId = crypto.randomUUID().slice(0, 8)
 
   try {
-    const rateLimit = await checkRateLimit(request, 'logs')
+    const rateLimit = await checkRateLimit(request, 'workflows')
     if (!rateLimit.allowed) {
       return createRateLimitResponse(rateLimit)
     }
@@ -87,20 +87,21 @@ export async function GET(request: NextRequest) {
     if (params.cursor) {
       const cursorData = decodeCursor(params.cursor)
       if (cursorData) {
-        conditions.push(
-          or(
-            gt(workflow.sortOrder, cursorData.sortOrder),
-            and(
-              eq(workflow.sortOrder, cursorData.sortOrder),
-              gt(workflow.createdAt, new Date(cursorData.createdAt))
-            ),
-            and(
-              eq(workflow.sortOrder, cursorData.sortOrder),
-              eq(workflow.createdAt, new Date(cursorData.createdAt)),
-              gt(workflow.id, cursorData.id)
-            )
-          )!
+        const cursorCondition = or(
+          gt(workflow.sortOrder, cursorData.sortOrder),
+          and(
+            eq(workflow.sortOrder, cursorData.sortOrder),
+            gt(workflow.createdAt, new Date(cursorData.createdAt))
+          ),
+          and(
+            eq(workflow.sortOrder, cursorData.sortOrder),
+            eq(workflow.createdAt, new Date(cursorData.createdAt)),
+            gt(workflow.id, cursorData.id)
+          )
         )
+        if (cursorCondition) {
+          conditions.push(cursorCondition)
+        }
       }
     }
 

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/components/general/components/api-info-modal.tsx

@@ -137,15 +137,19 @@ export function ApiInfoModal({ open, onOpenChange, workflowId }: ApiInfoModalPro
   const handleSave = useCallback(async () => {
     if (!workflowId) return
 
+    const activeWorkflowId = useWorkflowRegistry.getState().activeWorkflowId
+    if (activeWorkflowId !== workflowId) {
+      return
+    }
+
     setIsSaving(true)
     try {
       if (description.trim() !== (workflowMetadata?.description || '')) {
         updateWorkflow(workflowId, { description: description.trim() || 'New workflow' })
       }
 
       if (starterBlockId) {
-        const currentValue = subBlockValues[starterBlockId]?.inputFormat || inputFormat
-        const updatedValue = (currentValue as NormalizedField[]).map((field) => ({
+        const updatedValue = inputFormat.map((field) => ({
           ...field,
           description: paramDescriptions[field.name]?.trim() || undefined,
         }))
@@ -162,7 +166,6 @@ export function ApiInfoModal({ open, onOpenChange, workflowId }: ApiInfoModalPro
     workflowMetadata,
     updateWorkflow,
     starterBlockId,
-    subBlockValues,
     inputFormat,
     paramDescriptions,
     setValue,