Skip to content

Commit 48ff55d

Browse files
authored
opcm: Update upgrade method for disputegame-v2 (#17874)
* opcm: Update upgrade method for disputegame-v2 * update opcm semver * check game args; run post-upgrade checks * update std validator semver lock * fix assertGameArgsLength error prefix * opcm and validator fixes * lint LibGameArgs * fix interfaces; natspec * fix natspec in opcm * forwards-compatible challenger/proposer retrieval * assert deployed CANNON game parameters * use v2 getters * review comments * fix asr getter name * use v1 functions for existing upgrade path
1 parent a492f73 commit 48ff55d

File tree

11 files changed

+740
-85
lines changed

11 files changed

+740
-85
lines changed

packages/contracts-bedrock/interfaces/L1/IOPContractsManagerStandardValidator.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ interface IOPContractsManagerStandardValidator {
3434
address challenger;
3535
}
3636

37+
error InvalidGameArgsLength();
38+
3739
function version() external view returns (string memory);
3840
function anchorStateRegistryImpl() external view returns (address);
3941
function challenger() external view returns (address);

packages/contracts-bedrock/snapshots/abi/OPContractsManagerStandardValidator.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@
289289
"type": "string"
290290
}
291291
],
292-
"stateMutability": "pure",
292+
"stateMutability": "view",
293293
"type": "function"
294294
},
295295
{
@@ -465,5 +465,10 @@
465465
],
466466
"stateMutability": "view",
467467
"type": "function"
468+
},
469+
{
470+
"inputs": [],
471+
"name": "InvalidGameArgsLength",
472+
"type": "error"
468473
}
469474
]

packages/contracts-bedrock/snapshots/abi/OPContractsManagerUpgrader.json

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -366,6 +366,22 @@
366366
"name": "IdentityPrecompileCallFailed",
367367
"type": "error"
368368
},
369+
{
370+
"inputs": [
371+
{
372+
"internalType": "bytes32",
373+
"name": "devFeature",
374+
"type": "bytes32"
375+
}
376+
],
377+
"name": "InvalidDevFeatureAccess",
378+
"type": "error"
379+
},
380+
{
381+
"inputs": [],
382+
"name": "InvalidGameArgsLength",
383+
"type": "error"
384+
},
369385
{
370386
"inputs": [],
371387
"name": "NotABlueprint",

packages/contracts-bedrock/snapshots/semver-lock.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,12 @@
2020
"sourceCodeHash": "0xfca613b5d055ffc4c3cbccb0773ddb9030abedc1aa6508c9e2e7727cc0cd617b"
2121
},
2222
"src/L1/OPContractsManager.sol:OPContractsManager": {
23-
"initCodeHash": "0x4025118658a1c56c4fc2d0166081e6d27da980318e749e7811c7178115e4413e",
24-
"sourceCodeHash": "0xdb243ac7475b0214b8f662aa04f696d9ff213dade6b1bdade379e84df419b75a"
23+
"initCodeHash": "0xc6a5146f158bc14297447cda1668f83e8dbfdd74f1b8bc2291d9626933bd23ad",
24+
"sourceCodeHash": "0xd883ebfcc637ba9c6de12a872015098614cd288aaee84cda9d4c746aa1dcba18"
2525
},
2626
"src/L1/OPContractsManagerStandardValidator.sol:OPContractsManagerStandardValidator": {
27-
"initCodeHash": "0x57d6a6729d887ead009d518e8f17fa0d26bfc97b8efe1494ab4ef8dbb000d109",
28-
"sourceCodeHash": "0x1d58891954cf782d2fe4f112b0c7fd25be991c2b8873f10d8545c653b517cac9"
27+
"initCodeHash": "0x2eaa345ba05582c67b40a1eb7ec9d54823aa08468e697e2d6c04bb74cc574abc",
28+
"sourceCodeHash": "0x30d7e4243a3bab7fea8504be2cd24f2c5ab3335b802d62282a374a458a08feec"
2929
},
3030
"src/L1/OptimismPortal2.sol:OptimismPortal2": {
3131
"initCodeHash": "0x5bf576ea7f566e402a997204988471fc9b971410aa9dff8fe810b10baf6b7456",

0 commit comments

Comments
 (0)