Skip to content

cleanup(contracts): rename super permissioned game type#21513

Open
Inphi wants to merge 2 commits into
developfrom
inphi/game-rename
Open

cleanup(contracts): rename super permissioned game type#21513
Inphi wants to merge 2 commits into
developfrom
inphi/game-rename

Conversation

@Inphi

@Inphi Inphi commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rename SUPER_PERMISSIONED_CANNON to SUPER_PERMISSIONED while preserving game type value 5
  • Update OPCM deploy, upgrade, validation, devstack, op-deployer, tests, and docs references
  • Move the legacy Cannon prestate fixture out of the shared super-game test helper
  • Bump affected contract patch versions and update semver-lock.json

fixes #20933

@mintlify

mintlify Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Optimism-Docs 🟢 Ready View Preview Jun 22, 2026, 9:06 PM

@Inphi Inphi force-pushed the inphi/game-rename branch from d37bebf to 58336c8 Compare June 22, 2026 21:24
@Inphi Inphi marked this pull request as ready for review June 22, 2026 22:09
@Inphi Inphi requested review from a team and maurelian June 22, 2026 22:09
@Inphi Inphi requested review from a team as code owners June 22, 2026 22:09
@Inphi

Inphi commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

@OptimismBot plz review

@wwared wwared left a comment

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.

LGTM

Remaining references this PR should also update:

  • op-devstack/sysgo/superroot.go still has a superPermissionedCannonGameType. Either we rename that or drop the duplicated const and replace it with gameTypes.SuperPermissionedGameType
  • packages/contracts-bedrock/test/L1/OPContractsManagerStandardValidator.t.sol has test_validate_superPermissionedCannonNotRegistered_succeeds, should rename to test_validate_superPermissionedNotRegistered_succeeds

@digorithm digorithm left a comment

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.

:shipit:

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.

cleanup(contracts): Rename SUPER_PERMISSIONED_CANNON game

4 participants