Skip to content
This repository was archived by the owner on Oct 21, 2026. It is now read-only.

fix: update deposit wallet beacon integration - #2

Merged
ibruno merged 2 commits into
mainfrom
fix/update-deposit-wallet-beacon
Jul 1, 2026
Merged

ibruno merged 2 commits into
mainfrom
fix/update-deposit-wallet-beacon

Conversation

@ibruno

@ibruno ibruno commented Jul 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary by cubic

Switches deposit wallet address derivation to ERC1967 Beacon proxy bytecode and configures DepositWalletBeacon, aligning CREATE2 addresses with on-chain deployments to prevent misrouted deposits. No runtime overhead change. Affects outputs of deriveDepositWallet and RelayClient.getDepositWalletAddress. Released as @kuestcom/builder-relayer-client v2.0.6.

  • Bug Fixes

    • Compute CREATE2 initCodeHash from ERC1967 Beacon proxy bytecode (new constants/prefix).
    • RelayClient.getDepositWalletAddress now uses config.DepositWalletBeacon; default factory/beacon/implementation addresses updated.
    • Tests updated to the new deterministic address.
  • Migration

    • Add DepositWalletBeacon to your contract config (validation now requires it).
    • Recompute any cached/persisted deposit wallet addresses; results have changed.
    • No public method signatures changed.

Written for commit 5533177. Summary will update on new commits.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 4 files

Re-trigger cubic

@ibruno
ibruno merged commit b74c17d into main Jul 1, 2026
2 checks passed
@ibruno
ibruno deleted the fix/update-deposit-wallet-beacon branch July 1, 2026 13:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant