STR-453: Rework strata-primitives Buf serialization to get rid of 0x prefix

[evgenyzdanovich]

Description

• remove 0x prefix from strata-primitives Buf serialization,
• remove revm-primitives and reth-primitives from deps list of strata-primitives,
• adjust fntests.

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactor
• New or updated tests
• Dependency Update

Notes to Reviewers

Checklist

• I have performed a self-review of my code.
• I have commented my code where necessary.
• I have updated the documentation if needed.
• My changes do not introduce new warnings.
• I have added tests that prove my changes are effective or that my feature works.
• New and existing tests pass with my changes.

Related Issues

[evgenyzdanovich]

@delbonis
Got rid of reth-primitives and revm-primitives to simplify deps list of strata-primitives.
Tbh, I got somewhat controversial feelings about that, the usage of Buf became little less nice where a conversion to FixedBytes is needed. Would be glad to hear back your opinion.

Please take a look.

[delbonis]

Fantastic! Thank you!

[codecov]

Codecov Report

Attention: Patch coverage is 75.69444% with 35 lines in your changes missing coverage. Please review.

Project coverage is 54.85%. Comparing base (a0d413d) to head (0846553).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/primitives/src/macros.rs	75.00%	30 Missing ⚠️
crates/evmexec/src/fork_choice_state.rs	0.00%	3 Missing ⚠️
bin/prover-client/src/operators/cl_stf.rs	0.00%	1 Missing ⚠️
bin/prover-client/src/operators/evm_ee.rs	0.00%	1 Missing ⚠️

@@            Coverage Diff             @@
##             main     #652      +/-   ##
==========================================
+ Coverage   54.84%   54.85%   +0.01%     
==========================================
  Files         313      313              
  Lines       33526    33639     +113     
==========================================
+ Hits        18386    18452      +66     
- Misses      15140    15187      +47     

Files with missing lines	Coverage Δ
crates/evmexec/src/block.rs	100.00% <100.00%> (ø)
crates/evmexec/src/engine.rs	66.16% <100.00%> (ø)
crates/primitives/src/buf.rs	90.51% <100.00%> (+1.09%)	⬆️
crates/primitives/src/l1/btc.rs	82.56% <100.00%> (-0.05%)	⬇️
crates/proof-impl/evm-ee-stf/src/utils.rs	67.44% <100.00%> (ø)
bin/prover-client/src/operators/cl_stf.rs	0.00% <0.00%> (ø)
bin/prover-client/src/operators/evm_ee.rs	0.00% <0.00%> (ø)
crates/evmexec/src/fork_choice_state.rs	0.00% <0.00%> (ø)
crates/primitives/src/macros.rs	79.73% <75.00%> (-7.67%)	⬇️

... and 6 files with indirect coverage changes

[github-actions]

Commit: b2acd2a

SP1 Performance Test Results

program	cycles	success
BTC_BLOCKSPACE	30,370,673	✅
EL_BLOCK	97,975	✅
CL_BLOCK	91,310	✅
L1_BATCH	30,420,921	✅
L2_BATCH	5,473	✅
CHECKPOINT	28,005	✅

[delbonis]

Force merging to keep things moving.