refactor: resolved all protocol's todo

[0xEgao]

This pr aims to resolve all marked TODO in protocol module, and fixes #703 partially,

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.74%. Comparing base (776b75d) to head (34aaacb).
⚠️ Report is 88 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master     #725       +/-   ##
===========================================
+ Coverage   68.87%   80.74%   +11.87%     
===========================================
  Files          35       49       +14     
  Lines        4932    14706     +9774     
===========================================
+ Hits         3397    11875     +8478     
- Misses       1535     2831     +1296     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[stark-3k]

We might not need this now because it doesn't really change anything and we will also be upgrading our rust bitcoin version which will resolve the need for a compatibility layer.

[mojoX911]

Yeah this is probably redundant. But as its done already will merge it in to reduce review scope.

[mojoX911]

Ack