Remove 24-host path functions

[damiannolan]

Summary

Now that MerklePath handles bytes directly we can remove the Path functions in 24-host host.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned

[vishal-kanna]

@damiannolan, I would like to work on this issue

[crodriguezvega]

Thank you, @vishal-kanna!

[vishal-kanna]

@crodriguezvega Can you give more info about this issue?

[vishal-kanna]

@crodriguezvega @damiannolan Are these ChannelPath ChannelCapabilityPath ChannelUpgradeErrorPath path functions?

[crodriguezvega]

Hi @vishal-kanna, sorry for the delay. This issue refers to all the ...Path functions in the files _paths.go in the host package. Most of these functions should now only be used once (in the corresponding ...Key function), so the implementation can be moved to the ...Key function and get rid of the ...Path function. However, there might be some functions are used in more than one place, so maybe we need to see where it's best to keep those. You can open a draft PR and we can advise. Thank you for picking this issue!

[colin-axner]

Hi @vishal-kanna. Any updates or questions? As this work is in our current iteration, we would be happy to take over this work if you no longer have the opportunity to complete the issue

[DimitrisJim]

you can try inlining the Path suffixed functions into the Key ones for time being, some of these might still need to exist (capability one iirc) so can be left for now (that one will be removed soon).

We should be able to just do a bytes.Join too but that might get messier and can be done in follow up.

[colin-axner]

Thanks for your interest @vishal-kanna. I decided to complete the pr