Skip to content

fix: enable cgt overrides on OPCMv2#724

Merged
0xOneTony merged 2 commits intosc-feat/opcm2-add-cgtfrom
fix/opcm2-enable-cgt-override
Dec 3, 2025
Merged

fix: enable cgt overrides on OPCMv2#724
0xOneTony merged 2 commits intosc-feat/opcm2-add-cgtfrom
fix/opcm2-enable-cgt-override

Conversation

@0xiamflux
Copy link
Copy Markdown

No description provided.

@0xiamflux 0xiamflux requested a review from 0xOneTony December 3, 2025 19:39
@0xiamflux 0xiamflux self-assigned this Dec 3, 2025
@0xiamflux 0xiamflux changed the title fix: Enable CGT overrides on OPCMv2 fix: enable cgt overrides on OPCMv2 Dec 3, 2025
@0xOneTony 0xOneTony merged commit 2614984 into sc-feat/opcm2-add-cgt Dec 3, 2025
1 check passed
0xniha pushed a commit that referenced this pull request Dec 8, 2025
* fix: check CGT is not enabled before enabling it

* fix: enable CGT overrides
0xniha pushed a commit that referenced this pull request Dec 9, 2025
* fix: check CGT is not enabled before enabling it

* fix: enable CGT overrides
0xOneTony pushed a commit that referenced this pull request Dec 12, 2025
* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* refactor: change disputeGameConfigs order in _loadFullConfig

* fix: comment slash rules

* feat: add opcmv2-cgt matrix in CI

* fix: enable cgt overrides on OPCMv2 (#724)

* fix: check CGT is not enabled before enabling it

* fix: enable CGT overrides

* fix: add extra instruction overrides for useCustomGasToken on OPCMv2 (#725)

* chore: link TODO to issue (#727)

* fix: opcm2 revert upgrade cgt (#728)

* fix: add check enabled CGT during upgrades

* test: set CGT false by default during forked tests

* chore: expand comment on OPCMv2 regarding CGT

* chore: pre-pr ready

* refactor: remove unnecessary check for CGT feature (#735)

* fix: cgt upgrade in opcm v2 and semver (#738)

* fix: Full checkout for FFI build (ethereum-optimism#18527)

* chore(op-acceptance-tests): delete very old logs (ethereum-optimism#18529)

* chore(op-acceptance-tests): op-acceptor v3.8.0 (ethereum-optimism#18530)

* jovian: remove feature toggles (ethereum-optimism#17978)

* jovian: remove feature toggles

scope is now fixed

* Updated op-geth to v1.101604.0-synctest.0.0.20251120150812-e50f80a16afc

* lint

* just update-op-geth f48f382

* Use Jovian in test error message

* just update-op-geth ba6bdcfef42341fe2b5ce124c31ff2d6b264e9e4

* chore(ai-eng): add ReinitializableBase test to exclusion list (ethereum-optimism#18531)

* feat: have OPCM upgrade allowances be upgrade specific (ethereum-optimism#18462)

* feat: have OPCM upgrade allowances be upgrade specific

Updates the OPCMv2 check for allowed extra instructions to be
specific to releases. When release versions are bumped, the
allowances become automatically invalid and would reveal
anywhere in the codebase where the allowance is being used.

* fix: semver lock

* fix: emit instruction key in error

* fix: better semantics

* docs: proper OPCM versioning policy

* fix: final version tweaks

* fix(op-acceptance-test): flake-shake; empty slack notifications. (ethereum-optimism#18542)

* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* refactor: change disputeGameConfigs order in _loadFullConfig

* fix: enable cgt overrides on OPCMv2 (#724)

* fix: check CGT is not enabled before enabling it

* fix: enable CGT overrides

* fix: opcm2 revert upgrade cgt (#728)

* fix: add check enabled CGT during upgrades

* test: set CGT false by default during forked tests

* chore: expand comment on OPCMv2 regarding CGT

* fix: cgt upgrade in opcm v2 and semver (#738)

* fix: merge conflicts

* fix: pre-pr

* fix: add missing isMatchingInstruction for cgt in false

* refactor: opcm2 extra instruction keymatch (#747)

* refactor: add helper function to match instruction by key only

* chore: pre-pr ready

* test: refactor key matching test

* test: add test suite for IsMatchingKey helper function

* chore: pre-pr ready

* fix: opcmv2 semver

---------

Co-authored-by: Flux <175354924+0xiamflux@users.noreply.github.com>
Co-authored-by: Ján Jakub Naništa <jan.jakub.nanista@gmail.com>
Co-authored-by: Stefano Charissis <stefano@oplabs.co>
Co-authored-by: George Knee <georgeknee@googlemail.com>
Co-authored-by: Ariel Diaz <65925295+aliersh@users.noreply.github.com>
Co-authored-by: smartcontracts <14298799+smartcontracts@users.noreply.github.com>
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.

2 participants