Skip to content

feat: deploy AdapterStore + Arbitrum_Adapter after OFT audit #1063

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Aug 14, 2025
Merged

feat: deploy AdapterStore + Arbitrum_Adapter after OFT audit #1063

merged 9 commits into from
Aug 14, 2025

Conversation

@grasphoper
Copy link
Contributor

@grasphoper grasphoper commented Aug 7, 2025 •
edited
Loading

Integration tests (performed with slightly modified version of the adapter to support receiving ETH to test from EOA rather than with delegatecalls):

@grasphoper
deploy adapterStore + arbitrumAdapter
1b2dce5 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
bump package
e6cfab9 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
undo tsconfig change
30ea851 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
bmzig
bmzig previously approved these changes Aug 7, 2025
View reviewed changes
Copy link
Contributor

@bmzig bmzig left a comment

Choose a reason for hiding this comment

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

Nice. Do you need to transfer ownership of the adapter store still?

@grasphoper
add bytecode verification script
a19ba2c 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
Copy link
Contributor Author

grasphoper commented Aug 7, 2025 •
edited
Loading

@bmzig transferred ownership here

Also added a script to check the deployed real on-chain bytecode matches the bytecode from the contracts/ repo here.

Used via yarn hardhat verify-bytecode --contract Arbitrum_Adapter --network mainnet

Confirmed both deployments via the script

@grasphoper grasphoper requested a review from bmzig August 7, 2025 23:03
@grasphoper
move verifyBytecode script to a hardhat task
9566298 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
nicholaspai
nicholaspai previously approved these changes Aug 8, 2025
View reviewed changes
@grasphoper
deploy spokepool
e731034 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
verify deployment tx inputs rather than runtime bytecode
79eec83 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
improve verifyBytecode script
b1dbccf 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
update adapter address
a60f596 
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper grasphoper requested a review from nicholaspai August 11, 2025 23:54
@grasphoper grasphoper merged commit f2415a2 into master Aug 14, 2025
10 checks passed
@grasphoper grasphoper deleted the if/deploy-arbitrum-adapter branch August 14, 2025 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nicholaspai nicholaspai nicholaspai approved these changes

@mrice32 mrice32 Awaiting requested review from mrice32 mrice32 is a code owner

@pxrl pxrl Awaiting requested review from pxrl pxrl is a code owner

@bmzig bmzig Awaiting requested review from bmzig bmzig is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@grasphoper @nicholaspai @bmzig