feat(amplifier)!: amplifier gateway v2

[milapsheth]

AXE-3660

• fix tests

Breaking events

• ContractCallApproved now includes the messageId instead of tx hash and event index separately
• SignersRotated event is used instead of OperatorshipTransferred to adhere to the updated naming convention

commandId derivation has changed. See messageToCommandId here. It's still guaranteed to be globally unique.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.14%. Comparing base (aa421a9) to head (a006f21).

❗ Current head a006f21 differs from pull request most recent head 8d5160a. Consider uploading reports for the commit 8d5160a to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #154      +/-   ##
==========================================
+ Coverage   96.80%   98.14%   +1.33%     
==========================================
  Files          47       47              
  Lines         752      755       +3     
  Branches      158      156       -2     
==========================================
+ Hits          728      741      +13     
+ Misses          4        1       -3     
+ Partials       20       13       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Foivos]

I think one could pass rotate signers into approve messages and vice versa, which is potentially dangerous.

[milapsheth]

I think one could pass rotate signers into approve messages and vice versa, which is potentially dangerous.

Discussed offline, but adding here for completeness. The data hash being signed also depends on CommandType which isn't user controlled and avoids this issue.