search

valory-xyz / open-autonomy

A framework for the creation of autonomous agent services.
https://docs.autonolas.network
Apache License 2.0
79 stars 25 forks source link

`chain-id` should be a property #2211

Open Adamantios opened 5 months ago

Adamantios commented 5 months ago

The chain-id should be a property in the transaction settlement skill:

https://github.com/valory-xyz/open-autonomy/blob/v0.14.10/packages/valory/skills/transaction_settlement_abci/rounds.py#L248-L250

If the default chain id needs to be used as a fallback, then we can utilize the default chain ID from the params.

The skills that utilize the tx settlement skill should use this property instead of redefining it such as here for example:

https://github.com/valory-xyz/IEKit/blob/v0.9.2/packages/valory/skills/mech_interact_abci/states/base.py#L129-L132