Skip to content

fix(runtime): el código de execute puede correr hasta 60 minutos - #22

Merged
fforres merged 1 commit into
skywardfrom
fforres/execute-timeout-60m
Oct 13, 2026
Merged

fforres merged 1 commit into
skywardfrom
fforres/execute-timeout-60m

Conversation

@fforres

@fforres fforres commented Oct 12, 2026

Copy link
Copy Markdown

Qué cambia

Un script que corre con execute ahora tiene 60 minutos antes de que se corte, en vez de 5. Así un script que espera un trabajo largo (consultar una ejecución en una máquina, una llamada lenta a otra API) termina en una sola llamada, sin que quien lo usa tenga que partirlo en pedazos de menos de 5 minutos.

Detalles

  • DEFAULT_TIMEOUT_MS pasa de 5 a 60 minutos en los cuatro runtimes: dynamic-worker (el que usa executor-cloudflare con LOADER), QuickJS (el respaldo sin LOADER), Deno y workerd. Quien pase un timeoutMs más corto lo sigue teniendo.
  • El watchdog del dynamic-worker espera el timeout más su margen de 30 s, sin cambios.
  • Changeset execute-one-hour.md. Los tests de los cuatro runtimes pasan (28, 76, 19 y 12).

Límites que quedan

  • Cloudflare corta por CPU, no por tiempo de pared: 30 s de CPU por request por defecto (limits.cpu_ms, hasta 5 min en el plan pago). Esperar I/O, como llamadas a herramientas o sleep, no cuenta, así que un script que mayormente espera puede correr los 60 minutos.
  • La pausa por aprobación (PAUSED_APPROVAL_TIMEOUT_MS) sigue en 4 minutos; es aparte.

https://claude.ai/code/session_018KRLKV5j8vbaAyH3UgPVtw

Raise DEFAULT_TIMEOUT_MS from five to sixty minutes in the dynamic-worker,
QuickJS, Deno and workerd runtimes, so a script that waits on a long job
finishes in one execute call. Callers can still pass a shorter timeoutMs.

Claude-Session: https://claude.ai/code/session_018KRLKV5j8vbaAyH3UgPVtw
@fforres
fforres force-pushed the fforres/execute-timeout-60m branch from 0c4ee71 to e682f46 Compare October 13, 2026 22:31
@fforres
fforres merged commit d8393e3 into skyward Oct 13, 2026
4 of 40 checks passed
@fforres
fforres deleted the fforres/execute-timeout-60m branch October 13, 2026 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant