feat(prover): improve prover implementation

pkg/sender/sender.go

@@ -151,7 +151,7 @@ func (s *Sender) Close() {
 }
 
 // GetOpts returns the transaction options of the sender.
-func (s *Sender) GetOpts() *bind.TransactOpts {
+func (s *Sender) GetOpts(ctx context.Context) *bind.TransactOpts {
 	return &bind.TransactOpts{
 		From:      s.opts.From,
 		Nonce:     s.opts.Nonce,
@@ -161,7 +161,7 @@ func (s *Sender) GetOpts() *bind.TransactOpts {
 		GasFeeCap: s.opts.GasFeeCap,
 		GasTipCap: s.opts.GasTipCap,
 		GasLimit:  s.opts.GasLimit,
-		Context:   s.opts.Context,
+		Context:   ctx,
 		NoSend:    s.opts.NoSend,
 	}
 }
@@ -200,6 +200,7 @@ func (s *Sender) GetUnconfirmedTx(txID string) *types.Transaction {
 
 // SendRawTransaction sends a transaction to the given Ethereum node.
 func (s *Sender) SendRawTransaction(
+	ctx context.Context,
 	nonce uint64,
 	target *common.Address,
 	value *big.Int,
@@ -216,7 +217,7 @@ func (s *Sender) SendRawTransaction(
 
 	var (
 		originalTx types.TxData
-		opts       = s.GetOpts()
+		opts       = s.GetOpts(ctx)
 		gasLimit   = s.GasLimit
 		err        error
 	)

pkg/sender/sender_test.go

@@ -25,7 +25,7 @@ type SenderTestSuite struct {
 
 func (s *SenderTestSuite) TestSendTransaction() {
 	var (
-		opts   = s.sender.GetOpts()
+		opts   = s.sender.GetOpts(context.Background())
 		client = s.RPCClient.L1
 		eg     errgroup.Group
 	)
@@ -57,7 +57,7 @@ func (s *SenderTestSuite) TestSendTransaction() {
 }
 
 func (s *SenderTestSuite) TestSendRawTransaction() {
-	nonce, err := s.RPCClient.L1.NonceAt(context.Background(), s.sender.GetOpts().From, nil)
+	nonce, err := s.RPCClient.L1.NonceAt(context.Background(), s.sender.Address(), nil)
 	s.Nil(err)
 
 	var eg errgroup.Group
@@ -66,7 +66,14 @@ func (s *SenderTestSuite) TestSendRawTransaction() {
 		i := i
 		eg.Go(func() error {
 			addr := common.BigToAddress(big.NewInt(int64(i)))
-			_, err := s.sender.SendRawTransaction(nonce+uint64(i), &addr, big.NewInt(1), nil, nil)
+			_, err := s.sender.SendRawTransaction(
+				context.Background(),
+				nonce+uint64(i),
+				&addr,
+				big.NewInt(1),
+				nil,
+				nil,
+			)
 			return err
 		})
 	}
@@ -82,7 +89,7 @@ func (s *SenderTestSuite) TestSendRawTransaction() {
 func (s *SenderTestSuite) TestReplacement() {
 	send := s.sender
 	client := s.RPCClient.L1
-	opts := send.GetOpts()
+	opts := send.GetOpts(context.Background())
 
 	// Let max gas price be 2 times of the gas fee cap.
 	send.MaxGasFee = opts.GasFeeCap.Uint64() * 2
@@ -108,18 +115,19 @@ func (s *SenderTestSuite) TestReplacement() {
 		Value:     big.NewInt(1),
 		Data:      nil,
 	}
-	rawTx, err := send.GetOpts().Signer(send.GetOpts().From, types.NewTx(baseTx))
+	rawTx, err := opts.Signer(opts.From, types.NewTx(baseTx))
 	s.Nil(err)
 	err = client.SendTransaction(context.Background(), rawTx)
 	s.Nil(err)
 
+	ctx := context.Background()
 	// Replace the transaction with a higher nonce.
-	_, err = send.SendRawTransaction(nonce, &common.Address{}, big.NewInt(1), nil, nil)
+	_, err = send.SendRawTransaction(ctx, nonce, &common.Address{}, big.NewInt(1), nil, nil)
 	s.Nil(err)
 
 	time.Sleep(time.Second * 6)
 	// Send a transaction with a next nonce and let all the transactions be confirmed.
-	_, err = send.SendRawTransaction(nonce-1, &common.Address{}, big.NewInt(1), nil, nil)
+	_, err = send.SendRawTransaction(ctx, nonce-1, &common.Address{}, big.NewInt(1), nil, nil)
 	s.Nil(err)
 
 	for _, confirmCh := range send.TxToConfirmChannels() {
@@ -139,7 +147,7 @@ func (s *SenderTestSuite) TestReplacement() {
 func (s *SenderTestSuite) TestNonceTooLow() {
 	client := s.RPCClient.L1
 	send := s.sender
-	opts := s.sender.GetOpts()
+	opts := s.sender.GetOpts(context.Background())
 
 	nonce, err := client.NonceAt(context.Background(), opts.From, nil)
 	s.Nil(err)
@@ -150,7 +158,14 @@ func (s *SenderTestSuite) TestNonceTooLow() {
 		return
 	}
 
-	txID, err := send.SendRawTransaction(nonce-3, &common.Address{}, big.NewInt(1), nil, nil)
+	txID, err := send.SendRawTransaction(
+		context.Background(),
+		nonce-3,
+		&common.Address{},
+		big.NewInt(1),
+		nil,
+		nil,
+	)
 	s.Nil(err)
 	confirm := <-send.TxToConfirmChannel(txID)
 	s.Nil(confirm.Err)

proposer/proposer.go

@@ -322,7 +322,7 @@ func (p *Proposer) ProposeTxList(
 	tx, err := p.txBuilder.Build(
 		ctx,
 		p.tierFees,
-		p.sender.GetOpts(),
+		p.sender.GetOpts(p.ctx),
 		p.IncludeParentMetaHash,
 		txListBytes,
 	)

proposer/proposer_test.go

@@ -159,7 +159,14 @@ func (s *ProposerTestSuite) TestSendProposeBlockTx() {
 	nonce, err := s.RPCClient.L1.PendingNonceAt(context.Background(), s.p.proposerAddress)
 	s.Nil(err)
 
-	txID, err := sender.SendRawTransaction(nonce, &common.Address{}, common.Big1, nil, nil)
+	txID, err := sender.SendRawTransaction(
+		context.Background(),
+		nonce,
+		&common.Address{},
+		common.Big1,
+		nil,
+		nil,
+	)
 	s.Nil(err)
 	tx := sender.GetUnconfirmedTx(txID)
 
@@ -169,14 +176,21 @@ func (s *ProposerTestSuite) TestSendProposeBlockTx() {
 	newTx, err := s.p.txBuilder.Build(
 		context.Background(),
 		s.p.tierFees,
-		sender.GetOpts(),
+		sender.GetOpts(context.Background()),
 		false,
 		encoded,
 	)
 
 	s.Nil(err)
 
-	txID, err = sender.SendRawTransaction(nonce, newTx.To(), newTx.Value(), newTx.Data(), nil)
+	txID, err = sender.SendRawTransaction(
+		context.Background(),
+		nonce,
+		newTx.To(),
+		newTx.Value(),
+		newTx.Data(),
+		nil,
+	)
 	s.Nil(err)
 	newTx = sender.GetUnconfirmedTx(txID)
 

proposer/transaction_builder/calldata_test.go

@@ -13,7 +13,7 @@ func (s *TransactionBuilderTestSuite) TestBuildCalldata() {
 		{Tier: encoding.TierOptimisticID, Fee: common.Big256},
 		{Tier: encoding.TierSgxID, Fee: common.Big256},
 		{Tier: encoding.TierSgxAndZkVMID, Fee: common.Big257},
-	}, s.sender.GetOpts(), false, []byte{1})
+	}, s.sender.GetOpts(context.Background()), false, []byte{1})
 	s.Nil(err)
 	s.Equal(types.DynamicFeeTxType, int(tx.Type()))
 }

prover/event_handler/assignment_expired.go

@@ -51,14 +51,20 @@ func (h *AssignmentExpiredEventHandler) Handle(
 	if err != nil {
 		return err
 	}
-	if proofStatus.IsSubmitted {
-		// If there is already a proof submitted and there is no need to contest
-		// it, we skip proving this block here.
-		if !proofStatus.Invalid || !h.contesterMode {
-			return nil
-		}
+	if !proofStatus.IsSubmitted {
+		go func() {
+			h.proofSubmissionCh <- &proofProducer.ProofRequestBody{Tier: e.Meta.MinTier, Event: e}
+		}()
+		return nil
+	}
+	// If there is already a proof submitted and there is no need to contest
+	// it, we skip proving this block here.
+	if !proofStatus.Invalid || !h.contesterMode {
+		return nil
+	}
 
-		// If there is no contester, we submit a contest to protocol.
+	// If there is no contester, we submit a contest to protocol.
+	go func() {
 		if proofStatus.CurrentTransitionState.Contester == rpc.ZeroAddress {
 			h.proofContestCh <- &proofProducer.ContestRequestBody{
 				BlockID:    e.BlockId,
@@ -67,22 +73,13 @@ func (h *AssignmentExpiredEventHandler) Handle(
 				Meta:       &e.Meta,
 				Tier:       proofStatus.CurrentTransitionState.Tier,
 			}
-
-			return nil
-		}
-
-		go func() {
+		} else {
 			h.proofSubmissionCh <- &proofProducer.ProofRequestBody{
 				Tier:  proofStatus.CurrentTransitionState.Tier + 1,
 				Event: e,
 			}
-		}()
-
-		return nil
-	}
-
-	go func() {
-		h.proofSubmissionCh <- &proofProducer.ProofRequestBody{Tier: e.Meta.MinTier, Event: e}
+		}
 	}()
+
 	return nil
 }

prover/event_handler/block_proposed.go

@@ -150,7 +150,10 @@ func (h *BlockProposedEventHandler) Handle(
 				}
 				return nil
 			},
-			backoff.WithMaxRetries(backoff.NewConstantBackOff(h.backOffRetryInterval), h.backOffMaxRetrys),
+			backoff.WithContext(
+				backoff.WithMaxRetries(backoff.NewConstantBackOff(h.backOffRetryInterval), h.backOffMaxRetrys),
+				ctx,
+			),
 		); err != nil {
 			log.Error("Handle new BlockProposed event error", "error", err)
 		}
@@ -204,7 +207,7 @@ func (h *BlockProposedEventHandler) checkL1Reorg(
 		)
 
 		return fmt.Errorf(
-			"L1 block hash mismatch due to L1 reorg: %s != %s",
+			"l1 block hash mismatch due to L1 reorg: %s != %s",
 			lastL1OriginHeader.Hash(),
 			e.Meta.L1Hash,
 		)
@@ -357,7 +360,7 @@ func (h *BlockProposedEventHandler) checkExpirationAndSubmitProof(
 
 // ========================= Guardian Prover =========================
 
-// NewBlockProposedEventHandlerOps is the options for creating a new BlockProposedEventHandler.
+// NewBlockProposedGuardianEventHandlerOps is the options for creating a new BlockProposedEventHandler.
 type NewBlockProposedGuardianEventHandlerOps struct {
 	*NewBlockProposedEventHandlerOps
 	GuardianProverHeartbeater guardianProverHeartbeater.BlockSenderHeartbeater

prover/event_handler/transition_contested.go

@@ -58,7 +58,7 @@ func (h *TransitionContestedEventHandler) Handle(
 	}
 
 	// Compare the contested transition to the block in local L2 canonical chain.
-	isValidProof, err := isValidProof(
+	isValid, err := isValidProof(
 		ctx,
 		h.rpc,
 		e.BlockId,
@@ -69,7 +69,7 @@ func (h *TransitionContestedEventHandler) Handle(
 	if err != nil {
 		return err
 	}
-	if isValidProof {
+	if isValid {
 		log.Info(
 			"Contested transition is valid to local canonical chain, ignore the contest",
 			"blockID", e.BlockId,

prover/init.go

@@ -51,8 +51,7 @@ func (p *Prover) setApprovalAmount(ctx context.Context, contract common.Address)
 		return nil
 	}
 
-	opts := p.txSender.GetOpts()
-	opts.Context = ctx
+	opts := p.txSender.GetOpts(ctx)
 
 	log.Info("Approving the contract for taiko token", "allowance", p.cfg.Allowance.String(), "contract", contract)
 
@@ -89,7 +88,7 @@ func (p *Prover) setApprovalAmount(ctx context.Context, contract common.Address)
 		return err
 	}
 
-	log.Info("New allowance for the contract", "allowance", allowance.String(), "contract", contract)
+	log.Info("NewProofSubmitter allowance for the contract", "allowance", allowance.String(), "contract", contract)
 
 	return nil
 }
@@ -107,28 +106,22 @@ func (p *Prover) initProofSubmitters(
 		)
 		switch tier.ID {
 		case encoding.TierOptimisticID:
-			producer = &proofProducer.OptimisticProofProducer{DummyProofProducer: new(proofProducer.DummyProofProducer)}
+			producer = &proofProducer.OptimisticProofProducer{}
 		case encoding.TierSgxID:
-			sgxProducer, err := proofProducer.NewSGXProducer(
-				p.cfg.RaikoHostEndpoint,
-				p.cfg.L1HttpEndpoint,
-				p.cfg.L1BeaconEndpoint,
-				p.cfg.L2HttpEndpoint,
-			)
-			if err != nil {
-				return err
-			}
-			if p.cfg.Dummy {
-				sgxProducer.DummyProofProducer = new(proofProducer.DummyProofProducer)
+			producer = &proofProducer.SGXProofProducer{
+				RaikoHostEndpoint: p.cfg.RaikoHostEndpoint,
+				L1Endpoint:        p.cfg.L1HttpEndpoint,
+				L1BeaconEndpoint:  p.cfg.L1BeaconEndpoint,
+				L2Endpoint:        p.cfg.L2HttpEndpoint,
+				Dummy:             p.cfg.Dummy,
 			}
-			producer = sgxProducer
 		case encoding.TierGuardianID:
 			producer = proofProducer.NewGuardianProofProducer(p.cfg.EnableLivenessBondProof)
 		default:
 			return fmt.Errorf("unsupported tier: %d", tier.ID)
 		}
 
-		if submitter, err = proofSubmitter.New(
+		if submitter, err = proofSubmitter.NewProofSubmitter(
 			p.rpc,
 			producer,
 			p.proofGenerationCh,

prover/proof_producer/dummy_producer_test.go

@@ -30,7 +30,7 @@ func TestDummyProducerRequestProof(t *testing.T) {
 	}
 
 	var (
-		producer        = &DummyProofProducer{}
+		producer        = DummyProofProducer{}
 		tier     uint16 = 1024
 		blockID         = common.Big32
 	)

prover/proof_producer/guardian_producer.go

@@ -15,19 +15,13 @@ import (
 // GuardianProofProducer always returns an optimistic (dummy) proof.
 type GuardianProofProducer struct {
 	returnLivenessBond bool
-	*DummyProofProducer
+	DummyProofProducer
 }
 
 func NewGuardianProofProducer(returnLivenessBond bool) *GuardianProofProducer {
-	gp := &GuardianProofProducer{
+	return &GuardianProofProducer{
 		returnLivenessBond: returnLivenessBond,
 	}
-
-	if !returnLivenessBond {
-		gp.DummyProofProducer = new(DummyProofProducer)
-	}
-
-	return gp
 }
 
 // RequestProof implements the ProofProducer interface.
@@ -64,8 +58,3 @@ func (g *GuardianProofProducer) RequestProof(
 func (g *GuardianProofProducer) Tier() uint16 {
 	return encoding.TierGuardianID
 }
-
-// Cancellable implements the ProofProducer interface.
-func (g *GuardianProofProducer) Cancellable() bool {
-	return false
-}

prover/proof_producer/optimistic_producer.go

@@ -12,7 +12,7 @@ import (
 )
 
 // OptimisticProofProducer always returns an optimistic (dummy) proof.
-type OptimisticProofProducer struct{ *DummyProofProducer }
+type OptimisticProofProducer struct{ DummyProofProducer }
 
 // RequestProof implements the ProofProducer interface.
 func (o *OptimisticProofProducer) RequestProof(
@@ -37,8 +37,3 @@ func (o *OptimisticProofProducer) RequestProof(
 func (o *OptimisticProofProducer) Tier() uint16 {
 	return encoding.TierOptimisticID
 }
-
-// Cancellable implements the ProofProducer interface.
-func (o *OptimisticProofProducer) Cancellable() bool {
-	return false
-}