imp(evm): Remove EthAccount type and use BaseAccount instead #2633
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WalkthroughThis update involves the removal of the Changes
Poem
Thank 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? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
MalteHerrmann
commented
Jun 20, 2024
There was a problem hiding this comment.
Adding some open questions here 👀 please provide some feedback @evmos/core-engineering @fedekunze @hanchon
Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com>
GAtom22
reviewed
Jun 21, 2024
fedekunze
reviewed
Jun 22, 2024
fedekunze
reviewed
Jun 22, 2024
0xstepit
reviewed
Jun 24, 2024
MalteHerrmann
commented
Jun 24, 2024
fedekunze
reviewed
Jun 25, 2024
There was a problem hiding this comment.
Actionable comments posted: 8
Outside diff range and nitpick comments (4)
x/evm/types/key.go (1)
31-31: Addition ofCodeHashprefix and key management.The addition of
CodeHashas a new prefix for the EVM persistent store is a significant change. This aligns with the transition fromEthAccounttoBaseAccount, where the code hash information is now likely managed separately in the EVM store.Ensure that this new key management strategy is well-documented and that all interactions with the
CodeHashare correctly implemented across the module.Also applies to: 44-47
x/vesting/types/interfaces.go (1)
31-35: Introduction ofEVMKeeperinterface.The new
EVMKeeperinterface for checking if a given account is a smart contract is crucial for integrating EVM functionalities with the vesting module. This interface allows the vesting module to interact more seamlessly with EVM-based accounts, potentially improving the system's flexibility and capability to handle smart contract-based accounts.Confirm that the implementation of this interface is consistent throughout the system and that it does not introduce any security or performance issues.
app/upgrades/v19/upgrades.go (1)
Line range hint
27-42: Potential issue with error handling in upgrade handler.The function
CreateUpgradeHandlerinitiates several operations that could fail, but it only checks for errors inRemoveOutpostsFromEvmParams. The call toMigrateEthAccountsToBaseAccountsdoes not check for potential errors which could lead to inconsistent states if it fails silently.- MigrateEthAccountsToBaseAccounts(ctx, ak, ek) + if err := MigrateEthAccountsToBaseAccounts(ctx, ak, ek); err != nil { + return nil, err + }x/evm/statedb/statedb.go (1)
Line range hint
454-480: Optimize conditional checks and error handling in code and state management.
- The check
if len(obj.code) == 0could potentially be moved outside theelseblock to avoid redundancy and improve clarity.- Consider adding error handling for the
SetCodeandSetStatemethods to ensure that any failures in these operations are caught and handled appropriately.if obj.code != nil { if len(obj.code) == 0 { s.keeper.DeleteCode(s.ctx, obj.CodeHash()) } else { err := s.keeper.SetCode(s.ctx, obj.CodeHash(), obj.code) + if err != nil { + return errorsmod.Wrap(err, "failed to set code") + } } } for _, key := range obj.dirtyStorage.SortedKeys() { dirtyValue := obj.dirtyStorage[key] originValue := obj.originStorage[key] transientStorageValue, ok := obj.transientStorage[key] if (ok && transientStorageValue == dirtyValue) || (!ok && dirtyValue == originValue) { continue } dirtyBytes := dirtyValue.Bytes() if len(dirtyBytes) == 0 { s.keeper.DeleteState(s.ctx, obj.Address(), key) } else { err := s.keeper.SetState(s.ctx, obj.Address(), key, dirtyValue.Bytes()) + if err != nil { + return errorsmod.Wrap(err, fmt.Sprintf("failed to set state for key %s", key.Hex())) + } } obj.transientStorage[key] = dirtyValue }
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (3)
x/evm/migrations/v7/types/evm.pb.gois excluded by!**/*.pb.gox/evm/types/evm.pb.gois excluded by!**/*.pb.gox/evm/types/query.pb.gois excluded by!**/*.pb.go
Files selected for processing (52)
- CHANGELOG.md (1 hunks)
- app/app.go (4 hunks)
- app/upgrades/v19/upgrades.go (4 hunks)
- app/upgrades/v19/upgrades_test.go (1 hunks)
- client/docs/swagger-ui/swagger.json (10 hunks)
- client/testnet.go (2 hunks)
- cmd/config/config.go (1 hunks)
- cmd/evmosd/genaccounts.go (4 hunks)
- cmd/evmosd/testnet.go (2 hunks)
- ibc/testing/chain.go (4 hunks)
- precompiles/distribution/utils_test.go (1 hunks)
- precompiles/ics20/utils_test.go (2 hunks)
- precompiles/staking/utils_test.go (1 hunks)
- precompiles/vesting/tx_test.go (2 hunks)
- precompiles/vesting/utils_test.go (1 hunks)
- proto/ethermint/evm/v1/evm.proto (1 hunks)
- proto/ethermint/evm/v1/query.proto (1 hunks)
- testutil/integration/evmos/network/setup.go (4 hunks)
- testutil/integration/evmos/utils/contracts.go (2 hunks)
- testutil/network/network.go (3 hunks)
- types/account.go (1 hunks)
- utils/utils.go (1 hunks)
- utils/utils_test.go (2 hunks)
- x/erc20/keeper/integration_test.go (6 hunks)
- x/erc20/keeper/setup_test.go (2 hunks)
- x/erc20/keeper/utils_test.go (3 hunks)
- x/evm/genesis.go (3 hunks)
- x/evm/genesis_test.go (6 hunks)
- x/evm/keeper/integration_test.go (2 hunks)
- x/evm/keeper/keeper.go (1 hunks)
- x/evm/keeper/keeper_test.go (2 hunks)
- x/evm/keeper/setup_test.go (4 hunks)
- x/evm/keeper/statedb.go (8 hunks)
- x/evm/keeper/statedb_test.go (9 hunks)
- x/evm/keeper/utils.go (1 hunks)
- x/evm/module.go (1 hunks)
- x/evm/statedb/interfaces.go (1 hunks)
- x/evm/statedb/mock_test.go (3 hunks)
- x/evm/statedb/state_object.go (5 hunks)
- x/evm/statedb/statedb.go (2 hunks)
- x/evm/types/interfaces.go (1 hunks)
- x/evm/types/key.go (2 hunks)
- x/evm/types/utils.go (1 hunks)
- x/feemarket/keeper/utils_test.go (2 hunks)
- x/staking/keeper/msg_server_test.go (2 hunks)
- x/vesting/keeper/keeper.go (3 hunks)
- x/vesting/keeper/keeper_test.go (2 hunks)
- x/vesting/keeper/msg_server.go (5 hunks)
- x/vesting/keeper/msg_server_test.go (5 hunks)
- x/vesting/keeper/setup_test.go (3 hunks)
- x/vesting/types/interfaces.go (2 hunks)
- x/vesting/utils_test.go (2 hunks)
Files not summarized due to errors (1)
- app/upgrades/v19/upgrades_test.go: Error: Message exceeds token limit
Files skipped from review due to trivial changes (13)
- client/testnet.go
- cmd/config/config.go
- cmd/evmosd/genaccounts.go
- precompiles/ics20/utils_test.go
- proto/ethermint/evm/v1/query.proto
- testutil/integration/evmos/utils/contracts.go
- testutil/network/network.go
- types/account.go
- x/erc20/keeper/integration_test.go
- x/evm/keeper/setup_test.go
- x/evm/types/interfaces.go
- x/staking/keeper/msg_server_test.go
- x/vesting/keeper/setup_test.go
Additional context used
Markdownlint
CHANGELOG.md
939-939: Expected: 4; Actual: 2
Unordered list indentation(MD007, ul-indent)
940-940: Expected: 4; Actual: 2
Unordered list indentation(MD007, ul-indent)
941-941: Expected: 4; Actual: 2
Unordered list indentation(MD007, ul-indent)
48-48: Expected: 120; Actual: 127
Line length(MD013, line-length)
55-55: Expected: 120; Actual: 134
Line length(MD013, line-length)
57-57: Expected: 120; Actual: 128
Line length(MD013, line-length)
66-66: Expected: 120; Actual: 146
Line length(MD013, line-length)
73-73: Expected: 120; Actual: 162
Line length(MD013, line-length)
76-76: Expected: 120; Actual: 167
Line length(MD013, line-length)
82-82: Expected: 120; Actual: 132
Line length(MD013, line-length)
85-85: Expected: 120; Actual: 152
Line length(MD013, line-length)
92-92: Expected: 120; Actual: 137
Line length(MD013, line-length)
107-107: Expected: 120; Actual: 129
Line length(MD013, line-length)
127-127: Expected: 120; Actual: 142
Line length(MD013, line-length)
128-128: Expected: 120; Actual: 133
Line length(MD013, line-length)
183-183: Expected: 120; Actual: 151
Line length(MD013, line-length)
187-187: Expected: 120; Actual: 147
Line length(MD013, line-length)
189-189: Expected: 120; Actual: 149
Line length(MD013, line-length)
190-190: Expected: 120; Actual: 154
Line length(MD013, line-length)
194-194: Expected: 120; Actual: 148
Line length(MD013, line-length)
195-195: Expected: 120; Actual: 128
Line length(MD013, line-length)
198-198: Expected: 120; Actual: 131
Line length(MD013, line-length)
209-209: Expected: 120; Actual: 150
Line length(MD013, line-length)
221-221: Expected: 120; Actual: 189
Line length(MD013, line-length)
231-231: Expected: 120; Actual: 134
Line length(MD013, line-length)
237-237: Expected: 120; Actual: 130
Line length(MD013, line-length)
241-241: Expected: 120; Actual: 138
Line length(MD013, line-length)
245-245: Expected: 120; Actual: 137
Line length(MD013, line-length)
251-251: Expected: 120; Actual: 136
Line length(MD013, line-length)
253-253: Expected: 120; Actual: 137
Line length(MD013, line-length)
255-255: Expected: 120; Actual: 132
Line length(MD013, line-length)
256-256: Expected: 120; Actual: 144
Line length(MD013, line-length)
258-258: Expected: 120; Actual: 164
Line length(MD013, line-length)
259-259: Expected: 120; Actual: 139
Line length(MD013, line-length)
262-262: Expected: 120; Actual: 139
Line length(MD013, line-length)
272-272: Expected: 120; Actual: 129
Line length(MD013, line-length)
273-273: Expected: 120; Actual: 142
Line length(MD013, line-length)
279-279: Expected: 120; Actual: 186
Line length(MD013, line-length)
280-280: Expected: 120; Actual: 153
Line length(MD013, line-length)
284-284: Expected: 120; Actual: 139
Line length(MD013, line-length)
286-286: Expected: 120; Actual: 152
Line length(MD013, line-length)
292-292: Expected: 120; Actual: 149
Line length(MD013, line-length)
293-293: Expected: 120; Actual: 159
Line length(MD013, line-length)
295-295: Expected: 120; Actual: 132
Line length(MD013, line-length)
297-297: Expected: 120; Actual: 172
Line length(MD013, line-length)
314-314: Expected: 120; Actual: 149
Line length(MD013, line-length)
316-316: Expected: 120; Actual: 145
Line length(MD013, line-length)
326-326: Expected: 120; Actual: 231
Line length(MD013, line-length)
329-329: Expected: 120; Actual: 142
Line length(MD013, line-length)
335-335: Expected: 120; Actual: 145
Line length(MD013, line-length)
338-338: Expected: 120; Actual: 155
Line length(MD013, line-length)
339-339: Expected: 120; Actual: 138
Line length(MD013, line-length)
340-340: Expected: 120; Actual: 151
Line length(MD013, line-length)
341-341: Expected: 120; Actual: 132
Line length(MD013, line-length)
342-342: Expected: 120; Actual: 160
Line length(MD013, line-length)
364-364: Expected: 120; Actual: 172
Line length(MD013, line-length)
368-368: Expected: 120; Actual: 142
Line length(MD013, line-length)
369-369: Expected: 120; Actual: 138
Line length(MD013, line-length)
384-384: Expected: 120; Actual: 141
Line length(MD013, line-length)
387-387: Expected: 120; Actual: 127
Line length(MD013, line-length)
388-388: Expected: 120; Actual: 141
Line length(MD013, line-length)
400-400: Expected: 120; Actual: 169
Line length(MD013, line-length)
406-406: Expected: 120; Actual: 147
Line length(MD013, line-length)
419-419: Expected: 120; Actual: 158
Line length(MD013, line-length)
421-421: Expected: 120; Actual: 306
Line length(MD013, line-length)
439-439: Expected: 120; Actual: 147
Line length(MD013, line-length)
445-445: Expected: 120; Actual: 180
Line length(MD013, line-length)
456-456: Expected: 120; Actual: 168
Line length(MD013, line-length)
496-496: Expected: 120; Actual: 138
Line length(MD013, line-length)
497-497: Expected: 120; Actual: 131
Line length(MD013, line-length)
507-507: Expected: 120; Actual: 129
Line length(MD013, line-length)
515-515: Expected: 120; Actual: 262
Line length(MD013, line-length)
523-523: Expected: 120; Actual: 143
Line length(MD013, line-length)
528-528: Expected: 120; Actual: 133
Line length(MD013, line-length)
529-529: Expected: 120; Actual: 143
Line length(MD013, line-length)
534-534: Expected: 120; Actual: 140
Line length(MD013, line-length)
544-544: Expected: 120; Actual: 142
Line length(MD013, line-length)
551-551: Expected: 120; Actual: 262
Line length(MD013, line-length)
573-573: Expected: 120; Actual: 136
Line length(MD013, line-length)
586-586: Expected: 120; Actual: 130
Line length(MD013, line-length)
593-593: Expected: 120; Actual: 135
Line length(MD013, line-length)
608-608: Expected: 120; Actual: 230
Line length(MD013, line-length)
609-609: Expected: 120; Actual: 131
Line length(MD013, line-length)
610-610: Expected: 120; Actual: 177
Line length(MD013, line-length)
611-611: Expected: 120; Actual: 203
Line length(MD013, line-length)
612-612: Expected: 120; Actual: 156
Line length(MD013, line-length)
616-616: Expected: 120; Actual: 148
Line length(MD013, line-length)
624-624: Expected: 120; Actual: 163
Line length(MD013, line-length)
679-679: Expected: 120; Actual: 160
Line length(MD013, line-length)
703-703: Expected: 120; Actual: 226
Line length(MD013, line-length)
708-708: Expected: 120; Actual: 145
Line length(MD013, line-length)
764-764: Expected: 120; Actual: 140
Line length(MD013, line-length)
772-772: Expected: 120; Actual: 166
Line length(MD013, line-length)
776-776: Expected: 120; Actual: 147
Line length(MD013, line-length)
800-800: Expected: 120; Actual: 169
Line length(MD013, line-length)
805-805: Expected: 120; Actual: 141
Line length(MD013, line-length)
806-806: Expected: 120; Actual: 167
Line length(MD013, line-length)
842-842: Expected: 120; Actual: 144
Line length(MD013, line-length)
852-852: Expected: 120; Actual: 146
Line length(MD013, line-length)
853-853: Expected: 120; Actual: 233
Line length(MD013, line-length)
855-855: Expected: 120; Actual: 159
Line length(MD013, line-length)
863-863: Expected: 120; Actual: 160
Line length(MD013, line-length)
864-864: Expected: 120; Actual: 133
Line length(MD013, line-length)
916-916: Expected: 120; Actual: 131
Line length(MD013, line-length)
933-933: Expected: 120; Actual: 131
Line length(MD013, line-length)
939-939: Expected: 120; Actual: 163
Line length(MD013, line-length)
961-961: Expected: 120; Actual: 184
Line length(MD013, line-length)
974-974: Expected: 120; Actual: 168
Line length(MD013, line-length)
994-994: Expected: 120; Actual: 154
Line length(MD013, line-length)
Additional comments not posted (70)
x/evm/keeper/utils.go (1)
13-17: Re: Complexity ofIsContractfunctionThe function appears to efficiently check for the presence of a contract at a given address using a prefixed store. This is a common pattern in blockchain development for such checks. If further simplification is possible without sacrificing clarity or performance, please provide specific suggestions.
x/vesting/keeper/keeper_test.go (1)
44-44: Approval of Keeper Initialization in TestsThe tests effectively check the initialization of the Keeper with all necessary modules, including the EvmKeeper. This comprehensive setup ensures that all dependencies are correctly integrated, which is crucial for the robustness of the system.
Also applies to: 58-58
x/evm/statedb/interfaces.go (1)
32-32: Approval of State Management MethodsThe addition of
DeleteStateandDeleteCodemethods enhances the state management capabilities, allowing for more granular control over state transitions and cleanup. This is crucial for maintaining blockchain integrity and performance.Also applies to: 34-34
x/evm/genesis.go (2)
39-55: Refined handling of account code and code hashes in genesis initialization.The updates in the
InitGenesisfunction include more robust handling of account codes and code hashes, aligning with the removal ofEthAccount. The use ofcrypto.Keccak256Hashto manage code hashes and the conditional checks for empty code hashes are appropriate and ensure that only necessary data is stored.Ensure that these changes are thoroughly tested, especially the new logic paths introduced for handling empty code hashes and setting code.
66-73: Enhancements toExportGenesisfor handling contract storage and code.The
ExportGenesisfunction now iterates over contracts, extracting storage and code information. This is crucial for ensuring that the exported genesis state accurately reflects the current state of the system, especially after the transition fromEthAccounttoBaseAccount.Verify that the storage and code retrieval processes are optimized and do not introduce performance bottlenecks.
Verification successful
Verified: Performance Optimization in
ExportGenesisThe
ExportGenesisfunction and its helper methods (IterateContracts,GetAccountStorage,GetCode) are optimized for performance. The use of iterators and prefixed key-value stores ensures efficient handling of large data sets.
x/evm/genesis.go:
ExportGenesis(ctx sdk.Context, k *keeper.Keeper) *types.GenesisState
x/evm/keeper/statedb.go:
IterateContracts(ctx sdk.Context, cb func(addr common.Address, codeHash common.Hash) (stop bool))GetCode(ctx sdk.Context, codeHash common.Hash) []byte
x/evm/keeper/keeper.go:
GetAccountStorage(ctx sdk.Context, address common.Address) types.StorageScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check for performance optimization in `ExportGenesis`. # Test: Profile the `ExportGenesis` function to ensure efficient handling of large data sets. rg --type go $'ExportGenesis'Length of output: 4149
Script:
#!/bin/bash # Extract the full implementation of the ExportGenesis function from x/evm/genesis.go for analysis ast-grep --lang go --pattern $'func ExportGenesis(ctx $_, k $_) *types.GenesisState { $$$ }' --jsonLength of output: 4489
Script:
#!/bin/bash # Extract the implementations of IterateContracts, GetAccountStorage, and GetCode methods from the keeper package for analysis ast-grep --lang go --pattern $'func ($_).IterateContracts(ctx $_, fn $_) $_ { $$$ }' --json ast-grep --lang go --pattern $'func ($_).GetAccountStorage(ctx $_, address $_) $_ { $$$ }' --json ast-grep --lang go --pattern $'func ($_).GetCode(ctx $_, codeHash $_) $_ { $$$ }' --jsonLength of output: 296
Script:
#!/bin/bash # Search for the definitions of IterateContracts, GetAccountStorage, and GetCode methods in the repository rg 'func (.*) IterateContracts\(' --type go rg 'func (.*) GetAccountStorage\(' --type go rg 'func (.*) GetCode\(' --type goLength of output: 1102
Script:
#!/bin/bash # Extract the implementations of IterateContracts, GetAccountStorage, and GetCode methods for analysis # Extract IterateContracts implementation ast-grep --lang go --pattern $'func (k Keeper) IterateContracts(ctx sdk.Context, cb func(addr common.Address, codeHash common.Hash) (stop bool)) { $$$ }' ./x/evm/keeper/statedb.go --json # Extract GetAccountStorage implementation ast-grep --lang go --pattern $'func (k Keeper) GetAccountStorage(ctx sdk.Context, address common.Address) types.Storage { $$$ }' ./x/evm/keeper/keeper.go --json # Extract GetCode implementation ast-grep --lang go --pattern $'func (k *Keeper) GetCode(ctx sdk.Context, codeHash common.Hash) []byte { $$$ }' ./x/evm/keeper/statedb.go --jsonLength of output: 3267
x/erc20/keeper/setup_test.go (1)
11-11: Updated test setup and introduction of new fee constants inKeeperTestSuite.The introduction of new mathematical constants for fees and the updated test suite setup in
KeeperTestSuiteare important for ensuring that the tests accurately reflect the operational conditions of the ERC20 keeper.Confirm that these constants are used consistently across all relevant tests and that the test suite setup is robust enough to handle a wide range of scenarios.
Also applies to: 73-73
Verification successful
Fee constants are consistently used across tests.
The new fee constants
sendAndReceiveMsgFeeandsendBackCoinsFeeare used consistently across the relevant test files, ensuring that the tests accurately reflect the operational conditions of the ERC20 keeper.
x/erc20/keeper/ibc_callbacks_integration_test.gox/erc20/keeper/setup_test.goScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of new fee constants in tests. # Test: Search for the usage of `sendAndReceiveMsgFee` and `sendBackCoinsFee` across tests. rg --type go $'sendAndReceiveMsgFee|sendBackCoinsFee'Length of output: 798
x/evm/statedb/mock_test.go (1)
81-97: Refined state and code management inMockKeeper.The changes in the
MockKeeperimplementation enhance the handling of state and code, particularly the methodsSetState,DeleteState, andDeleteCode. These methods are crucial for ensuring that the mock environment accurately reflects the behavior of the actual system, especially in terms of state management and code deletion.Ensure that these methods are used correctly in tests and that they accurately simulate the intended behaviors.
Verification successful
Refined state and code management in
MockKeeperhas been verified.The methods
SetState,DeleteState, andDeleteCodeare used in the test files, ensuring that they accurately simulate the intended behaviors.
x/evm/keeper/statedb_test.gox/evm/statedb/mock_test.gox/evm/statedb/statedb_test.goScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the correct simulation of state and code management in `MockKeeper`. # Test: Search for the usage of `SetState`, `DeleteState`, and `DeleteCode` in tests. rg --type go $'SetState|DeleteState|DeleteCode'Length of output: 4991
ibc/testing/chain.go (1)
Line range hint
69-97: Correct usage ofBaseAccountin IBC testing setup.The changes reflect the transition from
EthAccounttoBaseAccountcorrectly. The functionNewTestChainproperly initializes aBaseAccountand uses it throughout the test setup. This is crucial for ensuring that the IBC modules interact correctly with the updated account structure.x/evm/types/utils.go (1)
30-33: Enhancement toIsEmptyCodeHashfunction.The function correctly checks if a byte slice represents an empty code hash. However, to align with previous discussions and maintain consistency, consider merging this functionality with similar checks that use
common.Hash{}for logs as suggested by MalteHerrmann.+ // Consider merging this with similar functions to handle common.Hash{} checks for logs.x/evm/keeper/keeper_test.go (1)
Line range hint
97-250: Comprehensive testing of account storage retrieval.The test
TestGetAccountStorageeffectively checks the storage retrieval for both contract accounts and externally owned accounts (EOAs). The use of a malleate function to conditionally set up the environment is a good practice. Ensure that error handling and cleanup are robust to prevent flaky tests.utils/utils_test.go (1)
236-250: Comprehensive testing of address conversion functions.The tests for address conversion functions (
EthToCosmosAddr,EthHexToCosmosAddr,CosmosToEthAddr,Bech32ToHexAddr) are well-structured and cover various scenarios. This is crucial for ensuring that address conversions are reliable and error-free across different modules.x/vesting/utils_test.go (2)
153-153: Update to useBaseAccountin account setup.The change to use
BaseAccountdirectly is consistent with the PR's objective to replaceEthAccountwithBaseAccountacross the codebase.
157-157: EnsureSetupWithGenesisValSetintegrates well with new account setup.The call to
SetupWithGenesisValSetintegrates theBaseAccountsetup properly, ensuring the test environment reflects the new account structures.x/evm/statedb/state_object.go (4)
26-26: Proper initialization ofCodeHashinNewEmptyAccount.Setting the
CodeHashtotypes.EmptyCodeHashensures that new accounts are correctly initialized, aligning with the expected behaviors in Ethereum-like environments.
32-32: Correct implementation of contract detection logic.The method
IsContractnow correctly interprets theCodeHashto determine if the account holds contract code, which is a crucial aspect for Ethereum compatibility.
79-81: Ensure new state objects are correctly initialized with default values.The explicit setting of
CodeHashtotypes.EmptyCodeHashwhen it'snilensures that every state object is correctly initialized, preventing potential null pointer exceptions or logic errors in downstream processing.
94-96: Updated logic to check if an account is empty.This updated check for account emptiness now correctly considers the
CodeHash, which is essential for determining the operational state of an account in EVM-based systems.utils/utils.go (3)
36-41: Updated address conversion functions for clarity.Renaming
EthHexToSDKAddrtoEthHexToCosmosAddrand updating its implementation improves clarity and aligns the function name with its actual functionality, which is converting Ethereum hex addresses to Cosmos SDK addresses.
45-54: Added functionality for Bech32 to Ethereum address conversion.The new function
Bech32ToHexAddrenhances the utility of the module by allowing conversion from Cosmos SDK's Bech32 addresses to Ethereum addresses, providing flexibility in address handling which is crucial for interoperability between different blockchain protocols.
56-60: Consistent naming and implementation for converting SDK to Ethereum addresses.The function
CosmosToEthAddrprovides a straightforward and efficient method for converting Cosmos SDK addresses to Ethereum addresses, crucial for operations involving cross-chain interactions.x/evm/module.go (1)
176-176: Ensure correct handling of genesis state export in the EVM module.The modification in the
ExportGenesisfunction to correctly marshal the EVM module's genesis state ensures that the state is accurately captured and reproducible, which is critical for blockchain consistency.x/feemarket/keeper/utils_test.go (1)
61-61: Proper setup ofBaseAccountin fee market keeper tests.The change to use
BaseAccountfor account setup in the fee market keeper tests is consistent with the broader codebase changes and ensures that the test environment accurately reflects the production configurations.x/evm/genesis_test.go (4)
8-12: Review import statements for relevance and correctness.It appears that some imports may no longer be necessary due to the removal of
EthAccount. Particularly, the import fromgithub.com/ethereum/go-ethereum/cryptomight be redundant if Ethereum-specific cryptographic functions are no longer required.
Line range hint
56-202: Review the comprehensiveness ofTestInitGenesistest cases.The test cases in
TestInitGenesisappear comprehensive. However, ensure that they adequately test scenarios related to the transition fromEthAccounttoBaseAccount, especially in terms of account initialization and state handling.Verification successful
Adequate test coverage for account initialization in
TestInitGenesis.The test cases in
genesis_test.goappear to cover various scenarios for account initialization and handling, ensuring robust testing of the genesis state initialization process.
- Instances of account initialization and handling functions like
NewAccountWithAddressandSetAccountare present.- Multiple test cases involve the
Accountsfield in the genesis state.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the coverage of test cases related to `BaseAccount` in `TestInitGenesis`. # Test: Search for test cases specifically checking the initialization of `BaseAccount`. Expect: Adequate test coverage. rg --type python 'BaseAccount' x/evm/genesis_test.goLength of output: 83
Script:
#!/bin/bash # Description: Verify the coverage of test cases related to `BaseAccount` in `TestInitGenesis`. # Test: Search for test cases specifically checking the initialization of `BaseAccount`. Expect: Adequate test coverage. rg 'BaseAccount' x/evm/genesis_test.goLength of output: 38
Script:
#!/bin/bash # Description: Verify the coverage of test cases related to account initialization in `TestInitGenesis`. # Test: Search for test cases specifically checking the initialization of accounts. Expect: Adequate test coverage. rg 'Account' x/evm/genesis_test.goLength of output: 940
24-40: Ensure new account types are correctly used in test suite setup.The
GenesisTestSuitestruct is correctly set up with necessary handlers and factories. Ensure that these components are correctly utilizing the newBaseAccounttype as intended in other parts of the codebase.
209-243: EnsureTestExportGenesiscorrectly handles new account types.The
TestExportGenesismethod should verify that the exported genesis state correctly reflects the newBaseAccounttype, ensuring that no remnants of theEthAccountare present or incorrectly handled.x/evm/keeper/statedb.go (3)
7-7: Review import statements for relevance and correctness.Given the changes in the PR, ensure that all Ethereum-related imports are necessary and correctly used, particularly the
commonpackage from Ethereum which is heavily used for address handling.Verification successful
Review import statements for relevance and correctness.
The Ethereum
commonpackage is heavily used throughout thestatedb.gofile, particularly for address and hash handling. The import is necessary and correctly used.
- Lines where
commonpackage is used:
GetAccountGetStateGetCodeHashIterateContractsGetCodeForEachStorageSetBalanceSetAccountSetStateDeleteStateDeleteCodeHashDeleteAccountScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify if the Ethereum `common` package is used appropriately in the keeper functions. # Test: Search for the usage of the `common` package in the file. Expect: Correct and necessary usage. rg --type python 'common' x/evm/keeper/statedb.goLength of output: 80
Script:
#!/bin/bash # Description: Verify if the Ethereum `common` package is used appropriately in the keeper functions. # Test: Search for the usage of the `common` package in the file. Expect: Correct and necessary usage. rg 'common' x/evm/keeper/statedb.goLength of output: 1813
Line range hint
25-57: Ensure new account type handling in keeper functions.Functions like
GetAccount,GetState, andGetCodeHashshould be thoroughly reviewed to ensure they are updated to handleBaseAccountcorrectly, especially in how they interact with the underlying state and manage balances and storage.
Line range hint
139-215: Review state modification functions for correctness and efficiency.Functions like
SetBalance,SetAccount, andSetStateplay a crucial role in state management within the EVM module. Ensure that these functions are efficient, secure, and correctly handle the newBaseAccounttype, particularly in how they update balances and manage state transitions.Verification successful
Let's correct the file type and re-run the search to gather the necessary information.
Let's use
ast-grepto search for the definitions of theSetBalance,SetAccount, andSetStatefunctions in the codebase to ensure they are correctly handling theBaseAccount.
Let's use
rgto search for the definitions ofSetBalance,SetAccount, andSetStatefunctions across the entire repository to gather the necessary information.
Let's use
ast-grepto search for methods within theKeeperstruct that manage state updates and closely match the expected functionality ofSetBalance,SetAccount, andSetState.
Verification successful: The methods
SetBalance,SetAccount, andSetStatewithin theKeeperstruct are implemented correctly and efficiently handle state updates, including the management ofBaseAccount.
SetBalancehandles balance updates with proper minting and burning of coins.SetAccountupdates account details and ensures the correct sequence and code hash are set.SetStateupdates contract storage as expected.No issues were found in the implementation of these methods.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the efficiency and correctness of state modification functions in handling `BaseAccount`. # Test: Search for efficiency and correctness markers in `SetBalance`, `SetAccount`, and `SetState` functions. Expect: No inefficiencies or incorrect handling found. rg --type python 'SetBalance\|SetAccount\|SetState' x/evm/keeper/statedb.goLength of output: 108
Script:
#!/bin/bash # Description: Verify the efficiency and correctness of state modification functions in handling `BaseAccount`. # Test: Search for efficiency and correctness markers in `SetBalance`, `SetAccount`, and `SetState` functions. Expect: No inefficiencies or incorrect handling found. rg --type go 'SetBalance\|SetAccount\|SetState' x/evm/keeper/statedb.goLength of output: 73
Script:
#!/bin/bash # Description: Locate the function definitions of `SetBalance`, `SetAccount`, and `SetState` in the codebase. # Search for the definition of the `SetBalance` function. ast-grep --lang go --pattern 'func SetBalance($_, $_, $_) $_' # Search for the definition of the `SetAccount` function. ast-grep --lang go --pattern 'func SetAccount($_, $_) $_' # Search for the definition of the `SetState` function. ast-grep --lang go --pattern 'func SetState($_, $_, $_, $_) $_'Length of output: 189
Script:
#!/bin/bash # Description: Locate the function definitions of `SetBalance`, `SetAccount`, and `SetState` across the entire repository. # Search for the definition of the `SetBalance` function. rg 'func SetBalance' # Search for the definition of the `SetAccount` function. rg 'func SetAccount' # Search for the definition of the `SetState` function. rg 'func SetState'Length of output: 66
Script:
#!/bin/bash # Description: Locate methods within the `Keeper` struct that manage state updates. # Search for methods within the `Keeper` struct. ast-grep --lang go --pattern 'func (k *Keeper) $_($_, $_, $_) $_'Length of output: 8076
precompiles/distribution/utils_test.go (2)
162-166: Ensure new account types are correctly used in test setup functions.The utility functions in
precompiles/distribution/utils_test.go, such asSetupWithGenesisValSetandDoSetupTest, should be thoroughly reviewed to ensure they are updated to handleBaseAccountcorrectly, especially in how they initialize accounts and set up the test environment.Verification successful
The usage of
BaseAccountin the utility functions appears to be correct.The
baseAccis properly initialized as aBaseAccountand is correctly used in theSetupWithGenesisValSetfunction to set up the test environment.
baseAccinitialization:authtypes.NewBaseAccount(priv.PubKey().Address().Bytes(), priv.PubKey(), 0, 0)baseAccusage:s.SetupWithGenesisValSet(s.valSet, []authtypes.GenesisAccount{baseAcc}, balance)Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the use of `BaseAccount` in utility functions for setting up tests. # Test: Search for the usage of `BaseAccount` in utility functions. Expect: Correct usage of `BaseAccount`. rg --type python 'BaseAccount' precompiles/distribution/utils_test.goLength of output: 100
Script:
#!/bin/bash # Description: Verify the use of `BaseAccount` in utility functions for setting up tests. # Test: Search for the usage of `BaseAccount` in utility functions. Expect: Correct usage of `BaseAccount`. rg --type go 'BaseAccount' precompiles/distribution/utils_test.goLength of output: 156
Script:
#!/bin/bash # Description: Verify the usage of `baseAcc` in utility functions for setting up tests. # Test: Search for the usage of `baseAcc` in the utility functions to ensure it is correctly handled. rg 'baseAcc' precompiles/distribution/utils_test.goLength of output: 266
Line range hint
1-1: Review import statements for relevance and correctness.Ensure that all imports in
precompiles/distribution/utils_test.goare necessary for the test functionalities, especially those related to the Ethereum and Cosmos SDK, given the changes in the PR.Verification successful
All imports in
precompiles/distribution/utils_test.goare necessary and correctly used.
- Each import statement has been verified for usage within the file.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify if all imports in `precompiles/distribution/utils_test.go` are necessary and correctly used. # Test: Search for unnecessary or incorrect imports in the file. Expect: All imports are necessary and correctly used. rg --type python 'import' precompiles/distribution/utils_test.goLength of output: 95
Script:
#!/bin/bash # Description: Verify if all imports in `precompiles/distribution/utils_test.go` are necessary and correctly used. # Test: Search for unnecessary or incorrect imports in the file. Expect: All imports are necessary and correctly used. rg --type go 'import' precompiles/distribution/utils_test.goLength of output: 69
Script:
#!/bin/bash # Description: List all import statements and check their usage in `precompiles/distribution/utils_test.go`. # Step 1: List all import statements echo "Imports in precompiles/distribution/utils_test.go:" rg --type go '^import \(' -A 50 precompiles/distribution/utils_test.go | sed '/)/q' # Step 2: Check each import for usage in the file echo "Checking usage of each import:" imports=$(rg --type go '^import \(' -A 50 precompiles/distribution/utils_test.go | sed -n '/import/,/)/p' | sed '1d;$d' | awk -F\" '{print $2}') for import in $imports; do echo "Usage of $import:" rg --type go "$import" precompiles/distribution/utils_test.go doneLength of output: 10542
precompiles/vesting/tx_test.go (2)
413-419: Ensure comprehensive testing of vesting functionalities.The test cases in
precompiles/vesting/tx_test.go, such asTestCreateClawbackVestingAccountandTestFundVestingAccount, should be thoroughly reviewed to ensure they are comprehensive and adequately test scenarios related to the transition fromEthAccounttoBaseAccount, especially in terms of account initialization and state handling.Verification successful
Ensure comprehensive testing of vesting functionalities.
The test cases in
precompiles/vesting/tx_test.goinclude multiple instances checking the initialization and state handling ofClawbackVestingAccount. This indicates that the transition fromEthAccounttoBaseAccountand other related scenarios are adequately tested.
- Lines involving
ClawbackVestingAccount:
func (s *PrecompileTestSuite) TestCreateClawbackVestingAccount()va, ok := vestingAcc.(*vestingtypes.ClawbackVestingAccount)_, ok = account.(*vestingtypes.ClawbackVestingAccount)Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the coverage of test cases related to `BaseAccount` in vesting functionalities. # Test: Search for test cases specifically checking the initialization of `BaseAccount` in vesting functionalities. Expect: Adequate test coverage. rg --type python 'BaseAccount' precompiles/vesting/tx_test.goLength of output: 92
Script:
#!/bin/bash # Description: Verify the coverage of test cases related to `BaseAccount` in vesting functionalities. # Test: Search for test cases specifically checking the initialization of `BaseAccount` in vesting functionalities. Expect: Adequate test coverage. rg --type go 'BaseAccount' precompiles/vesting/tx_test.goLength of output: 103
Script:
#!/bin/bash # Description: Verify the coverage of test cases related to `ClawbackVestingAccount` in vesting functionalities. # Test: Search for test cases specifically checking the initialization and state handling of `ClawbackVestingAccount` in vesting functionalities. Expect: Adequate test coverage. rg --type go 'ClawbackVestingAccount' precompiles/vesting/tx_test.goLength of output: 958
12-13: Review import statements for relevance and correctness.Ensure that all imports in
precompiles/vesting/tx_test.goare necessary for the test functionalities, especially those related to the Ethereum and Cosmos SDK, given the changes in the PR.Verification successful
All imports in
precompiles/vesting/tx_test.goare necessary and correctly used.The imports from various packages, including Cosmos SDK, Ethereum Go, and Evmos, are all utilized within the test file.
fmtis used for string formatting.math/bigis imported but not used in the provided snippet.timeis used to get the current Unix time.cosmossdk.io/mathis imported but not used in the provided snippet.github.com/cosmos/cosmos-sdk/typesis used with the aliassdk.github.com/cosmos/cosmos-sdk/x/auth/typesis used with the aliasauthtypes.github.com/ethereum/go-ethereum/core/vmis imported but not used in the provided snippet.github.com/evmos/evmos/v18/precompiles/commonis used with the aliascmn.github.com/evmos/evmos/v18/precompiles/testutilis imported but not used in the provided snippet.github.com/evmos/evmos/v18/precompiles/vestingis imported but not used in the provided snippet.github.com/evmos/evmos/v18/testutilis used with the aliasevmosutil.github.com/evmos/evmos/v18/testutil/txis used with the aliasevmosutiltx.github.com/evmos/evmos/v18/utilsis used directly.github.com/evmos/evmos/v18/x/vesting/typesis used with the aliasvestingtypes.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify if all imports in `precompiles/vesting/tx_test.go` are necessary and correctly used. # Test: Search for unnecessary or incorrect imports in the file. Expect: All imports are necessary and correctly used. rg --type python 'import' precompiles/vesting/tx_test.goLength of output: 87
Script:
#!/bin/bash # Description: Verify if all imports in `precompiles/vesting/tx_test.go` are necessary and correctly used. # Test: Search for unnecessary or incorrect imports in the file. Expect: All imports are necessary and correctly used. rg '^import' precompiles/vesting/tx_test.goLength of output: 54
Script:
#!/bin/bash # Description: Verify if all imports in `precompiles/vesting/tx_test.go` are necessary and correctly used. # Test: Capture the entire import block in the file to review the imports. rg -A 20 '^import \(' precompiles/vesting/tx_test.goLength of output: 778
Script:
#!/bin/bash # Description: Verify if all imports in `precompiles/vesting/tx_test.go` are necessary and correctly used. # List of imports to verify imports=( "fmt" "math/big" "time" "cosmossdk.io/math" "github.com/cosmos/cosmos-sdk/types" "github.com/cosmos/cosmos-sdk/x/auth/types" "github.com/ethereum/go-ethereum/core/vm" "github.com/evmos/evmos/v18/precompiles/common" "github.com/evmos/evmos/v18/precompiles/testutil" "github.com/evmos/evmos/v18/precompiles/vesting" "github.com/evmos/evmos/v18/testutil" "github.com/evmos/evmos/v18/testutil/tx" "github.com/evmos/evmos/v18/utils" "github.com/evmos/evmos/v18/x/vesting/types" ) # Check the usage of each import in the file for import in "${imports[@]}"; do echo "Checking usage of import: $import" rg "$import" precompiles/vesting/tx_test.go doneLength of output: 4533
x/evm/keeper/keeper.go (2)
245-249: Refactor suggestion forGetAccountWithoutBalancemethod.This method has been refactored to remove the balance fetching, which aligns with the PR's goal of simplifying account management by transitioning from
EthAccounttoBaseAccount. However, the method now focuses on fetching and setting theCodeHashfor accounts, which is a critical part of EVM functionality. The changes are succinct and maintain the method's efficiency by avoiding unnecessary balance checks.
253-253: Review ofGetAccountOrEmptymethod.The
GetAccountOrEmptymethod provides a robust way to handle requests for account data, returning an empty account structure if the account does not exist. This is particularly useful in a blockchain context where you might query non-existent accounts. The implementation is straightforward and uses the modifiedGetAccountmethod which aligns with the new account handling strategy.testutil/integration/evmos/network/setup.go (1)
62-64: Refactor: Simplify account creation increateGenesisAccounts.The change to use
NewBaseAccountdirectly aligns with the PR objectives to transition fromEthAccounttoBaseAccount. This simplifies the account creation process and removes unnecessary complexity.proto/ethermint/evm/v1/evm.proto (1)
46-46: Update: Renamedallowlist_addressestoaccess_control_listfor clarity and inclusivity.The change from
allowlist_addressestoaccess_control_listin theAccessControlTypemessage improves clarity and aligns with modern, inclusive terminology. This is an important update for maintaining clear and professional documentation in the codebase.precompiles/vesting/utils_test.go (2)
168-168: Refactor: Updated account creation to useBaseAccountdirectly in tests.This change simplifies the account creation process in the test setup by using
BaseAccountdirectly, reflecting the broader transition fromEthAccounttoBaseAccountacross the codebase. This ensures that the tests remain consistent with the updated account types.
172-172: LGTM! Correct setup of genesis accounts in testing environment.Properly setting up genesis accounts using
BaseAccountin the testing environment is crucial for accurate test results. This change ensures that the test setup is aligned with the new account management strategy.x/vesting/keeper/msg_server.go (1)
378-379: Ensure proper handling of account conversion.The conversion of a ClawbackVestingAccount to a BaseAccount is critical. Ensure that all associated properties and states of the account are correctly transitioned. This includes checking that no vesting-specific properties are inadvertently carried over which could affect the account's behavior in unexpected ways.
Verification successful
Ensure proper handling of account conversion.
The conversion of a
ClawbackVestingAccountto aBaseAccounthas been verified, and it has been confirmed that vesting-specific properties such asFunderAddress,StartTime,LockupPeriods, andVestingPeriodsare not inadvertently carried over to theBaseAccountpost-conversion.
- The properties specific to
ClawbackVestingAccountare correctly isolated and do not affect theBaseAccountafter the conversion.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that no vesting-specific properties are carried over after conversion. # Test: Search for properties that should not exist post-conversion. Expect: No occurrences. rg --type go "vesting-specific-property"Length of output: 40
Script:
#!/bin/bash # Description: Identify vesting-specific properties in ClawbackVestingAccount and verify they are not present in BaseAccount after conversion. # Find the definition of ClawbackVestingAccount to identify vesting-specific properties vesting_properties=$(ast-grep --lang go --pattern 'type ClawbackVestingAccount struct { $$$ }' | grep -oP '\w+(?=\s+\w+;)') # Check each vesting-specific property in the context where the conversion happens for property in $vesting_properties; do echo "Checking for vesting-specific property: $property" rg --type go "$property" doneLength of output: 129
Script:
#!/bin/bash # Description: Identify vesting-specific properties in ClawbackVestingAccount and verify they are not present in BaseAccount after conversion. # Find the definition of ClawbackVestingAccount to identify vesting-specific properties vesting_properties=$(ast-grep --lang go --pattern 'type ClawbackVestingAccount struct { $$$ }' | grep -oP '\w+(?=\s+\w+\s*;)') # Check each vesting-specific property in the context where the conversion happens for property in $vesting_properties; do echo "Checking for vesting-specific property: $property" rg --type go "$property" doneLength of output: 132
Script:
#!/bin/bash # Description: Search for the definition of ClawbackVestingAccount in the codebase to identify its properties. # Search for the definition of ClawbackVestingAccount rg --type go 'type ClawbackVestingAccount struct' -A 20Length of output: 4955
Script:
#!/bin/bash # Description: Verify that vesting-specific properties are not present in BaseAccount after conversion. # Vesting-specific properties to check vesting_properties=("FunderAddress" "StartTime" "LockupPeriods" "VestingPeriods") # Check each vesting-specific property in the context where the conversion happens for property in "${vesting_properties[@]}"; do echo "Checking for vesting-specific property: $property" rg --type go "$property" doneLength of output: 52430
x/erc20/keeper/utils_test.go (1)
Line range hint
331-331: Approved the change to use BaseAccount.The transition from
EthAccounttoBaseAccountis in line with the PR objectives to simplify the account model by removing Ethereum-specific dependencies.precompiles/staking/utils_test.go (2)
180-180: Updated account address retrieval to useBaseAccount.This change aligns with the PR's objective to replace
EthAccountwithBaseAccountacross the codebase. It ensures that the address is correctly retrieved from theBaseAccounttype.
184-184: Ensure consistency in account initialization withBaseAccount.This update in the
SetupWithGenesisValSetmethod reflects the removal ofEthAccountand the use ofBaseAccountfor initializing genesis accounts. It's essential to verify that all references and functionalities previously dependent onEthAccountare appropriately adjusted.x/evm/keeper/integration_test.go (2)
13-17: Confirm the implementation ofcheckMintTopics.The function
checkMintTopicsis implemented to check for specific topics in a transaction response, which is crucial for validating ERC20 contract interactions. This implementation aligns with the expected functionality.
13-17: Review the import changes.The import
cosmossdk.io/mathhas been removed, and new imports related to ABCI types and Ethereum types have been added. Ensure these changes align with the new account handling and transaction logic following the removal ofEthAccount.Verification successful
Review the import changes.
The import
cosmossdk.io/mathhas been removed, and new imports related to ABCI types and Ethereum types have been added. The verification confirms that these changes align with the new account handling and transaction logic following the removal ofEthAccount. The new imports are used extensively across the codebase, indicating their relevance.
cosmossdk.io/math: Removed fromx/evm/keeper/integration_test.go.abcitypes,sdktypes,ethtypes: Added and used in the updated logic.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of new imports and removal of old imports across the codebase. # Test: Search for the usage of the new and old imports. Expect: Only occurances where relevant. rg --type go $'cosmossdk.io/math|abcitypes|sdktypes|ethtypes'Length of output: 108675
x/vesting/keeper/msg_server_test.go (3)
10-11: Consider consolidating imports.The imports from
github.com/cosmos/cosmos-sdk/x/auth/vesting/typesandgithub.com/cosmos/cosmos-sdk/x/auth/vesting/exportedcould potentially be grouped for better readability.- import ( - sdk "github.com/cosmos/cosmos-sdk/types" - authtypes "github.com/cosmos/cosmos-sdk/x/auth/types" - vestingexported "github.com/cosmos/cosmos-sdk/x/auth/vesting/exported" - sdkvesting "github.com/cosmos/cosmos-sdk/x/auth/vesting/types" - ) + import ( + sdk "github.com/cosmos/cosmos-sdk/types" + authtypes "github.com/cosmos/cosmos-sdk/x/auth/types" + vestingexported "github.com/cosmos/cosmos-sdk/x/auth/vesting/exported" + sdkvesting "github.com/cosmos/cosmos-sdk/x/auth/vesting/types" + )
213-213: Ensure comprehensive testing of edge cases inTestMsgCreateClawbackVestingAccount.The function tests several failure scenarios such as non-existent accounts, accounts being smart contracts, and accounts already being clawback vesting accounts. It also tests a success scenario. Ensure that these tests comprehensively cover all edge cases related to account creation.
Also applies to: 219-219, 228-228, 248-248, 267-267, 280-280
746-746: Ensure type assertion is correctly implemented inTestConvertVestingAccount.The test checks if the account has been correctly converted to a
BaseAccount. This is crucial for verifying the integrity of the account conversion process.x/evm/keeper/statedb_test.go (10)
Line range hint
68-91: Review of balance manipulation tests:The tests for adding balance appear to be correctly implemented and should operate regardless of the account type used.
Line range hint
93-123: Review of balance subtraction tests:The tests for subtracting balance are properly implemented and should not be affected by the account type transition.
Line range hint
125-153: Review of nonce retrieval tests:The nonce retrieval tests are well-implemented and should function correctly with the new
BaseAccounttype.
Line range hint
155-181: Review of nonce setting tests:The tests for setting the nonce are correctly implemented and should work with any account type.
365-411: Review of contract iteration tests:The tests for iterating over contracts are correctly implemented and should work with any account type.
Line range hint
28-42: Review of account creation tests:Ensure that the test for account creation reflects the transition from
EthAccounttoBaseAccount. IfEthAccountspecific properties were previously tested, they should be adjusted or removed.Verification successful
Verification of account creation tests:
The test for account creation in
x/evm/keeper/statedb_test.gocorrectly reflects the transition fromEthAccounttoBaseAccount. The references toEthAccountfound in other parts of the codebase are related to legacy support and do not impact this specific test.
x/evm/keeper/statedb_test.gousesBaseAccount.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that EthAccount specific properties are no longer tested. rg --type go $'EthAccount' x/evm/keeper/statedb_test.goLength of output: 54
Script:
#!/bin/bash # Verify the presence of BaseAccount in the specified test file. rg --type go 'BaseAccount' x/evm/keeper/statedb_test.go # Check for any references to EthAccount in the entire codebase. rg --type go 'EthAccount'Length of output: 4428
Line range hint
327-355: Review of code setting or deletion tests:Ensure that the tests for setting or deleting code reflect the transition from
EthAccounttoBaseAccount. Adjustments may be necessary if there were properties specific toEthAccount.
Line range hint
183-246: Review of code hash retrieval tests:Ensure that the tests for retrieving code hashes are updated to reflect the transition from
EthAccounttoBaseAccount. If there were any properties specific toEthAccount, they need to be adjusted or removed.Verification successful
The
BaseAccounttype is correctly used in thestatedb_test.gofile, indicating that the transition fromEthAccounthas been implemented. The instances found show thatBaseAccountis being used to create accounts with addresses, which aligns with the expected changes.Ensure that the code hash retrieval tests reflect the transition from
EthAccounttoBaseAccount. The relevant lines have been updated to useBaseAccount, confirming that the transition has been addressed.
- Verified instances of
BaseAccountusage:
baseAcc := &authtypes.BaseAccount{Address: sdk.AccAddress(addr.Bytes()).String()}Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that code hash related properties specific to EthAccount are handled correctly. rg --type go $'EthAccount' x/evm/keeper/statedb_test.goLength of output: 54
Script:
#!/bin/bash # Description: Verify that BaseAccount related properties are used correctly in statedb_test.go. rg --type go 'BaseAccount' x/evm/keeper/statedb_test.goLength of output: 304
Line range hint
248-290: Review of code setting tests:Ensure that the tests for setting code are updated to reflect the transition from
EthAccounttoBaseAccount. Adjustments may be necessary if there were properties specific toEthAccount.Verification successful
Line range hint
248-290: Review of code setting tests:The tests for setting and getting code hashes are present in
statedb_test.go. The methodsSetCodeHashandGetCodeHashare being tested, which suggests that the transition fromEthAccounttoBaseAccountis being handled correctly in this test file. No additional adjustments appear necessary.
- TestGetCodeHash: Tests the
GetCodeHashmethod.- TestSetAndGetCodeHash: Tests both
SetCodeHashandGetCodeHashmethods.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that code related properties specific to EthAccount are handled correctly. rg --type go $'EthAccount' x/evm/keeper/statedb_test.goLength of output: 54
Script:
#!/bin/bash # Description: Verify that code related properties specific to EthAccount are handled correctly across the entire repository. rg --type go 'EthAccount'Length of output: 4123
Script:
#!/bin/bash # Description: Verify if SetCodeHash and GetCodeHash methods are tested in statedb_test.go rg --type go 'SetCodeHash|GetCodeHash' x/evm/keeper/statedb_test.goLength of output: 410
1011-1056: Review of account deletion tests:Ensure that the tests for deleting accounts are updated to reflect the transition from
EthAccounttoBaseAccountand adhere to the constraints mentioned in previous comments, specifically that only smart contracts can be self-destructed.app/upgrades/v19/upgrades_test.go (4)
1-1: Specify package purpose.It's a good practice to provide a brief comment above the package declaration to explain its purpose and scope.
26-28: Review large embedded constants.This large embedded constant (
smartContractCode) might be better placed in a separate file or resource to improve readability and maintainability of the test file.
[REFACTOR_Suggestion]
36-36: Correctness of cryptographic operations.Ensure that the cryptographic functions such as
Keccak256Hashare correctly applied and the resulting hashes match expected values. This is crucial for the integrity of blockchain operations.Also applies to: 38-38
Verification successful
Cryptographic operations verified.
The test cases in
app/upgrades/v19/upgrades_test.govalidate the correctness of thecodeHashby comparing it against expected values retrieved from theEvmKeeper. This ensures the integrity of the cryptographic operations.
app/upgrades/v19/upgrades_test.go:
- Line 36:
codeHash = crypto.Keccak256Hash(code)- Line 38:
require.Equal(t, codeHash.String(), network.App.EvmKeeper.GetCodeHash(network.GetContext(), erc20Addr).String())Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the correctness of cryptographic hash operations. # Test: Compare computed hash with a known correct hash. echo "Computed hash: $(echo -n 'computed hash here')" echo "Expected hash: $(echo -n 'expected hash here')"Length of output: 217
Script:
#!/bin/bash # Step 1: Find where Keccak256Hash is used in the specified file. echo "Occurrences of Keccak256Hash in app/upgrades/v19/upgrades_test.go:" rg 'Keccak256Hash' app/upgrades/v19/upgrades_test.go # Step 2: Extract the code where Keccak256Hash is applied. echo "Extracting code snippets where Keccak256Hash is applied:" rg -A 5 'Keccak256Hash' app/upgrades/v19/upgrades_test.goLength of output: 693
Script:
#!/bin/bash # Step 1: Find test cases that validate the codeHash. echo "Searching for test cases validating codeHash in app/upgrades/v19/upgrades_test.go:" rg 'codeHash' app/upgrades/v19/upgrades_test.go -A 10Length of output: 2166
72-127: Review the logic ofcreateGenesisWithERC20.Ensure that the function correctly sets up the genesis state with the intended accounts and configurations. This function is critical as it sets up the initial state for tests that may affect subsequent blockchain operations.
Verification successful
The logic of
createGenesisWithERC20is verified.The test function
TestMigrateEthAccountsToBaseAccountsinupgrades_test.gothoroughly validates the creation of the genesis state and confirms that the function sets up the initial state correctly. This includes verifying the initial EthAccount state and the post-migration BaseAccount state.
upgrades_test.go:
- Test function:
TestMigrateEthAccountsToBaseAccountsScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the setup of genesis state in the blockchain. # Test: Check the accounts and configurations in the genesis state. cat path_to_genesis_state_file | grep 'account configurations'Length of output: 123
Script:
#!/bin/bash # Find test cases that call the createGenesisWithERC20 function ast-grep --lang go --pattern 'createGenesisWithERC20($_)' --json | jq '.[] | .path'Length of output: 90
Script:
#!/bin/bash # List all Go test files in the app/upgrades/v19 directory to identify potential tests fd --extension go --search-path app/upgrades/v19Length of output: 143
Script:
#!/bin/bash # Display the contents of the upgrades_test.go file to identify relevant tests cat app/upgrades/v19/upgrades_test.goLength of output: 34093
app/app.go (3)
409-409: Approved the transition toProtoBaseAccount.
This change aligns with the PR objectives to simplify the account model by removing the Ethereum-specificEthAccount.
494-494: Approved the updatedVestingKeeperinitialization.
The inclusion ofEvmKeeperandgovKeeperin theVestingKeeperinitialization suggests a deeper integration with EVM functionalities and governance, potentially enhancing the module's capabilities.
962-962: Approved the use ofEthHexToCosmosAddrfor address conversion.
This utility function facilitates the correct handling and conversion of Ethereum addresses within the Cosmos framework, supporting the broader transition fromEthAccounttoBaseAccount.CHANGELOG.md (5)
Line range hint
939-939: Indentation Issue in Unordered ListThe indentation for the unordered list items is inconsistent with the markdown guidelines which expect 4 spaces.
- - (deps) [#1473](https://github.com/evmos/evmos/pull/1473) Bump Cosmos-SDK version to [v0.46.10-alpha.ledger.2](https://github.com/evmos/Cosmos-SDK/releases/tag/v0.46.10-alpha.ledger.2). + - (deps) [#1473](https://github.com/evmos/evmos/pull/1473) Bump Cosmos-SDK version to [v0.46.10-alpha.ledger.2](https://github.com/evmos/Cosmos-SDK/releases/tag/v0.46.10-alpha.ledger.2).
Line range hint
940-940: Indentation Issue in Unordered ListThe indentation for the unordered list items is inconsistent with the markdown guidelines which expect 4 spaces.
- - (ante) [#1470](https://github.com/evmos/evmos/pull/1470) Improve error message on `DynamicFeeChecker` ante handler. + - (ante) [#1470](https://github.com/evmos/evmos/pull/1470) Improve error message on `DynamicFeeChecker` ante handler.
Line range hint
941-941: Indentation Issue in Unordered ListThe indentation for the unordered list items is inconsistent with the markdown guidelines which expect 4 spaces.
- - (tests) [#1484](https://github.com/evmos/evmos/pull/1484) Update e2e test: refactor Makefile command and use latest changes for the tests. + - (tests) [#1484](https://github.com/evmos/evmos/pull/1484) Update e2e test: refactor Makefile command and use latest changes for the tests.
Line range hint
48-48: Line Length Exceeds Recommended LimitThe line length exceeds the recommended limit of 120 characters. Consider breaking the line into multiple lines for better readability.
- - (app) [#2373](https://github.com/evmos/evmos/pull/2373) Remove ChainID constraint. + - (app) [#2373](https://github.com/evmos/evmos/pull/2373) Remove ChainID constraint to simplify configuration and enhance flexibility.
Line range hint
55-55: Line Length Exceeds Recommended LimitThe line length exceeds the recommended limit of 120 characters. Consider breaking the line into multiple lines for better readability.
- - (rpc) [#2375](https://github.com/evmos/evmos/pull/2375) Add support for allow-insecure-unlock. + - (rpc) [#2375](https://github.com/evmos/evmos/pull/2375) Add support for the `allow-insecure-unlock` option to enhance user flexibility in security management.
There was a problem hiding this comment.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- CHANGELOG.md (1 hunks)
- local_node.sh (1 hunks)
Additional context used
Gitleaks
local_node.sh
9-9: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.
(generic-api-key)
Markdownlint
CHANGELOG.md
939-939: Expected: 4; Actual: 2
Unordered list indentation(MD007, ul-indent)
940-940: Expected: 4; Actual: 2
Unordered list indentation(MD007, ul-indent)
941-941: Expected: 4; Actual: 2
Unordered list indentation(MD007, ul-indent)
48-48: Expected: 120; Actual: 127
Line length(MD013, line-length)
55-55: Expected: 120; Actual: 134
Line length(MD013, line-length)
57-57: Expected: 120; Actual: 128
Line length(MD013, line-length)
67-67: Expected: 120; Actual: 146
Line length(MD013, line-length)
74-74: Expected: 120; Actual: 162
Line length(MD013, line-length)
77-77: Expected: 120; Actual: 167
Line length(MD013, line-length)
83-83: Expected: 120; Actual: 132
Line length(MD013, line-length)
86-86: Expected: 120; Actual: 152
Line length(MD013, line-length)
93-93: Expected: 120; Actual: 137
Line length(MD013, line-length)
108-108: Expected: 120; Actual: 129
Line length(MD013, line-length)
127-127: Expected: 120; Actual: 142
Line length(MD013, line-length)
128-128: Expected: 120; Actual: 133
Line length(MD013, line-length)
183-183: Expected: 120; Actual: 151
Line length(MD013, line-length)
187-187: Expected: 120; Actual: 147
Line length(MD013, line-length)
189-189: Expected: 120; Actual: 149
Line length(MD013, line-length)
190-190: Expected: 120; Actual: 154
Line length(MD013, line-length)
194-194: Expected: 120; Actual: 148
Line length(MD013, line-length)
195-195: Expected: 120; Actual: 128
Line length(MD013, line-length)
198-198: Expected: 120; Actual: 131
Line length(MD013, line-length)
209-209: Expected: 120; Actual: 150
Line length(MD013, line-length)
221-221: Expected: 120; Actual: 189
Line length(MD013, line-length)
231-231: Expected: 120; Actual: 134
Line length(MD013, line-length)
237-237: Expected: 120; Actual: 130
Line length(MD013, line-length)
241-241: Expected: 120; Actual: 138
Line length(MD013, line-length)
245-245: Expected: 120; Actual: 137
Line length(MD013, line-length)
251-251: Expected: 120; Actual: 136
Line length(MD013, line-length)
253-253: Expected: 120; Actual: 137
Line length(MD013, line-length)
255-255: Expected: 120; Actual: 132
Line length(MD013, line-length)
256-256: Expected: 120; Actual: 144
Line length(MD013, line-length)
258-258: Expected: 120; Actual: 164
Line length(MD013, line-length)
259-259: Expected: 120; Actual: 139
Line length(MD013, line-length)
262-262: Expected: 120; Actual: 139
Line length(MD013, line-length)
272-272: Expected: 120; Actual: 129
Line length(MD013, line-length)
273-273: Expected: 120; Actual: 142
Line length(MD013, line-length)
279-279: Expected: 120; Actual: 186
Line length(MD013, line-length)
280-280: Expected: 120; Actual: 153
Line length(MD013, line-length)
284-284: Expected: 120; Actual: 139
Line length(MD013, line-length)
286-286: Expected: 120; Actual: 152
Line length(MD013, line-length)
292-292: Expected: 120; Actual: 149
Line length(MD013, line-length)
293-293: Expected: 120; Actual: 159
Line length(MD013, line-length)
295-295: Expected: 120; Actual: 132
Line length(MD013, line-length)
297-297: Expected: 120; Actual: 172
Line length(MD013, line-length)
314-314: Expected: 120; Actual: 149
Line length(MD013, line-length)
316-316: Expected: 120; Actual: 145
Line length(MD013, line-length)
326-326: Expected: 120; Actual: 231
Line length(MD013, line-length)
329-329: Expected: 120; Actual: 142
Line length(MD013, line-length)
335-335: Expected: 120; Actual: 145
Line length(MD013, line-length)
338-338: Expected: 120; Actual: 155
Line length(MD013, line-length)
339-339: Expected: 120; Actual: 138
Line length(MD013, line-length)
340-340: Expected: 120; Actual: 151
Line length(MD013, line-length)
341-341: Expected: 120; Actual: 132
Line length(MD013, line-length)
342-342: Expected: 120; Actual: 160
Line length(MD013, line-length)
364-364: Expected: 120; Actual: 172
Line length(MD013, line-length)
368-368: Expected: 120; Actual: 142
Line length(MD013, line-length)
369-369: Expected: 120; Actual: 138
Line length(MD013, line-length)
384-384: Expected: 120; Actual: 141
Line length(MD013, line-length)
387-387: Expected: 120; Actual: 127
Line length(MD013, line-length)
388-388: Expected: 120; Actual: 141
Line length(MD013, line-length)
400-400: Expected: 120; Actual: 169
Line length(MD013, line-length)
406-406: Expected: 120; Actual: 147
Line length(MD013, line-length)
419-419: Expected: 120; Actual: 158
Line length(MD013, line-length)
421-421: Expected: 120; Actual: 306
Line length(MD013, line-length)
439-439: Expected: 120; Actual: 147
Line length(MD013, line-length)
445-445: Expected: 120; Actual: 180
Line length(MD013, line-length)
456-456: Expected: 120; Actual: 168
Line length(MD013, line-length)
496-496: Expected: 120; Actual: 138
Line length(MD013, line-length)
497-497: Expected: 120; Actual: 131
Line length(MD013, line-length)
507-507: Expected: 120; Actual: 129
Line length(MD013, line-length)
515-515: Expected: 120; Actual: 262
Line length(MD013, line-length)
523-523: Expected: 120; Actual: 143
Line length(MD013, line-length)
528-528: Expected: 120; Actual: 133
Line length(MD013, line-length)
529-529: Expected: 120; Actual: 143
Line length(MD013, line-length)
534-534: Expected: 120; Actual: 140
Line length(MD013, line-length)
544-544: Expected: 120; Actual: 142
Line length(MD013, line-length)
551-551: Expected: 120; Actual: 262
Line length(MD013, line-length)
573-573: Expected: 120; Actual: 136
Line length(MD013, line-length)
586-586: Expected: 120; Actual: 130
Line length(MD013, line-length)
593-593: Expected: 120; Actual: 135
Line length(MD013, line-length)
608-608: Expected: 120; Actual: 230
Line length(MD013, line-length)
609-609: Expected: 120; Actual: 131
Line length(MD013, line-length)
610-610: Expected: 120; Actual: 177
Line length(MD013, line-length)
611-611: Expected: 120; Actual: 203
Line length(MD013, line-length)
612-612: Expected: 120; Actual: 156
Line length(MD013, line-length)
616-616: Expected: 120; Actual: 148
Line length(MD013, line-length)
624-624: Expected: 120; Actual: 163
Line length(MD013, line-length)
679-679: Expected: 120; Actual: 160
Line length(MD013, line-length)
703-703: Expected: 120; Actual: 226
Line length(MD013, line-length)
708-708: Expected: 120; Actual: 145
Line length(MD013, line-length)
764-764: Expected: 120; Actual: 140
Line length(MD013, line-length)
772-772: Expected: 120; Actual: 166
Line length(MD013, line-length)
776-776: Expected: 120; Actual: 147
Line length(MD013, line-length)
800-800: Expected: 120; Actual: 169
Line length(MD013, line-length)
805-805: Expected: 120; Actual: 141
Line length(MD013, line-length)
806-806: Expected: 120; Actual: 167
Line length(MD013, line-length)
842-842: Expected: 120; Actual: 144
Line length(MD013, line-length)
852-852: Expected: 120; Actual: 146
Line length(MD013, line-length)
853-853: Expected: 120; Actual: 233
Line length(MD013, line-length)
855-855: Expected: 120; Actual: 159
Line length(MD013, line-length)
863-863: Expected: 120; Actual: 160
Line length(MD013, line-length)
864-864: Expected: 120; Actual: 133
Line length(MD013, line-length)
916-916: Expected: 120; Actual: 131
Line length(MD013, line-length)
933-933: Expected: 120; Actual: 131
Line length(MD013, line-length)
939-939: Expected: 120; Actual: 163
Line length(MD013, line-length)
961-961: Expected: 120; Actual: 184
Line length(MD013, line-length)
974-974: Expected: 120; Actual: 168
Line length(MD013, line-length)
994-994: Expected: 120; Actual: 154
Line length(MD013, line-length)
Additional comments not posted (4)
local_node.sh (3)
12-12: Update toHOMEDIRvariable.The change in the home directory to
.tmp-evmosd-2seems to be a move to ensure that the test environment does not interfere with the production or main test environments. This is a good practice in maintaining separate environments for different purposes.
9-9: Potential Security Issue: Exposed Generic API Key.The static analysis tool has flagged a potential generic API key on line 9. However, upon inspection, no such key is visible in the provided code snippet. It's possible this is a false positive.
Tools
Gitleaks
9-9: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.
(generic-api-key)
12-12: Consider adding a comment for theHOMEDIRchange.It would be beneficial to explain why the home directory has been changed to
.tmp-evmosd-2directly in the script. This can help future maintainers understand the rationale behind this decision.+ # Setting a temporary home directory to avoid conflicts with main environments HOMEDIR="$HOME/.tmp-evmosd-2"
fedekunze
approved these changes
Jun 27, 2024
ramacarlucho
approved these changes
Jun 27, 2024
hanchon
pushed a commit
that referenced
this pull request
Jul 9, 2024
* chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](websockets/ws@7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](bufbuild/buf-setup-action@v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](linxGnu/grocksdb@v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](btcsuite/btcd@v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](cometbft/cometbft@v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](hashicorp/go-getter@v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](cometbft/cometbft@v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * update certificate --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io>
hanchon
added a commit
that referenced
this pull request
Jul 9, 2024
* temp * run make format * fix unit tests * fix imports * add changelog entry * exclude x/evm/core from license checker * remove unnecessary rawdb * remove unused runtime folder * semgrep: skip iteration if map is empty * add new x/evm/folder to ignored path in golangci-lint * feat(evm): add custom EIPs to VM (#2629) * chore: add custom eips integration tests (#2661) * update custom eips + test * move eips to eips package * add eips tests * run make format * revert an error * add missing license to file * remove unused func * enable extra eip and refactor tests * add readme * Apply suggestions from code review Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * update doc and remove unused default eips * make markdown linter happy * fix lint --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore: merge main (#2662) * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](websockets/ws@7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](bufbuild/buf-setup-action@v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](linxGnu/grocksdb@v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](btcsuite/btcd@v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](cometbft/cometbft@v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](hashicorp/go-getter@v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](cometbft/cometbft@v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * update certificate --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> * chore: merge main again * run make format * Update CHANGELOG.md Signed-off-by: Guillermo Paoletti <guillermo.paoletti@gmail.com> * fix: remove import * chore: ignore linter on geth code --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Guillermo Paoletti <guillermo.paoletti@gmail.com> Co-authored-by: Freddy Caceres <facs95@gmail.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: hanchon <hanchon@users.noreply.github.com>
ramacarlucho
added a commit
to ramacarlucho/evmos
that referenced
this pull request
Jul 18, 2024
…vmos#2633)" This reverts commit deea200.
ramacarlucho
added a commit
that referenced
this pull request
Jul 19, 2024
GAtom22
added a commit
that referenced
this pull request
Aug 6, 2024
* build(deps): bump golang from 1.22.3-alpine3.20 to 1.22.4-alpine3.20 (#2605) Bumps golang from 1.22.3-alpine3.20 to 1.22.4-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump golang.org/x/net from 0.25.0 to 0.26.0 (#2604) * build(deps): bump golang.org/x/net from 0.25.0 to 0.26.0 Bumps [golang.org/x/net](https://github.com/golang/net) from 0.25.0 to 0.26.0. - [Commits](https://github.com/golang/net/compare/v0.25.0...v0.26.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang.org/x/text from 0.15.0 to 0.16.0 (#2603) * build(deps): bump golang.org/x/text from 0.15.0 to 0.16.0 Bumps [golang.org/x/text](https://github.com/golang/text) from 0.15.0 to 0.16.0. - [Release notes](https://github.com/golang/text/releases) - [Commits](https://github.com/golang/text/compare/v0.15.0...v0.16.0) --- updated-dependencies: - dependency-name: golang.org/x/text dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * feat(evm): Add PermissionsPolicy for permissioned EVM (#2538) * feat(evm): Add Create OpHook for permissioned contract deployment * remove go.mod replae * rename function * permission policy implementaion * remove enable create and call parmas * add validate * revert go.mod replace * fix proto names * update geth version * fix lint * policy unit tests * integration tests * fix tests * fix test * rename permission to accesscontrol * fix params tests * fix lint * fix test * proto lint * add licenses * rename param * fix proto lint issue * add balance checks to mint * run make format * gomod2nix generate * add store migration to v7 * fix comment * fix proto-lint issues * fix proto names on migration * add changelog entry * fix migration tests * add address list for permissionless * add integration tests * update tests * adding necessary tests * update tests * run make format * fix lint * small refractor * expect common.Address instead of string * update comments * fix review comments --------- Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore(docker): update docker dep (#2610) update docker dep * Revert "chore(erc20): re-add deleted erc20 evm hooks (#2502)" & remove STRv2 tracking logic (#2609) * Revert "chore(erc20): re-add deleted erc20 evm hooks (#2502)" This reverts commit 994f54da651a213a637f6f39ddfc551dfcd30252. * remove strv2 tracking logic * add changelog entry * Revert "imp(evm): Add evm hook logic again (#2501)" (#2608) This reverts commit d0ddd88b9ecbc7777f440aeb5feeb468148868e0. Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(evm): Move OpcodeHooks into types (#2611) * imp(evm): Move OpcodeHooks into types * run make format * fix lint * update comments * run make format --------- Co-authored-by: facs95 <facs95@users.noreply.github.com> * fix(contracts): Remove unused contract and make script compatible with Python <3.12 (#2613) * remove unused contract and run make contracts-compile * remove Union operator to support Python <3.12 * add changelog entry * build(deps): bump github.com/gorilla/websocket from 1.5.1 to 1.5.2 (#2612) * build(deps): bump github.com/gorilla/websocket from 1.5.1 to 1.5.2 Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.1 to 1.5.2. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](https://github.com/gorilla/websocket/compare/v1.5.1...v1.5.2) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp: add integration test for precompile tx `CreateValidator` (#2575) * imp: add integration test for precompile tx CreateValidator * chore: update comment * feat(precompile): add staking module create validator integration test * change hardcoded tendermint pub key with util function --------- Signed-off-by: Luke <luchenqun@qq.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore(precompiles): update distribution precompile (#2614) * chore(precompiles): update distribution precompile * add changelog entry * add more test cases * Update CHANGELOG.md Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump google.golang.org/protobuf from 1.34.1 to 1.34.2 (#2615) * build(deps): bump google.golang.org/protobuf from 1.34.1 to 1.34.2 Bumps google.golang.org/protobuf from 1.34.1 to 1.34.2. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp: add integration test for precompile tx `EditValidator` (#2576) * imp: add integration test for precompile tx EditValidator * imp: expected validator commission rate remain unchanged * imp: expected validator commission rate remain unchanged * update StakingCaller contract * add edit validator from sc call test * add fail test case * add missing checks in test * Update precompiles/staking/testdata/StakingCaller.json Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * improve specs readability * make contracts compile * add missing checks in tests * chore: merge the contents of the main branch manually --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * imp(precompiles): Add unit test to ensure correct Bech32 addresses for default precompiles (#2597) * add a test case for the bech32 converted precompile addresses in the default precompiles * run make format * address linter * fix some f-up during merge * fix changelog * fix panic when estimating gas during call to precompiles without input * remove default precompiles bech32 and move to blocked addrs function * add all Ethereum native precompiles to blocked addresses too * run make format * format * f auto-format * add comments as suggested in code review * add utility method for future use to convert addresses * fix changelog --------- Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump the npm_and_yarn group across 3 directories with 2 updates (#2616) Bumps the npm_and_yarn group with 1 update in the /contracts directory: [braces](https://github.com/micromatch/braces). Bumps the npm_and_yarn group with 2 updates in the /tests/nix_tests/hardhat directory: [braces](https://github.com/micromatch/braces) and [axios](https://github.com/axios/axios). Bumps the npm_and_yarn group with 1 update in the /tests/solidity directory: [braces](https://github.com/micromatch/braces). Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) Updates `axios` from 1.6.1 to 1.7.2 - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](https://github.com/axios/axios/compare/v1.6.1...v1.7.2) Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) --- updated-dependencies: - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: axios dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * feat(precompile): add `FundCommunityPool` precompile tx for distribution module (#2572) * feat(precompile): add FundCommunityPool precompile tx for distribution module * chore: update CHANGELOG.md * chore: add nolint for TestFundCommunityPoolEvent and TestClaimRewardsEvent * chore: add nolint for TestFundCommunityPoolEvent and TestClaimRewardsEvent * feat(precompile): add contract can call FundCommunityPool tx * chore: lint check * chore: fix make lint and rename delegator to depositor * update DistributionCaller contract * add nix tests * use base denom as in msg build stage * fix gh action yml * fix doc * add sender post balance check * Update precompiles/distribution/types.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: bring back deposit and tesatFundCommunityPool methods --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.4.0 to 7.5.0 (#2556) * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.4.0 to 7.5.0 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.4.0 to 7.5.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.5.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/compare/v7.4.0...v7.5.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * gomod tidy * update go mod * add query router to ICA host keeper * update deps * add changelog entry * go mod tidy * fix chlog * update ICS20 precompile with new TransferAuth field * run make format * fix chlog * fix chlog * update ics20 nix tests --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump bufbuild/buf-setup-action from 1.32.2 to 1.33.0 (#2621) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.32.2 to 1.33.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.32.2...v1.33.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump github.com/gorilla/websocket from 1.5.2 to 1.5.3 (#2620) * build(deps): bump github.com/gorilla/websocket from 1.5.2 to 1.5.3 Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.2 to 1.5.3. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](https://github.com/gorilla/websocket/compare/v1.5.2...v1.5.3) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump docker/build-push-action from 5 to 6 (#2622) Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5 to 6. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](https://github.com/docker/build-push-action/compare/v5...v6) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/spf13/cobra from 1.8.0 to 1.8.1 (#2623) * build(deps): bump github.com/spf13/cobra from 1.8.0 to 1.8.1 Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.8.0 to 1.8.1. - [Release notes](https://github.com/spf13/cobra/releases) - [Commits](https://github.com/spf13/cobra/compare/v1.8.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Change anteutils.TxFeeChecker to authante.TxFeeChecker (#2619) * change back to use cosmos TxFeeChecker type instead of utils type * remove unused code * nit * nit * add change log * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](https://github.com/linxGnu/grocksdb/compare/v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](https://github.com/btcsuite/btcd/compare/v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](https://github.com/hashicorp/go-getter/compare/v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * merge 'main' to GAtom22/sdk50 * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * feat(vm): move geth vm from fork to evmOS codebase (#2663) * temp * run make format * fix unit tests * fix imports * add changelog entry * exclude x/evm/core from license checker * remove unnecessary rawdb * remove unused runtime folder * semgrep: skip iteration if map is empty * add new x/evm/folder to ignored path in golangci-lint * feat(evm): add custom EIPs to VM (#2629) * chore: add custom eips integration tests (#2661) * update custom eips + test * move eips to eips package * add eips tests * run make format * revert an error * add missing license to file * remove unused func * enable extra eip and refactor tests * add readme * Apply suggestions from code review Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * update doc and remove unused default eips * make markdown linter happy * fix lint --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore: merge main (#2662) * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](https://github.com/linxGnu/grocksdb/compare/v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](https://github.com/btcsuite/btcd/compare/v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](https://github.com/hashicorp/go-getter/compare/v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * update certificate --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> * chore: merge main again * run make format * Update CHANGELOG.md Signed-off-by: Guillermo Paoletti <guillermo.paoletti@gmail.com> * fix: remove import * chore: ignore linter on geth code --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Guillermo Paoletti <guillermo.paoletti@gmail.com> Co-authored-by: Freddy Caceres <facs95@gmail.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: hanchon <hanchon@users.noreply.github.com> * build(deps): bump golang.org/x/net from 0.26.0 to 0.27.0 (#2660) * build(deps): bump golang.org/x/net from 0.26.0 to 0.27.0 Bumps [golang.org/x/net](https://github.com/golang/net) from 0.26.0 to 0.27.0. - [Commits](https://github.com/golang/net/compare/v0.26.0...v0.27.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Vladislav Varadinov <vladislav.varadinov@gmail.com> * build(deps): bump github.com/cometbft/cometbft from 0.37.8 to 0.37.9 (#2671) * build(deps): bump github.com/cometbft/cometbft from 0.37.8 to 0.38.10 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.8 to 0.38.10. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.10/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.8...v0.38.10) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * build(deps): bump github.com/cometbft/cometbft from 0.37.8 to 0.37.9 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * feat(str): Single Token Representation V2 (#2607) * working version * opcode changes * run make format * tests compiling * fix lint * run make format * fix erc20 precompiles tests * fix genesis state * fix some tests * fix lint * update erc20 contracts * delete registercoin method. Delete proposal register. fix some tests * add native token check on ibc transfer. fix tests. * fix linter * fix tests * run make format * add nix tests * fix nix tests * fix protos * feat(str): migration code (#2632) * minor changes * missing tests * run make format * update docstring + lint * remove redundant check * move duplicate util to utils package * move number to constant * small opt + remove unused * Update app/app.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update ibc/utils.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update x/evm/types/params.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update x/erc20/module.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update x/erc20/module.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * fix merge * minor changes * add check on errors * add error check * fix token factory * run migration * feat(str): revert test for erc20 precompile (#2670) * wip: reverter test * run transfer * transfer and send tests and fix * check sender balance * run make format * test with try * lint * lint * lint * add licenses --------- Signed-off-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: ramacarlucho <ramacarlucho@users.noreply.github.com> * fix comments. Add changelog --------- Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: ramacarlucho <ramacarlucho@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore(ci): add action to auto-format python files (#2673) * chore(ci): add action to auto-format python files * run make format-python * update action job name * run make format-python * add job to existing yaml * fix linter call order * run make format-python * add line length on black formatter * run make format-python * revert - add line length on black formatter * run make format-python * add pylint config file * run make format-python * fix lint err * run make format-python * fix lint err * keep only changed config param --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: dev-evmos <dev-evmos@users.noreply.github.com> * imp(changelog): Minor changelog clean ups to fix linter (#2675) clean changelog * make contracts-all * make format * fix some compilation issues * bump cosmos-sdk to v0.50.8 * run make format-python * fix compilation issues * update app.go * chore(distr-precompile): refactor new tests (#2680) * fix unit tests * start int_test refactor * fix unit tests * fix integration tests * remove duplicated test case * Update precompiles/distribution/tx_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * Update precompiles/distribution/tx_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update precompiles/bank/bank.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * fix bank precompile integ_tests * chore(stk-precompile): refactor new tests (#2686) * fix unit tests * fix integration tests * fix evm ante tests * fix eip tests * fix app_test.go * chore(erc20-prec): refactor tests (#2688) * fix unit tests * start int_test refactor * fix integration tests * fix protogen script and run make proto-gen * make proto-gen * wire up modules tx and query cmds * fix bank precompile unit tests * fix vesting mod unit test * chore(erc20): refactor module and tests (#2701) * fix some unit tests * add custom get signers for convertERC20 msg * update test utils and msg server tests * fix msg_server_test.go * fix int_test * refactor ibc callback int_tests * add cosmos.msg.v1.service annotation to tx.proto * run make format-python * refactor toggleConversion & RegisterERC20 props * refactor ibc callback integration tests * wire up mods CLI commands * remove unused imports * run make format-python * update config * remove unused code --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> …
GAtom22
added a commit
that referenced
this pull request
Aug 19, 2024
* build(deps): bump golang from 1.22.3-alpine3.20 to 1.22.4-alpine3.20 (#2605) Bumps golang from 1.22.3-alpine3.20 to 1.22.4-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump golang.org/x/net from 0.25.0 to 0.26.0 (#2604) * build(deps): bump golang.org/x/net from 0.25.0 to 0.26.0 Bumps [golang.org/x/net](https://github.com/golang/net) from 0.25.0 to 0.26.0. - [Commits](https://github.com/golang/net/compare/v0.25.0...v0.26.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang.org/x/text from 0.15.0 to 0.16.0 (#2603) * build(deps): bump golang.org/x/text from 0.15.0 to 0.16.0 Bumps [golang.org/x/text](https://github.com/golang/text) from 0.15.0 to 0.16.0. - [Release notes](https://github.com/golang/text/releases) - [Commits](https://github.com/golang/text/compare/v0.15.0...v0.16.0) --- updated-dependencies: - dependency-name: golang.org/x/text dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * feat(evm): Add PermissionsPolicy for permissioned EVM (#2538) * feat(evm): Add Create OpHook for permissioned contract deployment * remove go.mod replae * rename function * permission policy implementaion * remove enable create and call parmas * add validate * revert go.mod replace * fix proto names * update geth version * fix lint * policy unit tests * integration tests * fix tests * fix test * rename permission to accesscontrol * fix params tests * fix lint * fix test * proto lint * add licenses * rename param * fix proto lint issue * add balance checks to mint * run make format * gomod2nix generate * add store migration to v7 * fix comment * fix proto-lint issues * fix proto names on migration * add changelog entry * fix migration tests * add address list for permissionless * add integration tests * update tests * adding necessary tests * update tests * run make format * fix lint * small refractor * expect common.Address instead of string * update comments * fix review comments --------- Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore(docker): update docker dep (#2610) update docker dep * Revert "chore(erc20): re-add deleted erc20 evm hooks (#2502)" & remove STRv2 tracking logic (#2609) * Revert "chore(erc20): re-add deleted erc20 evm hooks (#2502)" This reverts commit 994f54da651a213a637f6f39ddfc551dfcd30252. * remove strv2 tracking logic * add changelog entry * Revert "imp(evm): Add evm hook logic again (#2501)" (#2608) This reverts commit d0ddd88b9ecbc7777f440aeb5feeb468148868e0. Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(evm): Move OpcodeHooks into types (#2611) * imp(evm): Move OpcodeHooks into types * run make format * fix lint * update comments * run make format --------- Co-authored-by: facs95 <facs95@users.noreply.github.com> * fix(contracts): Remove unused contract and make script compatible with Python <3.12 (#2613) * remove unused contract and run make contracts-compile * remove Union operator to support Python <3.12 * add changelog entry * build(deps): bump github.com/gorilla/websocket from 1.5.1 to 1.5.2 (#2612) * build(deps): bump github.com/gorilla/websocket from 1.5.1 to 1.5.2 Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.1 to 1.5.2. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](https://github.com/gorilla/websocket/compare/v1.5.1...v1.5.2) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp: add integration test for precompile tx `CreateValidator` (#2575) * imp: add integration test for precompile tx CreateValidator * chore: update comment * feat(precompile): add staking module create validator integration test * change hardcoded tendermint pub key with util function --------- Signed-off-by: Luke <luchenqun@qq.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore(precompiles): update distribution precompile (#2614) * chore(precompiles): update distribution precompile * add changelog entry * add more test cases * Update CHANGELOG.md Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump google.golang.org/protobuf from 1.34.1 to 1.34.2 (#2615) * build(deps): bump google.golang.org/protobuf from 1.34.1 to 1.34.2 Bumps google.golang.org/protobuf from 1.34.1 to 1.34.2. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp: add integration test for precompile tx `EditValidator` (#2576) * imp: add integration test for precompile tx EditValidator * imp: expected validator commission rate remain unchanged * imp: expected validator commission rate remain unchanged * update StakingCaller contract * add edit validator from sc call test * add fail test case * add missing checks in test * Update precompiles/staking/testdata/StakingCaller.json Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * improve specs readability * make contracts compile * add missing checks in tests * chore: merge the contents of the main branch manually --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * imp(precompiles): Add unit test to ensure correct Bech32 addresses for default precompiles (#2597) * add a test case for the bech32 converted precompile addresses in the default precompiles * run make format * address linter * fix some f-up during merge * fix changelog * fix panic when estimating gas during call to precompiles without input * remove default precompiles bech32 and move to blocked addrs function * add all Ethereum native precompiles to blocked addresses too * run make format * format * f auto-format * add comments as suggested in code review * add utility method for future use to convert addresses * fix changelog --------- Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump the npm_and_yarn group across 3 directories with 2 updates (#2616) Bumps the npm_and_yarn group with 1 update in the /contracts directory: [braces](https://github.com/micromatch/braces). Bumps the npm_and_yarn group with 2 updates in the /tests/nix_tests/hardhat directory: [braces](https://github.com/micromatch/braces) and [axios](https://github.com/axios/axios). Bumps the npm_and_yarn group with 1 update in the /tests/solidity directory: [braces](https://github.com/micromatch/braces). Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) Updates `axios` from 1.6.1 to 1.7.2 - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](https://github.com/axios/axios/compare/v1.6.1...v1.7.2) Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) --- updated-dependencies: - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: axios dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * feat(precompile): add `FundCommunityPool` precompile tx for distribution module (#2572) * feat(precompile): add FundCommunityPool precompile tx for distribution module * chore: update CHANGELOG.md * chore: add nolint for TestFundCommunityPoolEvent and TestClaimRewardsEvent * chore: add nolint for TestFundCommunityPoolEvent and TestClaimRewardsEvent * feat(precompile): add contract can call FundCommunityPool tx * chore: lint check * chore: fix make lint and rename delegator to depositor * update DistributionCaller contract * add nix tests * use base denom as in msg build stage * fix gh action yml * fix doc * add sender post balance check * Update precompiles/distribution/types.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: bring back deposit and tesatFundCommunityPool methods --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.4.0 to 7.5.0 (#2556) * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.4.0 to 7.5.0 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.4.0 to 7.5.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.5.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/compare/v7.4.0...v7.5.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * gomod tidy * update go mod * add query router to ICA host keeper * update deps * add changelog entry * go mod tidy * fix chlog * update ICS20 precompile with new TransferAuth field * run make format * fix chlog * fix chlog * update ics20 nix tests --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump bufbuild/buf-setup-action from 1.32.2 to 1.33.0 (#2621) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.32.2 to 1.33.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.32.2...v1.33.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump github.com/gorilla/websocket from 1.5.2 to 1.5.3 (#2620) * build(deps): bump github.com/gorilla/websocket from 1.5.2 to 1.5.3 Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.2 to 1.5.3. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](https://github.com/gorilla/websocket/compare/v1.5.2...v1.5.3) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump docker/build-push-action from 5 to 6 (#2622) Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5 to 6. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](https://github.com/docker/build-push-action/compare/v5...v6) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/spf13/cobra from 1.8.0 to 1.8.1 (#2623) * build(deps): bump github.com/spf13/cobra from 1.8.0 to 1.8.1 Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.8.0 to 1.8.1. - [Release notes](https://github.com/spf13/cobra/releases) - [Commits](https://github.com/spf13/cobra/compare/v1.8.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Change anteutils.TxFeeChecker to authante.TxFeeChecker (#2619) * change back to use cosmos TxFeeChecker type instead of utils type * remove unused code * nit * nit * add change log * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](https://github.com/linxGnu/grocksdb/compare/v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](https://github.com/btcsuite/btcd/compare/v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](https://github.com/hashicorp/go-getter/compare/v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * feat(vm): move geth vm from fork to evmOS codebase (#2663) * temp * run make format * fix unit tests * fix imports * add changelog entry * exclude x/evm/core from license checker * remove unnecessary rawdb * remove unused runtime folder * semgrep: skip iteration if map is empty * add new x/evm/folder to ignored path in golangci-lint * feat(evm): add custom EIPs to VM (#2629) * chore: add custom eips integration tests (#2661) * update custom eips + test * move eips to eips package * add eips tests * run make format * revert an error * add missing license to file * remove unused func * enable extra eip and refactor tests * add readme * Apply suggestions from code review Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * update doc and remove unused default eips * make markdown linter happy * fix lint --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore: merge main (#2662) * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](https://github.com/linxGnu/grocksdb/compare/v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](https://github.com/btcsuite/btcd/compare/v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](https://github.com/hashicorp/go-getter/compare/v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * update certificate --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> * chore: merge main again * run make format * Update CHANGELOG.md Signed-off-by: Guillermo Paoletti <guillermo.paoletti@gmail.com> * fix: remove import * chore: ignore linter on geth code --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Guillermo Paoletti <guillermo.paoletti@gmail.com> Co-authored-by: Freddy Caceres <facs95@gmail.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: Amine <34750005+Aminechakr@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: hanchon <hanchon@users.noreply.github.com> * build(deps): bump golang.org/x/net from 0.26.0 to 0.27.0 (#2660) * build(deps): bump golang.org/x/net from 0.26.0 to 0.27.0 Bumps [golang.org/x/net](https://github.com/golang/net) from 0.26.0 to 0.27.0. - [Commits](https://github.com/golang/net/compare/v0.26.0...v0.27.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Vladislav Varadinov <vladislav.varadinov@gmail.com> * build(deps): bump github.com/cometbft/cometbft from 0.37.8 to 0.37.9 (#2671) * build(deps): bump github.com/cometbft/cometbft from 0.37.8 to 0.38.10 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.8 to 0.38.10. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.10/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.8...v0.38.10) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * build(deps): bump github.com/cometbft/cometbft from 0.37.8 to 0.37.9 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * feat(str): Single Token Representation V2 (#2607) * working version * opcode changes * run make format * tests compiling * fix lint * run make format * fix erc20 precompiles tests * fix genesis state * fix some tests * fix lint * update erc20 contracts * delete registercoin method. Delete proposal register. fix some tests * add native token check on ibc transfer. fix tests. * fix linter * fix tests * run make format * add nix tests * fix nix tests * fix protos * feat(str): migration code (#2632) * minor changes * missing tests * run make format * update docstring + lint * remove redundant check * move duplicate util to utils package * move number to constant * small opt + remove unused * Update app/app.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update ibc/utils.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update x/evm/types/params.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update x/erc20/module.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update x/erc20/module.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * fix merge * minor changes * add check on errors * add error check * fix token factory * run migration * feat(str): revert test for erc20 precompile (#2670) * wip: reverter test * run transfer * transfer and send tests and fix * check sender balance * run make format * test with try * lint * lint * lint * add licenses --------- Signed-off-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: ramacarlucho <ramacarlucho@users.noreply.github.com> * fix comments. Add changelog --------- Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: ramacarlucho <ramacarlucho@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore(ci): add action to auto-format python files (#2673) * chore(ci): add action to auto-format python files * run make format-python * update action job name * run make format-python * add job to existing yaml * fix linter call order * run make format-python * add line length on black formatter * run make format-python * revert - add line length on black formatter * run make format-python * add pylint config file * run make format-python * fix lint err * run make format-python * fix lint err * keep only changed config param --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: dev-evmos <dev-evmos@users.noreply.github.com> * imp(changelog): Minor changelog clean ups to fix linter (#2675) clean changelog * chore(app): bump version to v19 (#2674) bump version to v19 * build(deps): bump github.com/holiman/uint256 from 1.2.1 to 1.3.0 (#2676) * build(deps): bump github.com/holiman/uint256 from 1.2.1 to 1.3.0 Bumps [github.com/holiman/uint256](https://github.com/holiman/uint256) from 1.2.1 to 1.3.0. - [Release notes](https://github.com/holiman/uint256/releases) - [Commits](https://github.com/holiman/uint256/compare/v1.2.1...v1.3.0) --- updated-dependencies: - dependency-name: github.com/holiman/uint256 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * chore: fix some comments (#2681) Signed-off-by: pudongair <744355276@qq.com> * chore: migration to enable custom eips (#2667) * add eips migration + tests * run make format * Apply suggestions from code review Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * Update app/upgrades/v19/upgrades.go Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update app/upgrades/v19/upgrades.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * Update app/upgrades/v19/upgrades.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * review comments * better logging * Update app/upgrades/v19/upgrades_test.go Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * apply suggestions * fix * update activators type to use string keys * move proto to v8 and update types name * regenerate new …
GAtom22
added a commit
that referenced
this pull request
Sep 6, 2024
* wip - bump sdk to v0.50 * update chlog * fix Dec imports and tests NewContext * address more issues * refactor deliverTx abci method * replace cometbft-db dep * replace AccountI imports and context in Keepers interface * update keeper instantiation and interfaces * update more keepers interfaces * fix keepers and modules interfaces * fix more issues * fix more issues * fix more issues on return signatures * refactor api server start func * refactor rosetta config * refactor rosetta start func * fix more issues * fix more issues * fix more issues * update proto * fix migration proto types * fix more issues * update ibc v8 * fix more issues * add getMsgsV2 to msgEthTx * fix signByAddress calls * remove deprecated Handlers * refactor BeginBlock and EndBlock in modules * fix context type issues * fix testutil network setup * fix more issues * fix more issues * fix more issues * remove simapp pkg * update sdk to evmos fork * remove rosetta dep temprarily to compile * fix comment * update init cmd * set expedited_voting_period in local_node.sh * changes to support auto cli * update proto with amino.proto * fix encoding and interface registration * add version and name * fix genesis parsing * run make format * fix createTx util func * make format * go mod tidy and update cosmossdk pkgs imports * fix some types * make proto-all * fix merge issues * update bank mock * address some lint issues * fix compilation issues * fix delegation creation * update helper and setup functions * update go version in gh workflows * add pulsar proto gen and start getSigner logic * finish customGetSigner logic * add rosetta with patched version * add rosetta config to configTemplate * gomod2nix generate * make format * chore(tests): update testing setup for sdk v0.50 (#2193) * update testing setup for custom genesis * fix evm integration test * refactor p256 integration tests * refactor update params * refactor v16 upgrade test * fix comment * add gov setup and generic type * refactor genesis setup * add comments * update gen setup funcs * funder functions refactor * fix nextBlockAfter func * wip - start fixing erc20 precompile integration tests * add next block to update acc nonce * changes based on review comments * refactor update params funcs * improve err msg * remove unused imports * add comments * make format * refactor based on review comments * refactor evm integration tests based on review comments * make format * handle error on iterations (export.go) * chore(test): migrate evm keeper tests (#2209) * fix problems * add bank genesis custom fn * add feemkt custom gen fn * wip - start evm test refactor * fix bugs on setup * refactor fees tests * refactor statedb tests * fix setAccount panic * add events utils * add new suite * fix(evm): Fix EVM msg_server tests (#2281) * imp(evm): Fix EVM msg_server tests * add comment * fix(evm): fix `state_transition` unit tests (#2283) * fix TestGetHashFn test * convert validator address to hex * fix test IntrinsicGas * fix TestApplyMessage * fix TestApplyMessage * reformat GasRefundTest * TestGasRefund fix * run format * fix all tests * fix imports * target review comments * add licenses * update comment * rename function * rename file * fix licenses * fix(evm): Fix `grpc_query` tests (#2287) * fix QueryAccount PR * fix TestQueryCosmosAccount * fix TestQuery * fix TestQueryStorage * fix TestQueryCode * fix TestQueryParams * fix TestEstimateGas * temporary tests * trace transaction update * refactor TraceTx tests * refractor traceBlock * refractor functions * refactor tracing tests * refractor test query BaseFee * refractor eth call * refractor TestEmptyRequest * refractor TestQueryValidatorAccount * refractor TestEstimateGas * refractor TestTrace * Update x/evm/keeper/grpc_query_test.go Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * fix non determinism * fix comments * review comments --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * chore(tests): update nix setup for sdk v0.50 (#2318) * bump go version to 1.21 * comment out unsoported rocksdb build * remove unnecessary flag * update config and patch * bump hermes version to v1.8.0 * run make format * address lint issues * address lint issues --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * fix(tests): Fix statedb and abci tests on the EVM module (#2325) * fix suicide statedb test * fix abci_test * run make format --------- Co-authored-by: facs95 <facs95@users.noreply.github.com> * add missing licenses * add wait for blocks fn using http * Apply suggestions from code review Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * changes based on review comments * fix(tests): refactor genesis and handler tests for sdk v0.50 (#2328) * fix(tests): refactor genesis and handler tests for sdk v0.50 * add corresponding acc number * fund fee collector for refunds * address lint issues --------- Co-authored-by: Freddy Caceres <facs95@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore(tests): refactor EVM anteHandler tests (#2338) * fix problems * add bank genesis custom fn * add feemkt custom gen fn * wip - start evm test refactor * fix bugs on setup * refactor fees tests * refactor statedb tests * fix setAccount panic * add events utils * add new suite * fix(evm): Fix EVM msg_server tests (#2281) * imp(evm): Fix EVM msg_server tests * add comment * fix(evm): fix `state_transition` unit tests (#2283) * fix TestGetHashFn test * convert validator address to hex * fix test IntrinsicGas * fix TestApplyMessage * fix TestApplyMessage * reformat GasRefundTest * TestGasRefund fix * run format * fix all tests * fix imports * target review comments * add licenses * update comment * rename function * rename file * fix licenses * fix(evm): Fix `grpc_query` tests (#2287) * fix QueryAccount PR * fix TestQueryCosmosAccount * fix TestQuery * fix TestQueryStorage * fix TestQueryCode * fix TestQueryParams * fix TestEstimateGas * temporary tests * trace transaction update * refactor TraceTx tests * refractor traceBlock * refractor functions * refactor tracing tests * refractor test query BaseFee * refractor eth call * refractor TestEmptyRequest * refractor TestQueryValidatorAccount * refractor TestEstimateGas * refractor TestTrace * Update x/evm/keeper/grpc_query_test.go Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * fix non determinism * fix comments * review comments --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * chore(tests): update nix setup for sdk v0.50 (#2318) * bump go version to 1.21 * comment out unsoported rocksdb build * remove unnecessary flag * update config and patch * bump hermes version to v1.8.0 * run make format * address lint issues * address lint issues --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * fix(tests): Fix statedb and abci tests on the EVM module (#2325) * fix suicide statedb test * fix abci_test * run make format --------- Co-authored-by: facs95 <facs95@users.noreply.github.com> * add missing licenses * add wait for blocks fn using http * Apply suggestions from code review Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * changes based on review comments * fix(tests): refactor genesis and handler tests for sdk v0.50 (#2328) * fix(tests): refactor genesis and handler tests for sdk v0.50 * add corresponding acc number * fund fee collector for refunds * address lint issues * refactor for UpdateFeemarket in gas_wanted * refactor setup and fix compilation errors * refactor tests * fix string to addr convertion in vesting ante * add regression amino testing to encoding config * refactor denom used on tests to use suite denom * fix eip712 get signers logic * remove MsgEthereumTx signers annotation cause has a customGetSigners * refactor math pkg import name * remove redundant code in test setup * fix EIP712 legacy verification * fix lint warning * add empty tx check * fix eth tx feemarket tests * fix sigs_tests and refactor utils funcs * remove refactored code * fix consensus params customization * fix refactoring on integration test tx factory * add comment * update comments * remove commented code --------- Co-authored-by: Freddy Caceres <facs95@gmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore(tests): refactor cosmos anteHandler and ante utils tests (#2346) * refactor setup to reuse in cosmos ante tests * refactor cosmos ante setup and fix compilation errors * refactor tests * fix RejectMsgsInAuthz tests * run make format * fix lint issue * fix ante utils tests * changes based on review comments --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * chore(test): refactor anteHandler integration test (#2353) * refactor integration test * run make format * run make format * fix comment * fix lint issue * Update testutil/integration/common/grpc/distribution.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: dev-evmos <dev-evmos@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * remove duplicated helper func * chore(tests): fix EIP712 tests (#2354) * fix eip712 test * fix preprocess tx tests * add comment * chore(tests): fix bech32 precompiled tests * chore(tests): migrate bank precompiles tests (#2363) * fix nil pointer * refactor ctx * fix bank precompile unit tests * precompiles(bank) start integration test refactor * fix setup for integration test * remove unnecessary code * fix tests * run make format * changes based on review comments * add missing licenses --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * chore(tests): migrate epochs module tests (#2369) * refactor suite * update suite and tests * complete begin block and init genesis test * complete abci test refactoring * refactor epoch info tests * update grpc query tests * run make format * remove genesis handler function check * move frequently used names as const * refactor test structure * run make format * Update x/epochs/keeper/setup_test.go Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * apply review suggestion --------- Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * chore(tests): migrate distribution precompiled tests (#2364) * update setup and fix compilation errors * fix testClaimRewards * remove unnecessary code * fix testClaimsEvents * fix testDelegationRewards * fix testDelegationTotalRewards * fix lint issue * fix WithdrawDelegatorRewards * fix testRun * fix validatorCommision test * fix validatorDistrInfo test * fix ValidatorOutstandingRewards query test * fix ValidatorSlashes query test * run make format * fix WithdrawValCommission tests * fix events tests * fix setup for integration test * refactor some test cases * make format * run make format * add withdraw rewards to withdrawer tests * fix flag on distr precompile and fix withdrawer tests * fix contract as delegator tests * fix some queries tests * add comment * fix slashing tests * fix tests * fix expected rewards * remove unnecessary code * run make format * run make format * fix account num in evm keeper test * remove global test suite variable * run make format * Update precompiles/distribution/tx_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * changes based on review comments * Apply suggestions from code review Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * break on iteration error * Update precompiles/distribution/query_test.go Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * fix issues * refactor based on review comments * run make format * Revert "refactor based on review comments" This reverts commit 8ebee30c5fc4a9299e82551409afc8b28d6cd033. * refactor based on new factory design * run make format * separate unit and integration tests --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: dev-evmos <dev-evmos@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * chore(tests): migrate staking precompile tests (#2384) * update setup and fix compilation errors * fix testClaimRewards * remove unnecessary code * fix testClaimsEvents * fix testDelegationRewards * fix testDelegationTotalRewards * fix lint issue * fix WithdrawDelegatorRewards * fix testRun * fix validatorCommision test * fix validatorDistrInfo test * fix ValidatorOutstandingRewards query test * fix ValidatorSlashes query test * run make format * fix WithdrawValCommission tests * fix events tests * fix setup for integration test * refactor some test cases * make format * run make format * add withdraw rewards to withdrawer tests * fix flag on distr precompile and fix withdrawer tests * fix contract as delegator tests * fix some queries tests * add comment * fix slashing tests * fix tests * fix expected rewards * remove unnecessary code * run make format * run make format * fix account num in evm keeper test * remove global test suite variable * run make format * refactor stk precompile setup and fix compilation errors * fix events tests * Update precompiles/distribution/tx_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * refactor helper func * changes based on review comments * fix more tests * Apply suggestions from code review Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * break on iteration error * Update precompiles/distribution/query_test.go Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * fix issues * fix testDelegate * fix testRedelegations * fix testRun * fix util func check * start refactor on int_tests * refactor based on review comments * run make format * continue refactor * Revert "refactor based on review comments" This reverts commit 8ebee30c5fc4a9299e82551409afc8b28d6cd033. * refactor based on new factory design * run make format * continue refactor int test * refactor helper func * fix some test cases * refactor keepers query with grpc method * refactor to use grpc instead of jeepers * fix some test cases * fix some test cases * fix some test cases * fix more tests * run make format * run make format * remove unnecessary address conversions * commit WIP addressing review comments * address further review comments * wip * fix test * address unhandled error linters * fix tests and avoid self-approvals * run make format * address review comment --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: dev-evmos <dev-evmos@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <malte@evmos.org> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * chore(tests): migrate vesting precompile tests (#2387) * chore(tests): migrate feemarket module tests (#2402) * refactor setup and fix compilation errors * fix gen account type * fix some test cases * fix some test cases * fix exp err messages * fix setup for test cases * fix block time bug * remove unnecessary test case * fix lint warnings * fix distr tests * fix evm keeper tests * make format * fix epochs tests * use utc time * chore(tests): start fix feemarket mod tests * fix abci tests * fix eip1559 tests * grpc query tests * keeper unit tests * fix migrations tests * fix msg_server tests * fix params tests * refactor integration test * fix test cases * fix integration tests * run make format * remove unnecessary code --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * chore(tests): migrate inflation module tests (#2405) * migrate epoch info tests * migrate gen tests * migrate grpc query tests * migrate infl unit tests * migrate hooks tests * remove utils.go * refactor setup.go * finish unit tests refactor * fix integration tests * run make format * fix lint warnings * Apply suggestions from code review * remove using App in integration tests where possible * add inflation queries to grpc handler and refactor integration tests * address more review comments * Update x/inflation/v1/keeper/hooks_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * rename var * Update x/inflation/v1/keeper/inflation_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * refactor based on review comments * refactor based on review comments --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <malte@evmos.org> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * fix(grpc): account address return in ValidatorAccount query response (#2434) * update files to fix build * run make format * remove related stuff to removed x/claims module * fix staking precompile query tests * update inflation mod tests with new setup exp values * build(deps): bump cosmossdk.io/api to 0.7.4 * add missing line in IBC init params * fix app export func test * fix wallet tests * fix evm initGenesis test * fix statedb int_tests * add store migrations in ibc transfer wrapper * bump ibc to v8.2.0 * remove deprecated GetSigners func * fix flaky evm keeper test * fix CreateValidator del amount check * fix precompile staking tests * fix lint warnings * build(deps): bump github.com/cometbft/cometbft to v0.38.7 * go mod tidy * chore(tests): migrate staking module wrapper tests (#2551) * fix msg server unit tests * fix integration tests * run make format --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * build(deps): bump cosmossdk.io/api to 0.7.5 * chore(tests): migrate vesting module tests (#2408) * refactor unit tests * run make format * start refactor on int_tests * wip - integration tests * fix some test cases * fix some test cases * fix some tests and fix setup * fix some tests * run make format * remove unnecessary code * migrate some tests * fix vesting tests * fix BroadcastTxSync finalize block req * fix vesting integration tests * update comment * Update x/vesting/keeper/integration_test.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * address review comments --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> * build(deps): bump github.com/spf13/viper to 1.19.0 * fix compilation issues * refactor new tests * chore(deps): bump to cosmos-sdk v0.50.7 * build(deps): bump github.com/cosmos/gogoproto to 1.5.0 * chore(tests): refactor ICS20 precompile tests (#2600) * start refactor * fix bech32 prefixes * refactor ICS20 authorization tests * run make format * add events check on tests * add denom traces query test * refactor ICS20 queries tests * refactor ICS20 tx tests * remove old files --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * address lint issues * build(deps): bump github.com/cometbft/cometbft from 0.38.7 to 0.38.9 * build(deps): bump github.com/cometbft/cometbft from 0.38.9 to 0.38.10 * bump ibc to v8.3.2 * chore: sync with main branch (#2678) * build(deps): bump golang from 1.22.3-alpine3.20 to 1.22.4-alpine3.20 (#2605) Bumps golang from 1.22.3-alpine3.20 to 1.22.4-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump golang.org/x/net from 0.25.0 to 0.26.0 (#2604) * build(deps): bump golang.org/x/net from 0.25.0 to 0.26.0 Bumps [golang.org/x/net](https://github.com/golang/net) from 0.25.0 to 0.26.0. - [Commits](https://github.com/golang/net/compare/v0.25.0...v0.26.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang.org/x/text from 0.15.0 to 0.16.0 (#2603) * build(deps): bump golang.org/x/text from 0.15.0 to 0.16.0 Bumps [golang.org/x/text](https://github.com/golang/text) from 0.15.0 to 0.16.0. - [Release notes](https://github.com/golang/text/releases) - [Commits](https://github.com/golang/text/compare/v0.15.0...v0.16.0) --- updated-dependencies: - dependency-name: golang.org/x/text dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * feat(evm): Add PermissionsPolicy for permissioned EVM (#2538) * feat(evm): Add Create OpHook for permissioned contract deployment * remove go.mod replae * rename function * permission policy implementaion * remove enable create and call parmas * add validate * revert go.mod replace * fix proto names * update geth version * fix lint * policy unit tests * integration tests * fix tests * fix test * rename permission to accesscontrol * fix params tests * fix lint * fix test * proto lint * add licenses * rename param * fix proto lint issue * add balance checks to mint * run make format * gomod2nix generate * add store migration to v7 * fix comment * fix proto-lint issues * fix proto names on migration * add changelog entry * fix migration tests * add address list for permissionless * add integration tests * update tests * adding necessary tests * update tests * run make format * fix lint * small refractor * expect common.Address instead of string * update comments * fix review comments --------- Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: facs95 <facs95@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore(docker): update docker dep (#2610) update docker dep * Revert "chore(erc20): re-add deleted erc20 evm hooks (#2502)" & remove STRv2 tracking logic (#2609) * Revert "chore(erc20): re-add deleted erc20 evm hooks (#2502)" This reverts commit 994f54da651a213a637f6f39ddfc551dfcd30252. * remove strv2 tracking logic * add changelog entry * Revert "imp(evm): Add evm hook logic again (#2501)" (#2608) This reverts commit d0ddd88b9ecbc7777f440aeb5feeb468148868e0. Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(evm): Move OpcodeHooks into types (#2611) * imp(evm): Move OpcodeHooks into types * run make format * fix lint * update comments * run make format --------- Co-authored-by: facs95 <facs95@users.noreply.github.com> * fix(contracts): Remove unused contract and make script compatible with Python <3.12 (#2613) * remove unused contract and run make contracts-compile * remove Union operator to support Python <3.12 * add changelog entry * build(deps): bump github.com/gorilla/websocket from 1.5.1 to 1.5.2 (#2612) * build(deps): bump github.com/gorilla/websocket from 1.5.1 to 1.5.2 Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.1 to 1.5.2. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](https://github.com/gorilla/websocket/compare/v1.5.1...v1.5.2) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp: add integration test for precompile tx `CreateValidator` (#2575) * imp: add integration test for precompile tx CreateValidator * chore: update comment * feat(precompile): add staking module create validator integration test * change hardcoded tendermint pub key with util function --------- Signed-off-by: Luke <luchenqun@qq.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore(precompiles): update distribution precompile (#2614) * chore(precompiles): update distribution precompile * add changelog entry * add more test cases * Update CHANGELOG.md Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump google.golang.org/protobuf from 1.34.1 to 1.34.2 (#2615) * build(deps): bump google.golang.org/protobuf from 1.34.1 to 1.34.2 Bumps google.golang.org/protobuf from 1.34.1 to 1.34.2. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp: add integration test for precompile tx `EditValidator` (#2576) * imp: add integration test for precompile tx EditValidator * imp: expected validator commission rate remain unchanged * imp: expected validator commission rate remain unchanged * update StakingCaller contract * add edit validator from sc call test * add fail test case * add missing checks in test * Update precompiles/staking/testdata/StakingCaller.json Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * improve specs readability * make contracts compile * add missing checks in tests * chore: merge the contents of the main branch manually --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * imp(precompiles): Add unit test to ensure correct Bech32 addresses for default precompiles (#2597) * add a test case for the bech32 converted precompile addresses in the default precompiles * run make format * address linter * fix some f-up during merge * fix changelog * fix panic when estimating gas during call to precompiles without input * remove default precompiles bech32 and move to blocked addrs function * add all Ethereum native precompiles to blocked addresses too * run make format * format * f auto-format * add comments as suggested in code review * add utility method for future use to convert addresses * fix changelog --------- Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump the npm_and_yarn group across 3 directories with 2 updates (#2616) Bumps the npm_and_yarn group with 1 update in the /contracts directory: [braces](https://github.com/micromatch/braces). Bumps the npm_and_yarn group with 2 updates in the /tests/nix_tests/hardhat directory: [braces](https://github.com/micromatch/braces) and [axios](https://github.com/axios/axios). Bumps the npm_and_yarn group with 1 update in the /tests/solidity directory: [braces](https://github.com/micromatch/braces). Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) Updates `axios` from 1.6.1 to 1.7.2 - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](https://github.com/axios/axios/compare/v1.6.1...v1.7.2) Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3) --- updated-dependencies: - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: axios dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * feat(precompile): add `FundCommunityPool` precompile tx for distribution module (#2572) * feat(precompile): add FundCommunityPool precompile tx for distribution module * chore: update CHANGELOG.md * chore: add nolint for TestFundCommunityPoolEvent and TestClaimRewardsEvent * chore: add nolint for TestFundCommunityPoolEvent and TestClaimRewardsEvent * feat(precompile): add contract can call FundCommunityPool tx * chore: lint check * chore: fix make lint and rename delegator to depositor * update DistributionCaller contract * add nix tests * use base denom as in msg build stage * fix gh action yml * fix doc * add sender post balance check * Update precompiles/distribution/types.go Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: bring back deposit and tesatFundCommunityPool methods --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.4.0 to 7.5.0 (#2556) * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.4.0 to 7.5.0 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.4.0 to 7.5.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.5.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/compare/v7.4.0...v7.5.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * gomod tidy * update go mod * add query router to ICA host keeper * update deps * add changelog entry * go mod tidy * fix chlog * update ICS20 precompile with new TransferAuth field * run make format * fix chlog * fix chlog * update ics20 nix tests --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump bufbuild/buf-setup-action from 1.32.2 to 1.33.0 (#2621) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.32.2 to 1.33.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.32.2...v1.33.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump github.com/gorilla/websocket from 1.5.2 to 1.5.3 (#2620) * build(deps): bump github.com/gorilla/websocket from 1.5.2 to 1.5.3 Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.2 to 1.5.3. - [Release notes](https://github.com/gorilla/websocket/releases) - [Commits](https://github.com/gorilla/websocket/compare/v1.5.2...v1.5.3) --- updated-dependencies: - dependency-name: github.com/gorilla/websocket dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * build(deps): bump docker/build-push-action from 5 to 6 (#2622) Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5 to 6. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](https://github.com/docker/build-push-action/compare/v5...v6) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/spf13/cobra from 1.8.0 to 1.8.1 (#2623) * build(deps): bump github.com/spf13/cobra from 1.8.0 to 1.8.1 Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.8.0 to 1.8.1. - [Release notes](https://github.com/spf13/cobra/releases) - [Commits](https://github.com/spf13/cobra/compare/v1.8.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Change anteutils.TxFeeChecker to authante.TxFeeChecker (#2619) * change back to use cosmos TxFeeChecker type instead of utils type * remove unused code * nit * nit * add change log * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](https://github.com/linxGnu/grocksdb/compare/v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https://github.com/btcsuite/btcd/blob/master/CHANGES) - [Commits](https://github.com/btcsuite/btcd/compare/v0.24.0...v0.24.2) --- updated-dependencies: - dependency-name: github.com/btcsuite/btcd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(app): Split chain configuration template into EVM config and MemIAVL config (#2644) * separate EVM config template from full Evmos chain config template for use in other repos * add changelog entry * address review comments * imp(ante): Fix typos in ante package (#2647) * fix typos in ante package * address review comments * build(deps): bump github.com/cometbft/cometbft from 0.37.5 to 0.37.7 (#2646) * build(deps): bump the go_modules group with 2 updates Bumps the go_modules group with 2 updates: [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) and [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/cometbft/cometbft` from 0.37.5 to 0.38.8 - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.8/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.5...v0.38.8) Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](https://github.com/hashicorp/go-getter/compare/v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update cometbft version bump --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * imp(ante): Decouple EVM decorator utilities from decorator struct (#2648) * decouple evm decorator utils from concrete implementation * add changelog entry * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.37.8 (#2649) * build(deps): bump github.com/cometbft/cometbft from 0.37.7 to 0.38.9 Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.37.7 to 0.38.9. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.9/CHANGELOG.md) - [Commits](https://github.com/cometbft/cometbft/compare/v0.37.7...v0.38.9) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update bump version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * imp(scripts): Adjust contract compilation util to include abi.json files (#2650) * adjust script to include abi.json files too * rename vesting interface file to match interface name * adjust utils and precompile ABI loading for Hardhat setup * clean up after compiling contracts with all target * recompile abis * add changelog entry * run black formatter * address some more linters --------- Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * build(deps): bump golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20 (#2654) Bumps golang from 1.22.4-alpine3.20 to 1.22.5-alpine3.20. --- updated-dependencies: - dependency-name: golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#2652) * build(deps): bump google.golang.org/grpc from 1.64.0 to 1.65.0 Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.65.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.64.0...v1.65.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * Merge pull request from GHSA-68fc-7mhg-6f6c * fix(stk-precompile): update dirty state * add create validator tests cases * fix(distr-precompile): update dirty state * add additional condition in isContract * refactor test * add more functions to the DistrCaller.sol * fix(ics20-precompile) update dirty state * chore(tests): add more test cases for ics20 precompile * add multi-transfer tx test cases * chore(tests): add more test cases for distr precompile * Update precompiles/common/precompile.go Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * refactor getWithdrawerHexAddr func * chore(staking-precompile): update CreateValidator and EditValidator checks * chore(tests): update staking precompile tests * chore(distr-precompile): remove claimRewards method * make format * add back claimRewards func commented out * comment out claimRewards from distr precompile logic * add staking test with transfer to bonded tokens acc * chore(ibc-transfer): add escrow account to dirties * chore(test): add test cases for ics20 precompile and escrow addr * Update tests/nix_tests/test_ics20_precompile.py Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * remove hardcoded exp balance * fix(precompile): use cache ctx on stateDB * add journal entry for precompiles * keep only last writeFn * add MaxPrecompileCalls limit * refactor precompile calls logic * remove unnecessary code * changes based on review comments * add StakingReverter tests * add revert test case for distr precompile * refactor cache ctx on stateDB * refactor: move CommitWithCacheCtx to RunSetup * chore: add events to snapshot * update ICS20 tests * fix err msg * renamve var * add a func on precompile cmn for journal entry * rename var * add comment * update comment description * update expected gas in ibc transfer test * address review comments * update comt * tests(ics20): add helper func to setup contract * tests(ics20): add test for nested revert * address review comments * add distr precompile tests * refactor to remove UpdateDirties func * Update precompiles/distribution/tx.go Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update vesting prec * refactor balance change entries setter func * add comments * update comment * fix sdk fork deps * comment claimRewards method and test * add smart contract balance check on test * remove transient storage logic * add edge test case with revert on storage change * revert changes to claimRewards * refactor var name * refactor claimRewards logic * refactor claimRewards logic * update cosmos-sdk fork with latest changes (no need for store keys deep copy) * change require with assert for invariant * update comment * update cosmos-sdk fork version * update commit function with latest changes merged from main * add changelog entry * update commit func * fix claimRewards comt --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com> Co-authored-by: stepit <stefanofrancesco.pitton@gmail.com> * chore: update changelog (#2655) * Merge pull request from GHSA-q6hg-6m9x-5g9c * fix(precompile): add funder and dest check to update balance * add more test cases * restrict funder to be origin or contract caller * restrict dest address when calling from sc * add test case with funds transfer within precompile call * remove print * add check for dirty addresses * add UpdateDirties func * update dirties helper func * Update precompiles/vesting/utils_test.go Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> * update comment description * changes based on review comments * refactor updateDirties * add test cases for updating funder balance before and after precompile call * add revert state test * update tx logic * update integration tests * refactor auth logic * update tests * add comt * fix: add funder and vest acc to dirties * add test cases for latest changes * address review comments * address review comments * address review comments * fix FlashLoan contract compilation * update logic * use pointer in precompile txs * remove unnecessary comment --------- Signed-off-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> * chore: update CHANGELOG & fix linter (#2657) * add changelog entry * run make format * fix lint warnings * update known exceptions * fix lint warnings * fix lint warnings * fix lint warnings * update solidity test suite deps --------- Co-authored-by: GAtom22 <GAtom22@users.noreply.github.com> * merge 'main' to GAtom22/sdk50 * [Snyk] Security upgrade golang from 1.22.3-bullseye to 1.23rc1-bullseye (#2656) fix: tests/e2e/Dockerfile.repo to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159417 - https://snyk.io/vuln/SNYK-DEBIAN11-GNUTLS28-6159419 - https://snyk.io/vuln/SNYK-DEBIAN11-LIBSSH2-5861756 - https://snyk.io/vuln/SNYK-DEBIAN11-CURL-3320493 - https://snyk.io/vuln/SNYK-DEBIAN11-ZLIB-6008961 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> * feat(vm): move geth vm from fork to evmOS codebase (#2663) * temp * run make format * fix unit tests * fix imports * add changelog entry * exclude x/evm/core from license checker * remove unnecessary rawdb * remove unused runtime folder * semgrep: skip iteration if map is empty * add new x/evm/folder to ignored path in golangci-lint * feat(evm): add custom EIPs to VM (#2629) * chore: add custom eips integration tests (#2661) * update custom eips + test * move eips to eips package * add eips tests * run make format * revert an error * add missing license to file * remove unused func * enable extra eip and refactor tests * add readme * Apply suggestions from code review Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * update doc and remove unused default eips * make markdown linter happy * fix lint --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: 0xstepit <0xstepit@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * chore: merge main (#2662) * chore: clean changelog (#2628) remove duplicate and fix style * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0 (#2631) * build(deps): bump github.com/cosmos/ibc-go/v7 Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.5.2-0.20240607065443-0f1cf8bf766b to 7.6.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/v7.6.0/CHANGELOG.md) - [Commits](https://github.com/cosmos/ibc-go/commits/v7.6.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * bump sdk and add chlog entry --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * build(deps-dev): bump ws from 7.5.9 to 7.5.10 in /contracts in the npm_and_yarn group across 1 directory (#2634) build(deps-dev): bump ws Bumps the npm_and_yarn group with 1 update in the /contracts directory: [ws](https://github.com/websockets/ws). Updates `ws` from 7.5.9 to 7.5.10 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/7.5.9...7.5.10) --- updated-dependencies: - dependency-name: ws dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * imp(tests): Add function to get ERC-20 balance to integration utils (#2635) * add get erc20 balance util * add changelog entry * add missing license * build(deps): bump bufbuild/buf-setup-action from 1.33.0 to 1.34.0 (#2637) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.33.0 to 1.34.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](https://github.com/bufbuild/buf-setup-action/compare/v1.33.0...v1.34.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 (#2638) * build(deps): bump github.com/linxGnu/grocksdb from 1.9.1 to 1.9.2 Bumps [github.com/linxGnu/grocksdb](https://github.com/linxGnu/grocksdb) from 1.9.1 to 1.9.2. - [Release notes](https://github.com/linxGnu/grocksdb/releases) - [Commits](https://github.com/linxGnu/grocksdb/compare/v1.9.1...v1.9.2) --- updated-dependencies: - dependency-name: github.com/linxGnu/grocksdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * update gomod2nix.toml file * update rocksdb version --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: stepit <48993133+0xstepit@users.noreply.github.com> Co-authored-by: tom <tomasguerraalda@hotmail.com> * fix(ci): Ignore Swagger docs in CheckOV linter (#2639) * ignore client folder in checkov CI * add changelog entry * add minor change in go file to trigger cla flow * Add: Support PebbleDB image (#2630) * add: config & workflow step for building pebble images * test: comment out GIT_DIFF condition in build workflow * Revert "test: comment out GIT_DIFF condition in build workflow" * test: force a workflow trigger to push test images * Revert "test: force a workflow trigger to push test images" * docs: update changelog * Update: lint * Revert: build.yml * Update CHANGELOG.md Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> --------- Signed-off-by: Amine <34750005+Aminechakr@users.noreply.github.com> Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fix(tests): Bump Dockerfile dependency and fix make test-e2e target in Makefile (#2642) * fix dockerfile and makefile * fix e2e tests * add changelog entry * imp(evm): Remove EthAccount type and use BaseAccount instead (#2633) * remove use of EthAccount in repository * remove EthAccount implementation * add missing license * wip adding migration logic * fix migration * add tests for migration * run make format * Update x/evm/genesis.go Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> * address review comments and add test for exporting EVM genesis * add iterator and corresponding test * use contract iterator for exporting EVM genesis state * address more review comments * remove unused method from interface definition * apply suggested renaming * address review comments * add changelog entry * add gitleaks allow directive to example contract storage * move changelog entry to breaking section * revert change in local node script --------- Signed-off-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com> Co-authored-by: MalteHerrmann <MalteHerrmann@users.noreply.github.com> * imp(dist): Improve efficiency of reward claiming with distribution precompile (#2643) * improve reward claim method * add changelog entry * address review comments --------- Signed-off-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 (#2641) * build(deps): bump github.com/btcsuite/btcd from 0.24.0 to 0.24.2 Bumps [github.com/btcsuite/btcd](https://github.com/btcsuite/btcd) from 0.24.0 to 0.24.2. - [Release notes](https://github.com/btcsuite/btcd/releases) - [Changelog](https…
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR removes the
EthAccounttype which was creating unnecessary complication for importing our modules into customer codebases that are not started as an Ethermint or Evmos based chain.The
EthAccountonly held the associated code hash of a potential smart contract at this address as extra information over the underlyingBaseAccounttype. This information can easily be stored in a separate EVM store with a corresponding code hash prefix.Summary by CodeRabbit
New Features
Bug Fixes
Refactor
EthAccountwithBaseAccountacross the application to align with the new account standard.Tests