-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Description
Problem
The lend markets and lend rates commands validate --chain and --asset manually in the RunE handler (via parseChainAsset), but the flags are not marked as required using MarkFlagRequired. This means:
defi lend markets --helpdoesn't show these flags as requireddefi schema lend marketsdoesn't reflect them as required in machine-readable output- Error messages are less standard — cobra gives a clear
required flag(s) "chain" not setvs the current custom error
Compare with bridge quote which correctly calls MarkFlagRequired for --from, --to, and --asset.
Suggested Fix
Add _ = marketsCmd.MarkFlagRequired("chain") and _ = marketsCmd.MarkFlagRequired("asset")) (and same for ratesCmd) in newLendCommand() in internal/app/runner.go.
Similarly, --protocol is validated manually but could also be marked required.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels