Skip to content

docs(base-contracts): fix Sepolia batch sender, refresh AggregateVerifier addresses - #1960

Open
kriss39 wants to merge 1 commit into
base:masterfrom
kriss39:fix/base-contracts-sepolia-batcher-verifier
Open

docs(base-contracts): fix Sepolia batch sender, refresh AggregateVerifier addresses#1960
kriss39 wants to merge 1 commit into
base:masterfrom
kriss39:fix/base-contracts-sepolia-batcher-verifier

Conversation

@kriss39

@kriss39 kriss39 commented Sep 13, 2026

Copy link
Copy Markdown

What changed? Why?

Three rows on specifications/reference/base-contracts no longer match what is on chain:

  • Base Sepolia → Batch Sender: 0x6CDEbe94…3E470xfc56E727…8bA3. Sepolia SystemConfig.batcherHash() returns 0xfc56e727…, and that is the only address posting to the 0xff…84532 batch inbox right now.
  • AggregateVerifier (mainnet and Sepolia): the listed addresses are older implementations. DisputeGameFactory.gameImpls(621) (621 is the Portal's respectedGameType) returns 0xeE303bA0…1ba2 on mainnet and 0x48cB1A96…764C on Sepolia, and the games being created today are clones of those.
  • Base Sepolia → FaultDisputeGame (Kona): removed. It was a duplicate of the FaultDisputeGame row above it (same address, gameImpls(0)).

Notes to reviewers

Everything above is a read-only eth_call you can repeat:

# Sepolia SystemConfig 0xf272670e…6194, batcherHash()
cast call 0xf272670eb55e895584501d564AfEB048bEd26194 'batcherHash()(bytes32)' --rpc-url <sepolia>
# mainnet DisputeGameFactory 0x43edB88C…B40e, gameImpls(621)
cast call 0x43edB88C4B80fDD2AdFF2412A7BebF9dF42cB40e 'gameImpls(uint32)(address)' 621 --rpc-url <mainnet>
# Sepolia DisputeGameFactory 0xd6E6dBf4…06E1
cast call 0xd6E6dBf4F7EA0ac412fD8b65ED297e64BB7a06E1 'gameImpls(uint32)(address)' 621 --rpc-url <sepolia>

The mainnet Batch Sender (0x5050f69a…) already matches batcherHash(), so it is untouched. I left the other rows alone (e.g. SystemDictator, the mainnet legacy OptimismMintableERC20Factory) since those are judgement calls rather than wrong values.

How has it been tested?

node scripts/lint-mdx.js docs/specifications/reference/base-contracts.mdx and node scripts/validate-docs-structure.js pass.

Screenshots

N/A (table cell edits only)

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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.

2 participants