Skip to content

fix(refactor): skip transient selfdestruct storage wipes#26433

Draft
mediocregopher wants to merge 7 commits into
mainfrom
mediocregopher/filter-transient-selfdestruct-wipes
Draft

fix(refactor): skip transient selfdestruct storage wipes#26433
mediocregopher wants to merge 7 commits into
mainfrom
mediocregopher/filter-transient-selfdestruct-wipes

Conversation

@mediocregopher

@mediocregopher mediocregopher commented Jul 18, 2026

Copy link
Copy Markdown
Member

Prevents streamed hashed-state conversion from emitting a storage wipe when an account is created and self-destructed in the same transaction. Such an account has no prior storage update to clear, and we don't want to support wipes in the engine live-sync code paths.

@github-project-automation github-project-automation Bot moved this to Backlog in Reth Tracker Jul 18, 2026
@mediocregopher mediocregopher added A-trie Related to Merkle Patricia Trie implementation C-bug An unexpected or incorrect behavior labels Jul 18, 2026
@mediocregopher
mediocregopher requested a review from klkvr July 20, 2026 14:00
@mediocregopher mediocregopher removed the C-bug An unexpected or incorrect behavior label Jul 20, 2026
@mediocregopher mediocregopher changed the title fix(trie): skip transient selfdestruct storage wipes fix(refactor): skip transient selfdestruct storage wipes Jul 20, 2026
@mediocregopher mediocregopher self-assigned this Jul 20, 2026
@mediocregopher

Copy link
Copy Markdown
Member Author

cyclops audit

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

cc @mediocregopher

Cyclops audit event published. View workflow run

Config: config: default, iterations: default, hours: default

@mediocregopher

Copy link
Copy Markdown
Member Author

cyclops audit

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

cc @mediocregopher

Cyclops audit event published. View workflow run

Config: config: default, iterations: default, hours: default

@tempoxyz-bot

tempoxyz-bot commented Jul 21, 2026

Copy link
Copy Markdown
Member

👁️ Cyclops Security Review

b8aa2c7

❌ Workflow failed · no review posted

Cyclops failed before it could finish the PR review. This status was repaired by the workflow exit handler.

Field Value
Run label pr-26433
Cause workflow exited with status Error
Workflow cyclops-pr-review-49xqg
Namespace argo-workflows

⚠️ Stale audit: this run targeted b8aa2c7, but the current PR head is 76133c9.

No Cyclops review was posted for this run.

…-transient-selfdestruct-wipes

# Conflicts:
#	crates/trie/parallel/src/state_root_task.rs
@mediocregopher

Copy link
Copy Markdown
Member Author

cyclops audit

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

cc @mediocregopher

Cyclops audit event failed to publish. View workflow run

Config: config: default, iterations: default, hours: default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-trie Related to Merkle Patricia Trie implementation

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants