Skip to content

fix(jseval): increase QuickJS memory limit from 32MB to 256MB#9860

Open
salarkhannn wants to merge 1 commit into
windmill-labs:mainfrom
salarkhannn:fix/quickjs-memory-limit-8073
Open

fix(jseval): increase QuickJS memory limit from 32MB to 256MB#9860
salarkhannn wants to merge 1 commit into
windmill-labs:mainfrom
salarkhannn:fix/quickjs-memory-limit-8073

Conversation

@salarkhannn

@salarkhannn salarkhannn commented Jul 1, 2026

Copy link
Copy Markdown

Flow expression evaluation uses QuickJS to evaluate step input transforms. The 32MB memory limit was too restrictive for payloads that construct large arrays — for example Array.from({length: 1e6}, ...).flat() exceeds 32MB and causes QuickJS to throw Exception generated by quickjs, surfacing as an opaque eval failure.

Increasing the limit to 256MB. QuickJS evaluations are always authenticated and run inside the worker per-job isolation, so this is safe.

Closes #8073.


Summary by cubic

Increase the QuickJS memory limit for flow expression evaluation from 32MB to 256MB to prevent opaque eval failures on large payloads (e.g., million-element arrays). Evaluations are authenticated and run in per‑job isolated workers, so this increase is safe.

Written for commit 5c5b6b9. Summary will update on new commits.

Review in cubic

Flow expression evaluation uses QuickJS to evaluate step input
transforms. The 32MB memory limit was too restrictive for payloads
that construct large arrays — e.g. Array.from({length: 1e6}, ...).flat()
exceeds 32MB and causes QuickJS to throw 'Exception generated by
quickjs' which surfaces as an opaque eval failure.

Increase the limit to 256MB. This is safe because QuickJS evaluations
are always authenticated, run inside the worker's per-job isolation,
and 256MB is well within typical server capacity.

Closes windmill-labs#8073.
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@salarkhannn

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Jul 1, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Re-trigger cubic

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.

bug: expression evaluation fails with large payloads in step input transforms (QuickJS)

1 participant