Skip to content

Transaction level execution#2699

Merged
pdrobnjak merged 1 commit intopd/evmc-vmfrom
pd/transaction-level-execution
Jan 14, 2026
Merged

Transaction level execution#2699
pdrobnjak merged 1 commit intopd/evmc-vmfrom
pd/transaction-level-execution

Conversation

@pdrobnjak
Copy link
Copy Markdown
Contributor

@pdrobnjak pdrobnjak commented Jan 14, 2026

Describe your changes and provide context

Refactor of the evmc and geth VM.

First an improvement on the abstraction - we're working on transaction executor level, not on the VM level.

Second, we can leverage the same executor implementation to cover both evmone and geth cases.

Removing everything that's redundant since second point holds true.

Testing performed to validate your change

Ran giga/tests.

@pdrobnjak pdrobnjak self-assigned this Jan 14, 2026
@pdrobnjak pdrobnjak changed the base branch from main to pd/evmc-vm January 14, 2026 15:01
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 14, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedJan 14, 2026, 3:03 PM

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.94%. Comparing base (08055a7) to head (ce9b17b).
⚠️ Report is 2 commits behind head on pd/evmc-vm.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff               @@
##           pd/evmc-vm    #2699      +/-   ##
==============================================
- Coverage       43.69%   41.94%   -1.76%     
==============================================
  Files            1911     1017     -894     
  Lines          159430    84784   -74646     
==============================================
- Hits            69670    35565   -34105     
+ Misses          83360    45875   -37485     
+ Partials         6400     3344    -3056     
Flag Coverage Δ
sei-chain ?
sei-cosmos 38.20% <ø> (-0.01%) ⬇️
sei-db 69.43% <ø> (ø)
sei-tendermint 47.17% <ø> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 904 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pdrobnjak pdrobnjak merged commit 911ffff into pd/evmc-vm Jan 14, 2026
40 of 43 checks passed
@pdrobnjak pdrobnjak deleted the pd/transaction-level-execution branch January 14, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants