fix: address key rotation CI failures

Author: randygrok

.github/workflows/proto.yml

@@ -15,3 +15,17 @@ jobs:
       - uses: bufbuild/buf-action@v1
         with:
           format: false
+          breaking: false
+      - name: Check protobuf breaking changes
+        env:
+          BASE_SHA: ${{ github.event.pull_request.base.sha || github.event.merge_group.base_sha || github.event.before }}
+        run: |
+          if [ -z "$BASE_SHA" ] || [[ "$BASE_SHA" =~ ^0+$ ]]; then
+            echo "No base SHA available for buf breaking check"
+            exit 0
+          fi
+
+          buf breaking proto \
+            --limit-to-input-files \
+            --error-format github-actions \
+            --against "https://github.com/${{ github.repository }}.git#format=git,commit=${BASE_SHA}"

block/internal/syncing/syncer.go

@@ -750,6 +750,9 @@ func (s *Syncer) trySyncNextBlockWithState(ctx context.Context, event *common.DA
 		s.cache.RemoveHeaderDAIncluded(headerHash)
 		s.cache.RemoveDataDAIncluded(data.DACommitment().String())
 
+		if errors.Is(err, types.ErrUnexpectedProposer) {
+			return errors.Join(errInvalidBlock, err)
+		}
 		if !errors.Is(err, errInvalidState) && !errors.Is(err, errInvalidBlock) {
 			return errors.Join(errInvalidBlock, err)
 		}

docs/index.md

@@ -4,6 +4,8 @@ title: Evolve Documentation
 titleTemplate: ':title'
 ---
 
+<!-- markdownlint-disable MD025 MD033 -->
+
 <script setup>
 import constants from './.vitepress/constants/constants.js'
 </script>

execution/evm/test/execution_test.go

@@ -129,8 +129,9 @@ func TestEngineExecution(t *testing.T) {
 			allTimestamps = append(allTimestamps, blockTimestamp)
 
 			// Execute transactions and get the new state root
-			newStateRoot, err := executionClient.ExecuteTxs(ctx, payload, blockHeight, blockTimestamp, prevStateRoot)
+			executeResult, err := executionClient.ExecuteTxs(ctx, payload, blockHeight, blockTimestamp, prevStateRoot)
 			require.NoError(tt, err)
+			newStateRoot := executeResult.UpdatedStateRoot
 
 			err = executionClient.SetFinal(ctx, blockHeight)
 			require.NoError(tt, err)
@@ -201,8 +202,9 @@ func TestEngineExecution(t *testing.T) {
 
 			// Use timestamp from build phase for each block to ensure proper ordering
 			blockTimestamp := allTimestamps[blockHeight-1]
-			newStateRoot, err := executionClient.ExecuteTxs(ctx, payload, blockHeight, blockTimestamp, prevStateRoot)
+			executeResult, err := executionClient.ExecuteTxs(ctx, payload, blockHeight, blockTimestamp, prevStateRoot)
 			require.NoError(t, err)
+			newStateRoot := executeResult.UpdatedStateRoot
 			if len(payload) == 0 {
 				require.Equal(tt, prevStateRoot, newStateRoot)
 			} else {

types/state.go

@@ -2,6 +2,7 @@ package types
 
 import (
 	"bytes"
+	"errors"
 	"fmt"
 	"sync"
 	"time"
@@ -16,6 +17,10 @@ var InitStateVersion = Version{
 	App:   0,
 }
 
+// ErrUnexpectedProposer is returned when a block was signed by a proposer
+// different from the proposer expected by the current state.
+var ErrUnexpectedProposer = errors.New("unexpected proposer")
+
 // State contains information about current state of the blockchain.
 type State struct {
 	Version Version
@@ -77,7 +82,7 @@ func (s State) AssertValidForNextState(header *SignedHeader, data *Data) error {
 		return fmt.Errorf("header-data validation failed: %w", err)
 	}
 	if len(s.NextProposerAddress) > 0 && !bytes.Equal(header.ProposerAddress, s.NextProposerAddress) {
-		return fmt.Errorf("unexpected proposer - got: %x, want: %x", header.ProposerAddress, s.NextProposerAddress)
+		return fmt.Errorf("%w - got: %x, want: %x", ErrUnexpectedProposer, header.ProposerAddress, s.NextProposerAddress)
 	}
 	return nil
 }