cli: refactor flag reading

[alexanderbez]

Description

Cleaner and consistent flag reading in ALL CLI auxiliary functions for client.Context building.

1. client.Context field not pre-populated & flag not set -> uses default flag value ✅
2. client.Context field not pre-populated & flag set -> uses set flag value ✅
3. client.Context field pre-populated & flag not set -> uses pre-populated value ✅
4. client.Context field pre-populated & flag set -> uses set flag value ✅

Checking clientCtx allows us to cover cases 1-3 and checking flagSet.Changed allows us to handle case 4.

ref: #6717 (comment)

---

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

• Targeted PR against correct branch (see CONTRIBUTING.md)
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards.
• Wrote unit and integration tests
• Updated relevant documentation (docs/) or specification (x/<module>/spec/)
• Added relevant godoc comments.
• Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• Re-reviewed Files changed in the Github PR explorer
• Review Codecov Report in the comment section below once CI passes

[codecov]

Codecov Report

Merging #6884 into master will increase coverage by 0.02%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master    #6884      +/-   ##
==========================================
+ Coverage   61.40%   61.42%   +0.02%     
==========================================
  Files         517      517              
  Lines       32050    32059       +9     
==========================================
+ Hits        19679    19692      +13     
+ Misses      10802    10799       -3     
+ Partials     1569     1568       -1     

[aaronc]

ACK'ed but removed automerge. I'd leave it up to you how best to manage these flags.