Bug and crash with Mantra OM

[LoulouCrypto]

I just have a bug with Mantra / OM

tenderduty | 2026/02/02 10:52:09 tenderduty | ✅ loaded bank metadata for chain Nolus from cosmos.directory tenderduty | panic: invalid denom: om tenderduty | tenderduty | goroutine 41 [running]: tenderduty | github.com/cosmos/cosmos-sdk/types.mustValidateDenom(...) tenderduty | github.com/cosmos/cosmos-sdk@v0.45.11/types/coin.go:802 tenderduty | github.com/cosmos/cosmos-sdk/types.NewDecCoinFromDec({0xc000113f34, 0x2}, {0xc000d52ba0?}) tenderduty | github.com/cosmos/cosmos-sdk@v0.45.11/types/dec_coin.go:26 +0xb2 tenderduty | github.com/firstset/tenderduty/v2/td2/utils.ConvertDecCoinToDisplayUnit({0xc0012758f0, 0x1, 0xc000d49d50?}, {{0x0, 0x0}, {0xc00003f410, 0x2, 0x2}, {0xc000113f30, 0x3}, ...}) tenderduty | github.com/firstset/tenderduty/v2/td2/utils/price-conversion.go:311 +0xa2f tenderduty | github.com/firstset/tenderduty/v2/td2.(*ChainConfig).GetValInfo(0xc000438408, 0x1) tenderduty | github.com/firstset/tenderduty/v2/td2/validator.go:224 +0x1147 tenderduty | github.com/firstset/tenderduty/v2/td2.Run.func4(0xc000438408, {0xc0001f3060, 0x6}) tenderduty | github.com/firstset/tenderduty/v2/td2/run.go:120 +0x2dd tenderduty | created by github.com/firstset/tenderduty/v2/td2.Run in goroutine 1 tenderduty | github.com/firstset/tenderduty/v2/td2/run.go:95 +0x53f tenderduty exited with code 2 (restarting)

Seems there is a problem with om

[LoulouCrypto]

I'm using Docker installation and i'm at the last update

[gluckzhang]

Hi @LoulouCrypto , thanks for reporting the issue. Seems that there is indeed a bug in price conversion. I will further check it and see how to fix.

[LoulouCrypto]

Maybe due to the $MANTRA To OM
Or it search for OM and its uom ?

[gluckzhang]

Maybe due to the $MANTRA To OM
Or it search for OM and its uom ?

It is because for Nolus I see that there can be different types of tokens as self-delegation rewards and commission. The current implementation fails to get the denom data for its native token nls. This PR #70 should fix the issue.

By the way, it seems that you are not using the latest version right? We made some improvements in error handling and this issue should not crash Tenderduty even if it fails to convert the tokens into their display denoms.

If you run Tenderduty as a container, you can try the latest tag that includes the fix :) https://hub.docker.com/layers/firstset/tenderduty/latest/

[LoulouCrypto]

I use it a docker
I will try to update

[LoulouCrypto]

Ok it stop crash

Now i have this error :

tenderduty  | 2026/02/03 15:15:00 tenderduty |  🧊 mantra-1     block 12284600
tenderduty  | 2026/02/03 15:15:00 tenderduty |  cannot convert rewards/commission to display unit for Mantra, err: invalid display denom "om" for base "uom": invalid denom: om
tenderduty  | 2026/02/03 15:15:00 tenderduty |  invalid target denom "om" for Mantra: invalid denom: om; falling back to "uom"
tenderduty  | 2026/02/03 15:15:02 tenderduty |  cannot convert rewards/commission to display unit for Mantra, err: invalid display denom "om" for base "uom": invalid denom: om
tenderduty  | 2026/02/03 15:15:02 tenderduty |  invalid target denom "om" for Mantra: invalid denom: om; falling back to "uom"
tenderduty  | 2026/02/03 15:15:03 tenderduty |  ❌ warning      Snow Fall missed block 20967001 on gravity-bridge-3
tenderduty  | 2026/02/03 15:15:04 tenderduty |  cannot convert rewards/commission to display unit for Mantra, err: invalid display denom "om" for base "uom": invalid denom: om
tenderduty  | 2026/02/03 15:15:04 tenderduty |  invalid target denom "om" for Mantra: invalid denom: om; falling back to "uom"
tenderduty  | 2026/02/03 15:15:05 tenderduty |  🧊 osmosis-1    block 54363000
tenderduty  | 2026/02/03 15:15:05 tenderduty |  🧊 laozi-mainnet block 57142120
tenderduty  | 2026/02/03 15:15:06 tenderduty |  cannot convert rewards/commission to display unit for Mantra, err: invalid display denom "om" for base "uom": invalid denom: om
tenderduty  | 2026/02/03 15:15:06 tenderduty |  invalid target denom "om" for Mantra: invalid denom: om; falling back to "uom"
tenderduty  | 2026/02/03 15:15:08 tenderduty |  cannot convert rewards/commission to display unit for Mantra, err: invalid display denom "om" for base "uom": invalid denom: om
tenderduty  | 2026/02/03 15:15:08 tenderduty |  invalid target denom "om" for Mantra: invalid denom: om; falling back to "uom"

[gluckzhang]

Oh I see. There is another issue about denom validations. In Cosmos SDK, denominations should be 3 ~ 128 characters long (ref) and om is treated as an invalid denom when the code tries to convert the numbers from uom to om.

I will see how to fix this, and also maybe remove this error log. The error does not affect the functionality, but it looks quite annoying 😅

[LoulouCrypto]

Oh I see. There is another issue about denom validations. In Cosmos SDK, denominations should be 3 ~ 128 characters long (ref) and om is treated as an invalid denom when the code tries to convert the numbers from uom to om.

I will see how to fix this, and also maybe remove this error log. The error does not affect the functionality, but it looks quite annoying 😅

Ah ah the logs are not annoying ;)

BTW Thx for the work !!!