Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@realZachi
Copy link
Contributor

Summary

The checks-macos test job fails with a JavaScript heap out of memory error:

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
[24520:0x128018000] 35210 ms: Scavenge 2021.4 (2076.8) -> 2019.1 (2078.1) MB

This increases the Node.js heap size to 4GB for the macOS test runner, matching the available memory on GitHub's macOS runners.

Test Plan

  • CI should pass on this PR (meta-test)

@nadesh00
Copy link

Looked into this — the fix in #1890 is straightforward: it sets NODE_OPTIONS=--max-old-space-size=4096 for the macOS test job, bumping the heap from ~2GB to 4GB. Linux and Windows pass fine, so this appears to be a macOS runner memory constraint rather than a code issue. The change is low-risk and should unblock flaky CI failures on PRs. 👍 from me on merging.

thewilloftheshadow added a commit that referenced this pull request Jan 26, 2026
@thewilloftheshadow
Copy link
Member

Squash-merged into main.

  • Squash commit: 9c8e8c5
  • Included: macOS checks NODE_OPTIONS bump + changelog entry (thanks)
  • Tests: pnpm lint && pnpm build && pnpm test (lint failed in existing src/line/* types)

@thewilloftheshadow
Copy link
Member

Closed after squash-merge to main (commit 9c8e8c5).

steipete pushed a commit to mertcicekci0/clawdbot that referenced this pull request Jan 26, 2026
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.

3 participants