refactor(raiko): prune deprecated forks by smtmfft · Pull Request #676 · taikoxyz/raiko

smtmfft · 2026-03-07T11:04:16Z

No description provided.

Copilot

Pull request overview

This PR refactors the Raiko codebase toward a Shasta-only workflow by removing deprecated fork support (Pacaya/Ontake/Hekla) across core parsing, host APIs, and utilities, and by updating stress tooling to more efficiently discover proposal boundaries.

Changes:

Remove legacy fork/batch proving paths (including v3 /proof/batch handler and Pacaya-related utilities/types) in favor of Shasta proposal flow.
Update Shasta stress tooling to cache anchor parsing results and use bounded/binary searches to discover proposal ranges.
Adjust default chain specs/config/tests to reflect Shasta-focused defaults (e.g., taiko_a7 removal, verifier updates).

Reviewed changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
script/stress_shasta_proposal.py	Adds anchor-info caching and new boundary/proposal-group discovery logic; updates Raiko submission context.
script/prove-shasta.sh	Updates usage text to reference `taiko_hoodi` instead of `taiko_a7`.
reqactor/src/backend.rs	Removes legacy batch-request Raiko construction (now hard-errors).
lib/src/utils/txs.rs	Drops Pacaya tx generation; Shasta-only batch tx generation.
lib/src/utils/pacaya.rs	Deletes Pacaya tx generation implementation.
lib/src/utils.rs	Removes `pacaya` module export.
lib/src/protocol_instance.rs	Removes non-Shasta protocol-instance logic; Shasta-only batch instance hashing/verification remains.
lib/src/input/{pacaya,ontake,hekla}.rs	Deletes deprecated fork ABI/input definitions.
lib/src/input.rs	Makes `BlockProposedFork` Shasta-only; prunes legacy fork-specific helpers.
lib/src/consts.rs	Updates fork/verifier tests and adds assertions for shasta-only defaults.
host/src/server/api/v3/proof/mod.rs	Removes legacy batch handler routes/docs; keeps Shasta batch endpoint.
host/src/server/api/v3/proof/batch_handler.rs	Deletes legacy v3 batch handler.
host/src/server/api/v3/proof/batch/pacaya.rs	Deletes Pacaya batch request processing helper.
host/src/server/api/v3/proof/batch/mod.rs	Removes Pacaya batch module export.
host/config/config.taiko_hoodi.json	Removes legacy fork SGX instance-id entries.
host/config/chain_spec_list_default.json	Removes Holesky + taiko_a7; prunes PACAYA L1 contract entries; adds/keeps hoodi + shasta taiko networks.
core/src/preflight/util.rs	Removes single-block + Pacaya preflight paths; Shasta-only event decode/filter behavior.
core/src/preflight/mod.rs	Stops exporting Pacaya batch proposal parsing entrypoint.
core/src/preflight/lru.rs	Makes test-only cache-clear helper `#[cfg(test)]`.
core/src/lib.rs	Removes Pacaya batch proving tests; adds a Shasta fixture regression test.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 22 out of 22 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 48 out of 49 changed files in this pull request and generated 6 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 65 out of 68 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

host/src/server/api/v3/proof/report.rs:18

OpenAPI annotation declares post for /proof/report, but the router registers this handler with routing::get. This will generate incorrect API docs/clients; update the #[utoipa::path(...)] method to get (or change the route).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 85 out of 87 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…ly mode - Removed all references to the pacaya fork, including related functions and test cases. - Updated the tests to ensure compatibility with shasta-only specifications. - Adjusted chain specifications to reflect the removal of pacaya and ensure proper handling of shasta proposals. - Cleaned up unused imports and code related to pacaya processing in various modules.

- core: extract helpers (build_batch_blocks, verify_parent_hash_chain, get_prover_config), fix EventFilterConditioin typo, add constants - lib: extract reth_chain_spec_for_name, active_fork_spec_id, simplify matches!/clamp/is_empty, dedupe keccak in mpt, add docs Made-with: Cursor

Made-with: Cursor

- Removed unused BatchMetadata and BatchProofRequest structs from interfaces.rs. - Simplified various function implementations and improved readability across multiple files. - Adjusted logging statements for better clarity and consistency. - Enhanced type handling and reduced complexity in preflight and provider modules. Made-with: Cursor

- Added patterns for .env files, log files, and patch files to .gitignore. - Updated submodule reference for gaiko.

- Removed AggregationGuestInput and its associated handling from multiple files, including interfaces, prover, and SGX modules. - Updated Dockerfile to exclude the boundless aggregation binary. - Cleaned up environment variable configurations in sample files. - Streamlined the entrypoint script by removing unnecessary instance ID updates. - Adjusted test imports to reflect the removal of aggregation input handling. This refactor simplifies the codebase by eliminating unused components and improving overall clarity.

- Changed L1_NETWORK references from "holesky" to "hoodi" in various configuration files, scripts, and documentation. - Updated environment variable configurations in .env files and Docker Compose files to reflect the new L1 network. - Adjusted related scripts to ensure compatibility with the new network naming. This update aligns the codebase with the current network naming conventions and improves clarity in configuration settings.

