rename for clarity

Author: julienrbrt

block/internal/submitting/submitter.go

@@ -337,9 +337,9 @@ func (s *Submitter) processDAInclusionLoop() {
 
 				s.logger.Debug().Uint64("height", nextHeight).Msg("advancing DA included height")
 
-				// Set sequencer height to DA height mapping using already retrieved data
-				if err := s.setSequencerHeightToDAHeight(s.ctx, nextHeight, header, data, currentDAIncluded == 0); err != nil {
-					s.logger.Error().Err(err).Uint64("height", nextHeight).Msg("failed to set sequencer height to DA height mapping")
+				// Set node height to DA height mapping using already retrieved data
+				if err := s.setNodeHeightToDAHeight(s.ctx, nextHeight, header, data, currentDAIncluded == 0); err != nil {
+					s.logger.Error().Err(err).Uint64("height", nextHeight).Msg("failed to set node height to DA height mapping")
 					break
 				}
 
@@ -427,14 +427,14 @@ func (s *Submitter) sendCriticalError(err error) {
 	}
 }
 
-// setSequencerHeightToDAHeight stores the mapping from a ev-node block height to the corresponding
+// setNodeHeightToDAHeight stores the mapping from a ev-node block height to the corresponding
 // DA (Data Availability) layer heights where the block's header and data were included.
 // This mapping is persisted in the store metadata and is used to track which DA heights
 // contain the block components for a given ev-node height.
 //
 // For blocks with empty transactions, both header and data use the same DA height since
 // empty transaction data is not actually published to the DA layer.
-func (s *Submitter) setSequencerHeightToDAHeight(ctx context.Context, height uint64, header *types.SignedHeader, data *types.Data, genesisInclusion bool) error {
+func (s *Submitter) setNodeHeightToDAHeight(ctx context.Context, height uint64, header *types.SignedHeader, data *types.Data, genesisInclusion bool) error {
 	headerHash, dataHash := header.Hash(), data.DACommitment()
 
 	headerDaHeightBytes := make([]byte, 8)

block/internal/submitting/submitter_test.go

@@ -191,10 +191,10 @@ func TestSubmitter_setSequencerHeightToDAHeight(t *testing.T) {
 	mockStore.On("SetMetadata", mock.Anything, dataKey, dBz).Return(nil).Once()
 	mockStore.On("SetMetadata", mock.Anything, store.GenesisDAHeightKey, gBz).Return(nil).Once()
 
-	require.NoError(t, s.setSequencerHeightToDAHeight(ctx, 1, h, d, true))
+	require.NoError(t, s.setNodeHeightToDAHeight(ctx, 1, h, d, true))
 }
 
-func TestSubmitter_setSequencerHeightToDAHeight_Errors(t *testing.T) {
+func TestSubmitter_setNodeHeightToDAHeight_Errors(t *testing.T) {
 	ctx := t.Context()
 	cm, st := newTestCacheAndStore(t)
 
@@ -205,11 +205,11 @@ func TestSubmitter_setSequencerHeightToDAHeight_Errors(t *testing.T) {
 	// No cache entries -> expect error on missing header
 	_, ok := cm.GetHeaderDAIncluded(h.Hash().String())
 	assert.False(t, ok)
-	assert.Error(t, s.setSequencerHeightToDAHeight(ctx, 1, h, d, false))
+	assert.Error(t, s.setNodeHeightToDAHeight(ctx, 1, h, d, false))
 
 	// Add header, missing data
 	cm.SetHeaderDAIncluded(h.Hash().String(), 10, 1)
-	assert.Error(t, s.setSequencerHeightToDAHeight(ctx, 1, h, d, false))
+	assert.Error(t, s.setNodeHeightToDAHeight(ctx, 1, h, d, false))
 }
 
 func TestSubmitter_initializeDAIncludedHeight(t *testing.T) {