remove unnecessary helpers, cleanup

Author: waleedlatif1

apps/sim/blocks/blocks/api.ts

@@ -93,9 +93,9 @@ Example:
       id: 'retries',
       title: 'Retries',
       type: 'short-input',
-      placeholder: '2',
+      placeholder: '0',
       description:
-        'Number of retry attempts for timeouts, 429 responses, and 5xx errors (default: 2 for GET/PUT/DELETE/HEAD)',
+        'Number of retry attempts for timeouts, 429 responses, and 5xx errors (default: 0, no retries)',
       mode: 'advanced',
     },
     {

apps/sim/tools/http/request.ts

@@ -55,23 +55,23 @@ export const requestTool: ToolConfig<RequestParams, RequestResponse> = {
     },
     retries: {
       type: 'number',
-      visibility: 'user-only',
+      visibility: 'hidden',
       description:
         'Number of retry attempts for retryable failures (timeouts, 429, 5xx). Defaults to 2 for idempotent methods (GET/PUT/DELETE/HEAD) and 0 otherwise.',
     },
     retryDelayMs: {
       type: 'number',
-      visibility: 'user-only',
+      visibility: 'hidden',
       description: 'Initial retry delay in milliseconds (default: 500)',
     },
     retryMaxDelayMs: {
       type: 'number',
-      visibility: 'user-only',
+      visibility: 'hidden',
       description: 'Maximum delay between retries in milliseconds (default: 30000)',
     },
     retryNonIdempotent: {
       type: 'boolean',
-      visibility: 'user-only',
+      visibility: 'hidden',
       description:
         'Allow retries for non-idempotent methods like POST/PATCH (may create duplicate requests).',
     },
@@ -144,22 +144,10 @@ export const requestTool: ToolConfig<RequestParams, RequestResponse> = {
 
     retry: {
       enabled: true,
-      maxRetries: 2,
-      maxRetriesLimit: 10,
+      maxRetries: 0,
       initialDelayMs: 500,
       maxDelayMs: 30000,
-      retryOnStatusCodes: [429],
-      retryOnStatusRanges: [{ min: 500, max: 599 }],
-      retryOnTimeout: true,
-      retryOnNetworkError: true,
-      respectRetryAfter: true,
-      retryableMethods: ['GET', 'HEAD', 'PUT', 'DELETE'],
-      paramOverrides: {
-        retries: 'retries',
-        initialDelayMs: 'retryDelayMs',
-        maxDelayMs: 'retryMaxDelayMs',
-        nonIdempotent: 'retryNonIdempotent',
-      },
+      retryIdempotentOnly: true,
     },
   },
 

apps/sim/tools/index.test.ts

@@ -1000,6 +1000,21 @@ describe('MCP Tool Execution', () => {
       expect((result.output as any).status).toBe(200)
     })
 
+    it('does not retry when retries is not specified (default: 0)', async () => {
+      global.fetch = Object.assign(
+        vi.fn().mockResolvedValue(makeJsonResponse(500, { error: 'server error' })),
+        { preconnect: vi.fn() }
+      ) as typeof fetch
+
+      const result = await executeTool('http_request', {
+        url: '/api/test',
+        method: 'GET',
+      })
+
+      expect(global.fetch).toHaveBeenCalledTimes(1)
+      expect(result.success).toBe(false)
+    })
+
     it('stops retrying after max attempts for http_request', async () => {
       global.fetch = Object.assign(
         vi.fn().mockResolvedValue(makeJsonResponse(502, { error: 'bad gateway' })),