Skip to content

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Sep 6, 2025

Increase memory limit of frontend build process to 4GiB to fix out-of-memory build failures

@ntindle ntindle requested a review from a team as a code owner September 6, 2025 17:11
@ntindle ntindle requested review from Pwuts and majdyz and removed request for a team September 6, 2025 17:11
@github-project-automation github-project-automation bot moved this to 🆕 Needs initial review in AutoGPT development kanban Sep 6, 2025
Copy link

netlify bot commented Sep 6, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 4a1d9ec
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/68bc6e0a39a8240008cabc56

@github-actions github-actions bot added the platform/frontend AutoGPT Platform - Front end label Sep 6, 2025
Copy link

qodo-merge-pro bot commented Sep 6, 2025

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Environment Precedence

Setting NODE_OPTIONS inline may be overridden by an ENV earlier/later in the Dockerfile or ignored if the shell treats it as a separate command; confirm the inline assignment reliably applies to the pnpm process in this multi-stage build context and that no global ENV NODE_OPTIONS is needed for consistency.

RUN if [ "$NEXT_PUBLIC_PW_TEST" = "true" ]; then NEXT_PUBLIC_PW_TEST=true NODE_OPTIONS="--max-old-space-size=4096" pnpm build; else NODE_OPTIONS="--max-old-space-size=4096" pnpm build; fi
Build Reproducibility

The conditional sets NEXT_PUBLIC_PW_TEST only when the incoming value is "true" but always sets NODE_OPTIONS; verify that this does not change build outputs across CI and local builds unintentionally and that memory limit is appropriate for builder resources.

RUN if [ "$NEXT_PUBLIC_PW_TEST" = "true" ]; then NEXT_PUBLIC_PW_TEST=true NODE_OPTIONS="--max-old-space-size=4096" pnpm build; else NODE_OPTIONS="--max-old-space-size=4096" pnpm build; fi

@github-actions github-actions bot added the size/s label Sep 6, 2025
Copy link

deepsource-io bot commented Sep 6, 2025

Here's the code health analysis summary for commits 0325ec0..4a1d9ec. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Copy link

netlify bot commented Sep 6, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 4a1d9ec
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs/deploys/68bc6e0ac602bd0008db5cf7

@Pwuts Pwuts changed the title ci: Add NODE_OPTIONS for memory limit during build ci(frontend): Add NODE_OPTIONS for memory limit during build Sep 6, 2025
@Pwuts Pwuts changed the title ci(frontend): Add NODE_OPTIONS for memory limit during build build(frontend): Add NODE_OPTIONS for memory limit during build Sep 6, 2025
@Pwuts Pwuts changed the title build(frontend): Add NODE_OPTIONS for memory limit during build build(frontend): Increase build process memory limit to 4 GiB Sep 6, 2025
@ntindle ntindle enabled auto-merge September 7, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🆕 Needs initial review
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants