coderabbitai[bot]
commented
1 week ago 📝 Walkthrough
📝 Walkthrough
📝 Walkthrough
## Walkthrough The pull request introduces several significant changes to the blockchain library, including the addition of a new dependency, enhancements to the `EvmClient` class for transaction broadcasting, and the introduction of new classes such as `WardenRegistryClient` and `WardenClient`. Additionally, various types and interfaces have been defined to improve the structure of the codebase, particularly for handling signature requests and transaction processing. The changes also include updates to configuration files to accommodate new parameters for EVMOS, along with enhancements to error handling and retry mechanisms. ## Changes | File | Change Summary | |----------------------------------------------------------------------|-------------------------------------------------------------------------------------------------------| | `automated-orders/packages/blockchain-library/package.json` | Added new dependency `"cockatiel": "^3.2.1"` to `dependencies`. | | `automated-orders/packages/blockchain-library/src/clients/evm.ts` | Updated `broadcastTx` method signature and implementation; added imports for `parseTransaction`, `parseSignature`, and `serializeTransaction`. | | `automated-orders/packages/blockchain-library/src/clients/registry.ts` | Introduced `WardenRegistryClient` class with `getTransaction` method. | | `automated-orders/packages/blockchain-library/src/clients/warden.ts` | Added `WardenClient` class with `pollSignatureRequests` method for handling signature requests. | | `automated-orders/packages/blockchain-library/src/index.ts` | Added new exports for various types and clients, including `WardenClient` and `WardenRegistryClient`. | | `automated-orders/packages/blockchain-library/src/processors/newSignatureProcessor.ts` | Updated `NewSignatureProcessor` to include retry and circuit breaker policies using `cockatiel`. | | `automated-orders/packages/blockchain-library/src/types/registry/configuration.ts` | Added new interface `IWardenRegistryConfiguration` with `contractAddress`. | | `automated-orders/packages/blockchain-library/src/types/registry/functions.ts` | Added new ABI function constants: `AddTransactionAbi`, `ExecutionsAbi`, `RegisterAbi`, `TransactionsAbi`. | | `automated-orders/packages/blockchain-library/src/types/warden/configuration.ts` | Modified `IWardenConfiguration` interface by removing `rpcURL` and adding new properties. | | `automated-orders/packages/blockchain-library/src/types/warden/functions.ts` | Added enums and interfaces for managing sign requests and broadcasting options. | | `automated-orders/packages/blockchain-library/src/types/warden/newSignatureRequest.ts` | Modified `INewSignatureRequest` interface to replace `signedData` with `signature` and `transactionHash`. | | `automated-orders/packages/relayer/src/config/config.ts` | Updated configuration schema with new required fields and properties for EVMOS. | | `automated-orders/packages/relayer/src/config/env.ts` | Modified `Env` interface to include new properties related to EVMOS. | | `automated-orders/packages/relayer/src/index.ts` | Enhanced `main` function for blockchain interactions, including instantiation of multiple clients. | | `automated-orders/packages/utils-library/src/index.ts` | Added new exports `empty` and `notEmpty` from `buffer.js`. | | `automated-orders/packages/utils-library/src/services/buffer.ts` | Introduced `notEmpty` and `empty` functions for utility in handling `Uint8Array`. | | `CHANGELOG.md` | Updated to include new feature `(relayer) #992 Added a service to relay transactions into Ethereum`. | | `precompiles/warden/query_types.go` | Modified error handling and input validation in request functions. | ## Possibly related PRs - **#840**: This PR involves the removal of the `template_id` fields, which may relate to the overall structure of requests and responses in the blockchain library, potentially impacting how dependencies are managed. - **#975**: The addition of the "slinky" precompile could be relevant as it may interact with the blockchain library's functionality, especially if it involves transaction handling or event processing. - **#1042**: The introduction of a new type field for signature requests enhances the functionality related to signature management, which may connect with the changes in the main PR regarding the addition of a new dependency. - **#1077**: This PR focuses on improving typings by utilizing enums consistently, which could relate to the overall type safety and structure of the blockchain library, aligning with the changes made in the main PR. - **#1078**: The transition to using `viem` and `wagmi` libraries in the blockchain library indicates a significant change in how blockchain interactions are handled, which is directly related to the updates made in the main PR regarding dependencies. ## Suggested labels `automated-uniswap`, `basic-automated-orders` ## Suggested reviewers - artur-abliazimov - mn13 - Svetomech - PitasiThank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
github-actions[bot]
vercel[bot]
Previous PR: https://github.com/warden-protocol/wardenprotocol/pull/1043 Closed due to too many differences after migration to viem.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation