Skip to content

Conversation

@pathoam
Copy link
Contributor

@pathoam pathoam commented Nov 5, 2025

Noticed that when RUST_LOG=trace the logs flood stdout and create parsing errors. This adapts the script to work with trace logs:

  • Harden packages/contracts/script/anvil-base-node.ts:20 by parsing cast stdout defensively (strip ANSI/log noise, retry up to three times) before extracting base fee, gas price, and
    fork block numbers.
  • Sanitize packages/contracts/script/anvil-token-paymaster-deposit.ts:17 so the deposit amount survives verbose cast to-wei output, preventing vm.envUint parse failures.

Also found some lint errors got into the contracts so fixed those

@pathoam pathoam requested a review from 0xBigBoss as a code owner November 5, 2025 21:19
Copy link
Contributor

Choose a reason for hiding this comment

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

oh this is cool. is this new?

@pathoam
Copy link
Contributor Author

pathoam commented Nov 7, 2025

ok I stripped out the lint stuff related to the forge version. The remaining parts of this stabilize anvil so it will run with rust trace logs on instead of crashing

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

Vercel Unique URL: https://sendapp-17sofinyr-0xsend.vercel.app
Vercel Preview URL: sendapp-anvil-log-fix-0xsend.vercel.app
Last Commit: 173a02c

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