Conversation
There was a problem hiding this comment.
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.
No description provided.