fix(tinyfish): reject a malformed numeric cap instead of dropping it

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ej1gsFPVzN5xZZ3QzvK8bD

Author: waleedlatif1

apps/sim/blocks/blocks/tinyfish.ts

@@ -501,14 +501,30 @@ Return ONLY the comma-separated URL list - no explanations, no extra text.`,
       params: (params) => {
         const result: Record<string, unknown> = {}
 
+        /**
+         * A numeric text input that does not parse is rejected rather than
+         * dropped. Omitting it silently hands the run TinyFish's default — no
+         * wall-clock limit at all for Max Duration — while the block still
+         * shows the value the user typed.
+         */
+        const toFiniteNumber = (value: string, field: string): number => {
+          const parsed = Number(value)
+          if (!Number.isFinite(parsed)) {
+            throw new Error(`Invalid numeric value for ${field}: ${value}`)
+          }
+          return parsed
+        }
+
         const maxSteps = String(params.maxSteps ?? '').trim()
-        if (maxSteps) result.maxSteps = Number(maxSteps)
+        if (maxSteps) result.maxSteps = toFiniteNumber(maxSteps, 'Max Steps')
 
         const maxDurationSeconds = String(params.maxDurationSeconds ?? '').trim()
-        if (maxDurationSeconds) result.maxDurationSeconds = Number(maxDurationSeconds)
+        if (maxDurationSeconds) {
+          result.maxDurationSeconds = toFiniteNumber(maxDurationSeconds, 'Max Duration (seconds)')
+        }
 
         const limit = String(params.limit ?? '').trim()
-        if (limit) result.limit = Number(limit)
+        if (limit) result.limit = toFiniteNumber(limit, 'Limit')
 
         /**
          * The list filter has its own sub-block id so it does not collide with
@@ -594,7 +610,7 @@ Return ONLY the comma-separated URL list - no explanations, no extra text.`,
     runs: {
       type: 'json',
       description:
-        'Runs matching the list filters [{runId, status, goal, createdAt, startedAt, finishedAt, numOfSteps, result, schemaValidation, error, streamingUrl, browserConfig}]',
+        'Runs matching the list filters [{runId, status, goal, createdAt, startedAt, finishedAt, numOfSteps, result, schemaValidation, error, streamingUrl, browserConfig, profileAttached, profileId, profileHint}]',
     },
     total: { type: 'number', description: 'Total runs matching the list filters' },
     nextCursor: { type: 'string', description: 'Cursor for the next page of runs' },
@@ -628,7 +644,7 @@ Return ONLY the comma-separated URL list - no explanations, no extra text.`,
     profiles: {
       type: 'json',
       description:
-        'Browser Context Profiles a run can start from [{profileId, name, proxyCountryCode, fingerprintSeed, createdAt, isDefault}]. Every field but profileId and name can be null when the API omits it',
+        'Browser Context Profiles a run can start from [{profileId, name, proxyCountryCode, fingerprintSeed, domainCount, createdAt, updatedAt, isDefault}]. Every field but profileId and name can be null when the API omits it',
     },
   },
 }

apps/sim/tools/tinyfish/tinyfish.test.ts

@@ -820,6 +820,31 @@ describe('TinyFish block', () => {
     expect(params).not.toHaveProperty('goal')
   })
 
+  it('rejects a numeric text input that does not parse instead of dropping the cap', () => {
+    expect(() =>
+      TinyFishBlock.tools.config?.params?.({
+        operation: 'tinyfish_run',
+        maxDurationSeconds: '5 minutes',
+      })
+    ).toThrow('Invalid numeric value for Max Duration (seconds): 5 minutes')
+
+    expect(() =>
+      TinyFishBlock.tools.config?.params?.({ operation: 'tinyfish_run', maxSteps: 'lots' })
+    ).toThrow('Invalid numeric value for Max Steps: lots')
+  })
+
+  it('coerces the duration cap and drops it when blank', () => {
+    expect(
+      TinyFishBlock.tools.config?.params?.({
+        operation: 'tinyfish_run',
+        maxDurationSeconds: '300',
+      })
+    ).toMatchObject({ maxDurationSeconds: 300 })
+    expect(
+      TinyFishBlock.tools.config?.params?.({ operation: 'tinyfish_run', maxDurationSeconds: '  ' })
+    ).not.toHaveProperty('maxDurationSeconds')
+  })
+
   it('coerces the numeric text inputs and drops them when blank', () => {
     expect(
       TinyFishBlock.tools.config?.params?.({ operation: 'tinyfish_run', maxSteps: '50' })