update trigger-save UI, increase auto disable to 100 consecutive from 10

Author: waleedlatif1

apps/docs/content/docs/en/triggers/schedule.mdx

@@ -56,7 +56,7 @@ You must deploy your workflow for the schedule to start running. Configure the s
 
 ## Automatic Disabling
 
-Schedules automatically disable after **10 consecutive failures** to prevent runaway errors. When disabled:
+Schedules automatically disable after **100 consecutive failures** to prevent runaway errors. When disabled:
 
 - A warning badge appears on the schedule block
 - The schedule stops executing

apps/sim/app/api/schedules/route.test.ts

@@ -144,7 +144,7 @@ describe('Schedule GET API', () => {
   it('indicates disabled schedule with failures', async () => {
     mockDbChain([
       [{ userId: 'user-1', workspaceId: null }],
-      [{ id: 'sched-1', status: 'disabled', failedCount: 10 }],
+      [{ id: 'sched-1', status: 'disabled', failedCount: 100 }],
     ])
 
     const res = await GET(createRequest('http://test/api/schedules?workflowId=wf-1'))

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/schedule-info/schedule-info.tsx

@@ -46,7 +46,7 @@ export function ScheduleInfo({ blockId, isPreview = false }: ScheduleInfoProps)
       {(failedCount > 0 || isDisabled) && (
         <div className='space-y-1'>
           <div className='flex flex-wrap items-center gap-2'>
-            {failedCount >= 10 && isDisabled ? (
+            {failedCount >= 100 && isDisabled ? (
               <Badge
                 variant='outline'
                 className='cursor-pointer'
@@ -70,9 +70,9 @@ export function ScheduleInfo({ blockId, isPreview = false }: ScheduleInfoProps)
               </Badge>
             ) : null}
           </div>
-          {failedCount >= 10 && isDisabled && (
+          {failedCount >= 100 && isDisabled && (
             <p className='text-[12px] text-[var(--text-tertiary)]'>
-              Disabled after 10 consecutive failures
+              Disabled after 100 consecutive failures
             </p>
           )}
         </div>

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/trigger-save/trigger-save.tsx

@@ -8,7 +8,6 @@ import {
   ModalHeader,
 } from '@/components/emcn/components'
 import { Trash } from '@/components/emcn/icons/trash'
-import { Alert, AlertDescription } from '@/components/ui/alert'
 import { cn } from '@/lib/core/utils/cn'
 import { createLogger } from '@/lib/logs/console/logger'
 import { useCollaborativeWorkflow } from '@/hooks/use-collaborative-workflow'
@@ -367,12 +366,7 @@ export function TriggerSave({
             saveStatus === 'error' && 'bg-red-600 hover:bg-red-700'
           )}
         >
-          {saveStatus === 'saving' && (
-            <>
-              <div className='mr-2 h-4 w-4 animate-spin rounded-full border-[1.5px] border-current border-t-transparent' />
-              Saving...
-            </>
-          )}
+          {saveStatus === 'saving' && 'Saving...'}
           {saveStatus === 'saved' && 'Saved'}
           {saveStatus === 'error' && 'Error'}
           {saveStatus === 'idle' && (webhookId ? 'Update Configuration' : 'Save Configuration')}
@@ -394,59 +388,48 @@ export function TriggerSave({
         )}
       </div>
 
-      {errorMessage && (
-        <Alert variant='destructive' className='mt-2'>
-          <AlertDescription>{errorMessage}</AlertDescription>
-        </Alert>
-      )}
+      {errorMessage && <p className='mt-2 text-[12px] text-[var(--text-error)]'>{errorMessage}</p>}
 
       {webhookId && hasWebhookUrlDisplay && (
-        <div className='mt-2 space-y-1'>
+        <div className='mt-4 space-y-2'>
           <div className='flex items-center justify-between'>
-            <span className='font-medium text-sm'>Test Webhook URL</span>
+            <span className='font-medium text-[13px] text-[var(--text-primary)]'>
+              Test Webhook URL
+            </span>
             <Button
-              variant='outline'
+              variant='ghost'
               onClick={generateTestUrl}
               disabled={isGeneratingTestUrl || isProcessing}
-              className='h-[32px] rounded-[8px] px-[12px]'
             >
-              {isGeneratingTestUrl ? (
-                <>
-                  <div className='mr-2 h-3 w-3 animate-spin rounded-full border-[1.5px] border-current border-t-transparent' />
-                  Generating…
-                </>
-              ) : testUrl ? (
-                'Regenerate'
-              ) : (
-                'Generate'
-              )}
+              {isGeneratingTestUrl ? 'Generating…' : testUrl ? 'Regenerate' : 'Generate'}
             </Button>
           </div>
           {testUrl ? (
-            <ShortInput
-              blockId={blockId}
-              subBlockId={`${subBlockId}-test-url`}
-              config={{
-                id: `${subBlockId}-test-url`,
-                type: 'short-input',
-                readOnly: true,
-                showCopyButton: true,
-              }}
-              value={testUrl}
-              readOnly={true}
-              showCopyButton={true}
-              disabled={isPreview || disabled}
-              isPreview={isPreview}
-            />
+            <>
+              <ShortInput
+                blockId={blockId}
+                subBlockId={`${subBlockId}-test-url`}
+                config={{
+                  id: `${subBlockId}-test-url`,
+                  type: 'short-input',
+                  readOnly: true,
+                  showCopyButton: true,
+                }}
+                value={testUrl}
+                readOnly={true}
+                showCopyButton={true}
+                disabled={isPreview || disabled}
+                isPreview={isPreview}
+              />
+              {testUrlExpiresAt && (
+                <p className='text-[12px] text-[var(--text-tertiary)]'>
+                  Expires {new Date(testUrlExpiresAt).toLocaleString()}
+                </p>
+              )}
+            </>
           ) : (
-            <p className='text-muted-foreground text-xs'>
-              Generate a temporary URL that executes this webhook against the live (undeployed)
-              workflow state.
-            </p>
-          )}
-          {testUrlExpiresAt && (
-            <p className='text-muted-foreground text-xs'>
-              Expires at {new Date(testUrlExpiresAt).toLocaleString()}
+            <p className='text-[12px] text-[var(--text-tertiary)]'>
+              Generate a temporary URL to test against the live (undeployed) workflow state.
             </p>
           )}
         </div>

apps/sim/background/schedule-execution.ts

@@ -30,7 +30,7 @@ import { mergeSubblockState } from '@/stores/workflows/server-utils'
 
 const logger = createLogger('TriggerScheduleExecution')
 
-const MAX_CONSECUTIVE_FAILURES = 10
+const MAX_CONSECUTIVE_FAILURES = 100
 
 type WorkflowRecord = typeof workflow.$inferSelect
 type WorkflowScheduleUpdate = Partial<typeof workflowSchedule.$inferInsert>

apps/sim/lib/webhooks/gmail-polling-service.ts

@@ -11,7 +11,7 @@ import { downloadAttachments, extractAttachmentInfo } from '@/tools/gmail/utils'
 
 const logger = createLogger('GmailPollingService')
 
-const MAX_CONSECUTIVE_FAILURES = 10
+const MAX_CONSECUTIVE_FAILURES = 100
 
 interface GmailWebhookConfig {
   labelIds: string[]

apps/sim/lib/webhooks/outlook-polling-service.ts

@@ -10,7 +10,7 @@ import { getOAuthToken, refreshAccessTokenIfNeeded } from '@/app/api/auth/oauth/
 
 const logger = createLogger('OutlookPollingService')
 
-const MAX_CONSECUTIVE_FAILURES = 10
+const MAX_CONSECUTIVE_FAILURES = 100
 
 async function markWebhookFailed(webhookId: string) {
   try {

apps/sim/lib/webhooks/rss-polling-service.ts

@@ -10,7 +10,7 @@ import { createLogger } from '@/lib/logs/console/logger'
 
 const logger = createLogger('RssPollingService')
 
-const MAX_CONSECUTIVE_FAILURES = 10
+const MAX_CONSECUTIVE_FAILURES = 100
 const MAX_GUIDS_TO_TRACK = 100 // Track recent guids to prevent duplicates
 
 interface RssWebhookConfig {