feat(target_chains/ethereum): add WithdrawFee action and implement related functionality in governance payload

[cctdaniel]

Summary

Added support for the WithdrawFee governance action that allows withdrawing collected fees from Pyth contracts to a specified address. The changes include:

• New WithdrawFee action in governance payload system
• Added serialization/deserialization support in xc_admin_common
• Added Solidity implementation for fee withdrawal
• Added test coverage for the new action

Rationale

This feature is necessary to:

1. Enable fee collection from Pyth contracts
2. Provide a secure, governance-controlled mechanism to withdraw accumulated fees
3. Allow flexible fee distribution to specified addresses

How has this been tested?

• Current tests cover my changes

  • Added WithdrawFee case to existing governance payload serialization tests
  • Added WithdrawFee to property-based testing via governanceActionArb
  • Existing governance message verification tests cover the new action

• Added new tests

  • Added serialization/deserialization tests for WithdrawFee action
  • Added test coverage for hex string to Buffer conversion in governance payload tests

• Manually tested the code

  • Verified correct encoding/decoding of withdrawal amounts and target addresses
  • Confirmed proper integration with existing governance message flow
  • Validated error handling for insufficient fees

The changes are well-tested through:

1. Unit tests for TypeScript serialization
2. Integration with existing governance payload system
3. Property-based testing for round-trip serialization
4. Solidity contract tests for fee withdrawal functionality

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
component-library	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 15, 2025 4:44am
entropy-debugger	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 15, 2025 4:44am
proposals	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 15, 2025 4:44am

3 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
api-reference	⬜️ Ignored (Inspect)	Visit Preview		Apr 15, 2025 4:44am
insights	⬜️ Ignored (Inspect)	Visit Preview		Apr 15, 2025 4:44am
staking	⬜️ Ignored (Inspect)	Visit Preview		Apr 15, 2025 4:44am

[ali-bahjati]

This is very nice. I made a comment to change the schema to be consistent; will approve after that.