Skip to content

refactor(rpc): make v8 self contained#3547

Merged
thiagodeev merged 13 commits intomainfrom
thiagodeev/self-contained-rpcv8
Apr 15, 2026
Merged

refactor(rpc): make v8 self contained#3547
thiagodeev merged 13 commits intomainfrom
thiagodeev/self-contained-rpcv8

Conversation

@thiagodeev
Copy link
Copy Markdown
Contributor

@thiagodeev thiagodeev commented Apr 13, 2026

Make RPC v8 self-contained, so that it no longer depends on code from v6 pkg.

@thiagodeev thiagodeev force-pushed the thiagodeev/self-contained-rpcv8 branch from e996f4b to 998860e Compare April 13, 2026 16:50
@thiagodeev thiagodeev changed the title Thiagodeev/self-contained-rpcv8 refactor(rpc): make v8 self contained Apr 13, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

❌ Patch coverage is 88.12155% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.72%. Comparing base (56989f1) to head (c93fbc0).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
rpc/v8/block.go 77.77% 14 Missing ⚠️
rpc/v8/simulation.go 18.18% 9 Missing ⚠️
rpc/v8/events.go 85.71% 4 Missing and 4 partials ⚠️
rpc/v8/sync.go 75.86% 6 Missing and 1 partial ⚠️
rpc/v8/class.go 92.59% 3 Missing and 1 partial ⚠️
rpc/v8/state_update.go 98.50% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3547      +/-   ##
==========================================
+ Coverage   75.62%   75.72%   +0.09%     
==========================================
  Files         397      401       +4     
  Lines       35389    35692     +303     
==========================================
+ Hits        26762    27026     +264     
- Misses       6763     6798      +35     
- Partials     1864     1868       +4     

☔ 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.

@thiagodeev thiagodeev marked this pull request as ready for review April 13, 2026 22:27
@thiagodeev thiagodeev force-pushed the thiagodeev/self-contained-rpcv8 branch from 998860e to c93fbc0 Compare April 14, 2026 12:27
@thiagodeev thiagodeev enabled auto-merge (squash) April 14, 2026 12:27
@thiagodeev thiagodeev merged commit ce20a30 into main Apr 15, 2026
25 checks passed
@thiagodeev thiagodeev deleted the thiagodeev/self-contained-rpcv8 branch April 15, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants