osmosis-labs / osmosis

The AMM Laboratory
https://app.osmosis.zone
Apache License 2.0
888 stars 590 forks source link

[Bug]: Auto cli adds extra commands for certain tx command #8599

Open PaddyMc opened 2 months ago

PaddyMc commented 2 months ago

What happened?

Description

Autocli somehow adds extra commands for certain tx commands. e.g

osmosisd tx | grep - 
  ibc-transfer           IBC fungible token transfer transaction subcommands
  ibc-transfer           IBC fungible token transfer transaction subcommands
  ibc-wasm               IBC wasm manager module transaction subcommands
  ibc-wasm               IBC wasm manager module transaction subcommands
  interchain-accounts    IBC interchain accounts transaction subcommands
  interchain-accounts    IBC interchain accounts transaction subcommands

Steps to complete:

  1. Find the issues between the txCommand() function in root.go and autoCli options
  2. Fix the issues
  3. Submit a PR and merge to main

Osmosis Version

v26.0.0

How to reproduce?

osmosisd tx | grep - 
  ibc-transfer           IBC fungible token transfer transaction subcommands
  ibc-transfer           IBC fungible token transfer transaction subcommands
  ibc-wasm               IBC wasm manager module transaction subcommands
  ibc-wasm               IBC wasm manager module transaction subcommands
  interchain-accounts    IBC interchain accounts transaction subcommands
  interchain-accounts    IBC interchain accounts transaction subcommands
zac-scheiwe commented 2 weeks ago

I noticed we are using both AutoCLI v1 and AutoCLI v2. Could this be the source of the issue?