refactor: rename HeightChecker to HeightProvider for clarity in execution layer

Author: randygrok

block/internal/executing/executor.go

@@ -588,12 +588,10 @@ func (e *Executor) applyBlock(ctx context.Context, header types.Header, data *ty
 // This is called during initialization to handle crash recovery scenarios where ev-node
 // is ahead of the execution layer.
 func (e *Executor) syncExecutionLayer(ctx context.Context, state types.State) error {
-	// Check if the executor supports height checking
-	execHeightChecker, ok := e.exec.(interface {
-		GetLatestHeight(ctx context.Context) (uint64, error)
-	})
+	// Check if the executor implements HeightProvider
+	execHeightProvider, ok := e.exec.(coreexecutor.HeightProvider)
 	if !ok {
-		e.logger.Debug().Msg("executor does not support height checking, skipping sync")
+		e.logger.Debug().Msg("executor does not implement HeightProvider, skipping sync")
 		return nil
 	}
 
@@ -605,7 +603,7 @@ func (e *Executor) syncExecutionLayer(ctx context.Context, state types.State) er
 		return nil
 	}
 
-	execHeight, err := execHeightChecker.GetLatestHeight(ctx)
+	execHeight, err := execHeightProvider.GetLatestHeight(ctx)
 	if err != nil {
 		return fmt.Errorf("failed to get execution layer height: %w", err)
 	}

block/internal/executing/sync_test.go

@@ -14,7 +14,7 @@ import (
 	"github.com/evstack/ev-node/types"
 )
 
-// MockHeightAwareExecutor is a mock that implements both Executor and HeightChecker
+// MockHeightAwareExecutor is a mock that implements both Executor and HeightProvider
 type MockHeightAwareExecutor struct {
 	mock.Mock
 }

core/execution/execution.go

@@ -86,9 +86,9 @@ type Executor interface {
 	SetFinal(ctx context.Context, blockHeight uint64) error
 }
 
-// HeightChecker is an optional interface that execution clients can implement
+// HeightProvider is an optional interface that execution clients can implement
 // to support height synchronization checks between ev-node and the execution layer.
-type HeightChecker interface {
+type HeightProvider interface {
 	// GetLatestHeight returns the current block height of the execution layer.
 	// This is useful for detecting desynchronization between ev-node and the execution layer
 	// after crashes or restarts.

go.mod

@@ -161,3 +161,5 @@ require (
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 	lukechampine.com/blake3 v1.4.1 // indirect
 )
+
+replace github.com/evstack/ev-node/core => ./core

go.sum

@@ -62,8 +62,6 @@ github.com/envoyproxy/go-control-plane v0.9.0/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymF
 github.com/envoyproxy/go-control-plane v0.9.1-0.20191026205805-5f8ba28d4473/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymFceY/DCBVvsKhRF0iEA4=
 github.com/envoyproxy/go-control-plane v0.9.4/go.mod h1:6rpuAdCZL397s3pYoYcLgu1mIlRU8Am5FuJP05cCM98=
 github.com/envoyproxy/protoc-gen-validate v0.1.0/go.mod h1:iSmxcyjqTsJpI2R4NaDN7+kN2VEUnK/pcBlmesArF7c=
-github.com/evstack/ev-node/core v1.0.0-beta.3 h1:01K2Ygm3puX4m2OBxvg/HDxu+he54jeNv+KDmpgujFc=
-github.com/evstack/ev-node/core v1.0.0-beta.3/go.mod h1:n2w/LhYQTPsi48m6lMj16YiIqsaQw6gxwjyJvR+B3sY=
 github.com/filecoin-project/go-clock v0.1.0 h1:SFbYIM75M8NnFm1yMHhN9Ahy3W5bEZV9gd6MPfXbKVU=
 github.com/filecoin-project/go-clock v0.1.0/go.mod h1:4uB/O4PvOjlx1VCMdZ9MyDZXRm//gkj1ELEbxfI1AZs=
 github.com/flynn/go-shlex v0.0.0-20150515145356-3f9db97f8568/go.mod h1:xEzjJPgXI435gkrCt3MPfRiAkVrwSbHsst4LCFVfpJc=