fix!: reportMessageValidationResult to accept peer id string

[twoeths]

Motivation

• In lodestar, passing peer id through thread boundary has to be in string, it has to convert back to a PeerId instance at lodestar side and reportMessageValidationResult to call toString() again, it causes lodestar network thread to waste 2.8% of cpu time

Description

• reportMessageValidationResult to accept peer id string instead, it's a breaking change

part of https://github.com/ChainSafe/lodestar/issues/5604

[codecov-commenter]

Codecov Report

Patch coverage: 25.00% and project coverage change: -0.03 :warning:

Comparison is base (e2505d6) 82.52% compared to head (586ba79) 82.49%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #432 +/- ## ========================================== - Coverage 82.52% 82.49% -0.03% ========================================== Files 48 48 Lines 11968 11968 Branches 1279 1278 -1 ========================================== - Hits 9876 9873 -3 - Misses 2092 2095 +3 ``` | [Impacted Files](https://app.codecov.io/gh/ChainSafe/js-libp2p-gossipsub/pull/432?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ChainSafe) | Coverage Δ | | |---|---|---| | [src/index.ts](https://app.codecov.io/gh/ChainSafe/js-libp2p-gossipsub/pull/432?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ChainSafe#diff-c3JjL2luZGV4LnRz) | `70.00% <25.00%> (-0.11%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.