fix: resolve CI lint and benchmark failures

Author: randygrok

README.md

@@ -58,7 +58,7 @@ Check out our tutorials on our [website][docs].
 
 ## Onboarding with Claude Code
 
-Use [Claude Code](https://claude.ai/code) to explore the codebase:
+Use [Claude Code](https://code.claude.com/docs/en/overview) to explore the codebase:
 
 ```bash
 cd ev-node && claude

block/internal/syncing/syncer_test.go

@@ -462,7 +462,7 @@ func TestSyncer_TrySyncNextBlock_SelectiveCacheCleanup(t *testing.T) {
 	}
 
 	tests := map[string]struct {
-		event           func(testing.TB) common.DAHeightEvent
+		event             func(testing.TB) common.DAHeightEvent
 		wantHeaderInCache bool
 		wantDataInCache   bool
 	}{

docs/adr/adr-023-execution-owned-proposer-rotation.md

@@ -21,14 +21,13 @@ The replacement design moves proposer selection into the execution environment.
 - `UpdatedStateRoot`: the state root after executing the block.
 - `NextProposerAddress`: the address expected to sign the next block.
 
-`GetExecutionInfo` also exposes `NextProposerAddress` for startup. If execution returns an empty proposer at startup, ev-node falls back to `genesis.proposer_address`.
-
 An empty `NextProposerAddress` from `ExecuteTxs` means the proposer is unchanged. ev-node must not write a redundant header field in that case, preserving compatibility with existing headers and hash chains.
 
 When execution returns a non-empty next proposer:
 
 - `State.NextProposerAddress` is updated and used as the expected signer for `LastBlockHeight + 1`.
 - Full nodes validate the next block signer against the previous state's `NextProposerAddress`.
+- Nodes recover the expected signer after restart from the persisted state. If legacy state has no stored proposer, ev-node falls back to `genesis.proposer_address`.
 - Header encoding remains unchanged. `Header.ProposerAddress` continues to identify the signer of the current block only.
 
 The execution result is the authority for proposer rotation. Header-only paths cannot derive proposer transitions without either replaying execution or using a future proof/certificate mechanism. This preserves header compatibility while keeping the rotation rule deterministic for full nodes.
@@ -49,7 +48,7 @@ The system contract must restrict writes to the configured authority. Unauthoriz
 
 ev-node validates each block's signer against the proposer address stored in the previous state. A malicious proposer cannot rotate the next signer through node-local configuration; the rotation must be derived from execution.
 
-If the execution interface returns an empty proposer, ev-node treats the proposer as unchanged. At startup, empty execution info falls back to genesis so existing execution implementations remain usable.
+If the execution interface returns an empty proposer from `ExecuteTxs`, ev-node treats the proposer as unchanged. At startup, missing proposer data in legacy state falls back to genesis so existing execution implementations remain usable.
 
 Compromise of the security council can still rotate the proposer to an attacker. This ADR reduces node configuration risk; it does not eliminate governance-key risk.
 

test/e2e/evm_test_common.go

@@ -545,18 +545,6 @@ func SetupCommonEVMEnv(t testing.TB, sut *SystemUnderTest, client tastoratypes.T
 	// Reset global nonce for each test to ensure clean state
 	globalNonce = 0
 
-	// Construct dynamic test ports (rollkit + DA)
-	dynEndpoints, err := generateTestEndpoints()
-	require.NoError(t, err, "failed to generate dynamic test endpoints")
-
-	// Start local DA explicitly on the chosen port
-	localDABinary := "local-da"
-	if evmSingleBinaryPath != "evm" {
-		localDABinary = filepath.Join(filepath.Dir(evmSingleBinaryPath), "local-da")
-	}
-	sut.ExecCmd(localDABinary, "-port", dynEndpoints.DAPort, "-block-time", "1s")
-	t.Logf("Started local DA on port %s", dynEndpoints.DAPort)
-
 	require.NotNil(t, client, "docker client is required")
 	require.NotEmpty(t, networkID, "docker networkID is required")
 	rethNode := evmtest.SetupTestRethNode(t, client, networkID, cfg.rethOpts...)
@@ -577,6 +565,19 @@ func SetupCommonEVMEnv(t testing.TB, sut *SystemUnderTest, client tastoratypes.T
 	genesisHash, err := rethNode.GenesisHash(context.Background())
 	require.NoError(t, err, "failed to get genesis hash")
 
+	// Reserve local ports after Docker has allocated reth host ports so Docker
+	// cannot later claim a port selected for local DA or ev-node listeners.
+	dynEndpoints, err := generateTestEndpoints()
+	require.NoError(t, err, "failed to generate dynamic test endpoints")
+
+	// Start local DA explicitly on the chosen port
+	localDABinary := "local-da"
+	if evmSingleBinaryPath != "evm" {
+		localDABinary = filepath.Join(filepath.Dir(evmSingleBinaryPath), "local-da")
+	}
+	sut.ExecCmd(localDABinary, "-port", dynEndpoints.DAPort, "-block-time", "1s")
+	t.Logf("Started local DA on port %s", dynEndpoints.DAPort)
+
 	// Populate endpoints with both dynamic rollkit ports and dynamic engine ports
 	dynEndpoints.SequencerEthPort = networkInfo.External.Ports.RPC
 	dynEndpoints.SequencerEnginePort = networkInfo.External.Ports.Engine