Skip to content

refactor(L1): remove legacy Nitro verifier - #400

Merged
leopoldjoy merged 1 commit into
mainfrom
feat/nitro-validator-c4
Aug 12, 2026
Merged

refactor(L1): remove legacy Nitro verifier#400
leopoldjoy merged 1 commit into
mainfrom
feat/nitro-validator-c4

Conversation

@leopoldjoy

@leopoldjoy leopoldjoy commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the legacy NitroEnclaveVerifier, interface, tests, snapshots, and RISC Zero deployment script
  • remove rollback-only deployment config/output wiring and the unused risc0-ethereum dependency
  • preserve the hinted NitroValidator registration path, SP1 state-proof verification, and historical audit artifacts

Stack

Testing

  • just deps
  • just snapshots
  • just semver-lock
  • just test --match-path test/deploy/SystemDeploy.t.sol
  • just test --match-path test/L1/proofs/TEEProverRegistry.t.sol
  • just test --match-test test_cannotReinitialize_succeeds
  • just test (1,206 passed, 0 failed, 1 skipped)

Linear: https://linear.app/coinbase/issue/CHAIN-4831/contracts-remove-the-legacy-nitro-zk-verifier-after-rollback-window

@leopoldjoy
leopoldjoy force-pushed the feat/nitro-validator-c4 branch from 8f3dbd0 to fe9bf6b Compare August 12, 2026 15:13
@linear

linear Bot commented Aug 12, 2026

Copy link
Copy Markdown

CHAIN-4831

@leopoldjoy
leopoldjoy force-pushed the feat/nitro-validator-c4 branch from fe9bf6b to ca44561 Compare August 12, 2026 16:32
Base automatically changed from feat/nitro-validator-c3 to main August 12, 2026 18:27
@cb-heimdall

cb-heimdall commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
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 0
Sum 1

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@leopoldjoy
leopoldjoy force-pushed the feat/nitro-validator-c4 branch from ca44561 to 56b1573 Compare August 12, 2026 18:39
@leopoldjoy
leopoldjoy merged commit 06c49c6 into main Aug 12, 2026
6 checks passed
@leopoldjoy
leopoldjoy deleted the feat/nitro-validator-c4 branch August 12, 2026 19:32
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.

3 participants