Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

Comments

Remove deprecated RuntimeActor from kona-node#2535

Merged
refcell merged 4 commits intomainfrom
copilot/fix-fd535611-2b49-4562-a080-0cc7703b30f4
Jul 23, 2025
Merged

Remove deprecated RuntimeActor from kona-node#2535
refcell merged 4 commits intomainfrom
copilot/fix-fd535611-2b49-4562-a080-0cc7703b30f4

Conversation

Copy link
Contributor

Copilot AI commented Jul 23, 2025

Overview

This PR completely removes the deprecated RuntimeActor from the kona-node service, along with all related functionality and documentation as requested in the issue.

Changes Made

Removed Files

  • docs/docs/pages/node/design/runtime.mdx - Complete Runtime system documentation
  • crates/node/service/src/actors/runtime.rs - RuntimeActor implementation

Modified Files

  • docs/sidebar.ts - Removed Runtime link from navigation menu
  • crates/node/service/src/actors/mod.rs - Removed runtime module and exports
  • crates/node/service/src/lib.rs - Removed RuntimeActor, RuntimeContext, RuntimeState exports
  • crates/node/service/src/service/core.rs - Removed RuntimeActor associated type and runtime_builder method from RollupNodeService trait
  • crates/node/service/src/service/standard/node.rs - Removed RuntimeActor type alias and runtime_builder implementation
  • crates/node/service/src/service/standard/builder.rs - Removed runtime_load_interval field and with_runtime_load_interval method
  • bin/node/src/commands/node.rs - Removed --l1.runtime-config-reload-interval CLI argument and related functionality

What Was Removed

The RuntimeActor system provided dynamic configuration management and protocol version signaling by:

  • Monitoring L1 contracts for runtime parameter changes
  • Fetching configuration from System Config and Protocol Versions contracts
  • Signaling protocol version updates to the execution engine via engine_signalSuperchainV1
  • Managing periodic configuration reload intervals

Verification

  • kona-node-service package compiles successfully
  • kona-node binary compiles successfully
  • ✅ All existing unit tests pass (13/13 tests)
  • ✅ Node binary runs and shows version information correctly
  • ✅ No breaking changes to existing functionality

The removal was surgical and precise, maintaining all existing functionality while cleanly removing the entire deprecated RuntimeActor system.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: refcell <21288394+refcell@users.noreply.github.com>
Copilot AI changed the title [WIP] The RuntimeActor has been deprecated. Can you remove the runtime actor from the kona-node entirely. Remove the Runtime documentation as well in the docs/ directory. Remove deprecated RuntimeActor from kona-node Jul 23, 2025
Copilot AI requested a review from refcell July 23, 2025 20:27
@refcell refcell marked this pull request as ready for review July 23, 2025 20:30
Copilot AI review requested due to automatic review settings July 23, 2025 20:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the deprecated RuntimeActor system from the kona-node service, eliminating dynamic runtime configuration management and protocol version signaling functionality. The removal includes all associated components, documentation, and CLI options while maintaining the existing node service architecture.

  • Complete removal of RuntimeActor implementation and related state management
  • Elimination of runtime configuration loading from L1 contracts
  • Removal of protocol version signaling to execution engine via engine_signalSuperchainV1

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/sidebar.ts Removed Runtime navigation link from documentation menu
docs/docs/pages/node/design/runtime.mdx Deleted complete Runtime system documentation
crates/node/service/src/actors/runtime.rs Removed entire RuntimeActor implementation
crates/node/service/src/actors/mod.rs Removed runtime module exports
crates/node/service/src/lib.rs Removed RuntimeActor, RuntimeContext, RuntimeState exports
crates/node/service/src/service/core.rs Removed RuntimeActor trait association and builder method
crates/node/service/src/service/standard/node.rs Removed RuntimeActor type alias and implementation
crates/node/service/src/service/standard/builder.rs Removed runtime load interval configuration

Copy link
Contributor

@refcell refcell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work

@codecov
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.2%. Comparing base (8c180a6) to head (9d53365).
Report is 2 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 refcell merged commit 5a96316 into main Jul 23, 2025
30 checks passed
@refcell refcell deleted the copilot/fix-fd535611-2b49-4562-a080-0cc7703b30f4 branch July 23, 2025 22:33
github-merge-queue bot pushed a commit that referenced this pull request Jul 28, 2025
## Overview

Removes the mention of the `RuntimeActor` from the docs. It was removed
in #2535
github-merge-queue bot pushed a commit that referenced this pull request Aug 12, 2025
## Description

It seems the AI has missed some references in
#2535. Removes the remaining runtime
libraries in the node.
matrix-rider609op added a commit to matrix-rider609op/kona that referenced this pull request Sep 29, 2025
## Description

It seems the AI has missed some references in
op-rs/kona#2535. Removes the remaining runtime
libraries in the node.
aPTRDgvm5ui3dkEtFYWc added a commit to aPTRDgvm5ui3dkEtFYWc/kona that referenced this pull request Oct 2, 2025
## Description

It seems the AI has missed some references in
op-rs/kona#2535. Removes the remaining runtime
libraries in the node.
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
## Overview

