fix: simulate gasless bundle

core/types/bundle_gasless.go

@@ -12,10 +12,9 @@ type SimulateGaslessBundleArgs struct {
 type GaslessTxSimResult struct {
 	Hash    common.Hash
 	GasUsed uint64
-	Valid   bool
 }
 
 type SimulateGaslessBundleResp struct {
-	Results          []GaslessTxSimResult
+	ValidResults     []GaslessTxSimResult
 	BasedBlockNumber int64
 }

miner/worker_builder.go

@@ -503,36 +503,28 @@ func (w *worker) simulateGaslessBundle(env *environment, bundle *types.Bundle) (
 		env.state.SetTxContext(tx.Hash(), txIdx)
 
 		var (
-			snap           = env.state.Snapshot()
-			gp             = env.gasPool.Gas()
-			valid          = true
-			gasUsed uint64 = 0
+			snap = env.state.Snapshot()
+			gp   = env.gasPool.Gas()
 		)
 
 		receipt, err := core.ApplyTransaction(w.chainConfig, w.chain, &w.coinbase, env.gasPool, env.state, env.header, tx,
 			&env.header.GasUsed, *w.chain.GetVMConfig())
 		if err != nil {
 			env.state.RevertToSnapshot(snap)
 			env.gasPool.SetGas(gp)
-			valid = false
 			log.Error("fail to simulate gasless tx, skipped", "hash", tx.Hash(), "err", err)
 		} else {
 			txIdx++
-		}
 
-		if receipt != nil {
-			gasUsed = receipt.GasUsed
+			result = append(result, types.GaslessTxSimResult{
+				Hash:    tx.Hash(),
+				GasUsed: receipt.GasUsed,
+			})
 		}
-
-		result = append(result, types.GaslessTxSimResult{
-			Hash:    tx.Hash(),
-			GasUsed: gasUsed,
-			Valid:   valid,
-		})
 	}
 
 	return &types.SimulateGaslessBundleResp{
-		Results:          result,
+		ValidResults:     result,
 		BasedBlockNumber: env.header.Number.Int64(),
 	}, nil
 }