Skip to content

fix: heap memory allocation issue#5728

Open
therealemjy wants to merge 1 commit into
mainfrom
ci/heap-allocation
Open

fix: heap memory allocation issue#5728
therealemjy wants to merge 1 commit into
mainfrom
ci/heap-allocation

Conversation

@therealemjy

@therealemjy therealemjy commented Jul 23, 2026

Copy link
Copy Markdown
Member

Changes

  • fix heap memory allocation issue
  • use slim versions of Venus contract packages as we only need addresses from them

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: de971a7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dapp-preview Ready Ready Preview Jul 23, 2026 9:12pm
dapp-testnet Ready Ready Preview Jul 23, 2026 9:12pm
venus.io Ready Ready Preview Jul 23, 2026 9:12pm

Request Review

@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown

Greptile Summary

This PR reduces contract-package footprint and increases release-build memory headroom.

  • Replaces Venus contract dependencies with slim package variants.
  • Sources BSC contract addresses from package deployment manifests instead of hardcoded constants.
  • Raises the Node.js heap limit, pins the release runner to Ubuntu 22.04, and adjusts npm setup.
  • Regenerates the dependency lockfile for the slim packages.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete changed-code failure identified.

The package-derived contract addresses preserve the previous BSC targets, and the release and dependency changes align with the intended lower-memory build and trusted-publishing paths.

Important Files Changed

Filename Overview
.github/workflows/cd.yaml Adds a 4 GB Node.js heap limit, pins Ubuntu 22.04, and removes setup-node registry configuration while retaining the npm trusted-publishing setup.
apps/evm/package.json Moves Venus contract dependencies to pinned slim releases to reduce installation and build memory usage.
apps/evm/src/libs/contracts/config/index.ts Replaces hardcoded BSC addresses with equivalent values from Venus protocol and periphery deployment manifests.
packages/chains/package.json Pins chain-generation dependencies to slim Venus contract packages.
yarn.lock Updates dependency resolution to the selected slim contract packages and removes their unused transitive dependency trees.

Reviews (1): Last reviewed commit: "ci: use slim versions of Venus contract ..." | Re-trigger Greptile

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for ./apps/evm

Status Category Percentage Covered / Total
🔵 Lines 83.21% 52596 / 63208
🔵 Statements 83.21% 52596 / 63208
🔵 Functions 63.57% 698 / 1098
🔵 Branches 72.94% 6073 / 8326
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/evm/src/libs/contracts/config/index.ts 0% 100% 100% 0% 3-954
Generated in workflow #13930 for commit de971a7 by the Vitest Coverage Report Action

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