Skip to content

Conversation

@mablr
Copy link
Contributor

@mablr mablr commented Dec 30, 2025

Motivation

Closes #12922

Solution

  • Quick fix handling specifically the browser wallet signer with new EitherSigner helper struct.
  • Bumped foundry-browser-wallet to v0.1.0 (fix the bug when sending txs without to)

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

Copy link
Contributor Author

@mablr mablr left a comment

Choose a reason for hiding this comment

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

It works fine in dev mode w/ foundry-rs/foundry-browser-wallet#15 (pnpm dev).

For example to deploy Counter contract:

../foundry/target/debug/forge script script/Counter.s.sol --rpc-url http://127.0.0.1:8545 --broadcast --browser --browser-disable-open --browser-development

@mablr mablr marked this pull request as ready for review January 16, 2026 18:18
Copy link
Contributor Author

@mablr mablr left a comment

Choose a reason for hiding this comment

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

Let's go ! Foundry browser wallet bumped, it's now ready for merge.

cc @zerosnacks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

feat(forge script): add browser wallet support to forge script

1 participant