Skip to content

feat(verification): Forbid transactions in the mempool that spend from voided outputs that are confirmed#1443

Merged
msbrogli merged 1 commit intomasterfrom
feat/verification-input-voided-and-confirmed
Oct 9, 2025
Merged

feat(verification): Forbid transactions in the mempool that spend from voided outputs that are confirmed#1443
msbrogli merged 1 commit intomasterfrom
feat/verification-input-voided-and-confirmed

Conversation

@msbrogli
Copy link
Member

@msbrogli msbrogli commented Oct 3, 2025

Motivation

After nano is activated, nano transactions that fail execution will be voided and confirmed by blocks. This scenario was impossible before (except for soft voided transaction) but now they should be way more common. This prevents blocks from confirming transactions that will certainly be voided.

Acceptance Criteria

  1. Reject transactions in the mempool that spend from voided outputs that are confirmed.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@msbrogli msbrogli requested a review from jansegre as a code owner October 3, 2025 15:19
@msbrogli msbrogli self-assigned this Oct 3, 2025
@msbrogli msbrogli added the enhancement New feature or request label Oct 3, 2025
@msbrogli msbrogli moved this from Todo to In Progress (WIP) in Hathor Network Oct 3, 2025
@msbrogli msbrogli force-pushed the feat/verification-input-voided-and-confirmed branch from 741d8f3 to 9f32b4c Compare October 3, 2025 15:20
@github-actions
Copy link

github-actions bot commented Oct 3, 2025

🐰 Bencher Report

Branchfeat/verification-input-voided-and-confirmed
Testbedubuntu-22.04

🚨 1 Alert

BenchmarkMeasure
Units
ViewBenchmark Result
(Result Δ%)
Upper Boundary
(Limit %)
sync-v2 (up to 20000 blocks)Latency
minutes (m)
📈 plot
🚷 threshold
🚨 alert (🔔)
2.44 m
(+41.51%)Baseline: 1.73 m
2.07 m
(117.93%)

Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
🚨 view alert (🔔)
2.44 m
(+41.51%)Baseline: 1.73 m
1.55 m
(63.60%)
2.07 m
(117.93%)

🐰 View full continuous benchmarking report in Bencher

@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.97%. Comparing base (2387a69) to head (37061c6).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1443      +/-   ##
==========================================
- Coverage   86.03%   85.97%   -0.07%     
==========================================
  Files         434      434              
  Lines       33739    33742       +3     
  Branches     5291     5292       +1     
==========================================
- Hits        29029    29009      -20     
- Misses       3673     3686      +13     
- Partials     1037     1047      +10     

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

@msbrogli msbrogli force-pushed the feat/verification-input-voided-and-confirmed branch 2 times, most recently from 74cb27e to 58e2b51 Compare October 9, 2025 17:07
@msbrogli msbrogli moved this from In Progress (WIP) to In Progress (Done) in Hathor Network Oct 9, 2025
@msbrogli msbrogli force-pushed the feat/verification-input-voided-and-confirmed branch from 58e2b51 to d1298a5 Compare October 9, 2025 17:20
@msbrogli msbrogli force-pushed the feat/verification-input-voided-and-confirmed branch from d1298a5 to 37061c6 Compare October 9, 2025 18:05
@github-project-automation github-project-automation bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Oct 9, 2025
@glevco glevco moved this from In Review (WIP) to In Review (Done) in Hathor Network Oct 9, 2025
@msbrogli msbrogli merged commit 37061c6 into master Oct 9, 2025
7 of 8 checks passed
@msbrogli msbrogli deleted the feat/verification-input-voided-and-confirmed branch October 9, 2025 18:40
@github-project-automation github-project-automation bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Oct 9, 2025
@jansegre jansegre mentioned this pull request Oct 13, 2025
2 tasks
@jansegre jansegre moved this from Waiting to be deployed to Done in Hathor Network Oct 15, 2025
@jansegre jansegre mentioned this pull request Oct 16, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants