Skip to content

feat(L1): deploy hinted nitro validator stack - #395

Merged
leopoldjoy merged 3 commits into
mainfrom
feat/nitro-validator-c2
Aug 11, 2026
Merged

feat(L1): deploy hinted nitro validator stack#395
leopoldjoy merged 3 commits into
mainfrom
feat/nitro-validator-c2

Conversation

@leopoldjoy

@leopoldjoy leopoldjoy commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a standalone Solidity 0.8.26 deployment script for P384Verifier -> CertManager -> NitroValidator
  • use named direct lib/... imports and scoped 10,000-run compiler restrictions matching upstream
  • require explicit nonzero CertManager owner and revoker arguments
  • test deployment wiring, root-cache initialization, role assignment, and EIP-170 runtime sizes

Behavior

The script is opt-in and does not change the standard system deployment. TEEProverRegistry continues to use the configured legacy NitroEnclaveVerifier until C3.

Testing

  • clean just test --match-path test/deploy/DeployNitroValidatorStack.t.sol (3 passed)
  • targeted script size build: CertManager 23,784 bytes with 792 bytes of EIP-170 margin
  • CI-equivalent lint, build, snapshots, and full test workflow (1,268 passed, 1 skipped)
  • just snapshots with no snapshot diff

Stack

Base automatically changed from feat/nitro-validator-c1 to main August 10, 2026 15:29
@cb-heimdall

cb-heimdall commented Aug 10, 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-c2 branch from deae8dd to 5456ed5 Compare August 11, 2026 10:38
Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
Comment thread justfile Outdated
Comment thread test/deploy/DeployNitroValidatorStack.t.sol Outdated
Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@leopoldjoy
leopoldjoy requested a review from jackchuma August 11, 2026 22:53
@leopoldjoy
leopoldjoy merged commit 40a4042 into main Aug 11, 2026
6 checks passed
@leopoldjoy
leopoldjoy deleted the feat/nitro-validator-c2 branch August 11, 2026 23:05
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.

4 participants