Inspector trait design question

[emnul]

I'm curious why EvmContext is not passed to the selfdestruct method within the Inspector trait, but it's passed to all other functions? I noticed even all the way back to when methods in the Inspector took EvmData instead of EvmContext, selfdestruct still only took a target and contract

[emnul]

After doing some digging, I realized that the SELFDESTRUCT opcode halts EVM execution and returns the empty tuple (). See section 9.4.4 of the Ethereum Yellow Paper. With this in mind, my best guess is that EvmContext is not passed to the selfdestruct method because an Inspector should not be able to modify EVM execution via access to EvmContext once this opcode is executed.

@rakita please let me know if my understand is correct here or if I'm missing anything

[rakita]

Hey, the reason for not including Context is rather simple, there were no requests to add it.
If needed you can always use step to filter selfdestruct opcode and act on it

btw there is a Revm tg group. Link is in main page, I would respond there faster