feat(bin/node): Change sequencer.enabled -> mode

[clabby]

Overview

Changes the sequencer.enabled flag to a more generic mode flag that can be used to specify different modes of operation for the node. Also wires this flag up to the NodeBuilder.

closes #2307

[Copilot]

Pull Request Overview

Updates the node configuration flag from sequencer.enabled to a generic mode option and connects it through the builder.

• Introduces with_mode on RollupNodeBuilder and moves mode into the RollupNode struct
• Defines FromStr for NodeMode to parse CLI input
• Removes duplicate mode assignment in RollupNodeBuilder::build

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File	Description
crates/node/service/src/service/standard/builder.rs	Added with_mode setter and wired mode into RollupNode
crates/node/service/src/service/mode.rs	Implemented FromStr for NodeMode

Comments suppressed due to low confidence (3)

crates/node/service/src/service/mode.rs:18

• Consider defining a custom error type (e.g., ParseNodeModeError) instead of using String so you can implement std::error::Error and provide richer context.

    type Err = String;

crates/node/service/src/service/mode.rs:17

• Add unit tests for parsing NodeMode with valid inputs ("validator", "sequencer") and invalid cases to ensure the FromStr implementation behaves as expected.

impl std::str::FromStr for NodeMode {

crates/node/service/src/service/mode.rs:7

• The NodeMode enum lacks a doc comment. Consider adding a brief description of its purpose and the meaning of each variant for maintainability.

pub enum NodeMode {

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.6%. Comparing base (8e309de) to head (4c0033c).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[refcell]

Nice