This PR completely removes the deprecated `RuntimeActor` from the
`kona-node` service, along with all related functionality and
documentation as requested in the issue.

## Changes Made

### Removed Files
- **`docs/docs/pages/node/design/runtime.mdx`** - Complete Runtime
system documentation
- **`crates/node/service/src/actors/runtime.rs`** - RuntimeActor
implementation

### Modified Files
- **`docs/sidebar.ts`** - Removed Runtime link from navigation menu
- **`crates/node/service/src/actors/mod.rs`** - Removed runtime module
and exports
- **`crates/node/service/src/lib.rs`** - Removed RuntimeActor,
RuntimeContext, RuntimeState exports
- **`crates/node/service/src/service/core.rs`** - Removed RuntimeActor
associated type and runtime_builder method from RollupNodeService trait
- **`crates/node/service/src/service/standard/node.rs`** - Removed
RuntimeActor type alias and runtime_builder implementation
- **`crates/node/service/src/service/standard/builder.rs`** - Removed
runtime_load_interval field and with_runtime_load_interval method
- **`bin/node/src/commands/node.rs`** - Removed
`--l1.runtime-config-reload-interval` CLI argument and related
functionality

## What Was Removed

The RuntimeActor system provided dynamic configuration management and
protocol version signaling by:
- Monitoring L1 contracts for runtime parameter changes
- Fetching configuration from System Config and Protocol Versions
contracts
- Signaling protocol version updates to the execution engine via
`engine_signalSuperchainV1`
- Managing periodic configuration reload intervals

## Verification

- ✅ `kona-node-service` package compiles successfully
- ✅ `kona-node` binary compiles successfully  
- ✅ All existing unit tests pass (13/13 tests)
- ✅ Node binary runs and shows version information correctly
- ✅ No breaking changes to existing functionality

The removal was surgical and precise, maintaining all existing
functionality while cleanly removing the entire deprecated RuntimeActor
system.

<!-- START COPILOT CODING AGENT TIPS -->
---

💡 You can make Copilot smarter by setting up custom instructions,
customizing its development environment and configuring Model Context
Protocol (MCP) servers. Learn more [Copilot coding agent
tips](https://gh.io/copilot-coding-agent-tips) in the docs.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: refcell <21288394+refcell@users.noreply.github.com>
Co-authored-by: refcell <abigger87@gmail.com>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
## Overview

Removes the mention of the `RuntimeActor` from the docs. It was removed
in op-rs/kona#2535
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
…rs/kona#2566)

## Description

It seems the AI has missed some references in
op-rs/kona#2535. Removes the remaining runtime
libraries in the node.
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
## Overview

This PR completely removes the deprecated `RuntimeActor` from the
`kona-node` service, along with all related functionality and
documentation as requested in the issue.

## Changes Made

### Removed Files
- **`docs/docs/pages/node/design/runtime.mdx`** - Complete Runtime
system documentation
- **`crates/node/service/src/actors/runtime.rs`** - RuntimeActor
implementation

### Modified Files
- **`docs/sidebar.ts`** - Removed Runtime link from navigation menu
- **`crates/node/service/src/actors/mod.rs`** - Removed runtime module
and exports
- **`crates/node/service/src/lib.rs`** - Removed RuntimeActor,
RuntimeContext, RuntimeState exports
- **`crates/node/service/src/service/core.rs`** - Removed RuntimeActor
associated type and runtime_builder method from RollupNodeService trait
- **`crates/node/service/src/service/standard/node.rs`** - Removed
RuntimeActor type alias and runtime_builder implementation
- **`crates/node/service/src/service/standard/builder.rs`** - Removed
runtime_load_interval field and with_runtime_load_interval method
- **`bin/node/src/commands/node.rs`** - Removed
`--l1.runtime-config-reload-interval` CLI argument and related
functionality

## What Was Removed

The RuntimeActor system provided dynamic configuration management and
protocol version signaling by:
- Monitoring L1 contracts for runtime parameter changes
- Fetching configuration from System Config and Protocol Versions
contracts
- Signaling protocol version updates to the execution engine via
`engine_signalSuperchainV1`
- Managing periodic configuration reload intervals

## Verification

- ✅ `kona-node-service` package compiles successfully
- ✅ `kona-node` binary compiles successfully  
- ✅ All existing unit tests pass (13/13 tests)
- ✅ Node binary runs and shows version information correctly
- ✅ No breaking changes to existing functionality

The removal was surgical and precise, maintaining all existing
functionality while cleanly removing the entire deprecated RuntimeActor
system.

<!-- START COPILOT CODING AGENT TIPS -->
---

💡 You can make Copilot smarter by setting up custom instructions,
customizing its development environment and configuring Model Context
Protocol (MCP) servers. Learn more [Copilot coding agent
tips](https://gh.io/copilot-coding-agent-tips) in the docs.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: refcell <21288394+refcell@users.noreply.github.com>
Co-authored-by: refcell <abigger87@gmail.com>
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
…rs/kona#2566)

## Description

It seems the AI has missed some references in
op-rs/kona#2535. Removes the remaining runtime
libraries in the node.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants