Skip to content

CHAIN-3892: Upgrade TEEProverRegistry to point to the new NitroEnclaveVerifier (Zeronet)#663

Merged
xenoliss merged 6 commits intomainfrom
bo/chain-3892
Apr 17, 2026
Merged

CHAIN-3892: Upgrade TEEProverRegistry to point to the new NitroEnclaveVerifier (Zeronet)#663
xenoliss merged 6 commits intomainfrom
bo/chain-3892

Conversation

@xenoliss
Copy link
Copy Markdown
Collaborator

Upgrade TEEProverRegistry Nitro Pointer (Zeronet)

This task updates TEEProverRegistry to use the new NitroEnclaveVerifier (latest Nitro changes).
Contracts are already deployed.

Scripts

  • script/DeployAndSetupNitro.s.sol: deploy + setup new NitroEnclaveVerifier
  • script/DeployTEEProverRegistryImpl.s.sol: deploy new TEEProverRegistry implementation pointing to the new Nitro
  • script/UpgradeTEEProverRegistry.s.sol: multisig ProxyAdmin.upgrade of TEE_PROVER_REGISTRY_PROXY

@linear
Copy link
Copy Markdown

linear bot commented Apr 10, 2026

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 10, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/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 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

Comment thread zeronet/2026-04-07-upgrade-tee-registry-nitro/.env Outdated
TEE_PROVER_REGISTRY_PROXY=0xf1d1441B7d98f191f5CFBE4660D3684FEa004AFc
TEE_PROVER_REGISTRY_OWNER=0x856611ed7e07d83243b15e93f6321f2df6865852

# NitroEnclaveVerifier deployment/config
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It might be good to just take these from the existing contract next time. Similar to how we usually do absolute prestate updates

@@ -0,0 +1,21 @@
BASE_CONTRACTS_COMMIT=ffe2af8cbffefa44bbb1a3917507f8bd8ebec7a2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Recently updated the contracts due to audit fixes

@cb-heimdall cb-heimdall dismissed roger-bai-coinbase’s stale review April 16, 2026 19:07

Approved review 4116596172 from roger-bai-coinbase is now dismissed due to new commit. Re-request for approval.

@xenoliss xenoliss merged commit 85d9dac into main Apr 17, 2026
14 checks passed
@xenoliss xenoliss deleted the bo/chain-3892 branch April 17, 2026 13:13
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