chore: simple rename of cmd helper, ParseFromCtx to NewClientFromContext

[ramin]

discussed we'd do as a follow up after rpc merge yesterday with @vgonkivs, happy to wholeheartedly rejected here, but my feeling is

NewClientFromContext is a more idiomatic name for this than ParseClientFromCtx

[codecov-commenter]

Codecov Report

Merging #2818 (5ee17e9) into main (0b5fc0b) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2818      +/-   ##
==========================================
- Coverage   51.44%   51.30%   -0.14%     
==========================================
  Files         167      167              
  Lines       10752    10752              
==========================================
- Hits         5531     5516      -15     
- Misses       4742     4755      +13     
- Partials      479      481       +2     

Files	Coverage Δ
cmd/rpc.go	0.00% <0.00%> (ø)

... and 6 files with indirect coverage changes

[Wondertan]

My concern here is that we don't instantiate the client, while New makes it seems like so. Get instead of New would be more appropriate, imo.

However, I do think that we should instantiate the Client in this func in future, rather than creating it elsewhere and passing through ctx, thus not opposed to keep the change.

[ramin]

@Wondertan oh man, i missed that is what was happening, I'm going to modify this so it really does take some of the flag stuff, set in context, and then instantiate the client, will be a little more "what you'd expect" then.

[renaynay]

I'm fine with either way