fix: [3273] @libp2p/utils/merge-options is blowing the stack

[SuchitraSwain]

…ironment

Title

Description

Notes & open questions

Change checklist

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation if necessary (this includes comments as well)
• I have added tests that prove my fix is effective or that my feature works

[dozyio]

why dependencies are being added?

[achingbrain]

Thanks for opening this. In addition to @dozyio's comment, in #3273 I suggest removing the mergeOptions function entirely, it's only used in the keychain and to create the libp2p config so is a bit superfluous.

Each component that receives an init object should be in charge of inspecting it and/or setting default values, ideally through shallow (e.g. one level deep) property access, then we don't need the deep-merge functionality of mergeOptions.