fix(docker): increase Node heap size to prevent OOM crash on macOS

[danieljrc888]

Problem

On macOS (particularly Apple Silicon / M1/M2/M3), docker compose build fails on the frontend service with:

FATAL ERROR: Ineffective mark-compacts near heap limit
Allocation failed - JavaScript heap out of memory
ERROR: "build-only" exited with 134.

This happens because Docker Desktop on macOS enforces stricter default memory limits per container, and the Vite/Vue build exhausts Node's default heap (~1.5 GB).

Fix

Added ENV NODE_OPTIONS=--max-old-space-size=4096 in the builder stage of docker/Dockerfile.frontend, giving Node 4 GB of heap to complete the build successfully.

Test plan

• Run docker compose build frontend on macOS — should complete without OOM error
• Verify the built image starts correctly with docker compose up frontend
• Confirm no regression on Linux builds (CI)

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Improved build stability by increasing memory allocation during the build process.

[coderabbitai]

📝 Walkthrough

Walkthrough

A single environment variable is added to the Dockerfile frontend builder stage to increase Node.js memory allocation during the build process. The change specifies a 4096MB maximum heap size for Node.js via the NODE_OPTIONS environment variable, enabling larger builds without modification to the build steps themselves.

Changes

Cohort / File(s)	Summary
Docker Build Configuration docker/Dockerfile.frontend	Adds ENV NODE_OPTIONS=--max-old-space-size=4096 to the builder stage to increase Node.js heap memory limit during build compilation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A memory boost, oh what a delight,
Four gigs of space, Node builds just right!
The builder stage now runs so free,
No more crashes—just harmony! 🚀

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title clearly and specifically summarizes the main change: adding NODE_OPTIONS to increase heap size for preventing OOM crashes on macOS. It directly reflects the primary fix in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The pull request description is comprehensive and well-structured, addressing the problem, fix, and testing approach clearly.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/frontend-docker-heap-oom

📝 Coding Plan

• Generate coding plan for human review comments

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

🎉 This PR is included in version 0.110.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