Skip to content

chore: Remove L2CM and CANNON_KONA Dev Feature Flags#21591

Draft
0xCoati wants to merge 9 commits into
developfrom
chore/remove-l2cm-flag
Draft

chore: Remove L2CM and CANNON_KONA Dev Feature Flags#21591
0xCoati wants to merge 9 commits into
developfrom
chore/remove-l2cm-flag

Conversation

@0xCoati

@0xCoati 0xCoati commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Removes the L2CM and CannonKona dev feature flags, making both the default codepath.

L2CM:

  • Drops the flag constant and its hardcoded short-circuit from both the Solidity (DevFeatures.sol) and Go (devfeatures.go) definitions.
  • Makes the ConditionalDeployer and L2DevFeatureFlags predeploys unconditional in the Predeploys registry.
  • Cleans up tests and removes redundant test-cases.

CannonKona:

  • Drops the flag constant and short-circuit the same way (bit 0x10 retired).
  • Collapses the only production branch — the startingRespectedGameType override gate in OPContractsManagerV2._isPermittedInstruction — to the always-on path (behavior-neutral, the shim already forced it to true).
  • Removes the DEV_FEATURE__CANNON_KONA env plumbing (Config.sol, FeatureFlags.sol), the *AlwaysEnabled* override tests, fuzz-test exclusions, and the commented-out CI matrix entry.
  • Bumps OPContractsManagerV2 7.1.237.1.24; regenerates semver-lock.json and the NUT bundle snapshot.

Note: on-chain L2DevFeatureFlags.isDevFeatureEnabled(0x10) now reports the actual bitmap instead of always-true — same precedent as the retired L2CM bit; nothing on-chain consumes it.

Notes

Addresses both flag removals from #20084 . The removal of the entire dev features system should be tracked in a separate issue.

@0xCoati 0xCoati changed the title remove l2cm dev feature flag and clean up chore: remove l2cm dev feature flag Jun 26, 2026
@0xarktos 0xarktos changed the title chore: remove l2cm dev feature flag chore: Remove L2CM and CANNON_KONA Dev Feature Flags Jul 9, 2026
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.

3 participants