clearer error

Author: smartcontracts

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

@@ -31,6 +31,7 @@ interface IOptimismPortal2 {
     error UnexpectedList();
     error UnexpectedString();
     error Unproven();
+    error LegacyGame();
 
     event DisputeGameBlacklisted(IDisputeGame indexed disputeGame);
     event Initialized(uint8 version);

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

@@ -33,6 +33,7 @@ interface IOptimismPortalInterop {
     error UnexpectedList();
     error UnexpectedString();
     error Unproven();
+    error LegacyGame();
 
     event DisputeGameBlacklisted(IDisputeGame indexed disputeGame);
     event Initialized(uint8 version);

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

@@ -20,11 +20,11 @@
     "sourceCodeHash": "0x2d21506cc51ebe0b60bcf89883aff5e9b1269567ce44ee779de3d3940e23fb65"
   },
   "src/L1/OptimismPortal2.sol": {
-    "initCodeHash": "0xb906cca6c1d2ef7d783f94d502bbf8704f39488bcbe4b1633f21d5a25f39d750",
-    "sourceCodeHash": "0x2fa2648a82059eb6fc052254678d3bd2bf7952be83ab53c545c9302df3b9d1a0"
+    "initCodeHash": "0x3f30480b47af0a965af7d38eff673fe1ef0e8704c1776eaef5648d242cfe376d",
+    "sourceCodeHash": "0x6f66b68d0c5c480941e2d47b456f748356e2670ee3157a08380063afad57dee9"
   },
   "src/L1/OptimismPortalInterop.sol": {
-    "initCodeHash": "0x92e46899517ecb927a6fea35995ebf6cc534443351d1f5806a7d34c5696c0648",
+    "initCodeHash": "0x9e12728cae72a1c701ddc3e09fe9e3b82aa38674ab76e6d9e74f3a384dfe25d2",
     "sourceCodeHash": "0xc04a7f9c14a13ec3587f5cc351c8e9f27fbbe9f1291a1aba07de29edbeef418a"
   },
   "src/L1/ProtocolVersions.sol": {

packages/contracts-bedrock/src/L1/OptimismPortal2.sol

@@ -28,7 +28,8 @@ import {
     Blacklisted,
     Unproven,
     ProposalNotValidated,
-    AlreadyFinalized
+    AlreadyFinalized,
+    LegacyGame
 } from "src/libraries/PortalErrors.sol";
 import { GameStatus, GameType, Claim, Timestamp } from "src/dispute/lib/Types.sol";
 
@@ -309,7 +310,11 @@ contract OptimismPortal2 is Initializable, ResourceMetering, ISemver {
         if (gameType.raw() != respectedGameType.raw()) revert InvalidGameType();
 
         // The game type of the DisputeGame must have been the respected game type at creation.
-        if (!gameProxy.wasRespectedGameTypeWhenCreated()) revert InvalidGameType();
+        try gameProxy.wasRespectedGameTypeWhenCreated() returns (bool wasRespected) {
+            if (!wasRespected) revert InvalidGameType();
+        } catch {
+            revert LegacyGame();
+        }
 
         // Creation time must be greater than or equal to the respected game type updated time.
         // Games created before this timestamp are not valid and cannot be used to finalize a
@@ -541,7 +546,12 @@ contract OptimismPortal2 is Initializable, ResourceMetering, ISemver {
         // time. We check that the game type is the respected game type at proving time, but it's
         // possible that the respected game type has since changed. Users can still use this game
         // to finalize a withdrawal as long as it has not been otherwise invalidated.
-        if (!disputeGameProxy.wasRespectedGameTypeWhenCreated()) revert InvalidGameType();
+        // The game type of the DisputeGame must have been the respected game type at creation.
+        try disputeGameProxy.wasRespectedGameTypeWhenCreated() returns (bool wasRespected) {
+            if (!wasRespected) revert InvalidGameType();
+        } catch {
+            revert LegacyGame();
+        }
 
         // The game must have been created after `respectedGameTypeUpdatedAt`. This is to prevent users from creating
         // invalid disputes against a deployed game type while the off-chain challenge agents are not watching.

packages/contracts-bedrock/src/libraries/PortalErrors.sol

@@ -38,3 +38,5 @@ error Unproven();
 error ProposalNotValidated();
 /// @notice Error for when a withdrawal has already been finalized.
 error AlreadyFinalized();
+/// @notice Error for when a game is a legacy game.
+error LegacyGame();

packages/contracts-bedrock/test/dispute/AnchorStateRegistry.t.sol

@@ -235,7 +235,7 @@ contract AnchorStateRegistry_IsGameRetired_Test is AnchorStateRegistry_Init {
 
 contract AnchorStateRegistry_IsGameProper_Test is AnchorStateRegistry_Init {
     /// @notice Tests that isGameProper will return true if the game meets all conditions.
-    function test_isGameProper_meetsAllConditions_succeeds() public {
+    function test_isGameProper_meetsAllConditions_succeeds() public view {
         // Game will meet all conditions by default.
         assertTrue(anchorStateRegistry.isGameProper(gameProxy));
     }