-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(op-challenger): add EventCall, TxCall and GetSubClaims #63
Conversation
Is there an overall description for this PR instead of implementation details? |
Description added |
I may lack some context, but from the point of view of OP, it seems still quite hard to understand the rationale of the changes from the description alone, which is important if you want it to be accepted by upstream. |
Great, taking another look and rethinking the rationale sounds like a good approach. A clearer explanation will definitely help with upstream acceptance. |
Anymore comments? Are we good to merge? |
deprecated in #84 |
We aim to address the issue of performing an attack transaction on a disputed claim by introducing a feature to decode the claims from calldata/blob in the FaultDisputeGameN.sol contract. The purpose of this feature is to allow the challenger to assess the validity of the claim and decide whether to initiate an attack.
LogFilterer
(FilterLogs
andSubscribeFilterLogs
func) interface forMultiCaller
to stub Event log responseCall
(ToBatchElemCreator
andHandleResult
func) interface for TxCall and EventCall to supporteth_getTransactionByHash
andeth_getFilterLogs
RPCsevent_stub.go
andtx_stub.go
to support mocking RPC testsGetSubClaims
function to retrieve subClaims from callData(done) and blobs(todo)