refactor(p2p): use PeerId type instead of str

[jansegre]

Motivation

This is the second refactor in this series. It makes it so Peer.id is a PeerId instead of a str, that way it is already checked to be a valid hash.

Acceptance Criteria

• Change the type of hathor.p2p.peer.Peer.id from str to hathor.p2p.peer_id.PeerId

Checklist

• If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

[codecov]

Codecov Report

Attention: Patch coverage is 98.07692% with 1 line in your changes missing coverage. Please review.

Project coverage is 84.88%. Comparing base (2c817fd) to head (ada19b8).
Report is 2 commits behind head on master.

Files	Patch %	Lines
hathor/p2p/utils.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1114      +/-   ##
==========================================
+ Coverage   84.83%   84.88%   +0.05%     
==========================================
  Files         315      316       +1     
  Lines       24071    24079       +8     
  Branches     3647     3645       -2     
==========================================
+ Hits        20421    20440      +19     
+ Misses       2936     2924      -12     
- Partials      714      715       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

The base branch was changed.

[github-actions]

Bencher

Report	Thu, August 22, 2024 at 14:47:51 UTC
Project	hathor-core
Branch	refactor/p2p-peer-use-peer-id
Testbed	ubuntu-22.04

Click to view all benchmark results

Benchmark	Latency	Latency Results nanoseconds (ns) | (Δ%)	Latency Lower Boundary nanoseconds (ns) | (%)	Latency Upper Boundary nanoseconds (ns) | (%)
sync-v2 (up to 20000 blocks)	✅ (view plot)	104,022,524,163.80 (+0.99%)	92,698,520,376.58 (89.11%)	113,298,191,571.37 (91.81%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help