Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What 💻
Backed::inspect
that was previously removed due to re-entrancy issues.inspect_ctx: Box<dyn Any>
is now added toBackend::inspect
to tightly couple any zk-specific data we'd need to pass.ctx
as it must not be persisted beyond the call (and also immediately used), which isn't guaranteed due to the additional indirection viaBackend::inspect
.CowBackend
to setinspect_ctx
on the inner backend context as it will always clone the&Backend
immediately.inspect
calls directly in executor (as was done previously) but then we diverge from foundry code.Why ✋
Evidence 📷
Tests pass