Copilot

Pull request overview

Copilot reviewed 126 out of 128 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (2)

host/src/server/api/v3/proof/list.rs:17

This handler is wired into the router but still calls todo!(), which will panic on the first request to /v3/proof/list (and /proof/list). Either implement the listing logic or remove/disable the route until it’s ready (e.g., return a structured error response instead of panicking).
host/src/server/api/v3/proof/report.rs:85
The OpenAPI annotation declares this as POST /proof/report, but the actual router registers GET /proof/report. This mismatch will generate incorrect docs/clients; update the #[utoipa::path(...)] method to get (or change the router to post) so the spec matches runtime behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 126 out of 128 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (2)

host/src/server/api/v3/proof/report.rs:86

The OpenAPI annotation declares this endpoint as post (#[utoipa::path(post, ...)]), but the router registers it with get(report_handler). This will generate incorrect API docs / client bindings; update the utoipa::path method to get to match the actual route.
host/src/server/api/v3/proof/list.rs:17
list_handler is wired into the router but still contains todo!(), which will panic and return 500 on any request. Either implement the listing behavior now, or return an explicit “not implemented” response (e.g., a structured Status::Error / 501) until it’s ready.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings March 7, 2026 11:04

Copilot started reviewing on behalf of smtmfft March 7, 2026 11:04 View session

Copilot AI reviewed Mar 7, 2026

View reviewed changes

Comment thread lib/src/input.rs Outdated

Comment thread host/config/chain_spec_list_default.json

Comment thread host/config/chain_spec_list_default.json

Comment thread script/stress_shasta_proposal.py

Comment thread script/stress_shasta_proposal.py Outdated

Comment thread lib/src/consts.rs

smtmfft requested a review from Copilot March 8, 2026 05:13

Copilot started reviewing on behalf of smtmfft March 8, 2026 05:13 View session

Copilot AI reviewed Mar 8, 2026

View reviewed changes

Comment thread script/stress_shasta_proposal.py

Comment thread script/stress_shasta_proposal.py Outdated

Comment thread lib/src/protocol_instance.rs Outdated

Comment thread core/src/preflight/util.rs Outdated

Comment thread reqactor/src/backend.rs Outdated

smtmfft requested a review from Copilot March 9, 2026 03:44

Copilot started reviewing on behalf of smtmfft March 9, 2026 03:44 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

smtmfft requested a review from Copilot March 9, 2026 06:06

Copilot started reviewing on behalf of smtmfft March 9, 2026 06:06 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

smtmfft requested a review from Copilot March 10, 2026 01:44

Copilot started reviewing on behalf of smtmfft March 10, 2026 01:45 View session

Copilot AI reviewed Mar 10, 2026

View reviewed changes

Comment thread lib/src/utils/txs.rs

smtmfft added 13 commits April 7, 2026 10:12

fix: fix warnings and refine script log

a1bec77

fix: fix review comments

217ee6d

chore: keep pruning

384c2db

chore: keep pruning

fb99c56

fix: fix CI

e17faff

chore: refine by code simplifier

cffd030

refactor: streamline action handling and improve queue management

87a5bfa

Made-with: Cursor

feat: add alert script

f25d57e

chore: refine script

21c6d39

chore: update .gitignore and submodule reference

6143d5c

- Added patterns for .env files, log files, and patch files to .gitignore. - Updated submodule reference for gaiko.

smtmfft force-pushed the prune-old-forks branch from d35ec24 to 6143d5c Compare April 7, 2026 02:21

smtmfft added 5 commits April 8, 2026 10:27

chore: cargo fmt

f2c4927

fix: fix CI

8f57b5c

fix: add missing script back

d80b97a

chore: fix typo

3c58447

smtmfft requested a review from Copilot April 8, 2026 06:33

Copilot started reviewing on behalf of smtmfft April 8, 2026 06:34 View session

Copilot AI reviewed Apr 8, 2026

View reviewed changes

Comment thread reqpool/src/memory_backend.rs

chore: fix typo

c4830fb

johntaiko requested review from Copilot and johntaiko April 8, 2026 13:59

Copilot started reviewing on behalf of johntaiko April 8, 2026 14:00 View session

Copilot AI reviewed Apr 8, 2026

View reviewed changes

Comment thread lib/src/utils/shasta.rs

chore: update gaiko

f2c5515

johntaiko approved these changes Apr 10, 2026

View reviewed changes

smtmfft enabled auto-merge April 10, 2026 07:48

smtmfft added this pull request to the merge queue Apr 10, 2026

Merged via the queue into main with commit 9e13667 Apr 10, 2026
21 of 22 checks passed

smtmfft deleted the prune-old-forks branch April 10, 2026 07:51

smtmfft mentioned this pull request Apr 11, 2026

chore(main): release 1.17.0 #678

Merged

Uh oh!

Conversation

smtmfft commented Mar 7, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants