[sontrinh16]

Summary

Addresses audit findings C-01–C-06 by correcting error propagation, preserving lock state on failure, and preventing ERC20 conversion reverts from halting unbonding.

• Return mint failures and unexpected unbonding lookup errors.
• Enforce denomination checks for zero-balance coins and initialize new locks with their denomination.
• Validate lock/unlock keys before slicing and return decoding errors.
• Apply lock moves to copies, committing only after both remove and add succeed.
• Isolate ERC20 conversion state and gas consumption so failed conversions leave the recipient’s Cosmos coins intact.
• Add regression tests, including a reverting contract exercised through the real EVM, and fix simulated-app keeper wiring.

Compatibility

• GetOrCreateMultiStakingLock now requires a denomination.
• Both lock/unlock key decoders now return an error.
• Stored formats remain unchanged.

Validation

• Full suite passed: go test -tags=test ./...
• Types tests passed after the final lock-move refactor.

[hieuvubk]

Overall look good!
Left one comment for convert erc20 coins

[sontrinh16]

moving to other target branch