Shrink the spread of modules

[daviddias]

This is a continuation of the work initiated by ipfs/js-ipfs#2222

Outstanding proposals

• Merge js-libp2p-peer-book into js-libp2p (done on a branch) Merge peer-id & peer-info into one, Merge peer-book into js-libp2p #453
• Merge js-libp2p-peer-id and js-libp2p-peer-info into a new module - js-libp2p-peer
• Merge https://github.com/libp2p/js-libp2p-keychain into js-libp2p
• Deprecate https://github.com/libp2p/js-libp2p-nat-mngr and/or merge into js-libp2p
• Deprecate https://github.com/libp2p/js-libp2p-utils (not used anywhere) seems that it is actually used in unreleased branches

[vasco-santos]

👍

Just want to point out that libp2p/js-libp2p-utils is being used in some transports and the idea would be to continue collecting specific utils (unlikely to change) common to several repos and move them there. As a result, I think we should continue with this module

Regarding the other ones, I can take ownership of getting them done over the next weeks

[daviddias]

Thanks for the clarification, @vasco-santos. I scratched it off the list

[daviddias]

Hi folks! Thought of passing by and doing a check in, here are a few questions:

[daviddias]

Ah, there is one more. What about https://github.com/libp2p/js-libp2p-keychain ? Will it be merged to js-libp2p as well?

[vasco-santos]

libp2p-peer

We still not decided the exact way we are going to do this, but we will get rid of peer-info and keep peer-id as it is, or rename it if appropriate. If everything goes as planned, I should start working on peerstore v2 during this quarter and one of the things we intend to do during that initiative is this.

libp2p-pubsub

Currently both libp2p-floodsub and libp2p-gossipsub extend libp2p-pubsub, which contains the base building blocks for a pubsub system. We have talked before about changing this a bit though. In this case, I would say that we should create a pubsub interface instead, under the libp2p interfaces repo, which would also remove a lot of "duplicated" tests on gossipsub and floodsub. This is not a priority for now, but I think we should get to it when there is significant new work to tackle on pubsub land.

libp2p-nat-mgr

Yes. It is still not being used and as a result as not been moved there yet.

libp2p-keychain

I already created a PR to move the codebase libp2p/js-libp2p#525. However, I agree with jacob that we should change libp2p api to include it, as well as to change its usage. I also plan on taking this work during this quarter

[jacobheun]

Released in 0.28