Conversation
Fold the August 2026 hinted Nitro registry upgrade into the existing Cobalt task as a sixth, env-gated ProxyAdmin.upgrade. Zeronet stays on the original five-call transaction. Sepolia Cobalt rotates TEE_IMAGE_HASH and both ZK hashes. The registry state digest therefore covers only stored values; the post-check asserts the registry reads its expected hash from the new AggregateVerifier. Networks that opt in via OLD_TEE_PROVER_REGISTRY_IMPL cannot generate validations until the new implementation is deployed. Co-authored-by: Cursor <cursoragent@cursor.com>
Collaborator
🟡 Heimdall Review Status
|
mw2000
marked this pull request as ready for review
September 17, 2026 18:56
jackchuma
reviewed
Sep 17, 2026
jackchuma
reviewed
Sep 17, 2026
CatfishbillyIII
approved these changes
Sep 17, 2026
Collaborator
|
Review Error for CatfishbillyIII @ 2026-09-17 19:01:04 UTC |
jackchuma
reviewed
Sep 17, 2026
jackchuma
reviewed
Sep 17, 2026
jackchuma
reviewed
Sep 17, 2026
jackchuma
reviewed
Sep 17, 2026
jackchuma
reviewed
Sep 17, 2026
0xth4nh
reviewed
Sep 17, 2026
0xth4nh
reviewed
Sep 17, 2026
Initialize ProtocolVersions at v1.4.0 and use the same Cobalt TEE image and ZK program hashes as Zeronet. Rely on the deployment scripts and Foundry's automatic remappings instead of duplicating validation targets and dependency configuration. Co-authored-by: Cursor <cursoragent@cursor.com>
0xth4nh
previously approved these changes
Sep 17, 2026
Record the Sepolia deployment addresses and chain-11155111 broadcast artifacts for the Cobalt implementations and hinted Nitro stack. Add the Coinbase and Security Council validation files for the six-call upgrade transaction, including the v1.4.0 ProtocolVersions schedule and TEE registry cutover with no balance changes. Co-authored-by: Cursor <cursoragent@cursor.com>
cb-heimdall
dismissed
0xth4nh’s stale review
September 17, 2026 23:10
Approved review 5241047870 from 0xth4nh is now dismissed due to new commit. Re-request for approval.
Co-authored-by: Cursor <cursoragent@cursor.com>
CatfishbillyIII
approved these changes
Sep 18, 2026
Collaborator
|
Review Error for CatfishbillyIII @ 2026-09-18 10:07:55 UTC |
roger-bai-coinbase
approved these changes
Sep 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Folds the August 2026 hinted Nitro registry upgrade into the existing Cobalt task as a sixth, env-gated
ProxyAdmin.upgrade. Zeronet stays on the original five-call transaction.Sepolia initializes
ProtocolVersionsat v1.4.0 and uses the same Cobalt TEE image and ZK program hashes as Zeronet. BecauseTEE_IMAGE_HASHrotates, the registry state digest covers only stored values and the post-check asserts that the registry reads its expected hash from the newAggregateVerifier.