fix: keep sequencer for promotable followers and require DA for promotable mode

Promotable nodes start as followers but can be promoted to proposer at
runtime, which hands the sequencer to the aggregator components. The
non-aggregator guard (added to testapp in #3386 and mirrored here for
the EVM app) would leave promotable nodes with a nil sequencer and
break promotion. Create the sequencer for promotable nodes too, and
require a DA address for promotable mode at config validation, same as
aggregator mode.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: tac0turtle

apps/evm/cmd/run.go

@@ -160,7 +160,9 @@ func createSequencer(
 	daClient block.FullDAClient,
 	executor execution.Executor,
 ) (coresequencer.Sequencer, error) {
-	if !nodeConfig.Node.Aggregator {
+	// Only aggregators and promotable nodes need a sequencer; promotable
+	// followers must have one ready before they are promoted to proposer.
+	if !nodeConfig.Node.Aggregator && !nodeConfig.Node.Promotable {
 		return nil, nil
 	}
 

apps/testapp/cmd/run.go

@@ -113,7 +113,9 @@ func createSequencer(
 	genesis genesis.Genesis,
 	executor execution.Executor,
 ) (coresequencer.Sequencer, error) {
-	if !nodeConfig.Node.Aggregator {
+	// Only aggregators and promotable nodes need a sequencer; promotable
+	// followers must have one ready before they are promoted to proposer.
+	if !nodeConfig.Node.Aggregator && !nodeConfig.Node.Promotable {
 		return nil, nil
 	}
 

pkg/cmd/run_node.go

@@ -48,8 +48,8 @@ func ParseStartConfig(cmd *cobra.Command) (rollconf.Config, error) {
 	if err != nil {
 		return rollconf.Config{}, err
 	}
-	if nodeConfig.Node.Aggregator && nodeConfig.GetDAAddress() == "" {
-		return rollconf.Config{}, fmt.Errorf("DA address is required when aggregator mode is enabled")
+	if (nodeConfig.Node.Aggregator || nodeConfig.Node.Promotable) && nodeConfig.GetDAAddress() == "" {
+		return rollconf.Config{}, fmt.Errorf("DA address is required when aggregator or promotable mode is enabled")
 	}
 
 	return nodeConfig, nil

pkg/cmd/run_node_test.go

@@ -175,20 +175,22 @@ func TestAggregatorFlagInvariants(t *testing.T) {
 }
 
 func TestParseStartConfig_AggregatorRequiresDAAddress(t *testing.T) {
-	executor, sequencer, keyProvider, nodeKey, ds, stopDAHeightTicker := createTestComponents(context.Background(), t)
-	defer stopDAHeightTicker()
+	for _, flag := range []string{"--rollkit.node.aggregator", "--rollkit.node.promotable"} {
+		executor, sequencer, keyProvider, nodeKey, ds, stopDAHeightTicker := createTestComponents(context.Background(), t)
+		defer stopDAHeightTicker()
 
-	nodeConfig := rollconf.DefaultConfig()
-	nodeConfig.RootDir = t.TempDir()
+		nodeConfig := rollconf.DefaultConfig()
+		nodeConfig.RootDir = t.TempDir()
 
-	cmd := newRunNodeCmd(t.Context(), executor, sequencer, keyProvider, nodeKey, ds, nodeConfig)
-	require.NoError(t, cmd.ParseFlags([]string{"start", "--rollkit.node.aggregator"}))
+		cmd := newRunNodeCmd(t.Context(), executor, sequencer, keyProvider, nodeKey, ds, nodeConfig)
+		require.NoError(t, cmd.ParseFlags([]string{"start", flag}))
 
-	_, err := ParseConfig(cmd)
-	require.NoError(t, err)
+		_, err := ParseConfig(cmd)
+		require.NoError(t, err)
 
-	_, err = ParseStartConfig(cmd)
-	require.EqualError(t, err, "DA address is required when aggregator mode is enabled")
+		_, err = ParseStartConfig(cmd)
+		require.EqualError(t, err, "DA address is required when aggregator or promotable mode is enabled")
+	}
 }
 
 func TestPromotableFlagInvariants(t *testing.T) {