Skip to content

feat: low gas child frames - #26854

Draft
gkozyryatskyy wants to merge 8 commits into
mainfrom
feature/low-gas-child-frame
Draft

feat: low gas child frames#26854
gkozyryatskyy wants to merge 8 commits into
mainfrom
feature/low-gas-child-frame

Conversation

@gkozyryatskyy

Copy link
Copy Markdown
Contributor

Description:
If we have INSUFFICIENT_GAS child frame, we should charge execution gas from the parent frame

Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
@gkozyryatskyy gkozyryatskyy added this to the v0.79 milestone Aug 14, 2026
@gkozyryatskyy gkozyryatskyy self-assigned this Aug 14, 2026
@lfdt-bot

lfdt-bot commented Aug 14, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...p/service/contract/impl/exec/utils/FrameUtils.java 40.00% 1 Missing and 2 partials ⚠️
...pl/exec/processors/CustomMessageCallProcessor.java 80.00% 0 Missing and 2 partials ⚠️

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #26854   +/-   ##
=========================================
  Coverage     70.44%   70.45%           
- Complexity    11676    11680    +4     
=========================================
  Files          2583     2583           
  Lines        108304   108319   +15     
  Branches      12118    12123    +5     
=========================================
+ Hits          76299    76311   +12     
  Misses        28020    28020           
- Partials       3985     3988    +3     
Files with missing lines Coverage Δ Complexity Δ
...pl/exec/processors/CustomMessageCallProcessor.java 85.41% <80.00%> (-0.30%) 48.00 <0.00> (+2.00) ⬇️
...p/service/contract/impl/exec/utils/FrameUtils.java 86.06% <40.00%> (-1.97%) 60.00 <1.00> (+1.00) ⬇️

... and 1 file with indirect coverage changes

Impacted file tree graph

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

Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
Signed-off-by: Glib Kozyryatskyy <glib.k@swirldslabs.com>
@testlens-app

testlens-app Bot commented Aug 17, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

⚠️ TestLens detected flakiness ⚠️

Test Summary

600: [FLOW] PR Checks / MATS / HAPI Tests / HAPI Tests (State Throttling) > :test-clients:testSubprocess

Test Runs Flakiness
StreamValidationTest > streamsAreValid() > as written ❌ ✅ 2% 🟡

🏷️ Commit: 800f9e1
▶️ Tests: 8 executed
⚪️ Checks: 48/48 completed


Learn more about TestLens at testlens.app.

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