Skip to content

update op succinct to v4.2.0#123

Open
xzav3r wants to merge 1 commit intomainfrom
update-op-succinct
Open

update op succinct to v4.2.0#123
xzav3r wants to merge 1 commit intomainfrom
update-op-succinct

Conversation

@xzav3r
Copy link
Contributor

@xzav3r xzav3r commented Mar 12, 2026

No description provided.

Copy link

@claude claude bot left a comment

Choose a reason for hiding this comment

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

LGTM — straightforward devnet config update adding ANCHOR_STATE_REGISTRY_ADDRESS for op-succinct v4.2.0 and fixing a port conflict.

Extended reasoning...

Overview

This PR updates 4 devnet configuration files to support op-succinct v4.2.0. It adds a new ANCHOR_STATE_REGISTRY_ADDRESS config parameter to the example env files for both the proposer and challenger, wires the dynamic value through the existing setup script (5-run-op-succinct.sh), fixes a host port conflict on port 9001 between op-succinct-challenger and op-reth-seq, and adds RUST_LOG=info to the challenger service to match the proposer.

Security risks

None. All changes are confined to devnet configuration files. The private keys in the env files are well-known test keys, not production credentials. No new attack surface is introduced.

Level of scrutiny

Low scrutiny is appropriate. These are mechanical devnet/docker-compose config changes that follow established patterns already present in the codebase. The $ANCHOR_STATE_REGISTRY variable used in the script is already fetched earlier in the same file (via cast call).

Other factors

No bugs were found by the automated bug hunting system. The changes are small (4 files, ~10 lines changed), self-contained, and consistent with the existing configuration patterns in the repository.

@xzav3r xzav3r requested review from googgoog and zjg555543 March 13, 2026 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant