[Bug]: AutoCli cannot serialize Coin positional args

[NeverHappened]

Is there an existing issue for this?

• I have searched the existing issues

What happened?

We're integrating Autocli into neutron and have a problem with Coin as a positional argument.

If you're trying make cli tx with chaind command 1token_a, it returns error.

Can you help me understand? Since I believe there is not much more that can be changed.

Autocli link to code here:
https://github.com/neutron-org/neutron/blob/98f6c79c6092f37aa95f71035ad6841c40f1526d/x/tokenfactory/autocli.go#L58
Proto type:
https://github.com/neutron-org/neutron/blob/e855c8f1901a6954dec97748baabd7f98b5e02bc/proto/osmosis/tokenfactory/v1beta1/tx.proto#L59

I also have problems with some queries returning '{}' empty response, but this is another story

Cosmos SDK Version

0.50.7

How to reproduce?

1. clone neutron: https://github.com/neutron-org/neutron
2. checkout feat/autocli-all branch
3. make build
4. make init it can error, do not mind it for now (you can see working chain logs in tail -f ./data/test-1/test-1.log)
5. ./build/neutrond tx tokenfactory mint "100tokenaa" --from demowallet1 --keyring-backend test --home ./data/test-1 --chain-id test-1 --fees 1000untr

[julienrbrt]

Thanks! I'll check this ASAP.
My first thought on discord was a missing annotation, but I'll dig it further, as for instance, on main it works fine with tx bank send (or any other cosmos.base.v1beta1.Coin)

[NeverHappened]

Yep, I saw that. But in tx bank send it's not a single Coin, but repeated Coins in the proto types.
Is this any different?

[NeverHappened]

@julienrbrt did you had any chance looking at it?

[julienrbrt]

I haven't yet. I am currently off. I'll look when I'm back.

[julienrbrt]

Looking now, sorry for the delay

[julienrbrt]

I am unable to reproduce:

main...julien/autocli-debug

make install
make init-simapp
simd start &
simd q counter count # {}
simd tx counter increase-couter 100stake 5 --yes
simd q counter count # total_count: "5"

[julienrbrt]

Can you make sure you use the latest version of autocli and of x/tx?
Additionally, are you still encountering the problem?

[julienrbrt]

Closing this as stale. Feel free to re-open.

[julienrbrt]

Re-opening as Noble is encountering the same issue:

You can run make build to build the SimApp, and then can test the issue with ./simapp/build/simd tx globalfee update-gas-prices 0.1uusdc
https://github.com/noble-assets/globalfee/tree/autocli
AutoCLI is configured in module.go#138-183

[JulianToledano]

I could not reproduce the issue for the Coin type. However, in the case of Noble, it fails due to the use of DecCoin. Currently, we don’t have a flag for the DecCoin type. As a result, it represents the argument as jsonMessageFlagValue and fails during decoding because it expects a number.